pi-crew 0.1.49 → 0.2.0

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

@@ -19,7 +19,7 @@ export type TeamContext = Pick<ExtensionContext, "cwd"> & Partial<Pick<Extension
 };
 
 export function withSessionId<T extends Pick<ExtensionContext, "sessionManager">>(ctx: T): T & { sessionId?: string } {
-	const sessionId = ctx.sessionManager.getSessionId();
+	const sessionId = ctx.sessionManager?.getSessionId?.();
 	return sessionId ? { ...ctx, sessionId } : { ...ctx };
 }
 

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

@@ -10,6 +10,7 @@ import { DEFAULT_PATHS } from "../../config/defaults.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { getPiSpawnCommand } from "../../runtime/pi-spawn.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";
 import type { PiTeamsToolResult } from "../tool-result.ts";
 import { configRecord, result, type TeamContext } from "./context.ts";
@@ -112,9 +113,19 @@ export interface TeamDoctorReportInput {
 export interface TeamDoctorReport {
 	text: string;
 	hasErrors: boolean;
+	drift?: DriftReport;
 }
 
 export function buildTeamDoctorReport(input: TeamDoctorReportInput): TeamDoctorReport {
+	// Compute drift once — reused in both Drift section and return value
+	const driftResult = detectDrift(
+		{
+			agents: allAgents(discoverAgents(input.cwd)).map((a) => a.name),
+			teams: allTeams(discoverTeams(input.cwd)).map((t) => t.name),
+			workflows: allWorkflows(discoverWorkflows(input.cwd)).map((w) => w.name),
+		},
+		loadConfig(input.cwd).config,
+	);
 	const sections = [
 		section("Runtime", () => {
 			const git = commandExists("git", ["--version"]);
@@ -156,6 +167,15 @@ export function buildTeamDoctorReport(input: TeamDoctorReportInput): TeamDoctorR
 			ok: input.validationErrors === 0,
 			detail: `${input.validationErrors} errors, ${input.validationWarnings} warnings`,
 		}]),
+		section("Drift", () => {
+			const driftErrors = driftResult.items.filter((item) => item.severity === "error").length;
+			const driftWarnings = driftResult.items.filter((item) => item.severity === "warning").length;
+			return [{
+				label: "config drift",
+				ok: !driftResult.hasDrift || driftErrors === 0,
+				detail: driftResult.hasDrift ? `${driftErrors} errors, ${driftWarnings} warnings` : "no drift detected",
+			}];
+		}),
 		section("Schema", () => {
 			const schemaIssues = auditJsonSchema(TeamToolParams);
 			return [{ label: "strict-provider schema", ok: schemaIssues.length === 0, detail: schemaIssues.length ? schemaIssues.slice(0, 3).join("; ") : "team tool schema compatible" }];
@@ -181,7 +201,7 @@ export function buildTeamDoctorReport(input: TeamDoctorReportInput): TeamDoctorR
 	}
 	if (lines.at(-1) === "") lines.pop();
 	const text = lines.join("\n");
-	return { text, hasErrors: sections.some((sectionLines) => sectionLines.some((line) => line.includes("FAIL"))) };
+	return { text, hasErrors: sections.some((sectionLines) => sectionLines.some((line) => line.includes("FAIL"))), drift: driftResult.hasDrift ? driftResult : undefined };
 }
 
 export function handleDoctor(ctx: TeamContext, params: TeamToolParamsValue = {}): PiTeamsToolResult {
@@ -203,7 +223,7 @@ export function handleDoctor(ctx: TeamContext, params: TeamToolParamsValue = {})
 		}
 	}
 	const validation = validateResources(ctx.cwd);
-	const { text, hasErrors } = buildTeamDoctorReport({
+	const { text, hasErrors, drift } = buildTeamDoctorReport({
 		cwd: ctx.cwd,
 		configPath: loadedConfig.path,
 		configErrors: loadedConfig.error ? [loadedConfig.error] : [],
@@ -213,5 +233,10 @@ export function handleDoctor(ctx: TeamContext, params: TeamToolParamsValue = {})
 		validationWarnings: validation.issues.filter((issue) => issue.level === "warning").length,
 		smokeChildPi,
 	});
-	return result(text, { action: "doctor", status: hasErrors ? "error" : "ok" }, hasErrors);
+	// Append detailed drift section if any drift was detected
+	let finalText = text;
+	if (drift?.hasDrift) {
+		finalText = `${text}\n\nDrift details:\n${formatDriftReport(drift)}`;
+	}
+	return result(finalText, { action: "doctor", status: hasErrors ? "error" : "ok" }, hasErrors);
 }

package/src/extension/team-tool/handle-settings.ts

@@ -1,188 +1,195 @@
-import type { TeamContext } from "../team-tool/context.ts";
-import { loadConfig, updateConfig } from "../../config/config.ts";
-import { configPatchFromConfig } from "../team-tool/config-patch.ts";
-import { result } from "../team-tool/context.ts";
-import type { PiTeamsToolResult } from "../tool-result.ts";
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-function setNested(obj: Record<string, unknown>, path: string, value: unknown): void {
-	const keys = path.split(".");
-	let target: Record<string, unknown> = obj;
-	for (let i = 0; i < keys.length - 1; i++) {
-		if (!target[keys[i]] || typeof target[keys[i]] !== "object") {
-			target[keys[i]] = {};
-		}
-		target = target[keys[i]] as Record<string, unknown>;
-	}
-	target[keys[keys.length - 1]] = value;
-}
-
-function getNested(obj: Record<string, unknown>, path: string): unknown {
-	const keys = path.split(".");
-	let current: unknown = obj;
-	for (const key of keys) {
-		if (!current || typeof current !== "object") return undefined;
-		current = (current as Record<string, unknown>)[key];
-	}
-	return current;
-}
-
-function formatValue(value: unknown): string {
-	if (value === undefined) return "<not set>";
-	if (typeof value === "object") return JSON.stringify(value);
-	return String(value);
-}
-
-function parseValue(raw: string): unknown {
-	// JSON handles strings (quoted), numbers, booleans, null, arrays, objects.
-	try { return JSON.parse(raw); } catch { /* keep as string */ }
-	return raw;
-}
-
-// ---------------------------------------------------------------------------
-// Known config keys — mirrors config-schema.ts + config.ts.
-// When adding new config fields, add the dotted path here so team-settings
-// can discover and display them.
-// ---------------------------------------------------------------------------
-
-const KNOWN_KEYS = new Set([
-	// top-level
-	"asyncByDefault",
-	"executeWorkers",
-	"notifierIntervalMs",
-	"requireCleanWorktreeLeader",
-	// runtime
-	"runtime.mode",
-	"runtime.preferLiveSession",
-	"runtime.allowChildProcessFallback",
-	"runtime.maxTurns",
-	"runtime.graceTurns",
-	"runtime.inheritContext",
-	"runtime.promptMode",
-	"runtime.groupJoin",
-	"runtime.groupJoinAckTimeoutMs",
-	"runtime.requirePlanApproval",
-	"runtime.completionMutationGuard",
-	// limits
-	"limits.maxConcurrentWorkers",
-	"limits.allowUnboundedConcurrency",
-	"limits.maxTaskDepth",
-	"limits.maxChildrenPerTask",
-	"limits.maxRunMinutes",
-	"limits.maxRetriesPerTask",
-	"limits.maxTasksPerRun",
-	"limits.heartbeatStaleMs",
-	// control
-	"control.enabled",
-	"control.needsAttentionAfterMs",
-	// autonomous
-	"autonomous.profile",
-	"autonomous.enabled",
-	"autonomous.injectPolicy",
-	"autonomous.preferAsyncForLongTasks",
-	"autonomous.allowWorktreeSuggestion",
-	// tools
-	"tools.enableClaudeStyleAliases",
-	"tools.enableSteer",
-	"tools.terminateOnForeground",
-	// agents
-	"agents.disableBuiltins",
-	// observability
-	"observability.prometheus.enabled",
-	"observability.otlp.enabled",
-	// worktree
-	"worktree.enabled",
-]);
-
-const KNOWN_SORTED = [...KNOWN_KEYS].sort();
-
-// ---------------------------------------------------------------------------
-// Detail objects – all require { action, status } from TeamToolDetails.
-// Extras (count, key, value, path) are passed as never to bypass the narrow
-// TeamToolDetails interface (consistent with the rest of the codebase).
-// ---------------------------------------------------------------------------
-
-const OK = { action: "settings", status: "ok" as const };
-const ERR = { action: "settings", status: "error" as const };
-
-// ---------------------------------------------------------------------------
-// Main handler
-// ---------------------------------------------------------------------------
-
-export function handleSettings(params: { config?: Record<string, unknown> }, ctx: TeamContext): PiTeamsToolResult {
-	const cfg = (params.config ?? {}) as Record<string, unknown>;
-	const args = typeof cfg.args === "string" ? cfg.args.trim() : "";
-	const scope = cfg.scope === "project" ? "project" : "user";
-	const loaded = loadConfig(ctx.cwd);
-	const effective = loaded.config as Record<string, unknown>;
-
-	// team-settings list
-	if (!args || args === "list") {
-		const lines = ["pi-crew settings:", `Path: ${loaded.path}`, ""];
-		for (const key of KNOWN_SORTED) {
-			const value = getNested(effective, key);
-			lines.push(`  ${key} = ${formatValue(value)}`);
-		}
-		lines.push("", "Usage: team-settings [list|get <key>|set <key> <value>|unset <key>|path|scope]");
-		return result(lines.join("\n"), { ...OK, count: KNOWN_KEYS.size } as never);
-	}
-
-	// team-settings path
-	if (args === "path") {
-		return result(`pi-crew config path: ${loaded.path}`, { ...OK, path: loaded.path } as never);
-	}
-
-	// team-settings scope
-	if (args === "scope") {
-		return result(`Current scope: ${scope}\nConfig at: ${loaded.path}`, { ...OK, scope } as never);
-	}
-
-	// team-settings get <key>
-	if (args.startsWith("get ")) {
-		const key = args.slice(4).trim();
-		if (!key) return result("Usage: team-settings get <key>", { ...ERR }, true);
-		const value = getNested(effective, key);
-		const note = KNOWN_KEYS.has(key) ? "" : " (unknown key — may not take effect)";
-		return result(`${key} = ${formatValue(value)}${note}`, { ...OK, key, value } as never);
-	}
-
-	// team-settings unset <key>
-	if (args.startsWith("unset ")) {
-		const key = args.slice(6).trim();
-		if (!key) return result("Usage: team-settings unset <key>", { ...ERR }, true);
-		try {
-			const saved = updateConfig({}, { cwd: ctx.cwd, scope, unsetPaths: [key] });
-			return result(`Unset ${key}\nPath: ${saved.path}`, { ...OK, key } as never);
-		} catch (error) {
-			return result(error instanceof Error ? error.message : String(error), { ...ERR }, true);
-		}
-	}
-
-	// team-settings set <key> <value>
-	if (args.startsWith("set ")) {
-		const rest = args.slice(4).trim();
-		const spaceIdx = rest.indexOf(" ");
-		if (spaceIdx === -1) return result("Usage: team-settings set <key> <value>", { ...ERR }, true);
-		const key = rest.slice(0, spaceIdx);
-		const rawValue = rest.slice(spaceIdx + 1).trim();
-		if (!key) return result("Usage: team-settings set <key> <value>", { ...ERR }, true);
-
-		const value = parseValue(rawValue);
-		const patch = {};
-		setNested(patch as Record<string, unknown>, key, value);
-
-		try {
-			const converted = configPatchFromConfig({ config: patch as Record<string, unknown> });
-			const saved = updateConfig(converted, { cwd: ctx.cwd, scope });
-			const warning = KNOWN_KEYS.has(key) ? "" : "\nWarning: unknown key — verify it exists in config schema.";
-			return result(`Set ${key} = ${formatValue(value)}\nPath: ${saved.path}${warning}`, { ...OK, key, value } as never);
-		} catch (error) {
-			return result(error instanceof Error ? error.message : String(error), { ...ERR }, true);
-		}
-	}
-
-	return result("Unknown subcommand. Usage: team-settings [list|get <key>|set <key> <value>|unset <key>|path|scope]", { ...ERR }, true);
-}
+import type { TeamContext } from "../team-tool/context.ts";
+import { loadConfig, updateConfig } from "../../config/config.ts";
+import { configPatchFromConfig } from "../team-tool/config-patch.ts";
+import { result } from "../team-tool/context.ts";
+import type { PiTeamsToolResult } from "../tool-result.ts";
+import { suggestConfigKey } from "../../config/suggestions.ts";
+
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+
+function setNested(obj: Record<string, unknown>, path: string, value: unknown): void {
+	const keys = path.split(".");
+	let target: Record<string, unknown> = obj;
+	for (let i = 0; i < keys.length - 1; i++) {
+		if (!target[keys[i]] || typeof target[keys[i]] !== "object") {
+			target[keys[i]] = {};
+		}
+		target = target[keys[i]] as Record<string, unknown>;
+	}
+	target[keys[keys.length - 1]] = value;
+}
+
+function getNested(obj: Record<string, unknown>, path: string): unknown {
+	const keys = path.split(".");
+	let current: unknown = obj;
+	for (const key of keys) {
+		if (!current || typeof current !== "object") return undefined;
+		current = (current as Record<string, unknown>)[key];
+	}
+	return current;
+}
+
+function formatValue(value: unknown): string {
+	if (value === undefined) return "<not set>";
+	if (typeof value === "object") return JSON.stringify(value);
+	return String(value);
+}
+
+function parseValue(raw: string): unknown {
+	// JSON handles strings (quoted), numbers, booleans, null, arrays, objects.
+	try { return JSON.parse(raw); } catch { /* keep as string */ }
+	return raw;
+}
+
+// ---------------------------------------------------------------------------
+// Known config keys — mirrors config-schema.ts + config.ts.
+// When adding new config fields, add the dotted path here so team-settings
+// can discover and display them.
+// ---------------------------------------------------------------------------
+
+const KNOWN_KEYS = new Set([
+	// top-level
+	"asyncByDefault",
+	"executeWorkers",
+	"notifierIntervalMs",
+	"requireCleanWorktreeLeader",
+	// runtime
+	"runtime.mode",
+	"runtime.preferLiveSession",
+	"runtime.allowChildProcessFallback",
+	"runtime.maxTurns",
+	"runtime.graceTurns",
+	"runtime.inheritContext",
+	"runtime.promptMode",
+	"runtime.groupJoin",
+	"runtime.groupJoinAckTimeoutMs",
+	"runtime.requirePlanApproval",
+	"runtime.completionMutationGuard",
+	// limits
+	"limits.maxConcurrentWorkers",
+	"limits.allowUnboundedConcurrency",
+	"limits.maxTaskDepth",
+	"limits.maxChildrenPerTask",
+	"limits.maxRunMinutes",
+	"limits.maxRetriesPerTask",
+	"limits.maxTasksPerRun",
+	"limits.heartbeatStaleMs",
+	// control
+	"control.enabled",
+	"control.needsAttentionAfterMs",
+	// autonomous
+	"autonomous.profile",
+	"autonomous.enabled",
+	"autonomous.injectPolicy",
+	"autonomous.preferAsyncForLongTasks",
+	"autonomous.allowWorktreeSuggestion",
+	// tools
+	"tools.enableClaudeStyleAliases",
+	"tools.enableSteer",
+	"tools.terminateOnForeground",
+	// agents
+	"agents.disableBuiltins",
+	// observability
+	"observability.prometheus.enabled",
+	"observability.otlp.enabled",
+	// worktree
+	"worktree.enabled",
+]);
+
+const KNOWN_SORTED = [...KNOWN_KEYS].sort();
+
+// ---------------------------------------------------------------------------
+// Detail objects – all require { action, status } from TeamToolDetails.
+// Extras (count, key, value, path) are passed as never to bypass the narrow
+// TeamToolDetails interface (consistent with the rest of the codebase).
+// ---------------------------------------------------------------------------
+
+const OK = { action: "settings", status: "ok" as const };
+const ERR = { action: "settings", status: "error" as const };
+
+// ---------------------------------------------------------------------------
+// Main handler
+// ---------------------------------------------------------------------------
+
+export function handleSettings(params: { config?: Record<string, unknown> }, ctx: TeamContext): PiTeamsToolResult {
+	const cfg = (params.config ?? {}) as Record<string, unknown>;
+	const args = typeof cfg.args === "string" ? cfg.args.trim() : "";
+	const scope = cfg.scope === "project" ? "project" : "user";
+	const loaded = loadConfig(ctx.cwd);
+	const effective = loaded.config as Record<string, unknown>;
+
+	// team-settings list
+	if (!args || args === "list") {
+		const lines = ["pi-crew settings:", `Path: ${loaded.path}`, ""];
+		for (const key of KNOWN_SORTED) {
+			const value = getNested(effective, key);
+			lines.push(`  ${key} = ${formatValue(value)}`);
+		}
+		lines.push("", "Usage: team-settings [list|get <key>|set <key> <value>|unset <key>|path|scope]");
+		return result(lines.join("\n"), { ...OK, count: KNOWN_KEYS.size } as never);
+	}
+
+	// team-settings path
+	if (args === "path") {
+		return result(`pi-crew config path: ${loaded.path}`, { ...OK, path: loaded.path } as never);
+	}
+
+	// team-settings scope
+	if (args === "scope") {
+		return result(`Current scope: ${scope}\nConfig at: ${loaded.path}`, { ...OK, scope } as never);
+	}
+
+	// team-settings get <key>
+	if (args.startsWith("get ")) {
+		const key = args.slice(4).trim();
+		if (!key) return result("Usage: team-settings get <key>", { ...ERR }, true);
+		const value = getNested(effective, key);
+		let note = KNOWN_KEYS.has(key) ? "" : " (unknown key — may not take effect)";
+		if (!KNOWN_KEYS.has(key)) {
+			const suggestion = suggestConfigKey(key, KNOWN_SORTED);
+			if (suggestion) note += ` (did you mean '${suggestion}'?)`;
+		}
+		return result(`${key} = ${formatValue(value)}${note}`, { ...OK, key, value } as never);
+	}
+
+	// team-settings unset <key>
+	if (args.startsWith("unset ")) {
+		const key = args.slice(6).trim();
+		if (!key) return result("Usage: team-settings unset <key>", { ...ERR }, true);
+		try {
+			const saved = updateConfig({}, { cwd: ctx.cwd, scope, unsetPaths: [key] });
+			return result(`Unset ${key}\nPath: ${saved.path}`, { ...OK, key } as never);
+		} catch (error) {
+			return result(error instanceof Error ? error.message : String(error), { ...ERR }, true);
+		}
+	}
+
+	// team-settings set <key> <value>
+	if (args.startsWith("set ")) {
+		const rest = args.slice(4).trim();
+		const spaceIdx = rest.indexOf(" ");
+		if (spaceIdx === -1) return result("Usage: team-settings set <key> <value>", { ...ERR }, true);
+		const key = rest.slice(0, spaceIdx);
+		const rawValue = rest.slice(spaceIdx + 1).trim();
+		if (!key) return result("Usage: team-settings set <key> <value>", { ...ERR }, true);
+
+		const value = parseValue(rawValue);
+		const patch = {};
+		setNested(patch as Record<string, unknown>, key, value);
+
+		try {
+			const converted = configPatchFromConfig({ config: patch as Record<string, unknown> });
+			const saved = updateConfig(converted, { cwd: ctx.cwd, scope });
+			const warning = KNOWN_KEYS.has(key) ? "" : "\nWarning: unknown key — verify it exists in config schema.";
+			const suggestion = KNOWN_KEYS.has(key) ? null : suggestConfigKey(key, KNOWN_SORTED);
+			const hint = suggestion ? `\nDid you mean '${suggestion}'?` : "";
+			return result(`Set ${key} = ${formatValue(value)}\nPath: ${saved.path}${warning}${hint}`, { ...OK, key, value } as never);
+		} catch (error) {
+			return result(error instanceof Error ? error.message : String(error), { ...ERR }, true);
+		}
+	}
+
+	return result("Unknown subcommand. Usage: team-settings [list|get <key>|set <key> <value>|unset <key>|path|scope]", { ...ERR }, true);
+}

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

@@ -1,41 +1,41 @@
-import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
-import { readEvents } from "../../state/event-log.ts";
-import { loadRunManifestById } from "../../state/state-store.ts";
-import { aggregateUsage, formatUsage } from "../../state/usage.ts";
-import type { PiTeamsToolResult } from "../tool-result.ts";
-import { result, type TeamContext } from "./context.ts";
-
-export function handleEvents(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
-	if (!params.runId) return result("Events requires runId.", { action: "events", status: "error" }, true);
-	const loaded = loadRunManifestById(ctx.cwd, params.runId);
-	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "events", status: "error" }, true);
-	const events = readEvents(loaded.manifest.eventsPath);
-	const lines = [`Events for ${loaded.manifest.runId}:`, ...(events.length ? events.map((event) => `${event.time} ${event.type}${event.taskId ? ` ${event.taskId}` : ""}${event.message ? `: ${event.message}` : ""}${event.data ? ` ${JSON.stringify(event.data)}` : ""}`) : ["(none)"])];
-	return result(lines.join("\n"), { action: "events", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
-}
-
-export function handleArtifacts(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
-	if (!params.runId) return result("Artifacts requires runId.", { action: "artifacts", status: "error" }, true);
-	const loaded = loadRunManifestById(ctx.cwd, params.runId);
-	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "artifacts", status: "error" }, true);
-	const lines = [`Artifacts for ${loaded.manifest.runId}:`, ...(loaded.manifest.artifacts.length ? loaded.manifest.artifacts.map((artifact) => `- ${artifact.kind}: ${artifact.path}${artifact.sizeBytes !== undefined ? ` (${artifact.sizeBytes} bytes)` : ""}${artifact.contentHash ? ` sha256=${artifact.contentHash.slice(0, 12)}` : ""}`) : ["- (none)"])];
-	return result(lines.join("\n"), { action: "artifacts", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
-}
-
-export function handleSummary(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
-	if (!params.runId) return result("Summary requires runId.", { action: "summary", status: "error" }, true);
-	const loaded = loadRunManifestById(ctx.cwd, params.runId);
-	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "summary", status: "error" }, true);
-	const usage = aggregateUsage(loaded.tasks);
-	const lines = [
-		`Summary for ${loaded.manifest.runId}`,
-		`Status: ${loaded.manifest.status}`,
-		`Team: ${loaded.manifest.team}`,
-		`Workflow: ${loaded.manifest.workflow ?? "(none)"}`,
-		`Goal: ${loaded.manifest.goal}`,
-		`Usage: ${formatUsage(usage)}`,
-		"Tasks:",
-		...loaded.tasks.map((task) => `- ${task.id}: ${task.status} (${task.role} -> ${task.agent})${task.error ? ` - ${task.error}` : ""}`),
-	];
-	return result(lines.join("\n"), { action: "summary", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
-}
+import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
+import { readEvents } from "../../state/event-log.ts";
+import { loadRunManifestById } from "../../state/state-store.ts";
+import { aggregateUsage, formatUsage } from "../../state/usage.ts";
+import type { PiTeamsToolResult } from "../tool-result.ts";
+import { result, type TeamContext } from "./context.ts";
+
+export function handleEvents(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
+	if (!params.runId) return result("Events requires runId.", { action: "events", status: "error" }, true);
+	const loaded = loadRunManifestById(ctx.cwd, params.runId);
+	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "events", status: "error" }, true);
+	const events = readEvents(loaded.manifest.eventsPath);
+	const lines = [`Events for ${loaded.manifest.runId}:`, ...(events.length ? events.map((event) => `${event.time} ${event.type}${event.taskId ? ` ${event.taskId}` : ""}${event.message ? `: ${event.message}` : ""}${event.data ? ` ${JSON.stringify(event.data)}` : ""}`) : ["(none)"])];
+	return result(lines.join("\n"), { action: "events", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
+}
+
+export function handleArtifacts(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
+	if (!params.runId) return result("Artifacts requires runId.", { action: "artifacts", status: "error" }, true);
+	const loaded = loadRunManifestById(ctx.cwd, params.runId);
+	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "artifacts", status: "error" }, true);
+	const lines = [`Artifacts for ${loaded.manifest.runId}:`, ...(loaded.manifest.artifacts.length ? loaded.manifest.artifacts.map((artifact) => `- ${artifact.kind}: ${artifact.path}${artifact.sizeBytes !== undefined ? ` (${artifact.sizeBytes} bytes)` : ""}${artifact.contentHash ? ` sha256=${artifact.contentHash.slice(0, 12)}` : ""}`) : ["- (none)"])];
+	return result(lines.join("\n"), { action: "artifacts", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
+}
+
+export function handleSummary(params: TeamToolParamsValue, ctx: TeamContext): PiTeamsToolResult {
+	if (!params.runId) return result("Summary requires runId.", { action: "summary", status: "error" }, true);
+	const loaded = loadRunManifestById(ctx.cwd, params.runId);
+	if (!loaded) return result(`Run '${params.runId}' not found.`, { action: "summary", status: "error" }, true);
+	const usage = aggregateUsage(loaded.tasks);
+	const lines = [
+		`Summary for ${loaded.manifest.runId}`,
+		`Status: ${loaded.manifest.status}`,
+		`Team: ${loaded.manifest.team}`,
+		`Workflow: ${loaded.manifest.workflow ?? "(none)"}`,
+		`Goal: ${loaded.manifest.goal}`,
+		`Usage: ${formatUsage(usage)}`,
+		"Tasks:",
+		...loaded.tasks.map((task) => `- ${task.id}: ${task.status} (${task.role} -> ${task.agent})${task.error ? ` - ${task.error}` : ""}`),
+	];
+	return result(lines.join("\n"), { action: "summary", status: "ok", runId: loaded.manifest.runId, artifactsRoot: loaded.manifest.artifactsRoot });
+}