pi-crew 0.2.2 → 0.2.4

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

@@ -1,110 +1,110 @@
-import { loadConfig } from "../../config/config.ts";
-import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
-import { appendEvent, readEvents } from "../../state/event-log.ts";
-import { readDeliveryState, readMailbox } from "../../state/mailbox.ts";
-import { loadRunManifestById, updateRunStatus, saveRunTasks } from "../../state/state-store.ts";
-import { aggregateUsage, formatUsage } from "../../state/usage.ts";
-import { applyAttentionState, formatActivityAge, resolveCrewControlConfig } from "../../runtime/agent-control.ts";
-import { readCrewAgents } from "../../runtime/crew-agent-records.ts";
-import { checkProcessLiveness, isActiveRunStatus } from "../../runtime/process-status.ts";
-import { formatTaskGraphLines, waitingReason } from "../../runtime/task-display.ts";
-import { verifyTaskCompletion, formatOutputPreview } from "../../runtime/completion-guard.ts";
-import { evaluateRunEffectiveness } from "../../runtime/effectiveness.ts";
-import type { PiTeamsToolResult } from "../tool-result.ts";
-import { result, type TeamContext } from "./context.ts";
-
-export function handleStatus(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
-	if (!params.runId) return result("Status requires runId.", { action: "status", status: "error" }, true);
-	const loaded = loadRunManifestById(ctx.cwd, params.runId);
-	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "status", status: "error" }, true);
-	let { manifest, tasks } = loaded;
-	let asyncLivenessLine: string | undefined;
-	if (manifest.async) {
-		const asyncState = manifest.async;
-		const liveness = checkProcessLiveness(asyncState.pid);
-		asyncLivenessLine = `Async: pid=${asyncState.pid ?? "unknown"} alive=${liveness.alive ? "true" : "false"} detail=${liveness.detail} log=${asyncState.logPath} spawnedAt=${asyncState.spawnedAt}`;
-		if (!liveness.alive && isActiveRunStatus(manifest.status)) {
-			manifest = updateRunStatus(manifest, "failed", `Async process stale: ${liveness.detail}`);
-			tasks = tasks.map((task) => task.status === "running" ? { ...task, status: "cancelled" as const, finishedAt: new Date().toISOString(), error: "Async process died; task was not completed." } : task);
-			saveRunTasks(manifest, tasks);
-			appendEvent(manifest.eventsPath, { type: "async.stale", runId: manifest.runId, message: liveness.detail, data: { pid: asyncState.pid } });
-		}
-	}
-	const counts = new Map<string, number>();
-	for (const task of tasks) counts.set(task.status, (counts.get(task.status) ?? 0) + 1);
-	const allEvents = readEvents(manifest.eventsPath);
-	const events = allEvents.slice(-8);
-	const attentionByTask = new Map(allEvents.filter((event) => event.type === "task.attention" && event.taskId).map((event) => [event.taskId!, event]));
-	const controlConfig = resolveCrewControlConfig(loadConfig(ctx.cwd).config);
-	const crewAgents = readCrewAgents(manifest).map((agent) => applyAttentionState(manifest, agent, controlConfig));
-	const artifactLines = manifest.artifacts.slice(-10).map((artifact) => `- ${artifact.kind}: ${artifact.path}${artifact.sizeBytes !== undefined ? ` (${artifact.sizeBytes} bytes)` : ""}`);
-	const deliveryState = readDeliveryState(manifest);
-	const ackTimeoutMs = loadConfig(ctx.cwd).config.runtime?.groupJoinAckTimeoutMs;
-	const groupJoinLines: string[] = [];
-	for (const message of readMailbox(manifest, "outbox").filter((m) => m.data?.kind === "group_join").slice(-5)) {
-		const ack = deliveryState.messages[message.id] === "acknowledged" ? "acknowledged" : "pending";
-		const ageMs = Date.now() - new Date(message.createdAt).getTime();
-		const requestId = String(message.data?.requestId ?? "unknown");
-		const timedOut = ack === "pending" && ackTimeoutMs !== undefined && Number.isFinite(ageMs) && ageMs > ackTimeoutMs;
-		if (timedOut && !allEvents.some((event) => event.type === "agent.group_join.ack_timeout" && event.data?.requestId === requestId)) {
-			appendEvent(manifest.eventsPath, { type: "agent.group_join.ack_timeout", runId: manifest.runId, message: "Group join delivery ack timed out; mailbox delivery remains the fallback.", data: { requestId, messageId: message.id, batchId: message.data?.batchId, partial: message.data?.partial, ageMs, ackTimeoutMs } });
-		}
-		groupJoinLines.push(`- ${String(message.data?.partial) === "true" ? "partial" : "completed"} request=${requestId} message=${message.id} ack=${timedOut ? "timeout" : ack}`);
-	}
-	const totalUsage = aggregateUsage(tasks);
-	const completedTasks = tasks.filter((task) => task.status === "completed");
-	const effectiveness = evaluateRunEffectiveness({ manifest, tasks, executeWorkers: manifest.runtimeResolution?.kind !== "scaffold", runtimeConfig: loadConfig(ctx.cwd).config.runtime });
-	const noObservedWorkTasks = effectiveness.noObservedWorkTaskIds.map((id) => tasks.find((task) => task.id === id)).filter((task): task is typeof tasks[number] => task !== undefined);
-	const attentionTasks = effectiveness.needsAttentionTaskIds.map((id) => tasks.find((task) => task.id === id)).filter((task): task is typeof tasks[number] => task !== undefined);
-	const activeAgents = crewAgents.filter((agent) => agent.status === "running");
-	const completedAgents = crewAgents.filter((agent) => agent.status !== "running");
-	const waitingTasks = tasks.filter((task) => task.status === "queued" || task.status === "waiting");
-	const agentLine = (agent: typeof crewAgents[number]): string => `- ${agent.id} [${agent.status}] ${agent.role} -> ${agent.agent} runtime=${agent.runtime}${agent.model ? ` model=${agent.model}` : ""}${agent.usage ? ` usage=${formatUsage(agent.usage)}` : ""}${agent.progress?.activityState ? ` activityState=${agent.progress.activityState}` : ""}${formatActivityAge(agent) ? ` activity=${formatActivityAge(agent)}` : ""}${agent.progress?.currentTool ? ` tool=${agent.progress.currentTool}` : ""}${agent.toolUses ? ` tools=${agent.toolUses}` : ""}${!agent.usage && agent.progress?.tokens ? ` tokens=${agent.progress.tokens}` : ""}${agent.progress?.turns ? ` turns=${agent.progress.turns}` : ""}${agent.jsonEvents !== undefined ? ` jsonEvents=${agent.jsonEvents}` : ""}${agent.outputPath ? ` output=${agent.outputPath}` : ""}${agent.transcriptPath ? ` transcript=${agent.transcriptPath}` : ""}${agent.statusPath ? ` status=${agent.statusPath}` : ""}${agent.error ? ` error=${agent.error}` : ""}`;
-	const lines = [
-		`Run: ${manifest.runId}`,
-		`Team: ${manifest.team}`,
-		`Workflow: ${manifest.workflow ?? "(none)"}`,
-		`Status: ${manifest.status}`,
-		`Workspace mode: ${manifest.workspaceMode}`,
-		...(manifest.runtimeResolution ? [`Runtime: ${manifest.runtimeResolution.kind}`, `Runtime safety: ${manifest.runtimeResolution.safety}`, `Runtime requested: ${manifest.runtimeResolution.requestedMode}${manifest.runtimeResolution.reason ? ` (${manifest.runtimeResolution.reason})` : ""}`] : []),
-		`Goal: ${manifest.goal}`,
-		`Created: ${manifest.createdAt}`,
-		`Updated: ${manifest.updatedAt}`,
-		`State: ${manifest.stateRoot}`,
-		`Artifacts: ${manifest.artifactsRoot}`,
-		...(asyncLivenessLine ? [asyncLivenessLine] : []),
-		"Task graph:",
-		...formatTaskGraphLines(tasks),
-		"Tasks:",
-		...(tasks.length ? tasks.map((task) => `- ${task.id} [${task.status}] ${task.role} -> ${task.agent}${task.taskPacket ? ` scope=${task.taskPacket.scope}` : ""}${task.verification ? ` green=${task.verification.observedGreenLevel}/${task.verification.requiredGreenLevel}` : ""}${task.modelAttempts?.length ? ` attempts=${task.modelAttempts.length}` : ""}${task.modelRouting ? ` modelRouting=${task.modelRouting.requested ? `${task.modelRouting.requested}->` : ""}${task.modelRouting.resolved}${task.modelRouting.usedAttempt ? ` attempt=${task.modelRouting.usedAttempt + 1}` : ""}` : ""}${task.agentProgress?.activityState ? ` activityState=${task.agentProgress.activityState}` : ""}${attentionByTask.get(task.id)?.data?.reason ? ` attention=${String(attentionByTask.get(task.id)?.data?.reason)}` : ""}${task.jsonEvents !== undefined ? ` jsonEvents=${task.jsonEvents}` : ""}${task.usage ? ` usage=${JSON.stringify(task.usage)}` : ""}${task.resultArtifact ? ` result=${task.resultArtifact.path}` : ""}${task.transcriptArtifact ? ` transcript=${task.transcriptArtifact.path}` : ""}${task.worktree ? ` worktree=${task.worktree.path}` : ""}${task.error ? ` error=${task.error}` : ""}`) : ["- (none)"]),
-		`Task counts: ${[...counts.entries()].map(([status, count]) => `${status}=${count}`).join(", ") || "none"}`,
-		"Effectiveness:",
-		`- observable=${effectiveness.observable}/${Math.max(1, effectiveness.completed)} completed tasks`,
-		`- workerExecution=${effectiveness.workerExecution} guard=${effectiveness.guardMode} severity=${effectiveness.severity}`,
-		`- noObservedWork=${effectiveness.noObservedWorkTaskIds.length ? effectiveness.noObservedWorkTaskIds.join(",") : "none"}`,
-		`- needsAttention=${effectiveness.needsAttentionTaskIds.length ? effectiveness.needsAttentionTaskIds.join(",") : "none"}`,
-		"Completion verification",
-		...(tasks.filter((t) => t.status === "completed").length ? tasks.filter((t) => t.status === "completed").map((t) => {
-			const guard = verifyTaskCompletion(t, manifest);
-			return `- ${t.id} green=${guard.greenLevel}/3${guard.warnings.length ? ` warnings=[${guard.warnings.join(", ")}]` : ""}`;
-		}) : ["- (no completed tasks)"]),
-		"Active agents:",
-		...(activeAgents.length ? activeAgents.map(agentLine) : ["- (none)"]),
-		"Waiting tasks:",
-		...(waitingTasks.length ? waitingTasks.map((task) => `- ${task.id} [queued] ${task.role} -> ${task.agent} ${waitingReason(task, tasks) ?? "waiting"}`) : ["- (none)"]),
-		"Completed agents:",
-		...(completedAgents.length ? completedAgents.map(agentLine) : ["- (none)"]),
-		"Policy decisions:",
-		...(manifest.policyDecisions?.length ? manifest.policyDecisions.map((item) => `- ${item.action} (${item.reason})${item.taskId ? ` ${item.taskId}` : ""}: ${item.message}`) : ["- (none)"]),
-		`Total usage: ${formatUsage(totalUsage)}`,
-		"Group joins:",
-		...(groupJoinLines.length ? groupJoinLines : ["- (none)"]),
-		"",
-		"Recent artifacts:",
-		...(artifactLines.length ? artifactLines : ["- (none)"]),
-		"",
-		"Recent events:",
-		...(events.length ? events.map((event) => `- ${event.time} ${event.type}${event.taskId ? ` ${event.taskId}` : ""}${event.message ? `: ${event.message}` : ""}`) : ["- (none)"]),
-	];
-	return result(lines.join("\n"), { action: "status", status: "ok", runId: manifest.runId, artifactsRoot: manifest.artifactsRoot, intent: `status ${manifest.runId}: ${manifest.status}` });
-}
+import { loadConfig } from "../../config/config.ts";
+import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
+import { appendEvent, readEvents } from "../../state/event-log.ts";
+import { readDeliveryState, readMailbox } from "../../state/mailbox.ts";
+import { loadRunManifestById, updateRunStatus, saveRunTasks } from "../../state/state-store.ts";
+import { aggregateUsage, formatUsage } from "../../state/usage.ts";
+import { applyAttentionState, formatActivityAge, resolveCrewControlConfig } from "../../runtime/agent-control.ts";
+import { readCrewAgents } from "../../runtime/crew-agent-records.ts";
+import { checkProcessLiveness, isActiveRunStatus } from "../../runtime/process-status.ts";
+import { formatTaskGraphLines, waitingReason } from "../../runtime/task-display.ts";
+import { verifyTaskCompletion, formatOutputPreview } from "../../runtime/completion-guard.ts";
+import { evaluateRunEffectiveness } from "../../runtime/effectiveness.ts";
+import type { PiTeamsToolResult } from "../tool-result.ts";
+import { result, type TeamContext } from "./context.ts";
+
+export function handleStatus(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
+	if (!params.runId) return result("Status requires runId.", { action: "status", status: "error" }, true);
+	const loaded = loadRunManifestById(ctx.cwd, params.runId);
+	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "status", status: "error" }, true);
+	let { manifest, tasks } = loaded;
+	let asyncLivenessLine: string | undefined;
+	if (manifest.async) {
+		const asyncState = manifest.async;
+		const liveness = checkProcessLiveness(asyncState.pid);
+		asyncLivenessLine = `Async: pid=${asyncState.pid ?? "unknown"} alive=${liveness.alive ? "true" : "false"} detail=${liveness.detail} log=${asyncState.logPath} spawnedAt=${asyncState.spawnedAt}`;
+		if (!liveness.alive && isActiveRunStatus(manifest.status)) {
+			manifest = updateRunStatus(manifest, "failed", `Async process stale: ${liveness.detail}`);
+			tasks = tasks.map((task) => task.status === "running" ? { ...task, status: "cancelled" as const, finishedAt: new Date().toISOString(), error: "Async process died; task was not completed." } : task);
+			saveRunTasks(manifest, tasks);
+			appendEvent(manifest.eventsPath, { type: "async.stale", runId: manifest.runId, message: liveness.detail, data: { pid: asyncState.pid } });
+		}
+	}
+	const counts = new Map<string, number>();
+	for (const task of tasks) counts.set(task.status, (counts.get(task.status) ?? 0) + 1);
+	const allEvents = readEvents(manifest.eventsPath);
+	const events = allEvents.slice(-8);
+	const attentionByTask = new Map(allEvents.filter((event) => event.type === "task.attention" && event.taskId).map((event) => [event.taskId!, event]));
+	const controlConfig = resolveCrewControlConfig(loadConfig(ctx.cwd).config);
+	const crewAgents = readCrewAgents(manifest).map((agent) => applyAttentionState(manifest, agent, controlConfig));
+	const artifactLines = manifest.artifacts.slice(-10).map((artifact) => `- ${artifact.kind}: ${artifact.path}${artifact.sizeBytes !== undefined ? ` (${artifact.sizeBytes} bytes)` : ""}`);
+	const deliveryState = readDeliveryState(manifest);
+	const ackTimeoutMs = loadConfig(ctx.cwd).config.runtime?.groupJoinAckTimeoutMs;
+	const groupJoinLines: string[] = [];
+	for (const message of readMailbox(manifest, "outbox").filter((m) => m.data?.kind === "group_join").slice(-5)) {
+		const ack = deliveryState.messages[message.id] === "acknowledged" ? "acknowledged" : "pending";
+		const ageMs = Date.now() - new Date(message.createdAt).getTime();
+		const requestId = String(message.data?.requestId ?? "unknown");
+		const timedOut = ack === "pending" && ackTimeoutMs !== undefined && Number.isFinite(ageMs) && ageMs > ackTimeoutMs;
+		if (timedOut && !allEvents.some((event) => event.type === "agent.group_join.ack_timeout" && event.data?.requestId === requestId)) {
+			appendEvent(manifest.eventsPath, { type: "agent.group_join.ack_timeout", runId: manifest.runId, message: "Group join delivery ack timed out; mailbox delivery remains the fallback.", data: { requestId, messageId: message.id, batchId: message.data?.batchId, partial: message.data?.partial, ageMs, ackTimeoutMs } });
+		}
+		groupJoinLines.push(`- ${String(message.data?.partial) === "true" ? "partial" : "completed"} request=${requestId} message=${message.id} ack=${timedOut ? "timeout" : ack}`);
+	}
+	const totalUsage = aggregateUsage(tasks);
+	const completedTasks = tasks.filter((task) => task.status === "completed");
+	const effectiveness = evaluateRunEffectiveness({ manifest, tasks, executeWorkers: manifest.runtimeResolution?.kind !== "scaffold", runtimeConfig: loadConfig(ctx.cwd).config.runtime });
+	const noObservedWorkTasks = effectiveness.noObservedWorkTaskIds.map((id) => tasks.find((task) => task.id === id)).filter((task): task is typeof tasks[number] => task !== undefined);
+	const attentionTasks = effectiveness.needsAttentionTaskIds.map((id) => tasks.find((task) => task.id === id)).filter((task): task is typeof tasks[number] => task !== undefined);
+	const activeAgents = crewAgents.filter((agent) => agent.status === "running");
+	const completedAgents = crewAgents.filter((agent) => agent.status !== "running");
+	const waitingTasks = tasks.filter((task) => task.status === "queued" || task.status === "waiting");
+	const agentLine = (agent: typeof crewAgents[number]): string => `- ${agent.id} [${agent.status}] ${agent.role} -> ${agent.agent} runtime=${agent.runtime}${agent.model ? ` model=${agent.model}` : ""}${agent.usage ? ` usage=${formatUsage(agent.usage)}` : ""}${agent.progress?.activityState ? ` activityState=${agent.progress.activityState}` : ""}${formatActivityAge(agent) ? ` activity=${formatActivityAge(agent)}` : ""}${agent.progress?.currentTool ? ` tool=${agent.progress.currentTool}` : ""}${agent.toolUses ? ` tools=${agent.toolUses}` : ""}${!agent.usage && agent.progress?.tokens ? ` tokens=${agent.progress.tokens}` : ""}${agent.progress?.turns ? ` turns=${agent.progress.turns}` : ""}${agent.jsonEvents !== undefined ? ` jsonEvents=${agent.jsonEvents}` : ""}${agent.outputPath ? ` output=${agent.outputPath}` : ""}${agent.transcriptPath ? ` transcript=${agent.transcriptPath}` : ""}${agent.statusPath ? ` status=${agent.statusPath}` : ""}${agent.error ? ` error=${agent.error}` : ""}`;
+	const lines = [
+		`Run: ${manifest.runId}`,
+		`Team: ${manifest.team}`,
+		`Workflow: ${manifest.workflow ?? "(none)"}`,
+		`Status: ${manifest.status}`,
+		`Workspace mode: ${manifest.workspaceMode}`,
+		...(manifest.runtimeResolution ? [`Runtime: ${manifest.runtimeResolution.kind}`, `Runtime safety: ${manifest.runtimeResolution.safety}`, `Runtime requested: ${manifest.runtimeResolution.requestedMode}${manifest.runtimeResolution.reason ? ` (${manifest.runtimeResolution.reason})` : ""}`] : []),
+		`Goal: ${manifest.goal}`,
+		`Created: ${manifest.createdAt}`,
+		`Updated: ${manifest.updatedAt}`,
+		`State: ${manifest.stateRoot}`,
+		`Artifacts: ${manifest.artifactsRoot}`,
+		...(asyncLivenessLine ? [asyncLivenessLine] : []),
+		"Task graph:",
+		...formatTaskGraphLines(tasks),
+		"Tasks:",
+		...(tasks.length ? tasks.map((task) => `- ${task.id} [${task.status}] ${task.role} -> ${task.agent}${task.taskPacket ? ` scope=${task.taskPacket.scope}` : ""}${task.verification ? ` green=${task.verification.observedGreenLevel}/${task.verification.requiredGreenLevel}` : ""}${task.modelAttempts?.length ? ` attempts=${task.modelAttempts.length}` : ""}${task.modelRouting ? ` modelRouting=${task.modelRouting.requested ? `${task.modelRouting.requested}->` : ""}${task.modelRouting.resolved}${task.modelRouting.usedAttempt ? ` attempt=${task.modelRouting.usedAttempt + 1}` : ""}` : ""}${task.agentProgress?.activityState ? ` activityState=${task.agentProgress.activityState}` : ""}${attentionByTask.get(task.id)?.data?.reason ? ` attention=${String(attentionByTask.get(task.id)?.data?.reason)}` : ""}${task.jsonEvents !== undefined ? ` jsonEvents=${task.jsonEvents}` : ""}${task.usage ? ` usage=${JSON.stringify(task.usage)}` : ""}${task.resultArtifact ? ` result=${task.resultArtifact.path}` : ""}${task.transcriptArtifact ? ` transcript=${task.transcriptArtifact.path}` : ""}${task.worktree ? ` worktree=${task.worktree.path}` : ""}${task.error ? ` error=${task.error}` : ""}`) : ["- (none)"]),
+		`Task counts: ${[...counts.entries()].map(([status, count]) => `${status}=${count}`).join(", ") || "none"}`,
+		"Effectiveness:",
+		`- observable=${effectiveness.observable}/${Math.max(1, effectiveness.completed)} completed tasks`,
+		`- workerExecution=${effectiveness.workerExecution} guard=${effectiveness.guardMode} severity=${effectiveness.severity}`,
+		`- noObservedWork=${effectiveness.noObservedWorkTaskIds.length ? effectiveness.noObservedWorkTaskIds.join(",") : "none"}`,
+		`- needsAttention=${effectiveness.needsAttentionTaskIds.length ? effectiveness.needsAttentionTaskIds.join(",") : "none"}`,
+		"Completion verification",
+		...(tasks.filter((t) => t.status === "completed").length ? tasks.filter((t) => t.status === "completed").map((t) => {
+			const guard = verifyTaskCompletion(t, manifest);
+			return `- ${t.id} green=${guard.greenLevel}/3${guard.warnings.length ? ` warnings=[${guard.warnings.join(", ")}]` : ""}`;
+		}) : ["- (no completed tasks)"]),
+		"Active agents:",
+		...(activeAgents.length ? activeAgents.map(agentLine) : ["- (none)"]),
+		"Waiting tasks:",
+		...(waitingTasks.length ? waitingTasks.map((task) => `- ${task.id} [queued] ${task.role} -> ${task.agent} ${waitingReason(task, tasks) ?? "waiting"}`) : ["- (none)"]),
+		"Completed agents:",
+		...(completedAgents.length ? completedAgents.map(agentLine) : ["- (none)"]),
+		"Policy decisions:",
+		...(manifest.policyDecisions?.length ? manifest.policyDecisions.map((item) => `- ${item.action} (${item.reason})${item.taskId ? ` ${item.taskId}` : ""}: ${item.message}`) : ["- (none)"]),
+		`Total usage: ${formatUsage(totalUsage)}`,
+		"Group joins:",
+		...(groupJoinLines.length ? groupJoinLines : ["- (none)"]),
+		"",
+		"Recent artifacts:",
+		...(artifactLines.length ? artifactLines : ["- (none)"]),
+		"",
+		"Recent events:",
+		...(events.length ? events.map((event) => `- ${event.time} ${event.type}${event.taskId ? ` ${event.taskId}` : ""}${event.message ? `: ${event.message}` : ""}`) : ["- (none)"]),
+	];
+	return result(lines.join("\n"), { action: "status", status: "ok", runId: manifest.runId, artifactsRoot: manifest.artifactsRoot, intent: `status ${manifest.runId}: ${manifest.status}` });
+}

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

