@fitchmultz/pi-subagents 0.52.0 → 0.52.2

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 ## [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
+
+- On macOS, use the system `ps` directly for process-birth and shell-group checks instead of searching every PATH entry. This reduces the process events and synchronous spawn work caused by frequent parent-liveness checks without changing PID-reuse proof, cancellation, or non-macOS behavior.
+
 ## [0.52.0] - 2026-10-26
 
 ### Added

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/agent/shell.js

@@ -2,7 +2,7 @@ import { randomUUID } from "node:crypto";
 import { execFileSync } from "node:child_process";
 import * as path from "node:path";
 import * as fs from "node:fs";
-import { readChildProcessIdentity, readProcessClock } from "../shared/post-exit-stdio-guard.js";
+import { PS_COMMAND, readChildProcessIdentity, readProcessClock, } from "../shared/post-exit-stdio-guard.js";
 import { read, write } from "./storage.js";
 import { processSchema } from "./contracts.js";
 import { verifyProcess } from "./process.js";
@@ -16,7 +16,7 @@ if (directory === undefined ||
     process.ppid !== pid) {
     throw new Error("Native shell registration requires its actual parent shell.");
 }
-const group = Number(execFileSync("ps", ["-p", String(pid), "-o", "pgid="], {
+const group = Number(execFileSync(PS_COMMAND, ["-p", String(pid), "-o", "pgid="], {
     encoding: "utf8",
     timeout: 1000,
     stdio: ["ignore", "pipe", "ignore"],

package/dist/build-info.js

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

package/dist/shared/post-exit-stdio-guard.js

@@ -3,6 +3,8 @@ import { readFileSync, readlinkSync } from "node:fs";
 import { isLinuxProcessReplaced, readLinuxProcess } from "../pi-intercom/broker/pid.js";
 import { hasErrorCode } from "./unknown.js";
 const BIRTH_FIELDS = ["lstart=", "pgid="];
+// macOS ships /bin/ps; resolving "ps" through PATH can spawn once per missing entry.
+export const PS_COMMAND = process.platform === "darwin" ? "/bin/ps" : "ps";
 /** Capture only a real, cross-process clock context; unavailable context proves nothing. */
 export function readProcessClock() {
     if (process.platform !== "android") {
@@ -45,7 +47,7 @@ function psOptions(zone) {
 }
 function readPsFields(pid, fields, zone) {
     try {
-        return execFileSync("ps", psArguments(pid, fields), {
+        return execFileSync(PS_COMMAND, psArguments(pid, fields), {
             ...psOptions(zone),
             stdio: ["ignore", "pipe", "ignore"],
         })
@@ -59,7 +61,7 @@ function readPsFields(pid, fields, zone) {
 /** Nonblocking `readPsFields`; a failed run likewise observes nothing. */
 function observePsFields(pid, fields, zone) {
     return new Promise((resolve) => {
-        execFile("ps", psArguments(pid, fields), psOptions(zone), (error, stdout) => {
+        execFile(PS_COMMAND, psArguments(pid, fields), psOptions(zone), (error, stdout) => {
             resolve(error ? undefined : stdout.trim().replace(/\s+/g, " "));
         });
     });

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.0",
+  "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 {

package/src/agent/shell.ts

@@ -2,7 +2,11 @@ import { randomUUID } from "node:crypto";
 import { execFileSync } from "node:child_process";
 import * as path from "node:path";
 import * as fs from "node:fs";
-import { readChildProcessIdentity, readProcessClock } from "../shared/post-exit-stdio-guard.ts";
+import {
+  PS_COMMAND,
+  readChildProcessIdentity,
+  readProcessClock,
+} from "../shared/post-exit-stdio-guard.ts";
 import { read, write } from "./storage.ts";
 import { processSchema } from "./contracts.ts";
 import { verifyProcess } from "./process.ts";
@@ -22,7 +26,7 @@ if (
   throw new Error("Native shell registration requires its actual parent shell.");
 }
 const group = Number(
-  execFileSync("ps", ["-p", String(pid), "-o", "pgid="], {
+  execFileSync(PS_COMMAND, ["-p", String(pid), "-o", "pgid="], {
     encoding: "utf8",
     timeout: 1000,
     stdio: ["ignore", "pipe", "ignore"],

package/src/shared/post-exit-stdio-guard.ts

@@ -5,6 +5,9 @@ import { hasErrorCode } from "./unknown.ts";
 
 const BIRTH_FIELDS = ["lstart=", "pgid="];
 
+// macOS ships /bin/ps; resolving "ps" through PATH can spawn once per missing entry.
+export const PS_COMMAND = process.platform === "darwin" ? "/bin/ps" : "ps";
+
 interface AgentProcessExit {
   readonly pid?: number;
   readonly code: number | null;
@@ -75,7 +78,7 @@ function psOptions(zone: string | undefined) {
 
 function readPsFields(pid: number, fields: readonly string[], zone?: string): string | undefined {
   try {
-    return execFileSync("ps", psArguments(pid, fields), {
+    return execFileSync(PS_COMMAND, psArguments(pid, fields), {
       ...psOptions(zone),
       stdio: ["ignore", "pipe", "ignore"],
     })
@@ -93,7 +96,7 @@ function observePsFields(
   zone?: string,
 ): Promise<string | undefined> {
   return new Promise((resolve) => {
-    execFile("ps", psArguments(pid, fields), psOptions(zone), (error, stdout) => {
+    execFile(PS_COMMAND, psArguments(pid, fields), psOptions(zone), (error, stdout) => {
       resolve(error ? undefined : stdout.trim().replace(/\s+/g, " "));
     });
   });