fix(models): sanitize formatter input before measuring width

vincentkoc · vincentkoc · commit 8218479d9fce · 2026-07-09T06:37:33.000-07:00
diff --git a/src/commands/models/list.format.test.ts b/src/commands/models/list.format.test.ts
@@ -43,6 +43,10 @@ describe("truncate", () => {
     expect(result).toBe("");
     expect(hasLoneSurrogate(result)).toBe(false);
   });
+
+  it("sanitizes terminal controls before measuring visible width", () => {
+    expect(truncate("ab\u001B]2;hidden\u0007cd", 6)).toBe("abcd");
+  });
 });
 
 describe("pad", () => {
diff --git a/src/commands/models/list.format.ts b/src/commands/models/list.format.ts
@@ -1,5 +1,6 @@
 /** Formatting helpers for model-list terminal tables. */
 import { truncateToVisibleWidth, visibleWidth } from "../../../packages/terminal-core/src/ansi.js";
+import { sanitizeTerminalText } from "../../../packages/terminal-core/src/safe-text.js";
 import { isRich as isRichTerminal, theme } from "../../../packages/terminal-core/src/theme.js";
 
 const TRUNCATED_SUFFIX = "...";
@@ -45,11 +46,12 @@ export const formatTag = (tag: string, rich: boolean) => {
 
 /** Truncates model-list cells to terminal visible width with an ASCII ellipsis. */
 export const truncate = (value: string, max: number) => {
-  if (visibleWidth(value) <= max) {
-    return value;
+  const sanitized = sanitizeTerminalText(value);
+  if (visibleWidth(sanitized) <= max) {
+    return sanitized;
   }
   if (max <= TRUNCATED_SUFFIX.length) {
-    return truncateToVisibleWidth(value, max);
+    return truncateToVisibleWidth(sanitized, max);
   }
-  return `${truncateToVisibleWidth(value, max - TRUNCATED_SUFFIX.length)}${TRUNCATED_SUFFIX}`;
+  return `${truncateToVisibleWidth(sanitized, max - TRUNCATED_SUFFIX.length)}${TRUNCATED_SUFFIX}`;
 };
