fix(agents): detect C1 control characters in hasControlCharacter

lsr911 · claude · lsr911 · commit 6e43dd4b4566 · 2026-07-10T12:45:06.000+08:00
hasControlCharacter() checked for C0 control characters (0x00-0x1f) and DEL (0x7f) but did not check for C1 controls (0x80-0x9f). The C1 range includes the CSI introducer (0x9b) and other control codes that should not appear in user-facing error text. Add the C1 range to the check. Same pattern as the C1 fixes in sanitizeForConsole (#103226) and renderTerminalBufferText (#103274). Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/src/agents/failover-error.ts b/src/agents/failover-error.ts
@@ -686,7 +686,7 @@ export function buildProviderReauthCommand(
 function hasControlCharacter(value: string): boolean {
   for (let i = 0; i < value.length; i += 1) {
     const code = value.charCodeAt(i);
-    if (code < 0x20 || code === 0x7f) {
+    if (code < 0x20 || code === 0x7f || (code >= 0x80 && code <= 0x9f)) {
       return true;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -686,7 +686,7 @@ export function buildProviderReauthCommand(`
`686`	`686`	`function hasControlCharacter(value: string): boolean {`
`687`	`687`	`for (let i = 0; i < value.length; i += 1) {`
`688`	`688`	`const code = value.charCodeAt(i);`
`689`		`- if (code < 0x20 \|\| code === 0x7f) {`
	`689`	`+ if (code < 0x20 \|\| code === 0x7f \|\| (code >= 0x80 && code <= 0x9f)) {`
`690`	`690`	`return true;`
`691`	`691`	`}`
`692`	`692`	`}`