fix(tui): keep every approval choice on screen under a tall body

Author: bwachman

src/tui/approval-prompt-visibility.test.ts

@@ -5,7 +5,9 @@
  * box's growth and over the overlay's own context text.
  */
 import { describe, expect, test } from "bun:test";
+import { EventEmitter } from "node:events";
 import { defined } from "../../testkit/defined.js";
+import { wireGates } from "./gate-wire.js";
 import { makePermissionItems, withTestRenderer } from "./harness.js";
 import { appendStreamRow } from "./shell/chrome.js";
 import { createAppShell } from "./shell/index.js";
@@ -178,3 +180,60 @@ describe("approval overlay keeps the prompt box on screen (CL-5750)", () => {
     );
   });
 });
+
+// A long multi-line command makes the permission body taller than the host.
+// Only the first choice was guaranteed rows, so the prompt showed Reject alone
+// and Accept once sat scrolled out of view (CL-11362).
+const LONG_COMMAND = [
+  "set +e",
+  "TARGET=/Users/operator/code/worktrees/corbits-code/stack-1559-1561-1562-1563-cl-10768",
+  'FIXTURE="$(mktemp -d /tmp/corbits-linear-release.XXXXXX)"',
+  ...Array.from(
+    { length: 28 },
+    (_, i) =>
+      `echo "probe step ${i}: checking the release issue list" >> "$FIXTURE/log"`,
+  ),
+].join("\n");
+
+const FIXED_CHOICES = ["Reject", "Reject and stop", "Accept once"] as const;
+
+describe("a tall approval body never hides a choice (CL-11362)", () => {
+  for (const [columns, rows] of [
+    [80, 24],
+    [100, 30],
+  ] as const) {
+    test(`every choice is painted for a long command at ${columns}x${rows}`, async () => {
+      await withTestRenderer(
+        async (h) => {
+          const shell = createAppShell(h.renderer, {
+            terminal: { columns, rows },
+            run: "idle",
+          });
+          const emitter = new EventEmitter();
+          const dispose = wireGates(emitter, shell);
+          try {
+            primeSession(shell);
+            emitter.emit("permission.gate", {
+              id: "gate-1",
+              request: {
+                tool: "run_shell",
+                action: "Run shell command",
+                subject: LONG_COMMAND,
+                scopes: [],
+              },
+              resolve: () => undefined,
+            });
+            await h.renderOnce();
+            await h.renderOnce();
+            const frame = h.captureCharFrame();
+            for (const choice of FIXED_CHOICES) expect(frame).toContain(choice);
+          } finally {
+            dispose();
+            shell.dispose();
+          }
+        },
+        { width: columns, height: rows },
+      );
+    });
+  }
+});

src/tui/shell/chrome.ts

@@ -840,7 +840,13 @@ function fitOverlayListToHost(shell: AppShell, hostH: number): void {
   const hasAnswer = overlayAnswerState(shell) !== null;
   const perItem = overlayRowsPerItem(shell.overlayKind);
   const hasItems = shell.overlayItems.length > 0;
-  const choiceWant = hasItems ? perItem : 0;
+  // A decision's first choice is Reject, so reserving one choice can leave
+  // the only approving choice scrolled out of view under a tall body.
+  const choiceWant = !hasItems
+    ? 0
+    : isDecisionOverlay(shell.overlayKind)
+      ? shell.overlayItems.length * perItem
+      : perItem;
   const choiceMin = hasItems ? 1 : 0;
   let bodyCount = shell.overlayBodyLines.length;
   const chromeOf = (n: number): number =>