pi-cohort 7.4.3 → 7.4.4

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## [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.4",
   "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/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);