pi-cohort 7.4.3 → 7.4.5

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [7.4.5] - 2026-10-23
+
+### Fixed
+- Child personas that allow `subagent` and inherit extensions now start without a duplicate-tool error; the explicit child extension is the only registrant. (#26)
+
+## [7.4.4] - 2026-10-22
+
+### Fixed
+- Async run status reads no longer serve a stale `running` state after an atomic `status.json` replacement that lands on the same mtime; the read cache now also compares inode and size.
+- The Tests workflow passes on Windows again: CRLF, POSIX file-mode, ENOTDIR, and SIGUSR2 assumptions in tests are normalized or skipped on win32, and run-dir cleanup retries while detached runners finish writing.
+
 ## [7.4.3] - 2026-10-20
 
 ### Changed

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-cohort",
-  "version": "7.4.3",
+  "version": "7.4.5",
   "description": "Delegate Pi work to focused child agents: code review, scouting, implementation, parallel audits, saved chains, and background jobs.",
   "author": "Jacek Juraszek",
   "license": "MIT",

package/src/extension/index.ts

@@ -34,8 +34,7 @@ import { registerSlashSubagentBridge } from "../slash/slash-bridge.ts";
 import { clearSlashSnapshots, getSlashRenderableSnapshot, resolveSlashMessageDetails, restoreSlashFinalSnapshots, type SlashMessageDetails } from "../slash/slash-live-state.ts";
 import { inspectSubagentStatus } from "../runs/background/run-status.ts";
 import registerSubagentNotify, { type SubagentNotifyDetails } from "../runs/background/notify.ts";
-import { SUBAGENT_CHILD_ENV, SUBAGENT_FANOUT_CHILD_ENV } from "../runs/shared/pi-args.ts";
-import registerFanoutChildSubagentExtension from "./fanout-child.ts";
+import { SUBAGENT_CHILD_ENV } from "../runs/shared/pi-args.ts";
 import { formatDuration, shortenPath } from "../shared/formatters.ts";
 import { loadConfig } from "./config.ts";
 import { deriveForwardedFlags } from "../runs/shared/forward-flags.ts";
@@ -242,7 +241,6 @@ function formatSubagentRoster(cwd: string): string | undefined {
 
 export default function registerSubagentExtension(pi: ExtensionAPI): void {
 	if (process.env[SUBAGENT_CHILD_ENV] === "1") {
-		if (process.env[SUBAGENT_FANOUT_CHILD_ENV] === "1") registerFanoutChildSubagentExtension(pi);
 		return;
 	}
 	const globalStore = globalThis as Record<string, unknown>;

package/src/shared/utils.ts

@@ -16,7 +16,9 @@ import type { AgentProgress, AsyncStatus, Details, DisplayItem, ErrorInfo, Singl
 
 export { getAgentDir } from "./agent-dir.ts";
 
-const statusCache = new Map<string, { mtime: number; status: AsyncStatus }>();
+// Keyed on inode too: writeAtomicJson renames a fresh file in, and a coarse
+// filesystem clock can give the replacement the same mtime as the original.
+const statusCache = new Map<string, { mtime: number; ino: number; size: number; status: AsyncStatus }>();
 
 function getErrorMessage(error: unknown): string {
 	return error instanceof Error ? error.message : String(error);
@@ -35,7 +37,7 @@ function isNotFoundError(error: unknown): boolean {
 }
 
 /**
- * Read async job status from disk (with mtime-based caching)
+ * Read async job status from disk (with stat-based caching)
  */
 export function readStatus(asyncDir: string): AsyncStatus | null {
 	const statusPath = path.join(asyncDir, "status.json");
@@ -51,7 +53,7 @@ export function readStatus(asyncDir: string): AsyncStatus | null {
 	}
 
 	const cached = statusCache.get(statusPath);
-	if (cached && cached.mtime === stat.mtimeMs) {
+	if (cached && cached.mtime === stat.mtimeMs && cached.ino === stat.ino && cached.size === stat.size) {
 		return cached.status;
 	}
 
@@ -74,7 +76,7 @@ export function readStatus(asyncDir: string): AsyncStatus | null {
 		});
 	}
 
-	statusCache.set(statusPath, { mtime: stat.mtimeMs, status });
+	statusCache.set(statusPath, { mtime: stat.mtimeMs, ino: stat.ino, size: stat.size, status });
 	if (statusCache.size > 50) {
 		const firstKey = statusCache.keys().next().value;
 		if (firstKey) statusCache.delete(firstKey);