fix(branding): replace remaining user-facing "Roo" strings with "Zoo"#343
fix(branding): replace remaining user-facing "Roo" strings with "Zoo"#343proyectoauraorg wants to merge 2 commits into
Conversation
Several user-visible strings still showed the upstream "Roo" brand instead of "Zoo": - the missing-tool-parameter retry notice (was hardcoded in Task.ts; now localized via tools:missingToolParameter / missingToolParameterWithPath across all 18 locales, matching the existing unknownToolError pattern) - the editor tab title and the webview <title> - the diff editor label and the integrated terminal name - the "no visible instances" output-channel line - the credit-balance and LM Studio context-length error notices - the router/cloud removal messages (common.json + routerRemoval.ts) Internal identifiers are intentionally left unchanged: the legacy provider id "roo", .roo* config file references, i18n key paths, OpenRouter X-Title attribution headers, MCP/checkpoint client names, and console logs.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughReplaces user-facing "Roo" branding with "Zoo" across UI strings, webview titles, diff/terminal labels, router removal messages, LM Studio error text, adds localized missing-parameter messages in 18 locales, and updates tests to match the new strings. ChangesBranding rename: Roo to Zoo
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
src/core/task/__tests__/Task.spec.tsESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox. 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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Exercises both relPath branches of the now-localized missing-tool-parameter error so the changed lines in Task.ts are covered (addresses the codecov patch-coverage gap on Zoo-Code-Org#343).
|
What do you think about it: Zoo-Code/src/api/providers/vscode-lm.ts Lines 396 to 406 in b91a0c1 |
What & why
Several user-visible strings still showed the upstream "Roo" brand instead of "Zoo". The most visible one was the missing-tool-parameter retry notice in chat:
That message was hardcoded in
Task.ts(unlike its siblingsunknownToolError/invalidJsonArgument, which are already localized to "Zoo"). The editor tab and several other surfaces had the same leftover.Changes
Task.ts— the missing-parameter notice is now localized via new keystools:missingToolParameter/tools:missingToolParameterWithPath, added and translated across all 18 locales (mirroring the existingunknownToolErrorpattern).createWebviewPanel), the webview<title>, the diff editor label (DIFF_VIEW_LABEL_CHANGES), and the integrated terminal name.common.json(18 locales) and the fallback constants inrouterRemoval.ts.Tests updated to assert the new strings.
Intentionally not changed
To avoid breaking internals or rewriting history, these "Roo" occurrences are left as-is:
rooand.roo*config file references;errors.roo.*);X-Titleattribution headers and MCP/checkpoint client names (sent to external services, not shown in the VS Code UI);Verification
node scripts/find-missing-translations.js→ ✅ all locales completetsc --noEmiton the changed files → cleaneslint→ cleanviteston the affected specs → 131 passingSummary by CodeRabbit
New Features
Documentation
Tests