@gotgenes/pi-subagents 6.12.1 → 6.13.1

package/docs/plans/0136-decompose-agent-menu.md

@@ -0,0 +1,300 @@
+---
+issue: 136
+issue_title: "Decompose `agent-menu.ts`"
+---
+
+# Decompose agent-menu into config editor, creation wizard, and file-ops abstraction
+
+## Problem Statement
+
+`agent-menu.ts` (668 lines) has 8 distinct responsibilities: menu FSM orchestration, agent listing, config editing, agent ejection, two creation wizards, running-agent viewer, and settings form.
+Filesystem operations (`readFileSync`, `writeFileSync`, `unlinkSync`, `existsSync`, `mkdirSync`) are scattered across 10+ call sites with no abstraction layer, forcing tests to use `vi.mock("node:fs")` rather than injecting stubs.
+
+## Goals
+
+- Extract an `AgentFileOps` interface abstracting all filesystem calls, with a production `FsAgentFileOps` implementation.
+- Extract `ui/agent-config-editor.ts` (~170 lines) containing `showAgentDetail` with eject/disable/enable/edit/delete/reset transitions.
+- Extract `ui/agent-creation-wizard.ts` (~200 lines) containing both the AI-generation and manual-form creation paths.
+- Leave menu FSM, agent listing, running-agent viewer, and settings form in `agent-menu.ts` (~280 lines).
+- Each extracted module receives dependencies via injection — no direct `node:fs` imports outside `FsAgentFileOps`.
+- Enable unit testing of config editor and creation wizard without `vi.mock("node:fs")`.
+
+## Non-Goals
+
+- Refactoring the settings form or running-agent viewer — they stay in `agent-menu.ts`.
+- Changing any user-facing behavior or menu flow.
+- Extracting `showAllAgentsList` — it is menu orchestration (presents the list, delegates to editor for detail).
+- Deduplicating the YAML frontmatter builders in eject and manual wizard (structurally different content shapes).
+
+## Background
+
+### Prerequisite
+
+Issue #135 (extract display helpers) is **implemented** — `ui/display.ts` exists and provides `formatDuration`, `getDisplayName`, and other formatters.
+Extracted menu sub-modules can import display helpers without pulling in the widget.
+
+### Existing IO-injection convention
+
+The codebase uses injectable IO interfaces to decouple domain logic from `node:fs` and the Pi SDK:
+
+- `AssemblerIO` in `session-config.ts` — 4 methods for prompt assembly IO.
+- `RunnerIO` in `agent-runner.ts` — 7 methods for session creation IO.
+
+Both follow the same pattern: interface defined in the module that uses it, production implementation wired at the edge (`index.ts`), test stubs injected directly.
+
+### Current fs call inventory in agent-menu.ts
+
+| Function                         | fs calls                                                      |
+| -------------------------------- | ------------------------------------------------------------- |
+| `findAgentFile`                  | `existsSync` ×2                                               |
+| `showAgentDetail` (edit)         | `readFileSync`, `writeFileSync`                               |
+| `showAgentDetail` (delete/reset) | `unlinkSync`                                                  |
+| `ejectAgent`                     | `mkdirSync`, `existsSync`, `writeFileSync`                    |
+| `disableAgent`                   | `readFileSync`, `writeFileSync`, `mkdirSync`, `writeFileSync` |
+| `enableAgent`                    | `readFileSync`, `writeFileSync`, `unlinkSync`                 |
+| `showGenerateWizard`             | `mkdirSync`, `existsSync` ×2                                  |
+| `showManualWizard`               | `mkdirSync`, `existsSync`, `writeFileSync`                    |
+
+### Current test file
+
+`agent-menu.test.ts` (212 lines) uses `vi.mock("node:fs")` with `vi.hoisted` stubs.
+Only 7 tests exist — they cover the top-level menu, agent listing, projectAgentsDir injection, and settings delegation.
+No tests exercise config-editor transitions (edit/delete/eject/disable/enable) or creation wizards.
+
+## Design Overview
+
+### AgentFileOps interface
+
+A narrow interface abstracting all agent `.md` file operations:
+
+```typescript
+interface AgentFileOps {
+  exists(filePath: string): boolean;
+  read(filePath: string): string | undefined;
+  write(filePath: string, content: string): void;
+  remove(filePath: string): void;
+  ensureDir(dirPath: string): void;
+  findAgentFile(name: string, dirs: string[]): string | undefined;
+}
+```
+
+Design notes:
+
+- `read` returns `undefined` when the file does not exist (wraps `readFileSync` with a try/catch).
+- `write` ensures parent directories exist before writing (internalizes `mkdirSync`).
+- `ensureDir` is needed separately because `showGenerateWizard` creates the directory before spawning an agent that writes via Pi's tool (not via `AgentFileOps.write`).
+- `findAgentFile` takes an ordered list of directories and returns the first matching path.
+  The current code returns `{ path, location }` but only `showAgentDetail` uses `location` (for a confirmation dialog) — the full path already conveys the location to the user, so a plain `string | undefined` return suffices.
+
+### FsAgentFileOps production implementation
+
+Thin wrapper over `node:fs` synchronous APIs:
+
+```typescript
+class FsAgentFileOps implements AgentFileOps {
+  exists(filePath: string): boolean {
+    return existsSync(filePath);
+  }
+  read(filePath: string): string | undefined {
+    try { return readFileSync(filePath, "utf-8"); }
+    catch { return undefined; }
+  }
+  write(filePath: string, content: string): void {
+    mkdirSync(dirname(filePath), { recursive: true });
+    writeFileSync(filePath, content, "utf-8");
+  }
+  remove(filePath: string): void {
+    unlinkSync(filePath);
+  }
+  ensureDir(dirPath: string): void {
+    mkdirSync(dirPath, { recursive: true });
+  }
+  findAgentFile(name: string, dirs: string[]): string | undefined {
+    for (const dir of dirs) {
+      const p = join(dir, `${name}.md`);
+      if (existsSync(p)) return p;
+    }
+    return undefined;
+  }
+}
+```
+
+### Extracted module patterns
+
+Both extracted modules follow the existing factory-function pattern (`createAgentsMenuHandler(deps)`) with narrow deps interfaces per ISP:
+
+Config editor call-site sketch (from orchestrator):
+
+```typescript
+const editor = createAgentConfigEditor({
+  fileOps: deps.fileOps,
+  registry: deps.registry,
+  personalAgentsDir: deps.personalAgentsDir,
+  projectAgentsDir: deps.projectAgentsDir,
+});
+// ...
+await editor.showAgentDetail(ctx, agentName);
+```
+
+Creation wizard call-site sketch (from orchestrator):
+
+```typescript
+const wizard = createAgentCreationWizard({
+  fileOps: deps.fileOps,
+  manager: deps.manager,
+  registry: deps.registry,
+  personalAgentsDir: deps.personalAgentsDir,
+  projectAgentsDir: deps.projectAgentsDir,
+});
+// ...
+await wizard.showCreateWizard(ctx);
+```
+
+### Deps interfaces (ISP-compliant)
+
+```typescript
+// agent-config-editor.ts
+interface AgentConfigEditorDeps {
+  fileOps: AgentFileOps;
+  registry: AgentTypeRegistry;
+  personalAgentsDir: string;
+  projectAgentsDir: string;
+}
+
+// agent-creation-wizard.ts
+interface AgentCreationWizardDeps {
+  fileOps: AgentFileOps;
+  manager: AgentMenuManager;
+  registry: AgentTypeRegistry;
+  personalAgentsDir: string;
+  projectAgentsDir: string;
+}
+```
+
+The config editor does not need `manager` (no agent spawning).
+The creation wizard does not need `agentActivity`, `getModelLabel`, or `settings`.
+
+### Updated AgentMenuDeps
+
+```typescript
+interface AgentMenuDeps {
+  manager: AgentMenuManager;
+  registry: AgentTypeRegistry;
+  agentActivity: AgentActivityReader;
+  getModelLabel: (type: string, registry?: ModelRegistry) => string;
+  settings: AgentMenuSettings;
+  fileOps: AgentFileOps;          // ← new
+  personalAgentsDir: string;
+  projectAgentsDir: string;
+}
+```
+
+The single new field (`fileOps`) replaces all direct `node:fs` imports.
+
+## Module-Level Changes
+
+### New files
+
+1. `src/ui/agent-file-ops.ts` — `AgentFileOps` interface + `FsAgentFileOps` class.
+2. `src/ui/agent-config-editor.ts` — `AgentConfigEditorDeps` interface, `createAgentConfigEditor` factory.
+   Moves in: `showAgentDetail`, `ejectAgent`, `disableAgent`, `enableAgent`.
+3. `src/ui/agent-creation-wizard.ts` — `AgentCreationWizardDeps` interface, `createAgentCreationWizard` factory.
+   Moves in: `showCreateWizard`, `showGenerateWizard`, `showManualWizard`.
+4. `test/ui/agent-file-ops.test.ts` — unit tests for `FsAgentFileOps`.
+5. `test/ui/agent-config-editor.test.ts` — unit tests for config editor transitions with injected `AgentFileOps` stubs.
+6. `test/ui/agent-creation-wizard.test.ts` — unit tests for wizard paths with injected stubs.
+
+### Modified files
+
+1. `src/ui/agent-menu.ts`:
+   - Remove `import { existsSync, mkdirSync, readFileSync, unlinkSync } from "node:fs"`.
+   - Remove `findAgentFile`, `showAgentDetail`, `ejectAgent`, `disableAgent`, `enableAgent`, `showCreateWizard`, `showGenerateWizard`, `showManualWizard` (~375 lines removed).
+   - Add `fileOps: AgentFileOps` to `AgentMenuDeps`.
+   - Import and instantiate `createAgentConfigEditor` and `createAgentCreationWizard`.
+   - Update `showAllAgentsList` to call `editor.showAgentDetail(ctx, agentName)`.
+   - Update `showAgentsMenu` to call `wizard.showCreateWizard(ctx)`.
+   - Re-export `AgentMenuManager` (still needed by the wizard deps).
+   - Net result: ~280 lines (orchestration, listing, running agents, settings).
+2. `src/index.ts`:
+   - Import `FsAgentFileOps` from `./ui/agent-file-ops.js`.
+   - Construct `new FsAgentFileOps()` and pass as `fileOps` in the `createAgentsMenuHandler` call.
+3. `test/ui/agent-menu.test.ts`:
+   - Remove `vi.mock("node:fs")` and `vi.hoisted` stubs entirely.
+   - Add `fileOps` stub to `makeDeps` factory.
+   - Rewrite the "projectAgentsDir injection" test to verify that the orchestrator delegates to the editor (or move to `agent-config-editor.test.ts`).
+
+### Removed symbols
+
+Grep confirms these functions are only referenced within `agent-menu.ts` (closures inside `createAgentsMenuHandler`) — no external consumers:
+
+- `findAgentFile` — moves to `agent-config-editor.ts` (reimplemented via `AgentFileOps.findAgentFile`).
+- `showAgentDetail`, `ejectAgent`, `disableAgent`, `enableAgent` — move to `agent-config-editor.ts`.
+- `showCreateWizard`, `showGenerateWizard`, `showManualWizard` — move to `agent-creation-wizard.ts`.
+
+## Test Impact Analysis
+
+### New unit tests enabled by extraction
+
+1. **Config editor transitions** — `showAgentDetail` has 6 menu paths (edit, delete, reset, eject, disable, enable) with sub-branches (confirm/cancel, file exists/not-exists, default/custom agent).
+   Currently untestable in isolation because the functions are closures inside `createAgentsMenuHandler`.
+   After extraction, each transition is testable with injected `AgentFileOps` stubs — no `vi.mock("node:fs")` needed.
+2. **Creation wizard flows** — generate wizard (spawn + check result) and manual wizard (multi-step form → write file) are currently untested.
+   After extraction, both are testable with injected stubs for `AgentFileOps` and `AgentMenuManager.spawnAndWait`.
+3. **FsAgentFileOps** — thin tests verifying the production fs wrapper (read returns undefined on missing file, write ensures parent dirs, findAgentFile checks directories in order).
+
+### Existing tests that become redundant
+
+The "projectAgentsDir injection" test (`agent-menu.test.ts`) navigates through the main menu → agent types → agent detail, then asserts `mockExistsSync` was called with the correct path.
+After extraction, this end-to-end path tests orchestration + editor together; a focused test in `agent-config-editor.test.ts` replaces it.
+The orchestrator test can be simplified to verify it delegates to the editor.
+
+### Existing tests that stay as-is
+
+- Menu structure tests (shows options, reload, running agents) — these test the orchestrator directly.
+- Settings delegation tests — these test code that remains in `agent-menu.ts`.
+
+## TDD Order
+
+### Cycle 1: AgentFileOps interface and FsAgentFileOps
+
+1. `test:` write `test/ui/agent-file-ops.test.ts` — tests for `read` (existing file, missing file), `write` (ensures parent dir), `remove`, `exists`, `ensureDir`, `findAgentFile` (first-match ordering, no match).
+2. `feat:` implement `src/ui/agent-file-ops.ts` — interface + `FsAgentFileOps` class.
+3. Commit: `feat: add AgentFileOps interface and FsAgentFileOps (#136)`.
+
+### Cycle 2: Extract agent-config-editor
+
+1. `test:` write `test/ui/agent-config-editor.test.ts` — tests for `showAgentDetail` transitions: edit (save/cancel), delete (confirm/cancel), reset-to-default (confirm/cancel), eject (project/personal location, overwrite check), disable (existing file toggle, new disable-only file), enable (remove enabled:false line, remove empty override file).
+   All tests inject stub `AgentFileOps` — no `vi.mock`.
+2. `refactor:` create `src/ui/agent-config-editor.ts` — move `showAgentDetail`, `ejectAgent`, `disableAgent`, `enableAgent` from `agent-menu.ts`.
+   Replace direct fs calls with `deps.fileOps.*` calls.
+   Replace the closure `findAgentFile` with `deps.fileOps.findAgentFile(name, [deps.projectAgentsDir, deps.personalAgentsDir])`.
+3. `refactor:` update `src/ui/agent-menu.ts` — add `fileOps` to `AgentMenuDeps`, import `createAgentConfigEditor`, wire into `showAllAgentsList`.
+4. `refactor:` update `src/index.ts` — import `FsAgentFileOps`, pass `fileOps: new FsAgentFileOps()` in deps.
+5. `test:` update `test/ui/agent-menu.test.ts` — add `fileOps` stub to `makeDeps`, rewrite the "projectAgentsDir injection" test to verify orchestrator→editor delegation.
+6. Run `pnpm run check` (interface change).
+7. Commit: `refactor: extract agent-config-editor from agent-menu (#136)`.
+
+### Cycle 3: Extract agent-creation-wizard
+
+1. `test:` write `test/ui/agent-creation-wizard.test.ts` — tests for `showCreateWizard` (location + method selection), `showGenerateWizard` (spawn success, spawn error, overwrite check), `showManualWizard` (full form flow, overwrite check, tool/model/thinking selections).
+   All tests inject stub `AgentFileOps` + `AgentMenuManager` — no `vi.mock`.
+2. `refactor:` create `src/ui/agent-creation-wizard.ts` — move `showCreateWizard`, `showGenerateWizard`, `showManualWizard` from `agent-menu.ts`.
+   Replace direct fs calls with `deps.fileOps.*` calls.
+3. `refactor:` update `src/ui/agent-menu.ts` — import `createAgentCreationWizard`, wire into `showAgentsMenu`.
+   Remove the `import { ... } from "node:fs"` line (no longer needed).
+4. `test:` remove `vi.mock("node:fs")` and `vi.hoisted` stubs from `test/ui/agent-menu.test.ts` entirely.
+5. Run `pnpm run check`.
+6. Commit: `refactor: extract agent-creation-wizard from agent-menu (#136)`.
+
+## Risks and Mitigations
+
+| Risk                                                                                                      | Mitigation                                                                                                                                                                                            |
+| --------------------------------------------------------------------------------------------------------- | ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| Circular dependency between orchestrator and editor (editor calling back to menu)                         | The editor's `showAgentDetail` does not recurse to the agent list — the orchestrator does (`showAllAgentsList` calls editor then recurses itself). No circular dependency.                            |
+| `FsAgentFileOps.write` silently creates parent directories when callers expect the directory to not exist | `write` mirrors current behavior — every call site already calls `mkdirSync` before `writeFileSync`. The consolidation only changes where the `mkdirSync` happens, not whether it runs.               |
+| Adding `fileOps` to `AgentMenuDeps` breaks existing test factory                                          | Cycle 2 updates `makeDeps` in the same commit — the interface change and call-site update land together per the testing skill's single-call-site rule.                                                |
+| Generate wizard spawns an agent that writes via Pi tools, not via `AgentFileOps.write`                    | The wizard uses `fileOps.ensureDir` for directory creation and `fileOps.exists` for the post-spawn success check. The spawned agent's file write is outside the menu's control and is not abstracted. |
+
+## Open Questions
+
+- None — the issue's proposed changes section is unambiguous and the prerequisite (#135) is already implemented.

