@fitchmultz/pi-subagents 0.52.1 → 0.52.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [0.52.2] - 2026-10-29
+
+### Fixed
+
+- Avoid rereading historical shell records on every idle parking check while a live shell still blocks parking. Retain one publication path and revalidate its current record, owner token and process birth. Read the full inventory again when that witness disappears or dies. Stop and cold continuation keep their complete inventory checks. No records are pruned and live jobs are not interrupted.
+
 ## [0.52.1] - 2026-10-29
 
 ### Fixed

package/dist/agent/parking.js

@@ -4,7 +4,7 @@ import * as path from "node:path";
 import { parkedSchema } from "./contracts.js";
 import { errorText, loadSDK } from "./host.js";
 import { conversation, latest, launchAt, list, read, runDirectory, write } from "./storage.js";
-import { alive, shells, wakeActor } from "./process.js";
+import { alive, shell, shells, wakeActor } from "./process.js";
 import { isHistoryRequest, PARK, PARKED_HISTORY, WAKE, } from "../shared/parking.js";
 import { nativePeerId } from "../pi-intercom/session-targets.js";
 function parkable(session) {
@@ -74,6 +74,7 @@ export class Parking {
     restored;
     since = 0;
     leaf;
+    shell;
     base;
     launch;
     token;
@@ -111,9 +112,7 @@ export class Parking {
     }
     /** True once this generation has exited its runtime with or without parked authority. */
     async park(runtime, actor) {
-        if (!actor.pi ||
-            !this.due(runtime.session, actor.busy) ||
-            shells(this.base, this.launch, this.token).some(alive)) {
+        if (!actor.pi || !this.due(runtime.session, actor.busy) || this.hasShell()) {
             return false;
         }
         const hold = await this.prepare(runtime, actor.pi, actor);
@@ -122,6 +121,13 @@ export class Parking {
         }
         return hold !== undefined;
     }
+    hasShell() {
+        const witness = this.shell === undefined ? undefined : shell(this.shell, this.token);
+        if (!witness || !alive(witness)) {
+            this.shell = shells(this.base, this.launch, this.token).find(alive)?.file;
+        }
+        return this.shell !== undefined;
+    }
     /** Final idle: no run, turn, journal change or viewer since the period began. */
     due(session, busy) {
         const leaf = session.sessionManager.getLeafId();

package/dist/agent/process.js

@@ -346,6 +346,13 @@ async function waitForExit(targets, timeout) {
         await delay(25);
     }
 }
+export function shell(file, token) {
+    const identity = published(file, read(file, processSchema));
+    if (identity && identity.token !== token) {
+        throw new Error("Foreign or missing native shell identity.");
+    }
+    return identity;
+}
 export function shells(base, launch, token) {
     const dir = path.join(conversation(base, launch), "shells", token);
     if (!fs.existsSync(dir)) {
@@ -356,11 +363,11 @@ export function shells(base, launch, token) {
         .filter((name) => name.endsWith(".json"))
         .map((name) => {
         const file = path.join(dir, name);
-        const identity = published(file, read(file, processSchema));
-        if (!identity || identity.token !== token) {
+        const identity = shell(file, token);
+        if (!identity) {
             throw new Error("Foreign or missing native shell identity.");
         }
-        return identity;
+        return Object.assign(identity, { file });
     });
 }
 function quote(value) {

package/dist/build-info.js

@@ -1 +1 @@
-export const EXTENSION_BUILD = Object.freeze({"version":"0.52.1","sha256":"d950f3bebb47cbe1dc09b2f7f9cc042535701b14cfe39cc518107a5626698bf4"});
+export const EXTENSION_BUILD = Object.freeze({"version":"0.52.2","sha256":"cb74b5f47c1e4f8a3a5f27aa2ce24dcb7c240b43aba9472b276a9a815ec30b2a"});

package/docs/performance.md

@@ -165,6 +165,27 @@ is not an indefinite-idle bound. The
 [issue 238 evidence](evidence/issue-238-idle/README.md) has the samples, host
 hashes, native causal chain and limits.
 
+### Idle parking shell records
+
+An idle child with a live shell now retains one shell publication path. Each
+parking check rereads that record and uses the existing token and process-birth
+validation. A missing or dead witness causes a full inventory read. Stop and cold
+continuation still read the complete inventory.
+
+On macOS arm64 with Node 24.22.1, five alternating before/after pairs exercised
+`Parking.park` against one real actor's 615 shell records. Both versions refused
+to park on every check. After one untimed initial inventory, each recurring check
+opened 615 records before and one record after. Median wall time changed from
+80.96 ms to 4.30 ms. Observed ranges were 73.40 to 84.01 ms before and 4.19 to
+4.43 ms after. The baseline was release 0.52.1 at `3650e11`.
+
+This isolated helper measurement advanced the idle clock and supplied an idle
+session stub. It read the real records without changing them or the active actor.
+The initial scan and witness-loss scan still read the full inventory. This is not
+an end-to-end model, TUI or fleet CPU speedup claim. The native `park-shells` test
+separately exercises repeated checks, witness death, missing and replaced
+publications, and continuation on official Pi and the maintained fork.
+
 ## Necessary costs
 
 - Native Pi owns context loading and model execution. The extension cannot remove

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fitchmultz/pi-subagents",
-  "version": "0.52.1",
+  "version": "0.52.2",
   "description": "Pi extension for subagent orchestration and local session coordination",
   "license": "MIT",
   "type": "module",

package/src/agent/parking.ts

@@ -5,7 +5,7 @@ import type { AgentSession, ExtensionAPI } from "@earendil-works/pi-coding-agent
 import { parkedSchema, type Launch, type Parked } from "./contracts.ts";
 import { errorText, loadSDK, type WorkingState } from "./host.ts";
 import { conversation, latest, launchAt, list, read, runDirectory, write } from "./storage.ts";
-import { alive, shells, wakeActor } from "./process.ts";
+import { alive, shell, shells, wakeActor } from "./process.ts";
 import {
   isHistoryRequest,
   PARK,
@@ -138,6 +138,7 @@ export class Parking {
   private restored?: string;
   private since = 0;
   private leaf?: string | null;
+  private shell?: string;
   private readonly base: string;
   private readonly launch: Launch;
   private readonly token: string;
@@ -175,11 +176,7 @@ export class Parking {
   }
   /** True once this generation has exited its runtime with or without parked authority. */
   async park(runtime: Runtime, actor: Resident): Promise<boolean> {
-    if (
-      !actor.pi ||
-      !this.due(runtime.session, actor.busy) ||
-      shells(this.base, this.launch, this.token).some(alive)
-    ) {
+    if (!actor.pi || !this.due(runtime.session, actor.busy) || this.hasShell()) {
       return false;
     }
     const hold = await this.prepare(runtime, actor.pi, actor);
@@ -188,6 +185,13 @@ export class Parking {
     }
     return hold !== undefined;
   }
+  private hasShell(): boolean {
+    const witness = this.shell === undefined ? undefined : shell(this.shell, this.token);
+    if (!witness || !alive(witness)) {
+      this.shell = shells(this.base, this.launch, this.token).find(alive)?.file;
+    }
+    return this.shell !== undefined;
+  }
   /** Final idle: no run, turn, journal change or viewer since the period began. */
   private due(session: AgentSession, busy: boolean): boolean {
     const leaf = session.sessionManager.getLeafId();

package/src/agent/process.ts

@@ -393,7 +393,18 @@ async function waitForExit(targets: readonly OwnedProcess[], timeout: number): P
     await delay(25);
   }
 }
-export function shells(base: string, launch: Launch, token: string): OwnedProcess[] {
+export function shell(file: string, token: string): OwnedProcess | undefined {
+  const identity = published(file, read(file, processSchema));
+  if (identity && identity.token !== token) {
+    throw new Error("Foreign or missing native shell identity.");
+  }
+  return identity;
+}
+export function shells(
+  base: string,
+  launch: Launch,
+  token: string,
+): (OwnedProcess & { readonly file: string })[] {
   const dir = path.join(conversation(base, launch), "shells", token);
   if (!fs.existsSync(dir)) {
     return [];
@@ -403,11 +414,11 @@ export function shells(base: string, launch: Launch, token: string): OwnedProces
     .filter((name) => name.endsWith(".json"))
     .map((name) => {
       const file = path.join(dir, name);
-      const identity = published(file, read(file, processSchema));
-      if (!identity || identity.token !== token) {
+      const identity = shell(file, token);
+      if (!identity) {
         throw new Error("Foreign or missing native shell identity.");
       }
-      return identity;
+      return Object.assign(identity, { file });
     });
 }
 function quote(value: string): string {