fix(agent-runner-execution): use truncateUtf16Safe for external run failure detail

zhangqueping · zhangqueping · commit aa530d6289b6 · 2026-07-09T18:37:04.000+08:00
Replace raw UTF-16 slice in formatForwardedExternalRunFailureText with truncateUtf16Safe to prevent surrogate pair splitting. The module already imports truncateUtf16Safe for other truncation paths — this commit extends coverage to the remaining call site. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
diff --git a/src/auto-reply/reply/agent-runner-execution.ts b/src/auto-reply/reply/agent-runner-execution.ts
@@ -952,7 +952,7 @@ function formatForwardedExternalRunFailureText(message: string): string {
   }
   const detail =
     sanitized.length > EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS
-      ? `${sanitized.slice(0, EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS - 1).trimEnd()}…`
+      ? `${truncateUtf16Safe(sanitized, EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS - 1).trimEnd()}…`
       : sanitized;
   const suffix = /[.!?]$/u.test(detail) ? "" : ".";
   return `⚠️ Agent failed before reply: ${detail}${suffix} Please try again, or use /new to start a fresh session.`;

Original file line number	Diff line number	Diff line change
`@@ -952,7 +952,7 @@ function formatForwardedExternalRunFailureText(message: string): string {`
`952`	`952`	`}`
`953`	`953`	`const detail =`
`954`	`954`	`sanitized.length > EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS`
`955`		- ? `${sanitized.slice(0, EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS - 1).trimEnd()}…`
	`955`	+ ? `${truncateUtf16Safe(sanitized, EXTERNAL_RUN_FAILURE_DETAIL_MAX_CHARS - 1).trimEnd()}…`
`956`	`956`	`: sanitized;`
`957`	`957`	`const suffix = /[.!?]$/u.test(detail) ? "" : ".";`
`958`	`958`	return `⚠️ Agent failed before reply: ${detail}${suffix} Please try again, or use /new to start a fresh session.`;