[Chore] Add test coverage for Roo-to-Zoo user-facing string changes#1003
Closed
zoomote[bot] wants to merge 6 commits into
Closed
[Chore] Add test coverage for Roo-to-Zoo user-facing string changes#1003zoomote[bot] wants to merge 6 commits into
zoomote[bot] wants to merge 6 commits into
Conversation
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.
Contributor
Author
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Contributor
Author
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
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
Related to #551, builds on #971
Description
This PR stacks additional test coverage on top of #971 (the Roo -> Zoo user-facing string rebrand). It contains all of #971's commits plus one test-only commit. No production code changes.
New coverage for #971's changed code:
createMessageandcompletePrompterror tests now assert the full branded message ("...larger context length to work with Zoo Code's prompts."), guarding both error-wrapping paths inlm-studio.ts.tools.jsondefinesmissingToolParameterandmissingToolParameterWithPathwith all interpolation placeholders ({{toolName}},{{relPath}},{{paramName}}), and that the English copy says "Zoo", not "Roo". The locale list is read from the filesystem, so future locales are covered automatically.Test Procedure
pnpm test— all packages pass (10/10 turbo tasks)pnpm lint— all checks successfulpnpm check-types— all checks successfulnpx vitest run api/providers/__tests__/lmstudio.spec.ts api/providers/__tests__/vscode-lm.spec.ts i18n/__tests__/tools-missing-param.spec.ts— 39/39 passedPre-Submission Checklist
Screenshots / Videos
Test-only change; no UI impact.
Documentation Updates
No documentation updates are required.
Additional Notes
This branch is based on #971's head (
fix/rename-roo-to-zoo-user-stringsfrom the contributor fork) since maintainer pushes to the fork were not available. Once #971 merges, this PR's diff againstmainreduces to the single test-only commit. Alternatively, a maintainer can cherry-pick the last commit (a01f9f57e) onto #971's branch.