pi-herdr-agents 2.0.2 → 2.0.4

package/CHANGELOG.md

@@ -7,13 +7,30 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
-## [v2.0.2](https://github.com/giuseppecrj/pi-herdr-agents/compare/v2.0.1...v2.0.2)
+## [v2.0.4](https://github.com/giuseppecrj/pi-herdr-agents/compare/v2.0.3...v2.0.4)
+
+### Merged
+
+- perf: batch persistent delivery ledger reads [`#58`](https://github.com/giuseppecrj/pi-herdr-agents/pull/58)
+
+## [v2.0.3](https://github.com/giuseppecrj/pi-herdr-agents/compare/v2.0.2...v2.0.3) - 2026-09-21
+
+### Commits
+
+- fix: skip startup worktree inventory outside Herdr [`34800d7`](https://github.com/giuseppecrj/pi-herdr-agents/commit/34800d79464575063a61ad6f240cd49cc8acd730)
+- chore: release v2.0.3 [`60888f2`](https://github.com/giuseppecrj/pi-herdr-agents/commit/60888f2dea48317257909abbcd0dd374ed861d77)
+
+## [v2.0.2](https://github.com/giuseppecrj/pi-herdr-agents/compare/v2.0.1...v2.0.2) - 2026-09-19
 
 ### Merged
 
 - fix(subagents): allow isolated same-family review fallback [`#54`](https://github.com/giuseppecrj/pi-herdr-agents/pull/54)
 - fix(subagents): remove conflicting Ctrl+J shortcut [`#53`](https://github.com/giuseppecrj/pi-herdr-agents/pull/53)
 
+### Commits
+
+- chore: release v2.0.2 [`49efa52`](https://github.com/giuseppecrj/pi-herdr-agents/commit/49efa52bf671fbef00ad1be2d8b679188c5d1de3)
+
 ## [v2.0.1](https://github.com/giuseppecrj/pi-herdr-agents/compare/v2.0.0...v2.0.1) - 2026-09-18
 
 ### Merged

package/README.md

@@ -23,7 +23,7 @@ Delegate investigation, implementation, and review without blocking the parent s
 - [Herdr](https://herdr.dev) and its CLI
 - `HERDR_ENV=1` — start Pi from inside Herdr
 
-Other terminal multiplexers are not supported. Worktrees isolate Git checkouts, not processes or permissions; child agents and installed Pi packages run with your user account's access.
+Other terminal multiplexers are not supported. Outside Herdr, the extension skips automatic startup worktree inventory. Explicit inventory tools still report unavailable inspection as unknown. Worktrees isolate Git checkouts, not processes or permissions; child agents and installed Pi packages run with your user account's access.
 
 ## Install
 

package/docs/worktree-subagents.md

@@ -231,7 +231,7 @@ Ignored files do not block cleanup. Inventory shows their exact file count, and
 
 Open workspaces are removed through Herdr. Git-only orphans use `git worktree remove`, verify checkout absence, and then prune stale Git registrations. Failed removal is reported, never forced. A reachable owned manifest is merge-updated to `removed` with `workspaceRemovedAt` only after success; absent cross-session manifests are reported and are not rewritten. A manifest update failure after removal is a removed-with-warning result, not a failed removal. Removed manifests provide already-removed no-op evidence only; a new checkout at the same path is classified independently. Missing and dangling manifest paths are non-matches. Permission errors, symlink loops, and other undecidable manifest identities remain blockers, as do genuine identity disagreements. Cleanup Git and Herdr calls have 30-second timeouts; a timed-out operation fails closed rather than proceeding with removal.
 
-Branches and their commits are always retained: cleanup never deletes, force-updates, or rewrites a branch. It never runs on completion, shutdown, or a timer. Parent session start emits only one compact inventory count when contained worktrees remain. Unsupported platforms or failed global process enumeration block removal; individual visibility gaps are disclosed warnings, never proof that an orphan is idle.
+Branches and their commits are always retained: cleanup never deletes, force-updates, or rewrites a branch. It never runs on completion, shutdown, or a timer. Inside Herdr with its CLI available, parent session start emits only one compact inventory count when contained worktrees remain. Outside Herdr, startup skips the automatic inventory; explicit inventory tools remain available and report unavailable inspection as unknown. Unsupported platforms or failed global process enumeration block removal; individual visibility gaps are disclosed warnings, never proof that an orphan is idle.
 
 ## Current limits
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-herdr-agents",
-  "version": "2.0.2",
+  "version": "2.0.4",
   "description": "Asynchronous Pi subagents in Herdr, with optional isolated Git worktrees",
   "keywords": [
     "pi-package",

package/pi-extension/subagents/herdr.ts

@@ -108,6 +108,7 @@ function extractHerdrWorktree(output: string): HerdrWorktreeSurface {
 
 function herdrExec(args: string[], timeout?: number): string {
 	return execFileSync("herdr", args, {
+		stdio: "pipe",
 		encoding: "utf8",
 		timeout,
 		killSignal: "SIGKILL",

package/pi-extension/subagents/index.ts

@@ -2422,6 +2422,7 @@ export const __test__ = {
 	resolveUnexpectedErrorPresentation,
 	shouldAdvanceToFallback,
 	deliverPersistentTaskEvent,
+	drainPersistentTaskEvents,
 	notifyPersistentCrash,
 	sendSubagentResult,
 	shouldRetainSubagentSurface,
@@ -2657,11 +2658,13 @@ function deliverPersistentTaskEvent(
 	running: RunningSubagent,
 	event: ReturnType<typeof readPersistentTaskEvents>[number],
 	api: Pick<ExtensionAPI, "sendMessage">,
+	ledgerSnapshot?: ReturnType<typeof readPersistentDeliveryLedger>,
 ): void {
 	if (!running.persistent || event.generation !== running.generationId) return;
 	const deliveryKey = `${running.id}:${event.type}:${event.task}`;
 	if (inFlightPersistentTaskDeliveries.has(deliveryKey)) return;
-	const ledger = readPersistentDeliveryLedger(running.sessionFile);
+	const ledger =
+		ledgerSnapshot ?? readPersistentDeliveryLedger(running.sessionFile);
 	if (event.type === "help-request") {
 		if (
 			ledger.some(
@@ -2685,13 +2688,15 @@ function deliverPersistentTaskEvent(
 				},
 				{ triggerTurn: true, deliverAs: "steer" },
 			);
-			appendPersistentDeliveryLedger(running.sessionFile, {
-				task: event.task,
-				outcome: "help-requested",
-				generation: running.generationId!,
-				logicalId: running.logicalId!,
-				policyHash: running.policyHash!,
-			});
+			ledger.push(
+				appendPersistentDeliveryLedger(running.sessionFile, {
+					task: event.task,
+					outcome: "help-requested",
+					generation: running.generationId!,
+					logicalId: running.logicalId!,
+					policyHash: running.policyHash!,
+				}),
+			);
 			if (running.taskId === event.task) running.taskId = undefined;
 		} finally {
 			inFlightPersistentTaskDeliveries.delete(deliveryKey);
@@ -2724,13 +2729,15 @@ function deliverPersistentTaskEvent(
 				policyHash: running.policyHash!,
 			},
 		);
-		appendPersistentDeliveryLedger(running.sessionFile, {
-			task: event.task,
-			outcome: "delivered",
-			generation: running.generationId!,
-			logicalId: running.logicalId!,
-			policyHash: running.policyHash!,
-		});
+		ledger.push(
+			appendPersistentDeliveryLedger(running.sessionFile, {
+				task: event.task,
+				outcome: "delivered",
+				generation: running.generationId!,
+				logicalId: running.logicalId!,
+				policyHash: running.policyHash!,
+			}),
+		);
 		running.tasksCompleted = completed;
 		if (running.taskId === event.task) running.taskId = undefined;
 		if (running.stopState === "pending")
@@ -2743,13 +2750,25 @@ function deliverPersistentTaskEvent(
 function drainPersistentTaskEvents(
 	running: RunningSubagent,
 	api: Pick<ExtensionAPI, "sendMessage">,
+	readLedger = readPersistentDeliveryLedger,
 ): void {
 	const events = readPersistentTaskEvents(running.sessionFile);
+	let ledger: ReturnType<typeof readPersistentDeliveryLedger> | undefined;
 	for (const event of events.slice(running.observedTaskEvents ?? 0)) {
+		if (!running.persistent || event.generation !== running.generationId)
+			continue;
+		if (
+			inFlightPersistentTaskDeliveries.has(
+				`${running.id}:${event.type}:${event.task}`,
+			)
+		)
+			continue;
+		ledger ??= readLedger(running.sessionFile);
 		deliverPersistentTaskEvent(
 			running,
 			event,
 			selectCompletionApi(api, runtime.pi),
+			ledger,
 		);
 	}
 	running.observedTaskEvents = events.length;
@@ -3131,7 +3150,7 @@ export default function subagentsExtension(
 			startStatusRefresh(pi);
 			updateWidget();
 		}
-		if (parentSession && ctx.cwd && ctx.hasUI) {
+		if (parentSession && ctx.cwd && ctx.hasUI && isTerminalAvailable()) {
 			try {
 				const notice = worktreeInventoryNotice(
 					await listContainedWorktrees(cleanupInput(ctx)),