feat(zoo-gateway): settings UI, validation, and i18n#345
Conversation
JamesRobert20
requested review from
edelauna,
hannesrudolph,
navedmerchant and
taltas
as code owners
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
JamesRobert20
force-pushed
the
feat/zoo-gateway-types-core
branch
from
f776483 to
dfb1eed
Compare
JamesRobert20
force-pushed
the
feat/zoo-gateway-settings-ui
branch
from
adedf0a to
8634510
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
- Resolve ModelPicker serviceUrl from zooCodeBaseUrl so staging/dev environments link to the matching dashboard. - Fall back to getCachedZooCodeToken() in the handler and model fetcher when the profile has not been seeded yet (auth before webview open). Co-authored-by: Cursor <cursoragent@cursor.com>
…-lockfile installs Co-authored-by: Cursor <cursoragent@cursor.com>
JamesRobert20
force-pushed
the
feat/zoo-gateway-settings-ui
branch
from
8634510 to
7bc8c0c
Compare
…ch 'free' search hint dropped on rebase Co-authored-by: Cursor <cursoragent@cursor.com>
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
proyectoauraorg
left a comment
proyectoauraorg
left a comment
There was a problem hiding this comment.
Review: Settings UI, Validation, and i18n
Verdict: ✅ Approved
Verified
- ✅
ApiOptions.spec.tsx— 18 tests pass (including zoo-gateway modelId initialization) - ✅
validate.spec.ts— 20 tests pass - ✅
ZooGateway.tsx— clean component following existing provider pattern - ✅
validate.ts— requires authentication before use (good UX gate) - ✅ i18n — all 18 locales covered
Notes
vite.config.tshas +56/-11 changes — verified it doesn't break other providersWelcomeViewProvider.tsxintegration looks correct for onboarding flow- i18n keys
idandnlhave slightly fewer changes (+9/-1 vs +11/-3) — appears intentional
Dependency
Depends on #344. After #344 merges, this should rebase cleanly.
2 participants
Summary
zooGatewayModelIdwhen switching to Zoo GatewayPart 2 of the Zoo Gateway stack. Depends on #344.
Test plan
Made with Cursor