@@ -1,13 +1,13 @@
-export interface TeamToolDetails {
-	action: string;
-	status: "ok" | "error" | "planned";
-	runId?: string;
-	artifactsRoot?: string;
-	abortedIds?: string[];
-	missingIds?: string[];
-	foreignIds?: string[];
-	intent?: string;
-	resumedIds?: string[];
-	retriedTaskIds?: string[];
-	mailboxIds?: string[];
-}
+export interface TeamToolDetails {
+	action: string;
+	status: "ok" | "error" | "planned";
+	runId?: string;
+	artifactsRoot?: string;
+	abortedIds?: string[];
+	missingIds?: string[];
+	foreignIds?: string[];
+	intent?: string;
+	resumedIds?: string[];
+	retriedTaskIds?: string[];
+	mailboxIds?: string[];
+}

package/src/extension/team-tool.ts

@@ -32,6 +32,7 @@ import type { executeTeamRun as ExecuteTeamRunFn } from "../runtime/team-runner.
 let _cachedExecuteTeamRun: typeof ExecuteTeamRunFn | undefined;
 async function executeTeamRun(...args: Parameters<typeof ExecuteTeamRunFn>): Promise<Awaited<ReturnType<typeof ExecuteTeamRunFn>>> {
 	if (!_cachedExecuteTeamRun) {
+		// LAZY: heavy runtime — defer 1.4s import cost until team run actually executes.
 		const mod = await import("../runtime/team-runner.ts");
 		_cachedExecuteTeamRun = mod.executeTeamRun;
 	}
@@ -48,7 +49,18 @@ import { parsePiJsonOutput } from "../runtime/pi-json-output.ts";
 import { buildParentContext, configRecord, formatScoped, result, type TeamContext } from "./team-tool/context.ts";
 import { autonomousPatchFromConfig, configPatchFromConfig, effectiveRunConfig, formatAutonomyStatus } from "./team-tool/config-patch.ts";
 import { handleApi } from "./team-tool/api.ts";
-import { handleRun } from "./team-tool/run.ts";
+// Lazy-loaded: run.ts pulls in spawnBackgroundTeamRun, resolveCrewRuntime, etc.
+// Static import fails silently in some jiti contexts (child-process), leaving handleRun undefined.
+import type { handleRun as HandleRunFn } from "./team-tool/run.ts";
+let _cachedHandleRun: typeof HandleRunFn | undefined;
+async function handleRun(...args: Parameters<typeof HandleRunFn>): Promise<Awaited<ReturnType<typeof HandleRunFn>>> {
+	if (!_cachedHandleRun) {
+		// LAZY: run.ts pulls in spawnBackgroundTeamRun + resolveCrewRuntime; also avoids jiti import race in child-process contexts.
+		const mod = await import("./team-tool/run.ts");
+		_cachedHandleRun = mod.handleRun;
+	}
+	return _cachedHandleRun(...args);
+}
 import { handleDoctor } from "./team-tool/doctor.ts";
 import { handleStatus } from "./team-tool/status.ts";
 import { handleArtifacts, handleEvents, handleSummary } from "./team-tool/inspect.ts";
@@ -62,7 +74,7 @@ import { normalizeSkillOverride } from "../runtime/skill-instructions.ts";
 
 export type { TeamToolDetails } from "./team-tool-types.ts";
 export type { TeamContext } from "./team-tool/context.ts";
-export { handleRun } from "./team-tool/run.ts";
+export { handleRun };
 export { handleDoctor } from "./team-tool/doctor.ts";
 export { handleStatus } from "./team-tool/status.ts";
 export { handleArtifacts, handleEvents, handleSummary } from "./team-tool/inspect.ts";
@@ -191,7 +203,7 @@ export async function handleResume(params: TeamToolParamsValue, ctx: TeamContext
 		const loadedConfig = loadConfig(ctx.cwd);
 		const recovered = recoverCheckpointedTasks(loaded.manifest, loaded.tasks);
 		const resumeManifest = recovered.manifest;
-		const executedConfig = effectiveRunConfig(loadedConfig.config, params.config);
+		const executedConfig = { ...effectiveRunConfig(loadedConfig.config, params.config) };
 		// Preserve original manifest scaffold mode when resume has no explicit mode override
 		// AND workers are not explicitly disabled. If workers are disabled, let
 		// resolveCrewRuntime detect it and return blocked safety.
@@ -225,7 +237,7 @@ export async function handleResume(params: TeamToolParamsValue, ctx: TeamContext
 		if (replay.messages.length) appendEvent(runtimeManifest.eventsPath, { type: "mailbox.replayed", runId: runtimeManifest.runId, message: `Replayed ${replay.messages.length} pending inbox message(s).`, data: { messageIds: replay.messages.map((message) => message.id), taskIds: replay.messages.map((message) => message.taskId).filter(Boolean) } });
 		const executeWorkers = runtime.kind !== "scaffold";
 		const resumeSkillOverride = normalizeSkillOverride(params.skill) ?? runtimeManifest.skillOverride;
-		const executed = await executeTeamRun({ manifest: runtimeManifest, tasks: resetTasks, team, workflow, agents, executeWorkers, limits: executedConfig.limits, runtime, runtimeConfig: executedConfig.runtime, parentContext: buildParentContext(ctx), parentModel: ctx.model, modelRegistry: ctx.modelRegistry, modelOverride: params.model, skillOverride: resumeSkillOverride, signal: ctx.signal, reliability: executedConfig.reliability, metricRegistry: ctx.metricRegistry });
+		const executed = await executeTeamRun({ manifest: runtimeManifest, tasks: resetTasks, team, workflow, agents, executeWorkers, limits: executedConfig.limits, runtime, runtimeConfig: executedConfig.runtime, parentContext: buildParentContext(ctx), parentModel: ctx.model, modelRegistry: ctx.modelRegistry, modelOverride: params.model, skillOverride: resumeSkillOverride, signal: ctx.signal, reliability: executedConfig.reliability, metricRegistry: ctx.metricRegistry, workspaceId: ctx.cwd });
 		return result([`Resumed run ${executed.manifest.runId}.`, `Status: ${executed.manifest.status}`, `Tasks: ${executed.tasks.length}`, `Artifacts: ${executed.manifest.artifactsRoot}`].join("\n"), { action: "resume", status: executed.manifest.status === "failed" ? "error" : "ok", runId: executed.manifest.runId, artifactsRoot: executed.manifest.artifactsRoot }, executed.manifest.status === "failed");
 	});
 }

