Skip to content

[Chore] Add test coverage for Roo-to-Zoo user-facing string changes#1003

Closed
zoomote[bot] wants to merge 6 commits into
mainfrom
test/bump-coverage-pr-971-07omcv8qp9t8a
Closed

[Chore] Add test coverage for Roo-to-Zoo user-facing string changes#1003
zoomote[bot] wants to merge 6 commits into
mainfrom
test/bump-coverage-pr-971-07omcv8qp9t8a

Conversation

@zoomote

@zoomote zoomote Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Opened on behalf of Naved Merchant. View the task or mention @zoomote for follow-up asks.

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:

  • LM Studio provider: the existing createMessage and completePrompt error tests now assert the full branded message ("...larger context length to work with Zoo Code's prompts."), guarding both error-wrapping paths in lm-studio.ts.
  • VS Code LM provider: new tests assert the LM authorization justification is branded ("Zoo Code would like to use ...") and that client initialization failures throw the Zoo Code branded error, including the nested "Failed to select model" wrap.
  • i18n locales: a new spec verifies every shipped locale's tools.json defines missingToolParameter and missingToolParameterWithPath with 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 successful
  • pnpm check-types — all checks successful
  • Targeted: npx vitest run api/providers/__tests__/lmstudio.spec.ts api/providers/__tests__/vscode-lm.spec.ts i18n/__tests__/tools-missing-param.spec.ts — 39/39 passed

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

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-strings from the contributor fork) since maintainer pushes to the fork were not available. Once #971 merges, this PR's diff against main reduces to the single test-only commit. Alternatively, a maintainer can cherry-pick the last commit (a01f9f57e) onto #971's branch.

rrewll and others added 6 commits July 22, 2026 20:22
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.
@zoomote

zoomote Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

No code issues found. See task

Reviewed a01f9f5

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 32.43243% with 25 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/api/providers/vscode-lm.ts 30.30% 23 Missing ⚠️
src/api/transform/vscode-lm-format.ts 0.00% 2 Missing ⚠️

📢 Thoughts on this report? Let us know!

@zoomote

zoomote Bot commented Jul 24, 2026

Copy link
Copy Markdown
Contributor Author

Closing: coverage commit pushed directly to #971's branch instead (a7ae66a).

@zoomote zoomote Bot closed this Jul 24, 2026
@zoomote
zoomote Bot deleted the test/bump-coverage-pr-971-07omcv8qp9t8a branch July 24, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants