pi-crew 0.8.6 → 0.8.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-crew",
-  "version": "0.8.6",
+  "version": "0.8.8",
   "description": "Pi extension for coordinated AI teams, workflows, worktrees, and async task orchestration",
   "author": "baphuongna",
   "license": "MIT",

package/src/extension/registration/compaction-guard.ts

@@ -1,5 +1,6 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { listRecentRuns } from "../run-index.ts";
+import { findRepoRoot } from "../../utils/paths.ts";
 import { extractSessionId } from "../../utils/session-utils.ts";
 import type { ArtifactDescriptor, TeamRunManifest } from "../../state/types.ts";
 
@@ -69,6 +70,29 @@ function formatCrewArtifactIndex(entries: CrewArtifactIndexEntry[]): string {
 	return lines.join("\n");
 }
 
+/**
+ * Project-scope filter: keep `run` only if it belongs to the SAME repo as
+ * `queryCwd` (or is a user-level / legacy run with no repo). This is the
+ * version-independent, reliable barrier against cross-project leaks: even
+ * when the session-id filter below cannot fire (ctx.sessionId is absent on
+ * some pi versions — observed on pi 0.79.6 ExtensionContext), the cwd filter
+ * stops another project's in-flight runs (e.g. edge-ai-agent) from bleeding
+ * into this project's ambient status or compaction-resume directive.
+ *
+ * Note: listRecentRuns already scopes its filesystem scan via scopedRunRoots,
+ * BUT it ALSO merges the GLOBAL activeRunEntries() registry (the cross-
+ * project dashboard view). That global merge is intentional for the
+ * dashboard but wrong for "what should THIS project's session do" — hence
+ * this filter at the consumption site.
+ */
+function isInProjectScope(run: TeamRunManifest, queryCwd: string): boolean {
+	const queryRepo = findRepoRoot(queryCwd);
+	if (queryRepo === undefined) return true; // viewer not in a repo → user-level view
+	const runRepo = typeof run.cwd === "string" && run.cwd.length > 0 ? findRepoRoot(run.cwd) : undefined;
+	if (runRepo === undefined) return true; // run is user-level / legacy / not a repo → include
+	return runRepo === queryRepo; // same project only
+}
+
 /**
  * Collect in-flight (non-terminal) crew runs that must be resumable after
  * compaction. These are runs the agent was actively working on or awaiting.
@@ -88,7 +112,13 @@ function formatCrewArtifactIndex(entries: CrewArtifactIndexEntry[]): string {
 export function collectInFlightRuns(cwd: string, currentSessionId?: string): TeamRunManifest[] {
 	return listRecentRuns(cwd, MAX_ARTIFACT_INDEX_RUNS).filter((run) => {
 		if (!IN_FLIGHT_RUN_STATUSES.has(run.status)) return false;
-		if (currentSessionId === undefined) return true; // no filter → back-compat
+		// Reliable barrier (2026-06-17): never leak another project's runs into
+		// THIS project's resume directive / ambient status, regardless of
+		// whether the session-id filter is available. This fixes the live
+		// cross-session leak that persisted after 4bd6f5b because ctx.sessionId
+		// is absent on pi 0.79.6.
+		if (!isInProjectScope(run, cwd)) return false;
+		if (currentSessionId === undefined) return true; // no session filter → back-compat (still project-scoped)
 		return run.ownerSessionId === currentSessionId; // strict: legacy ownerless runs excluded
 	});
 }

package/src/extension/run-index.ts

@@ -76,7 +76,7 @@ function scopedRunRoots(cwd: string): string[] {
 	return [...roots];
 }
 
-function collectActiveRuns(): TeamRunManifest[] {
+function collectActiveRuns(cwd?: string): TeamRunManifest[] {
 	return activeRunEntries()
 		.map((entry) => readManifest(entry.manifestPath))
 		.filter((manifest): manifest is TeamRunManifest => manifest !== undefined);

package/src/extension/team-tool/doctor.ts

@@ -9,6 +9,7 @@ import { projectCrewRoot, userCrewRoot } from "../../utils/paths.ts";
 import { DEFAULT_PATHS } from "../../config/defaults.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { getPiSpawnCommand } from "../../runtime/pi-spawn.ts";
+import { getRuntimeWarmupStatus } from "../../runtime/runtime-warmup.ts";
 import { validateResources } from "../validate-resources.ts";
 import { detectDrift, formatDriftReport, type DriftReport } from "../../config/drift-detector.ts";
 import { TeamToolParams } from "../../schema/team-tool-schema.ts";
@@ -188,6 +189,37 @@ export function buildTeamDoctorReport(input: TeamDoctorReportInput): TeamDoctorR
 			{ label: "leader repository", ok: true, detail: input.cwd },
 			{ label: "cleanup policy", ok: true, detail: "dirty worktrees preserved unless force is set" },
 		]),
+		section("Runtime warmup (cold-start fix v0.8.6)", () => {
+			// Surface whether the general cold-start-race fix is active + how long
+			// the graph warmup took, so a session can confirm the fix loaded
+			// (post-restart) and isn't pathologically slow. An UNWARMED graph is
+			// the documented cause of `Cannot read properties of undefined
+			// (reading '<binding>')` under concurrent subagent spawn.
+			//
+			// "Not started" is NOT a doctor error: it is the normal state in unit
+			// tests and in any caller that invokes buildTeamDoctorReport directly
+			// without going through registerPiTeams. Only a STARTED-but-FAILED
+			// warmup is an error (something genuinely went wrong during pre-warm).
+			const status = getRuntimeWarmupStatus();
+			const checks: DoctorCheck[] = [
+				{
+					label: "warmup started",
+					ok: true, // informational — "not started" is not a failure
+					detail: status.started ? "module graph pre-warmed at registration" : "not started in this process (normal for direct unit-test calls; in a live Pi session, started at extension load)",
+				},
+			];
+			if (status.started) {
+				checks.push({
+					label: "warmup completed",
+					ok: status.completed,
+					detail: status.completed ? (status.durationMs !== undefined ? `graph warm in ${status.durationMs}ms` : "completed") : "in progress",
+				});
+				if (status.error) {
+					checks.push({ label: "warmup error", ok: false, detail: status.error });
+				}
+			}
+			return checks;
+		}),
 	];
 	if (input.smokeChildPi) {
 		sections.push([`Child check`, `- ${input.smokeChildPi.ok ? "OK" : "FAIL"} child Pi smoke: ${input.smokeChildPi.detail}`]);

package/src/runtime/runtime-warmup.ts

@@ -61,6 +61,9 @@ const HOT_PEER_DEPS = ["@earendil-works/pi-coding-agent"] as const;
 
 let warmupPromise: Promise<void> | undefined;
 let warmupStarted = false;
+let warmupCompleted = false;
+let warmupDurationMs: number | undefined;
+let warmupError: string | undefined;
 
 /**
  * Start the runtime warmup (idempotent). Fires eager `import()` of the hot
@@ -74,6 +77,7 @@ let warmupStarted = false;
 export function startRuntimeWarmup(): void {
 	if (warmupStarted) return;
 	warmupStarted = true;
+	const startedAt = Date.now();
 	warmupPromise = (async (): Promise<void> => {
 		const imports: Array<Promise<unknown>> = [];
 		for (const spec of HOT_MODULE_SPECIFIERS) {
@@ -91,9 +95,15 @@ export function startRuntimeWarmup(): void {
 			);
 		}
 		await Promise.all(imports);
-	})().catch(() => {
-		// final safety net — warmup must never reject
-	});
+	})()
+		.then(() => {
+			warmupCompleted = true;
+			warmupDurationMs = Date.now() - startedAt;
+		})
+		.catch((err: unknown) => {
+			// final safety net — warmup must never reject. Record for diagnostics.
+			warmupError = err instanceof Error ? err.message : String(err ?? "unknown");
+		});
 }
 
 /**
@@ -113,9 +123,34 @@ export async function awaitRuntimeWarmup(): Promise<void> {
 export function resetRuntimeWarmupForTest(): void {
 	warmupPromise = undefined;
 	warmupStarted = false;
+	warmupCompleted = false;
+	warmupDurationMs = undefined;
+	warmupError = undefined;
 }
 
 /** Test seam: has startRuntimeWarmup() been called? */
 export function isRuntimeWarmupStarted(): boolean {
 	return warmupStarted;
 }
+
+/**
+ * Diagnostic snapshot of warmup state for `team doctor`. Surfaces whether the
+ * v0.8.6 cold-start fix is active and how long the graph warmup took, so a
+ * session can confirm the fix loaded (post-restart) and isn't pathologically
+ * slow.
+ */
+export interface RuntimeWarmupStatus {
+	started: boolean;
+	completed: boolean;
+	durationMs: number | undefined;
+	error: string | undefined;
+}
+
+export function getRuntimeWarmupStatus(): RuntimeWarmupStatus {
+	return {
+		started: warmupStarted,
+		completed: warmupCompleted,
+		durationMs: warmupDurationMs,
+		error: warmupError,
+	};
+}