pi-crew 0.9.16 → 0.9.17

package/src/runtime/manifest-cache.ts

@@ -1,11 +1,11 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
+import { DEFAULT_CACHE, DEFAULT_PATHS } from "../config/defaults.ts";
+import { activeRunEntries } from "../state/active-run-registry.ts";
+import type { TeamRunManifest } from "../state/types.ts";
 import { closeWatcher, watchWithErrorHandler } from "../utils/fs-watch.ts";
 import { findRepoRoot, projectCrewRoot, userCrewRoot } from "../utils/paths.ts";
-import { activeRunEntries } from "../state/active-run-registry.ts";
 import { isSafePathId, resolveContainedRelativePath, resolveRealContainedPath } from "../utils/safe-paths.ts";
-import type { TeamRunManifest } from "../state/types.ts";
-import { DEFAULT_CACHE, DEFAULT_PATHS } from "../config/defaults.ts";
 
 export interface ManifestCache {
 	list(limit?: number): TeamRunManifest[];
@@ -74,7 +74,14 @@ function validateManifestForRoot(root: string, runId: string, manifest: TeamRunM
 		const stateRoot = resolveContainedRelativePath(root, runId, "runId");
 		const crewRoot = path.dirname(path.dirname(root));
 		const artifactsRoot = resolveContainedRelativePath(path.join(crewRoot, DEFAULT_PATHS.state.artifactsSubdir), runId, "runId");
-		if (manifest.runId !== runId || !sameFilesystemPath(manifest.stateRoot, stateRoot) || !sameFilesystemPath(manifest.tasksPath, path.join(stateRoot, DEFAULT_PATHS.state.tasksFile)) || !sameFilesystemPath(manifest.eventsPath, path.join(stateRoot, DEFAULT_PATHS.state.eventsFile)) || !sameFilesystemPath(manifest.artifactsRoot, artifactsRoot)) return false;
+		if (
+			manifest.runId !== runId ||
+			!sameFilesystemPath(manifest.stateRoot, stateRoot) ||
+			!sameFilesystemPath(manifest.tasksPath, path.join(stateRoot, DEFAULT_PATHS.state.tasksFile)) ||
+			!sameFilesystemPath(manifest.eventsPath, path.join(stateRoot, DEFAULT_PATHS.state.eventsFile)) ||
+			!sameFilesystemPath(manifest.artifactsRoot, artifactsRoot)
+		)
+			return false;
 		if (fs.existsSync(artifactsRoot)) {
 			if (fs.lstatSync(artifactsRoot).isSymbolicLink()) return false;
 			resolveRealContainedPath(path.dirname(artifactsRoot), path.basename(artifactsRoot));
@@ -125,7 +132,10 @@ function collectRoots(root: string): ParsedEntry[] {
 	}
 	return entries
 		.filter((entry) => entry.length > 0 && isSafePathId(entry))
-		.map((entry) => ({ runId: entry, path: manifestPathForRun(root, entry) }))
+		.map((entry) => ({
+			runId: entry,
+			path: manifestPathForRun(root, entry),
+		}))
 		.filter((entry): entry is ParsedEntry => entry.path !== undefined);
 }
 
@@ -201,7 +211,10 @@ export function createManifestCache(cwd: string, options: ManifestCacheOptions =
 		}
 		const parsedEntries = [
 			...roots.flatMap((root) => collectRoots(root)),
-			...activeRunEntries().map((entry) => ({ runId: entry.runId, path: entry.manifestPath })),
+			...activeRunEntries().map((entry) => ({
+				runId: entry.runId,
+				path: entry.manifestPath,
+			})),
 		];
 		const unique = new Map<string, CachedManifest | undefined>();
 		for (const entry of parsedEntries) {
@@ -216,7 +229,6 @@ export function createManifestCache(cwd: string, options: ManifestCacheOptions =
 			if (cached) unique.set(entry.runId, cached);
 		}
 
-
 		const runs = [...unique.values()].filter((value): value is CachedManifest => value !== undefined).map((value) => value.manifest);
 		const sorted = runs.sort((a, b) => (b.createdAt ?? "").localeCompare(a.createdAt ?? ""));
 		const limited = sorted.slice(0, Math.max(0, limit));
@@ -238,11 +250,15 @@ export function createManifestCache(cwd: string, options: ManifestCacheOptions =
 
 	if (options.watch ?? true) {
 		for (const root of roots) {
-			const watcher = watchWithErrorHandler(root, () => {
-				scheduleListRefresh();
-			}, () => {
-				scheduleListRefresh();
-			});
+			const watcher = watchWithErrorHandler(
+				root,
+				() => {
+					scheduleListRefresh();
+				},
+				() => {
+					scheduleListRefresh();
+				},
+			);
 			if (watcher) {
 				watcher.unref();
 				watchers.push(watcher);

package/src/runtime/mcp-proxy.ts

@@ -34,10 +34,7 @@ export interface McpProxyConfig {
  * @param options.parentMcpTools — MCP tool names from the parent session (if available)
  * @param options.shareMcp — Whether to share MCP connections (default: true)
  */
-export function buildMcpProxyConfig(options: {
-	parentMcpTools?: string[];
-	shareMcp?: boolean;
-}): McpProxyConfig {
+export function buildMcpProxyConfig(options: { parentMcpTools?: string[]; shareMcp?: boolean }): McpProxyConfig {
 	if (options.shareMcp === false) {
 		return { enableMcp: true, proxyTools: [], proxyToolNames: [] };
 	}
@@ -74,9 +71,7 @@ export function buildMcpProxyConfig(options: {
  * In a future iteration, these can be wired to the actual MCP connections
  * via an inter-process bridge.
  */
-function createMcpProxyTools(
-	toolNames: string[],
-): Array<ToolDefinition<TSchema, unknown>> {
+function createMcpProxyTools(toolNames: string[]): Array<ToolDefinition<TSchema, unknown>> {
 	// For now, we don't create individual proxy tools because we can't
 	// forward MCP calls without the parent's MCP manager reference.
 	//
@@ -93,10 +88,7 @@ function createMcpProxyTools(
  */
 export function discoverMcpToolNames(activeToolNames: string[]): string[] {
 	return activeToolNames.filter(
-		(name) =>
-			name.startsWith("mcp__") ||
-			name.startsWith("mcp-") ||
-			(name.includes("__") && !name.startsWith("submit_result")),
+		(name) => name.startsWith("mcp__") || name.startsWith("mcp-") || (name.includes("__") && !name.startsWith("submit_result")),
 	);
 }
 
@@ -104,10 +96,10 @@ export function discoverMcpToolNames(activeToolNames: string[]): string[] {
  * Build MCP proxy config from a real Pi SDK session's active tools.
  * This is the preferred way — inspect what the parent session has available.
  */
-export function buildMcpProxyFromSession(
-	activeToolNames: string[],
-	options?: { shareMcp?: boolean },
-): McpProxyConfig {
+export function buildMcpProxyFromSession(activeToolNames: string[], options?: { shareMcp?: boolean }): McpProxyConfig {
 	const mcpTools = discoverMcpToolNames(activeToolNames);
-	return buildMcpProxyConfig({ parentMcpTools: mcpTools, shareMcp: options?.shareMcp });
+	return buildMcpProxyConfig({
+		parentMcpTools: mcpTools,
+		shareMcp: options?.shareMcp,
+	});
 }

package/src/runtime/metric-parser.ts

@@ -1,11 +1,7 @@
 /**
  * Metric names that are denied to prevent prototype pollution.
  */
-export const DENIED_METRIC_NAMES: ReadonlySet<string> = new Set([
-	"__proto__",
-	"constructor",
-	"prototype",
-]);
+export const DENIED_METRIC_NAMES: ReadonlySet<string> = new Set(["__proto__", "constructor", "prototype"]);
 
 const METRIC_LINE_RE = /^CREW_METRIC\s+(\w+)=(\S+)$/;
 

package/src/runtime/model-fallback.ts

@@ -2,8 +2,8 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import { errors } from "../errors.ts";
-import { checkModelScope } from "./model-scope.ts";
 import { fuzzyResolveModelId } from "./model-resolver.ts";
+import { checkModelScope } from "./model-scope.ts";
 
 export interface AvailableModelInfo {
 	provider: string;
@@ -46,13 +46,22 @@ function modelInfoFromUnknown(value: unknown): AvailableModelInfo | undefined {
 	if (!value || typeof value !== "object" || Array.isArray(value)) return undefined;
 	const record = value as ModelLike;
 	if (typeof record.provider !== "string" || typeof record.id !== "string") return undefined;
-	return { provider: record.provider, id: record.id, fullId: `${record.provider}/${record.id}` };
+	return {
+		provider: record.provider,
+		id: record.id,
+		fullId: `${record.provider}/${record.id}`,
+	};
 }
 
 export function availableModelInfosFromRegistry(registry: unknown): AvailableModelInfo[] | undefined {
 	if (!registry || typeof registry !== "object" || Array.isArray(registry)) return undefined;
 	const candidate = registry as ModelRegistryLike;
-	const raw = typeof candidate.getAvailable === "function" ? candidate.getAvailable() : typeof candidate.getAll === "function" ? candidate.getAll() : undefined;
+	const raw =
+		typeof candidate.getAvailable === "function"
+			? candidate.getAvailable()
+			: typeof candidate.getAll === "function"
+				? candidate.getAll()
+				: undefined;
 	if (!Array.isArray(raw)) return undefined;
 	return raw.map(modelInfoFromUnknown).filter((entry): entry is AvailableModelInfo => entry !== undefined);
 }
@@ -74,7 +83,7 @@ function readJsonObject(filePath: string): Record<string, unknown> | undefined {
 	try {
 		if (!fs.existsSync(filePath)) return undefined;
 		const parsed = JSON.parse(fs.readFileSync(filePath, "utf-8"));
-		return parsed && typeof parsed === "object" && !Array.isArray(parsed) ? parsed as Record<string, unknown> : undefined;
+		return parsed && typeof parsed === "object" && !Array.isArray(parsed) ? (parsed as Record<string, unknown>) : undefined;
 	} catch {
 		return undefined;
 	}
@@ -92,7 +101,11 @@ function piAgentDir(): string {
 
 function settingsModelInfo(settings: PiSettingsLike | undefined): AvailableModelInfo | undefined {
 	if (typeof settings?.defaultProvider !== "string" || typeof settings.defaultModel !== "string") return undefined;
-	return { provider: settings.defaultProvider, id: settings.defaultModel, fullId: `${settings.defaultProvider}/${settings.defaultModel}` };
+	return {
+		provider: settings.defaultProvider,
+		id: settings.defaultModel,
+		fullId: `${settings.defaultProvider}/${settings.defaultModel}`,
+	};
 }
 
 function modelsJsonInfos(modelsJson: PiModelsJsonLike | undefined): AvailableModelInfo[] {
@@ -118,8 +131,11 @@ function modelsJsonInfos(modelsJson: PiModelsJsonLike | undefined): AvailableMod
 export function configuredModelInfosFromPiConfig(cwd?: string): AvailableModelInfo[] {
 	const agentDir = piAgentDir();
 	const globalSettings = readJsonObject(path.join(agentDir, "settings.json")) as PiSettingsLike | undefined;
-	const projectSettings = cwd ? readJsonObject(path.join(cwd, ".pi", "settings.json")) as PiSettingsLike | undefined : undefined;
-	const effectiveSettings = { ...(globalSettings ?? {}), ...(projectSettings ?? {}) };
+	const projectSettings = cwd ? (readJsonObject(path.join(cwd, ".pi", "settings.json")) as PiSettingsLike | undefined) : undefined;
+	const effectiveSettings = {
+		...(globalSettings ?? {}),
+		...(projectSettings ?? {}),
+	};
 	const defaultModel = settingsModelInfo(effectiveSettings);
 	return uniqueModelInfos([
 		...(defaultModel ? [defaultModel] : []),
@@ -127,7 +143,10 @@ export function configuredModelInfosFromPiConfig(cwd?: string): AvailableModelIn
 	]);
 }
 
-export function splitThinkingSuffix(model: string): { baseModel: string; thinkingSuffix: string } {
+export function splitThinkingSuffix(model: string): {
+	baseModel: string;
+	thinkingSuffix: string;
+} {
 	const colonIdx = model.lastIndexOf(":");
 	if (colonIdx === -1) return { baseModel: model, thinkingSuffix: "" };
 	return {
@@ -251,7 +270,9 @@ export function isRetryableModelFailure(error: string | undefined): boolean {
 
 export function formatModelAttemptNote(attempt: ModelAttemptSummary, nextModel?: string): string {
 	const failure = attempt.error?.trim() || `exit ${attempt.exitCode ?? 1}`;
-	return nextModel ? `[fallback] ${attempt.model} failed: ${failure}. Retrying with ${nextModel}.` : `[fallback] ${attempt.model} failed: ${failure}.`;
+	return nextModel
+		? `[fallback] ${attempt.model} failed: ${failure}. Retrying with ${nextModel}.`
+		: `[fallback] ${attempt.model} failed: ${failure}.`;
 }
 
 export function buildModelCandidates(
@@ -321,16 +342,31 @@ export function buildConfiguredModelRouting(input: {
 }): ConfiguredModelRouting {
 	const registryModels = availableModelInfosFromRegistry(input.modelRegistry);
 	const configModels = configuredModelInfosFromPiConfig(input.cwd);
-	const availableModels = registryModels && registryModels.length > 0 ? registryModels : configModels.length > 0 ? configModels : registryModels;
+	const availableModels =
+		registryModels && registryModels.length > 0 ? registryModels : configModels.length > 0 ? configModels : registryModels;
 	const parentModel = modelStringFromUnknown(input.parentModel);
 	const preferredProvider = parentModel?.split("/")[0] ?? availableModels?.[0]?.provider;
 	// B3: Parent model inheritance — when agent has no model specified,
 	// inherit from parent session model before falling back to defaults.
 	const effectiveAgentModel = input.agentModel?.trim() ? input.agentModel : parentModel;
-	const requested = [input.overrideModel, input.stepModel, input.teamRoleModel, effectiveAgentModel].find((model): model is string => Boolean(model?.trim()));
-	if (availableModels && availableModels.length === 0) return { requested, candidates: [], reason: "no configured Pi models available" };
+	const requested = [input.overrideModel, input.stepModel, input.teamRoleModel, effectiveAgentModel].find((model): model is string =>
+		Boolean(model?.trim()),
+	);
+	if (availableModels && availableModels.length === 0)
+		return {
+			requested,
+			candidates: [],
+			reason: "no configured Pi models available",
+		};
 	const rawModels = availableModels
-		? [input.overrideModel, input.stepModel, input.teamRoleModel, effectiveAgentModel, ...(input.fallbackModels ?? []), ...availableModels.map((model) => model.fullId)]
+		? [
+				input.overrideModel,
+				input.stepModel,
+				input.teamRoleModel,
+				effectiveAgentModel,
+				...(input.fallbackModels ?? []),
+				...availableModels.map((model) => model.fullId),
+			]
 		: [input.overrideModel, input.stepModel, input.teamRoleModel, effectiveAgentModel, ...(input.fallbackModels ?? []), parentModel];
 	// Fix (Round 18): when an agent has `model: false` (frontmatter) the
 	// inherited `parentModel` (= session chính's model, e.g. minimax-M3) IS the
@@ -345,11 +381,12 @@ export function buildConfiguredModelRouting(input: {
 			return isAvailableModel(model.trim(), availableModels);
 		});
 	const candidates = buildModelCandidates(configuredModels[0], configuredModels.slice(1), availableModels, preferredProvider);
-	const reason = requested && candidates[0] && resolveModelCandidate(requested, availableModels, preferredProvider) !== candidates[0]
-		? "requested model unavailable; selected configured Pi fallback"
-		: candidates.length > 1
-			? "configured Pi fallback chain"
-			: undefined;
+	const reason =
+		requested && candidates[0] && resolveModelCandidate(requested, availableModels, preferredProvider) !== candidates[0]
+			? "requested model unavailable; selected configured Pi fallback"
+			: candidates.length > 1
+				? "configured Pi fallback chain"
+				: undefined;
 	// F7 scope gate: when `scopeModelsPatterns` is configured, check the
 	// resolved model. Caller-supplied (override/step/team role) out-of-scope
 	// is a HARD ERROR (we surface it via the verdict AND throw, so spawn aborts

package/src/runtime/model-resolver.ts

@@ -60,9 +60,7 @@ export function resolveModel(input: string, registry: ModelRegistry): Model | st
 		} else if (name.includes(query)) {
 			score = 40 + (query.length / name.length) * 20;
 		} else if (
-			query
-				.split(/[\s\-/]+/)
-				.every((part) => id.includes(part) || name.includes(part) || m.provider.toLowerCase().includes(part))
+			query.split(/[\s\-/]+/).every((part) => id.includes(part) || name.includes(part) || m.provider.toLowerCase().includes(part))
 		) {
 			score = 20;
 		}
@@ -113,9 +111,7 @@ export function fuzzyResolveModelId(input: string, models: SimpleModelEntry[]):
 		} else if (name.includes(query)) {
 			score = 40 + (query.length / (name.length || 1)) * 20;
 		} else if (
-			query
-				.split(/[\s\-/]+/)
-				.every((part) => id.includes(part) || name.includes(part) || m.provider.toLowerCase().includes(part))
+			query.split(/[\s\-/]+/).every((part) => id.includes(part) || name.includes(part) || m.provider.toLowerCase().includes(part))
 		) {
 			score = 20;
 		}

package/src/runtime/model-scope.ts

@@ -92,7 +92,12 @@ export function checkModelScope(
 	source: ModelScopeSource,
 ): ModelScopeCheck {
 	if (!modelId) {
-		return { inScope: true, source, model: "", reason: "no model specified" };
+		return {
+			inScope: true,
+			source,
+			model: "",
+			reason: "no model specified",
+		};
 	}
 	if (!patterns || patterns.length === 0) {
 		// No allowlist → not enforcing. The toggle is opt-in; the user hasn't
@@ -101,7 +106,12 @@ export function checkModelScope(
 	}
 	for (const pattern of patterns) {
 		if (matchesModelPattern(modelId, pattern)) {
-			return { inScope: true, source, model: modelId, matchedPattern: pattern };
+			return {
+				inScope: true,
+				source,
+				model: modelId,
+				matchedPattern: pattern,
+			};
 		}
 	}
 	return {
@@ -131,7 +141,13 @@ export async function readEnabledModelsPatterns(cwd: string, agentDir?: string):
 		// LAZY: defer dynamic import of @earendil-works/pi-coding-agent to its call site.
 		const mod = await import("@earendil-works/pi-coding-agent" as string).catch(() => null);
 		if (!mod) return [];
-		const SettingsManagerCtor = (mod as { SettingsManager?: { create?: (cwd: string, agentDir?: string) => { getEnabledModels?: () => string[] | undefined } } }).SettingsManager;
+		const SettingsManagerCtor = (
+			mod as {
+				SettingsManager?: {
+					create?: (cwd: string, agentDir?: string) => { getEnabledModels?: () => string[] | undefined };
+				};
+			}
+		).SettingsManager;
 		if (!SettingsManagerCtor?.create) return [];
 		const sm = SettingsManagerCtor.create(cwd, agentDir);
 		const patterns = sm.getEnabledModels?.();

package/src/runtime/notebook-helpers.ts

@@ -1,90 +1,88 @@
 export interface NotebookCell {
-  index: number;
-  cellType: "code" | "markdown" | "raw";
-  source: string;
-  outputs?: unknown[];
-  metadata?: Record<string, unknown>;
+	index: number;
+	cellType: "code" | "markdown" | "raw";
+	source: string;
+	outputs?: unknown[];
+	metadata?: Record<string, unknown>;
 }
 
 export interface Notebook {
-  cells: NotebookCell[];
-  metadata?: Record<string, unknown>;
-  nbformat: number;
+	cells: NotebookCell[];
+	metadata?: Record<string, unknown>;
+	nbformat: number;
 }
 
 /** Check if a file path is a notebook */
 export function isNotebookPath(filePath: string): boolean {
-  return filePath.toLowerCase().endsWith(".ipynb");
+	return filePath.toLowerCase().endsWith(".ipynb");
 }
 
 function normalizeSource(source: unknown): string {
-  if (typeof source === "string") return source;
-  if (Array.isArray(source)) return source.join("");
-  return "";
+	if (typeof source === "string") return source;
+	if (Array.isArray(source)) return source.join("");
+	return "";
 }
 
 /** Parse a .ipynb JSON file into a Notebook structure */
 export function parseNotebook(content: string): Notebook {
-  let raw: unknown;
-  try {
-    raw = JSON.parse(content);
-  } catch {
-    return { cells: [], nbformat: 4 };
-  }
-  if (typeof raw !== "object" || raw === null) return { cells: [], nbformat: 4 };
-  const obj = raw as Record<string, unknown>;
-  const nbformat = typeof obj.nbformat === "number" ? obj.nbformat : 4;
-  const rawCells = Array.isArray(obj.cells) ? obj.cells : [];
-  const cells: NotebookCell[] = rawCells
-    .map((c: unknown, i: number): NotebookCell | null => {
-      if (typeof c !== "object" || c === null) return null;
-      const cell = c as Record<string, unknown>;
-      const cellType = cell.cell_type;
-      if (cellType !== "code" && cellType !== "markdown" && cellType !== "raw") return null;
-      return {
-        index: i,
-        cellType,
-        source: normalizeSource(cell.source),
-        outputs: Array.isArray(cell.outputs) ? cell.outputs : undefined,
-        metadata:
-          cell.metadata && typeof cell.metadata === "object" && !Array.isArray(cell.metadata)
-            ? (cell.metadata as Record<string, unknown>)
-            : undefined,
-      };
-    })
-    .filter((c: NotebookCell | null): c is NotebookCell => c !== null);
-  const metadata =
-    obj.metadata && typeof obj.metadata === "object" && !Array.isArray(obj.metadata)
-      ? (obj.metadata as Record<string, unknown>)
-      : undefined;
-  return { cells, metadata, nbformat };
+	let raw: unknown;
+	try {
+		raw = JSON.parse(content);
+	} catch {
+		return { cells: [], nbformat: 4 };
+	}
+	if (typeof raw !== "object" || raw === null) return { cells: [], nbformat: 4 };
+	const obj = raw as Record<string, unknown>;
+	const nbformat = typeof obj.nbformat === "number" ? obj.nbformat : 4;
+	const rawCells = Array.isArray(obj.cells) ? obj.cells : [];
+	const cells: NotebookCell[] = rawCells
+		.map((c: unknown, i: number): NotebookCell | null => {
+			if (typeof c !== "object" || c === null) return null;
+			const cell = c as Record<string, unknown>;
+			const cellType = cell.cell_type;
+			if (cellType !== "code" && cellType !== "markdown" && cellType !== "raw") return null;
+			return {
+				index: i,
+				cellType,
+				source: normalizeSource(cell.source),
+				outputs: Array.isArray(cell.outputs) ? cell.outputs : undefined,
+				metadata:
+					cell.metadata && typeof cell.metadata === "object" && !Array.isArray(cell.metadata)
+						? (cell.metadata as Record<string, unknown>)
+						: undefined,
+			};
+		})
+		.filter((c: NotebookCell | null): c is NotebookCell => c !== null);
+	const metadata =
+		obj.metadata && typeof obj.metadata === "object" && !Array.isArray(obj.metadata)
+			? (obj.metadata as Record<string, unknown>)
+			: undefined;
+	return { cells, metadata, nbformat };
 }
 
 /** Get a specific cell by index */
 export function getCell(notebook: Notebook, index: number): NotebookCell | undefined {
-  return notebook.cells.find((c) => c.index === index);
+	return notebook.cells.find((c) => c.index === index);
 }
 
 /** Update a cell's source content, returning a new Notebook */
 export function updateCell(notebook: Notebook, index: number, source: string): Notebook {
-  const cells = notebook.cells.map((c) =>
-    c.index === index ? { ...c, source } : c,
-  );
-  return { ...notebook, cells };
+	const cells = notebook.cells.map((c) => (c.index === index ? { ...c, source } : c));
+	return { ...notebook, cells };
 }
 
 /** Serialize a Notebook back to .ipynb JSON string */
 export function serializeNotebook(notebook: Notebook): string {
-  const raw = {
-    nbformat: notebook.nbformat,
-    nbformat_minor: 5,
-    metadata: notebook.metadata ?? {},
-    cells: notebook.cells.map((c) => ({
-      cell_type: c.cellType,
-      source: c.source,
-      metadata: c.metadata ?? {},
-      ...(c.cellType === "code" ? { outputs: c.outputs ?? [], execution_count: null } : {}),
-    })),
-  };
-  return JSON.stringify(raw, null, 2) + "\n";
+	const raw = {
+		nbformat: notebook.nbformat,
+		nbformat_minor: 5,
+		metadata: notebook.metadata ?? {},
+		cells: notebook.cells.map((c) => ({
+			cell_type: c.cellType,
+			source: c.source,
+			metadata: c.metadata ?? {},
+			...(c.cellType === "code" ? { outputs: c.outputs ?? [], execution_count: null } : {}),
+		})),
+	};
+	return JSON.stringify(raw, null, 2) + "\n";
 }

package/src/runtime/orphan-worker-registry.ts

@@ -21,13 +21,14 @@
  * cleanupOrphanWorkers) are protected by file locking to prevent concurrent
  * writes from causing lost updates.
  */
+
+import { execFileSync, execSync } from "node:child_process";
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { execSync, execFileSync } from "node:child_process";
-import { userPiRoot } from "../utils/paths.ts";
-import { logInternalError } from "../utils/internal-error.ts";
-import { withFileLockSync } from "../state/locks.ts";
 import { isSymlinkSafePath } from "../state/atomic-write.ts";
+import { withFileLockSync } from "../state/locks.ts";
+import { logInternalError } from "../utils/internal-error.ts";
+import { userPiRoot } from "../utils/paths.ts";
 
 const STALE_REGISTRATION_MS = 60 * 60 * 1000; // 1 hour
 // Grace period before a fresh worker can be cleaned up when parent is dead.
@@ -95,7 +96,10 @@ function getProcessStartTimeLinux(pid: number): number | undefined {
 		// utime(12) stime(13) cutime(14) cstime(15) priority(16) nice(17)
 		// num_threads(18) itrealvalue(19) starttime(20) vsize(21) rss(22)
 		// ...but we only need starttime which is field 22 (index 21 after comm)
-		const fieldsAfterComm = stat.slice(lastParen + 1).trim().split(/\s+/);
+		const fieldsAfterComm = stat
+			.slice(lastParen + 1)
+			.trim()
+			.split(/\s+/);
 		// starttime is at index 19 (the 20th field after comm)
 		const startTimeClockTicks = Number(fieldsAfterComm[19]);
 		if (!Number.isFinite(startTimeClockTicks)) return undefined;
@@ -139,11 +143,10 @@ function getProcessStartTimeWindows(pid: number): number | undefined {
 	// For Node.js without native modules, use tasklist /v and parse output
 	try {
 		// /v verbose, /fo csv, /nh no header
-		const output = execFileSync(
-			"powershell",
-			["-Command", `Get-Process -Id ${pid} | Select-Object -ExpandProperty StartTime`],
-			{ encoding: "utf-8", timeout: 5000 },
-		).trim();
+		const output = execFileSync("powershell", ["-Command", `Get-Process -Id ${pid} | Select-Object -ExpandProperty StartTime`], {
+			encoding: "utf-8",
+			timeout: 5000,
+		}).trim();
 		if (!output) return undefined;
 		const date = new Date(output);
 		if (Number.isNaN(date.getTime())) return undefined;
@@ -153,7 +156,6 @@ function getProcessStartTimeWindows(pid: number): number | undefined {
 	}
 }
 
-
 let REGISTRY_PATH = path.join(userPiRoot(), "state", "orphan-workers.json");
 
 /** @internal Test-only: override the registry path. */
@@ -213,7 +215,11 @@ function writeRegistry(entries: OrphanWorkerEntry[]): void {
 	// writing to disk.
 	for (const entry of entries) {
 		if (!isValidId(entry.sessionId) || !isValidId(entry.runId)) {
-			logInternalError("orphan-worker-registry.write", new Error("Refusing to write: invalid sessionId or runId"), `sessionId=${entry.sessionId} runId=${entry.runId}`);
+			logInternalError(
+				"orphan-worker-registry.write",
+				new Error("Refusing to write: invalid sessionId or runId"),
+				`sessionId=${entry.sessionId} runId=${entry.runId}`,
+			);
 			return;
 		}
 	}
@@ -224,12 +230,20 @@ function writeRegistry(entries: OrphanWorkerEntry[]): void {
 	// isSymlinkSafePath walks the ancestor chain to detect any symlinks,
 	// preventing attacks where an intermediate ancestor is a symlink.
 	if (!isSymlinkSafePath(p)) {
-		logInternalError("orphan-worker-registry.write", new Error("Refusing to write: target is a symlink or inside untrusted directory"), `path=${p}`);
+		logInternalError(
+			"orphan-worker-registry.write",
+			new Error("Refusing to write: target is a symlink or inside untrusted directory"),
+			`path=${p}`,
+		);
 		return;
 	}
 	// Issue 2 fix: Check parent directory safety immediately before creating it.
 	if (!isSymlinkSafePath(dir)) {
-		logInternalError("orphan-worker-registry.write", new Error("Refusing to create: parent directory is a symlink or inside untrusted directory"), `dir=${dir}`);
+		logInternalError(
+			"orphan-worker-registry.write",
+			new Error("Refusing to create: parent directory is a symlink or inside untrusted directory"),
+			`dir=${dir}`,
+		);
 		return;
 	}
 	// Ensure parent directory exists to serialize directory
@@ -240,11 +254,7 @@ function writeRegistry(entries: OrphanWorkerEntry[]): void {
 	try {
 		fs.writeFileSync(p, JSON.stringify(entries, null, 2), { mode: 0o600 });
 	} catch (error) {
-		logInternalError(
-			"orphan-worker-registry.write",
-			error,
-			`path=${p} entries=${entries.length}`,
-		);
+		logInternalError("orphan-worker-registry.write", error, `path=${p} entries=${entries.length}`);
 	}
 }
 
@@ -266,9 +276,7 @@ export function registerWorker(
 	// Issue 3 fix: Validate sessionId and runId to prevent path traversal attacks.
 	if (!isValidId(sessionId) || !isValidId(runId)) return;
 	const startTime = getProcessStartTime(pid) ?? 0;
-	const parentPidStartTime = Number.isFinite(parentPid) && parentPid > 0
-		? (getProcessStartTime(parentPid) ?? 0)
-		: 0;
+	const parentPidStartTime = Number.isFinite(parentPid) && parentPid > 0 ? (getProcessStartTime(parentPid) ?? 0) : 0;
 	withFileLockSync(getRegistryPath(), () => {
 		const entries = readRegistry();
 		// Dedupe by PID
@@ -322,10 +330,13 @@ export interface CleanupOrphanWorkersResult {
  *   ALWAYS kept regardless of age. This protects concurrent sessions.
  *   Pass undefined for unconditional cleanup (e.g. from `pi-crew cleanup`).
  */
-export function cleanupOrphanWorkers(
-	currentSessionId?: string,
-): CleanupOrphanWorkersResult {
-	let result: CleanupOrphanWorkersResult = { scanned: 0, killed: 0, pruned: 0, kept: 0 };
+export function cleanupOrphanWorkers(currentSessionId?: string): CleanupOrphanWorkersResult {
+	let result: CleanupOrphanWorkersResult = {
+		scanned: 0,
+		killed: 0,
+		pruned: 0,
+		kept: 0,
+	};
 	withFileLockSync(getRegistryPath(), () => {
 		const entries = readRegistry();
 		const now = Date.now();