pi-crew 0.9.55 → 0.9.57

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

@@ -4,14 +4,24 @@ import { buildAgentDashboard, readAgentOutput } from "../../runtime/agent-observ
 import { buildCapabilityInventory } from "../../runtime/capability-inventory.ts";
 import { agentOutputPath, readCrewAgentEventsCursor, readCrewAgentStatus, readCrewAgents } from "../../runtime/crew-agent-records.ts";
 import { readForegroundControlStatus, writeForegroundInterruptRequest } from "../../runtime/foreground-control.ts";
-import { terminateLiveAgentsForRun } from "../../runtime/live-agent-manager.ts";
+import { touchWorkerHeartbeat } from "../../runtime/heartbeat/worker-heartbeat.ts";
+import { appendLiveAgentControlRequest } from "../../runtime/live-session/live-agent-control.ts";
+import {
+	followUpLiveAgent,
+	getLiveAgent,
+	listActiveLiveAgents,
+	resumeLiveAgent,
+	steerLiveAgent,
+	stopLiveAgent,
+	terminateLiveAgentsForRun,
+} from "../../runtime/live-session/live-agent-manager.ts";
+import { liveControlRealtimeMessage, publishLiveControlRealtime } from "../../runtime/live-session/live-control-realtime.ts";
+import { probeLiveSessionRuntime } from "../../runtime/live-session/live-session-runtime.ts";
+import { resolveCrewRuntime } from "../../runtime/model/runtime-resolver.ts";
 import { currentCrewRole, permissionForRole } from "../../runtime/role-permission.ts";
-import { resolveCrewRuntime } from "../../runtime/runtime-resolver.ts";
-import { touchWorkerHeartbeat } from "../../runtime/worker-heartbeat.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { canTransitionTaskStatus, isTeamTaskStatus } from "../../state/contracts.ts";
-import { appendEvent, readEvents, readEventsCursor } from "../../state/event-log.ts";
-import { withRunLock, withRunLockSync } from "../../state/locks.ts";
+import { withRunLock, withRunLockSync } from "../../state/coordination/locks.ts";
 import {
 	acknowledgeMailboxMessage,
 	appendFollowUpMessage,
@@ -25,25 +35,16 @@ import {
 	readMailbox,
 	readMailboxMessage,
 	validateMailbox,
-} from "../../state/mailbox.ts";
-import { loadRunManifestById, saveRunManifestAsync, saveRunTasks, updateRunStatus } from "../../state/state-store.ts";
-import { claimTask, releaseTaskClaim, transitionClaimedTaskStatus } from "../../state/task-claims.ts";
-import { appendLiveAgentControlRequest } from "../../subagents/live/control.ts";
-import {
-	followUpLiveAgent,
-	getLiveAgent,
-	listActiveLiveAgents,
-	resumeLiveAgent,
-	steerLiveAgent,
-	stopLiveAgent,
-} from "../../subagents/live/manager.ts";
-import { liveControlRealtimeMessage, publishLiveControlRealtime } from "../../subagents/live/realtime.ts";
-import { probeLiveSessionRuntime } from "../../subagents/live/session-runtime.ts";
+} from "../../state/coordination/mailbox.ts";
+import { claimTask, releaseTaskClaim, transitionClaimedTaskStatus } from "../../state/coordination/task-claims.ts";
+import { appendEvent, readEvents, readEventsCursor } from "../../state/event-log/event-log.ts";
+import { loadRunManifestById, saveRunManifestAsync, saveRunTasks, updateRunStatus } from "../../state/stores/state-store.ts";
 import { logInternalError } from "../../utils/internal-error.ts";
 import { resolveRealContainedPath } from "../../utils/safe-paths.ts";
 import { locateRunCwd } from "../team-tool.ts";
 import type { PiTeamsToolResult } from "../tool-result.ts";
 import { configRecord, result, type TeamContext } from "./context.ts";
+import { paramRequired } from "./param-error.ts";
 import { RUN_NOT_FOUND_HINT } from "./run-not-found.ts";
 
 export function globMatch(value: string, pattern: string): boolean {
@@ -121,7 +122,8 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 			status: "ok",
 		});
 	}
