refactor(init): use renderInlineMarkdown for spinner messages

Pipe spinner messages through the markdown rendering pipeline so file
names, commands, and project names display as styled inline code in the
terminal. Update truncateForTerminal to handle ANSI escape sequences
when measuring visible width.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: betegon

src/lib/init/wizard-runner.ts

@@ -25,6 +25,8 @@ import { CLI_VERSION } from "../constants.js";
 import { getAuthToken } from "../db/auth.js";
 import { WizardError } from "../errors.js";
 import { terminalLink } from "../formatters/colors.js";
+import { renderInlineMarkdown } from "../formatters/markdown.js";
+import { stripAnsi } from "../formatters/plain-detect.js";
 import { resolveOrCreateTeam } from "../resolve-team.js";
 import { slugify } from "../utils.js";
 import {
@@ -86,10 +88,28 @@ function nextPhase(
 function truncateForTerminal(message: string): string {
   // Reserve space for spinner (2 chars) + some padding
   const maxWidth = (process.stdout.columns || 80) - 4;
-  if (message.length <= maxWidth) {
+  if (stripAnsi(message).length <= maxWidth) {
     return message;
   }
-  return `${message.slice(0, maxWidth - 1)}…`;
+  // Truncate by visible width, preserving ANSI sequences
+  let visible = 0;
+  let i = 0;
+  let result = "";
+  while (i < message.length && visible < maxWidth - 1) {
+    if (message[i] === "\x1b") {
+      const end = message.indexOf("m", i);
+      if (end !== -1) {
+        result += message.slice(i, end + 1);
+        i = end + 1;
+        continue;
+      }
+    }
+    result += message[i];
+    visible += 1;
+    i += 1;
+  }
+  // Close any open ANSI sequences with a reset
+  return `${result}\x1b[0m…`;
 }
 
 /**
@@ -112,7 +132,7 @@ export function describeLocalOp(payload: LocalOpPayload): string {
       const first = patches[0];
       if (patches.length === 1 && first) {
         const verb = patchActionVerb(first.action);
-        return `${verb} ${basename(first.path)}...`;
+        return `${verb} \`${basename(first.path)}\`...`;
       }
       const counts = patchActionCounts(patches);
       return `Applying ${patches.length} file changes (${counts})...`;
@@ -121,14 +141,14 @@ export function describeLocalOp(payload: LocalOpPayload): string {
       const cmds = payload.params.commands;
       const first = cmds[0];
       if (cmds.length === 1 && first) {
-        return `Running ${first}...`;
+        return `Running \`${first}\`...`;
       }
-      return `Running ${cmds.length} commands (${first ?? "..."}, ...)...`;
+      return `Running ${cmds.length} commands (\`${first ?? "..."}\`, ...)...`;
     }
     case "list-dir":
       return "Listing directory...";
     case "create-sentry-project":
-      return `Creating project "${payload.params.name}" (${payload.params.platform})...`;
+      return `Creating project \`${payload.params.name}\` (${payload.params.platform})...`;
     case "detect-sentry":
       return "Checking for existing Sentry setup...";
     default:
@@ -144,12 +164,12 @@ function describeFilePaths(verb: string, paths: string[]): string {
     return `${verb} files...`;
   }
   if (paths.length === 1) {
-    return `${verb} ${basename(first)}...`;
+    return `${verb} \`${basename(first)}\`...`;
   }
   if (paths.length === 2 && second) {
-    return `${verb} ${basename(first)}, ${basename(second)}...`;
+    return `${verb} \`${basename(first)}\`, \`${basename(second)}\`...`;
   }
-  return `${verb} ${paths.length} files (${basename(first)}${second ? `, ${basename(second)}` : ""}, ...)...`;
+  return `${verb} ${paths.length} files (\`${basename(first)}\`${second ? `, \`${basename(second)}\`` : ""}, ...)...`;
 }
 
 /** Map a patch action to a user-facing verb. */
@@ -188,13 +208,13 @@ async function handleSuspendedStep(
   const label = STEP_LABELS[stepId] ?? stepId;
 
   if (payload.type === "local-op") {
-    const message = describeLocalOp(payload);
+    const message = renderInlineMarkdown(describeLocalOp(payload));
     spin.message(truncateForTerminal(message));
 
     const localResult = await handleLocalOp(payload, options);
 
     if (localResult.message) {
-      spin.stop(localResult.message);
+      spin.stop(renderInlineMarkdown(localResult.message));
       spin.start("Processing...");
     }
 

test/lib/init/wizard-runner.test.ts

@@ -28,6 +28,7 @@ import * as auth from "../../../src/lib/db/auth.js";
 // biome-ignore lint/performance/noNamespaceImport: spyOn requires object reference
 import * as userDb from "../../../src/lib/db/user.js";
 import { WizardError } from "../../../src/lib/errors.js";
+import { stripAnsi } from "../../../src/lib/formatters/plain-detect.js";
 // biome-ignore lint/performance/noNamespaceImport: spyOn requires object reference
 import * as fmt from "../../../src/lib/init/formatters.js";
 // biome-ignore lint/performance/noNamespaceImport: spyOn requires object reference
@@ -628,8 +629,8 @@ describe("runWizard", { timeout: TEST_TIMEOUT_MS }, () => {
         ) as string[] | undefined;
         expect(call).toBeDefined();
         const msg = call?.[0] ?? "";
-        expect(msg.length).toBeLessThanOrEqual(36);
-        expect(msg.endsWith("…")).toBe(true);
+        expect(stripAnsi(msg).length).toBeLessThanOrEqual(36);
+        expect(stripAnsi(msg).endsWith("…")).toBe(true);
       } finally {
         Object.defineProperty(process.stdout, "columns", {
           value: origColumns,
@@ -939,7 +940,7 @@ describe("describeLocalOp", () => {
           params: { paths: ["src/settings.py"] },
         })
       );
-      expect(msg).toBe("Reading settings.py...");
+      expect(msg).toBe("Reading `settings.py`...");
     });
 
     test("two files shows both basenames", () => {
@@ -949,7 +950,7 @@ describe("describeLocalOp", () => {
           params: { paths: ["src/settings.py", "src/urls.py"] },
         })
       );
-      expect(msg).toBe("Reading settings.py, urls.py...");
+      expect(msg).toBe("Reading `settings.py`, `urls.py`...");
     });
 
     test("three+ files shows count and first two basenames", () => {
@@ -961,7 +962,7 @@ describe("describeLocalOp", () => {
           },
         })
       );
-      expect(msg).toBe("Reading 4 files (one.py, two.py, ...)...");
+      expect(msg).toBe("Reading 4 files (`one.py`, `two.py`, ...)...");
     });
 
     test("empty paths array", () => {
@@ -980,7 +981,7 @@ describe("describeLocalOp", () => {
           params: { paths: ["requirements.txt"] },
         })
       );
-      expect(msg).toBe("Checking requirements.txt...");
+      expect(msg).toBe("Checking `requirements.txt`...");
     });
 
     test("multiple files shows count", () => {
@@ -990,7 +991,7 @@ describe("describeLocalOp", () => {
           params: { paths: ["a.py", "b.py", "c.py"] },
         })
       );
-      expect(msg).toBe("Checking 3 files (a.py, b.py, ...)...");
+      expect(msg).toBe("Checking 3 files (`a.py`, `b.py`, ...)...");
     });
   });
 
