@jwilger/pi-development-system 0.51.1 → 0.52.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.51.1",
+  "version": "0.52.0",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/src/core/test-runner.ts

@@ -65,7 +65,7 @@ export type ResultLike = {
 const NO_TEST_SCRIPT = /Missing script: "?test/;
 /** What npm prints around a missing script; any other line means something else ran or failed. */
 const NPM_MISSING_SCRIPT_NOISE =
-  /^(?:npm (?:error|ERR!)\s*)?(?:$|.*Missing script.*|.*To see a list of scripts.*|.*A complete log of this run.*|.*Did you mean.*|.*npm run.*)$/;
+  /^(?:npm (?:error|ERR!)\s*)?(?:$|.*Missing script.*|.*To see a list of scripts.*|.*A complete log of this run.*|.*Did you mean.*|.*npm run.*|Command exited with code \d+)$/;
 const onlyMissingScript = (text: string): boolean =>
   NO_TEST_SCRIPT.test(text) &&
   text.split("\n").every((l) => NPM_MISSING_SCRIPT_NOISE.test(l.trim()));

package/src/subagents/index.ts

@@ -1,25 +1,21 @@
 // @ts-nocheck: vendored upstream pi-subagent-manager 0.14.0 compiled under looser options; see src/subagents/VENDORED.md
 import { randomUUID } from "node:crypto";
-import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { buildSessionContext, getAgentDir } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { fuzzyFilter } from "@earendil-works/pi-tui";
-import { createInheritedToolSource } from "./orch/inherited-tools.ts";
-import { ThreadManager } from "./orch/manager.ts";
-import { subagentPrompt } from "./orch/prompt.ts";
-import { createDriverFactory } from "./orch/runtime.ts";
-import { agentTools } from "./orch/tools.ts";
-import {
-  IMPORT_REQUEST_PREFIX,
-  markImportOffered,
-  offerAgentImport,
-} from "./prefs/agent-import.ts";
 import { ConfigStore } from "./prefs/config.ts";
+import { IMPORT_REQUEST_PREFIX, markImportOffered, offerAgentImport } from "./prefs/agent-import.ts";
+import { ThreadManager } from "./orch/manager.ts";
 import { loadManagerSettings } from "./prefs/settings.ts";
-import type { SavedThread, ThreadEvent } from "./types.ts";
-import { AgentNavigationEditor } from "./ui/agent-navigation-editor.ts";
 import { configureAgents } from "./ui/settings-ui.ts";
 import { AgentNavigationController, showAgentStatus } from "./ui/status-ui.ts";
+import { AgentNavigationEditor } from "./ui/agent-navigation-editor.ts";
+import { createDriverFactory } from "./orch/runtime.ts";
+import { createInheritedToolSource } from "./orch/inherited-tools.ts";
+import { subagentPrompt } from "./orch/prompt.ts";
+import { agentTools } from "./orch/tools.ts";
 import { editAgentTypes, updateWidget } from "./ui/ui.ts";
+import type { SavedThread, ThreadEvent } from "./types.ts";
 
 const REGISTRY_ENTRY = "pi-subagent:registry:v1";
 const ROOT_MAILBOX_ENTRY = "pi-subagent:root-mailbox:v1";
@@ -85,55 +81,32 @@ export default function piSubagent(pi: ExtensionAPI): void {
     if (existingFactory && existingFactory !== navigationEditorFactory) return;
     // The host only transfers raw text, not cursor/undo/expanded-paste state.
     if (ctx.ui.getEditorText?.()) return;
-    const history = buildSessionContext(ctx.sessionManager.getBranch()).messages.flatMap(
-      (message) => {
-        if (message.role !== "user") return [];
-        const content = message.content;
-        return [
-          typeof content === "string"
-            ? content
-            : content
-                .filter((block) => block.type === "text")
-                .map((block) => block.text)
-                .join(""),
-        ];
-      },
-    );
+    const history = buildSessionContext(ctx.sessionManager.getBranch()).messages.flatMap((message) => {
+      if (message.role !== "user") return [];
+      const content = message.content;
+      return [typeof content === "string" ? content : content
+        .filter((block) => block.type === "text").map((block) => block.text).join("")];
+    });
     navigationEditorFactory = (tui, theme, keys) => {
       const editor = new AgentNavigationEditor(tui, theme, keys, {
         // Initial startup hydrates after session_start; replacement editors do not.
         initialHistory: seedHistory ? history : undefined,
         generation: () => navigationGeneration,
-        canOpen: () =>
-          manager !== undefined &&
-          context?.mode === "tui" &&
-          limits.subagentMode !== "off" &&
-          !navigation.isOpen &&
-          manager
-            .list()
-            .some(
-              (thread) =>
-                thread.path !== "/root" &&
-                (thread.state === "starting" || thread.state === "running"),
-            ),
-        openTree: () =>
-          manager && context
-            ? navigation.open(context, manager.scope("/root"), undefined, limits.nerdFontIcons)
-            : undefined,
-        canCollapse: () =>
-          manager !== undefined &&
-          context?.mode === "tui" &&
-          limits.subagentMode !== "off" &&
-          !navigation.isOpen &&
-          !widgetCollapsed &&
-          limits.widgetMode === "full" &&
-          widgetThreads().some((thread) => thread.path !== "/root"),
+        canOpen: () => manager !== undefined && context?.mode === "tui" &&
+          limits.subagentMode !== "off" && !navigation.isOpen &&
+          manager.list().some((thread) => thread.path !== "/root" &&
+            (thread.state === "starting" || thread.state === "running")),
+        openTree: () => manager && context
+          ? navigation.open(context, manager.scope("/root"), undefined, limits.nerdFontIcons) : undefined,
+        canCollapse: () => manager !== undefined && context?.mode === "tui" &&
+          limits.subagentMode !== "off" && !navigation.isOpen && !widgetCollapsed &&
+          limits.widgetMode === "full" && widgetThreads().some((thread) => thread.path !== "/root"),
         collapseWidget: () => {
           widgetCollapsed = true;
           if (context) refreshWidget(context);
         },
-        onError: (error) =>
-          context?.ui.notify(error instanceof Error ? error.message : String(error), "error"),
+        onError: (error) => context?.ui.notify(
+          error instanceof Error ? error.message : String(error), "error"),
       });
       navigationEditor = editor;
       return editor;
@@ -147,10 +120,7 @@ export default function piSubagent(pi: ExtensionAPI): void {
     // Migration requires the root's file tools and must not delegate source prompts.
     if (limits.subagentMode === "orchestration") {
       if (!firstRun)
-        ctx.ui.notify(
-          "Agent import requires Opportunistic mode. Change Subagent Mode in /agents settings.",
-          "info",
-        );
+        ctx.ui.notify("Agent import requires Opportunistic mode. Change Subagent Mode in /agents settings.", "info");
       return;
     }
     if (importInProgress) return;
@@ -170,15 +140,12 @@ export default function piSubagent(pi: ExtensionAPI): void {
     if (!manager) throw new Error("Subagent threads are not initialized; start a Pi session first");
     return manager;
   };
-  const widgetThreads = () =>
-    requireManager()
-      .list()
-      .filter((thread) => {
-        // Resuming a retained session makes it visible for this task, even after it settles again.
-        if (thread.state === "starting" || thread.state === "running")
-          hiddenWidgetThreads.delete(thread.path);
-        return !hiddenWidgetThreads.has(thread.path);
-      });
+  const widgetThreads = () => requireManager().list().filter((thread) => {
+    // Resuming a retained session makes it visible for this task, even after it settles again.
+    if (thread.state === "starting" || thread.state === "running")
+      hiddenWidgetThreads.delete(thread.path);
+    return !hiddenWidgetThreads.has(thread.path);
+  });
   const requireContext = () => {
     if (!context) throw new Error("No active Pi session");
     return context;
@@ -220,72 +187,44 @@ export default function piSubagent(pi: ExtensionAPI): void {
   };
   const pendingSummaries = (ctx: ExtensionContext) => {
     const entries = ctx.sessionManager.getBranch();
-    const summarized = new Set(
-      entries.flatMap((entry) => {
-        if (entry.type !== "custom_message" || entry.customType !== ROOT_SUMMARY_MESSAGE) return [];
-        const ids = (entry.details as { mailboxIds?: unknown } | undefined)?.mailboxIds;
-        return Array.isArray(ids) ? ids.filter((id): id is string => typeof id === "string") : [];
-      }),
-    );
+    const summarized = new Set(entries.flatMap((entry) => {
+      if (entry.type !== "custom_message" || entry.customType !== ROOT_SUMMARY_MESSAGE) return [];
+      const ids = (entry.details as { mailboxIds?: unknown } | undefined)?.mailboxIds;
+      return Array.isArray(ids) ? ids.filter((id): id is string => typeof id === "string") : [];
+    }));
     return entries.flatMap((entry) => {
       if (entry.type !== "custom" || entry.customType !== ROOT_MAILBOX_ENTRY) return [];
       const notification = entry.data as RootNotification | undefined;
       return notification?.rootSessionId === ctx.sessionManager.getSessionId() &&
-        notification.finalRecap === true &&
-        typeof notification.content === "string" &&
+        notification.finalRecap === true && typeof notification.content === "string" &&
         typeof notification.details?.mailboxId === "string" &&
-        !summarized.has(notification.details.mailboxId)
-        ? [notification]
-        : [];
+        !summarized.has(notification.details.mailboxId) ? [notification] : [];
     });
   };
   const scheduleSummary = (ctx: ExtensionContext) => {
-    if (
-      !limits.finalRecap ||
-      limits.subagentMode === "off" ||
-      summaryRunning ||
-      suspendingSummaries ||
-      summaryTimer !== undefined ||
-      ctx.isIdle() === false ||
-      !pendingSummaries(ctx).length
-    )
-      return;
+    if (!limits.finalRecap || limits.subagentMode === "off" || summaryRunning || suspendingSummaries ||
+        summaryTimer !== undefined || ctx.isIdle() === false || !pendingSummaries(ctx).length) return;
     const currentGeneration = generation;
     // Coalesce siblings finishing together and leave the settlement boundary before
     // starting a new turn. Recheck settings/session/idle state after the deferral.
     summaryTimer = setTimeout(() => {
       summaryTimer = undefined;
-      if (
-        currentGeneration !== generation ||
-        context?.sessionManager !== ctx.sessionManager ||
-        !limits.finalRecap ||
-        limits.subagentMode === "off" ||
-        suspendingSummaries ||
-        ctx.isIdle() === false
-      )
-        return;
+      if (currentGeneration !== generation || context?.sessionManager !== ctx.sessionManager ||
+          !limits.finalRecap || limits.subagentMode === "off" || suspendingSummaries || ctx.isIdle() === false) return;
       const notifications = pendingSummaries(ctx);
       if (!notifications.length) return;
       summaryRunning = true;
       try {
-        pi.sendMessage(
-          {
-            customType: ROOT_SUMMARY_MESSAGE,
-            content:
-              "Summarize the newly finished asynchronous subagent results for the user. " +
-              "Their results are in the preceding subagent notifications; use agent_output if more detail is needed. " +
-              "Give a concise main-thread answer covering results and any failures. " +
-              "Do not spawn or resume agents solely to summarize.\nAgents: " +
-              [...new Set(notifications.map((notification) => notification.details.path))].join(
-                ", ",
-              ),
-            display: false,
-            details: {
-              mailboxIds: notifications.map((notification) => notification.details.mailboxId),
-            },
-          },
-          { triggerTurn: true },
-        );
+        pi.sendMessage({
+          customType: ROOT_SUMMARY_MESSAGE,
+          content: "Summarize the newly finished asynchronous subagent results for the user. " +
+            "Their results are in the preceding subagent notifications; use agent_output if more detail is needed. " +
+            "Give a concise main-thread answer covering results and any failures. " +
+            "Do not spawn or resume agents solely to summarize.\nAgents: " +
+            [...new Set(notifications.map((notification) => notification.details.path))].join(", "),
+          display: false,
+          details: { mailboxIds: notifications.map((notification) => notification.details.mailboxId) },
+        }, { triggerTurn: true });
       } catch (error) {
         summaryRunning = false;
         warnDelivery(error);
@@ -339,10 +278,7 @@ export default function piSubagent(pi: ExtensionAPI): void {
           content: message,
           // Foreground results already handled by a busy main thread do not need
           // another turn. Completions during a summary do need a follow-up summary.
-          finalRecap:
-            limits.finalRecap &&
-            !suspendingSummaries &&
-            event.kind === "settled" &&
+          finalRecap: limits.finalRecap && !suspendingSummaries && event.kind === "settled" &&
             (thread.state === "completed" || thread.state === "failed") &&
             (requireContext().isIdle() || summaryRunning),
           details: {
@@ -399,14 +335,8 @@ export default function piSubagent(pi: ExtensionAPI): void {
   const refreshWidget = (ctx: ExtensionContext) => {
     if (limits.subagentMode === "off") {
       if (ctx.hasUI) ctx.ui.setWidget("pi-subagent", undefined);
-    } else
-      updateWidget(
-        ctx,
-        widgetThreads(),
-        widgetCollapsed ? "minimal" : limits.widgetMode,
-        limits.nerdFontIcons,
-        rootTurnEnded,
-      );
+    } else updateWidget(ctx, widgetThreads(), widgetCollapsed ? "minimal" : limits.widgetMode,
+      limits.nerdFontIcons, rootTurnEnded);
   };
   pi.on("agent_start", async (_event, ctx) => {
     // A new task must not resurrect the previous task's settled agents. Automatic
@@ -444,11 +374,7 @@ export default function piSubagent(pi: ExtensionAPI): void {
     };
   });
 
-  const attachSession = async (
-    ctx: ExtensionContext,
-    installEditor = false,
-    seedHistory = true,
-  ) => {
+  const attachSession = async (ctx: ExtensionContext, installEditor = false, seedHistory = true) => {
     cancelSummary();
     const token = ++generation;
     closeNavigation(installEditor);
@@ -605,19 +531,9 @@ export default function piSubagent(pi: ExtensionAPI): void {
           },
         });
       } else if (command === "status") {
-        await showAgentStatus(
-          ctx,
-          requireManager().scope("/root"),
-          undefined,
-          limits.nerdFontIcons,
-        );
+        await showAgentStatus(ctx, requireManager().scope("/root"), undefined, limits.nerdFontIcons);
       } else if (command === "tree") {
-        await navigation.open(
-          ctx,
-          requireManager().scope("/root"),
-          rest.join(" ") || undefined,
-          limits.nerdFontIcons,
-        );
+        await navigation.open(ctx, requireManager().scope("/root"), rest.join(" ") || undefined, limits.nerdFontIcons);
       } else
         ctx.ui.notify(
           "Usage: /agents [tree [path] | status | settings | types | import | reload]",

package/src/subagents/orch/inherited-tools.ts

@@ -43,7 +43,7 @@ export function createInheritedToolSource(pi: ExtensionAPI) {
       for (const tool of loadout.registered) {
         const exposure = loadout.getExposure(tool.name);
         // Inactive direct tools and hidden tools are not available in the root either.
-        if (!(callable.has(tool.name) || active.has(tool.name))) continue;
+        if (!callable.has(tool.name) && !active.has(tool.name)) continue;
         if (exposure === "hidden") continue;
         const metadata = info.get(tool.name);
         tools.push({
@@ -64,14 +64,14 @@ export function createInheritedToolSource(pi: ExtensionAPI) {
               // SDK orchestrators are instead recreated locally by runtime.
               throw new Error(
                 `Cannot safely inherit model-only tool ${tool.name}: the SDK does not ` +
-                  "provide a hook-preserving execution bridge. Run it in the main session.",
+                "provide a hook-preserving execution bridge. Run it in the main session.",
               );
             }
             const bridge = rootContext ?? context;
             if (!bridge) {
               throw new Error(
                 "Inherited tool execution needs a main-session agent_* tool call first. " +
-                  "Use agent_steer to resume this child instead of the thread dialog.",
+                "Use agent_steer to resume this child instead of the thread dialog.",
               );
             }
             const outcome = await bridge.executeTool(tool.name, args, { signal, onUpdate });

package/src/subagents/orch/runtime.ts

@@ -1,4 +1,4 @@
-// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: vendored upstream TUI/config code with no tests; rewriting it risks behaviour (see src/subagents/VENDORED.md)
+// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: the flagged functions (driver construction, settings resolution) are long sequences over pi sessions; only the pin and resume paths are covered by test/subagents/spawn-pins.test.ts, so splitting them would move untested behaviour (see src/subagents/VENDORED.md)
 import { existsSync } from "node:fs";
 import { mkdir, readFile, realpath } from "node:fs/promises";
 import path from "node:path";

package/src/subagents/prefs/agent-import.ts

@@ -159,8 +159,8 @@ export async function offerAgentImport(
       cwd: ctx.cwd,
       agentDir: options.agentDir,
       includeProject: ctx.isProjectTrusted(),
-      homeDir: options.homeDir,
-      extraAgentDirs: options.extraAgentDirs,
+      ...(options.homeDir === undefined ? {} : { homeDir: options.homeDir }),
+      ...(options.extraAgentDirs === undefined ? {} : { extraAgentDirs: options.extraAgentDirs }),
     });
     if (found.diagnostics.length > 0)
       ctx.ui.notify(dialogText(found.diagnostics.join("\n")), "warning");

package/src/subagents/prefs/config.ts

@@ -1,4 +1,4 @@
-// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: vendored upstream TUI/config code with no tests; rewriting it risks behaviour (see src/subagents/VENDORED.md)
+// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: agent-file parsing and layer merging are long validators kept close to upstream; only parseAgentType is covered (test/agents/agents.test.ts), so splitting the rest would move untested behaviour (see src/subagents/VENDORED.md)
 import { randomUUID } from "node:crypto";
 import {
   lstatSync,

package/src/subagents/prefs/import-discovery.ts

@@ -1,5 +1,5 @@
 // @ts-nocheck: vendored upstream pi-subagent-manager 0.14.0 compiled under looser options; see src/subagents/VENDORED.md
-import { type Dirent, lstatSync, readdirSync, readFileSync, type Stats } from "node:fs";
+import { lstatSync, readdirSync, readFileSync, type Dirent, type Stats } from "node:fs";
 import { homedir } from "node:os";
 import { delimiter, dirname, isAbsolute, join, normalize, relative, resolve, sep } from "node:path";
 import { parseDocument } from "yaml";
@@ -17,8 +17,8 @@ export interface DiscoverImportOptions {
   cwd: string;
   agentDir: string;
   includeProject: boolean;
-  homeDir?: string | undefined;
-  extraAgentDirs?: string | undefined;
+  homeDir?: string;
+  extraAgentDirs?: string;
 }
 
 const EXTRA_AGENT_DIRS_ENV = "PI_SUBAGENT_EXTRA_AGENT_DIRS";
@@ -227,7 +227,7 @@ function walk(
         continue;
       }
       if (isSkillMarkdownName(entry.name)) continue;
-      if (!(entry.isFile() && entry.name.endsWith(".md")) || entry.name.endsWith(".chain.md")) {
+      if (!entry.isFile() || !entry.name.endsWith(".md") || entry.name.endsWith(".chain.md")) {
         continue;
       }
       if (isExcluded(child, excludes) || isManagerOwned(child, managerDirs)) continue;
@@ -270,7 +270,7 @@ function considerFile(
   const rawName = fields.name?.trim() ?? "";
   const name = rawName || (permissive ? filenameName(resolved) : "");
   const description = fields.description ?? (permissive ? "" : undefined);
-  if (!name || description === undefined || !(permissive || description.trim())) return;
+  if (!name || description === undefined || (!permissive && !description.trim())) return;
   const packageName = fields.packageName?.trim() ?? "";
   candidates.push({
     id: resolved,
@@ -303,7 +303,7 @@ function looseDisplayFields(content: string): DisplayFields | undefined {
       /* Ambiguous display metadata is not selectable. */
     }
   }
-  if (!(fields.name && fields.description)) return undefined;
+  if (!fields.name || !fields.description) return undefined;
   return { name: fields.name, description: fields.description, packageName: fields.package };
 }
 

package/src/subagents/prefs/settings.ts

@@ -1,5 +1,5 @@
-// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: vendored upstream TUI/config code with no tests; rewriting it risks behaviour (see src/subagents/VENDORED.md)
-// biome-ignore-all lint/suspicious/noShadow: vendored upstream TUI/config code with no tests; rewriting it risks behaviour (see src/subagents/VENDORED.md)
+// biome-ignore-all lint/complexity/noExcessiveCognitiveComplexity: vendored upstream settings-file loader with no tests of its own; rewriting it risks behaviour (see src/subagents/VENDORED.md)
+// biome-ignore-all lint/suspicious/noShadow: vendored upstream settings-file loader with no tests of its own; rewriting it risks behaviour (see src/subagents/VENDORED.md)
 import { randomUUID } from "node:crypto";
 import {
   closeSync,

package/src/subagents/ui/agent-navigation-editor.ts

@@ -5,11 +5,11 @@ import {
   type KeybindingsManager,
 } from "@earendil-works/pi-coding-agent";
 import {
-  type EditorTheme,
   getKeybindings,
   Key,
-  type Keybinding,
   matchesKey,
+  type EditorTheme,
+  type Keybinding,
   type TUI,
 } from "@earendil-works/pi-tui";
 
@@ -87,11 +87,8 @@ export class AgentNavigationEditor extends CustomEditor {
 
   override handleInput(data: string): void {
     const outer = this.dispatchDepth === 0;
-    const direction = this.ordinaryArrow(data, "left")
-      ? "left"
-      : this.ordinaryArrow(data, "right")
-        ? "right"
-        : undefined;
+    const direction = this.ordinaryArrow(data, "left") ? "left"
+      : this.ordinaryArrow(data, "right") ? "right" : undefined;
     if (outer && (!direction || (this.pending && this.pending !== direction))) this.inputEpoch++;
     const previousPaste = this.pastePending;
     const wasPaste = previousPaste || data.includes("\x1b[200~");
@@ -103,12 +100,10 @@ export class AgentNavigationEditor extends CustomEditor {
     const cursor = this.getCursor();
     const text = this.getText();
     const lines = text.split("\n");
-    const atBoundary =
-      direction === "left"
-        ? cursor.line === 0 && cursor.col === 0
-        : direction === "right" &&
-          cursor.line === lines.length - 1 &&
-          cursor.col === lines.at(-1)!.length;
+    const atBoundary = direction === "left"
+      ? cursor.line === 0 && cursor.col === 0
+      : direction === "right" && cursor.line === lines.length - 1 &&
+        cursor.col === lines.at(-1)!.length;
     const eligible =
       outer &&
       direction !== undefined &&
@@ -175,10 +170,9 @@ export class AgentNavigationEditor extends CustomEditor {
     generation: unknown,
     direction: "left" | "right",
   ): void {
-    const allowed = () =>
-      direction === "left"
-        ? this.navigation.canOpen()
-        : !!this.navigation.collapseWidget && (this.navigation.canCollapse?.() ?? false);
+    const allowed = () => direction === "left"
+      ? this.navigation.canOpen()
+      : !!this.navigation.collapseWidget && (this.navigation.canCollapse?.() ?? false);
     if (this.pending || !allowed()) return;
     this.pending = direction;
     const epoch = this.inputEpoch;
@@ -201,9 +195,8 @@ export class AgentNavigationEditor extends CustomEditor {
       try {
         // Call within the checked microtask: an extra Promise.then would leave
         // a gap in which root ownership could change after validation.
-        void Promise.resolve(
-          direction === "left" ? this.navigation.openTree() : this.navigation.collapseWidget!(),
-        )
+        void Promise.resolve(direction === "left"
+          ? this.navigation.openTree() : this.navigation.collapseWidget!())
           .catch((error: unknown) => this.navigation.onError?.(error))
           .finally(() => {
             this.pending = undefined;

package/src/subagents/ui/live-agent-view.ts

@@ -13,7 +13,7 @@ import {
   wrapTextWithAnsi,
 } from "@earendil-works/pi-tui";
 import type { ThreadService, TranscriptSnapshot } from "../types.ts";
-import { type DialogHost, dialogText } from "./dialog.ts";
+import { dialogText, type DialogHost } from "./dialog.ts";
 import { AGENT_PROGRESS_INTERVAL, agentProgressIcon, agentTypeLabel } from "./ui.ts";
 
 /** Transcript detail level cycled by the t key: full → preview → compact. */
@@ -66,10 +66,7 @@ export class LiveAgentView {
   private unsubscribe: (() => void) | undefined;
   private renderTimer: ReturnType<typeof setTimeout> | undefined;
   private progressTimer: ReturnType<typeof setInterval> | undefined;
-  private cache = new WeakMap<
-    object,
-    { width: number; detail: TranscriptDetail; lines: string[] }
-  >();
+  private cache = new WeakMap<object, { width: number; detail: TranscriptDetail; lines: string[] }>();
   private scroll: ScrollView;
   private content: string[] = [];
   private restoreViewport = true;
@@ -191,10 +188,7 @@ export class LiveAgentView {
 
   private syncProgress(): void {
     const state = this.snapshot?.thread?.state;
-    const active =
-      this.nerdFontIcons &&
-      !this.disposed &&
-      !this.error &&
+    const active = this.nerdFontIcons && !this.disposed && !this.error &&
       (state === "starting" || state === "running");
     if (!active) {
       if (this.progressTimer !== undefined) clearInterval(this.progressTimer);
@@ -339,11 +333,7 @@ export class LiveAgentView {
         if (this.detail === "compact")
           return [this.theme.fg("dim", "[Thinking hidden · t cycles view]")];
         // Thinking is model-authored Markdown; preview truncates the rendered lines, not the source.
-        const rendered = this.markdown(
-          String(block.thinking ?? "[redacted thinking]"),
-          width,
-          true,
-        );
+        const rendered = this.markdown(String(block.thinking ?? "[redacted thinking]"), width, true);
         if (this.detail === "full" || rendered.length <= 3) return rendered;
         return [...rendered.slice(0, 3), this.theme.fg("dim", "...")];
       }
@@ -459,9 +449,7 @@ export class LiveAgentView {
       }
       case "ls": {
         const limit = num("limit");
-        lines = [
-          `${title("ls")} ${path ?? "."}${limit != null ? detail(` (limit ${limit})`) : ""}`,
-        ];
+        lines = [`${title("ls")} ${path ?? "."}${limit != null ? detail(` (limit ${limit})`) : ""}`];
         break;
       }
       default:

package/src/subagents/ui/thread-tree.ts

@@ -25,7 +25,11 @@ function comparePath(a: string, b: string): number {
   return a < b ? -1 : a > b ? 1 : 0;
 }
 
-function chainHas(parentOf: Map<string, string | null>, start: string, target: string): boolean {
+function chainHas(
+  parentOf: Map<string, string | null>,
+  start: string,
+  target: string,
+): boolean {
   const seen = new Set<string>();
   let current: string | null = start;
   while (current) {
@@ -46,7 +50,8 @@ export function buildStatusTree(
 ): StatusRow[] {
   const threadsByPath = new Map<string, ThreadView>();
   for (const thread of threads) {
-    if (thread?.path && !threadsByPath.has(thread.path)) threadsByPath.set(thread.path, thread);
+    if (thread?.path && !threadsByPath.has(thread.path))
+      threadsByPath.set(thread.path, thread);
   }
   const nodes = new Map<string, StatusNode>();
   const parentOf = new Map<string, string | null>();
@@ -61,7 +66,8 @@ export function buildStatusTree(
   const declaredParent = (path: string): string | null => {
     if (path === ROOT) return null;
     const thread = threadsByPath.get(path);
-    if (thread) return thread.parent && thread.parent !== path ? thread.parent : null;
+    if (thread)
+      return thread.parent && thread.parent !== path ? thread.parent : null;
     return lexicalParent(path);
   };
   const ensure = (path: string, seen = new Set<string>()): void => {