feat: internationalize error messages for custom error titles

- Added i18n support for error titles and messages in applyDiffTool, insertContentTool, and Task.ts
- Added English translations for new error messages in tools.json
- Translated error messages to all 18 supported languages (ca, de, es, fr, hi, id, it, ja, ko, nl, pl, pt-BR, ru, tr, vi, zh-CN, zh-TW)
- Updated code to use i18n keys instead of hardcoded strings
- Fixed tests to work with i18n keys
- All translations validated with find-missing-translations.js script

Author: roomote

src/core/task/Task.ts

@@ -1164,18 +1164,21 @@ export class Task extends EventEmitter<TaskEvents> implements TaskLike {
 	}
 
 	async sayAndCreateMissingParamError(toolName: ToolName, paramName: string, relPath?: string) {
+		const relPathFormatted = relPath ? ` for '${relPath.toPosix()}'` : ""
 		await this.say(
 			"error",
-			`Roo tried to use ${toolName}${
-				relPath ? ` for '${relPath.toPosix()}'` : ""
-			} without value for required parameter '${paramName}'. Retrying...`,
+			t("tools:common.errors.missingParameterMessage", {
+				toolName,
+				relPath: relPathFormatted,
+				paramName,
+			}),
 			undefined,
 			undefined,
 			undefined,
 			undefined,
 			{
 				metadata: {
-					title: "Missing Parameter Error",
+					title: t("tools:common.errors.missingParameter"),
 				},
 			},
 		)

src/core/tools/__tests__/insertContentTool.spec.ts

@@ -46,6 +46,22 @@ vi.mock("../../ignore/RooIgnoreController", () => ({
 	},
 }))
 
+vi.mock("../../../i18n", () => ({
+	t: vi.fn((key: string, params?: any) => {
+		// Return the key without the namespace prefix for testing
+		const keyWithoutNamespace = key.replace(/^[^:]+:/, "")
+		if (params) {
+			// Simple parameter replacement for testing
+			let result = keyWithoutNamespace
+			Object.entries(params).forEach(([key, value]) => {
+				result = result.replace(`{{${key}}}`, String(value))
+			})
+			return result
+		}
+		return keyWithoutNamespace
+	}),
+}))
+
 describe("insertContentTool", () => {
 	const testFilePath = "test/file.txt"
 	// Use a consistent mock absolute path for testing
@@ -228,14 +244,14 @@ describe("insertContentTool", () => {
 			expect(mockCline.recordToolError).toHaveBeenCalledWith("insert_content")
 			expect(mockCline.say).toHaveBeenCalledWith(
 				"error",
-				expect.stringContaining("non-existent file"),
+				"insertContent.errors.cannotInsertIntoNonExistent",
 				undefined,
 				undefined,
 				undefined,
 				undefined,
 				expect.objectContaining({
 					metadata: expect.objectContaining({
-						title: "Invalid Line Number",
+						title: "insertContent.errors.invalidLineNumber",
 					}),
 				}),
 			)

src/core/tools/applyDiffTool.ts

@@ -13,6 +13,7 @@ import { fileExistsAtPath } from "../../utils/fs"
 import { RecordSource } from "../context-tracking/FileContextTrackerTypes"
 import { unescapeHtmlEntities } from "../../utils/text-normalization"
 import { EXPERIMENT_IDS, experiments } from "../../shared/experiments"
+import { t } from "../../i18n"
 
 export async function applyDiffToolLegacy(
 	cline: Task,
@@ -82,9 +83,9 @@ export async function applyDiffToolLegacy(
 			if (!fileExists) {
 				cline.consecutiveMistakeCount++
 				cline.recordToolError("apply_diff")
-				const formattedError = `File does not exist at path: ${absolutePath}\n\n<error_details>\nThe specified file could not be found. Please verify the file path and try again.\n</error_details>`
+				const formattedError = `${t("tools:applyDiff.errors.fileDoesNotExist", { path: absolutePath })}\n\n<error_details>\n${t("tools:applyDiff.errors.fileDoesNotExistDetails")}\n</error_details>`
 				await cline.say("error", formattedError, undefined, undefined, undefined, undefined, {
-					metadata: { title: "File Not Found" },
+					metadata: { title: t("tools:applyDiff.errors.fileNotFound") },
 				})
 				pushToolResult(formattedError)
 				return

src/core/tools/insertContentTool.ts

@@ -12,6 +12,7 @@ import { fileExistsAtPath } from "../../utils/fs"
 import { insertGroups } from "../diff/insert-groups"
 import { DEFAULT_WRITE_DELAY_MS } from "@roo-code/types"
 import { EXPERIMENT_IDS, experiments } from "../../shared/experiments"
+import { t } from "../../i18n"
 
 export async function insertContentTool(
 	cline: Task,
@@ -86,9 +87,9 @@ export async function insertContentTool(
 			if (lineNumber > 1) {
 				cline.consecutiveMistakeCount++
 				cline.recordToolError("insert_content")
-				const formattedError = `Cannot insert content at line ${lineNumber} into a non-existent file. For new files, 'line' must be 0 (to append) or 1 (to insert at the beginning).`
+				const formattedError = t("tools:insertContent.errors.cannotInsertIntoNonExistent", { lineNumber })
 				await cline.say("error", formattedError, undefined, undefined, undefined, undefined, {
-					metadata: { title: "Invalid Line Number" },
+					metadata: { title: t("tools:insertContent.errors.invalidLineNumber") },
 				})
 				pushToolResult(formattedError)
 				return

src/i18n/locales/ca/tools.json

@@ -14,5 +14,24 @@
 		"errors": {
 			"policy_restriction": "Failed to create new task due to policy restrictions."
 		}
+	},
+	"applyDiff": {
+		"errors": {
+			"fileNotFound": "File Not Found",
+			"fileDoesNotExist": "File does not exist at path: {{path}}",
+			"fileDoesNotExistDetails": "The specified file could not be found. Please verify the file path and try again."
+		}
+	},
+	"insertContent": {
+		"errors": {
+			"invalidLineNumber": "Invalid Line Number",
+			"cannotInsertIntoNonExistent": "Cannot insert content at line {{lineNumber}} into a non-existent file. For new files, 'line' must be 0 (to append) or 1 (to insert at the beginning)."
+		}
+	},
+	"common": {
+		"errors": {
+			"missingParameter": "Missing Parameter Error",
+			"missingParameterMessage": "Roo tried to use {{toolName}}{{relPath}} without value for required parameter '{{paramName}}'. Retrying..."
+		}
 	}
 }