pi-pier 0.3.0 → 0.3.1

package/src/subagent-gc.ts

@@ -1,30 +1,87 @@
 /**
- * Tab/pane collection and isolate-worktree sweep.
- *
- * Why: GC mixed herdr close I/O with the plugin body. Predicates stay in
- * gc-core.ts; this adapter owns list/close, isolate git, and the ticker.
+ * Subagent board hygiene: registry projection + recovery, tab/pane collection, isolate-worktree sweep.
+ * Predicates live in gc-core.ts; this adapter owns herdr close I/O, isolate git commands, and the ticker.
  */
 import { appendFileSync, existsSync, rmSync } from 'node:fs';
 import type { Context } from '@deepseek-ai/cordis';
 import type { HerdrClientLike } from './herdr-client.ts';
-import { shouldClosePane, shouldCloseTaskTab } from './gc-core.ts';
+import { isPathInside, planIsolateSweep, shouldClosePane, shouldCloseTaskTab } from './gc-core.ts';
 import { runtimePolicy } from './runtime-policy.ts';
-import { evaluateRelease, parseWorktreePorcelain, type SubEntry } from './subagent-core.ts';
-import { planIsolateSweep, isPathInside } from './gc-core.ts';
+import { swallow } from './swallow.ts';
+import { evaluateRelease, foldSubsRegistry, makeRegistry, parseWorktreePorcelain, sleep, SUBS_CUSTOM_TYPE, type SubEntry } from './subagent-core.ts';
+import type { GitIo } from './subagent-spawn.ts';
+import type { TerminalStateSlot } from './plugins/terminal.ts';
 
-interface Cand { branch: string; wtPath: string; entry: SubEntry | null }
-import type { GitIo } from './subagent-git-io.ts';
-import type { TerminalStateSlot } from './core/terminal.ts';
+/* ── registry projection + recovery ─────────────────────────────── */
 
