pi-crew 0.9.16 → 0.9.18

package/src/runtime/anchor-manager.ts

@@ -10,13 +10,13 @@ const MAX_HANDOFFS_PER_ANCHOR = 100;
 
 /**
  * AnchorManager - Creates shared summary points where multiple handoffs accumulate.
- * 
+ *
  * Based on pi-boomerang's anchorMode pattern:
  * - setAnchor() creates a shared summary point for a session
  * - accumulateHandoff() adds handoffs to the anchor
  * - clearAnchor() finalizes and returns accumulated summaries
  * - getAnchorHandoff() retrieves accumulated summary without clearing
- * 
+ *
  * @see docs/pi-boomerang-integration-plan.md
  */
 
@@ -89,7 +89,7 @@ export class AnchorManager {
 	/**
 	 * Set an anchor point for a session.
 	 * All subsequent handoffs will accumulate to this anchor.
-	 * 
+	 *
 	 * @param sessionId - The session ID to create anchor for
 	 * @param context - Initial context for the anchor
 	 * @returns The anchor ID
@@ -121,7 +121,7 @@ export class AnchorManager {
 
 	/**
 	 * Get the current anchor for a session.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @returns The anchor if exists, null otherwise
 	 */
@@ -134,7 +134,7 @@ export class AnchorManager {
 
 	/**
 	 * Get the anchor ID for a session.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @returns The anchor ID if exists, undefined otherwise
 	 */
@@ -145,7 +145,7 @@ export class AnchorManager {
 	/**
 	 * Clear an anchor and return the accumulated handoff summary.
 	 * This removes the anchor and returns merged handoffs.
-	 * 
+	 *
 	 * @param anchorId - The anchor ID to clear
 	 * @returns The accumulated handoff summary
 	 * @throws Error if anchor not found or no handoffs accumulated
@@ -163,14 +163,17 @@ export class AnchorManager {
 		this.sessionAnchors.delete(anchor.sessionId);
 		this.anchors.delete(anchorId);
 
-		this.options.eventEmitter?.emit("anchor:cleared", { anchorId, accumulated });
+		this.options.eventEmitter?.emit("anchor:cleared", {
+			anchorId,
+			accumulated,
+		});
 
 		return accumulated;
 	}
 
 	/**
 	 * Clear anchor by session ID.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @returns The accumulated handoff summary
 	 */
@@ -183,7 +186,7 @@ export class AnchorManager {
 	/**
 	 * Accumulate a handoff to an anchor.
 	 * If anchor doesn't exist, creates an implicit anchor.
-	 * 
+	 *
 	 * @param anchorId - The anchor ID
 	 * @param handoff - The handoff summary to accumulate
 	 */
@@ -224,7 +227,7 @@ export class AnchorManager {
 
 	/**
 	 * Accumulate handoff by session ID.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @param handoff - The handoff summary to accumulate
 	 */
@@ -241,7 +244,7 @@ export class AnchorManager {
 
 	/**
 	 * Get the accumulated handoff for an anchor without clearing it.
-	 * 
+	 *
 	 * @param anchorId - The anchor ID
 	 * @returns The accumulated handoff summary, or null if anchor not found or no handoffs
 	 */
@@ -254,7 +257,7 @@ export class AnchorManager {
 
 	/**
 	 * Get accumulated handoff by session ID.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @returns The accumulated handoff summary, or null if no anchor or handoffs
 	 */
@@ -266,7 +269,7 @@ export class AnchorManager {
 
 	/**
 	 * Get status information for an anchor.
-	 * 
+	 *
 	 * @param anchorId - The anchor ID
 	 * @returns Status object or null if anchor not found
 	 */
@@ -279,21 +282,15 @@ export class AnchorManager {
 			sessionId: anchor.sessionId,
 			createdAt: anchor.createdAt,
 			handoffCount: anchor.handoffs.length,
-			totalTokens: anchor.handoffs.reduce(
-				(sum, h) => sum + h.metrics.tokensUsed,
-				0,
-			),
-			totalDuration: anchor.handoffs.reduce(
-				(sum, h) => sum + h.metrics.duration,
-				0,
-			),
+			totalTokens: anchor.handoffs.reduce((sum, h) => sum + h.metrics.tokensUsed, 0),
+			totalDuration: anchor.handoffs.reduce((sum, h) => sum + h.metrics.duration, 0),
 			context: anchor.context,
 		};
 	}
 
 	/**
 	 * Get status by session ID.
-	 * 
+	 *
 	 * @param sessionId - The session ID
 	 * @returns Status object or null if no anchor
 	 */
@@ -305,7 +302,7 @@ export class AnchorManager {
 
 	/**
 	 * Check if an anchor has handoffs accumulated.
-	 * 
+	 *
 	 * @param anchorId - The anchor ID
 	 * @returns True if anchor has handoffs
 	 */
@@ -316,7 +313,7 @@ export class AnchorManager {
 
 	/**
 	 * Get all anchors.
-	 * 
+	 *
 	 * @returns Array of all anchors
 	 */
 	getAllAnchors(): Anchor[] {
@@ -347,7 +344,12 @@ export class AnchorManager {
 				iterations: acc.iterations + h.metrics.iterations,
 				toolsUsed: [...acc.toolsUsed, ...h.metrics.toolsUsed],
 			}),
-			{ tokensUsed: 0, duration: 0, iterations: 0, toolsUsed: [] as string[] },
+			{
+				tokensUsed: 0,
+				duration: 0,
+				iterations: 0,
+				toolsUsed: [] as string[],
+			},
 		);
 
 		// Deduplicate tools

package/src/runtime/async-marker.ts

@@ -19,7 +19,13 @@ export function writeAsyncStartMarker(manifest: Pick<TeamRunManifest, "stateRoot
 export function hasAsyncStartMarker(manifest: Pick<TeamRunManifest, "stateRoot">): boolean {
 	try {
 		const raw = JSON.parse(fs.readFileSync(asyncStartMarkerPath(manifest), "utf-8")) as Partial<AsyncStartMarker>;
-		return typeof raw.pid === "number" && Number.isInteger(raw.pid) && raw.pid > 0 && typeof raw.startedAt === "string" && raw.startedAt.length > 0;
+		return (
+			typeof raw.pid === "number" &&
+			Number.isInteger(raw.pid) &&
+			raw.pid > 0 &&
+			typeof raw.startedAt === "string" &&
+			raw.startedAt.length > 0
+		);
 	} catch {
 		return false;
 	}

package/src/runtime/async-runner.ts

@@ -1,19 +1,16 @@
-import { spawn, type SpawnOptions } from "node:child_process";
-import { createRequire } from "node:module";
+import { type SpawnOptions, spawn } from "node:child_process";
 import * as fs from "node:fs";
+import { createRequire } from "node:module";
 import * as path from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
-import { WINDOWS_ESSENTIAL_ENV_VARS } from "../utils/env-allowlist.ts";
-import { logInternalError } from "../utils/internal-error.ts";
 import { appendEvent } from "../state/event-log.ts";
-import { sanitizeEnvSecrets } from "../utils/env-filter.ts";
-import { resolvePeerDepDir, PEER_DEP_DIR_ENV } from "./peer-dep.ts";
-import {
-	registerWorker,
-	unregisterWorker,
-} from "./orphan-worker-registry.ts";
 import type { TeamRunManifest } from "../state/types.ts";
-
+import { WINDOWS_ESSENTIAL_ENV_VARS } from "../utils/env-allowlist.ts";
+import { sanitizeEnvSecrets } from "../utils/env-filter.ts";
+import { logInternalError } from "../utils/internal-error.ts";
+import { packageRoot } from "../utils/paths.ts";
+import { registerWorker, unregisterWorker } from "./orphan-worker-registry.ts";
+import { PEER_DEP_DIR_ENV, resolvePeerDepDir } from "./peer-dep.ts";
 
 export type FileExists = (filePath: string) => boolean;
 
@@ -23,9 +20,7 @@ const requireFromHere = createRequire(import.meta.url);
 const STRIP_TYPES_MIN_MAJOR = 22;
 const STRIP_TYPES_MIN_MINOR = 6;
 
-export type LoaderSpec =
-	| { kind: "jiti"; path: string }
-	| { kind: "strip-types" };
+export type LoaderSpec = { kind: "jiti"; path: string } | { kind: "strip-types" };
 
 type LoaderInput = LoaderSpec | string | false | undefined;
 
@@ -130,12 +125,21 @@ export function getBackgroundRunnerCommand(
 	// don't want report files can opt out with PI_CREW_BG_REPORT_ON_FATAL=0.
 	const reportOn = !(process.env.PI_CREW_BG_REPORT_ON_FATAL === "0" || process.env.PI_TEAMS_BG_REPORT_ON_FATAL === "0");
 	const reportDir = reportDirectory ?? path.dirname(runnerPath);
-	const reportFlags = reportOn
-		? ["--report-on-fatalerror", "--report-compact", `--report-directory=${reportDir}`]
-		: [];
+	const reportFlags = reportOn ? ["--report-on-fatalerror", "--report-compact", `--report-directory=${reportDir}`] : [];
 	if (loader.kind === "jiti") {
 		return {
-			args: [memoryLimit, ...reportFlags, "--trace-uncaught", "--import", pathToFileURL(loader.path).href, runnerPath, "--cwd", cwd, "--run-id", runId],
+			args: [
+				memoryLimit,
+				...reportFlags,
+				"--trace-uncaught",
+				"--import",
+				pathToFileURL(loader.path).href,
+				runnerPath,
+				"--cwd",
+				cwd,
+				"--run-id",
+				runId,
+			],
 			loader: "jiti",
 		};
 	}
@@ -220,7 +224,17 @@ export const BACKGROUND_RUNNER_ENV_ALLOWLIST: string[] = [
 ];
 
 export async function spawnBackgroundTeamRun(manifest: TeamRunManifest): Promise<SpawnBackgroundTeamRunResult> {
-	const runnerPath = path.join(path.dirname(fileURLToPath(import.meta.url)), "background-runner.ts");
+	// FIX (2026-07-02, perf review F-critical): use packageRoot() instead of
+	// import.meta.url-relative path. The previous path.resolve walks
+	// <bundleDir>/background-runner.ts, which is correct in src/ but BROKEN
+	// in the bundle: esbuild's __esm helper does not preserve per-module
+	// import.meta.url, so the resolve lands at <pi-crew>/background-runner.ts
+	// (missing `src/runtime/`). The background-runner then ENOENTs at spawn
+	// and 3 explorer agents failed during the verification run. packageRoot()
+	// walks up to find pi-crew's package.json and works correctly from both
+	// src/ and dist/ entry points. Mirrors the fix in pi-args.ts:10 (commit
+	// 0dd93e0).
+	const runnerPath = path.join(packageRoot(), "src", "runtime", "background-runner.ts");
 	const logPath = path.join(manifest.stateRoot, "background.log");
 	fs.mkdirSync(manifest.stateRoot, { recursive: true });
 
@@ -238,14 +252,16 @@ export async function spawnBackgroundTeamRun(manifest: TeamRunManifest): Promise
 	// `npm root -g` probe. No-op when pi-crew and pi are co-located. See
 	// src/runtime/peer-dep.ts.
 	const peerDepDir = resolvePeerDepDir();
-	const childEnv = peerDepDir
-		? { ...filteredEnv, [PEER_DEP_DIR_ENV]: peerDepDir }
-		: filteredEnv;
+	const childEnv = peerDepDir ? { ...filteredEnv, [PEER_DEP_DIR_ENV]: peerDepDir } : filteredEnv;
 
 	const loader = resolveTypeScriptLoader();
 	if (!loader) {
 		const message = buildLoaderUnavailableMessage(packageRootFromRuntime());
-		appendEvent(manifest.eventsPath, { type: "async.failed", runId: manifest.runId, message });
+		appendEvent(manifest.eventsPath, {
+			type: "async.failed",
+			runId: manifest.runId,
+			message,
+		});
 		throw new Error(message);
 	}
 	// Pass manifest.stateRoot as report-directory so V8 fatal reports land
@@ -345,4 +361,3 @@ export async function spawnBackgroundTeamRun(manifest: TeamRunManifest): Promise
 
 	return { pid: child.pid, logPath };
 }
-

package/src/runtime/auto-summarize.ts

@@ -1,11 +1,11 @@
 /**
  * AutoSummarizeService - Enables auto-summarization with token/tool thresholds.
- * 
+ *
  * Based on pi-boomerang's autoBoomerang pattern:
  * - toggle() enables/disables auto-summarization
  * - shouldAutoSummarize() checks if task should auto-summarize
  * - Token and tool thresholds control when summarization triggers
- * 
+ *
  * @see docs/pi-boomerang-integration-plan.md
  */
 
@@ -72,11 +72,7 @@ export interface AutoSummarizeTriggeredEventData {
 /**
  * What triggered the auto-summarize.
  */
-export type AutoSummarizeTrigger =
-	| "token_threshold"
-	| "tools_threshold"
-	| "manual"
-	| "high_usage";
+export type AutoSummarizeTrigger = "token_threshold" | "tools_threshold" | "manual" | "high_usage";
 
 /**
  * AutoSummarizeService enables automatic summarization based on configurable thresholds.
@@ -143,7 +139,7 @@ export class AutoSummarizeService {
 
 	/**
 	 * Check if a task should auto-summarize.
-	 * 
+	 *
 	 * @param packet - The task packet
 	 * @param result - The task result
 	 * @returns True if the task should auto-summarize
@@ -180,7 +176,7 @@ export class AutoSummarizeService {
 
 	/**
 	 * Get the reason why a task should (or should not) auto-summarize.
-	 * 
+	 *
 	 * @param packet - The task packet
 	 * @param result - The task result
 	 * @returns Object with shouldSummarize flag and reason
@@ -343,8 +339,6 @@ export interface AutoSummarizeDecision {
 /**
  * Create an AutoSummarizeService with default options.
  */
-export function createAutoSummarizeService(
-	options?: AutoSummarizeServiceOptions,
-): AutoSummarizeService {
+export function createAutoSummarizeService(options?: AutoSummarizeServiceOptions): AutoSummarizeService {
 	return new AutoSummarizeService(options);
-}
+}