package/src/extension/tool-result.ts

@@ -1,16 +1,16 @@
-import type { AgentToolResult } from "@mariozechner/pi-agent-core";
-import type { TeamToolDetails } from "./team-tool-types.ts";
-
-export type PiTeamsToolResult<TDetails = TeamToolDetails> = AgentToolResult<TDetails> & { isError?: boolean };
-
-export function toolResult<TDetails>(text: string, details: TDetails, isError = false): PiTeamsToolResult<TDetails> {
-	return { content: [{ type: "text", text }], details, isError };
-}
-
-export function isToolError(result: { isError?: boolean }): boolean {
-	return result.isError === true;
-}
-
-export function textFromToolResult(result: { content?: Array<{ type: string; text?: string }> }): string {
-	return result.content?.map((item) => item.text ?? "").join("\n") ?? "";
-}
+import type { AgentToolResult } from "@mariozechner/pi-agent-core";
+import type { TeamToolDetails } from "./team-tool-types.ts";
+
+export type PiTeamsToolResult<TDetails = TeamToolDetails> = AgentToolResult<TDetails> & { isError?: boolean };
+
+export function toolResult<TDetails>(text: string, details: TDetails, isError = false): PiTeamsToolResult<TDetails> {
+	return { content: [{ type: "text", text }], details, isError };
+}
+
+export function isToolError(result: { isError?: boolean }): boolean {
+	return result.isError === true;
+}
+
+export function textFromToolResult(result: { content?: Array<{ type: string; text?: string }> }): string {
+	return result.content?.map((item) => item.text ?? "").join("\n") ?? "";
+}