package/docs/retro/0135-extract-display-helpers.md

@@ -0,0 +1,38 @@
+---
+issue: 135
+issue_title: "Extract display helpers from `agent-widget.ts`"
+---
+
+# Retro: #135 — Extract display helpers from agent-widget.ts
+
+## Final Retrospective (2026-05-22T19:00:00Z)
+
+### Session summary
+
+Extracted 11 helper functions, 3 constants, and 2 types from `agent-widget.ts` into a new `ui/display.ts` module.
+Updated 10 source consumers and 2 test consumers.
+Pure code-motion refactoring — no behavior change, no test-count delta (714 tests throughout).
+
+### Observations
+
+#### What went well
+
+- Plan accurately identified all 10 source and 2 test import sites with no misses — the consumer import table in the plan mapped 1:1 to actual changes.
+- TDD execution was mechanical and smooth: create module → update source imports → rename test file → verify.
+  Zero surprises or deviations from the plan.
+
+#### What caused friction (agent side)
+
+- `rabbit-hole` — During `/ship-issue`, wasted ~6 tool calls investigating whether `pi-subagents-v6.12.0` was at HEAD.
+  Ran `git log --oneline HEAD --not --remotes=origin/main` which dumped the entire repo history (50KB truncation), then misread `git describe --tags --abbrev=0` (nearest ancestor tag) as confirming the tag was at HEAD.
+  `git tag --points-at HEAD` returned empty, disproving the assumption, but I still spent cycles reasoning about CI release-please behavior.
+  Impact: added friction but no rework — the close comment and release-please merge were correct.
+
+#### What caused friction (user side)
+
+- None observed.
+  The issue was unambiguous, the architecture doc prescribed the exact extraction set, and no user intervention was needed during implementation.
+
+### Changes made
+
+1. Created `packages/pi-subagents/docs/retro/0135-extract-display-helpers.md` (this file).