-function sleep(ms: number): Promise<void> {
-  const wait = Promise.withResolvers<void>();
-  setTimeout(wait.resolve, ms);
-  return wait.promise;
+interface SubagentRegistryHost {
+  pi: { appendEntry?: (customType: string, data: unknown) => void };
+  client: HerdrClientLike;
+  subs: Map<string, SubEntry>;
+  writeHistory: (entry: SubEntry, patch?: { status?: SubEntry['status']; closedAt?: number }, via?: string) => void;
+}
+
+interface SubagentRegistry {
+  /** Append the registry snapshot to the session branch; duplicate snapshots are skipped. */
+  persist(): void;
+  rebuild(eventCtx: unknown): void;
+  sweepZombieRunning(): Promise<void>;
+}
+
+export function createSubagentRegistry(host: SubagentRegistryHost): SubagentRegistry {
+  let lastSnapshot = '';
+
+  function persist(): void {
+    try {
+      const registry = makeRegistry([...host.subs.values()]);
+      const snapshot = JSON.stringify(registry);
+      if (snapshot === lastSnapshot) return;
+      lastSnapshot = snapshot;
+      host.pi.appendEntry?.(SUBS_CUSTOM_TYPE, registry);
+    } catch (err) {
+      // Session logging must not break delegation; a silent failure here shows up as a ghost running row.
+      swallow('subagent.persist-subs', err);
+    }
+  }
+
+  function rebuild(eventCtx: unknown): void {
+    try {
+      const entries = (eventCtx as { sessionManager?: { getBranch?: () => readonly unknown[] } })
+        ?.sessionManager?.getBranch?.() ?? [];
+      const registry = foldSubsRegistry(entries as Parameters<typeof foldSubsRegistry>[0]);
+      for (const entry of registry.subs) host.subs.set(entry.paneId, entry);
+      lastSnapshot = ''; // branch replay replaced live state; force the next persist
+    } catch {
+      /* registry recovery failure must not block the live session */
+    }
+  }
+
+  async function sweepZombieRunning(): Promise<void> {
+    if (!host.client.available || host.subs.size === 0) return;
+    let livePaneIds: ReadonlySet<string>;
+    try {
+      livePaneIds = new Set((await host.client.listPanes()).map((pane) => pane.paneId));
+    } catch {
+      return; // do not close agents when the liveness lookup itself failed
+    }
+    let changed = false;
+    for (const [paneId, entry] of host.subs) {
+      if (entry.status !== 'running' || livePaneIds.has(paneId)) continue;
+      entry.status = 'closed';
+      host.writeHistory(entry, { status: 'closed', closedAt: Date.now() }, 'zombie-sweep');
+      changed = true;
+    }
+    if (changed) persist();
+  }
+
+  return { persist, rebuild, sweepZombieRunning };
 }
 
-export interface GcHost {
+/* ── GC ─────────────────────────────────────────────────────────── */
+
+interface GcHost {
   client: HerdrClientLike;
-  env: { tabId: string } | null;
+  env: { tabId: string; paneId?: string } | null;
   subs: Map<string, SubEntry>;
   persistSubs(): void;
   writeHistory(e: SubEntry, patch?: { outcome?: string | null; status?: SubEntry['status']; closedAt?: number }, via?: string): void;
@@ -35,7 +92,7 @@ export interface GcHost {
   injectNotice(content: string): Promise<void>;
 }
 
-export interface GcController {
+interface GcController {
   runGcSafely(): Promise<void>;
   onTurnStart(): Promise<void>;
   startTicker(ctx: Context): void;
@@ -66,66 +123,58 @@ export function createGcController(h: GcHost): GcController {
       arr.push(e);
       byTab.set(e.tabId, arr);
     }
-    const taskTabIds = new Set(byTab.keys());
     const mainTabId = h.env?.tabId ?? '';
 
+    const closeRow = (e: SubEntry): void => {
+      if (e.status === 'closed') return;
+      e.status = 'closed';
+      h.writeHistory(e, { status: 'closed', closedAt: Date.now() }, 'gc');
+    };
+
     for (const [tabId, entries] of byTab) {
       if (tabId === mainTabId) continue;
       const tabPanes = panesList.filter((p) => p.tabId === tabId);
       if (tabPanes.length === 0) {
-        for (const e of entries) {
-          if (e.status !== 'closed') {
-            e.status = 'closed';
-            h.writeHistory(e, { status: 'closed', closedAt: Date.now() }, 'gc');
-          }
-        }
+        for (const e of entries) closeRow(e);
         continue;
       }
-      const should = shouldCloseTaskTab({
+      if (!autoCloseTabs) continue;
+      if (!shouldCloseTaskTab({
         entries,
         paneStatuses: tabPanes.map((p) => p.agentStatus),
         ttlMs,
         now: Date.now(),
-      });
-      if (!autoCloseTabs || !should) continue;
+      })) continue;
+      // Live terminal panes and not-yet-delivered settlement notices keep the tab alive.
       if (tabPanes.some((p) => termPaneIds.has(p.paneId) || pendingNoticeIds.has(p.paneId))) continue;
       try {
         await h.client.tabClose(tabId);
-      } catch {
-        /* tab may already be gone */
-      }
-      for (const e of entries) {
-        if (e.status !== 'closed') {
-          e.status = 'closed';
-          h.writeHistory(e, { status: 'closed', closedAt: Date.now() }, 'gc');
-        }
-      }
+      } catch { /* tab may already be gone */ }
+      for (const e of entries) closeRow(e);
       await sleep(300);
     }
 
-    const closableTaskTabIds = new Set([...taskTabIds].filter((t) => t !== mainTabId));
+    const closableTaskTabIds = new Set([...byTab.keys()].filter((t) => t !== mainTabId));
     const candidates = [...h.subs.values()].filter(
       (e) => e.status === 'consumed' && !(e.tabId && closableTaskTabIds.has(e.tabId)),
     );
     for (const e of candidates) {
       if (termPaneIds.has(e.paneId) || pendingNoticeIds.has(e.paneId)) continue;
+      // Never collect self, whatever the registry says: a poisoned entry once closed the master's pane.
+      if (h.env?.paneId && e.paneId === h.env.paneId) continue;
       if (!shouldClosePane({
         consumedAt: e.consumedAt ?? null,
         herdrStatus: statuses.get(e.paneId),
         prevTurnStart,
       })) continue;
       if (statuses.get(e.paneId) === undefined) {
-        e.status = 'closed';
-        h.writeHistory(e, { status: 'closed', closedAt: Date.now() }, 'gc');
+        closeRow(e);
         continue;
       }
       try {
         await h.client.closePane(e.paneId);
-      } catch {
-        /* pane may already be gone */
-      }
-      e.status = 'closed';
-      h.writeHistory(e, { status: 'closed', closedAt: Date.now() }, 'gc');
+      } catch { /* pane may already be gone */ }
+      closeRow(e);
       await sleep(300);
     }
     h.persistSubs();
@@ -143,12 +192,11 @@ export function createGcController(h: GcHost): GcController {
     const wtPorcelain = await h.git.runGit(masterCwd, ['worktree', 'list', '--porcelain']);
     if (wtPorcelain === null) return;
     const wtByBranch = parseWorktreePorcelain(wtPorcelain);
-    // Session-owned isolates: settled or consumed entries whose worktree was not released yet.
     const sessionOwned = [...h.subs.values()]
       .filter((e) => e.isolate && e.isolate.releasedAt == null && e.status !== 'running')
       .map((e) => ({ branch: e.isolate!.branch, worktreePath: e.isolate!.worktreePath }));
-    // Untracked `pier/*` branches are only swept on explicit request: they may belong to
-    // another session (whose process may even be running inside them).
+    // Untracked `pier/*` branches are only swept on explicit request: they may belong to another
+    // session whose process is even running inside them.
     const sweepOrphans = process.env.PIER_ISOLATE_SWEEP_ORPHANS === '1';
     const branches = sweepOrphans
       ? (await h.git.runGit(masterCwd, ['for-each-ref', '--format=%(refname:short)', 'refs/heads/pier/']))
@@ -167,14 +215,12 @@ export function createGcController(h: GcHost): GcController {
     for (const e of h.subs.values()) {
       if (e.isolate && e.isolate.releasedAt == null && e.status !== 'running') entryByBranch.set(e.isolate.branch, e);
     }
-    const byBranch = new Map<string, Cand>();
-    for (const cand of plan.candidates) {
-      byBranch.set(cand.branch, { branch: cand.branch, wtPath: cand.worktreePath, entry: entryByBranch.get(cand.branch) ?? null });
-    }
-    trace?.(`cands=${[...byBranch.keys()].join(',') || 'none'} skipped=${plan.skipped.map((s) => `${s.branch}:${s.reason}`).join(',') || 'none'} subs=${[...h.subs.values()].map((s) => `${s.status}${s.isolate ? '/iso' : ''}`).join(',') || 'none'}`);
+    trace?.(`cands=${plan.candidates.map((c) => c.branch).join(',') || 'none'} skipped=${plan.skipped.map((s) => `${s.branch}:${s.reason}`).join(',') || 'none'} subs=${[...h.subs.values()].map((s) => `${s.status}${s.isolate ? '/iso' : ''}`).join(',') || 'none'}`);
     let persisted = false;
-    for (const cand of byBranch.values()) {
-      const { branch, wtPath, entry } = cand;
+    for (const cand of plan.candidates) {
+      const { branch, worktreePath: wtPath } = cand;
+      const entry = entryByBranch.get(branch) ?? null;
+      // The worktree is already gone (removed by hand or by a previous pass) → only release the row.
       if (entry && !wtByBranch.has(branch)) {
         if (existsSync(wtPath)) {
           try { rmSync(wtPath, { recursive: true, force: true }); } catch { continue; }
@@ -185,9 +231,10 @@ export function createGcController(h: GcHost): GcController {
         continue;
       }
       const mergedOut = await h.git.runGit(masterCwd, ['merge-base', '--is-ancestor', branch, 'HEAD']);
-      const merged = mergedOut !== null ? true : null;
-      let mergedFinal = merged;
-      if (merged === null) {
+      let mergedFinal: boolean | null = mergedOut !== null ? true : null;
+      if (mergedFinal === null) {
+        // A non-zero exit means "not an ancestor" OR "unknown branch": only a resolving ref
+        // makes it unmerged; otherwise the decision stays 'unknown' and the worktree is retained.
         const sha = await h.git.runGit(masterCwd, ['rev-parse', branch]);
         const headSha = await h.git.runGit(masterCwd, ['rev-parse', 'HEAD']);
         if (sha != null && headSha != null) mergedFinal = false;
@@ -196,11 +243,10 @@ export function createGcController(h: GcHost): GcController {
       const dirtyCount = dirtyOut === null ? null : dirtyOut.split('\n').filter((l) => l.trim() !== '').length;
       const decision = evaluateRelease({ merged: mergedFinal, dirtyCount });
       if (decision.action === 'release') {
-        // Defense in depth: planner already refuses the current worktree, but a removal here
-        // must never be able to delete the directory this process is running in.
+        // Defense in depth: the planner already refuses the current worktree, but a removal here
+        // must never delete the directory this process is running in.
         if (isPathInside(masterCwd, wtPath)) continue;
-        const removed = await h.git.runGit(masterCwd, ['worktree', 'remove', wtPath]);
-        let ok = removed !== null;
+        let ok = (await h.git.runGit(masterCwd, ['worktree', 'remove', wtPath])) !== null;
         if (!ok) {
           await sleep(2000);
           ok = (await h.git.runGit(masterCwd, ['worktree', 'remove', wtPath])) !== null;
@@ -227,14 +273,10 @@ export function createGcController(h: GcHost): GcController {
     gcRunning = true;
     try {
       await gcPass();
-    } catch {
-      /* next turn/tick retries */
-    }
+    } catch { /* next turn/tick retries */ }
     try {
       await isolateSweep();
-    } catch {
-      /* next turn/tick retries */
-    } finally {
+    } catch { /* next turn/tick retries */ } finally {
       gcRunning = false;
     }
   }