package/src/extension/validate-resources.ts

@@ -1,77 +1,77 @@
-import { allAgents, discoverAgents } from "../agents/discover-agents.ts";
-import { allTeams, discoverTeams } from "../teams/discover-teams.ts";
-import { allWorkflows, discoverWorkflows } from "../workflows/discover-workflows.ts";
-import { validateWorkflowForTeam } from "../workflows/validate-workflow.ts";
-
-export interface ValidationIssue {
-	level: "error" | "warning";
-	resource: string;
-	message: string;
-}
-
-export interface ValidationReport {
-	issues: ValidationIssue[];
-	agents: number;
-	teams: number;
-	workflows: number;
-}
-
-export function validateResources(cwd: string): ValidationReport {
-	const agents = allAgents(discoverAgents(cwd));
-	const teams = allTeams(discoverTeams(cwd));
-	const workflows = allWorkflows(discoverWorkflows(cwd));
-	const agentNames = new Set(agents.map((agent) => agent.name));
-	const workflowNames = new Set(workflows.map((workflow) => workflow.name));
-	const issues: ValidationIssue[] = [];
-
-	for (const agent of agents) {
-		const modelValues = [agent.model, ...(agent.fallbackModels ?? [])].filter((value): value is string => typeof value === "string" && value.length > 0);
-		for (const model of modelValues) {
-			if (/\s/.test(model)) {
-				issues.push({ level: "warning", resource: `agent:${agent.name}`, message: `Model reference '${model}' contains whitespace.` });
-			}
-			if (model.includes("/") && model.split("/").some((part) => part.trim() === "")) {
-				issues.push({ level: "warning", resource: `agent:${agent.name}`, message: `Model reference '${model}' has an empty provider/model segment.` });
-			}
-		}
-	}
-
-	for (const team of teams) {
-		for (const role of team.roles) {
-			if (!agentNames.has(role.agent)) {
-				issues.push({ level: "error", resource: `team:${team.name}`, message: `Role '${role.name}' references unknown agent '${role.agent}'.` });
-			}
-		}
-		if (team.defaultWorkflow && !workflowNames.has(team.defaultWorkflow)) {
-			issues.push({ level: "error", resource: `team:${team.name}`, message: `defaultWorkflow references unknown workflow '${team.defaultWorkflow}'.` });
-		}
-		const workflow = workflows.find((candidate) => candidate.name === team.defaultWorkflow);
-		if (workflow) {
-			for (const error of validateWorkflowForTeam(workflow, team)) {
-				issues.push({ level: "error", resource: `workflow:${workflow.name}`, message: `Team '${team.name}': ${error}` });
-			}
-		}
-	}
-
-	for (const workflow of workflows) {
-		if (workflow.steps.length === 0) {
-			issues.push({ level: "warning", resource: `workflow:${workflow.name}`, message: "Workflow has no steps." });
-		}
-	}
-
-	return { issues, agents: agents.length, teams: teams.length, workflows: workflows.length };
-}
-
-export function formatValidationReport(report: ValidationReport): string {
-	const lines = [
-		"pi-crew resource validation:",
-		`Agents: ${report.agents}`,
-		`Teams: ${report.teams}`,
-		`Workflows: ${report.workflows}`,
-		`Issues: ${report.issues.length}`,
-	];
-	if (report.issues.length > 0) {
-		lines.push("", ...report.issues.map((issue) => `- ${issue.level.toUpperCase()} ${issue.resource}: ${issue.message}`));
-	}
-	return lines.join("\n");
-}
+import { allAgents, discoverAgents } from "../agents/discover-agents.ts";
+import { allTeams, discoverTeams } from "../teams/discover-teams.ts";
+import { allWorkflows, discoverWorkflows } from "../workflows/discover-workflows.ts";
+import { validateWorkflowForTeam } from "../workflows/validate-workflow.ts";
+
+export interface ValidationIssue {
+	level: "error" | "warning";
+	resource: string;
+	message: string;
+}
+
+export interface ValidationReport {
+	issues: ValidationIssue[];
+	agents: number;
+	teams: number;
+	workflows: number;
+}
+
+export function validateResources(cwd: string): ValidationReport {
+	const agents = allAgents(discoverAgents(cwd));
+	const teams = allTeams(discoverTeams(cwd));
+	const workflows = allWorkflows(discoverWorkflows(cwd));
+	const agentNames = new Set(agents.map((agent) => agent.name));
+	const workflowNames = new Set(workflows.map((workflow) => workflow.name));
+	const issues: ValidationIssue[] = [];
+
+	for (const agent of agents) {
+		const modelValues = [agent.model, ...(agent.fallbackModels ?? [])].filter((value): value is string => typeof value === "string" && value.length > 0);
+		for (const model of modelValues) {
+			if (/\s/.test(model)) {
+				issues.push({ level: "warning", resource: `agent:${agent.name}`, message: `Model reference '${model}' contains whitespace.` });
+			}
+			if (model.includes("/") && model.split("/").some((part) => part.trim() === "")) {
+				issues.push({ level: "warning", resource: `agent:${agent.name}`, message: `Model reference '${model}' has an empty provider/model segment.` });
+			}
+		}
+	}
+
+	for (const team of teams) {
+		for (const role of team.roles) {
+			if (!agentNames.has(role.agent)) {
+				issues.push({ level: "error", resource: `team:${team.name}`, message: `Role '${role.name}' references unknown agent '${role.agent}'.` });
+			}
+		}
+		if (team.defaultWorkflow && !workflowNames.has(team.defaultWorkflow)) {
+			issues.push({ level: "error", resource: `team:${team.name}`, message: `defaultWorkflow references unknown workflow '${team.defaultWorkflow}'.` });
+		}
+		const workflow = workflows.find((candidate) => candidate.name === team.defaultWorkflow);
+		if (workflow) {
+			for (const error of validateWorkflowForTeam(workflow, team)) {
+				issues.push({ level: "error", resource: `workflow:${workflow.name}`, message: `Team '${team.name}': ${error}` });
+			}
+		}
+	}
+
+	for (const workflow of workflows) {
+		if (workflow.steps.length === 0) {
+			issues.push({ level: "warning", resource: `workflow:${workflow.name}`, message: "Workflow has no steps." });
+		}
+	}
+
+	return { issues, agents: agents.length, teams: teams.length, workflows: workflows.length };
+}
+
+export function formatValidationReport(report: ValidationReport): string {
+	const lines = [
+		"pi-crew resource validation:",
+		`Agents: ${report.agents}`,
+		`Teams: ${report.teams}`,
+		`Workflows: ${report.workflows}`,
+		`Issues: ${report.issues.length}`,
+	];
+	if (report.issues.length > 0) {
+		lines.push("", ...report.issues.map((issue) => `- ${issue.level.toUpperCase()} ${issue.resource}: ${issue.message}`));
+	}
+	return lines.join("\n");
+}

