@pellux/goodvibes-tui 2.0.20 → 2.1.0

package/src/{panels → views}/fleet-read-model.ts

@@ -1,11 +1,11 @@
 // ---------------------------------------------------------------------------
 // fleet-read-model.ts
 //
-// (Fleet tree panel), pure, testable read-model over the SDK's live
+// (Agents modal tree), pure, testable read-model over the SDK's live
 // process registry (@pellux/goodvibes-sdk/platform/runtime/fleet, landed on
-// SDK main as). No BasePanel/rendering dependency: this module owns the
+// SDK main as). No rendering dependency: this module owns the
 // flat ProcessNode[] -> tree transformation, the honest cost/token
-// aggregates, and the state->glyph/tone mapping that FleetPanel renders.
+// aggregates, and the state->glyph/tone mapping that the Agents modal renders.
 //
 // Design notes:
 //   - The brief predates the real SDK API and sketched a bespoke
@@ -55,7 +55,7 @@ export interface FleetTreeRow {
   readonly hasChildren: boolean;
 }
 
-/** Aggregate snapshot the fleet panel renders from. */
+/** Aggregate snapshot the Agents modal renders from. */
 export interface FleetSnapshot {
   readonly rows: readonly FleetTreeRow[];
   /** Sum of costUsd across nodes with a real (non-unpriced) reading; null when none do. */
@@ -68,7 +68,7 @@ export interface FleetSnapshot {
    * Ids of the nodes that are "blocked on me", waiting on a user approval or
    * user input right now (see isBlockedOnUserState), in display (row) order.
    * This is a DERIVED view over the same node states the rows already carry,
-   * never a second source of truth: the panel reads it to badge/jump, it is
+   * never a second source of truth: the Agents modal reads it to badge/jump, it is
    * not a store. Empty when nothing is waiting on the operator.
    */
   readonly blockedNodeIds: readonly string[];
@@ -85,9 +85,9 @@ export type FleetStateTone = 'active' | 'success' | 'failure' | 'warn' | 'muted'
 // contract's STATE_GLYPHS (@pellux/goodvibes-sdk/platform/presentation), that
 // one is a 4-state semantic alias (good/warn/bad/info) shared by the TUI and
 // agent renderers; this one is a 12-state ProcessNode taxonomy specific to the
-// Fleet tree panel (thinking/executing-tool/awaiting-approval/streaming/
+// Agents modal tree (thinking/executing-tool/awaiting-approval/streaming/
 // stalled/retrying/done/failed/killed/interrupted/idle/queued/paused), and no
-// other surface (checked: the agent has no fleet-panel renderer) consumes it
+// other surface (checked: the agent has no Agents modal renderer) consumes it
 // today. Per the presentation-parity brief, a table only moves to the SDK
 // contract when 2+ surfaces need it, this one stays TUI-renderer-local.
 const STATE_GLYPHS: Record<ProcessState, string> = {
@@ -176,7 +176,7 @@ const KIND_TAGS: Record<ProcessKind, string> = {
   // that goodvibes did NOT spawn, observed read-only from OS signals. The row's
   // first job is visibility ("it's in claude or codex"); it is never counted in
   // any our-fleet count, never carries a stop affordance, and steer is drill-in
-  // only. See ObservedProcess handling in fleet-panel-format.ts.
+  // only. See ObservedProcess handling in fleet-format.ts.
   'observed-external': 'observed',
 };
 
@@ -525,7 +525,7 @@ export function buildFleetSnapshot(nodes: readonly ProcessNode[], capturedAt: nu
     if (isFleetRunningLeaf(node)) runningCount++;
   }
 
-  // Blocked ids in display (row) order so the panel's jump key steps through
+  // Blocked ids in display (row) order so the Agents modal's jump key steps through
   // them top-to-bottom exactly as they appear on screen.
   const blockedNodeIds = rows
     .filter((row) => isBlockedOnUserNode(row.node))

package/src/{panels/fleet-tabs.ts → views/fleet-session-tabs.ts}

@@ -1,13 +1,13 @@
 // ---------------------------------------------------------------------------
-// fleet-tabs.ts
+// fleet-session-tabs.ts
 //
-// Pure, testable tab-state model for FleetPanel's
-// session tabs. No BasePanel/rendering dependency, mirroring the
+// Pure, testable tab-state model for the Agents modal's
+// session tabs. No rendering dependency, mirroring the
 // fleet-read-model.ts convention: this module owns attach/detach/switch as
 // pure state transitions over a small immutable FleetTabsState, so the tab
-// lifecycle is unit-testable without a live registry or a rendered panel.
+// lifecycle is unit-testable without a live registry or a rendered view.
 //
-// Tab-index convention: index 0 is always the fleet TREE (the panel's root
+// Tab-index convention: index 0 is always the fleet TREE (the Agents modal's root
 // view, never stored in `tabs`); `activeTabIndex` 1..N addresses
 // `tabs[0..N-1]`. This mirrors the brief's "tree is the root/home tab"
 // framing while keeping the array itself simple (only attached tabs).
@@ -16,7 +16,7 @@
 // attaching to (a transcript for an agent; a live member summary for a
 // chain, see fleet-transcript.ts). workflow/trigger/schedule/watcher/
 // background-process/wrfc-subtask nodes have no transcript and are never
-// attachable (FleetPanel shows a status message instead, matching the
+// attachable (the Agents modal shows a status message instead, matching the
 // existing i/K "not supported" convention).
 // ---------------------------------------------------------------------------
 
@@ -39,9 +39,9 @@ export function isAttachableFleetKind(kind: ProcessKind): kind is FleetAttachabl
  * event matching this badge's `messageId`). `dropped` is a TUI-side
  * inference: the SDK emits no "expired"/"cancelled" signal for a queued
  * steer, so if the target node goes terminal (done/failed/killed/
- * interrupted) while the badge is still `queued`, FleetPanel resolves it to
+ * interrupted) while the badge is still `queued`, the Agents modal resolves it to
  * `dropped` itself rather than leaving the badge hanging forever (see
- * fleet-panel.ts reconcileSteerBadges, cross-WO note: the SDK engineer
+ * src/input/agents-modal.ts reconcileSteerBadges, cross-WO note: the SDK engineer
  * confirmed no dropped signal exists).
  */
 export type SteerBadgeStatus = 'queued' | 'consumed' | 'dropped';
@@ -52,7 +52,7 @@ export interface SteerBadge {
   readonly status: SteerBadgeStatus;
   /** Present for 'dropped', a one-line honest explanation shown in the tab. */
   readonly note?: string;
-  /** epoch ms when status left 'queued' (consumed or dropped), drives FleetPanel's linger-then-clear tick. */
+  /** epoch ms when status left 'queued' (consumed or dropped), drives the Agents modal's linger-then-clear tick. */
   readonly resolvedAt?: number;
   /**
    * epoch ms when status entered 'queued' (set once, at submit time). Drives
@@ -119,10 +119,10 @@ export interface FleetTab {
   ledgerLoadStarted: boolean;
   /**
    * The one-line steer composer's in-progress text, or `null`
-   * when not composing. Mirrors git-panel.ts's `commitMessage` mutable-slot
-   * convention (FleetPanel.isCapturingTextBurst() gates on this being
+   * when not composing. Mirrors the git view's `commitMessage` mutable-slot
+   * convention (the Agents modal's text-capture check gates on this being
    * non-null so a burst/paste lands here char-by-char, never as tree/tab
-   * hotkeys, see FleetPanel.handleSteerInput).
+   * hotkeys, see the Agents modal's steer input handler).
    */
   steerDraft: string | null;
   /** This tab's most recent steer message's lifecycle, or null. */
@@ -173,7 +173,7 @@ export function activeFleetTab(state: FleetTabsState): FleetTab | null {
  * Attach a node as a tab (Enter on an attachable tree row). Re-focuses an
  * already-open tab for the same node instead of creating a duplicate.
  * Returns `state` unchanged if `node.kind` is not attachable, callers
- * (FleetPanel) are expected to guard with `isAttachableFleetKind` first and
+ * (the Agents modal) are expected to guard with `isAttachableFleetKind` first and
  * report a status message rather than silently doing nothing; this
  * function's own no-op fallback is defense in depth.
  */

package/src/{panels → views}/fleet-spawn.ts

@@ -1,7 +1,7 @@
 // ---------------------------------------------------------------------------
 // fleet-spawn.ts
 //
-// The Fleet panel's ACP spawn affordance: 'n' on the fleet surface lists the
+// The Agents modal's ACP spawn affordance: 'n' lists the
 // third-party coding agents the daemon discovered (acp.agents.list, read-only,
 // quiet when none), you pick one, then pick a working directory from the known
 // candidates (the registered workspaces + the current dir, no free-text path
@@ -13,24 +13,16 @@
 // never a hung row: the daemon bounds the handshake and returns a 'failed'
 // record, and this controller reports its three honest fields.
 //
-// The controller owns the fleet view + input while a pick is in flight (mirrors
-// FleetActs' pick mode), so fleet-panel.ts stays a thin delegator under the
-// architecture line cap. The gateway is injectable so the flow round-trips
+// The controller owns the Agents modal's input while a pick is in flight
+// (mirrors FleetActs' pick mode); the modal draws spawnView(). The gateway is
+// injectable so the flow round-trips
 // against a mocked daemon in tests; the live builder is createAcpSpawnGateway.
 // ---------------------------------------------------------------------------
 
 import type { ConfigManager } from '@pellux/goodvibes-sdk/platform/config';
 import type { OperatorMethodOutput } from '@pellux/goodvibes-sdk';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
-import type { Line } from '@pellux/goodvibes-sdk/platform/types';
 import { resolveOperatorRpc } from '../input/commands/operator-rpc.ts';
-import {
-  buildKeyboardHints,
-  buildPanelWorkspace,
-  buildPanelLine,
-  DEFAULT_PANEL_PALETTE,
-  type PanelPalette,
-} from './polish.ts';
 
 /** One discovered third-party ACP agent (acp.agents.list). */
 export type AcpDiscoveredAgent = OperatorMethodOutput<'acp.agents.list'>['agents'][number];
@@ -111,7 +103,15 @@ interface DirStep {
 }
 type SpawnMode = AgentStep | DirStep | null;
 
-const C = DEFAULT_PANEL_PALETTE;
+/** The spawn picker, as data. */
+export interface FleetSpawnView {
+  readonly step: 'agent' | 'dir';
+  readonly title: string;
+  readonly intro: string;
+  readonly options: ReadonlyArray<{ readonly label: string; readonly detail: string; readonly selected: boolean }>;
+  /** A create round-trip is in flight. */
+  readonly creating: boolean;
+}
 
 export class FleetSpawn {
   private mode: SpawnMode = null;
@@ -218,44 +218,28 @@ export class FleetSpawn {
     }
   }
 
-  /** The spawn picker view (replaces the tree while a pick is in flight). */
-  public renderSpawnMode(width: number, height: number, palette: PanelPalette = C): Line[] {
-    const P = palette;
-    if (!this.mode) return [];
-    const lines: Line[] = [];
-    if (this.creating) {
-      lines.push(buildPanelLine(width, [[' Hosting the agent…', P.dim]]));
-      return buildPanelWorkspace(width, height, { title: 'Fleet: Host an agent', sections: [{ lines }], footerLines: [], palette: P });
-    }
+  /** The spawn picker as data the Agents modal draws, or null when it is not open. */
+  public spawnView(): FleetSpawnView | null {
+    if (!this.mode) return null;
+    if (this.creating) return { step: this.mode.step, title: 'Hosting the agent…', intro: '', options: [], creating: true };
     if (this.mode.step === 'agent') {
-      lines.push(buildPanelLine(width, [[' Host a third-party coding agent; pick one, then a directory.', P.dim]]));
-      lines.push(buildPanelLine(width, [['', P.dim]]));
-      this.mode.agents.forEach((agent, i) => {
-        const selected = i === (this.mode as AgentStep).index;
-        lines.push(buildPanelLine(width, [
-          [selected ? ' ▸ ' : '   ', selected ? P.info : P.dim],
-          [agent.title, selected ? P.value : P.dim],
-          [`, ${agent.binaryPath}`, P.dim],
-        ]));
-      });
-    } else {
-      lines.push(buildPanelLine(width, [[` Directory for ${this.mode.agent.title}: a known dir, no path to type.`, P.dim]]));
-      lines.push(buildPanelLine(width, [['', P.dim]]));
-      this.mode.candidates.forEach((cand, i) => {
-        const selected = i === (this.mode as DirStep).index;
-        lines.push(buildPanelLine(width, [
-          [selected ? ' ▸ ' : '   ', selected ? P.info : P.dim],
-          [cand.label, selected ? P.value : P.dim],
-          [`, ${cand.path}`, P.dim],
-        ]));
-      });
+      const index = this.mode.index;
+      return {
+        step: 'agent',
+        title: 'Host an agent',
+        intro: 'Host a third-party coding agent here: pick one, then a directory.',
+        options: this.mode.agents.map((agent, i) => ({ label: agent.title, detail: agent.binaryPath, selected: i === index })),
+        creating: false,
+      };
     }
-    const footerLines = [buildKeyboardHints(width, [
-      { keys: '↑↓', label: 'choose' },
-      { keys: 'Enter', label: this.mode.step === 'agent' ? 'pick agent' : 'host here' },
-      { keys: 'Esc', label: 'cancel' },
-    ], P)];
-    return buildPanelWorkspace(width, height, { title: 'Fleet: Host an agent', sections: [{ lines }], footerLines, palette: P });
+    const index = this.mode.index;
+    return {
+      step: 'dir',
+      title: this.mode.agent.title,
+      intro: `Where ${this.mode.agent.title} runs: a known directory, no path to type.`,
+      options: this.mode.candidates.map((cand, i) => ({ label: cand.label, detail: cand.path, selected: i === index })),
+      creating: false,
+    };
   }
 
   private requireGateway(): AcpSpawnGateway | null {

package/src/{panels → views}/fleet-steer.ts

@@ -2,17 +2,17 @@
 // fleet-steer.ts
 //
 // Steer-badge rendering helpers and the "dropped inference"
-// reconciliation pass, split out of fleet-panel.ts to keep that file under
+// reconciliation pass, split out of src/input/agents-modal.ts to keep that file under
 // the architecture line cap (see check-architecture.ts's 800-line gate).
-// Pure functions only; FleetPanel still owns the mutable FleetTab.steerBadge
-// state itself (see fleet-tabs.ts's SteerBadge doc) and calls into this
+// Pure functions only; the Agents modal still owns the mutable FleetTab.steerBadge
+// state itself (see fleet-session-tabs.ts's SteerBadge doc) and calls into this
 // module rather than duplicating the logic inline.
 // ---------------------------------------------------------------------------
 
 import { STEER_TTL_MS, type ProcessNode } from '@pellux/goodvibes-sdk/platform/runtime/fleet';
 import { isTerminalProcessState } from './fleet-read-model.ts';
-import type { FleetTab, SteerBadge, SteerBadgeStatus } from './fleet-tabs.ts';
-import { buildPanelLine, DEFAULT_PANEL_PALETTE, type PanelPalette } from './polish.ts';
+import type { FleetTab, SteerBadge, SteerBadgeStatus } from './fleet-session-tabs.ts';
+import { buildViewLine, DEFAULT_VIEW_PALETTE, type ViewPalette } from './polish.ts';
 import type { Line } from '@pellux/goodvibes-sdk/platform/types';
 
 /** Linger before an auto-resolved (consumed/dropped) steer badge is cleared from the tab. */
@@ -26,11 +26,11 @@ export function steerBadgeGlyph(status: SteerBadgeStatus): string {
   }
 }
 
-export function steerBadgeTone(status: SteerBadgeStatus, palette: PanelPalette): string {
+export function steerBadgeTone(status: SteerBadgeStatus, palette: ViewPalette): string {
   switch (status) {
-    case 'queued': return palette.warn ?? DEFAULT_PANEL_PALETTE.warn;
-    case 'consumed': return palette.good ?? DEFAULT_PANEL_PALETTE.good;
-    case 'dropped': return palette.bad ?? DEFAULT_PANEL_PALETTE.bad;
+    case 'queued': return palette.warn ?? DEFAULT_VIEW_PALETTE.warn;
+    case 'consumed': return palette.good ?? DEFAULT_VIEW_PALETTE.good;
+    case 'dropped': return palette.bad ?? DEFAULT_VIEW_PALETTE.bad;
   }
 }
 
@@ -50,7 +50,7 @@ export function steerRefusalMessage(reason: string, siblingLabels: readonly stri
 }
 
 /** One-line honest status for a tab's steer badge; the queued line names the target and points at the ⧗ delivery badge (WO item 4). */
-export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette: PanelPalette, targetLabel?: string): Line {
+export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette: ViewPalette, targetLabel?: string): Line {
   const glyph = steerBadgeGlyph(badge.status);
   const tone = steerBadgeTone(badge.status, palette);
   const forTarget = targetLabel ? ` for ${targetLabel}` : '';
@@ -59,7 +59,7 @@ export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette:
     : badge.status === 'consumed'
       ? 'steer consumed'
       : `steer dropped: ${badge.note ?? 'the target ended before delivery'}`;
-  return buildPanelLine(width, [
+  return buildViewLine(width, [
     [' ', palette.dim],
     [glyph, tone],
     [` ${label}`, palette.dim],
@@ -75,7 +75,7 @@ export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette:
  * its short linger so a tab doesn't accumulate stale indicators.
  *
  * Mutates `tab.steerBadge` in place (same mutable-slot convention as
- * `FleetTab.ledgerEntries`, see fleet-panel.ts's ensureLedgerLoaded).
+ * `FleetTab.ledgerEntries`, see src/input/agents-modal.ts's ensureLedgerLoaded).
  * Returns true when anything changed, so the caller knows whether to mark
  * itself dirty.
  */

package/src/{panels → views}/fleet-stop.ts

@@ -1,9 +1,9 @@
 // ---------------------------------------------------------------------------
 // fleet-stop.ts
 //
-// d1, the "stopping…" write-window overlay, split out of fleet-panel.ts
+// d1, the "stopping…" write-window overlay, split out of src/input/agents-modal.ts
 // to keep that file under the architecture 800-line gate (same rationale as
-// fleet-steer.ts). FleetPanel owns nothing here beyond a FleetStopTracker
+// fleet-steer.ts). The Agents modal owns nothing here beyond a FleetStopTracker
 // instance; all the state->display logic lives in these pure helpers.
 // ---------------------------------------------------------------------------
 
@@ -76,7 +76,7 @@ export function fleetStateDisplay(state: ProcessState, stopping: boolean, blocke
   return { glyph: fleetStateGlyph(state), label: state, tone: fleetStateTone(state) };
 }
 
-/** The narrow action + side-effect surface `toggleFleetPause` needs from FleetPanel. */
+/** The narrow action + side-effect surface `toggleFleetPause` needs from the Agents modal. */
 export interface FleetPauseDeps {
   readonly interrupt: (id: string) => boolean;
   readonly resume: (id: string) => boolean;
@@ -196,7 +196,7 @@ export function countDescendantStats(rows: readonly FleetTreeRow[], nodeId: stri
 /**
  * Build the confirm-overlay arm options for a cascade Kill of one node, the
  * label carries the descendant count so the operator sees what a confirm takes
- * down. Extracted from FleetPanel to keep that file under the 800-line cap.
+ * down. Extracted from the Agents modal to keep that file under the 800-line cap.
  */
 export function fleetKillConfirmArgs(
   node: ProcessNode,

package/src/{panels → views}/fleet-transcript.ts

@@ -1,7 +1,7 @@
 // ---------------------------------------------------------------------------
 // fleet-transcript.ts
 //
-// Renders the content of an attached FleetPanel
+// Renders the content of an attached Agents modal
 // session tab. Three sources, chosen by what the SDK can actually provide
 // (see the design brief's "central reality check", a full-fidelity live
 // transcript needs a ConversationMessageSnapshot[] history source, which
@@ -19,7 +19,7 @@
 //   - 'unavailable'   , a terminal agent whose snapshot came back empty
 //     (evicted past the retention bound, or an agent kind, e.g. the WRFC
 //     owner, that never registered a live conversation source at all).
-//     FleetPanel degrades to the on-disk event ledger (renderFleetLedgerFallback)
+//     the Agents modal degrades to the on-disk event ledger (renderFleetLedgerFallback)
 //     for this case; this module never fabricates transcript content.
 //
 // A 'wrfc-chain' tab has no single conversation of its own (it coordinates
@@ -33,7 +33,7 @@ import { createEmptyLine, type Line } from '@pellux/goodvibes-sdk/platform/types
 import type { MessageLineCache } from '../core/conversation-line-cache.ts';
 import type { ConversationRenderContext } from '../core/conversation-rendering.ts';
 import { renderConversationEventLine, renderConversationNotice } from '../renderer/conversation-surface.ts';
-import { buildPanelLine, DEFAULT_PANEL_PALETTE, type PanelPalette } from './polish.ts';
+import { buildViewLine, DEFAULT_VIEW_PALETTE, type ViewPalette } from './polish.ts';
 import { formatElapsed } from '../utils/format-elapsed.ts';
 import { truncateDisplay } from '../utils/terminal-width.ts';
 import {
@@ -44,16 +44,16 @@ import {
   type FleetTreeRow,
 } from './fleet-read-model.ts';
 
-/** Duplicated from fleet-panel.ts's private helper (same tiny mapping) rather than
- *  cross-importing, fleet-panel.ts imports THIS module to render tab content, so
- *  importing back would form a cycle. See fleet-panel.ts's own toneColor for the
+/** Duplicated from src/input/agents-modal.ts's private helper (same tiny mapping) rather than
+ *  cross-importing, src/input/agents-modal.ts imports THIS module to render tab content, so
+ *  importing back would form a cycle. See src/input/agents-modal.ts's own toneColor for the
  *  canonical copy; keep both in sync if the tone set ever changes. */
-function toneColor(tone: FleetStateTone, palette: PanelPalette): string {
+function toneColor(tone: FleetStateTone, palette: ViewPalette): string {
   switch (tone) {
-    case 'active': return palette.info ?? DEFAULT_PANEL_PALETTE.info;
-    case 'success': return palette.good ?? DEFAULT_PANEL_PALETTE.good;
-    case 'failure': return palette.bad ?? DEFAULT_PANEL_PALETTE.bad;
-    case 'warn': return palette.warn ?? DEFAULT_PANEL_PALETTE.warn;
+    case 'active': return palette.info ?? DEFAULT_VIEW_PALETTE.info;
+    case 'success': return palette.good ?? DEFAULT_VIEW_PALETTE.good;
+    case 'failure': return palette.bad ?? DEFAULT_VIEW_PALETTE.bad;
+    case 'warn': return palette.warn ?? DEFAULT_VIEW_PALETTE.warn;
     case 'muted': return palette.dim;
   }
 }
@@ -83,11 +83,10 @@ export interface FleetTranscriptRender {
 const FROZEN_TRANSCRIPT_NOTICE = 'Read-only: this agent finished; not a live view.';
 
 function renderFrozenTranscriptNotice(width: number): Line[] {
-  const palette = DEFAULT_PANEL_PALETTE;
+  const palette = DEFAULT_VIEW_PALETTE;
   return renderConversationNotice(FROZEN_TRANSCRIPT_NOTICE, width, {
-    accent: palette.warn ?? DEFAULT_PANEL_PALETTE.warn,
+    accent: palette.warn ?? DEFAULT_VIEW_PALETTE.warn,
     text: palette.dim,
-    dim: true,
   });
 }
 
@@ -115,7 +114,7 @@ export function renderFleetAgentTranscript(
 ): FleetTranscriptRender {
   if (snapshot.length === 0) {
     if (isTerminal) return { mode: 'unavailable', lines: [] };
-    return { mode: 'live', lines: [buildPanelLine(width, [[' (no messages yet)', DEFAULT_PANEL_PALETTE.dim]])] };
+    return { mode: 'live', lines: [buildViewLine(width, [[' (no messages yet)', DEFAULT_VIEW_PALETTE.dim]])] };
   }
 
   const historyLines: Line[] = [];
@@ -174,17 +173,17 @@ export function renderFleetChainSummary(
   width: number,
   chainDoneOrAbsent: boolean,
 ): Line[] {
-  const C = DEFAULT_PANEL_PALETTE;
+  const C = DEFAULT_VIEW_PALETTE;
   if (memberRows.length === 0) {
     const message = chainDoneOrAbsent
       ? ' chain completed: members no longer tracked'
       : ' (no member agents yet)';
-    return [buildPanelLine(width, [[message, C.dim]])];
+    return [buildViewLine(width, [[message, C.dim]])];
   }
   return memberRows.map((row) => {
     const node = row.node;
     const color = toneColor(fleetStateTone(node.state), C);
-    return buildPanelLine(width, [
+    return buildViewLine(width, [
       [' ', C.dim],
       [fleetStateGlyph(node.state), color],
       [` ${fleetKindTag(node.kind)} `, C.dim],
@@ -224,7 +223,7 @@ function formatLedgerDuration(ms: unknown): string {
   return typeof ms === 'number' ? ` (${formatElapsed(ms)})` : '';
 }
 
-function renderLedgerEntry(width: number, entry: Record<string, unknown>, palette: PanelPalette): Line {
+function renderLedgerEntry(width: number, entry: Record<string, unknown>, palette: ViewPalette): Line {
   const type = typeof entry['type'] === 'string' ? entry['type'] : 'unknown';
   switch (type) {
     case 'meta': {
@@ -256,7 +255,7 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
     case 'tool_execution': {
       const toolName = typeof entry['toolName'] === 'string' ? entry['toolName'] : 'tool';
       const success = entry['success'] !== false;
-      const bad = palette.bad ?? DEFAULT_PANEL_PALETTE.bad;
+      const bad = palette.bad ?? DEFAULT_VIEW_PALETTE.bad;
       // the writer already truncates this to 500 chars (session.ts's
       // resultPreview field), passed through verbatim (collapsed to one
       // line), never re-summarized or fabricated, then tail-truncated again
@@ -276,9 +275,9 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
     }
     case 'session_end': {
       const status = typeof entry['status'] === 'string' ? entry['status'] : 'ended';
-      const good = palette.good ?? DEFAULT_PANEL_PALETTE.good;
-      const bad = palette.bad ?? DEFAULT_PANEL_PALETTE.bad;
-      const warn = palette.warn ?? DEFAULT_PANEL_PALETTE.warn;
+      const good = palette.good ?? DEFAULT_VIEW_PALETTE.good;
+      const bad = palette.bad ?? DEFAULT_VIEW_PALETTE.bad;
+      const warn = palette.warn ?? DEFAULT_VIEW_PALETTE.warn;
       const tone = status === 'completed' ? good : status === 'failed' ? bad : warn;
       return renderConversationEventLine(width, { marker: '■', markerFg: tone, label: 'session', labelFg: tone }, [
         { text: `${status}${formatLedgerDuration(entry['durationMs'])}`, fg: tone },
@@ -334,14 +333,14 @@ export function renderFleetLedgerFallback(
   width: number,
   height: number,
 ): Line[] {
-  const palette = DEFAULT_PANEL_PALETTE;
+  const palette = DEFAULT_VIEW_PALETTE;
   const notice = renderConversationNotice(
     'Read-only. Full transcript unavailable for this agent; showing its activity log instead.',
     width,
-    { accent: palette.warn, text: palette.dim, dim: true },
+    { accent: palette.warn, text: palette.dim },
   );
   const body = entries.length === 0
-    ? [buildPanelLine(width, [[' (no activity recorded)', palette.dim]])]
+    ? [buildViewLine(width, [[' (no activity recorded)', palette.dim]])]
     : entries.map((entry) => renderLedgerEntry(width, entry, palette));
   const budgetHeight = Math.max(0, height - notice.length);
   const visible = body.length > budgetHeight
@@ -352,5 +351,5 @@ export function renderFleetLedgerFallback(
 
 /** A blank tab-content placeholder (e.g. while a ledger load is in flight). */
 export function renderFleetTranscriptLoading(width: number): Line[] {
-  return [buildPanelLine(width, [[' Loading…', DEFAULT_PANEL_PALETTE.dim]]), createEmptyLine(width)];
+  return [buildViewLine(width, [[' Loading…', DEFAULT_VIEW_PALETTE.dim]]), createEmptyLine(width)];
 }

package/src/{panels/fleet-panel-worktree-detail.ts → views/fleet-worktree-detail.ts}

@@ -1,17 +1,17 @@
 // ---------------------------------------------------------------------------
-// fleet-panel-worktree-detail.ts
+// fleet-worktree-detail.ts
 //
-// NET-NEW helper (not moved code, it was authored here because fleet-panel.ts
+// NET-NEW helper (not moved code, it was authored here because src/input/agents-modal.ts
 // sits at the architecture check's 800-line cap, scripts/check-architecture.ts):
-// a single pure helper for the Fleet panel's per-work-item detail view, so a
-// new panel feature doesn't have to fight that cap inline.
+// a single pure helper for the Agents modal's per-work-item detail view, so a
+// new view feature doesn't have to fight that cap inline.
 // ---------------------------------------------------------------------------
 
 import type { WorkItem } from '@pellux/goodvibes-sdk/platform/orchestration';
 
 /**
- * Per-item worktree-isolation detail text for the Fleet panel's expanded
- * node detail (fleet-panel.ts renderDetail), or null for a shared-isolation
+ * Per-item worktree-isolation detail text for the Agents modal's expanded
+ * node detail (src/input/agents-modal.ts renderDetail), or null for a shared-isolation
  * item (or any node that isn't a work item). Inferred from the item's OWN
  * fields (worktreePath/worktreeBranch/mergeState) rather than a
  * workstream-level flag, adaptWorkItem's ProcessNode.raw carries
@@ -41,7 +41,7 @@ export function formatWorkItemIsolationDetail(item: WorkItem): string | null {
   }
 }
 
-/** `node.raw` for a 'work-item' ProcessNode is `{ item, workstreamId }` (unknown to the panel's own types), this narrows without fleet-panel.ts needing its own WorkItem import at the call site. Non-work-item nodes (raw is undefined/some other shape) fall through to null. */
+/** `node.raw` for a 'work-item' ProcessNode is `{ item, workstreamId }` (unknown to the Agents modal's own types), this narrows without src/input/agents-modal.ts needing its own WorkItem import at the call site. Non-work-item nodes (raw is undefined/some other shape) fall through to null. */
 export function formatWorkItemIsolationDetailFromRaw(raw: unknown): string | null {
   const item = (raw as { item?: WorkItem } | undefined)?.item;
   return item ? formatWorkItemIsolationDetail(item) : null;

package/src/{panels → views}/hosted-session-feed.ts

@@ -3,11 +3,11 @@
  * session it is attached to.
  *
  * One place holds it because two surfaces read it: the `/hosted` command family
- * (which prints status and drives the verbs) and the Hosted Session panel
+ * (which prints status and drives the verbs) and the Hosted Session view
  * (which renders the conversation). The same shared-feed shape
  * notifications-feed.ts already uses in this repo, a module-level instance a
- * command writes and a panel subscribes to, rather than threading a service
- * through the command context and the panel deps for one feature.
+ * command writes and a view subscribes to, rather than threading a service
+ * through the command context and the Hosted Session view deps for one feature.
  *
  * ── What is a fact here, and what is a rendering ──────────────────────────
  *
@@ -18,7 +18,7 @@
  *
  * The ROWS are this terminal's rendering of the transcript: the history
  * `attach` backfilled, plus what the live `turn` and `tools` frames have said
- * since. A row that is still being streamed is marked `streaming` so the panel
+ * since. A row that is still being streamed is marked `streaming` so the Hosted Session view
  * can show it growing rather than waiting for the turn to end, the same thing
  * a local turn does.
  *
@@ -57,7 +57,7 @@ export interface HostedRunningToolCall {
   readonly startedAt: number;
 }
 
-/** Everything the panel renders and the command reports. */
+/** Everything the Hosted Session view renders and the command reports. */
 export interface HostedSessionFeedState {
   readonly record: HostedSessionRecord | null;
   readonly rows: readonly HostedSessionRow[];
@@ -340,7 +340,7 @@ export class HostedSessionFeed {
 
 let sharedFeed: HostedSessionFeed | null = null;
 
-/** The one feed the `/hosted` command writes and the Hosted Session panel reads. */
+/** The one feed the `/hosted` command writes and the Hosted Session view reads. */
 export function getSharedHostedSessionFeed(): HostedSessionFeed {
   sharedFeed ??= new HostedSessionFeed();
   return sharedFeed;

package/src/views/modal-surface-registry.ts

@@ -0,0 +1,41 @@
+/**
+ * modal-surface-registry.ts, the config-modal surfaces by name.
+ *
+ * Every settings-style modal (services, providers, plugins, hooks, memory and
+ * the rest) is a ConfigModalSurface registered here once at startup, with the
+ * old names that should open it (for example 'providers' and 'accounts' both
+ * open 'providers-modal'). The config-modal host resolves a name through
+ * getModalSurface; ctx.openModal and the view router (views.ts) resolve an old
+ * name through getModalRedirect first.
+ */
+
+import type { ConfigModalSurface, ConfigModalSurfaceRegistry } from '../input/config-modal-types.ts';
+
+export class ModalSurfaceRegistry implements ConfigModalSurfaceRegistry {
+  private readonly surfaces = new Map<string, ConfigModalSurface>();
+  private readonly redirects = new Map<string, string>();
+
+  /** Register (or replace) the surface for its name. */
+  registerModalSurface(surface: ConfigModalSurface): void {
+    this.surfaces.set(surface.name, surface);
+  }
+
+  /** Make an old name open a modal (e.g. 'accounts' opens 'providers-modal'). */
+  registerModalRedirect(name: string, modalName: string): void {
+    this.redirects.set(name, modalName);
+  }
+
+  getModalSurface(name: string): ConfigModalSurface | undefined {
+    return this.surfaces.get(name);
+  }
+
+  /** The modal an old name opens, or undefined. */
+  getModalRedirect(name: string): string | undefined {
+    return this.redirects.get(name);
+  }
+
+  /** Every old name that opens a modal, with the modal it opens. */
+  listRedirects(): ReadonlyArray<readonly [string, string]> {
+    return [...this.redirects.entries()];
+  }
+}

package/src/{panels → views}/modals/devices-modal.ts

@@ -62,7 +62,7 @@ class DevicesModalSurface implements ConfigModalSurface {
     const now = this.deps.now?.() ?? Date.now();
     const tokens = this.deps.pairingTokens.list();
     const rows: ConfigModalRow[] = [];
-    rows.push(infoRow('intro', 'Devices paired to this daemon. Each has its own token; revoke one without affecting the others.', { dim: true }));
+    rows.push(infoRow('intro', 'Devices paired to this daemon. Each has its own token; revoke one without affecting the others.', { fg: MODAL_TONES.dim }));
 
     if (tokens.length === 0) {
       rows.push(infoRow('empty', 'No devices paired yet: run /pair (or open the pairing QR) to add one.'));
@@ -78,7 +78,7 @@ class DevicesModalSurface implements ConfigModalSurface {
       legacyRevoked
         ? 'Legacy shared token: revoked.'
         : 'Legacy shared token: active; migrate a device off it (m), then revoke it (s).',
-      legacyRevoked ? { dim: true } : { fg: MODAL_TONES.reasoning },
+      legacyRevoked ? { fg: MODAL_TONES.dim } : { fg: MODAL_TONES.reasoning },
     ));
 
     return {