@@ -1004,7 +1005,7 @@ describe("describeLocalOp", () => {
           },
         })
       );
-      expect(msg).toBe("Creating sentry.py...");
+      expect(msg).toBe("Creating `sentry.py`...");
     });
 
     test("single modify shows verb and basename", () => {
@@ -1016,7 +1017,7 @@ describe("describeLocalOp", () => {
           },
         })
       );
-      expect(msg).toBe("Modifying settings.py...");
+      expect(msg).toBe("Modifying `settings.py`...");
     });
 
     test("single delete shows verb and basename", () => {
@@ -1028,7 +1029,7 @@ describe("describeLocalOp", () => {
           },
         })
       );
-      expect(msg).toBe("Deleting old.js...");
+      expect(msg).toBe("Deleting `old.js`...");
     });
 
     test("multiple patches shows count and breakdown", () => {
@@ -1056,7 +1057,7 @@ describe("describeLocalOp", () => {
           params: { commands: ["pip install sentry-sdk"] },
         })
       );
-      expect(msg).toBe("Running pip install sentry-sdk...");
+      expect(msg).toBe("Running `pip install sentry-sdk`...");
     });
 
     test("multiple commands shows count and first", () => {
@@ -1068,7 +1069,7 @@ describe("describeLocalOp", () => {
           },
         })
       );
-      expect(msg).toBe("Running 2 commands (pip install sentry-sdk, ...)...");
+      expect(msg).toBe("Running 2 commands (`pip install sentry-sdk`, ...)...");
     });
   });
 
@@ -1089,7 +1090,7 @@ describe("describeLocalOp", () => {
           params: { name: "my-app", platform: "python-django" },
         })
       );
-      expect(msg).toBe('Creating project "my-app" (python-django)...');
+      expect(msg).toBe("Creating project `my-app` (python-django)...");
     });
   });
 });