fix(branding): replace remaining user-facing "Roo" strings with "Zoo"#971
Open
rrewll wants to merge 5 commits into
Open
fix(branding): replace remaining user-facing "Roo" strings with "Zoo"#971rrewll wants to merge 5 commits into
rrewll wants to merge 5 commits into
Conversation
Contributor
|
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 (2)
📝 WalkthroughWalkthroughThe changes replace remaining Roo branding with Zoo across runtime messages and webview locales, add localized missing-tool-parameter retry messages in supported languages, and add tests for tab-panel creation and both task error-message branches. ChangesBranding and localization updates
Estimated code review effort: 3 (Moderate) | ~20 minutes Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
rrewll
force-pushed
the
fix/rename-roo-to-zoo-user-strings
branch
2 times, most recently
from
July 21, 2026 13:26
59622cd to
fb30376
Compare
rrewll
marked this pull request as ready for review
July 21, 2026 13:47
rrewll
requested review from
JamesRobert20,
edelauna,
hannesrudolph,
navedmerchant and
taltas
as code owners
July 21, 2026 13:47
Replace user-facing product references from "Roo" to "Zoo": - Webview panel and editor tab titles - Integrated terminal names - Diff view labels - VS Code LM authorization justification - LM Studio context-length error messages - Missing-parameter tool error (now i18n-keyed via tools:missingToolParameter[WithPath] across all 16 locales) - Webview-ui localization strings (chat, settings, prompts across all locales) Per review feedback, references to the external Roo provider/router remain untouched: the routerRemoval messages, errors.roo/info.roo i18n keys, Roo credit-balance notices, and other internal identifiers that reference the legacy Roo provider rather than our brand.
rrewll
force-pushed
the
fix/rename-roo-to-zoo-user-strings
branch
from
July 21, 2026 13:51
fb30376 to
1d47890
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 2
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
.changeset/branding-roo-to-zoo-user-strings.md (1)
1-8: 📐 Maintainability & Code Quality | 🔴 Critical | ⚡ Quick winRemove the changeset file.
As per path instructions, do NOT create
.changesetfiles for each commit or code change. Changesets are managed separately by maintainers and should not be generated by agents during normal development.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.changeset/branding-roo-to-zoo-user-strings.md around lines 1 - 8, Remove the changeset file described by the diff; do not add or modify any other changeset files.Source: Path instructions
🧹 Nitpick comments (1)
src/core/task/__tests__/Task.spec.ts (1)
442-450: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winAssert that
relPathis present in the output.The test validates that
toolNameandparamNamemake it through the i18n interpolation, but it misses asserting that therelPath(src/foo.ts) is also properly injected in thewithPathNotice. Asserting this will prevent future regressions in the interpolation logic.♻️ Proposed refactor
const [withoutPathChannel, withoutPathNotice] = saySpy.mock.calls[1] expect(withPathChannel).toBe("error") expect(withoutPathChannel).toBe("error") expect(withPathNotice).toEqual(expect.stringContaining("read_file")) expect(withPathNotice).toEqual(expect.stringContaining("path")) + expect(withPathNotice).toEqual(expect.stringContaining("src/foo.ts")) expect(withoutPathNotice).toEqual(expect.stringContaining("execute_command")) expect(withoutPathNotice).toEqual(expect.stringContaining("command"))🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@src/core/task/__tests__/Task.spec.ts` around lines 442 - 450, Update the test assertions for withPathNotice in the saySpy call checks to also require the expected relPath value, “src/foo.ts”, while preserving the existing toolName and paramName assertions.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/api/providers/vscode-lm.ts`:
- Line 398: Complete the branding migration in the VS Code LM provider by
replacing remaining user-facing “Roo Code <Language Model API>” references in
the handler’s errors and diagnostics with the consistent “Zoo Code” branding.
Update the messages near the authorization justification and the referenced
error/diagnostic paths while preserving their existing behavior and details.
In `@webview-ui/src/i18n/locales/ko/settings.json`:
- Line 801: Update the Korean settings description string at the terminal
description entry to remove the corrupted replacement characters and restore the
intended Korean text, preserving the existing meaning, formatting, and embedded
<0>자세히 알아보기</0> link.
---
Outside diff comments:
In @.changeset/branding-roo-to-zoo-user-strings.md:
- Around line 1-8: Remove the changeset file described by the diff; do not add
or modify any other changeset files.
---
Nitpick comments:
In `@src/core/task/__tests__/Task.spec.ts`:
- Around line 442-450: Update the test assertions for withPathNotice in the
saySpy call checks to also require the expected relPath value, “src/foo.ts”,
while preserving the existing toolName and paramName assertions.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro Plus
Run ID: b657f42e-2d6a-4ce7-b26a-0a6a4981f609
📒 Files selected for processing (63)
.changeset/branding-roo-to-zoo-user-strings.mdsrc/activate/__tests__/registerCommands.spec.tssrc/activate/registerCommands.tssrc/api/providers/lm-studio.tssrc/api/providers/vscode-lm.tssrc/core/task/Task.tssrc/core/task/__tests__/Task.spec.tssrc/core/webview/ClineProvider.tssrc/i18n/locales/ca/tools.jsonsrc/i18n/locales/de/tools.jsonsrc/i18n/locales/en/tools.jsonsrc/i18n/locales/es/tools.jsonsrc/i18n/locales/fr/tools.jsonsrc/i18n/locales/hi/tools.jsonsrc/i18n/locales/id/tools.jsonsrc/i18n/locales/it/tools.jsonsrc/i18n/locales/ja/tools.jsonsrc/i18n/locales/ko/tools.jsonsrc/i18n/locales/nl/tools.jsonsrc/i18n/locales/pl/tools.jsonsrc/i18n/locales/pt-BR/tools.jsonsrc/i18n/locales/ru/tools.jsonsrc/i18n/locales/tr/tools.jsonsrc/i18n/locales/vi/tools.jsonsrc/i18n/locales/zh-CN/tools.jsonsrc/i18n/locales/zh-TW/tools.jsonwebview-ui/src/i18n/locales/ca/chat.jsonwebview-ui/src/i18n/locales/ca/settings.jsonwebview-ui/src/i18n/locales/de/chat.jsonwebview-ui/src/i18n/locales/de/prompts.jsonwebview-ui/src/i18n/locales/de/settings.jsonwebview-ui/src/i18n/locales/es/chat.jsonwebview-ui/src/i18n/locales/es/settings.jsonwebview-ui/src/i18n/locales/fr/chat.jsonwebview-ui/src/i18n/locales/fr/settings.jsonwebview-ui/src/i18n/locales/hi/chat.jsonwebview-ui/src/i18n/locales/hi/settings.jsonwebview-ui/src/i18n/locales/id/chat.jsonwebview-ui/src/i18n/locales/id/settings.jsonwebview-ui/src/i18n/locales/it/chat.jsonwebview-ui/src/i18n/locales/it/settings.jsonwebview-ui/src/i18n/locales/ja/chat.jsonwebview-ui/src/i18n/locales/ja/prompts.jsonwebview-ui/src/i18n/locales/ja/settings.jsonwebview-ui/src/i18n/locales/ko/chat.jsonwebview-ui/src/i18n/locales/ko/prompts.jsonwebview-ui/src/i18n/locales/ko/settings.jsonwebview-ui/src/i18n/locales/nl/chat.jsonwebview-ui/src/i18n/locales/nl/settings.jsonwebview-ui/src/i18n/locales/pl/chat.jsonwebview-ui/src/i18n/locales/pl/settings.jsonwebview-ui/src/i18n/locales/pt-BR/chat.jsonwebview-ui/src/i18n/locales/pt-BR/settings.jsonwebview-ui/src/i18n/locales/ru/chat.jsonwebview-ui/src/i18n/locales/ru/settings.jsonwebview-ui/src/i18n/locales/tr/chat.jsonwebview-ui/src/i18n/locales/tr/settings.jsonwebview-ui/src/i18n/locales/vi/chat.jsonwebview-ui/src/i18n/locales/vi/settings.jsonwebview-ui/src/i18n/locales/zh-CN/chat.jsonwebview-ui/src/i18n/locales/zh-CN/prompts.jsonwebview-ui/src/i18n/locales/zh-CN/settings.jsonwebview-ui/src/i18n/locales/zh-TW/chat.json
Contributor
|
Thanks for this, can you please fix the CodeRabbit comment? |
Contributor
Author
Ok, I have fixed it |
github-actions
Bot
added
the
awaiting-review
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related GitHub Issue
Closes: #551
Description
Replace user-facing product references from "Roo" to "Zoo":
Add test coverage for sayAndCreateMissingParamError to ensure the localized error messages correctly interpolate tool names and parameter names, preventing silent i18n regressions. Also add test coverage for openClineInNewTab to ensure the new editor tab title names as "Zoo Code" instead of "Roo Code".
Per review feedback, references to the external Roo provider/router remain untouched: the routerRemoval messages, errors.roo/info.roo i18n keys, Roo credit-balance notices, and other internal identifiers that reference the legacy Roo provider rather than our brand.
Test Procedure
pnpm test- 414 test files and 6794 tests passedpnpm lint- all checks are successfulpnpm check-types- all checks are successfulnode scripts/find-missing-translations.js- all translations are complete across all checked areasPre-Submission Checklist
Documentation Updates
No documentation updates are required
Additional Notes
This PR is the continuation of #343. Unfortunately, I couldn't get hold of the author of this PR, so this is the "fork" of the PR above.
Summary by CodeRabbit
New Features
Bug Fixes
Tests