package/src/hooks/registry.ts

@@ -1,61 +1,61 @@
-import type { HookDefinition, HookName, HookContext, HookResult, HookExecutionReport } from "./types.ts";
-import { appendEvent } from "../state/event-log.ts";
-import type { TeamRunManifest } from "../state/types.ts";
-import { runEventBus } from "../ui/run-event-bus.ts";
-
-const registry = new Map<HookName, HookDefinition[]>();
-
-export function registerHook(definition: HookDefinition): void {
-	const hooks = registry.get(definition.name) ?? [];
-	hooks.push(definition);
-	registry.set(definition.name, hooks);
-}
-
-export function clearHooks(): void {
-	registry.clear();
-}
-
-export function getHooks(name: HookName): HookDefinition[] {
-	return registry.get(name) ?? [];
-}
-
-export async function executeHook(name: HookName, ctx: HookContext): Promise<HookExecutionReport> {
-	const hooks = getHooks(name);
-	if (hooks.length === 0) return { hookName: name, outcome: "allow", durationMs: 0 };
-	const start = Date.now();
-	const diagnostics: string[] = [];
-	let capturedModifications: Record<string, unknown> | undefined;
-	for (const hook of hooks) {
-		try {
-			const result: HookResult = await hook.handler(ctx);
-			if (hook.mode === "blocking" && result.outcome === "block") {
-				return { hookName: name, outcome: "block", durationMs: Date.now() - start, reason: result.reason };
-			}
-			if (result.outcome === "modify" && result.data) {
-				Object.assign(ctx, result.data);
-				capturedModifications = { ...result.data };
-			}
-		} catch (error) {
-			const message = error instanceof Error ? error.message : String(error);
-			if (hook.mode === "blocking") {
-				return { hookName: name, outcome: "block", durationMs: Date.now() - start, reason: `Hook error: ${message}` };
-			}
-			// Non-blocking hook errors are accumulated as diagnostics; continue to next hook
-			diagnostics.push(message);
-			}
-	}
-	if (diagnostics.length > 0) {
-		return { hookName: name, outcome: "diagnostic", durationMs: Date.now() - start, reason: diagnostics.join("; "), modifiedData: capturedModifications };
-	}
-	return { hookName: name, outcome: "allow", durationMs: Date.now() - start, modifiedData: capturedModifications };
-}
-
-export function appendHookEvent(manifest: TeamRunManifest, report: HookExecutionReport): void {
-	appendEvent(manifest.eventsPath, {
-		type: "hook.executed",
-		runId: manifest.runId,
-		message: `Hook ${report.hookName} completed with outcome=${report.outcome}${report.reason ? `: ${report.reason}` : ""}`,
-		data: { hookName: report.hookName, outcome: report.outcome, durationMs: report.durationMs, reason: report.reason },
-	});
-	runEventBus.emit({ type: "effectiveness_changed", runId: manifest.runId, data: { hookName: report.hookName, outcome: report.outcome } });
-}
+import type { HookDefinition, HookName, HookContext, HookResult, HookExecutionReport } from "./types.ts";
+import { appendEvent } from "../state/event-log.ts";
+import type { TeamRunManifest } from "../state/types.ts";
+import { runEventBus } from "../ui/run-event-bus.ts";
+
+const registry = new Map<HookName, HookDefinition[]>();
+
+export function registerHook(definition: HookDefinition): void {
+	const hooks = registry.get(definition.name) ?? [];
+	hooks.push(definition);
+	registry.set(definition.name, hooks);
+}
+
+export function clearHooks(): void {
+	registry.clear();
+}
+
+export function getHooks(name: HookName): HookDefinition[] {
+	return registry.get(name) ?? [];
+}
+
+export async function executeHook(name: HookName, ctx: HookContext): Promise<HookExecutionReport> {
+	const hooks = getHooks(name);
+	if (hooks.length === 0) return { hookName: name, outcome: "allow", durationMs: 0 };
+	const start = Date.now();
+	const diagnostics: string[] = [];
+	let capturedModifications: Record<string, unknown> | undefined;
+	for (const hook of hooks) {
+		try {
+			const result: HookResult = await hook.handler(ctx);
+			if (hook.mode === "blocking" && result.outcome === "block") {
+				return { hookName: name, outcome: "block", durationMs: Date.now() - start, reason: result.reason };
+			}
+			if (result.outcome === "modify" && result.data) {
+				Object.assign(ctx, result.data);
+				capturedModifications = { ...result.data };
+			}
+		} catch (error) {
+			const message = error instanceof Error ? error.message : String(error);
+			if (hook.mode === "blocking") {
+				return { hookName: name, outcome: "block", durationMs: Date.now() - start, reason: `Hook error: ${message}` };
+			}
+			// Non-blocking hook errors are accumulated as diagnostics; continue to next hook
+			diagnostics.push(message);
+			}
+	}
+	if (diagnostics.length > 0) {
+		return { hookName: name, outcome: "diagnostic", durationMs: Date.now() - start, reason: diagnostics.join("; "), modifiedData: capturedModifications };
+	}
+	return { hookName: name, outcome: "allow", durationMs: Date.now() - start, modifiedData: capturedModifications };
+}
+
+export function appendHookEvent(manifest: TeamRunManifest, report: HookExecutionReport): void {
+	appendEvent(manifest.eventsPath, {
+		type: "hook.executed",
+		runId: manifest.runId,
+		message: `Hook ${report.hookName} completed with outcome=${report.outcome}${report.reason ? `: ${report.reason}` : ""}`,
+		data: { hookName: report.hookName, outcome: report.outcome, durationMs: report.durationMs, reason: report.reason },
+	});
+	runEventBus.emit({ type: "effectiveness_changed", runId: manifest.runId, data: { hookName: report.hookName, outcome: report.outcome } });
+}