Add tests for the MCP auth-marker clearing and retry-copy transitions

Author: TheGreatAxios

src/tui/command-surfaces.test.ts

@@ -1975,6 +1975,31 @@ describe("mcp surface", () => {
     });
   });
 
+  test("a timed-out authorization failure still offers Enter-retry copy", async () => {
+    await withWiredShell(async (shell, harness) => {
+      openCommandSurface(shell, "mcp", {
+        notify: () => undefined,
+        mcp: {
+          // Short timeout wording so the two-line describe zone has room
+          // left for the impact line — a `what` that wraps to both lines
+          // crowds `impact` out by design (see describeZoneLines).
+          list: () => [
+            {
+              name: "granola",
+              state: "failed",
+              error: "timed out waiting for the browser",
+            },
+          ],
+          openAuthURL: () => undefined,
+        },
+      });
+      await harness.renderOnce();
+      const frame = harness.captureCharFrame();
+      expect(frame).toContain("granola — failed");
+      expect(frame).toContain("Enter retries");
+    });
+  });
+
   test("Alt+R confirms before removing a custom server", async () => {
     await withWiredShell(async (shell, harness) => {
       const removed: string[] = [];

src/tui/runtime-channels.test.ts

@@ -169,6 +169,30 @@ describe("mcp.status channel", () => {
     }
   });
 
+  test("an ordinary failure after needs-auth clears the standing auth mark", async () => {
+    const { host, emitter, frame, cleanup } = await mountHeadless();
+    try {
+      emitter.emit("mcp.status", {
+        name: "granola",
+        state: "needs-auth",
+        url: "https://mcp.test/auth",
+      });
+      expect(await frame()).toContain("mcp !");
+
+      emitter.emit("mcp.status", {
+        name: "granola",
+        state: "failed",
+        error: "ECONNREFUSED",
+      });
+      const painted = await frame();
+      expect(host.shell.mcpNeedsAuth).toEqual([]);
+      expect(painted).not.toContain("mcp !");
+      expect(host.shell.statusFlash).toContain("mcp granola did not connect");
+    } finally {
+      cleanup();
+    }
+  });
+
   test("a failed connect keeps the landing mountain and rides the notice strip (CL-5600)", async () => {
     // Full product-host path: mcp.status → mcpNotice → surfaceSystemNotice.
     // The unit landing suite covers surfaceSystemNotice alone; this locks the