fix: remove incorrect manual string replacement in tool repetition error message

roomote · roomote · commit 55dabfe321d2 · 2025-07-18T12:48:17.000Z
The ToolRepetitionDetector already handles tool name interpolation via the i18n system when calling t() with the toolName option. The manual .replace() call was trying to replace {toolName} but the actual placeholder is {{toolName}} (double braces), causing the tool name to never be properly interpolated in the error message. This fix removes the redundant manual replacement and lets the i18n system handle the interpolation correctly. Fixes #5879
diff --git a/src/core/assistant-message/presentAssistantMessage.ts b/src/core/assistant-message/presentAssistantMessage.ts
@@ -379,7 +379,7 @@ export async function presentAssistantMessage(cline: Task) {
 					// Handle repetition similar to mistake_limit_reached pattern.
 					const { response, text, images } = await cline.ask(
 						repetitionCheck.askUser.messageKey as ClineAsk,
-						repetitionCheck.askUser.messageDetail.replace("{toolName}", block.name),
+						repetitionCheck.askUser.messageDetail,
 					)
 
 					if (response === "messageResponse") {

Original file line number	Diff line number	Diff line change
`@@ -379,7 +379,7 @@ export async function presentAssistantMessage(cline: Task) {`
`379`	`379`	`// Handle repetition similar to mistake_limit_reached pattern.`
`380`	`380`	`const { response, text, images } = await cline.ask(`
`381`	`381`	`repetitionCheck.askUser.messageKey as ClineAsk,`
`382`		`- repetitionCheck.askUser.messageDetail.replace("{toolName}", block.name),`
	`382`	`+ repetitionCheck.askUser.messageDetail,`
`383`	`383`	`)`
`384`	`384`
`385`	`385`	`if (response === "messageResponse") {`