Park unused chrome task and agents strips

Author: TheGreatAxios

src/tui/chrome-state.test.ts

@@ -30,15 +30,15 @@ describe("formatChromeZones", () => {
     })
   })
 
-  test("partial: task rows only", () => {
+  test("partial: task rows do not auto-paint (zones parked)", () => {
     const out = formatChromeZones({
       task: [{ title: "cutover readiness", status: "doing" }],
     })
-    expect(out.task).toEqual([{ label: "cutover readiness", status: "doing" }])
+    expect(out.task).toBeNull()
     expect(out.agents).toBeNull()
   })
 
-  test("running agents: agents zone stays null; checklist suppressed", () => {
+  test("running agents: both zones stay null", () => {
     const state: ChromeLiveState = {
       task: [
         { title: "chrome live helper", status: "doing" },
@@ -64,13 +64,12 @@ describe("formatChromeZones", () => {
       ],
     }
     const out = formatChromeZones(state, NOW)
-    // Transcript Task rows own live lane status — no FLEET board / agents zone.
-    // Checklist is suppressed while any lane is running.
+    // Both strips parked — live work stays on transcript ● Task rows.
     expect(out.task).toBeNull()
     expect(out.agents).toBeNull()
   })
 
-  test("idle with checklist still formats tasks when no lane is running", () => {
+  test("idle with open checklist still returns null (zones parked)", () => {
     const out = formatChromeZones(
       {
         task: [
@@ -89,10 +88,7 @@ describe("formatChromeZones", () => {
       NOW,
     )
     expect(out.agents).toBeNull()
-    expect(out.task).toEqual([
-      { label: "chrome live helper", status: "doing" },
-      { label: "wire chrome zone", status: "todo" },
-    ])
+    expect(out.task).toBeNull()
   })
 
   test("observe does not force an agents panel via formatChromeZones", () => {
@@ -113,8 +109,7 @@ describe("formatChromeZones", () => {
       },
       NOW,
     )
-    // Agents zone is always null from formatChromeZones; observe is not a
-    // chrome-zone surface here (agents zone stays empty; stack-only layout).
+    // Both zones always null from formatChromeZones (parked pending rebuild).
     expect(out.agents).toBeNull()
     expect(out.task).toBeNull()
   })
@@ -364,7 +359,7 @@ describe("chromeFromSession", () => {
     ])
 
     const zones = formatChromeZones(state, NOW)
-    // Running lanes suppress checklist; agents zone stays null (no FLEET board).
+    // Both chrome strips parked pending rebuild.
     expect(zones.task).toBeNull()
     expect(zones.agents).toBeNull()
   })

src/tui/chrome-state.ts

@@ -1,9 +1,16 @@
 /**
  * Live chrome zone formatter for setChromeZones.
  *
- * Pure: structured session state → one-line task / agents strings.
- * Heights stay with geometry (zones max 1 row each); this module never
- * invents row budgets.
+ * Pure: structured session state → task / agents zone rows.
+ * Heights stay with geometry; this module never invents row budgets.
+ *
+ * ## Parked auto-paint
+ *
+ * `formatChromeZones` currently always returns `{ task: null, agents: null }`
+ * — both chrome strips are parked pending rebuild. Live work stays on
+ * transcript `● Task …` rows. `formatTasksPanel` / `formatAgentsPanel` remain
+ * for demos, tests, and a future rebuild; manual `setChromeZones` can still
+ * feed preformatted rows.
  *
  * ## Product host push contract
  *
@@ -19,7 +26,8 @@
  *
  * Always pass the full snapshot so absent zones clear (`null` hides the zone).
  * Partial object fields mean “no data” → that zone line is null, not left
- * stale. Observe mode can override the agents line via `state.observe`.
+ * stale. Observe mode can override the agents line via `state.observe` when
+ * agents chrome is rebuilt.
  */
 
 import {
@@ -129,28 +137,20 @@ export type FormattedChromeZones = {
 /**
  * Format structured live state into chrome zone rows for setChromeZones.
  *
- * Empty / partial / inactive inputs yield null for the corresponding zone
- * so geometry collapses that strip (idleDefault 0).
- *
- * Live sub-agent work paints as `● Task …` transcript rows (runtime-bridge),
- * not as a standing FLEET board or dual-rail agents zone — those restated the
- * same lanes above the chat and made progress harder to read. The agents zone
- * stays empty so dual layout never engages. The manage_tasks checklist is
- * also suppressed while any lane is running; it returns once the fleet is dry.
+ * Both chrome strips (task checklist + agents/fleet board) are parked pending
+ * rebuild: this always returns `{ task: null, agents: null }` so nothing
+ * auto-paints in those zones. Live work stays on transcript `● Task …` rows
+ * (runtime-bridge). `formatTasksPanel` / `formatAgentsPanel` stay intact for
+ * demos, tests, and a future rebuild; manual `setChromeZones` / Alt+T can still
+ * feed preformatted rows into the shell.
  */
 export function formatChromeZones(
   state: ChromeLiveState,
   nowMs: number = Date.now(),
 ): FormattedChromeZones {
+  void state
   void nowMs
-  const hasLiveAgents =
-    state.agents !== null &&
-    state.agents !== undefined &&
-    state.agents.some((s) => s.status === "running")
-  // Fleet board chrome is off: transcript Task rows own live lane status.
-  const agents = null
-  const task = hasLiveAgents ? null : formatTasksPanel(state.task)
-  return { task, agents }
+  return { task: null, agents: null }
 }
 
 /**

src/tui/runner-host.test.ts

@@ -6,7 +6,7 @@ import type { KeyEvent } from "@opentui/core"
 import type { CostSummary } from "../cost/cost-summary.js"
 import type { SubAgentSession } from "../subagent/session-store.js"
 import { createHarness } from "./harness.js"
-import { acceptOverlaySelection, closeInsetOverlay, runOverlayAction, toggleTasksPanel } from "./shell.js"
+import { acceptOverlaySelection, closeInsetOverlay, runOverlayAction } from "./shell.js"
 import {
   mountRunnerHost,
   observeSessionFromSubAgents,
@@ -129,19 +129,11 @@ describe("observeSessionFromSubAgents", () => {
 })
 
 describe("mountRunnerHost chrome wiring", () => {
-  // CL-5731: the task-change callback was built (director writes tasks,
-  // getTasks()/onTasksChange exist) but had no live consumer — the chrome
-  // push mechanism type-checked fine with `subscribeChrome` omitted, so a
-  // director's task update never reached the shell. `subscribeChrome` is now
-  // a required dep (not optional) so that regression cannot type-check
-  // again, but the type alone does not prove the wiring actually runs: this
-  // test drives a real notify() call through mountRunnerHost end to end and
-  // asserts the task panel painted from it, the way the real runner's
-  // `emitter.emit("tasks", ...)` -> `subscribeChrome` -> `pushChrome` chain
-  // does. If `subscribeChrome`'s notify callback were ever dropped again
-  // (e.g. `deps.subscribeChrome?.(pushChrome)` silently no-op on undefined),
-  // this test fails because the second push never reaches the panel.
-  test("a live chrome push (subscribeChrome notify) repaints the task panel", async () => {
+  // CL-5731: subscribeChrome must stay wired end-to-end. formatChromeZones
+  // now parks both chrome strips (always null), so a tasks push must not
+  // paint the checklist — this test asserts the notify path still runs and
+  // leaves the task panel empty (rebuild later; live work is ● Task rows).
+  test("a live chrome push (subscribeChrome notify) does not auto-paint the task panel", async () => {
     const harness = await createHarness({ width: 80, height: 24 })
     let liveTasks: readonly { title: string; status: "todo" | "doing" | "done" | "cancelled" }[] = []
     let notify: (() => void) | undefined
@@ -168,20 +160,17 @@ describe("mountRunnerHost chrome wiring", () => {
       expect(host.shell.taskBox.visible).toBe(false)
       expect(notify).toBeDefined()
 
-      // Mirrors createChatDirector's onTasksChange firing after a
-      // manage_tasks tool call: the live source changes, then the runner
-      // notifies the host — it does not push the new snapshot itself.
+      // Mirrors createChatDirector's onTasksChange: live source changes, then
+      // the runner notifies the host. formatChromeZones parks the checklist.
       liveTasks = [{ title: "wire task panel", status: "doing" }]
       notify?.()
 
-      // CL-5847: the panel is hidden by default, so the live push lands in
-      // tasksRaw underneath without showing. It stays hidden until opt-in.
       expect(host.shell.taskBox.visible).toBe(false)
-      toggleTasksPanel(host.shell)
-      expect(host.shell.taskBox.visible).toBe(true)
       await harness.renderOnce()
       const frame = harness.captureCharFrame()
-      expect(frame).toContain("wire task panel")
+      expect(frame).not.toContain("wire task panel")
+      // Notify callback stayed registered — subscribe path ran without error.
+      expect(notify).toBeDefined()
     } finally {
       host.dispose()
       harness.destroy()