taskplane 0.29.2 → 0.30.1

package/extensions/taskplane/types.ts

@@ -24,6 +24,17 @@ export interface OrchestratorConfig {
 		operator_id: string;
 		/** How completed batches are integrated. manual = user runs /orch-integrate. supervised = supervisor proposes plan, asks confirmation. auto = supervisor executes without asking. */
 		integration: "manual" | "supervised" | "auto";
+		/**
+		 * Optional pre-resolved batch ID injected by callers that already
+		 * know the batch identity (e.g., resumed orchestrations). When
+		 * absent, callers fall back to the `ORCH_BATCH_ID` env var or a
+		 * timestamp. Read by `executeLaneV2` (execution.ts).
+		 *
+		 * @since TP-195 (#TBD) — documented field that was already being
+		 * read at runtime via `config.orchestrator?.batchId` and asserted
+		 * by the source-grep invariant in `runtime-model-fallback.test.ts`.
+		 */
+		batchId?: string;
 	};
 	dependencies: {
 		source: "prompt" | "agent";
@@ -158,13 +169,40 @@ export function parseSegmentIdRepo(segment: { repoId: string }): string {
 /** Build a dynamic segment expansion request ID (`exp-{timestamp}-{random5}`). */
 export function buildExpansionRequestId(timestamp = Date.now()): string {
 	const ts = Number.isFinite(timestamp) ? Math.floor(timestamp) : Date.now();
-	const base = Math.random().toString(36).slice(2).toLowerCase().replace(/[^a-z0-9]/g, "");
+	const base = Math.random()
+		.toString(36)
+		.slice(2)
+		.toLowerCase()
+		.replace(/[^a-z0-9]/g, "");
 	const random5 = (base + "00000").slice(0, 5);
 	return `exp-${ts}-${random5}`;
 }
 
 // ── Step-Segment Mapping (Phase A: segment-scoped worker visibility) ────
 
+/**
+ * Authoritative segment-scope mode for a single worker iteration.
+ *
+ * - `FULL_TASK`: the worker sees the entire PROMPT.md, all steps, all checkboxes.
+ *   No `Active segment ID` / `Your checkboxes for this step` prose is injected.
+ *   Segment-related environment variables (`TASKPLANE_ACTIVE_SEGMENT_ID`,
+ *   `TASKPLANE_SEGMENT_ID`) are hard-cleared so that runtime tools keyed on
+ *   them (e.g., `request_segment_expansion`) cannot accidentally register.
+ *
+ * - `SEGMENT_SCOPED`: the worker is iterating a specific segment of a
+ *   multi-segment task. Only that segment's steps and checkboxes are shown;
+ *   `Active segment ID` is announced; segment-related env vars carry the
+ *   active `segmentId`; the segment-overlay system prompt is appended.
+ *
+ * This is the single authoritative flag for the segment-scope decision
+ * (TP-196 / #502). Call sites should derive their behaviour from this mode
+ * rather than re-evaluating the underlying boolean conditions, which prevents
+ * the multiple branches drifting out of sync.
+ *
+ * @since TP-196
+ */
+export type SegmentScopeMode = "FULL_TASK" | "SEGMENT_SCOPED";
+
 /** A group of checkboxes scoped to a single repo within a step. */
 export interface SegmentCheckboxGroup {
 	repoId: string;
@@ -365,7 +403,6 @@ export interface PreflightCheck {
 	hint?: string;
 }
 
-
 // ── Defaults ─────────────────────────────────────────────────────────
 
 export const DEFAULT_ORCHESTRATOR_CONFIG: OrchestratorConfig = {
@@ -428,7 +465,6 @@ export const DEFAULT_TASK_RUNNER_CONFIG: TaskRunnerConfig = {
 	model_fallback: "inherit",
 };
 
-
 // ── Helpers ──────────────────────────────────────────────────────────
 
 export function freshBatchState(): BatchState {
@@ -598,7 +634,12 @@ export interface RemoveAllWorktreesResult {
 	/** All per-worktree outcomes in order */
 	outcomes: RemoveWorktreeOutcome[];
 	/** Branches preserved (had unmerged commits) */
-	preserved: Array<{ branch: string; savedBranch: string; laneNumber: number; unmergedCount?: number }>;
+	preserved: Array<{
+		branch: string;
+		savedBranch: string;
+		laneNumber: number;
+		unmergedCount?: number;
+	}>;
 }
 
 // ── Discovery Types ──────────────────────────────────────────────────
@@ -656,7 +697,6 @@ export interface DiscoveryResult {
 	errors: DiscoveryError[];
 }
 
-
 // ── Wave Computation Types ───────────────────────────────────────────
 
 /** Dependency graph: adjacency list (task → tasks it depends on) */
@@ -683,7 +723,6 @@ export interface WaveComputationResult {
 	segmentPlans?: TaskSegmentPlanMap;
 }
 
-
 // ── Lane Allocation (Phase 3) ────────────────────────────────────────
 
 /**
@@ -760,7 +799,6 @@ export interface AllocatedLane {
 	repoId?: string;
 }
 
-
 // ── Execution Types & Contracts ──────────────────────────────────────
 
 /**
@@ -921,7 +959,8 @@ export type ExecutionErrorCode =
 	| "EXEC_TASK_STAGE_FAILED"
 	| "EXEC_TASK_COMMIT_FAILED"
 	| "EXEC_TMUX_NOT_AVAILABLE"
-	| "EXEC_WORKTREE_MISSING";
+	| "EXEC_WORKTREE_MISSING"
+	| "EXEC_MISSING_TASK_FOLDER";
 
 /** Typed error for lane execution failures. */
 export class ExecutionError extends Error {
@@ -938,7 +977,6 @@ export class ExecutionError extends Error {
 	}
 }
 
-
 // ── Monitoring Types & Contracts ─────────────────────────────────────
 
 /**
@@ -1050,7 +1088,6 @@ export interface MtimeTracker {
 	stallTimerStart: number | null;
 }
 
-
 // ── Wave Execution Types & Contracts ─────────────────────────────────
 
 /**
@@ -1122,7 +1159,6 @@ export interface WaveExecutionResult {
 	} | null;
 }
 
-
 // ── Orchestrator Runtime State ───────────────────────────────────────
 
 /**
@@ -1135,7 +1171,16 @@ export interface WaveExecutionResult {
  *                   → paused (via /orch-pause)
  *   Any active state → idle (via cleanup after completion/failure)
  */
-export type OrchBatchPhase = "idle" | "launching" | "planning" | "executing" | "merging" | "paused" | "stopped" | "completed" | "failed";
+export type OrchBatchPhase =
+	| "idle"
+	| "launching"
+	| "planning"
+	| "executing"
+	| "merging"
+	| "paused"
+	| "stopped"
+	| "completed"
+	| "failed";
 
 /**
  * Runtime state for a batch execution.
@@ -1288,14 +1333,17 @@ export function freshOrchBatchState(): OrchBatchRuntimeState {
 	};
 }
 
-
 // ── Merge Types ──────────────────────────────────────────────────────
 
 /**
  * Valid merge result statuses.
  * Matches the contract in .pi/agents/task-merger.md.
  */
-export type MergeResultStatus = "SUCCESS" | "CONFLICT_RESOLVED" | "CONFLICT_UNRESOLVED" | "BUILD_FAILURE";
+export type MergeResultStatus =
+	| "SUCCESS"
+	| "CONFLICT_RESOLVED"
+	| "CONFLICT_UNRESOLVED"
+	| "BUILD_FAILURE";
 
 /** All valid status strings for runtime validation. */
 export const VALID_MERGE_STATUSES: ReadonlySet<string> = new Set([
@@ -1686,7 +1734,6 @@ export interface MergeSessionHealthState {
 	deadEmitted: boolean;
 }
 
-
 // ── Merge Retry Policy Matrix (TP-033 Step 2) ───────────────────────
 
 /**
@@ -1743,7 +1790,9 @@ export interface MergeRetryPolicy {
  *
  * @since TP-033
  */
-export const MERGE_RETRY_POLICY_MATRIX: Readonly<Record<MergeFailureClassification, MergeRetryPolicy>> = {
+export const MERGE_RETRY_POLICY_MATRIX: Readonly<
+	Record<MergeFailureClassification, MergeRetryPolicy>
+> = {
 	verification_new_failure: {
 		retriable: true,
 		maxAttempts: 1,
@@ -1788,7 +1837,6 @@ export const MERGE_FAILURE_CLASSIFICATIONS: readonly MergeFailureClassification[
 	"git_lock_file",
 ] as const;
 
-
 // ── Tier 0 Watchdog Recovery Types (TP-039) ──────────────────────────
 
 /**
@@ -1923,7 +1971,11 @@ export interface EscalationContext {
  *
  * @since TP-039
  */
-export function tier0ScopeKey(pattern: Tier0RecoveryPattern, taskId: string, waveIndex: number): string {
+export function tier0ScopeKey(
+	pattern: Tier0RecoveryPattern,
+	taskId: string,
+	waveIndex: number,
+): string {
 	return `t0:${pattern}:${taskId}:w${waveIndex}`;
 }
 
@@ -2066,7 +2118,6 @@ export interface EngineEvent {
  */
 export type EngineEventCallback = (event: EngineEvent) => void;
 
-
 // ── Supervisor Alert Types (TP-076) ──────────────────────────────────
 
 /**
@@ -2278,7 +2329,10 @@ export function buildSupervisorSegmentFrontierSnapshot(
 	preferredSegmentId?: string | null,
 ): SupervisorSegmentFrontierSnapshot | undefined {
 	const orderedSegmentIds = Array.isArray(segmentIds)
-		? segmentIds.filter((segmentId): segmentId is string => typeof segmentId === "string" && segmentId.trim().length > 0)
+		? segmentIds.filter(
+				(segmentId): segmentId is string =>
+					typeof segmentId === "string" && segmentId.trim().length > 0,
+			)
 		: [];
 	if (orderedSegmentIds.length === 0) return undefined;
 
@@ -2289,16 +2343,17 @@ export function buildSupervisorSegmentFrontierSnapshot(
 		}
 	}
 
-	const resolvedActiveSegmentId = (activeSegmentId && orderedSegmentIds.includes(activeSegmentId))
-		? activeSegmentId
-		: (preferredSegmentId && orderedSegmentIds.includes(preferredSegmentId)
-			? preferredSegmentId
-			: null);
+	const resolvedActiveSegmentId =
+		activeSegmentId && orderedSegmentIds.includes(activeSegmentId)
+			? activeSegmentId
+			: preferredSegmentId && orderedSegmentIds.includes(preferredSegmentId)
+				? preferredSegmentId
+				: null;
 
 	const segments = orderedSegmentIds.map((segmentId) => {
 		const persisted = bySegmentId.get(segmentId);
-		const status: PersistedSegmentStatus = persisted?.status
-			?? (resolvedActiveSegmentId === segmentId ? "running" : "pending");
+		const status: PersistedSegmentStatus =
+			persisted?.status ?? (resolvedActiveSegmentId === segmentId ? "running" : "pending");
 		return {
 			segmentId,
 			repoId: persisted ? parseSegmentIdRepo(persisted) : "unknown",
@@ -2307,11 +2362,12 @@ export function buildSupervisorSegmentFrontierSnapshot(
 		};
 	});
 
-	const terminalSegments = segments.filter((segment) =>
-		segment.status === "succeeded"
-		|| segment.status === "failed"
-		|| segment.status === "stalled"
-		|| segment.status === "skipped",
+	const terminalSegments = segments.filter(
+		(segment) =>
+			segment.status === "succeeded" ||
+			segment.status === "failed" ||
+			segment.status === "stalled" ||
+			segment.status === "skipped",
 	).length;
 
 	return {
@@ -2346,7 +2402,6 @@ export function buildEngineEventBase(
 	};
 }
 
-
 /**
  * Decision output from the merge retry policy evaluator.
  *
@@ -2383,50 +2438,50 @@ export interface MergeRetryDecision {
  */
 export type MergeRetryLoopOutcome =
 	| {
-		/** Retry succeeded — caller should continue normal post-merge flow */
-		kind: "retry_succeeded";
-		mergeResult: MergeWaveResult;
-		/** Classification of the failure that was retried */
-		classification: MergeFailureClassification | null;
-		/** Scope key used for retry counter tracking */
-		scopeKey: string;
-		/** Last retry decision (carries attempt/maxAttempts for event emission) */
-		lastDecision: MergeRetryDecision;
-	}
+			/** Retry succeeded — caller should continue normal post-merge flow */
+			kind: "retry_succeeded";
+			mergeResult: MergeWaveResult;
+			/** Classification of the failure that was retried */
+			classification: MergeFailureClassification | null;
+			/** Scope key used for retry counter tracking */
+			scopeKey: string;
+			/** Last retry decision (carries attempt/maxAttempts for event emission) */
+			lastDecision: MergeRetryDecision;
+	  }
 	| {
-		/** Safe-stop triggered during retry — caller should break the wave loop */
-		kind: "safe_stop";
-		mergeResult: MergeWaveResult;
-		/** Classification of the failure that was retried */
-		classification: MergeFailureClassification | null;
-		/** Scope key used for retry counter tracking */
-		scopeKey: string;
-		/** Last retry decision (carries attempt/maxAttempts for event emission) */
-		lastDecision: MergeRetryDecision;
-		errorMessage: string;
-		notifyMessage: string;
-	}
+			/** Safe-stop triggered during retry — caller should break the wave loop */
+			kind: "safe_stop";
+			mergeResult: MergeWaveResult;
+			/** Classification of the failure that was retried */
+			classification: MergeFailureClassification | null;
+			/** Scope key used for retry counter tracking */
+			scopeKey: string;
+			/** Last retry decision (carries attempt/maxAttempts for event emission) */
+			lastDecision: MergeRetryDecision;
+			errorMessage: string;
+			notifyMessage: string;
+	  }
 	| {
-		/**
-		 * Retry exhausted or failure is non-retriable — caller should
-		 * force `paused` regardless of on_merge_failure config.
-		 */
-		kind: "exhausted";
-		mergeResult: MergeWaveResult;
-		classification: MergeFailureClassification | null;
-		scopeKey: string;
-		lastDecision: MergeRetryDecision;
-		errorMessage: string;
-		notifyMessage: string;
-	}
+			/**
+			 * Retry exhausted or failure is non-retriable — caller should
+			 * force `paused` regardless of on_merge_failure config.
+			 */
+			kind: "exhausted";
+			mergeResult: MergeWaveResult;
+			classification: MergeFailureClassification | null;
+			scopeKey: string;
+			lastDecision: MergeRetryDecision;
+			errorMessage: string;
+			notifyMessage: string;
+	  }
 	| {
-		/** No retry attempted (unclassifiable or non-retriable with 0 attempts).
-		 *  Caller should fall through to standard on_merge_failure policy. */
-		kind: "no_retry";
-		mergeResult: MergeWaveResult;
-		classification: MergeFailureClassification | null;
-		scopeKey: string;
-	};
+			/** No retry attempted (unclassifiable or non-retriable with 0 attempts).
+			 *  Caller should fall through to standard on_merge_failure policy. */
+			kind: "no_retry";
+			mergeResult: MergeWaveResult;
+			classification: MergeFailureClassification | null;
+			scopeKey: string;
+	  };
 
 /**
  * Callbacks provided to `applyMergeRetryLoop()` for side effects
@@ -2510,7 +2565,6 @@ export interface OrchDashboardViewModel {
 	failurePolicy: string | null; // e.g., "stop-wave" if stopped by policy
 }
 
-
 // ── State Persistence Types (TS-009) ─────────────────────────────────
 
 // ── v3 Resilience & Diagnostics Sections (TP-030) ────────────────────
@@ -2832,7 +2886,13 @@ export interface PersistedTaskRecord {
  *
  * @since v4 (TP-081)
  */
-export type PersistedSegmentStatus = "pending" | "running" | "succeeded" | "failed" | "stalled" | "skipped";
+export type PersistedSegmentStatus =
+	| "pending"
+	| "running"
+	| "succeeded"
+	| "failed"
+	| "stalled"
+	| "skipped";
 
 /**
  * Persisted record of a single segment's execution state.
@@ -3095,7 +3155,6 @@ export interface PersistedBatchState {
 	_extraFields?: Record<string, unknown>;
 }
 
-
 // ── Resume (TS-009 Step 4) ───────────────────────────────────────────
 
 /**
@@ -3313,10 +3372,7 @@ export const DURATION_BASE_MINUTES = 30;
  * Get estimated duration in minutes for a task size.
  * Uses explicit mapping, falling back to weight × base.
  */
-export function getTaskDurationMinutes(
-	size: string,
-	sizeWeights: Record<string, number>,
-): number {
+export function getTaskDurationMinutes(size: string, sizeWeights: Record<string, number>): number {
 	if (SIZE_DURATION_MINUTES[size] !== undefined) {
 		return SIZE_DURATION_MINUTES[size];
 	}
@@ -3324,7 +3380,6 @@ export function getTaskDurationMinutes(
 	return weight * DURATION_BASE_MINUTES;
 }
 
-
 // ── Batch History ────────────────────────────────────────────────────
 
 /** Token counts for a task, wave, or batch. */
@@ -3341,8 +3396,8 @@ export interface BatchTaskSummary {
 	taskId: string;
 	taskName: string;
 	status: "succeeded" | "failed" | "skipped" | "blocked" | "stalled" | "pending";
-	wave: number;      // 1-based
-	lane: number;      // 1-based
+	wave: number; // 1-based
+	lane: number; // 1-based
 	durationMs: number;
 	tokens: TokenCounts;
 	exitReason: string | null;
@@ -3350,8 +3405,8 @@ export interface BatchTaskSummary {
 
 /** Per-wave summary for history. */
 export interface BatchWaveSummary {
-	wave: number;      // 1-based
-	tasks: string[];   // task IDs
+	wave: number; // 1-based
+	tasks: string[]; // task IDs
 	mergeStatus: "succeeded" | "failed" | "partial" | "skipped";
 	durationMs: number;
 	tokens: TokenCounts;
@@ -3380,7 +3435,6 @@ export interface BatchHistorySummary {
 /** Max number of batch history entries to retain. */
 export const BATCH_HISTORY_MAX_ENTRIES = 100;
 
-
 // ── Workspace Mode Types ─────────────────────────────────────────────
 
 /**
@@ -3518,7 +3572,6 @@ export interface ExecutionContext {
 	pointer: PointerResolution | null;
 }
 
-
 // ── Workspace Validation Error Types ─────────────────────────────────
 
 /**
@@ -3560,7 +3613,7 @@ export type WorkspaceConfigErrorCode =
 	| "WORKSPACE_TASK_AREA_OUTSIDE_TASKS_ROOT"
 	| "WORKSPACE_SETUP_REQUIRED"
 	| "WORKSPACE_DUPLICATE_REPO_PATH"
-	| "WORKSPACE_SCHEMA_INVALID";/**
+	| "WORKSPACE_SCHEMA_INVALID"; /**
  * Typed error class for workspace configuration failures.
  *
  * Thrown during workspace config loading/validation when the config file
@@ -3577,7 +3630,12 @@ export class WorkspaceConfigError extends Error {
 	/** Optional filesystem path related to the error */
 	relatedPath?: string;
 
-	constructor(code: WorkspaceConfigErrorCode, message: string, repoId?: string, relatedPath?: string) {
+	constructor(
+		code: WorkspaceConfigErrorCode,
+		message: string,
+		repoId?: string,
+		relatedPath?: string,
+	) {
 		super(message);
 		this.name = "WorkspaceConfigError";
 		this.code = code;
@@ -3586,7 +3644,6 @@ export class WorkspaceConfigError extends Error {
 	}
 }
 
-
 // ── Pointer Resolution Types ─────────────────────────────────────────
 
 /**
@@ -3653,7 +3710,6 @@ export interface PointerResolution {
 	warning?: string;
 }
 
-
 // ── Workspace Defaults ───────────────────────────────────────────────
 
 /**
@@ -3697,7 +3753,6 @@ export function createRepoModeContext(
 	};
 }
 
-
 // ── Agent Mailbox Types (TP-089) ─────────────────────────────────────
 
 /**
@@ -3735,7 +3790,12 @@ export type MailboxMessageType = "steer" | "query" | "abort" | "info" | "reply"
  * @since TP-089
  */
 export const MAILBOX_MESSAGE_TYPES: ReadonlySet<string> = new Set<MailboxMessageType>([
-	"steer", "query", "abort", "info", "reply", "escalate",
+	"steer",
+	"query",
+	"abort",
+	"info",
+	"reply",
+	"escalate",
 ]);
 
 /**
@@ -3838,7 +3898,10 @@ export type RuntimeAgentStatus =
 
 /** Set of terminal agent statuses (process is no longer alive). @since TP-102 */
 export const TERMINAL_AGENT_STATUSES: ReadonlySet<RuntimeAgentStatus> = new Set([
-	"exited", "crashed", "timed_out", "killed",
+	"exited",
+	"crashed",
+	"timed_out",
+	"killed",
 ]);
 
 /**
@@ -4173,7 +4236,11 @@ export function runtimeRoot(stateRoot: string, batchId: string): string {
  *
  * @since TP-102
  */
-export function runtimeAgentDir(stateRoot: string, batchId: string, agentId: RuntimeAgentId): string {
+export function runtimeAgentDir(
+	stateRoot: string,
+	batchId: string,
+	agentId: RuntimeAgentId,
+): string {
 	return `${stateRoot}/.pi/runtime/${batchId}/agents/${agentId}`;
 }
 
@@ -4182,7 +4249,11 @@ export function runtimeAgentDir(stateRoot: string, batchId: string, agentId: Run
  *
  * @since TP-102
  */
-export function runtimeManifestPath(stateRoot: string, batchId: string, agentId: RuntimeAgentId): string {
+export function runtimeManifestPath(
+	stateRoot: string,
+	batchId: string,
+	agentId: RuntimeAgentId,
+): string {
 	return `${runtimeAgentDir(stateRoot, batchId, agentId)}/manifest.json`;
 }
 
@@ -4191,7 +4262,11 @@ export function runtimeManifestPath(stateRoot: string, batchId: string, agentId:
  *
  * @since TP-102
  */
-export function runtimeAgentEventsPath(stateRoot: string, batchId: string, agentId: RuntimeAgentId): string {
+export function runtimeAgentEventsPath(
+	stateRoot: string,
+	batchId: string,
+	agentId: RuntimeAgentId,
+): string {
 	return `${runtimeAgentDir(stateRoot, batchId, agentId)}/events.jsonl`;
 }
 
@@ -4200,7 +4275,11 @@ export function runtimeAgentEventsPath(stateRoot: string, batchId: string, agent
  *
  * @since TP-102
  */
-export function runtimeLaneSnapshotPath(stateRoot: string, batchId: string, laneNumber: number): string {
+export function runtimeLaneSnapshotPath(
+	stateRoot: string,
+	batchId: string,
+	laneNumber: number,
+): string {
 	return `${stateRoot}/.pi/runtime/${batchId}/lanes/lane-${laneNumber}.json`;
 }
 
@@ -4244,7 +4323,11 @@ export interface RuntimeMergeSnapshot {
  *
  * @since TP-164
  */
-export function runtimeMergeSnapshotPath(stateRoot: string, batchId: string, mergeNumber: number): string {
+export function runtimeMergeSnapshotPath(
+	stateRoot: string,
+	batchId: string,
+	mergeNumber: number,
+): string {
 	return `${stateRoot}/.pi/runtime/${batchId}/lanes/merge-${mergeNumber}.json`;
 }
 
@@ -4309,15 +4392,28 @@ export function validateAgentManifest(manifest: unknown): string[] {
 	if (typeof m.role !== "string") errors.push("role must be a string");
 	else {
 		const validRoles: ReadonlySet<string> = new Set(["worker", "reviewer", "merger", "lane-runner"]);
-		if (!validRoles.has(m.role as string)) errors.push(`role must be one of: ${[...validRoles].join(", ")}`);
+		if (!validRoles.has(m.role as string))
+			errors.push(`role must be one of: ${[...validRoles].join(", ")}`);
 	}
-	if (typeof m.pid !== "number" || !Number.isFinite(m.pid) || m.pid <= 0) errors.push("pid must be a positive finite number");
-	if (typeof m.parentPid !== "number" || !Number.isFinite(m.parentPid) || m.parentPid <= 0) errors.push("parentPid must be a positive finite number");
-	if (typeof m.startedAt !== "number" || !Number.isFinite(m.startedAt)) errors.push("startedAt must be a finite number");
+	if (typeof m.pid !== "number" || !Number.isFinite(m.pid) || m.pid <= 0)
+		errors.push("pid must be a positive finite number");
+	if (typeof m.parentPid !== "number" || !Number.isFinite(m.parentPid) || m.parentPid <= 0)
+		errors.push("parentPid must be a positive finite number");
+	if (typeof m.startedAt !== "number" || !Number.isFinite(m.startedAt))
+		errors.push("startedAt must be a finite number");
 	if (typeof m.status !== "string") errors.push("status must be a string");
 	else {
-		const validStatuses: ReadonlySet<string> = new Set(["spawning", "running", "wrapping_up", "exited", "crashed", "timed_out", "killed"]);
-		if (!validStatuses.has(m.status as string)) errors.push(`status must be one of: ${[...validStatuses].join(", ")}`);
+		const validStatuses: ReadonlySet<string> = new Set([
+			"spawning",
+			"running",
+			"wrapping_up",
+			"exited",
+			"crashed",
+			"timed_out",
+			"killed",
+		]);
+		if (!validStatuses.has(m.status as string))
+			errors.push(`status must be one of: ${[...validStatuses].join(", ")}`);
 	}
 	if (typeof m.cwd !== "string" || !m.cwd) errors.push("cwd must be a non-empty string");
 	if (typeof m.repoId !== "string") errors.push("repoId must be a string");
@@ -4339,7 +4435,13 @@ export function validatePacketPaths(packet: unknown): string[] {
 	}
 	const p = packet as Record<string, unknown>;
 
-	for (const field of ["promptPath", "statusPath", "donePath", "reviewsDir", "taskFolder"] as const) {
+	for (const field of [
+		"promptPath",
+		"statusPath",
+		"donePath",
+		"reviewsDir",
+		"taskFolder",
+	] as const) {
 		if (typeof p[field] !== "string" || !(p[field] as string)) {
 			errors.push(`${field} must be a non-empty string`);
 		}
@@ -4347,4 +4449,3 @@ export function validatePacketPaths(packet: unknown): string[] {
 
 	return errors;
 }
-