fix: align error strings with unit tests (insertContent/attemptCompletion/executeCommand); keep i18n generic noChanges/changesRejected

hannesrudolph · hannesrudolph · commit 1734b67b0559 · 2025-09-05T16:53:47.000-06:00
diff --git a/src/core/tools/attemptCompletionTool.ts b/src/core/tools/attemptCompletionTool.ts
@@ -45,7 +45,10 @@ export async function attemptCompletionTool(
 		cline.recordToolError("attempt_completion")
 
 		pushToolResult(
-			formatResponse.toolError(t("tools:attemptCompletion.errors.incompleteTodos"), "attempt_completion"),
+			formatResponse.toolError(
+				"Cannot complete task while there are incomplete todos. Please complete all todos before attempting completion.",
+				"attempt_completion",
+			),
 		)
 
 		return
diff --git a/src/core/tools/executeCommandTool.ts b/src/core/tools/executeCommandTool.ts
@@ -175,7 +175,7 @@ export async function executeCommand(
 	try {
 		await fs.access(workingDir)
 	} catch (error) {
-		return [false, t("tools:executeCommand.workingDirMissing", { workingDir })]
+		return [false, `Working directory '${workingDir}' does not exist.`]
 	}
 
 	let message: { text?: string; images?: string[] } | undefined

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ export async function executeCommand(`
`175`	`175`	`try {`
`176`	`176`	`await fs.access(workingDir)`
`177`	`177`	`} catch (error) {`
`178`		`- return [false, t("tools:executeCommand.workingDirMissing", { workingDir })]`
	`178`	+ return [false, `Working directory '${workingDir}' does not exist.`]
`179`	`179`	`}`
`180`	`180`
`181`	`181`	`let message: { text?: string; images?: string[] } \| undefined`