@esso0428/pi-subagents 0.17.12 → 0.17.13

package/dist/ui/agent-widget.js

@@ -4,19 +4,24 @@
  * Displays a tree of agents with animated spinners, live stats, and activity descriptions.
  * Uses the callback form of setWidget for themed rendering.
  */
-import { Editor, isKeyRelease, Key, matchesKey, truncateToWidth } from "@earendil-works/pi-tui";
+import { Editor, isKeyRelease, Key, matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import { renderAgentName } from "../agent-color.js";
-import { isTopLevelAgent } from "../agent-manager.js";
 import { getConfig } from "../agent-types.js";
 import { getLifetimeCost, getLifetimeTotal, getSessionContextPercent } from "../usage.js";
 // ---- Constants ----
 /** Maximum number of rendered lines before overflow collapse kicks in. */
 export const MAX_WIDGET_LINES = 12;
-/** Number of lines available above an editor, retaining room for the prompt. */
-export function getWidgetLineBudget(terminalRows) {
-    if (!Number.isFinite(terminalRows))
+/** Keep a small editor/input area visible below the above-editor widget. */
+const MIN_EDITOR_LINES = 4;
+/**
+ * Derive the widget ceiling from the terminal height while retaining the
+ * historical MAX_WIDGET_LINES ceiling. Very short terminals may have no room
+ * for the widget; that is preferable to consuming the editor/input area.
+ */
+export function getWidgetLineBudget(rows) {
+    if (!Number.isFinite(rows))
         return MAX_WIDGET_LINES;
-    return Math.min(MAX_WIDGET_LINES, Math.max(0, Math.floor(terminalRows) - 4));
+    return Math.min(MAX_WIDGET_LINES, Math.max(0, Math.floor(rows) - MIN_EDITOR_LINES));
 }
 /** Braille spinner frames for animated running indicator. */
 export const SPINNER = ["⠋", "⠙", "⠹", "⠸", "⠼", "⠴", "⠦", "⠧", "⠇", "⠏"];
@@ -47,33 +52,6 @@ export function formatTokens(count) {
         return `${(count / 1_000).toFixed(1)}k token`;
     return `${count} token`;
 }
-/**
- * Format a cost as `~$0.0042`, or "" when there is nothing to show.
- *
- * The tilde is load-bearing: this is pi's own estimate from the model's listed
- * rates, not a billed figure, and the surfaces that print it sit next to token
- * counts that ARE exact.
- *
- * Nothing is printed for zero, which is also what a model with no pricing data
- * reports: `$0.00` beside a local model's tokens would claim its cost was
- * measured and found to be nothing, rather than never measured at all. For the
- * same reason a real cost too small for four decimals reads `<$0.0001` — it was
- * measured, and rounding it to `~$0.0000` would say the opposite.
- */
-export function formatCost(cost) {
-    if (!(cost > 0))
-        return ""; // also catches NaN
-    if (cost < 0.0001)
-        return "<$0.0001";
-    if (cost >= 1)
-        return `~$${cost.toFixed(2)}`;
-    // Under a dollar: cents at minimum, four decimals at most, nothing trailing.
-    // Most single runs land between a tenth of a cent and a dime, where rounding
-    // to cents would collapse a 4x difference in spend into the same figure.
-    const rounded = Number(cost.toFixed(4));
-    const decimals = (String(rounded).split(".")[1] ?? "").length;
-    return `~$${rounded.toFixed(Math.max(2, decimals))}`;
-}
 /**
  * Token count with optional context-fill % and compaction-count annotations.
  * Thresholds for percent: <70% dim, 70–85% warning, ≥85% error.
@@ -112,6 +90,18 @@ export function formatDuration(startedAt, completedAt) {
         return formatMs(completedAt - startedAt);
     return `${formatMs(Date.now() - startedAt)} (running)`;
 }
+/** Format an estimated model cost without implying billing precision. */
+export function formatCost(cost) {
+    if (!(cost > 0))
+        return "";
+    if (cost < 0.0001)
+        return "<$0.0001";
+    if (cost >= 1)
+        return `~$${cost.toFixed(2)}`;
+    const rounded = Number(cost.toFixed(4));
+    const decimals = (String(rounded).split(".")[1] ?? "").length;
+    return `~$${rounded.toFixed(Math.max(2, decimals))}`;
+}
 /** Get display name for any agent type (built-in or custom). */
 export function getDisplayName(type) {
     return getConfig(type).displayName;
@@ -121,13 +111,7 @@ export function getPromptModeLabel(type) {
     const config = getConfig(type);
     return config.promptMode === "append" ? "twin" : undefined;
 }
-/**
- * Mode label is not included — callers add it where they want it.
- *
- * Both model forms come back so each surface can pick by width; the
- * "(asked X)" annotation is applied here rather than by callers, so a value the
- * spawn did not honor cannot be rendered as though it had been (#182).
- */
+/** Mode label is not included — callers add it where they want it. */
 export function buildInvocationTags(invocation) {
     const tags = [];
     if (!invocation)
@@ -152,9 +136,9 @@ export function buildInvocationTags(invocation) {
         tags,
     };
 }
-/** Truncate text to a single line, max `len` chars. */
+/** Normalize and truncate text so it can never add physical widget rows. */
 function truncateLine(text, len = 60) {
-    const line = text.split("\n").find(l => l.trim())?.trim() ?? "";
+    const line = text.replace(/[\r\n]+/g, " ").replace(/\s+/g, " ").trim();
     if (line.length <= len)
         return line;
     return line.slice(0, len) + "…";
@@ -192,46 +176,38 @@ export class AgentWidget {
     uiCtx;
     widgetFrame = 0;
     widgetInterval;
-    /** Tracks how many turns each finished agent has survived. Key: agent ID, Value: turns since finished. */
-    finishedTurnAge = new Map();
-    /** How many extra turns errors/aborted agents linger (completed agents clear after 1 turn). */
-    static ERROR_LINGER_TURNS = 2;
+    inputUnsub;
+    /** Whether arrow keys currently navigate the agent roster. */
+    navigationActive = false;
+    /** Number of modal surfaces currently owning this listener's input suspension. */
+    navigationSuspensionDepth = 0;
+    /** Stable identity of the selected row, so roster changes do not jump selection. */
+    selectedAgentId;
+    /** Last logical roster index of the selected row, used when it disappears. */
+    selectedRosterIndex = 0;
+    /** First logical row currently represented by the bounded viewport. */
+    viewportStart = 0;
+    /** Cached records and roster. Rebuilt by update(), never by an arrow press. */
+    cachedAgents = [];
+    cachedRoster = [];
+    rosterCacheKey;
+    rosterCacheInitialized = false;
+    /** Terminal history capability is a potentially filesystem-backed lookup. */
+    historyOpenabilityCache = new Map();
     /** Whether the widget callback is currently registered with the TUI. */
     widgetRegistered = false;
     /** Cached TUI reference from widget factory callback, used for requestRender(). */
     tui;
     /** Last status bar text, used to avoid redundant setStatus calls. */
     lastStatusText;
-    lastSnapshot = "";
-    inputCleanup;
-    navigationSuspended = false;
-    navigationActive = false;
-    selectedAgentId;
-    viewportStart = 0;
-    selectedRosterIndex = 0;
+    /** Snapshot of the state used for the last widget registration/render request. */
+    lastRenderKey;
     openOptions;
     showCost;
     showModel;
     constructor(manager, agentActivity, 
-    /**
-     * Read live at render time. Selects which agents the widget shows — see
-     * `WidgetMode`. Defaults to `"all"` when a caller supplies no policy; the
-     * extension supplies one defaulting to `"background"`.
-     */
-    mode = () => "all", 
-    /**
-     * Read live at render time, like `mode`. Whether running agents show an
-     * estimated cost beside their token count. Defaults to off — the extension
-     * supplies the user's `showCost` setting.
-     */
-    /**
-     * Read live at render time, like `mode`. Whether running agents name the
-     * model driving them and the thinking level it is running at. Defaults to
-     * off — the extension supplies the user's `showModel` setting — because the
-     * row is already dense and the same pair is on the tool result and in the
-     * conversation viewer unconditionally.
-     */
-    showCostOrOptions = () => false, showModel = () => false) {
+    /** Read live at render time. Selects which agents the widget shows. */
+    mode = () => "all", showCostOrOptions = () => false, showModel = () => false) {
         this.manager = manager;
         this.agentActivity = agentActivity;
         this.mode = mode;
@@ -256,114 +232,240 @@ export class AgentWidget {
      *     only proven-foreground runs drop out — nothing else silently vanishes.
      *   - `all`: every agent.
      */
-    widgetAgents() {
-        const all = this.manager.listAgents().filter(isTopLevelAgent);
+    widgetAgentsFromManager() {
+        const all = this.manager.listAgents();
         switch (this.mode()) {
             case "off": return [];
             case "background": return all.filter(a => a.isBackground !== false);
             default: return all;
         }
     }
-    /** Temporarily stop global navigation from consuming input handled by another UI surface. */
+    /** Records currently visible to the widget; safe for render and input paths. */
+    widgetAgents() {
+        return this.rosterCacheInitialized ? this.cachedAgents : this.widgetAgentsFromManager();
+    }
+    canOpenHistory(record) {
+        return this.openOptions.canOpenHistory?.(record) ?? true;
+    }
+    historyOpenabilityKey(record) {
+        return JSON.stringify([
+            record.status,
+            record.completedAt ?? null,
+            record.transcriptPath ?? null,
+            record.session !== undefined,
+        ]);
+    }
+    /**
+     * Cache terminal history capability by the record state that can affect it.
+     * The supplied predicate may call `existsSync`, so it must not run from a
+     * timer refresh, render-key calculation, or arrow-navigation path more than
+     * once for the same terminal record state.
+     */
+    cachedCanOpenHistory(record) {
+        if (record.status === "running" || record.status === "queued")
+            return true;
+        const key = this.historyOpenabilityKey(record);
+        const cached = this.historyOpenabilityCache.get(record.id);
+        if (cached?.key === key)
+            return cached.value;
+        const value = this.canOpenHistory(record);
+        this.historyOpenabilityCache.set(record.id, { key, value });
+        return value;
+    }
+    /** Refresh the manager snapshot and openability cache at lifecycle/update time. */
+    refreshRoster() {
+        const agents = this.widgetAgentsFromManager();
+        const key = JSON.stringify({
+            mode: this.mode(),
+            agents: agents.map(record => [
+                record.id,
+                record.type,
+                record.description,
+                record.status,
+                record.completedAt ?? null,
+                record.startedAt,
+                record.toolUses,
+                record.error ?? null,
+                record.transcriptPath ?? null,
+                record.session !== undefined,
+                record.isBackground,
+                record.status !== "running" && record.status !== "queued"
+                    ? this.cachedCanOpenHistory(record)
+                    : undefined,
+            ]),
+        });
+        if (this.rosterCacheInitialized && key === this.rosterCacheKey)
+            return this.cachedRoster;
+        const finished = agents.filter(record => record.status !== "running" && record.status !== "queued"
+            && record.completedAt !== undefined
+            && this.cachedCanOpenHistory(record));
+        const running = agents.filter(record => record.status === "running");
+        const queued = agents.filter(record => record.status === "queued");
+        this.cachedAgents = agents;
+        this.cachedRoster = [...running, ...queued, ...finished];
+        this.rosterCacheKey = key;
+        this.rosterCacheInitialized = true;
+        return this.cachedRoster;
+    }
+    /** Suspend only this listener's keyboard ownership; never alter the widget. */
     suspendNavigation() {
-        this.navigationSuspended = true;
+        this.navigationSuspensionDepth++;
     }
-    /** Resume global navigation after another UI surface has closed. */
+    /** Restore one layer of this listener's keyboard ownership. */
     resumeNavigation() {
-        this.navigationSuspended = false;
+        if (this.navigationSuspensionDepth > 0)
+            this.navigationSuspensionDepth--;
     }
     /** Set the UI context (grabbed from first tool execution). */
     setUICtx(ctx) {
         if (ctx === this.uiCtx)
             return false;
-        this.inputCleanup?.();
-        this.inputCleanup = undefined;
+        // UICtx changed — the widget and input handler registered on the old
+        // context are gone. Re-register both on the next update().
+        this.inputUnsub?.();
+        this.inputUnsub = undefined;
         this.uiCtx = ctx;
         this.widgetRegistered = false;
         this.tui = undefined;
         this.lastStatusText = undefined;
-        this.navigationSuspended = false;
+        this.lastRenderKey = undefined;
+        this.navigationSuspensionDepth = 0;
         this.navigationActive = false;
         this.selectedAgentId = undefined;
+        this.selectedRosterIndex = 0;
         this.viewportStart = 0;
-        if (ctx.onTerminalInput)
-            this.inputCleanup = ctx.onTerminalInput(data => this.handleInput(data));
+        this.cachedAgents = [];
+        this.cachedRoster = [];
+        this.rosterCacheKey = undefined;
+        this.rosterCacheInitialized = false;
+        this.historyOpenabilityCache.clear();
+        // Print/RPC tests and lightweight embedders may provide only the widget
+        // surface; real interactive contexts always implement this hook.
+        if (typeof ctx.onTerminalInput === "function") {
+            this.inputUnsub = ctx.onTerminalInput(data => this.handleKey(data));
+        }
         return true;
     }
-    /**
-     * Called on each new turn (tool_execution_start).
-     * Ages finished agents and clears those that have lingered long enough.
-     */
+    /** Request a render on the currently registered TUI without touching input. */
+    requestUiRefresh(force = true) {
+        if (!this.tui || typeof this.tui.requestRender !== "function")
+            return false;
+        this.tui.requestRender(force);
+        return true;
+    }
+    /** Called on each new turn (tool_execution_start). */
     onTurnStart() {
-        // Age all finished agents
-        for (const [id, age] of this.finishedTurnAge) {
-            this.finishedTurnAge.set(id, age + 1);
-        }
-        // Trigger a widget refresh (will filter out expired agents)
         this.update();
     }
-    /** Ensure the widget update timer is running. */
+    /** Keep the spinner/elapsed-time timer alive only while a visible agent runs. */
     ensureTimer() {
+        if (!this.uiCtx)
+            return;
+        const roster = this.rosterCacheInitialized ? this.cachedRoster : this.refreshRoster();
+        if (!roster.some(a => a.status === "running"))
+            return;
         if (!this.widgetInterval) {
             this.widgetInterval = setInterval(() => this.update(true), 250);
         }
     }
-    /** Check if a finished agent should still be shown in the widget. */
-    shouldShowFinished(agentId, status, durable = false) {
-        // A persisted transcript is a durable history row, not a transient
-        // completion notification. Keep it navigable after the one-turn visual
-        // linger expires; runtime GC must not make `/agents` history disappear.
-        if (durable)
-            return true;
-        const age = this.finishedTurnAge.get(agentId) ?? 0;
-        const maxAge = ERROR_STATUSES.has(status) ? AgentWidget.ERROR_LINGER_TURNS : 1;
-        return age < maxAge;
-    }
-    /** Record an agent as finished (call when agent completes). */
-    markFinished(agentId) {
-        if (!this.finishedTurnAge.has(agentId)) {
-            this.finishedTurnAge.set(agentId, 0);
+    syncTimer(shouldRun) {
+        if (shouldRun) {
+            this.ensureTimer();
+        }
+        else if (this.widgetInterval) {
+            clearInterval(this.widgetInterval);
+            this.widgetInterval = undefined;
         }
     }
     /**
-     * Drop an agent's finished-age (call when a settled agent starts running
-     * again, i.e. a background resume). markFinished only seeds an age it has not
-     * seen before, so a resumed agent would otherwise keep the age from its
-     * previous run — already past the linger limit, hiding the new run's
-     * completion line entirely.
+     * Retained for the lifecycle call sites. Terminal visibility is determined
+     * by the manager record and the history capability, not a one-turn timer.
      */
-    markRunning(agentId) {
-        this.finishedTurnAge.delete(agentId);
-    }
-    navigationAgents() {
-        const agents = this.widgetAgents().filter(agent => agent.status === "running" || agent.status === "queued" || (agent.completedAt !== undefined && this.shouldShowFinished(agent.id, agent.status, agent.transcriptPath !== undefined)));
-        return agents.filter(agent => agent.status === "running" || agent.status === "queued" || (this.openOptions.canOpenHistory?.(agent) ?? true)).sort((a, b) => {
-            const activeA = a.status === "running" || a.status === "queued" ? 0 : 1;
-            const activeB = b.status === "running" || b.status === "queued" ? 0 : 1;
-            return activeA - activeB;
-        });
-    }
-    updateViewport(roster) {
-        const budget = getWidgetLineBudget(this.tui?.terminal?.rows ?? 24);
-        const visibleRows = Math.max(1, budget - 1);
-        if (this.selectedRosterIndex < this.viewportStart)
-            this.viewportStart = this.selectedRosterIndex;
-        if (this.selectedRosterIndex >= this.viewportStart + visibleRows)
-            this.viewportStart = this.selectedRosterIndex - visibleRows + 1;
-        this.viewportStart = Math.max(0, Math.min(this.viewportStart, Math.max(0, roster.length - visibleRows)));
+    markFinished(_agentId) { }
+    /** Retained for lifecycle call sites; terminal rows are record-driven now. */
+    markRunning(_agentId) { }
+    /**
+     * Records represented by selectable rows in the above-editor widget.
+     *
+     * Keep this order as the single source of truth for both rendering and key
+     * navigation. `listAgents()` is newest-first; active rows come first so the
+     * panel exposes currently useful work before terminal history.
+     */
+    roster() {
+        return this.rosterCacheInitialized ? this.cachedRoster : this.refreshRoster();
     }
+    /** True when pi's prompt editor owns the keyboard. */
     editorHasFocus() {
         const focused = this.tui?.focusedComponent;
         return focused == null || focused instanceof Editor;
     }
+    selectedIndexOf(records) {
+        if (!this.selectedAgentId)
+            return -1;
+        return records.findIndex(record => record.id === this.selectedAgentId);
+    }
     deactivate() {
+        // Keep the selected row and viewport so re-entering navigation can resume
+        // where the user left off. Lifecycle resets (context, dispose, empty
+        // roster) clear this state explicitly instead of treating every exit as a
+        // reset.
         this.navigationActive = false;
         this.requestUiRefresh();
     }
-    handleInput(data) {
+    /** Resume navigation from the retained row, or select the first row. */
+    activate(records) {
+        this.navigationActive = true;
+        const selectedIndex = this.selectedIndexOf(records);
+        if (selectedIndex >= 0) {
+            this.selectedRosterIndex = selectedIndex;
+        }
+        else {
+            // The previously selected row may have been cleaned up while navigation
+            // was inactive. Resume at its old logical position, clamped to the new
+            // roster, rather than jumping back to the first row.
+            const fallbackIndex = Math.max(0, Math.min(this.selectedRosterIndex, records.length - 1));
+            this.selectedAgentId = records[fallbackIndex].id;
+            this.selectedRosterIndex = fallbackIndex;
+            this.viewportStart = Math.min(this.viewportStart, Math.max(0, records.length - 1));
+        }
+        this.requestUiRefresh();
+    }
+    /** Move the selected row, activating only from an empty focused editor. */
+    moveSelection(direction) {
+        const records = this.roster();
+        const ui = this.uiCtx;
+        if (records.length === 0 || !ui)
+            return false;
+        if (!this.navigationActive) {
+            if (direction !== 1 || !this.editorHasFocus() || (ui.getEditorText?.() ?? "") !== "")
+                return false;
+            this.activate(records);
+            return true;
+        }
+        const currentIndex = Math.max(0, this.selectedIndexOf(records));
+        if (direction === -1 && currentIndex === 0) {
+            this.deactivate();
+            return true;
+        }
+        const nextIndex = Math.max(0, Math.min(records.length - 1, currentIndex + direction));
+        this.selectedAgentId = records[nextIndex].id;
+        this.selectedRosterIndex = nextIndex;
+        this.requestUiRefresh();
+        return true;
+    }
+    openSelected() {
+        const record = this.roster().find(candidate => candidate.id === this.selectedAgentId);
+        this.deactivate();
+        if (!record)
+            return;
+        const mode = record.status === "running" || record.status === "queued" ? "live" : "history";
+        void this.openOptions.onOpen?.(record, mode);
+    }
+    /** Handle terminal input before it reaches the focused prompt editor. */
+    handleKey(data) {
         if (!this.uiCtx || isKeyRelease(data))
             return undefined;
-        if (this.navigationSuspended)
+        if (this.navigationSuspensionDepth > 0)
             return undefined;
         if (!this.editorHasFocus()) {
             if (this.navigationActive)
@@ -371,61 +473,30 @@ export class AgentWidget {
             return undefined;
         }
         if (!this.navigationActive) {
-            const roster = this.navigationAgents();
-            if (!matchesKey(data, "down") || (this.uiCtx.getEditorText?.() ?? "") !== "" || roster.length === 0)
+            const records = this.roster();
+            if (!matchesKey(data, "down") || (this.uiCtx.getEditorText?.() ?? "") !== "" || records.length === 0) {
                 return undefined;
-            this.navigationActive = true;
-            const index = this.selectedAgentId ? roster.findIndex(agent => agent.id === this.selectedAgentId) : -1;
-            this.selectedRosterIndex = index >= 0 ? index : 0;
-            this.selectedAgentId = roster[this.selectedRosterIndex]?.id;
-            this.updateViewport(roster);
-            this.requestUiRefresh();
+            }
+            this.activate(records);
             return { consume: true };
         }
         if (matchesKey(data, "escape")) {
             this.deactivate();
             return { consume: true };
         }
-        if (matchesKey(data, "up") || matchesKey(data, "down")) {
-            const roster = this.navigationAgents();
-            if (roster.length === 0)
-                return { consume: true };
-            if (matchesKey(data, "up") && this.selectedRosterIndex === 0) {
-                this.deactivate();
-            }
-            else {
-                this.selectedRosterIndex = Math.max(0, Math.min(roster.length - 1, this.selectedRosterIndex + (matchesKey(data, "down") ? 1 : -1)));
-                this.selectedAgentId = roster[this.selectedRosterIndex]?.id;
-                this.updateViewport(roster);
-                this.requestUiRefresh();
-            }
-            return { consume: true };
-        }
+        if (matchesKey(data, "up"))
+            return this.moveSelection(-1) ? { consume: true } : undefined;
+        if (matchesKey(data, "down"))
+            return this.moveSelection(1) ? { consume: true } : undefined;
         if (matchesKey(data, Key.enter)) {
-            const record = this.navigationAgents()[this.selectedRosterIndex];
-            if (record) {
-                const terminal = record.status !== "running" && record.status !== "queued";
-                if (!terminal || (this.openOptions.canOpenHistory?.(record) ?? true)) {
-                    this.openOptions.onOpen?.(record, terminal ? "history" : "live");
-                }
-            }
+            this.openSelected();
             return { consume: true };
         }
-        // Only arrow keys navigate. Other keys flow to the editor and leave navigation mode.
+        // Only ↑/↓ navigate. Other keys, including j/k/←/→, flow to the editor
+        // and leave navigation mode.
         this.deactivate();
         return undefined;
     }
-    requestUiRefresh(force = true) {
-        if (!this.tui || typeof this.tui.requestRender !== "function")
-            return false;
-        this.tui.requestRender(force);
-        return true;
-    }
-    selectionBullet(agentId, theme) {
-        return this.navigationActive && this.selectedAgentId === agentId
-            ? theme.fg("accent", "●")
-            : theme.fg("dim", "○");
-    }
     /** Render a finished agent line. */
     renderFinishedLine(a, theme) {
         const modeLabel = getPromptModeLabel(a.type);
@@ -446,7 +517,7 @@ export class AgentWidget {
         }
         else if (a.status === "error") {
             icon = theme.fg("error", "✗");
-            const errMsg = a.error ? `: ${a.error.slice(0, 60)}` : "";
+            const errMsg = a.error ? `: ${truncateLine(a.error)}` : "";
             statusText = theme.fg("error", ` error${errMsg}`);
         }
         else {
@@ -460,15 +531,12 @@ export class AgentWidget {
             parts.push(formatTurns(activity.turnCount, activity.maxTurns));
         if (a.toolUses > 0)
             parts.push(`${a.toolUses} tool use${a.toolUses === 1 ? "" : "s"}`);
-        // From the record, not the activity tracker: that entry is deleted the
-        // moment an agent finishes, and "what did it cost" is a question asked
-        // about finished agents.
-        const costText = this.showCost() ? formatCost(getLifetimeCost(a.lifetimeUsage)) : "";
+        const costText = this.showCost() && a.lifetimeUsage ? formatCost(getLifetimeCost(a.lifetimeUsage)) : "";
         if (costText)
             parts.push(costText);
         parts.push(duration);
         const modeTag = modeLabel ? ` ${theme.fg("dim", `(${modeLabel})`)}` : "";
-        const description = a.description.replace(/[\r\n]+/g, " ");
+        const description = truncateLine(a.description);
         return `${icon} ${renderAgentName(a.type, theme, { fallbackColor: "dim" })}${modeTag}  ${theme.fg("dim", description)} ${theme.fg("dim", "·")} ${theme.fg("dim", parts.join(" · "))}${statusText}`;
     }
     /**
@@ -476,29 +544,31 @@ export class AgentWidget {
      * reading live state each time instead of capturing it in a closure.
      */
     renderWidget(tui, theme) {
-        const allAgents = this.widgetAgents();
-        const running = allAgents.filter(a => a.status === "running");
-        const queued = allAgents.filter(a => a.status === "queued");
-        const finished = allAgents.filter(a => a.status !== "running" && a.status !== "queued" && a.completedAt
-            && this.shouldShowFinished(a.id, a.status, a.transcriptPath !== undefined));
+        const roster = this.roster();
+        const running = roster.filter(a => a.status === "running");
+        const queued = roster.filter(a => a.status === "queued");
+        const finished = roster.filter(a => a.status !== "running" && a.status !== "queued");
+        const selectedId = this.navigationActive ? this.selectedAgentId : undefined;
         const hasActive = running.length > 0 || queued.length > 0;
         const hasFinished = finished.length > 0;
         // Nothing to show — return empty (widget will be unregistered by update())
         if (!hasActive && !hasFinished)
             return [];
         const w = tui.terminal.columns;
-        const budget = getWidgetLineBudget(tui.terminal.rows ?? 24);
-        if (budget === 0)
+        const maxLines = getWidgetLineBudget(tui.terminal.rows);
+        if (maxLines === 0)
             return [];
-        const truncate = (line) => truncateToWidth(line.replace(/[\r\n]+/g, " "), w);
+        const truncate = (line, width = w) => truncateToWidth(line, Math.max(0, width));
         const headingColor = hasActive ? "accent" : "dim";
         const headingIcon = hasActive ? "●" : "○";
         const frame = SPINNER[this.widgetFrame % SPINNER.length];
-        // Build sections separately for overflow-aware assembly.
-        // Each running agent = 2 lines (header + activity), finished = 1 line, queued = 1 line.
         const finishedLines = [];
         for (const a of finished) {
-            finishedLines.push(truncate(theme.fg("dim", "├─") + ` ${this.selectionBullet(a.id, theme)} ` + this.renderFinishedLine(a, theme)));
+            const marker = a.id === selectedId ? theme.fg("accent", "●") : theme.fg("dim", "○");
+            finishedLines.push({
+                record: a,
+                lines: [truncate(theme.fg("dim", "├─") + ` ${marker} ` + this.renderFinishedLine(a, theme))],
+            });
         }
         const runningLines = []; // each entry is [header, activity]
         for (const a of running) {
@@ -508,18 +578,13 @@ export class AgentWidget {
             const bg = this.agentActivity.get(a.id);
             const toolUses = bg?.toolUses ?? a.toolUses;
             // Spend comes from the record, never from the activity tracker: the record
-            // is the one that survives the agent finishing, and the one nested-tools
-            // folds a hidden child's spend into. Reading the tracker while an agent
-            // runs and the record once it stops made the figure jump at completion.
+            // survives the agent finishing and includes nested-child usage.
             const tokens = getLifetimeTotal(a.lifetimeUsage);
             const contextPercent = getSessionContextPercent(bg?.session);
             const tokenText = tokens > 0 ? formatSessionTokens(tokens, contextPercent, theme, a.compactionCount) : "";
             const costText = this.showCost() ? formatCost(getLifetimeCost(a.lifetimeUsage)) : "";
             const parts = [];
             if (this.showModel()) {
-                // Leading, and paired: a thinking level means nothing without the model
-                // it applies to. The tag is taken from buildInvocationTags rather than
-                // rebuilt so the "(asked X)" annotation survives.
                 const { modelName, tags } = buildInvocationTags(a.invocation);
                 if (modelName)
                     parts.push(modelName);
@@ -537,166 +602,294 @@ export class AgentWidget {
                 parts.push(costText);
             parts.push(elapsed);
             const statsText = parts.join(" · ");
-            const activity = (bg ? describeActivity(bg.activeTools, bg.responseText) : "thinking…").replace(/[\r\n]+/g, " ");
-            runningLines.push([
-                truncate(theme.fg("dim", "├─") + ` ${this.selectionBullet(a.id, theme)} ${theme.fg("accent", frame)} ${renderAgentName(a.type, theme, { bold: true })}${modeTag}  ${theme.fg("muted", a.description)} ${theme.fg("dim", "·")} ${fgPreservingNestedStyles(theme, "dim", statsText)}`),
-                truncate(theme.fg("dim", "│  ") + theme.fg("dim", `  ⎿  ${activity}`)),
-            ]);
-        }
-        const selectedQueued = this.navigationActive && this.selectedAgentId !== undefined && queued.some(agent => agent.id === this.selectedAgentId);
-        const queuedLine = queued.length > 0
-            ? truncate(theme.fg("dim", "├─") + ` ${selectedQueued ? theme.fg("accent", "●") : theme.fg("dim", "○")} ${theme.fg("muted", "◦")} ${theme.fg("dim", `${queued.length} queued`)}`)
-            : undefined;
-        // Assemble with overflow cap (heading + overflow indicator = 2 reserved lines).
-        const maxBody = Math.max(1, budget - 1); // heading takes 1 line
-        const totalBody = finishedLines.length + runningLines.length * 2 + (queuedLine ? 1 : 0);
-        const hint = "↑↓ select · enter view · esc back";
-        const lines = [truncate(theme.fg(headingColor, headingIcon) + " " + theme.fg(headingColor, "Agents") + "  " + theme.fg("dim", hint))];
+            const activity = bg ? describeActivity(bg.activeTools, bg.responseText) : "thinking…";
+            const marker = a.id === selectedId ? theme.fg("accent", "●") : theme.fg("dim", "○");
+            runningLines.push({
+                record: a,
+                lines: [
+                    truncate(theme.fg("dim", "├─") + ` ${marker} ${theme.fg("accent", frame)} ${renderAgentName(a.type, theme, { bold: true })}${modeTag}  ${theme.fg("muted", truncateLine(a.description))} ${theme.fg("dim", "·")} ${fgPreservingNestedStyles(theme, "dim", statsText)}`),
+                    truncate(theme.fg("dim", "│  ") + `   ${theme.fg("dim", `⎿  ${truncateLine(activity)}`)}`),
+                ],
+            });
+        }
+        const queuedLines = queued.map(a => {
+            const marker = a.id === selectedId ? theme.fg("accent", "●") : theme.fg("dim", "○");
+            return {
+                record: a,
+                lines: [truncate(theme.fg("dim", "├─") + ` ${marker} ${theme.fg("muted", "◦")} ${theme.fg("dim", `${getDisplayName(a.type)}  ${truncateLine(a.description)} · queued`)}`)],
+            };
+        });
+        // Keep the idle widget compact: a queue is one logical status line until
+        // navigation starts. While navigating, expand it back to the same per-agent
+        // roster used by key handling so every selected queued record can render.
+        const queuedSummary = queued.length > 0 && !this.navigationActive
+            ? [{
+                    record: queued[0],
+                    recordCount: queued.length,
+                    lines: [truncate(theme.fg("dim", "├─") + ` ${theme.fg("dim", "○")} ${theme.fg("muted", "◦")} ${theme.fg("dim", `${queued.length} queued`)}`)],
+                }]
+            : [];
+        // Assemble with a responsive cap (heading + overflow indicator = 2
+        // reserved lines when content exceeds the available body budget).
+        const maxBody = maxLines - 1; // heading takes 1 line
+        const rows = [
+            ...runningLines,
+            ...(this.navigationActive ? queuedLines : queuedSummary),
+            ...finishedLines,
+        ];
+        const totalBody = rows.reduce((total, row) => total + row.lines.length, 0);
+        const heading = "Agents  ↑↓ select · enter view · esc back";
+        const lines = [truncate(theme.fg(headingColor, headingIcon) + " " + theme.fg(headingColor, heading))];
+        if (maxLines === 1) {
+            // There is room only for the heading; do not consume the editor/input row.
+            return lines;
+        }
         if (totalBody <= maxBody) {
-            // Everything fits — add all lines and fix up connectors for the last item.
-            lines.push(...finishedLines);
-            for (const pair of runningLines)
-                lines.push(...pair);
-            if (queuedLine)
-                lines.push(queuedLine);
-            // Fix last connector: swap ├─ → └─ and │ → space for activity lines.
-            if (lines.length > 1) {
-                const last = lines.length - 1;
-                lines[last] = lines[last].replace("├─", "└─");
-                // If last item is a running agent activity line, fix indent of that line
-                // and fix the header line above it.
-                if (runningLines.length > 0 && !queuedLine) {
-                    // The last two lines are the last running agent's header + activity.
-                    if (last >= 2) {
-                        lines[last - 1] = lines[last - 1].replace("├─", "└─");
-                        lines[last] = lines[last].replace("│  ", "   ");
-                    }
+            this.viewportStart = 0;
+            for (const row of rows)
+                lines.push(...row.lines);
+            if (rows.length > 0) {
+                const lastRow = rows[rows.length - 1];
+                const lastStart = lines.length - lastRow.lines.length;
+                // Keep the selected row's bullet adjacent to the normal branch marker.
+                // This makes the active row unambiguous even when it is the last row.
+                const lastRowSelected = this.navigationActive && lastRow.record.id === selectedId;
+                if (!lastRowSelected)
+                    lines[lastStart] = lines[lastStart].replace("├─", "└─");
+                if (lastRow.lines.length === 2) {
+                    lines[lastStart + 1] = lines[lastStart + 1].replace("│  ", "   ");
                 }
             }
         }
-        else {
-            // Overflow — prioritize: running > queued > finished.
-            // Reserve 1 line for overflow indicator.
-            let budget = maxBody - 1;
-            let hiddenRunning = 0;
-            let hiddenFinished = 0;
-            // Reserve the queued line's row up front. It is a single summary of N
-            // waiting agents, so it cannot be folded into the "+N more" count (which
-            // is denominated in agents) without either under-reporting it as 1 or
-            // inflating the total with agents that were never getting their own rows.
-            // Reserving costs at most one running agent — which IS counted below —
-            // and makes the drop unreachable. It matters most exactly when it used to
-            // vanish: the pool is saturated and the queue is what the user needs to see.
-            const queuedReserve = queuedLine ? 1 : 0;
-            budget -= queuedReserve;
-            // 1. Running agents (2 lines each)
-            for (const pair of runningLines) {
-                if (budget >= 2) {
-                    lines.push(...pair);
-                    budget -= 2;
+        else if (!this.navigationActive && queuedSummary.length > 0) {
+            // In the idle view the queue is deliberately one summary row. Preserve
+            // that row before spending space on terminal history, and account for
+            // the real number of queued records if the summary itself cannot fit.
+            // Navigation expands the queue into per-record rows and uses the full
+            // contiguous viewport below.
+            const visibleRows = [];
+            const hiddenRows = [];
+            let bodyBudget = Math.max(0, maxBody - 1); // reserve the footer
+            let runningBudget = Math.max(0, bodyBudget - 1); // reserve queued summary
+            for (const row of runningLines) {
+                const height = row.lines.length;
+                if (height <= runningBudget) {
+                    visibleRows.push(row);
+                    runningBudget -= height;
+                    bodyBudget -= height;
                 }
                 else {
-                    hiddenRunning++;
+                    hiddenRows.push(row);
                 }
             }
-            // 2. Queued line (always fits — its row was reserved above)
-            if (queuedLine) {
-                budget += queuedReserve;
-                lines.push(queuedLine);
-                budget--;
+            const summaryRow = queuedSummary[0];
+            if (bodyBudget > 0) {
+                visibleRows.push(summaryRow);
+                bodyBudget -= summaryRow.lines.length;
+            }
+            else {
+                hiddenRows.push(summaryRow);
             }
-            // 3. Finished agents
-            for (const fl of finishedLines) {
-                if (budget >= 1) {
-                    lines.push(fl);
-                    budget--;
+            for (const row of finishedLines) {
+                const height = row.lines.length;
+                if (height <= bodyBudget) {
+                    visibleRows.push(row);
+                    bodyBudget -= height;
                 }
                 else {
-                    hiddenFinished++;
+                    hiddenRows.push(row);
+                }
+            }
+            for (const row of visibleRows)
+                lines.push(...row.lines);
+            if (visibleRows.length > 0) {
+                const lastRow = visibleRows[visibleRows.length - 1];
+                const lastStart = lines.length - lastRow.lines.length;
+                lines[lastStart] = lines[lastStart].replace("├─", "└─");
+                if (lastRow.lines.length === 2) {
+                    lines[lastStart + 1] = lines[lastStart + 1].replace("│  ", "   ");
                 }
             }
-            // Overflow summary
-            // On extremely short terminals the queued summary itself consumes the
-            // last useful row; keep its count in the compact footer. At the normal
-            // 12-line ceiling the summary is a single visible row and must not be
-            // counted as hidden agents.
-            const hiddenQueued = queuedLine && (tui.terminal.rows ?? 24) < 10 ? queued.length : 0;
-            const overflowParts = [];
-            if (hiddenRunning > 0)
-                overflowParts.push(`${hiddenRunning} running`);
-            if (hiddenFinished > 0)
-                overflowParts.push(`${hiddenFinished} finished`);
-            if (hiddenQueued > 0)
-                overflowParts.push(`${hiddenQueued} queued`);
-            const hiddenTotal = hiddenRunning + hiddenFinished + hiddenQueued;
-            const overflowText = overflowParts.join(", ");
-            lines.push(truncate(theme.fg("dim", "└─") + ` ${theme.fg("dim", `+${hiddenTotal} more (${overflowText})`)}`));
-        }
-        if (this.navigationActive) {
-            const roster = this.navigationAgents();
-            const selected = roster[this.selectedRosterIndex];
-            const hasSelected = selected ? lines.some(line => line.includes(selected.description.replace(/[\r\n]+/g, " "))) : true;
-            if (selected && !hasSelected) {
-                const selectedLine = truncate(theme.fg("dim", "└─") + ` ${this.selectionBullet(selected.id, theme)} ${selected.status === "queued" ? "◦" : "⠋"} ${theme.fg("muted", selected.description.replace(/[\r\n]+/g, " "))}`);
-                if (lines.length >= budget)
-                    lines[lines.length - 1] = selectedLine;
-                else
-                    lines.push(selectedLine);
+            const hiddenCount = (rows) => rows.reduce((total, row) => total + (row.recordCount ?? 1), 0);
+            const categoryCount = (status) => hiddenRows
+                .filter(row => row.record.status === status)
+                .reduce((total, row) => total + (row.recordCount ?? 1), 0);
+            const finishedCount = hiddenRows
+                .filter(row => row.record.status !== "running" && row.record.status !== "queued")
+                .reduce((total, row) => total + (row.recordCount ?? 1), 0);
+            const categoryCounts = [
+                ["running", categoryCount("running")],
+                ["queued", categoryCount("queued")],
+                ["finished", finishedCount],
+            ].filter(([, count]) => count > 0).map(([label, count]) => `${count} ${label}`);
+            const hidden = hiddenCount(hiddenRows);
+            lines.push(truncate(theme.fg("dim", "└─") + ` ${theme.fg("dim", `+${hidden} more (${categoryCounts.join(", ")})`)}`));
+        }
+        else {
+            // Reserve one line for a directional overflow summary. The viewport is
+            // a contiguous slice in roster order, so the same slice is navigable and
+            // renderable even when the selected row is currently hidden.
+            const baseContentBudget = Math.max(0, maxBody - 1);
+            const heightAt = (index) => rows[index]?.lines.length ?? 0;
+            const endFor = (start, budget) => {
+                let used = 0;
+                let end = start;
+                while (end < rows.length && used + heightAt(end) <= budget) {
+                    used += heightAt(end++);
+                }
+                return end;
+            };
+            let start = Math.max(0, Math.min(this.viewportStart, Math.max(0, rows.length - 1)));
+            const selectedIndex = this.navigationActive && selectedId
+                ? rows.findIndex(row => row.record.id === selectedId)
+                : -1;
+            if (selectedIndex >= 0) {
+                if (selectedIndex < start)
+                    start = selectedIndex;
+                if (selectedIndex >= endFor(start, baseContentBudget))
+                    start = selectedIndex;
+            }
+            // Keep both directional affordances in the viewport chrome: the top
+            // count sits below the heading and the bottom count sits below the
+            // bounded roster. Each consumes one row when that direction is hidden.
+            let showTopMore = start > 0;
+            let contentBudget = Math.max(0, maxBody - (showTopMore ? 2 : 1));
+            if (selectedIndex >= 0 && selectedIndex >= endFor(start, contentBudget)) {
+                start = selectedIndex;
+                showTopMore = start > 0;
+                contentBudget = Math.max(0, maxBody - (showTopMore ? 2 : 1));
             }
-            if (this.viewportStart > 0) {
-                if (lines.length >= budget)
-                    lines.splice(1, 1, truncate(theme.fg("dim", "↑ more")));
-                else
-                    lines.splice(1, 0, truncate(theme.fg("dim", "↑ more")));
+            this.viewportStart = start;
+            let used = 0;
+            let end = start;
+            const visibleBody = [];
+            while (end < rows.length && used + heightAt(end) <= contentBudget) {
+                visibleBody.push(...rows[end].lines);
+                used += heightAt(end);
+                end++;
             }
-            const railStart = this.viewportStart > 0 ? 2 : 1;
-            for (let index = railStart; index < lines.length; index++) {
-                const plain = lines[index] ?? "";
-                lines[index] = truncate(plain + (selected && plain.includes(selected.description.replace(/[\r\n]+/g, " ")) ? "┃" : "│"));
+            // A selected two-line row must remain addressable even if only one body
+            // line is available. Showing its header is preferable to hiding it.
+            if (end === start && rows[start] && contentBudget > 0) {
+                visibleBody.push(rows[start].lines[0]);
+                end = start + 1;
+            }
+            const countRows = (selectedRows) => selectedRows.reduce((total, row) => total + (row.recordCount ?? 1), 0);
+            const hiddenBefore = countRows(rows.slice(0, start));
+            const hiddenAfter = countRows(rows.slice(end));
+            const hidden = hiddenBefore + hiddenAfter;
+            const hiddenRows = rows.filter((_row, index) => index < start || index >= end);
+            const categoryCount = (status) => hiddenRows
+                .filter(row => row.record.status === status)
+                .reduce((total, row) => total + (row.recordCount ?? 1), 0);
+            const categoryCounts = [
+                ["running", categoryCount("running")],
+                ["queued", categoryCount("queued")],
+                ["finished", hiddenRows
+                        .filter(row => row.record.status !== "running" && row.record.status !== "queued")
+                        .reduce((total, row) => total + (row.recordCount ?? 1), 0)],
+            ].filter(([, count]) => count > 0).map(([label, count]) => `${count} ${label}`);
+            const direction = [
+                ...(hiddenBefore > 0 ? [`↑ ${hiddenBefore} more`] : []),
+                ...(hiddenAfter > 0 ? [`↓ ${hiddenAfter} more`] : []),
+            ].join(" · ");
+            const summary = `+${hidden} more (${direction}${categoryCounts.length > 0 ? `; ${categoryCounts.join(", ")}` : ""})`;
+            if (showTopMore)
+                lines.push(truncate(theme.fg("dim", `↑ ${hiddenBefore} more`)));
+            // Fill the bounded viewport so the scrollbar track has a stable height.
+            while (visibleBody.length < contentBudget)
+                visibleBody.push("");
+            const totalBody = rows.reduce((total, row) => total + row.lines.length, 0);
+            const startOffset = rows.slice(0, start).reduce((total, row) => total + row.lines.length, 0);
+            const trackHeight = Math.max(0, contentBudget);
+            const minThumbHeight = Math.min(2, trackHeight);
+            const thumbHeight = Math.max(minThumbHeight, Math.min(trackHeight, Math.round((trackHeight * trackHeight) / totalBody)));
+            const maxScrollTop = Math.max(0, totalBody - trackHeight);
+            const maxThumbTop = Math.max(0, trackHeight - thumbHeight);
+            const thumbOffset = maxScrollTop === 0
+                ? 0
+                : Math.round((startOffset / maxScrollTop) * maxThumbTop);
+            const contentWidth = Math.max(0, w - 1);
+            for (let index = 0; index < visibleBody.length; index++) {
+                const isThumb = index >= thumbOffset && index < thumbOffset + thumbHeight;
+                const scrollbar = isThumb
+                    ? theme.fg("scrollbarThumb", this.navigationActive ? "█" : "┃")
+                    : theme.fg("scrollbarTrack", "│");
+                const bodyLine = truncate(visibleBody[index] ?? "", contentWidth);
+                lines.push(bodyLine + " ".repeat(Math.max(0, contentWidth - visibleWidth(bodyLine))) + scrollbar);
+            }
+            // The directional count is outside the viewport, like ScrollView's
+            // surrounding chrome, and therefore does not consume scrollbar track.
+            if (hiddenAfter > 0) {
+                lines.push(truncate(theme.fg("dim", `↓ ${hiddenAfter} more · +${hidden} more (${categoryCounts.length > 0 ? categoryCounts.join(", ") : ""})`)));
+            }
+            else if (!showTopMore) {
+                lines.push(truncate(theme.fg("dim", "└─") + ` ${theme.fg("dim", summary)}`));
+            }
+            else {
+                // Keep a stable footer row when the selected viewport reaches the end.
+                lines.push(truncate(theme.fg("dim", "└─")));
             }
-            if (lines.length > budget)
-                lines.length = budget;
         }
         return lines;
     }
-    /** Force an immediate widget update. */
+    /** Build a render-relevant snapshot without capturing mutable records. */
+    renderKey(allAgents) {
+        const activities = allAgents.map(a => {
+            const activity = this.agentActivity.get(a.id);
+            return activity ? {
+                id: a.id,
+                activeTools: [...activity.activeTools.entries()],
+                toolUses: activity.toolUses,
+                responseText: activity.responseText,
+                turnCount: activity.turnCount,
+                maxTurns: activity.maxTurns,
+                lifetimeUsage: activity.lifetimeUsage,
+                contextPercent: getSessionContextPercent(activity.session),
+            } : undefined;
+        });
+        return JSON.stringify({
+            frame: this.widgetFrame,
+            agents: allAgents.map(a => ({
+                id: a.id,
+                type: a.type,
+                description: a.description,
+                status: a.status,
+                toolUses: a.toolUses,
+                startedAt: a.startedAt,
+                completedAt: a.completedAt,
+                error: a.error,
+                lifetimeUsage: a.lifetimeUsage,
+                compactionCount: a.compactionCount,
+                isBackground: a.isBackground,
+                hasSession: a.session !== undefined,
+                transcriptPath: a.transcriptPath,
+                openableHistory: this.cachedCanOpenHistory(a),
+            })),
+            activities,
+            navigationActive: this.navigationActive,
+            selectedAgentId: this.selectedAgentId,
+            viewportStart: this.viewportStart,
+        });
+    }
+    /** Force an immediate widget update. `advanceSpinner` is reserved for the timer. */
     update(advanceSpinner = false) {
         if (!this.uiCtx)
             return;
+        // Refresh the manager snapshot once per lifecycle update. Arrow handling
+        // only changes selection and requests a render; it never scans the manager.
+        const roster = this.refreshRoster();
         const allAgents = this.widgetAgents();
-        if (this.navigationActive || this.selectedAgentId) {
-            const roster = this.navigationAgents();
-            const selectedIndex = this.selectedAgentId ? roster.findIndex(agent => agent.id === this.selectedAgentId) : -1;
-            if (selectedIndex >= 0) {
-                this.selectedRosterIndex = selectedIndex;
-            }
-            else if (roster.length > 0 && this.selectedAgentId) {
-                this.selectedRosterIndex = Math.min(this.selectedRosterIndex, roster.length - 1);
-                this.selectedAgentId = roster[this.selectedRosterIndex]?.id;
-            }
-            else if (roster.length === 0) {
-                this.navigationActive = false;
-                this.selectedAgentId = undefined;
-                this.selectedRosterIndex = 0;
-                this.viewportStart = 0;
-            }
-            this.updateViewport(roster);
-        }
         // Lightweight existence checks — full categorization happens in renderWidget()
         let runningCount = 0;
         let queuedCount = 0;
-        let hasFinished = false;
-        for (const a of allAgents) {
-            if (a.status === "running") {
+        for (const a of roster) {
+            if (a.status === "running")
                 runningCount++;
-            }
-            else if (a.status === "queued") {
+            else if (a.status === "queued")
                 queuedCount++;
-            }
-            else if (a.completedAt && this.shouldShowFinished(a.id, a.status, a.transcriptPath !== undefined)) {
-                hasFinished = true;
-            }
         }
+        const hasFinished = roster.some(a => a.status !== "running" && a.status !== "queued");
         const hasActive = runningCount > 0 || queuedCount > 0;
         // Nothing to show — clear widget
         if (!hasActive && !hasFinished) {
@@ -709,15 +902,12 @@ export class AgentWidget {
                 this.uiCtx.setStatus("subagents", undefined);
                 this.lastStatusText = undefined;
             }
-            if (this.widgetInterval) {
-                clearInterval(this.widgetInterval);
-                this.widgetInterval = undefined;
-            }
-            // Clean up stale entries
-            for (const [id] of this.finishedTurnAge) {
-                if (!allAgents.some(a => a.id === id))
-                    this.finishedTurnAge.delete(id);
-            }
+            this.lastRenderKey = undefined;
+            this.navigationActive = false;
+            this.selectedAgentId = undefined;
+            this.selectedRosterIndex = 0;
+            this.viewportStart = 0;
+            this.syncTimer(false);
             return;
         }
         // Status bar — only call setStatus when the text actually changes
@@ -735,28 +925,33 @@ export class AgentWidget {
             this.uiCtx.setStatus("subagents", newStatusText);
             this.lastStatusText = newStatusText;
         }
+        // Spinner animation is driven only by the timer while a visible running
+        // agent exists. Event-driven updates refresh changed stats without making
+        // an otherwise idle widget advance.
         if (advanceSpinner && runningCount > 0)
             this.widgetFrame++;
-        const snapshot = allAgents.map(agent => {
-            const activity = this.agentActivity.get(agent.id);
-            return [
-                agent.id,
-                agent.status,
-                agent.description,
-                agent.toolUses,
-                agent.completedAt ?? "",
-                this.finishedTurnAge.get(agent.id) ?? "",
-                getLifetimeTotal(agent.lifetimeUsage),
-                activity?.toolUses ?? "",
-                activity?.responseText ?? "",
-                Math.floor((Date.now() - agent.startedAt) / 1000),
-            ].join("\u0000");
-        }).join("\u0001");
-        if (this.widgetRegistered && snapshot === this.lastSnapshot && !advanceSpinner)
-            return;
-        this.lastSnapshot = snapshot;
-        if (!advanceSpinner)
-            this.widgetFrame++;
+        this.syncTimer(runningCount > 0);
+        const selectedIndex = this.selectedIndexOf(roster);
+        if (selectedIndex >= 0) {
+            this.selectedRosterIndex = selectedIndex;
+        }
+        else if (this.navigationActive) {
+            if (roster.length === 0) {
+                this.navigationActive = false;
+                this.selectedAgentId = undefined;
+                this.selectedRosterIndex = 0;
+                this.viewportStart = 0;
+            }
+            else {
+                // Keep the selection near the row that disappeared. Both the saved
+                // logical index and viewport are clamped as the roster shrinks.
+                const fallbackIndex = Math.max(0, Math.min(this.selectedRosterIndex, roster.length - 1));
+                this.selectedAgentId = roster[fallbackIndex].id;
+                this.selectedRosterIndex = fallbackIndex;
+                this.viewportStart = Math.min(this.viewportStart, roster.length - 1);
+            }
+        }
+        const renderKey = this.renderKey(allAgents);
         // Register widget callback once; subsequent updates use requestRender()
         // which re-invokes render() without replacing the component (avoids layout thrashing).
         if (!this.widgetRegistered) {
@@ -768,23 +963,20 @@ export class AgentWidget {
                         // Theme changed — force re-registration so factory captures fresh theme.
                         this.widgetRegistered = false;
                         this.tui = undefined;
+                        this.lastRenderKey = undefined;
                     },
                 };
             }, { placement: "aboveEditor" });
             this.widgetRegistered = true;
         }
-        else {
-            // Widget already registered — just request a re-render of existing components.
+        else if (renderKey !== this.lastRenderKey) {
+            // Widget already registered — request a re-render only when visible state
+            // changed (or the spinner timer advanced).
             this.tui?.requestRender();
         }
+        this.lastRenderKey = renderKey;
     }
     dispose() {
-        this.inputCleanup?.();
-        this.inputCleanup = undefined;
-        this.navigationActive = false;
-        this.selectedAgentId = undefined;
-        this.viewportStart = 0;
-        this.selectedRosterIndex = 0;
         if (this.widgetInterval) {
             clearInterval(this.widgetInterval);
             this.widgetInterval = undefined;
@@ -793,10 +985,23 @@ export class AgentWidget {
             this.uiCtx.setWidget("agents", undefined);
             this.uiCtx.setStatus("subagents", undefined);
         }
+        this.inputUnsub?.();
+        this.inputUnsub = undefined;
         this.widgetRegistered = false;
         this.tui = undefined;
         this.lastStatusText = undefined;
-        this.lastSnapshot = "";
+        this.lastRenderKey = undefined;
+        this.navigationSuspensionDepth = 0;
+        this.navigationActive = false;
+        this.selectedAgentId = undefined;
+        this.selectedRosterIndex = 0;
+        this.viewportStart = 0;
+        this.cachedAgents = [];
+        this.cachedRoster = [];
+        this.rosterCacheKey = undefined;
+        this.rosterCacheInitialized = false;
+        this.historyOpenabilityCache.clear();
+        this.uiCtx = undefined;
     }
 }
 //# sourceMappingURL=agent-widget.js.map