package/docs/retro/0136-decompose-agent-menu.md

@@ -0,0 +1,43 @@
+---
+issue: 136
+issue_title: "Decompose `agent-menu.ts`"
+---
+
+# Retro: #136 — Decompose `agent-menu.ts`
+
+## Final Retrospective (2026-05-22T20:10:00-04:00)
+
+### Session summary
+
+Decomposed `agent-menu.ts` (668 lines) into four focused modules: `agent-file-ops.ts`, `agent-config-editor.ts`, `agent-creation-wizard.ts`, and a slimmed-down `agent-menu.ts` (296 lines).
+Three TDD cycles shipped cleanly, adding 47 tests (714 → 761) and eliminating `vi.mock("node:fs")` from the menu test file.
+Released as `pi-subagents-v6.13.0`.
+
+### Observations
+
+#### What went well
+
+- The three-cycle TDD plan (file-ops → config-editor → creation-wizard) produced clean incremental commits with no rework.
+  Each cycle left the repo green for both tests and type-check.
+- The creation wizard naturally produced narrower interfaces (`WizardManager`, `WizardRegistry`) than the plan specified — a positive deviation from the plan's `AgentTypeRegistry` concrete type, following ISP more strictly.
+- The large edit removing ~170 lines of extracted functions from `agent-menu.ts` in Cycle 2 landed correctly on the first attempt, thanks to using exact `oldText` matching with the full function bodies.
+
+#### What caused friction (agent side)
+
+- `missing-context` — The config-editor test factory used `Partial<AgentConfigEditorDeps>` for the overrides parameter.
+  The `...overrides` spread created a union type that erased `Mock<...>` methods from `fileOps`, producing 28 `TS2339` errors on `pnpm run check`.
+  The testing skill warns about return-type annotations but not about `Partial<Interface>` in overrides — the same erasure mechanism applies through a different path.
+  Impact: one extra edit-check cycle to remove the `Partial<>` annotation and overrides parameter.
+  Self-identified (caught on `pnpm run check` before commit).
+
+- `missing-context` — The config-editor test had an unused `ctx` variable from an earlier draft of the "disable-only file" test, caught only by the linter after the Cycle 3 commit.
+  Impact: required amending the Cycle 3 commit; added friction but no rework.
+  Self-identified (caught by `pnpm run lint`).
+
+#### What caused friction (user side)
+
+- None observed — the user's plan was unambiguous and the prerequisite (#135) was already implemented.
+
+### Changes made
+
+1. Added a `Partial<Interface>` type-erasure bullet to `.pi/skills/testing/SKILL.md`.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gotgenes/pi-subagents",
-  "version": "6.12.1",
+  "version": "6.13.1",
   "exports": {
     ".": "./src/service.ts"
   },

