fix(branding): replace remaining user-facing "Roo" strings with "Zoo" (#343)

Rename user-facing product references from "Roo" to "Zoo": webview panel and
editor-tab titles, the integrated terminal name, the diff-view label, the LM
Studio context-length guidance, the VS Code LM authorization justification, and
the missing-parameter tool notice (now i18n-keyed via
tools:missingToolParameter[WithPath] across all locales).

Per review feedback, references to the external Roo provider/router are left
untouched: the routerRemoval messages, the errors.roo/info.roo i18n keys, and
the Roo credit-balance notice name the legacy Roo provider, not our brand.

Strengthen the sayAndCreateMissingParamError test so it asserts the resolved,
interpolated notice actually names the tool and the missing parameter (the
previous expect.any(String) would have passed even on a silent i18n regression).

Author: proyectoauraorg

.changeset/branding-roo-to-zoo-user-strings.md

@@ -0,0 +1,5 @@
+---
+"zoo-code": patch
+---
+
+Replace remaining user-facing "Roo" brand strings with "Zoo": the missing-tool-parameter retry notice (now localized across all locales), the editor tab and webview `<title>`, the diff editor label, the terminal name, the "no visible instances" output, and the LM Studio context-length notice. References to the external Roo provider/router are intentionally left unchanged (the `roo` provider id and `.roo*` config files, the `errors.roo`/`info.roo` localized messages, the router-removal and Roo Cloud sign-in/credit-balance notices, i18n key paths, attribution headers, and console logs).

src/activate/__tests__/registerCommands.spec.ts

@@ -114,7 +114,7 @@ describe("getVisibleProviderOrLog", () => {
 		const result = getVisibleProviderOrLog(mockOutputChannel)
 
 		expect(result).toBeUndefined()
-		expect(mockOutputChannel.appendLine).toHaveBeenCalledWith("Cannot find any visible Roo Code instances.")
+		expect(mockOutputChannel.appendLine).toHaveBeenCalledWith("Cannot find any visible Zoo Code instances.")
 	})
 })
 

src/activate/registerCommands.ts