-	if (!params.runId) return result("API requires runId.", { action: "api", status: "error" }, true);
+	if (!params.runId)
+		return result(paramRequired("api", "runId", "{ action: 'api', runId: 'team_...' }"), { action: "api", status: "error" }, true);
 	const runCwd = locateRunCwd(params.runId, ctx.cwd);
 	if (!runCwd) return result(`Run '${params.runId}' not found.${RUN_NOT_FOUND_HINT}`, { action: "api", status: "error" }, true);
 	const loaded = loadRunManifestById(runCwd, params.runId); // NOTE: no withRunLock - best-effort only; concurrent writes may cause inconsistency
@@ -302,7 +304,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task)
 			return result(
-				"API read-task requires config.taskId matching a task id or step id.",
+				paramRequired(
+					"API read-task",
+					"config.taskId matching a task id or step id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-task', taskId: '01_01-agent' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -368,7 +374,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agent = readCrewAgents(loaded.manifest).find((item) => item.id === agentId || item.taskId === agentId);
 		if (!agent)
 			return result(
-				"API get-agent-result requires config.agentId matching an agent id or task id.",
+				paramRequired(
+					"API get-agent-result",
+					"config.agentId matching an agent id or task id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'get-agent-result', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -391,7 +401,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const status = agent ? (readCrewAgentStatus(loaded.manifest, agent.taskId) ?? agent) : undefined;
 		if (!status)
 			return result(
-				"API read-agent-status requires config.agentId matching an agent id or task id.",
+				paramRequired(
+					"API read-agent-status",
+					"config.agentId matching an agent id or task id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-agent-status', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -412,7 +426,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agent = agentId ? agents.find((item) => item.id === agentId || item.taskId === agentId) : agents[0];
 		if (!agent)
 			return result(
-				"API read-agent-events requires config.agentId matching an agent id or task id, or at least one agent in the run.",
+				paramRequired(
+					"API read-agent-events",
+					"config.agentId matching an agent id or task id, or at least one agent in the run",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-agent-events', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -438,7 +456,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agent = agentId ? agents.find((item) => item.id === agentId || item.taskId === agentId) : agents[0];
 		if (!agent)
 			return result(
-				"API read-agent-transcript requires config.agentId matching an agent id or task id, or at least one agent in the run.",
+				paramRequired(
+					"API read-agent-transcript",
+					"config.agentId matching an agent id or task id, or at least one agent in the run",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-agent-transcript', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -467,7 +489,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agent = agentId ? agents.find((item) => item.id === agentId || item.taskId === agentId) : agents[0];
 		if (!agent)
 			return result(
-				"API read-agent-output requires config.agentId matching an agent id or task id, or at least one agent in the run.",
+				paramRequired(
+					"API read-agent-output",
+					"config.agentId matching an agent id or task id, or at least one agent in the run",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-agent-output', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -513,7 +539,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agent = readCrewAgents(loaded.manifest).find((item) => item.id === agentId || item.taskId === agentId);
 		if (!agent)
 			return result(
-				"API nudge-agent requires config.agentId matching an agent id or task id.",
+				paramRequired(
+					"API nudge-agent",
+					"config.agentId matching an agent id or task id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'nudge-agent', agentId: 'agent-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -580,7 +610,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const agentId = typeof cfg.agentId === "string" ? cfg.agentId : undefined;
 		if (!agentId)
 			return result(
-				`API ${operation} requires config.agentId.`,
+				paramRequired(
+					`API ${operation}`,
+					"config.agentId",
+					`{ action: 'api', runId: 'team_...', config: { operation: '${operation}', agentId: 'agent-1' } }`,
+				),
 				{
 					action: "api",
 					status: "error",
@@ -637,7 +671,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 			if (operation === "follow-up-agent") {
 				if (!prompt)
 					return result(
-						"API follow-up-agent requires config.prompt or config.message.",
+						paramRequired(
+							"API follow-up-agent",
+							"config.prompt or config.message",
+							"{ action: 'api', runId: 'team_...', config: { operation: 'follow-up-agent', agentId: 'agent-1', prompt: '<next step>' } }",
+						),
 						{
 							action: "api",
 							status: "error",
@@ -662,7 +700,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 			if (operation === "resume-agent") {
 				if (!prompt)
 					return result(
-						"API resume-agent requires config.prompt or config.message.",
+						paramRequired(
+							"API resume-agent",
+							"config.prompt or config.message",
+							"{ action: 'api', runId: 'team_...', config: { operation: 'resume-agent', agentId: 'agent-1', prompt: '<resume instruction>' } }",
+						),
 						{
 							action: "api",
 							status: "error",
@@ -710,7 +752,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 				);
 			if (operation === "resume-agent" && !prompt)
 				return result(
-					"API resume-agent requires config.prompt or config.message.",
+					paramRequired(
+						"API resume-agent",
+						"config.prompt or config.message",
+						"{ action: 'api', runId: 'team_...', config: { operation: 'resume-agent', agentId: 'agent-1', prompt: '<resume instruction>' } }",
+					),
 					{
 						action: "api",
 						status: "error",
@@ -720,7 +766,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 				);
 			if (operation === "follow-up-agent" && !prompt)
 				return result(
-					"API follow-up-agent requires config.prompt or config.message.",
+					paramRequired(
+						"API follow-up-agent",
+						"config.prompt or config.message",
+						"{ action: 'api', runId: 'team_...', config: { operation: 'follow-up-agent', agentId: 'agent-1', prompt: '<next step>' } }",
+					),
 					{
 						action: "api",
 						status: "error",
@@ -867,7 +917,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const taskId = typeof cfg.taskId === "string" && cfg.taskId.trim() ? cfg.taskId.trim() : undefined;
 		if (!body)
 			return result(
-				"API send-message requires config.body.",
+				paramRequired(
+					"API send-message",
+					"config.body",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'send-message', body: '<message>' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -932,7 +986,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const messageId = typeof cfg.messageId === "string" ? cfg.messageId : undefined;
 		if (!messageId)
 			return result(
-				"API ack-message requires config.messageId.",
+				paramRequired(
+					"API ack-message",
+					"config.messageId",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'ack-message', messageId: 'msg-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -995,7 +1053,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task)
 			return result(
-				"API read-heartbeat requires config.taskId matching a task id or step id.",
+				paramRequired(
+					"API read-heartbeat",
+					"config.taskId matching a task id or step id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'read-heartbeat', taskId: '01_01-agent' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -1016,7 +1078,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task)
 			return result(
-				"API claim-task requires config.taskId matching a task id or step id.",
+				paramRequired(
+					"API claim-task",
+					"config.taskId matching a task id or step id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'claim-task', taskId: '01_01-agent' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -1066,7 +1132,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task || !owner || !token)
 			return result(
-				"API release-task-claim requires config.taskId, config.owner, and config.token.",
+				paramRequired(
+					"API release-task-claim",
+					"config.taskId, config.owner, and config.token",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'release-task-claim', taskId: '01_01-agent', owner: 'worker-1', token: 'tok-1' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -1113,7 +1183,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task || !owner || !token || !isTeamTaskStatus(to))
 			return result(
-				"API transition-task-status requires config.taskId, config.owner, config.token, and valid config.status.",
+				paramRequired(
+					"API transition-task-status",
+					"config.taskId, config.owner, config.token, and valid config.status",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'transition-task-status', taskId: '01_01-agent', owner: 'worker-1', token: 'tok-1', status: 'done' } }",
+				),
 				{
 					action: "api",
 					status: "error",
@@ -1167,7 +1241,11 @@ export async function handleApi(params: TeamToolParamsValue, ctx: TeamContext):
 		const task = loaded.tasks.find((item) => item.id === taskId || item.stepId === taskId);
 		if (!task)
 			return result(
-				"API write-heartbeat requires config.taskId matching a task id or step id.",
+				paramRequired(
+					"API write-heartbeat",
+					"config.taskId matching a task id or step id",
+					"{ action: 'api', runId: 'team_...', config: { operation: 'write-heartbeat', taskId: '01_01-agent' } }",
+				),
 				{
 					action: "api",
 					status: "error",

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

@@ -1,19 +1,24 @@
 import { appendHookEvent, executeHook } from "../../hooks/registry.ts";
-import { buildSyntheticTerminalEvidence, type CancellationReason, cancellationReasonFromUnknown } from "../../runtime/cancellation.ts";
-import { killProcessPid } from "../../runtime/child-pi.ts";
+import { killProcessPid } from "../../runtime/child-pi/child-pi.ts";
 import { recordFromTask, saveCrewAgents } from "../../runtime/crew-agent-records.ts";
 import { writeForegroundInterruptRequest } from "../../runtime/foreground-control.ts";
-import { terminateLiveAgentsForRun } from "../../runtime/live-agent-manager.ts";
+import { terminateLiveAgentsForRun } from "../../runtime/live-session/live-agent-manager.ts";
+import {
+	buildSyntheticTerminalEvidence,
+	type CancellationReason,
+	cancellationReasonFromUnknown,
+} from "../../runtime/process/cancellation.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
-import { appendEvent } from "../../state/event-log.ts";
-import { withRunLockSync } from "../../state/locks.ts";
-import { loadRunManifestById, saveRunTasks, updateRunStatus } from "../../state/state-store.ts";
+import { withRunLockSync } from "../../state/coordination/locks.ts";
+import { appendEvent } from "../../state/event-log/event-log.ts";
+import { loadRunManifestById, saveRunTasks, updateRunStatus } from "../../state/stores/state-store.ts";
 import { logInternalError } from "../../utils/internal-error.ts";
 import { locateRunCwd } from "../team-tool.ts";
 import type { PiTeamsToolResult } from "../tool-result.ts";
 import { type CacheControlDeps, invalidateSnapshot } from "./cache-control.ts";
 import { result, type TeamContext } from "./context.ts";
 import { enforceDestructiveIntent, intentFromConfig } from "./intent-policy.ts";
+import { paramRequired } from "./param-error.ts";
 import { RUN_NOT_FOUND_HINT } from "./run-not-found.ts";
 
 export interface AbortOwnedResult {
@@ -84,7 +89,12 @@ function cancelReasonFromParams(params: TeamToolParamsValue): CancellationReason
 }
 
 export async function handleRetry(params: TeamToolParamsValue, ctx: TeamContext, deps?: CacheControlDeps): Promise<PiTeamsToolResult> {
-	if (!params.runId) return result("Retry requires runId.", { action: "retry", status: "error" }, true);
+	if (!params.runId)
+		return result(
+			paramRequired("retry", "runId", "{ action: 'retry', runId: 'team_...' }"),
+			{ action: "retry", status: "error" },
+			true,
+		);
 	const runCwd = locateRunCwd(params.runId, ctx.cwd);
 	if (!runCwd) return result(`Run '${params.runId}' not found.${RUN_NOT_FOUND_HINT}`, { action: "retry", status: "error" }, true);
 	const loaded = loadRunManifestById(runCwd, params.runId); // NOTE: no withRunLock - best-effort only; concurrent writes may cause inconsistency
@@ -176,7 +186,12 @@ export async function handleRetry(params: TeamToolParamsValue, ctx: TeamContext,
 export async function handleCancel(params: TeamToolParamsValue, ctx: TeamContext, deps?: CacheControlDeps): Promise<PiTeamsToolResult> {
 	const intentError = enforceDestructiveIntent("cancel", params, ctx.config);
 	if (intentError) return intentError;
-	if (!params.runId) return result("Cancel requires runId.", { action: "cancel", status: "error" }, true);
+	if (!params.runId)
+		return result(
+			paramRequired("cancel", "runId", "{ action: 'cancel', runId: 'team_...' }"),
+			{ action: "cancel", status: "error" },
+			true,
+		);
 	const runCwd = locateRunCwd(params.runId, ctx.cwd);
 	if (!runCwd) return result(`Run '${params.runId}' not found.${RUN_NOT_FOUND_HINT}`, { action: "cancel", status: "error" }, true);
 	const loaded = loadRunManifestById(runCwd, params.runId); // NOTE: no withRunLock - best-effort only; concurrent writes may cause inconsistency

package/src/extension/team-tool/chain-executor.ts

@@ -31,7 +31,7 @@ import type { Decision, TaskPacket, TaskResult } from "../../runtime/handoff-man
 import { readIfSmallWithTee } from "../../runtime/task-output-context.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { atomicWriteFile } from "../../state/atomic-write.ts";
-import { createRunPaths, loadRunManifestById } from "../../state/state-store.ts";
+import { createRunPaths, loadRunManifestById } from "../../state/stores/state-store.ts";
 import type { TeamRunManifest, TeamTaskState } from "../../state/types.ts";
 import type { PiTeamsToolResult } from "../tool-result.ts";
 import { textFromToolResult } from "../tool-result.ts";

package/src/extension/team-tool/dispatch/automate.ts

@@ -22,8 +22,20 @@ import {
 import { result, type TeamContext } from "../context.ts";
 import { handleListScheduled, handleSchedule } from "../handle-schedule.ts";
 
+/**
+ * Actions owned by the Automate domain. Single source of truth for the switch
+ * below AND for the runtime exhaustiveness test
+ * (test/unit/dispatch-exhaustive.test.ts). The compile-time `never` sentinel in
+ * the `default` branch errors if an AutomateDomainAction is added here without
+ * a matching `case`.
+ */
+export const AUTOMATE_DOMAIN_ACTIONS = ["schedule", "scheduled", "anchor", "auto-summarize", "auto_boomerang", "api"] as const;
+type AutomateDomainAction = (typeof AUTOMATE_DOMAIN_ACTIONS)[number];
+
 export async function handleAutomateDomain(params: TeamToolParamsValue, ctx: TeamContext): Promise<PiTeamsToolResult> {
-	switch (params.action) {
+	// `domainForAction` routes only Automate-domain actions here, so narrowing is sound.
+	const action = params.action as AutomateDomainAction;
+	switch (action) {
 		case "api":
 			return handleApi(params, ctx);
 		case "schedule":
@@ -70,11 +82,14 @@ export async function handleAutomateDomain(params: TeamToolParamsValue, ctx: Tea
 					return handleAutoSummarizeStatus(params, ctx);
 			}
 		}
-		default:
+		default: {
+			// Compile-time exhaustiveness: errors if an AutomateDomainAction lacks a case above.
+			const _exhaustive: never = action;
 			return result(
 				`Unhandled automate-domain action: ${params.action}${formatActionSuggestion(String(params.action ?? ""))}`,
 				{ action: "unknown", status: "error" },
 				true,
 			);
+		}
 	}
 }

package/src/extension/team-tool/dispatch/control.ts

@@ -13,8 +13,20 @@ import { handleDoctor } from "../doctor.ts";
 import { handleCleanup, handleForget, handlePrune } from "../lifecycle-actions.ts";
 import { handleRespond } from "../respond.ts";
 
+/**
+ * Actions owned by the Control domain. Single source of truth for the switch
+ * below AND for the runtime exhaustiveness test
+ * (test/unit/dispatch-exhaustive.test.ts). The compile-time `never` sentinel in
+ * the `default` branch errors if a ControlDomainAction is added here without a
+ * matching `case`.
+ */
+export const CONTROL_DOMAIN_ACTIONS = ["cancel", "invalidate", "respond", "cleanup", "prune", "forget", "doctor"] as const;
+type ControlDomainAction = (typeof CONTROL_DOMAIN_ACTIONS)[number];
+
 export async function handleControlDomain(params: TeamToolParamsValue, ctx: TeamContext): Promise<PiTeamsToolResult> {
-	switch (params.action) {
+	// `domainForAction` routes only Control-domain actions here, so narrowing is sound.
+	const action = params.action as ControlDomainAction;
+	switch (action) {
 		case "doctor":
 			return handleDoctor(ctx, params);
 		case "cleanup":
@@ -29,7 +41,9 @@ export async function handleControlDomain(params: TeamToolParamsValue, ctx: Team
 			return handleInvalidate(params, ctx);
 		case "respond":
 			return handleRespond(params, ctx);
-		default:
+		default: {
+			// Compile-time exhaustiveness: errors if a ControlDomainAction lacks a case above.
+			const _exhaustive: never = action;
 			return result(
 				`Unhandled control-domain action: ${params.action}`,
 				{
@@ -38,5 +52,6 @@ export async function handleControlDomain(params: TeamToolParamsValue, ctx: Team
 				},
 				true,
 			);
+		}
 	}
 }

package/src/extension/team-tool/dispatch/manage.ts

@@ -27,8 +27,37 @@ import {
 	handleWorkflowSave,
 } from "../workflow-manage.ts";
 
+/**
+ * Actions owned by the Manage domain. Single source of truth for the switch
+ * below AND for the runtime exhaustiveness test
+ * (test/unit/dispatch-exhaustive.test.ts). The compile-time `never` sentinel in
+ * the `default` branch errors if a ManageDomainAction is added here without a
+ * matching `case`.
+ */
+export const MANAGE_DOMAIN_ACTIONS = [
+	"create",
+	"update",
+	"delete",
+	"init",
+	"config",
+	"validate",
+	"autonomy",
+	"settings",
+	"workflow-create",
+	"workflow-get",
+	"workflow-list",
+	"workflow-save",
+	"workflow-delete",
+	"import",
+	"imports",
+	"export",
+] as const;
+type ManageDomainAction = (typeof MANAGE_DOMAIN_ACTIONS)[number];
+
 export async function handleManageDomain(params: TeamToolParamsValue, ctx: TeamContext): Promise<PiTeamsToolResult> {
-	switch (params.action) {
+	// `domainForAction` routes only Manage-domain actions here, so narrowing is sound.
+	const action = params.action as ManageDomainAction;
+	switch (action) {
 		case "init": {
 			const cfg = configRecord(params.config);
 			const ignoreMethod =
@@ -151,11 +180,14 @@ export async function handleManageDomain(params: TeamToolParamsValue, ctx: TeamC
 			return handleWorkflowSave(params, ctx);
 		case "workflow-delete":
 			return handleWorkflowDelete(params, ctx);
-		default:
+		default: {
+			// Compile-time exhaustiveness: errors if a ManageDomainAction lacks a case above.
+			const _exhaustive: never = action;
 			return result(
 				`Unhandled manage-domain action: ${params.action}${formatActionSuggestion(String(params.action ?? ""))}`,
 				{ action: "unknown", status: "error" },
 				true,
 			);
+		}
 	}
 }

package/src/extension/team-tool/dispatch/run.ts

@@ -20,8 +20,20 @@ import { handleOrchestrate } from "../orchestrate.ts";
 import { handleParallel } from "../parallel-dispatch.ts";
 import { handlePlan } from "../plan.ts";
 
+/**
+ * Actions owned by the Run domain. Single source of truth for the switch
+ * below AND for the runtime exhaustiveness test
+ * (test/unit/dispatch-exhaustive.test.ts). The compile-time `never` sentinel in
+ * the `default` branch errors if a RunDomainAction is added here without a
+ * matching `case`.
+ */
+export const RUN_DOMAIN_ACTIONS = ["run", "parallel", "plan", "orchestrate", "resume", "retry", "wait", "steer", "goal"] as const;
+type RunDomainAction = (typeof RUN_DOMAIN_ACTIONS)[number];
+
 export async function handleRunDomain(params: TeamToolParamsValue, ctx: TeamContext): Promise<PiTeamsToolResult> {
-	switch (params.action) {
+	// `domainForAction` routes only Run-domain actions here, so narrowing is sound.
+	const action = params.action as RunDomainAction;
+	switch (action) {
 		case "run":
 			return handleRun(params, ctx);
 		case "parallel":
@@ -40,7 +52,9 @@ export async function handleRunDomain(params: TeamToolParamsValue, ctx: TeamCont
 			return handleSteer(params, ctx);
 		case "goal":
 			return handleGoal(params, ctx);
-		default:
+		default: {
+			// Compile-time exhaustiveness: errors if a RunDomainAction lacks a case above.
+			const _exhaustive: never = action;
 			return result(
 				`Unhandled run-domain action: ${params.action}`,
 				{
@@ -49,5 +63,6 @@ export async function handleRunDomain(params: TeamToolParamsValue, ctx: TeamCont
 				},
 				true,
 			);
+		}
 	}
 }

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

@@ -10,11 +10,11 @@
 import * as path from "node:path";
 import { allAgents, discoverAgents } from "../../../agents/discover-agents.ts";
 import { loadConfig } from "../../../config/config.ts";
-import { FileCheckpointStore } from "../../../runtime/checkpoint.ts";
+import { FileCheckpointStore } from "../../../runtime/recovery/checkpoint.ts";
 import { getSkillCacheStats } from "../../../runtime/skill-instructions.ts";
 import type { TeamToolParamsValue } from "../../../schema/team-tool-schema.ts";
-import { computeRunCacheKey, getCachedRun, getCacheStats } from "../../../state/run-cache.ts";
-import { listRunGraphs, loadRunGraph } from "../../../state/run-graph.ts";
+import { computeRunCacheKey, getCachedRun, getCacheStats } from "../../../state/stores/run-cache.ts";
+import { listRunGraphs, loadRunGraph } from "../../../state/stores/run-graph.ts";
 import { allTeams, discoverTeams } from "../../../teams/discover-teams.ts";
 import { searchAgents, searchTeams } from "../../../utils/bm25-search.ts";
 import { projectCrewRoot } from "../../../utils/paths.ts";
@@ -32,8 +32,37 @@ import { handleArtifacts, handleEvents, handleSummary } from "../inspect.ts";
 import { handleWorktrees } from "../lifecycle-actions.ts";
 import { handleStatus } from "../status.ts";
 
+/**
+ * Actions owned by the Status domain. Single source of truth for the switch
+ * below AND for the runtime exhaustiveness test
+ * (test/unit/dispatch-exhaustive.test.ts). The compile-time `never` sentinel in
+ * the `default` branch errors if a StatusDomainAction is added here without a
+ * matching `case`.
+ */
+export const STATUS_DOMAIN_ACTIONS = [
+	"status",
+	"list",
+	"get",
+	"events",
+	"artifacts",
+	"summary",
+	"graph",
+	"search",
+	"health",
+	"worktrees",
+	"checkpoint",
+	"cache",
+	"explain",
+	"onboard",
+	"recommend",
+	"help",
+] as const;
+type StatusDomainAction = (typeof STATUS_DOMAIN_ACTIONS)[number];
+
 export async function handleStatusDomain(params: TeamToolParamsValue, ctx: TeamContext): Promise<PiTeamsToolResult> {
-	switch (params.action) {
+	// `domainForAction` routes only Status-domain actions here, so narrowing is sound.
+	const action = params.action as StatusDomainAction;
+	switch (action) {
 		case "list":
 			return handleList(params, ctx);
 		case "get":
@@ -177,11 +206,14 @@ export async function handleStatusDomain(params: TeamToolParamsValue, ctx: TeamC
 				{ action: "checkpoint", status: "ok", data: { checkpoint } },
 			);
 		}
-		default:
+		default: {
+			// Compile-time exhaustiveness: errors if a StatusDomainAction lacks a case above.
+			const _exhaustive: never = action;
 			return result(
 				`Unhandled status-domain action: ${params.action}${formatActionSuggestion(String(params.action ?? ""))}`,
 				{ action: "unknown", status: "error" },
 				true,
 			);
+		}
 	}
 }

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

@@ -5,9 +5,9 @@ import { allAgents, discoverAgents } from "../../agents/discover-agents.ts";
 import { loadConfig } from "../../config/config.ts";
 import { DEFAULT_PATHS } from "../../config/defaults.ts";
 import { type DriftReport, detectDrift, formatDriftReport } from "../../config/drift-detector.ts";
+import { getRuntimeWarmupStatus } from "../../runtime/model/runtime-warmup.ts";
 import { getPiSpawnCommand } from "../../runtime/pi-spawn.ts";
-import { getRuntimeWarmupStatus } from "../../runtime/runtime-warmup.ts";
-import { formatZombieReport, scanZombieSubagents } from "../../runtime/zombie-scanner.ts";
+import { formatZombieReport, scanZombieSubagents } from "../../runtime/process/zombie-scanner.ts";
 import type { TeamToolParamsValue } from "../../schema/team-tool-schema.ts";
 import { TeamToolParams } from "../../schema/team-tool-schema.ts";
 import { atomicWriteFile } from "../../state/atomic-write.ts";

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

@@ -1,6 +1,6 @@
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { loadRunManifestById } from "../../state/state-store.ts";
+import { loadRunManifestById } from "../../state/stores/state-store.ts";
 import type { TeamRunManifest, TeamTaskState } from "../../state/types.ts";
 import { RUN_NOT_FOUND_HINT } from "./run-not-found.ts";