package/src/index.ts

@@ -41,6 +41,7 @@ import { createAgentTool } from "./tools/agent-tool.js";
 import { createGetResultTool } from "./tools/get-result-tool.js";
 import { getModelLabelFromConfig } from "./tools/helpers.js";
 import { createSteerTool } from "./tools/steer-tool.js";
+import { FsAgentFileOps } from "./ui/agent-file-ops.js";
 import { createAgentsMenuHandler } from "./ui/agent-menu.js";
 import {
   AgentWidget,
@@ -248,6 +249,7 @@ export default function (pi: ExtensionAPI) {
       return getModelLabelFromConfig(cfg.model);
     },
     settings,
+    fileOps: new FsAgentFileOps(),
     personalAgentsDir: join(getAgentDir(), 'agents'),
     projectAgentsDir: join(process.cwd(), '.pi', 'agents'),
   });

package/src/ui/agent-config-editor.ts

@@ -0,0 +1,202 @@
+/**
+ * agent-config-editor.ts — Agent detail view with edit/delete/eject/disable/enable transitions.
+ *
+ * Extracted from agent-menu.ts to give each concern a single responsibility.
+ * Receives dependencies via injection — no direct `node:fs` imports.
+ */
+
+import { join } from "node:path";
+
+import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
+import type { AgentTypeRegistry } from "../agent-types.js";
+import type { AgentConfig } from "../types.js";
+import type { AgentFileOps } from "./agent-file-ops.js";
+
+// ---- Deps interface ----
+
+export interface AgentConfigEditorDeps {
+  fileOps: AgentFileOps;
+  registry: AgentTypeRegistry;
+  personalAgentsDir: string;
+  projectAgentsDir: string;
+}
+
+// ---- Factory ----
+
+export function createAgentConfigEditor(deps: AgentConfigEditorDeps) {
+  function agentDirs(): string[] {
+    return [deps.projectAgentsDir, deps.personalAgentsDir];
+  }
+
+  async function showAgentDetail(ctx: ExtensionContext, name: string) {
+    if (deps.registry.resolveType(name) == null) {
+      ctx.ui.notify(`Agent config not found for "${name}".`, "warning");
+      return;
+    }
+    const cfg = deps.registry.resolveAgentConfig(name);
+
+    const file = deps.fileOps.findAgentFile(name, agentDirs());
+    const isDefault = cfg.isDefault === true;
+    const disabled = cfg.enabled === false;
+
+    let menuOptions: string[];
+    if (disabled && file) {
+      menuOptions = isDefault
+        ? ["Enable", "Edit", "Reset to default", "Delete", "Back"]
+        : ["Enable", "Edit", "Delete", "Back"];
+    } else if (isDefault && !file) {
+      menuOptions = ["Eject (export as .md)", "Disable", "Back"];
+    } else if (isDefault && file) {
+      menuOptions = ["Edit", "Disable", "Reset to default", "Delete", "Back"];
+    } else {
+      menuOptions = ["Edit", "Disable", "Delete", "Back"];
+    }
+
+    const choice = await ctx.ui.select(name, menuOptions);
+    if (!choice || choice === "Back") return;
+
+    if (choice === "Edit" && file) {
+      const content = deps.fileOps.read(file);
+      if (content !== undefined) {
+        const edited = await ctx.ui.editor(`Edit ${name}`, content);
+        if (edited !== undefined && edited !== content) {
+          deps.fileOps.write(file, edited);
+          deps.registry.reload();
+          ctx.ui.notify(`Updated ${file}`, "info");
+        }
+      }
+    } else if (choice === "Delete") {
+      if (file) {
+        const confirmed = await ctx.ui.confirm(
+          "Delete agent",
+          `Delete ${name} (${file})?`,
+        );
+        if (confirmed) {
+          deps.fileOps.remove(file);
+          deps.registry.reload();
+          ctx.ui.notify(`Deleted ${file}`, "info");
+        }
+      }
+    } else if (choice === "Reset to default" && file) {
+      const confirmed = await ctx.ui.confirm(
+        "Reset to default",
+        `Delete override ${file} and restore embedded default?`,
+      );
+      if (confirmed) {
+        deps.fileOps.remove(file);
+        deps.registry.reload();
+        ctx.ui.notify(`Restored default ${name}`, "info");
+      }
+    } else if (choice.startsWith("Eject")) {
+      await ejectAgent(ctx, name, cfg);
+    } else if (choice === "Disable") {
+      await disableAgent(ctx, name);
+    } else if (choice === "Enable") {
+      await enableAgent(ctx, name);
+    }
+  }
+
+  async function ejectAgent(ctx: ExtensionContext, name: string, cfg: AgentConfig) {
+    const location = await ctx.ui.select("Choose location", [
+      "Project (.pi/agents/)",
+      `Personal (${deps.personalAgentsDir})`,
+    ]);
+    if (!location) return;
+
+    const targetDir = location.startsWith("Project")
+      ? deps.projectAgentsDir
+      : deps.personalAgentsDir;
+
+    const targetPath = join(targetDir, `${name}.md`);
+    if (deps.fileOps.exists(targetPath)) {
+      const overwrite = await ctx.ui.confirm(
+        "Overwrite",
+        `${targetPath} already exists. Overwrite?`,
+      );
+      if (!overwrite) return;
+    }
+
+    const fmFields: string[] = [];
+    fmFields.push(`description: ${cfg.description}`);
+    if (cfg.displayName) fmFields.push(`display_name: ${cfg.displayName}`);
+    fmFields.push(`tools: ${cfg.builtinToolNames?.join(", ") || "all"}`);
+    if (cfg.model) fmFields.push(`model: ${cfg.model}`);
+    if (cfg.thinking) fmFields.push(`thinking: ${cfg.thinking}`);
+    if (cfg.maxTurns) fmFields.push(`max_turns: ${cfg.maxTurns}`);
+    fmFields.push(`prompt_mode: ${cfg.promptMode}`);
+    if (cfg.extensions === false) fmFields.push("extensions: false");
+    else if (Array.isArray(cfg.extensions))
+      fmFields.push(`extensions: ${cfg.extensions.join(", ")}`);
+    if (cfg.skills === false) fmFields.push("skills: false");
+    else if (Array.isArray(cfg.skills))
+      fmFields.push(`skills: ${cfg.skills.join(", ")}`);
+    if (cfg.disallowedTools?.length)
+      fmFields.push(`disallowed_tools: ${cfg.disallowedTools.join(", ")}`);
+    if (cfg.inheritContext) fmFields.push("inherit_context: true");
+    if (cfg.runInBackground) fmFields.push("run_in_background: true");
+    if (cfg.isolated) fmFields.push("isolated: true");
+    if (cfg.memory) fmFields.push(`memory: ${cfg.memory}`);
+    if (cfg.isolation) fmFields.push(`isolation: ${cfg.isolation}`);
+
+    const content = `---\n${fmFields.join("\n")}\n---\n\n${cfg.systemPrompt}\n`;
+
+    deps.fileOps.write(targetPath, content);
+    deps.registry.reload();
+    ctx.ui.notify(`Ejected ${name} to ${targetPath}`, "info");
+  }
+
+  async function disableAgent(ctx: ExtensionContext, name: string) {
+    const file = deps.fileOps.findAgentFile(name, agentDirs());
+    if (file) {
+      const content = deps.fileOps.read(file);
+      if (content?.includes("\nenabled: false\n")) {
+        ctx.ui.notify(`${name} is already disabled.`, "info");
+        return;
+      }
+      if (content) {
+        const updated = content.replace(/^---\n/, "---\nenabled: false\n");
+        deps.fileOps.write(file, updated);
+        deps.registry.reload();
+        ctx.ui.notify(`Disabled ${name} (${file})`, "info");
+      }
+      return;
+    }
+
+    const location = await ctx.ui.select("Choose location", [
+      "Project (.pi/agents/)",
+      `Personal (${deps.personalAgentsDir})`,
+    ]);
+    if (!location) return;
+
+    const targetDir = location.startsWith("Project")
+      ? deps.projectAgentsDir
+      : deps.personalAgentsDir;
+
+    const targetPath = join(targetDir, `${name}.md`);
+    deps.fileOps.write(targetPath, "---\nenabled: false\n---\n");
+    deps.registry.reload();
+    ctx.ui.notify(`Disabled ${name} (${targetPath})`, "info");
+  }
+
+  async function enableAgent(ctx: ExtensionContext, name: string) {
+    const file = deps.fileOps.findAgentFile(name, agentDirs());
+    if (!file) return;
+
+    const content = deps.fileOps.read(file);
+    if (!content) return;
+
+    const updated = content.replace(/^(---\n)enabled: false\n/, "$1");
+
+    if (updated.trim() === "---\n---" || updated.trim() === "---\n---\n") {
+      deps.fileOps.remove(file);
+      deps.registry.reload();
+      ctx.ui.notify(`Enabled ${name} (removed ${file})`, "info");
+    } else {
+      deps.fileOps.write(file, updated);
+      deps.registry.reload();
+      ctx.ui.notify(`Enabled ${name} (${file})`, "info");
+    }
+  }
+
+  return { showAgentDetail };
+}