@esso0428/pi-subagents 0.17.13 → 0.17.15

package/src/index.ts

@@ -58,10 +58,9 @@ import {
   type Theme,
   type UICtx,
 } from "./ui/agent-widget.js";
-import { FleetList, type FleetUICtx, type FleetWorkflow } from "./ui/fleet-list.js";
 import { showSchedulesMenu } from "./ui/schedule-menu.js";
 import { renderWorkflowCard, renderWorkflowEntryCard } from "./ui/workflow-card.js";
-import { openWorkflowFromFleet, showWorkflowsMenu, type WorkflowMenuDeps } from "./ui/workflow-menu.js";
+import { showWorkflowsMenu, type WorkflowMenuDeps } from "./ui/workflow-menu.js";
 import { getLifetimeCost, getLifetimeTotal, getSessionContextPercent, type LifetimeUsage, PendingUsagePool, toReportedUsage } from "./usage.js";
 import { decideWorkflowCollision, FOREIGN_WORKFLOW_TOOL_NAMES } from "./workflow/collisions.js";
 import { WORKFLOW_ENTRY_TYPE, type WorkflowEntryData, workflowEntryData } from "./workflow/entry.js";
@@ -425,7 +424,7 @@ export default function (pi: ExtensionAPI) {
   /** Show `~$X` next to token counts in the subagent surfaces. */
   let showCost = false;
   function isShowCostEnabled(): boolean { return showCost; }
-  function setShowCost(b: boolean): void { showCost = b; widget.update(); fleet.update(); }
+  function setShowCost(b: boolean): void { showCost = b; widget.update(); }
   /** Name the model and thinking level on the widget's running rows. */
   let showModel = false;
   function isShowModelEnabled(): boolean { return showModel; }
@@ -438,16 +437,6 @@ export default function (pi: ExtensionAPI) {
   let viewerMarkdown: ViewerMarkdownMode = "assistant";
   function getViewerMarkdown(): ViewerMarkdownMode { return viewerMarkdown; }
   function setViewerMarkdown(mode: ViewerMarkdownMode): void { viewerMarkdown = mode; }
-  /**
-   * The viewer's `m` key, from either entry point: set the mode and persist it,
-   * so the key and `/agents → Settings` stay one setting rather than one per
-   * entry point. `ctx` carries only the warning a failed write notifies with,
-   * and the fleet list may be acting without one.
-   */
-  function chooseViewerMarkdown(mode: ViewerMarkdownMode, ctx?: ExtensionCommandContext): void {
-    setViewerMarkdown(mode);
-    persistSettings(ctx, `Viewer markdown set to ${mode}`);
-  }
   const pendingUsage = new PendingUsagePool();
 
   // ---- Cancellable pending notifications ----
@@ -493,7 +482,6 @@ export default function (pi: ExtensionAPI) {
   function sendIndividualNudge(record: AgentRecord) {
     agentActivity.delete(record.id);
     widget.markFinished(record.id);
-    fleet.onAgentFinished(record.id);
     scheduleNudge(record.id, () => emitIndividualNudge(record));
     widget.update();
   }
@@ -501,7 +489,7 @@ export default function (pi: ExtensionAPI) {
   // ---- Group join manager ----
   const groupJoin = new GroupJoinManager(
     (records, partial) => {
-      for (const r of records) { agentActivity.delete(r.id); widget.markFinished(r.id); fleet.onAgentFinished(r.id); }
+      for (const r of records) { agentActivity.delete(r.id); widget.markFinished(r.id); }
 
       const groupKey = `group:${records.map(r => r.id).join(",")}`;
       scheduleNudge(groupKey, () => {
@@ -605,7 +593,6 @@ export default function (pi: ExtensionAPI) {
     if (record.resultConsumed) {
       agentActivity.delete(record.id);
       widget.markFinished(record.id);
-      fleet.onAgentFinished(record.id);
       widget.update();
       return;
     }
@@ -686,7 +673,7 @@ export default function (pi: ExtensionAPI) {
     // Every programmatic spawn lands here — cross-extension RPC, both `@handle`
     // mention paths, and the `Symbol.for("pi-subagents:manager")` registry — and
     // none came through the Agent tool, which is where the UI activity tracker is
-    // otherwise created. Without one the widget and FleetView have no tool name
+    // otherwise created. Without one the widget has no tool name
     // and no turn count, so the row reads `thinking…` for the agent's whole life
     // while the header's tool-use count climbs beside it (#181). Double-tracking
     // is not possible: the Agent tool calls `manager.spawn` directly. The tracker
@@ -700,7 +687,7 @@ export default function (pi: ExtensionAPI) {
     // leaves the displayed ceiling stale.
     const { state, callbacks } = createActivityTracker(resolveEffectiveMaxTurns(dispatch.type, options?.maxTurns));
     // Repaints are left to the manager's `onStart` callback, which already starts
-    // the widget/fleet timers for agents that enter this way.
+    // the widget timer for agents that enter this way.
     const id = manager.spawn(piRef, ctxRef, dispatch.type, prompt, { ...options, ...callbacks });
     agentActivity.set(id, state);
     return id;
@@ -812,8 +799,6 @@ export default function (pi: ExtensionAPI) {
     if (ctx.hasUI && (ctx.mode === undefined || ctx.mode === "tui")) {
       widget.setUICtx(ctx.ui);
       widget.update();
-      fleet.setUICtx(ctx.ui as any, false);
-      fleet.setCwd(ctx.cwd);
     }
     manager.clearCompleted(true);
     // Guard mirrors the `!scheduler.isActive()` pattern below: session_start
@@ -859,8 +844,8 @@ export default function (pi: ExtensionAPI) {
       ctx.ui.addAutocompleteProvider(current =>
         createMentionProvider(
           current,
-          // Plain text, not renderAgentName: the same label FleetView and the
-          // widget show, but the autocomplete description cannot carry ANSI.
+          // Plain text, not renderAgentName: the same label the widget shows,
+          // but the autocomplete description cannot carry ANSI.
           () => mentionRoster(manager, mentionTypes(), type => getConfig(type).displayName),
           isAgentMentionsEnabled,
         ),
@@ -970,7 +955,7 @@ export default function (pi: ExtensionAPI) {
 
     // Evicted, but its conversation is still on disk: reopen it. This is an
     // ordinary spawn carrying a session file, so the new record picks up the
-    // widget, fleet row, transcript and completion notification unchanged —
+    // widget row, transcript and completion notification unchanged —
     // and `reclaim` hands it back the names the tombstone was holding.
     if (resolved?.kind === "tombstone") {
       const entry = resolved.entry;
@@ -1092,8 +1077,8 @@ export default function (pi: ExtensionAPI) {
     try {
       // Nothing else to pass: runAgent resolves model, thinking and max turns
       // from the agent's own config when the spawn omits them, and the
-      // manager's onStart/onComplete callbacks own the widget, the fleet list
-      // and the completion notification — the same contract the scheduler and
+      // manager's onStart/onComplete callbacks own the widget and completion
+      // notification — the same contract the scheduler and
       // cross-extension RPC spawns run under.
       const id = spawnTopLevel(pi, ctx, type, mention.message, {
         description: describeMention(mention.message),
@@ -1138,7 +1123,6 @@ export default function (pi: ExtensionAPI) {
     for (const timer of pendingNudges.values()) clearTimeout(timer);
     pendingNudges.clear();
     widget.dispose();
-    fleet.dispose();
     // Awaited: it emits `session_shutdown` into every retained child session so
     // extensions bound there can release what they armed in `session_start` (#242).
     // pi awaits this handler, and the process exits right after — unawaited, those
@@ -1162,16 +1146,6 @@ export default function (pi: ExtensionAPI) {
   }, isShowModelEnabled);
   function setWidgetMode(m: WidgetMode): void { widgetMode = m; widget.update(); }
 
-  // Claude Code-style FleetView: navigable list of main + subagents below the editor.
-  // The last two arguments keep a conversation overlay opened here identical to
-  // one opened from `/agents`: same setting on the way in, same persist out.
-  const fleet = new FleetList(manager, agentActivity, isShowCostEnabled, getViewerMarkdown,
-    (mode) => chooseViewerMarkdown(mode, currentCtx as unknown as ExtensionCommandContext | undefined),
-    process.cwd());
-  let fleetViewEnabled = true;
-  function isFleetViewEnabled(): boolean { return fleetViewEnabled; }
-  function setFleetViewEnabled(b: boolean): void { fleetViewEnabled = b; fleet.setEnabled(b); }
-
   // Claude Code-style `@handle message` prompt mentions. Read live by both the
   // `input` hook and the stacked autocomplete provider, so the toggle applies
   // immediately — the provider itself can never be unregistered (pi's wrapper
@@ -1297,7 +1271,7 @@ export default function (pi: ExtensionAPI) {
   /**
    * Launch a detached resume of an existing agent and wire everything a
    * re-running agent needs: transcript anchoring, activity tracking, join-mode
-   * batching, the widget/fleet refresh, and the `subagents:created` event.
+   * batching, the widget refresh, and the `subagents:created` event.
    *
    * Shared by the Agent tool's `resume` + `run_in_background` branch and the
    * `@handle message` prompt mention — they differ only in how they report the
@@ -1371,8 +1345,6 @@ export default function (pi: ExtensionAPI) {
     widget.markRunning(id);
     widget.ensureTimer();
     widget.update();
-    fleet.ensureTimer();
-    fleet.update();
 
     // Resume ignores subagent_type (the record keeps the type it was
     // spawned with), so report the record's own identity — a "created"
@@ -1391,7 +1363,6 @@ export default function (pi: ExtensionAPI) {
   // Grab UI context from first tool execution + clear lingering widget on new turn
   pi.on("tool_execution_start", async (_event, ctx) => {
     if (ctx.hasUI && (ctx.mode === undefined || ctx.mode === "tui")) widget.setUICtx(ctx.ui as UICtx);
-    if (ctx.hasUI && ctx.mode === undefined) fleet.setUICtx(ctx.ui as unknown as FleetUICtx, true);
     widget.onTurnStart();
   });
 
@@ -1444,7 +1415,6 @@ export default function (pi: ExtensionAPI) {
       setStrictAgentFiles: (b) => { strictAgentFiles = b; },
       setDisableDefaultAgents: setDisableDefaultAgents,
       setToolDescriptionMode: setToolDescriptionMode,
-      setFleetView: setFleetViewEnabled,
       setAgentMentions: setAgentMentionMode,
       setRememberAgents,
       setWidgetMode: setWidgetMode,
@@ -2129,9 +2099,6 @@ Terse command-style prompts produce shallow, generic work.
         agentActivity.set(id, bgState);
         widget.ensureTimer();
         widget.update();
-        fleet.ensureTimer();
-        fleet.update();
-
         // Emit created event
         pi.events.emit("subagents:created", {
           id,
@@ -2214,8 +2181,6 @@ Terse command-style prompts produce shallow, generic work.
             fgId = a.id;
             agentActivity.set(a.id, fgState);
             widget.ensureTimer();
-            fleet.ensureTimer();
-            fleet.update();
             break;
           }
         }
@@ -2271,8 +2236,7 @@ Terse command-style prompts produce shallow, generic work.
         if (fgId) {
           agentActivity.delete(fgId);
           widget.markFinished(fgId);
-          fleet.onAgentFinished(fgId);
-        }
+            }
       }
 
       // Get final token count — from the record, like the cost below it, so the
@@ -2344,30 +2308,6 @@ Terse command-style prompts produce shallow, generic work.
    */
   const workflowTasks = new Map<string, WorkflowTask>();
 
-  /**
-   * Workflow runs as the fleet list wants them.
-   *
-   * Mapped here rather than handing `WorkflowTask` over the seam: the list is
-   * deliberately ignorant of the workflow engine, and a run's counters live in
-   * the progress log rather than on the record, so they are derived per call
-   * the same way the card derives them.
-   */
-  function fleetWorkflows(): FleetWorkflow[] {
-    // Cached counters only, no derivation: the fleet list calls this on a
-    // 200ms tick and reads the roster several times per update, so walking a
-    // run's progress log here would put O(log) work in the render loop.
-    return [...workflowTasks.values()].map(task => ({
-      id: task.id,
-      name: task.meta?.name ?? task.workflowName ?? task.id,
-      status: task.status,
-      doneCount: task.doneCount,
-      totalCount: task.agentCount,
-      startedAt: task.startTime,
-      ...(task.endTime !== undefined ? { completedAt: task.endTime } : {}),
-      tokens: task.totalTokens,
-    }));
-  }
-
   /**
    * Run a task to completion against the real manager, settling the record
    * either way. Never rejects: a run that cannot start (bad `meta`, oversized
@@ -2412,7 +2352,6 @@ Terse command-style prompts produce shallow, generic work.
    */
   function notifyWorkflowFinished(task: WorkflowTask) {
     widget.update();
-    fleet.update();
     const result = workflowResultText(task);
     scheduleNudge(task.id, () => {
       pi.sendMessage<NotificationDetails>({
@@ -2586,7 +2525,6 @@ Terse command-style prompts produce shallow, generic work.
       // surfaces — nothing else would register the widget for a run whose
       // first agent has not started yet.
       widget.update();
-      fleet.update();
 
       // Background, like Claude Code: the id comes back now and the run keeps
       // going without the tool call.
@@ -2665,7 +2603,6 @@ Terse command-style prompts produce shallow, generic work.
 
       workflowsEnabled = false; // not setWorkflowsEnabled: this is not the user pinning it
       widget.update();
-      fleet.update();
       warn(verdict.message);
 
       if (!verdict.withdraw) return;
@@ -2740,7 +2677,6 @@ Terse command-style prompts produce shallow, generic work.
     const task = createWorkflowTask({ id: workflowRunId(), script, scriptPath: path, meta });
     workflowTasks.set(task.id, task);
     widget.update();
-    fleet.update();
     report(`Running workflow ${meta.name}…`, "info");
 
     // Detached: session_start is awaited by the host, and a workflow can run for
@@ -2756,7 +2692,6 @@ Terse command-style prompts produce shallow, generic work.
         display: false,
       }, { deliverAs: "nextTurn" });
       widget.update();
-      fleet.update();
     });
   }
 
@@ -2978,15 +2913,7 @@ Terse command-style prompts produce shallow, generic work.
       ctx.ui.notify(noAgentsMsg, "info");
     }
 
-    // The widget also listens through the global terminal-input hook. Pause it
-    // while the top-level menu owns the same arrow/Enter keys.
-    widget.suspendNavigation();
-    let choice: string | undefined;
-    try {
-      choice = await ctx.ui.select("Agents", options);
-    } finally {
-      widget.resumeNavigation();
-    }
+    const choice = await ctx.ui.select("Agents", options);
     if (!choice) return;
 
     if (choice.startsWith("Running agents (")) {
@@ -3118,36 +3045,30 @@ Terse command-style prompts produce shallow, generic work.
       }
     };
 
-    widget.suspendNavigation();
-    let choice: string | undefined;
-    try {
-      choice = await ctx.ui.custom<string | undefined>((_tui, _theme, _kb, done) => {
-        const list = new SelectList(options, Math.min(options.length, 10), getSelectListTheme());
-        list.setSelectedIndex(initialIndex);
-        const initialItem = options[initialIndex];
-        if (initialItem) remember(initialItem.value);
-        list.onSelectionChange = item => remember(item.value);
-        list.onSelect = item => {
-          remember(item.value);
-          done(item.value);
-        };
-        list.onCancel = () => done(undefined);
+    const choice = await ctx.ui.custom<string | undefined>((_tui, _theme, _kb, done) => {
+      const list = new SelectList(options, Math.min(options.length, 10), getSelectListTheme());
+      list.setSelectedIndex(initialIndex);
+      const initialItem = options[initialIndex];
+      if (initialItem) remember(initialItem.value);
+      list.onSelectionChange = item => remember(item.value);
+      list.onSelect = item => {
+        remember(item.value);
+        done(item.value);
+      };
+      list.onCancel = () => done(undefined);
 
-        const container = new Container();
-        container.addChild(new Text(title, 0, 0));
-        container.addChild(new Spacer(1));
-        container.addChild(list);
-        return {
-          render: (width: number) => container.render(width),
-          invalidate: () => container.invalidate(),
-          handleInput: (data: string) => {
-            if (!isKeyRelease(data)) list.handleInput(data);
-          },
-        };
-      });
-    } finally {
-      widget.resumeNavigation();
-    }
+      const container = new Container();
+      container.addChild(new Text(title, 0, 0));
+      container.addChild(new Spacer(1));
+      container.addChild(list);
+      return {
+        render: (width: number) => container.render(width),
+        invalidate: () => container.invalidate(),
+        handleInput: (data: string) => {
+          if (!isKeyRelease(data)) list.handleInput(data);
+        },
+      };
+    });
 
     if (!choice) return undefined;
     return pairs.find(({ record }) => record.id === choice)?.record;
@@ -3589,7 +3510,6 @@ Write the file using the write tool. Only write the file, nothing else.`;
       strictAgentFiles,
       disableDefaultAgents: isDefaultsDisabled(),
       toolDescriptionMode: getToolDescriptionMode(),
-      fleetView: isFleetViewEnabled(),
       agentMentions: getAgentMentionMode(),
       rememberAgents: getRememberAgents(),
       widgetMode: getWidgetMode(),
@@ -3767,7 +3687,7 @@ Write the file using the write tool. Only write the file, nothing else.`;
           id: "showCost",
           label: "Show cost",
           description:
-            "Show an estimated `~$0.0042` beside subagent token counts in the widget, fleet view, results and notifications. Priced by pi from the model's rates — omitted entirely for a model it has no rates for.",
+            "Show an estimated `~$0.0042` beside subagent token counts in the widget, results and notifications. Priced by pi from the model's rates — omitted entirely for a model it has no rates for.",
           currentValue: isShowCostEnabled() ? "on" : "off",
           values: ["on", "off"],
         },
@@ -3787,13 +3707,6 @@ Write the file using the write tool. Only write the file, nothing else.`;
           currentValue: getViewerMarkdown(),
           values: ["off", "assistant", "all"],
         },
-        {
-          id: "fleetView",
-          label: "Fleet view",
-          description: "Claude Code-style main+subagents list below the editor (↓/← to navigate, Enter to view)",
-          currentValue: isFleetViewEnabled() ? "on" : "off",
-          values: ["on", "off"],
-        },
         {
           id: "agentMentions",
           label: "Agent mentions",
@@ -3961,10 +3874,6 @@ Write the file using the write tool. Only write the file, nothing else.`;
       } else if (id === "viewerMarkdown") {
         setViewerMarkdown(value as ViewerMarkdownMode);
         notifyApplied(ctx, `Viewer markdown set to ${value}`);
-      } else if (id === "fleetView") {
-        const enabled = value === "on";
-        setFleetViewEnabled(enabled);
-        notifyApplied(ctx, `Fleet view ${enabled ? "enabled" : "disabled"}`);
       } else if (id === "agentMentions") {
         const mode = value as AgentMentionMode;
         setAgentMentionMode(mode);
@@ -4069,31 +3978,6 @@ Write the file using the write tool. Only write the file, nothing else.`;
     }
   }
 
-  // Persist the current snapshot, emit `subagents:settings_changed`, and surface
-  // the right toast. Successful saves show info; persistence failures downgrade
-  // to warning so users aren't silently reverted on restart. Event fires regardless
-  // of outcome so listeners see the in-memory change.
-  /**
-   * Persist + broadcast the settings, silent on success — for a change whose
-   * feedback is the UI it just changed: the viewer's `m` key, where a
-   * notification per press would talk over the overlay it is describing.
-   *
-   * A *failed* write still speaks. Every other settings path warns when the
-   * value is session-only, and swallowing it here would leave a preference
-   * looking persisted when the next session will not have it.
-   */
-  function persistSettings(ctx: ExtensionCommandContext | undefined, changeMsg: string): void {
-    const { message, level } = saveAndEmitChanged(
-      snapshotSettings(),
-      changeMsg,
-      (event, payload) => pi.events.emit(event, payload),
-    );
-    // `ctx` is absent only on the fleet path between sessions, where
-    // `currentCtx` has been cleared and there is no UI to carry the warning to.
-    // The write still happens.
-    if (level === "warning") ctx?.ui.notify(message, level);
-  }
-
   function notifyApplied(ctx: ExtensionCommandContext, successMsg: string) {
     const { message, level } = saveAndEmitChanged(
       snapshotSettings(),
@@ -4108,19 +3992,11 @@ Write the file using the write tool. Only write the file, nothing else.`;
     handler: async (_args, ctx) => { await showAgentsMenu(ctx); },
   });
 
-  /**
-   * What `/agents → Workflows` and the fleet list's `workflow` rows need from
-   * here. One object, built once: both entry points open the same inspector,
-   * and handing them different views of the session would let the two drift.
-   */
+  /** Dependencies shared by `/agents → Workflows` and its inspector. */
   const workflowMenuDeps: WorkflowMenuDeps = {
     tasks: workflowTasks,
     getRecord: id => manager.getRecord(id),
     viewAgentConversation,
-    // Read lazily: `currentCtx` is rebound on every session_start, and the
-    // fleet list may act between sessions, when there is none.
-    getCtx: () => currentCtx as unknown as ExtensionCommandContext | undefined,
   };
 
-  fleet.setWorkflowSource(fleetWorkflows, id => openWorkflowFromFleet(id, workflowMenuDeps));
 }

package/src/prompts.ts

@@ -29,8 +29,8 @@ export interface PromptExtras {
    * that makes the documentation true.
    *
    * Deliberately NOT applied to every subagent. In pi an ordinary agent's
-   * output IS read by a human — through FleetView, the conversation viewer and
-   * `get_subagent_result` — so terse raw data would be the wrong answer there.
+   * output IS read by a human — through the Agents widget, the conversation
+   * viewer and `get_subagent_result` — so terse raw data would be wrong there.
    */
   workflowChild?: boolean;
 }

package/src/settings.ts

@@ -118,12 +118,6 @@ export interface SubagentsSettings {
    * next pi session.
    */
   toolDescriptionMode?: ToolDescriptionMode;
-  /**
-   * Whether the Claude Code-style FleetView (the navigable main+subagents list
-   * rendered below the editor) is shown. Defaults to `true`. Pure-UI: when off,
-   * the list never registers and the global key handler never captures input.
-   */
-  fleetView?: boolean;
   /**
    * Whether `@handle message` typed at the prompt is routed to that subagent
    * instead of the main model, and whether `@` offers running agents alongside
@@ -266,7 +260,7 @@ export interface SubagentsSettings {
   reportUsage?: boolean;
   /**
    * Whether the subagent surfaces show an estimated dollar cost next to their
-   * token counts (widget, FleetView, conversation viewer, foreground results,
+   * token counts (widget, conversation viewer, foreground results,
    * completion notifications). Defaults to `false`. Applied live.
    *
    * Rendered as `~$0.0042` — the tilde marks it as pi's reported estimate
@@ -319,7 +313,6 @@ export interface SettingsAppliers {
   setStrictAgentFiles: (b: boolean) => void;
   setDisableDefaultAgents: (b: boolean) => void;
   setToolDescriptionMode: (mode: ToolDescriptionMode) => void;
-  setFleetView: (b: boolean) => void;
   setAgentMentions: (mode: AgentMentionMode) => void;
   setRememberAgents: (b: boolean) => void;
   setWidgetMode: (mode: WidgetMode) => void;
@@ -414,9 +407,6 @@ function sanitize(raw: unknown): SubagentsSettings {
   if (typeof r.toolDescriptionMode === "string" && VALID_TOOL_DESCRIPTION_MODES.has(r.toolDescriptionMode)) {
     out.toolDescriptionMode = r.toolDescriptionMode as ToolDescriptionMode;
   }
-  if (typeof r.fleetView === "boolean") {
-    out.fleetView = r.fleetView;
-  }
   // Was a boolean before the `model` mode existed. A hand-written or
   // previously-written `true` means "on", which is now the default `model`.
   if (typeof r.agentMentions === "boolean") {
@@ -526,7 +516,6 @@ export function applySettings(s: SubagentsSettings, appliers: SettingsAppliers):
   if (typeof s.strictAgentFiles === "boolean") appliers.setStrictAgentFiles(s.strictAgentFiles);
   if (typeof s.disableDefaultAgents === "boolean") appliers.setDisableDefaultAgents(s.disableDefaultAgents);
   if (s.toolDescriptionMode) appliers.setToolDescriptionMode(s.toolDescriptionMode);
-  if (typeof s.fleetView === "boolean") appliers.setFleetView(s.fleetView);
   if (s.agentMentions) appliers.setAgentMentions(s.agentMentions);
   if (typeof s.rememberAgents === "boolean") appliers.setRememberAgents(s.rememberAgents);
   if (s.widgetMode) appliers.setWidgetMode(s.widgetMode);

package/src/ui/agent-mention.ts

@@ -50,8 +50,8 @@ import type { AgentRecord, AgentTombstone } from "../types.js";
 /**
  * One thing `@` can address, and what sending to it will do. `typeLabel` is the
  * agent's `display_name`, resolved by the caller: this module stays independent
- * of the type registry, but the popup must agree with FleetView and the widget,
- * which both render the label rather than the raw type.
+ * of the type registry, but the popup must agree with the Agents widget,
+ * which renders the label rather than the raw type.
  */
 export type MentionTarget =
   | { kind: "record"; handle: string; record: AgentRecord; typeLabel: string }

package/src/ui/agent-widget.ts

@@ -267,8 +267,6 @@ export class AgentWidget {
   private inputUnsub: (() => void) | undefined;
   /** Whether arrow keys currently navigate the agent roster. */
   private navigationActive = false;
-  /** Number of modal surfaces currently owning this listener's input suspension. */
-  private navigationSuspensionDepth = 0;
   /** Stable identity of the selected row, so roster changes do not jump selection. */
   private selectedAgentId: string | undefined;
   /** Last logical roster index of the selected row, used when it disappears. */
@@ -408,16 +406,6 @@ export class AgentWidget {
     return this.cachedRoster;
   }
 
-  /** Suspend only this listener's keyboard ownership; never alter the widget. */
-  suspendNavigation(): void {
-    this.navigationSuspensionDepth++;
-  }
-
-  /** Restore one layer of this listener's keyboard ownership. */
-  resumeNavigation(): void {
-    if (this.navigationSuspensionDepth > 0) this.navigationSuspensionDepth--;
-  }
-
   /** Set the UI context (grabbed from first tool execution). */
   setUICtx(ctx: UICtx): boolean {
     if (ctx === this.uiCtx) return false;
@@ -431,7 +419,6 @@ export class AgentWidget {
     this.tui = undefined;
     this.lastStatusText = undefined;
     this.lastRenderKey = undefined;
-    this.navigationSuspensionDepth = 0;
     this.navigationActive = false;
     this.selectedAgentId = undefined;
     this.selectedRosterIndex = 0;
@@ -450,7 +437,7 @@ export class AgentWidget {
   }
 
   /** Request a render on the currently registered TUI without touching input. */
-  requestUiRefresh(force = true): boolean {
+  requestUiRefresh(force = false): boolean {
     if (!this.tui || typeof this.tui.requestRender !== "function") return false;
     this.tui.requestRender(force);
     return true;
@@ -573,7 +560,6 @@ export class AgentWidget {
   /** Handle terminal input before it reaches the focused prompt editor. */
   handleKey(data: string): { consume?: boolean; data?: string } | undefined {
     if (!this.uiCtx || isKeyRelease(data)) return undefined;
-    if (this.navigationSuspensionDepth > 0) return undefined;
     if (!this.editorHasFocus()) {
       if (this.navigationActive) this.deactivate();
       return undefined;
@@ -1111,7 +1097,6 @@ export class AgentWidget {
     this.tui = undefined;
     this.lastStatusText = undefined;
     this.lastRenderKey = undefined;
-    this.navigationSuspensionDepth = 0;
     this.navigationActive = false;
     this.selectedAgentId = undefined;
     this.selectedRosterIndex = 0;

package/src/ui/workflow-menu.ts

@@ -3,13 +3,10 @@
  *
  * The same shape `schedule-menu.ts` has for `/agents → Scheduled jobs`: the
  * submenu and the overlay it opens live here, and everything they need arrives
- * as {@link WorkflowMenuDeps} rather than through a closure. The inspector is
- * reached from two places — this menu and a `workflow` row in the fleet list —
- * and both go through `showWorkflowDialog`, so the two entry points cannot
- * drift apart on what the keys do.
+ * as {@link WorkflowMenuDeps} rather than through a closure.
  *
- * Lives in the agents menu rather than as a top-level `/workflows` command: it
- * is one more view of the same fleet, and a second command name would only add
+ * Lives in the agents menu rather than as a top-level `/workflows` command: a
+ * second command name would only add
  * a collision surface (pi renames duplicate commands to `/workflows:1` and
  * `/workflows:2`, which breaks the bare name for both).
  */
@@ -31,12 +28,6 @@ export interface WorkflowMenuDeps {
   getRecord(id: string): AgentRecord | undefined;
   /** The conversation overlay `c` opens on an agent row. */
   viewAgentConversation(ctx: ExtensionCommandContext, record: AgentRecord): Promise<void>;
-  /**
-   * The session context, for the fleet-list entry point — that one is a
-   * keypress in a list that holds no `ctx` of its own. Undefined between
-   * sessions, which is a no-op rather than an error.
-   */
-  getCtx(): ExtensionCommandContext | undefined;
 }
 
 /**
@@ -54,9 +45,7 @@ export async function showWorkflowDialog(
   task: WorkflowTask,
   deps: WorkflowMenuDeps,
 ): Promise<void> {
-  // Overlaid on the same terms as the conversation viewer, because they are
-  // reached the same way: both are rows of the fleet list, and opening one
-  // must not behave unlike opening the other. Inline, the frame would render
+  // Use an overlay like the conversation viewer. Inline, the frame would render
   // into the conversation and stay in the scrollback after it closed.
   const { VIEWPORT_HEIGHT_PCT } = await import("./conversation-viewer.js");
   /**
@@ -147,22 +136,6 @@ export async function showWorkflowDialog(
   );
 }
 
-/**
- * Open a run from the fleet list.
- *
- * The list hands back an id rather than a task, so a run that settled and was
- * swept between render and keypress is a no-op instead of a crash. `esc` in the
- * dialog closes it and control returns to the list — which is why the promise
- * is handed back: the list puts the cursor back on the run rather than dropping
- * the reader at `main`.
- */
-export function openWorkflowFromFleet(id: string, deps: WorkflowMenuDeps): Promise<void> | void {
-  const task = deps.tasks.get(id);
-  const ctx = deps.getCtx();
-  if (task === undefined || ctx === undefined) return;
-  return showWorkflowDialog(ctx, task, deps);
-}
-
 /** `/agents → Workflows` — list this session's runs, open one. */
 export async function showWorkflowsMenu(
   ctx: ExtensionCommandContext,