@markjaquith/agency 3.2.14 → 3.3.0

package/src/commands/work.test.ts

@@ -1,1375 +0,0 @@
-import { describe, expect, test } from "bun:test"
-import { Effect } from "effect"
-import { FileSystemService } from "../services/FileSystemService"
-import { WorkbaseService } from "../services/WorkbaseService"
-import { EpicService } from "../services/EpicService"
-import { TaskService } from "../services/TaskService"
-import { PhaseService } from "../services/PhaseService"
-import { WorktreeService } from "../services/WorktreeService"
-import { ReadinessService } from "../services/ReadinessService"
-import { IntegrationService } from "../services/IntegrationService"
-import { captureErrors, captureLogs } from "../test-utils"
-import { work, workPrepare } from "./work"
-import type { PickWorkTarget } from "../workbase/work-target"
-import type { PickWorkbase } from "../workbase/workbase-choice"
-import type { Progress } from "../utils/progress"
-
-type ExecutionWorkspace = Effect.Effect.Success<
-	ReturnType<WorktreeService["materialize"]>
->
-
-const singlePhaseWorkspace: ExecutionWorkspace = {
-	root: "/workbase",
-	taskPath: "/workbase/tasks/example/TASK.md",
-	phasePath: null,
-	codePath: "/workbase/tasks/example/code",
-	writablePath: "/workbase/tasks/example/code/agency",
-	reviewPath: null,
-	repo: "agency",
-	repos: [],
-	dryRun: false,
-	checkouts: [],
-	operations: [],
-}
-
-const multiPhaseWorkspace: ExecutionWorkspace = {
-	root: "/workbase",
-	taskPath: "/workbase/tasks/example/TASK.md",
-	phasePath: "/workbase/tasks/example/phases/implementation/PHASE.md",
-	codePath: "/workbase/tasks/example/phases/implementation/code",
-	writablePath: "/workbase/tasks/example/phases/implementation/code/agency",
-	reviewPath: null,
-	repo: "agency",
-	repos: [],
-	dryRun: false,
-	checkouts: [],
-	operations: [],
-}
-
-const taskDirectory = "/workbase/tasks/example"
-const phaseDirectory = `${taskDirectory}/phases/implementation`
-
-interface HarnessOptions {
-	readonly workspace?: ExecutionWorkspace
-	readonly materializeError?: Error
-	readonly available?: Readonly<Record<string, boolean>>
-	readonly chooserCommand?: readonly string[]
-	readonly agent?: "opencode2" | "opencode" | "pi" | "claude"
-	readonly agents?: Record<
-		string,
-		{
-			command: readonly [string, ...string[]]
-			autoCommand?: readonly [string, ...string[]]
-			resumeCommand?: readonly [string, ...string[]]
-			autoResumeCommand?: readonly [string, ...string[]]
-			environment?: Record<string, string>
-		}
-	>
-	readonly multiPhaseTasks?: readonly string[]
-	readonly epicRecords?: readonly any[]
-	readonly taskRecords?: readonly any[]
-	readonly phaseRecords?: readonly any[]
-	readonly outsideWorkbase?: boolean
-	readonly registeredWorkbases?: readonly string[]
-	readonly existingDirectories?: readonly string[]
-	readonly guardError?: Error
-	readonly launchError?: Error
-	readonly workTargetIds?: readonly string[]
-	readonly taskStatus?: "open" | "working" | "delegated" | "done" | "dropped"
-	readonly phaseStatus?: "open" | "working" | "delegated" | "done" | "dropped"
-	readonly taskStatuses?: Readonly<
-		Record<string, "open" | "working" | "delegated" | "done" | "dropped">
-	>
-	readonly phaseStatuses?: Readonly<
-		Record<string, "open" | "working" | "delegated" | "done" | "dropped">
-	>
-	readonly taskRepo?: string
-	readonly existingPaths?: readonly string[]
-	readonly validationIssues?: readonly { path: string; message: string }[]
-}
-
-const createHarness = (options: HarnessOptions = {}) => {
-	const events: string[] = []
-	const probes: string[] = []
-	const statusUpdates: string[] = []
-	const shownTasks: string[] = []
-	const progressUpdates: string[] = []
-	let integrationSyncs = 0
-	const guards: Array<{ target: string; override?: boolean }> = []
-	const launches: Array<{
-		cli: string
-		args: readonly string[]
-		cwd: string
-	}> = []
-	const launchEnvironments: Array<Readonly<Record<string, string>>> = []
-	const launchProcessEnvironments: Array<Record<string, string | undefined>> =
-		[]
-	const materializeOptions: Array<
-		Parameters<WorktreeService["materialize"]>[3]
-	> = []
-	const taskStatuses = { ...options.taskStatuses }
-	const phaseStatuses = { ...options.phaseStatuses }
-	const worktrees = {
-		materialize: (
-			_taskId: string,
-			_phaseId?: string,
-			_root?: string,
-			commandOptions?: Parameters<WorktreeService["materialize"]>[3],
-		) => {
-			events.push("materialize")
-			materializeOptions.push(commandOptions)
-			return options.materializeError
-				? Effect.fail(options.materializeError)
-				: Effect.succeed(options.workspace ?? singlePhaseWorkspace)
-		},
-	}
-	const workbase = {
-		loadGlobalConfig: () => Effect.succeed({ agent: options.agent }),
-		discover: (path: string) =>
-			options.outsideWorkbase && path === "/outside"
-				? Effect.fail({
-						_tag: "WorkbaseNotFoundError" as const,
-						message: "No Agency workbase found from /outside",
-					})
-				: Effect.succeed("/workbase"),
-		listRegistered: () => Effect.succeed(options.registeredWorkbases ?? []),
-		getDefault: () => Effect.succeed(undefined),
-		loadConfig: () =>
-			Effect.succeed({
-				root: "/workbase",
-				config: {
-					version: 2 as const,
-					chooserCommand: options.chooserCommand,
-					agents: options.agents,
-				},
-			}),
-		repositoryAliases: () => Effect.succeed(["agency"]),
-		validate: () => {
-			const issues = options.validationIssues ?? []
-			return Effect.succeed({
-				root: "/workbase",
-				issues,
-				epicCount: 0,
-				taskCount: 1,
-				phaseCount: 0,
-				valid: issues.length === 0,
-			})
-		},
-	}
-	const epics = {
-		show: (id: string) =>
-			Effect.succeed({
-				id,
-				path: `/workbase/epics/${id}/EPIC.md`,
-				data: { tasks: [] },
-			}),
-		list: () => Effect.succeed(options.epicRecords ?? []),
-	}
-	const tasks = {
-		show: (id: string) => {
-			shownTasks.push(id)
-			return Effect.succeed({
-				id,
-				path: `/workbase/tasks/${id}/TASK.md`,
-				revision: "a".repeat(64),
-				data: options.multiPhaseTasks?.includes(id)
-					? { phases: [] }
-					: {
-							repo: options.taskRepo ?? "agency",
-							branch: `task/${id}`,
-							base: "main",
-							status: taskStatuses[id] ?? options.taskStatus ?? "open",
-						},
-			})
-		},
-		list: () => Effect.succeed(options.taskRecords ?? []),
-		setStatus: (id: string, status: string) => {
-			statusUpdates.push(`task:${id}:${status}`)
-			taskStatuses[id] = status as
-				| "open"
-				| "working"
-				| "delegated"
-				| "done"
-				| "dropped"
-			return Effect.void
-		},
-	}
-	const phases = {
-		show: (taskId: string, id: string) =>
-			Effect.succeed({
-				taskId,
-				id,
-				path: `/workbase/tasks/${taskId}/phases/${id}/PHASE.md`,
-				revision: "b".repeat(64),
-				data: {
-					repo: "agency",
-					branch: `task/${id}`,
-					base: "main",
-					status:
-						phaseStatuses[`${taskId}/${id}`] ?? options.phaseStatus ?? "open",
-				},
-			}),
-		list: () => Effect.succeed(options.phaseRecords ?? []),
-		setStatus: (taskId: string, id: string, status: string) => {
-			statusUpdates.push(`phase:${taskId}:${id}:${status}`)
-			phaseStatuses[`${taskId}/${id}`] = status as
-				| "open"
-				| "working"
-				| "delegated"
-				| "done"
-				| "dropped"
-			return Effect.void
-		},
-	}
-	const readiness = {
-		getWorkTargetIds: () =>
-			Effect.succeed(
-				new Set(
-					options.workTargetIds ?? [
-						...(options.epicRecords ?? []).map(
-							(record: any) => `epic:${record.id}`,
-						),
-						...(options.taskRecords ?? []).map((record: any) =>
-							"phases" in record.data
-								? `task:${record.id}`
-								: `execution-unit:task/${record.id}`,
-						),
-						...(options.phaseRecords ?? []).map(
-							(record: any) =>
-								`execution-unit:phase/${record.taskId}/${record.id}`,
-						),
-					],
-				),
-			),
-		guardWorkTarget: (target: string, _root: string, override?: boolean) => {
-			if (options.guardError || override) events.push("guard")
-			guards.push({ target, override })
-			return options.guardError && !override
-				? Effect.fail(options.guardError)
-				: Effect.void
-		},
-	}
-	const integrations = {
-		sync: () => {
-			integrationSyncs += 1
-			return Effect.succeed({
-				root: "/workbase",
-				files: [
-					{
-						name: "opencode",
-						state: "managed",
-					},
-				],
-			})
-		},
-		syncRoot: () => {
-			integrationSyncs += 1
-			return Effect.succeed({
-				root: "/workbase",
-				files: [
-					{
-						name: "opencode",
-						state: "managed",
-					},
-				],
-			})
-		},
-	}
-	const fs = {
-		isDirectory: (path: string) =>
-			Effect.succeed(
-				path === "/workbase/epics" || path === "/workbase/tasks"
-					? false
-					: (options.existingDirectories?.includes(path) ?? true),
-			),
-		readFile: (path: string) =>
-			Effect.succeed(path.endsWith("agency.json") ? '{"version":2}\n' : ""),
-		readDirectory: () => Effect.succeed([]),
-		exists: (path: string) =>
-			Effect.succeed(options.existingPaths?.includes(path) ?? false),
-		realPath: (path: string) => Effect.succeed(path),
-		runCommand: (args: readonly string[]) => {
-			const cli = args[1]!
-			events.push(`probe:${cli}`)
-			probes.push(cli)
-			return Effect.succeed({
-				exitCode: options.available?.[cli] === false ? 1 : 0,
-				stdout: "",
-				stderr: "",
-			})
-		},
-	}
-	const launch = (
-		cli: string,
-		args: readonly string[],
-		cwd: string,
-		environment: Readonly<Record<string, string>>,
-	) => {
-		events.push(`launch:${cli}`)
-		launches.push({ cli, args, cwd })
-		launchEnvironments.push(environment)
-		launchProcessEnvironments.push({
-			AGENCY_SESSION_ID: process.env.AGENCY_SESSION_ID,
-			AGENCY_TARGET: process.env.AGENCY_TARGET,
-			AGENCY_PROMPT: process.env.AGENCY_PROMPT,
-		})
-		if (options.launchError) throw options.launchError
-	}
-	const defaultPick: PickWorkTarget = () => Effect.succeed(null)
-	const defaultPickWorkbase: PickWorkbase = () => Effect.succeed(null)
-	const progress: Progress = {
-		start: (message) => progressUpdates.push(`start:${message}`),
-		succeed: (message) => progressUpdates.push(`succeed:${message}`),
-		fail: (message) => progressUpdates.push(`fail:${message}`),
-	}
-	const run = (
-		commandOptions: Parameters<typeof work>[0],
-		pick: PickWorkTarget = defaultPick,
-		pickBase: PickWorkbase = defaultPickWorkbase,
-		preserveSession = false,
-	) => {
-		const agencySessionId = process.env.AGENCY_SESSION_ID
-		if (!preserveSession) delete process.env.AGENCY_SESSION_ID
-		return Effect.runPromise(
-			work(commandOptions, launch, pick, progress, pickBase).pipe(
-				Effect.provideService(WorktreeService, worktrees as never),
-				Effect.provideService(FileSystemService, fs as never),
-				Effect.provideService(WorkbaseService, workbase as never),
-				Effect.provideService(EpicService, epics as never),
-				Effect.provideService(TaskService, tasks as never),
-				Effect.provideService(PhaseService, phases as never),
-				Effect.provideService(ReadinessService, readiness as never),
-				Effect.provideService(IntegrationService, integrations as never),
-			) as Effect.Effect<void, unknown, never>,
-		).finally(() => {
-			if (agencySessionId !== undefined) {
-				process.env.AGENCY_SESSION_ID = agencySessionId
-			}
-		})
-	}
-	const runPrepare = (commandOptions: Parameters<typeof workPrepare>[0]) =>
-		Effect.runPromise(
-			workPrepare(commandOptions).pipe(
-				Effect.provideService(WorktreeService, worktrees as never),
-				Effect.provideService(FileSystemService, fs as never),
-				Effect.provideService(WorkbaseService, workbase as never),
-				Effect.provideService(TaskService, tasks as never),
-				Effect.provideService(PhaseService, phases as never),
-				Effect.provideService(ReadinessService, readiness as never),
-			) as Effect.Effect<void, unknown, never>,
-		)
-
-	return {
-		events,
-		probes,
-		launches,
-		launchEnvironments,
-		launchProcessEnvironments,
-		materializeOptions,
-		statusUpdates,
-		taskStatuses,
-		phaseStatuses,
-		shownTasks,
-		progressUpdates,
-		guards,
-		get integrationSyncs() {
-			return integrationSyncs
-		},
-		run,
-		runPrepare,
-	}
-}
-
-describe("work command", () => {
-	test("reconciles managed integration files before preparing work", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example", opencode: true })
-
-		expect(harness.integrationSyncs).toBe(1)
-	})
-
-	test("guards execution targets before materialization and honors --force", async () => {
-		const blocked = createHarness({ guardError: new Error("blocked") })
-		await expect(
-			blocked.run({ taskId: "example", opencode: true }),
-		).rejects.toThrow("blocked")
-		expect(blocked.events).toEqual(["guard"])
-		expect(blocked.guards).toEqual([
-			{ target: "execution-unit:task/example", override: undefined },
-		])
-
-		const forced = createHarness({ guardError: new Error("blocked") })
-		await forced.run({ taskId: "example", opencode: true, force: true })
-		expect(forced.events).toEqual([
-			"guard",
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-		expect(forced.guards[0]).toEqual({
-			target: "execution-unit:task/example",
-			override: true,
-		})
-		expect(forced.materializeOptions[0]?.validationAlreadyPerformed).toBe(false)
-	})
-
-	test("reuses successful readiness validation during materialization", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example", opencode: true })
-
-		expect(harness.materializeOptions[0]?.validationAlreadyPerformed).toBe(true)
-	})
-
-	test("reopens forced terminal tasks through open before launching as working", async () => {
-		for (const previousStatus of ["done", "dropped"] as const) {
-			const harness = createHarness({
-				taskStatuses: { example: previousStatus },
-			})
-			const output = await captureLogs(() =>
-				harness.run({ taskId: "example", opencode: true, force: true }),
-			)
-
-			expect(harness.statusUpdates).toEqual([
-				"task:example:open",
-				"task:example:working",
-			])
-			expect(harness.taskStatuses.example).toBe("working")
-			expect(output).toEqual([
-				`Reopened task/example from ${previousStatus} as working`,
-			])
-		}
-	})
-
-	test("reopens forced terminal phases through open before launching as working", async () => {
-		for (const previousStatus of ["done", "dropped"] as const) {
-			const harness = createHarness({
-				workspace: multiPhaseWorkspace,
-				multiPhaseTasks: ["example"],
-				phaseStatuses: { "example/implementation": previousStatus },
-			})
-			const output = await captureLogs(() =>
-				harness.run({
-					taskId: "example",
-					phaseId: "implementation",
-					opencode: true,
-					force: true,
-				}),
-			)
-
-			expect(harness.statusUpdates).toEqual([
-				"phase:example:implementation:open",
-				"phase:example:implementation:working",
-			])
-			expect(harness.phaseStatuses["example/implementation"]).toBe("working")
-			expect(output).toEqual([
-				`Reopened phase/example/implementation from ${previousStatus} as working`,
-			])
-		}
-	})
-
-	test("reports a forced reopen as one machine result", async () => {
-		const harness = createHarness({ taskStatuses: { example: "done" } })
-		const output = await captureLogs(() =>
-			harness.run({
-				taskId: "example",
-				opencode: true,
-				force: true,
-				json: true,
-			}),
-		)
-
-		expect(JSON.parse(output.join("\n"))).toEqual({
-			target: "task/example",
-			reopened: true,
-			previousStatus: "done",
-			status: "working",
-		})
-	})
-
-	test("offers only launchable targets to the interactive chooser", async () => {
-		const harness = createHarness({
-			taskRecords: [
-				{
-					id: "ready",
-					path: "/workbase/tasks/ready/TASK.md",
-					data: { status: "open" },
-				},
-				{
-					id: "blocked",
-					path: "/workbase/tasks/blocked/TASK.md",
-					data: { status: "open" },
-				},
-			],
-			workTargetIds: ["execution-unit:task/ready"],
-		})
-		let labels: readonly string[] = []
-		const pick: PickWorkTarget = (choices) => {
-			labels = choices.map((choice) => choice.plainLabel)
-			return Effect.succeed(null)
-		}
-
-		await harness.run({ cwd: "/workbase" }, pick)
-		expect(labels).toEqual(["[open] task ready"])
-	})
-
-	test("prepares without launching or changing lifecycle status", async () => {
-		const harness = createHarness({ existingDirectories: [] })
-
-		const [output] = await captureLogs(() =>
-			harness.runPrepare({
-				cwd: "/workbase",
-				directory: "example",
-				json: true,
-				dryRun: true,
-			}),
-		)
-
-		expect(harness.events).toEqual(["materialize"])
-		expect(harness.launches).toEqual([])
-		expect(harness.statusUpdates).toEqual([])
-		expect(harness.materializeOptions[0]).toMatchObject({
-			json: true,
-			dryRun: true,
-			validationAlreadyPerformed: true,
-		})
-		const result = JSON.parse(output!)
-		expect(result.validationEvidence.status).toBe("refreshed")
-		expect(result.validationEvidence.reasons).toEqual(["not-supplied"])
-		expect(result.execution).toMatchObject({
-			capability: "agency-execution-v1",
-			mode: "preview",
-			executionIdentity: { target: "execution-unit:task/example" },
-			workspace: { state: "planned" },
-			commands: {
-				work: { argv: ["agency", "work", ".", "--auto"] },
-			},
-		})
-
-		const [reusedOutput] = await captureLogs(() =>
-			harness.runPrepare({
-				cwd: "/workbase",
-				directory: "example",
-				json: true,
-				dryRun: true,
-				evidence: JSON.stringify(result.validationEvidence.evidence),
-			}),
-		)
-		expect(JSON.parse(reusedOutput!).validationEvidence).toEqual(
-			expect.objectContaining({ status: "reused", reasons: [] }),
-		)
-
-		const conflicting = createHarness({
-			existingDirectories: [],
-			taskRepo: "other",
-		})
-		await expect(
-			conflicting.runPrepare({
-				cwd: "/workbase",
-				directory: "example",
-				dryRun: true,
-				evidence: JSON.stringify(result.validationEvidence.evidence),
-			}),
-		).rejects.toThrow("Recalled repository conflicts")
-	})
-
-	test("resolves task and phase document paths", async () => {
-		const taskDocument = "/workbase/tasks/example/TASK.md"
-		const taskHarness = createHarness({
-			existingDirectories: [],
-			existingPaths: [taskDocument],
-		})
-
-		await taskHarness.runPrepare({
-			directory: taskDocument,
-			dryRun: true,
-			silent: true,
-		})
-		expect(taskHarness.guards[0]?.target).toBe("execution-unit:task/example")
-		expect(taskHarness.shownTasks).toEqual(["example", "example"])
-
-		const phaseDocument =
-			"/workbase/tasks/example/phases/implementation/PHASE.md"
-		const phaseHarness = createHarness({
-			existingDirectories: [],
-			existingPaths: [phaseDocument],
-			multiPhaseTasks: ["example"],
-			workspace: multiPhaseWorkspace,
-		})
-
-		await phaseHarness.runPrepare({
-			directory: phaseDocument,
-			dryRun: true,
-			silent: true,
-		})
-		expect(phaseHarness.guards[0]?.target).toBe(
-			"execution-unit:phase/example/implementation",
-		)
-		expect(phaseHarness.materializeOptions).toHaveLength(1)
-	})
-
-	test("returns structured validation issues before readiness or materialization", async () => {
-		const harness = createHarness({
-			existingDirectories: [],
-			validationIssues: [
-				{
-					path: "tasks/example/TASK.md",
-					message: "Repository alias 'missing' is not configured",
-				},
-			],
-		})
-
-		await expect(
-			harness.runPrepare({
-				cwd: "/workbase",
-				directory: "example",
-				dryRun: true,
-			}),
-		).rejects.toThrow(
-			"Workbase validation failed with 1 issue:\n- tasks/example/TASK.md: Repository alias 'missing' is not configured",
-		)
-		expect(harness.guards).toEqual([])
-		expect(harness.events).toEqual([])
-	})
-
-	test("force prepares a phase blocked by an active dependency without launching or changing status", async () => {
-		const blocked = createHarness({
-			workspace: multiPhaseWorkspace,
-			multiPhaseTasks: ["example"],
-			guardError: new Error("Phase dependency is working"),
-		})
-		await expect(
-			blocked.runPrepare({
-				cwd: "/workbase",
-				taskId: "example",
-				phaseId: "implementation",
-			}),
-		).rejects.toThrow("Phase dependency is working")
-		expect(blocked.events).toEqual(["guard"])
-
-		const forced = createHarness({
-			workspace: multiPhaseWorkspace,
-			multiPhaseTasks: ["example"],
-			guardError: new Error("Phase dependency is working"),
-		})
-		await forced.runPrepare({
-			cwd: "/workbase",
-			taskId: "example",
-			phaseId: "implementation",
-			force: true,
-			silent: true,
-		})
-
-		expect(forced.guards).toEqual([
-			{
-				target: "execution-unit:phase/example/implementation",
-				override: true,
-			},
-		])
-		expect(forced.events).toEqual(["guard", "materialize"])
-		expect(forced.materializeOptions[0]).toMatchObject({
-			force: true,
-			validationAlreadyPerformed: true,
-		})
-		expect(forced.launches).toEqual([])
-		expect(forced.statusUpdates).toEqual([])
-	})
-
-	test("launches an epic agent from an epic directory", async () => {
-		const harness = createHarness()
-
-		await harness.run({
-			cwd: "/workbase/epics/delivery",
-			directory: ".",
-			opencode: true,
-			auto: true,
-		})
-
-		expect(harness.events).toEqual(["probe:opencode", "launch:opencode"])
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: [
-				"opencode",
-				"--prompt",
-				"Agency worker launch target: epic:delivery. Work on the epic. Read /workbase/epics/delivery/EPIC.md.",
-			],
-			cwd: "/workbase/epics/delivery",
-		})
-		expect(harness.launchEnvironments[0]?.OPENCODE_CONFIG).toBeUndefined()
-		expect(
-			harness.launchEnvironments[0]?.OPENCODE_CONFIG_CONTENT,
-		).toBeUndefined()
-	})
-
-	test("resolves an existing positional path before treating it as a task ID", async () => {
-		const harness = createHarness({
-			existingDirectories: ["/workbase/tasks/delivery"],
-		})
-
-		await harness.run({
-			cwd: "/workbase/tasks/delivery",
-			directory: ".",
-			opencode: true,
-		})
-
-		expect(harness.shownTasks).toEqual(["delivery", "delivery"])
-		expect(harness.launches[0]?.cwd).toBe(taskDirectory)
-	})
-
-	test("treats a positional value as a task ID when it is not a directory", async () => {
-		const harness = createHarness({ existingDirectories: [] })
-
-		await harness.run({
-			cwd: "/workbase",
-			directory: "delivery",
-			opencode: true,
-		})
-
-		expect(harness.shownTasks).toEqual(["delivery", "delivery"])
-		expect(harness.launches[0]?.cwd).toBe(taskDirectory)
-	})
-
-	test("launches a multi-phase task agent without materializing", async () => {
-		const harness = createHarness({ multiPhaseTasks: ["delivery"] })
-
-		await harness.run({
-			cwd: "/workbase/tasks/delivery",
-			directory: ".",
-			opencode: true,
-			auto: true,
-		})
-
-		expect(harness.events).toEqual(["probe:opencode", "launch:opencode"])
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: [
-				"opencode",
-				"--prompt",
-				"Agency worker launch target: task:delivery. Work on the task. Read /workbase/tasks/delivery/TASK.md.",
-			],
-			cwd: "/workbase/tasks/delivery",
-		})
-		expect(harness.launchEnvironments[0]?.OPENCODE_CONFIG).toBeUndefined()
-	})
-
-	test("infers a phase from a nested checkout directory", async () => {
-		const harness = createHarness({ workspace: multiPhaseWorkspace })
-
-		await harness.run({
-			cwd: "/workbase/tasks/example/phases/implementation/code/agency/src",
-			directory: ".",
-			opencode: true,
-			auto: true,
-		})
-
-		expect(harness.events).toEqual([
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: [
-				"opencode",
-				"--prompt",
-				"Agency worker launch target: execution-unit:phase/example/implementation. Start the task. Read /workbase/tasks/example/TASK.md and /workbase/tasks/example/phases/implementation/PHASE.md.",
-			],
-			cwd: phaseDirectory,
-		})
-		expect(harness.statusUpdates).toEqual([
-			"phase:example:implementation:working",
-		])
-		expect(harness.launchEnvironments[0]?.OPENCODE_CONFIG).toBeUndefined()
-	})
-
-	test("infers a single-phase task from a nested checkout directory", async () => {
-		const harness = createHarness()
-
-		await harness.run({
-			cwd: "/workbase/tasks/example/code/agency/src",
-			directory: ".",
-			opencode: true,
-		})
-
-		expect(harness.events[0]).toBe("materialize")
-		expect(harness.launches[0]?.cwd).toBe(taskDirectory)
-		expect(harness.launchEnvironments[0]?.OPENCODE_CONFIG).toBeUndefined()
-	})
-
-	test("selects a target when no directory is provided", async () => {
-		const phase = {
-			taskId: "delivery",
-			id: "build",
-			path: "/workbase/tasks/delivery/phases/build/PHASE.md",
-			data: {},
-		}
-		const harness = createHarness({
-			workspace: {
-				...multiPhaseWorkspace,
-				taskPath: "/workbase/tasks/delivery/TASK.md",
-				phasePath: "/workbase/tasks/delivery/phases/build/PHASE.md",
-			},
-			taskRecords: [
-				{
-					id: "delivery",
-					path: "/workbase/tasks/delivery/TASK.md",
-					data: { phases: [{ id: "build" }] },
-				},
-			],
-			phaseRecords: [phase],
-		})
-		const pick: PickWorkTarget = (choices) =>
-			Effect.succeed(
-				choices.find((choice) => choice.label.includes("build"))!.target,
-			)
-
-		await harness.run({ cwd: "/workbase/tasks/example", opencode: true }, pick)
-
-		expect(harness.events).toEqual([
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-		expect(harness.launches[0]?.cwd).toBe(
-			"/workbase/tasks/delivery/phases/build",
-		)
-	})
-
-	test("requires an explicit target when input is disabled", async () => {
-		const harness = createHarness()
-
-		await expect(
-			harness.run({
-				cwd: "/workbase",
-				opencode: true,
-				inputAllowed: false,
-			}),
-		).rejects.toThrow("provide a directory, task ID, or --epic")
-		expect(harness.events).toEqual([])
-	})
-
-	test("runs an explicit target when input is disabled", async () => {
-		const harness = createHarness({ existingDirectories: [] })
-
-		await harness.run({
-			cwd: "/workbase",
-			directory: "example",
-			opencode: true,
-			inputAllowed: false,
-		})
-
-		expect(harness.events).toEqual([
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-	})
-
-	test("selects a registered workbase when local discovery fails", async () => {
-		const harness = createHarness({
-			outsideWorkbase: true,
-			registeredWorkbases: ["/first", "/workbase"],
-		})
-		const selections: string[][] = []
-		const pickBase: PickWorkbase = (workbases) => {
-			selections.push([...workbases])
-			return Effect.succeed("/workbase")
-		}
-
-		await harness.run(
-			{ cwd: "/outside", taskId: "example", opencode: true },
-			undefined,
-			pickBase,
-		)
-
-		expect(selections).toEqual([["/first", "/workbase"]])
-		expect(harness.events).toEqual([
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-	})
-
-	test("does not select a registered workbase when input is disabled", async () => {
-		const harness = createHarness({
-			outsideWorkbase: true,
-			registeredWorkbases: ["/workbase"],
-			existingDirectories: [],
-		})
-
-		await expect(
-			harness.run({
-				cwd: "/outside",
-				directory: "example",
-				opencode: true,
-				inputAllowed: false,
-			}),
-		).rejects.toThrow("provide an explicit path or run Agency from a workbase")
-		expect(harness.events).toEqual([])
-	})
-
-	test("explains how to register a workbase when none are known", async () => {
-		const harness = createHarness({ outsideWorkbase: true })
-
-		await expect(harness.run({ cwd: "/outside" })).rejects.toThrow(
-			"agency workbase add <path>",
-		)
-		expect(harness.events).toEqual([])
-	})
-
-	test("passes the configured chooser command to the shared picker", async () => {
-		const harness = createHarness({
-			chooserCommand: ["gum", "filter"],
-			epicRecords: [
-				{
-					id: "delivery",
-					path: "/workbase/epics/delivery/EPIC.md",
-					data: { tasks: [] },
-				},
-			],
-		})
-		let command: readonly string[] | undefined
-		const pick: PickWorkTarget = (_choices, chooserCommand) => {
-			command = chooserCommand
-			return Effect.succeed(null)
-		}
-
-		await harness.run({ cwd: "/workbase" }, pick)
-
-		expect(command).toEqual(["gum", "filter"])
-		expect(harness.launches).toEqual([])
-	})
-
-	test("rejects conflicting agent flags before materialization", async () => {
-		const harness = createHarness()
-
-		await expect(
-			harness.run({ taskId: "example", opencode: true, claude: true }),
-		).rejects.toThrow("Cannot combine --agent, --opencode, and --claude")
-		expect(harness.events).toEqual([])
-	})
-
-	test("rejects combining explicit epic and task targets", async () => {
-		const harness = createHarness()
-
-		await expect(
-			harness.run({ epicId: "delivery", taskId: "example" }),
-		).rejects.toThrow("Cannot combine --epic")
-		expect(harness.events).toEqual([])
-	})
-
-	test("launches OpenCode in the task directory with explicit context", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example", opencode: true })
-
-		expect(harness.events).toEqual([
-			"materialize",
-			"probe:opencode",
-			"launch:opencode",
-		])
-		expect(harness.launches).toEqual([
-			{
-				cli: "opencode",
-				args: ["opencode"],
-				cwd: taskDirectory,
-			},
-		])
-		expect(harness.launchEnvironments[0]?.AGENCY_PROMPT).toBe("")
-		expect(harness.statusUpdates).toEqual(["task:example:working"])
-		expect(harness.launchEnvironments[0]?.OPENCODE_CONFIG).toBeUndefined()
-		expect(harness.progressUpdates).toEqual([
-			"start:Preparing workspace...",
-			"succeed:Workspace ready",
-		])
-	})
-
-	test("launches OpenCode in the phase directory with explicit context", async () => {
-		const harness = createHarness({ workspace: multiPhaseWorkspace })
-
-		await harness.run({
-			taskId: "example",
-			phaseId: "implementation",
-			opencode: true,
-		})
-
-		expect(harness.launches[0]?.cwd).toBe(phaseDirectory)
-	})
-
-	test("sends the generated prompt only with --auto", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example", opencode: true, auto: true })
-
-		expect(harness.launches[0]?.args).toEqual([
-			"opencode",
-			"--prompt",
-			"Agency worker launch target: execution-unit:task/example. Start the task. Read /workbase/tasks/example/TASK.md.",
-		])
-		expect(harness.launchEnvironments[0]?.AGENCY_PROMPT).toBe(
-			"Agency worker launch target: execution-unit:task/example. Start the task. Read /workbase/tasks/example/TASK.md.",
-		)
-		expect(harness.launchProcessEnvironments[0]).toEqual({
-			AGENCY_SESSION_ID: harness.launchEnvironments[0]?.AGENCY_SESSION_ID,
-			AGENCY_TARGET: "execution-unit:task/example",
-			AGENCY_PROMPT:
-				"Agency worker launch target: execution-unit:task/example. Start the task. Read /workbase/tasks/example/TASK.md.",
-		})
-	})
-
-	test("continues existing work with the resume command", async () => {
-		const harness = createHarness({ taskStatus: "working" })
-
-		await harness.run({ taskId: "example", opencode: true, auto: true })
-
-		expect(harness.launches[0]?.args).toEqual([
-			"opencode",
-			"--continue",
-			"--prompt",
-			"Agency worker launch target: execution-unit:task/example. Continue the task. Read /workbase/tasks/example/TASK.md.",
-		])
-		expect(harness.launchEnvironments[0]?.AGENCY_PROMPT).toBe(
-			"Agency worker launch target: execution-unit:task/example. Continue the task. Read /workbase/tasks/example/TASK.md.",
-		)
-	})
-
-	test("resumes OpenCode deterministically when a session identity exists", async () => {
-		const harness = createHarness({ workspace: multiPhaseWorkspace })
-		process.env.AGENCY_SESSION_ID = "existing-session"
-		try {
-			await harness.run(
-				{
-					taskId: "example",
-					phaseId: "implementation",
-					opencode: true,
-					auto: true,
-				},
-				undefined,
-				undefined,
-				true,
-			)
-		} finally {
-			delete process.env.AGENCY_SESSION_ID
-		}
-
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: [
-				"opencode",
-				"--continue",
-				"--prompt",
-				"Agency worker launch target: execution-unit:phase/example/implementation. Continue the task. Read /workbase/tasks/example/TASK.md and /workbase/tasks/example/phases/implementation/PHASE.md.",
-			],
-			cwd: phaseDirectory,
-		})
-	})
-
-	test("expands a named agent with shared context", async () => {
-		const harness = createHarness({
-			available: { codex: true },
-			agents: {
-				custom: {
-					command: ["codex"],
-					autoCommand: ["codex", "--task", "{task}", "{prompt}"],
-					environment: {
-						CUSTOM_TARGET: "{target}",
-						AGENCY_TARGET: "cannot-override",
-					},
-				},
-			},
-		})
-
-		await harness.run({ taskId: "example", agent: "custom", auto: true })
-
-		expect(harness.probes).toEqual(["codex"])
-		expect(harness.launches[0]).toEqual({
-			cli: "codex",
-			args: [
-				"codex",
-				"--task",
-				"example",
-				"Agency worker launch target: execution-unit:task/example. Start the task. Read /workbase/tasks/example/TASK.md.",
-			],
-			cwd: taskDirectory,
-		})
-		expect(harness.launchEnvironments[0]).toMatchObject({
-			AGENCY_AGENT: "custom",
-			AGENCY_WORKBASE: "/workbase",
-			AGENCY_TARGET: "execution-unit:task/example",
-			AGENCY_TASK_ID: "example",
-			AGENCY_PHASE_ID: "",
-			CUSTOM_TARGET: "execution-unit:task/example",
-		})
-	})
-
-	test("prints the exact command contract without launching and omits secrets", async () => {
-		const harness = createHarness({
-			available: { agent: true },
-			agents: {
-				custom: {
-					command: ["agent"],
-					autoCommand: ["agent", "{prompt}"],
-					environment: {
-						VISIBLE: "{task}",
-						API_TOKEN: "do-not-print",
-					},
-				},
-			},
-		})
-
-		const output = await captureLogs(() =>
-			harness.run({
-				taskId: "example",
-				agent: "custom",
-				printCommand: true,
-			}),
-		)
-		const printed = JSON.parse(output.join("\n"))
-
-		expect(harness.launches).toEqual([])
-		expect(harness.statusUpdates).toEqual([])
-		expect(printed.cwd).toBe(taskDirectory)
-		expect(printed.argv).toEqual(["agent"])
-		expect(printed.environment.AGENCY_PROMPT).toBe("")
-		expect(printed.environment.VISIBLE).toBe("example")
-		expect(printed.environment.API_TOKEN).toBeUndefined()
-	})
-
-	test("prints the phase directory in the command contract", async () => {
-		const harness = createHarness({ workspace: multiPhaseWorkspace })
-		const output = await captureLogs(() =>
-			harness.run({
-				taskId: "example",
-				phaseId: "implementation",
-				opencode: true,
-				printCommand: true,
-			}),
-		)
-
-		expect(JSON.parse(output.join("\n")).cwd).toBe(phaseDirectory)
-	})
-
-	test("does not reopen a forced terminal target in print-only mode", async () => {
-		const harness = createHarness({ taskStatuses: { example: "done" } })
-
-		await captureLogs(() =>
-			harness.run({
-				taskId: "example",
-				opencode: true,
-				force: true,
-				printCommand: true,
-			}),
-		)
-
-		expect(harness.statusUpdates).toEqual([])
-		expect(harness.taskStatuses.example).toBe("done")
-	})
-
-	test("leaves managed OpenCode access to the project plugin", async () => {
-		const harness = createHarness()
-		const output = await captureLogs(() =>
-			harness.run({ taskId: "example", opencode: true, printCommand: true }),
-		)
-		const printed = JSON.parse(output.join("\n"))
-
-		expect(printed.environment.OPENCODE_CONFIG).toBeUndefined()
-		expect(printed.environment.OPENCODE_CONFIG_CONTENT).toBeUndefined()
-	})
-
-	test("provides the writable checkout to plugins without changing the project", async () => {
-		const harness = createHarness()
-		await harness.run({ taskId: "example", opencode: true })
-
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: ["opencode"],
-			cwd: taskDirectory,
-		})
-		expect(harness.launchEnvironments[0]?.AGENCY_WRITABLE_CHECKOUT).toBe(
-			"/workbase/tasks/example/code/agency",
-		)
-		expect(
-			harness.launchEnvironments[0]?.OPENCODE_CONFIG_CONTENT,
-		).toBeUndefined()
-	})
-
-	test("automatically prefers opencode2", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example" })
-
-		expect(harness.probes).toEqual(["opencode2"])
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode2",
-			args: ["opencode2"],
-			cwd: taskDirectory,
-		})
-	})
-
-	test("automatically falls back from opencode2 to opencode", async () => {
-		const harness = createHarness({ available: { opencode2: false } })
-
-		await harness.run({ taskId: "example" })
-
-		expect(harness.probes).toEqual(["opencode2", "opencode"])
-		expect(harness.launches[0]).toEqual({
-			cli: "opencode",
-			args: ["opencode"],
-			cwd: taskDirectory,
-		})
-	})
-
-	test("automatically falls back from OpenCode to Claude", async () => {
-		const harness = createHarness({
-			available: { opencode2: false, opencode: false, pi: false },
-		})
-
-		await harness.run({ taskId: "example" })
-
-		expect(harness.probes).toEqual(["opencode2", "opencode", "pi", "claude"])
-		expect(harness.launches[0]).toEqual({
-			cli: "claude",
-			args: ["claude"],
-			cwd: taskDirectory,
-		})
-	})
-
-	test("automatically falls back from OpenCode to Pi", async () => {
-		const harness = createHarness({
-			available: { opencode2: false, opencode: false },
-		})
-
-		await harness.run({ taskId: "example" })
-
-		expect(harness.probes).toEqual(["opencode2", "opencode", "pi"])
-		expect(harness.launches[0]).toMatchObject({ cli: "pi", args: ["pi"] })
-	})
-
-	test("uses the global agent before automatic detection", async () => {
-		const harness = createHarness({ agent: "pi" })
-
-		await harness.run({ taskId: "example" })
-
-		expect(harness.probes).toEqual(["pi"])
-		expect(harness.launches[0]).toMatchObject({ cli: "pi", args: ["pi"] })
-	})
-
-	test("lets an invocation agent override the global agent", async () => {
-		const harness = createHarness({ agent: "pi" })
-
-		await harness.run({ taskId: "example", agent: "claude" })
-
-		expect(harness.probes).toEqual(["claude"])
-		expect(harness.launches[0]).toMatchObject({
-			cli: "claude",
-			args: ["claude"],
-		})
-	})
-
-	test("does not fall back when OpenCode is explicitly required", async () => {
-		const harness = createHarness({ available: { opencode: false } })
-
-		await expect(
-			harness.run({ taskId: "example", opencode: true }),
-		).rejects.toThrow("opencode CLI tool not found")
-		expect(harness.probes).toEqual(["opencode"])
-		expect(harness.launches).toEqual([])
-		expect(harness.statusUpdates).toEqual([])
-	})
-
-	test("launches explicitly requested Claude", async () => {
-		const harness = createHarness()
-
-		await harness.run({ taskId: "example", claude: true })
-
-		expect(harness.probes).toEqual(["claude"])
-		expect(harness.launches[0]).toEqual({
-			cli: "claude",
-			args: ["claude"],
-			cwd: taskDirectory,
-		})
-	})
-
-	test("fails when neither agent tool is available", async () => {
-		const harness = createHarness({
-			available: {
-				opencode2: false,
-				opencode: false,
-				pi: false,
-				claude: false,
-			},
-		})
-
-		await expect(harness.run({ taskId: "example" })).rejects.toThrow(
-			"claude CLI tool not found",
-		)
-		expect(harness.probes).toEqual(["opencode2", "opencode", "pi", "claude"])
-		expect(harness.launches).toEqual([])
-	})
-
-	test("does not probe or launch when materialization fails", async () => {
-		const harness = createHarness({
-			materializeError: new Error("materialization failed"),
-		})
-
-		await expect(harness.run({ taskId: "example" })).rejects.toThrow(
-			"materialization failed",
-		)
-		expect(harness.events).toEqual(["materialize"])
-		expect(harness.progressUpdates).toEqual([
-			"start:Preparing workspace...",
-			"fail:Workspace preparation failed",
-		])
-	})
-
-	test("does not reopen a terminal target when preparation fails", async () => {
-		const harness = createHarness({
-			taskStatuses: { example: "dropped" },
-			materializeError: new Error("materialization failed"),
-		})
-
-		await expect(
-			harness.run({ taskId: "example", force: true }),
-		).rejects.toThrow("materialization failed")
-		expect(harness.statusUpdates).toEqual([])
-		expect(harness.taskStatuses.example).toBe("dropped")
-	})
-
-	test("retains working when launch fails after a forced reopen", async () => {
-		const harness = createHarness({
-			taskStatuses: { example: "done" },
-			launchError: new Error("launch failed"),
-		})
-
-		await expect(
-			harness.run({ taskId: "example", force: true, silent: true }),
-		).rejects.toThrow("launch failed")
-		expect(harness.statusUpdates).toEqual([
-			"task:example:open",
-			"task:example:working",
-		])
-		expect(harness.taskStatuses.example).toBe("working")
-	})
-
-	test("respects silent and verbose logging options", async () => {
-		const verboseHarness = createHarness()
-		const verboseLogs = await captureErrors(() =>
-			verboseHarness.run({ taskId: "example", verbose: true }),
-		)
-		expect(verboseLogs).toEqual([
-			"Launching command: opencode2 (cwd: /workbase/tasks/example)",
-		])
-		expect(verboseHarness.materializeOptions[0]?.verbose).toBe(true)
-
-		const silentHarness = createHarness()
-		const silentLogs = await captureErrors(() =>
-			silentHarness.run({
-				taskId: "example",
-				verbose: true,
-				silent: true,
-			}),
-		)
-		expect(silentLogs).toEqual([])
-	})
-})