@@ -21,7 +21,7 @@ import { t } from "../i18n"
 export function getVisibleProviderOrLog(outputChannel: vscode.OutputChannel): ClineProvider | undefined {
 	const visibleProvider = ClineProvider.getVisibleInstance()
 	if (!visibleProvider) {
-		outputChannel.appendLine("Cannot find any visible Roo Code instances.")
+		outputChannel.appendLine("Cannot find any visible Zoo Code instances.")
 		return undefined
 	}
 	return visibleProvider
@@ -232,7 +232,7 @@ export const openClineInNewTab = async ({ context, outputChannel }: Omit<Registe
 
 	const targetCol = hasVisibleEditors ? Math.max(lastCol + 1, 1) : vscode.ViewColumn.Two
 
-	const newPanel = vscode.window.createWebviewPanel(ClineProvider.tabPanelId, "Roo Code", targetCol, {
+	const newPanel = vscode.window.createWebviewPanel(ClineProvider.tabPanelId, "Zoo Code", targetCol, {
 		enableScripts: true,
 		retainContextWhenHidden: true,
 		localResourceRoots: [context.extensionUri],

src/api/providers/lm-studio.ts

@@ -165,7 +165,7 @@ export class LmStudioHandler extends BaseProvider implements SingleCompletionHan
 			} as const
 		} catch (error) {
 			throw new Error(
-				"Please check the LM Studio developer logs to debug what went wrong. You may need to load the model with a larger context length to work with Roo Code's prompts.",
+				"Please check the LM Studio developer logs to debug what went wrong. You may need to load the model with a larger context length to work with Zoo Code's prompts.",
 			)
 		}
 	}
@@ -209,7 +209,7 @@ export class LmStudioHandler extends BaseProvider implements SingleCompletionHan
 			return response.choices[0]?.message.content || ""
 		} catch (error) {
 			throw new Error(
-				"Please check the LM Studio developer logs to debug what went wrong. You may need to load the model with a larger context length to work with Roo Code's prompts.",
+				"Please check the LM Studio developer logs to debug what went wrong. You may need to load the model with a larger context length to work with Zoo Code's prompts.",
 			)
 		}
 	}

src/api/providers/vscode-lm.ts

@@ -395,7 +395,7 @@ export class VsCodeLmHandler extends BaseProvider implements SingleCompletionHan
 		try {
 			// Create the response stream with required options
 			const requestOptions: vscode.LanguageModelChatRequestOptions = {
-				justification: `Roo Code would like to use '${client.name}' from '${client.vendor}', Click 'Allow' to proceed.`,
+				justification: `Zoo Code would like to use '${client.name}' from '${client.vendor}', Click 'Allow' to proceed.`,
 				tools: convertToVsCodeLmTools(metadata?.tools ?? []),
 			}
 

src/core/task/Task.ts

@@ -1723,9 +1723,13 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 	async sayAndCreateMissingParamError(toolName: ToolName, paramName: string, relPath?: string) {
 		await this.say(
 			"error",
-			`Roo tried to use ${toolName}${
-				relPath ? ` for '${relPath.toPosix()}'` : ""
-			} without value for required parameter '${paramName}'. Retrying...`,
+			relPath
+				? t("tools:missingToolParameterWithPath", {
+						toolName,
+						relPath: relPath.toPosix(),
+						paramName,
+					})
+				: t("tools:missingToolParameter", { toolName, paramName }),
 		)
 		return formatResponse.toolError(formatResponse.missingToolParameterError(paramName))
 	}

src/core/task/__tests__/Task.spec.ts

@@ -156,6 +156,25 @@ vi.mock("../../environment/getEnvironmentDetails", () => ({
 
 vi.mock("../../ignore/RooIgnoreController")
 
+// i18n is not initialized in this suite, so the real t() returns the raw translation key.
+// Interpolate just the missing-parameter keys so tests can assert on the resolved, user-facing
+// copy; every other key is delegated to the real t() to keep the rest of the suite unchanged.
+vi.mock("../../../i18n", async (importOriginal) => {
+	const actual = (await importOriginal()) as typeof import("../../../i18n")
+	return {
+		...actual,
+		t: (key: string, args?: Record<string, unknown>) => {
+			if (key === "tools:missingToolParameterWithPath") {
+				return `Zoo tried to use ${args?.toolName} for '${args?.relPath}' without value for required parameter '${args?.paramName}'. Retrying...`
+			}
+			if (key === "tools:missingToolParameter") {
+				return `Zoo tried to use ${args?.toolName} without value for required parameter '${args?.paramName}'. Retrying...`
+			}
+			return actual.t(key, args)
+		},
+	}
+})
+
 vi.mock("../../condense", async (importOriginal) => {
 	const actual = (await importOriginal()) as any
 	return {
@@ -396,6 +415,41 @@ describe("Cline", () => {
 		})
 	})
 
+	describe("sayAndCreateMissingParamError", () => {
+		it("surfaces a localized error notice and returns the missing-parameter tool error for both relPath branches", async () => {
+			const cline = new Task({
+				provider: mockProvider,
+				apiConfiguration: mockApiConfig,
+				task: "test task",
+				startTask: false,
+			})
+
+			const saySpy = vi.spyOn(cline, "say").mockResolvedValue(undefined)
+
+			// relPath provided -> the "...WithPath" message branch.
+			const withPath = await cline.sayAndCreateMissingParamError("read_file", "path", "src/foo.ts")
+			// relPath omitted -> the plain message branch.
+			const withoutPath = await cline.sayAndCreateMissingParamError("execute_command", "command")
+
+			// Both branches emit an "error" say whose resolved text names the tool and the
+			// missing parameter (guards against a silent i18n regression where t() would
+			// otherwise return the raw key or an empty string and still type-check as a String).
+			expect(saySpy).toHaveBeenCalledTimes(2)
+			const [withPathChannel, withPathNotice] = saySpy.mock.calls[0]
+			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(withoutPathNotice).toEqual(expect.stringContaining("execute_command"))
+			expect(withoutPathNotice).toEqual(expect.stringContaining("command"))
+
+			// The returned tool error names the missing parameter.
+			expect(withPath).toContain("path")
+			expect(withoutPath).toContain("command")
+		})
+	})
+
 	describe("getEnvironmentDetails", () => {
 		describe("API conversation handling", () => {
 			beforeEach(() => {

src/core/webview/ClineProvider.ts

@@ -1177,7 +1177,7 @@ export class ClineProvider
 						window.AUDIO_BASE_URI = "${audioUri}"
 						window.MATERIAL_ICONS_BASE_URI = "${materialIconsUri}"
 					</script>
-					<title>Roo Code</title>
+					<title>Zoo Code</title>
 				</head>
 				<body>
 					<div id="root"></div>
@@ -1256,7 +1256,7 @@ export class ClineProvider
 				window.AUDIO_BASE_URI = "${audioUri}"
 				window.MATERIAL_ICONS_BASE_URI = "${materialIconsUri}"
 			</script>
-            <title>Roo Code</title>
+            <title>Zoo Code</title>
           </head>
           <body>
             <noscript>You need to enable JavaScript to run this app.</noscript>