@esso0428/pi-subagents 0.17.35 → 0.17.37

package/CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.17.37] - 2026-10-05
+
+### Fixed
+- **The `/agents` menu no longer undercounts the project-wide history**: the entry advertised the in-memory record count while the list it opened also merged records recovered from older session files, so the label read 150 against 230 rows. Both now read one cached, filtered legacy set per cwd — cached because the scan is not cheap, and shared because separate reads would let the count and the list drift apart again.
+
+## [0.17.36] - 2026-10-05
+
+### Fixed
+- **Legacy session history no longer lists records that cannot be opened**: recovering `subagents:record` entries from older sessions bypassed the `canOpenAgentHistory` check that the this-session list, the project-wide list, and the roster all apply, so entries with no durable transcript were offered and could only ever answer "No agent history." They now clear the same bar, which drops roughly half of what the scan recovers.
+
 ## [0.17.35] - 2026-10-05
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@esso0428/pi-subagents",
-  "version": "0.17.35",
+  "version": "0.17.37",
   "description": "A pi extension that brings smart Claude Code-style autonomous sub-agents to pi, with npm:pi-subagents-style JSON agent overrides.",
   "author": "ESSO0428",
   "repository": {

package/src/agent-history-list.ts

@@ -83,6 +83,8 @@ export function buildAgentStatusMenuEntries(
   records: readonly AgentRecord[],
   cwd: string | undefined,
   sessionId?: string,
+  /** Recovered legacy records, already filtered to the openable ones. */
+  legacy: readonly AgentRecord[] = [],
 ): string[] {
   const { active, history } = splitAgentRecords(records, cwd);
   // This-session history is what you almost always want, so it is the default
@@ -91,9 +93,14 @@ export function buildAgentStatusMenuEntries(
   // branch or checkpoint are stamped with the current session, so an inherited
   // agent counts as present here rather than disappearing into the other bucket.
   const inSession = sessionId ? history.filter((record) => record.sessionId === sessionId) : [];
+  // Legacy records are merged into this list when it opens, so the count has to
+  // include the ones the in-memory set does not already carry — otherwise the
+  // menu advertises fewer entries than the list actually shows.
+  const inMemory = new Set(history.map((record) => record.id));
+  const allSessions = history.length + legacy.filter((record) => !inMemory.has(record.id)).length;
   return [
     ...(active.length > 0 ? [`Running agents (${active.length})`] : []),
     ...(inSession.length > 0 ? [`Agent history this session (${inSession.length})`] : []),
-    ...(history.length > 0 ? [`Agent history all sessions (${history.length})`] : []),
+    ...(allSessions > 0 ? [`Agent history all sessions (${allSessions})`] : []),
   ];
 }

package/src/index.ts

@@ -675,7 +675,25 @@ export default function (pi: ExtensionAPI) {
     }
   }
 
+  /**
+   * Legacy `subagents:record` entries recovered from session files, per cwd.
+   *
+   * The scan is not cheap, and both the menu's counts and the history list need
+   * it — reading it twice would double the cost on every `/agents` visit, and a
+   * stale count in the menu is worse than a slow one.
+   */
+  const legacyHistoryCache = new Map<string, AgentRecord[]>();
+  async function legacyHistoryFor(ctx: ExtensionCommandContext): Promise<AgentRecord[]> {
+    const key = ctx.cwd;
+    const cached = legacyHistoryCache.get(key);
+    if (cached) return cached;
+    const records = (await readLegacySessionRecords(key)).filter(record => canOpenAgentHistory(record, key));
+    legacyHistoryCache.set(key, records);
+    return records;
+  }
+
   function setWidgetMode(m: WidgetMode): void {
+    legacyHistoryCache.clear();
     widgetMode = m;
     widget.update();
   }
@@ -1701,7 +1719,8 @@ Terse command-style prompts produce shallow, generic work.
     // Keep active agents and terminal history in separate menu entries.
     const records = manager.listAgents();
     const { active, history } = splitAgentRecords(records, ctx.cwd);
-    options.push(...buildAgentStatusMenuEntries(records, ctx.cwd, ctx.sessionManager?.getSessionId?.()));
+    const legacy = await legacyHistoryFor(ctx);
+    options.push(...buildAgentStatusMenuEntries(records, ctx.cwd, ctx.sessionManager?.getSessionId?.(), legacy));
 
     // Agent types list
     if (allNames.length > 0) {
@@ -1931,9 +1950,13 @@ Terse command-style prompts produce shallow, generic work.
     // Sessions recorded before durable checkpoints only exist as
     // `subagents:record` entries in their session file. They are merged here for
     // the project-wide view; the this-session view never needs the scan.
+    //
+    // They must clear the same `canOpenAgentHistory` bar as everything else:
+    // most carry no durable transcript, and listing an entry that can only ever
+    // answer "No agent history." is a dead link, not history.
     const merged = scope === "this-session"
       ? scoped
-      : mergeLegacyRecords(scoped, await readLegacySessionRecords(ctx.cwd));
+      : mergeLegacyRecords(scoped, await legacyHistoryFor(ctx));
     const pairs = merged.map((record) => ({ record, label: formatAgentHistoryOption(record, Date.now()) }));
     makeUniqueAgentOptionLabels(pairs);
     const title = scope === "this-session" ? "Agent history — this session" : "Agent history — all sessions";