@selesai/code 0.8.3 → 0.8.4

package/dist/extensions/pi-subagents/test/unit/chain-root-attachment.test.ts

@@ -39,12 +39,12 @@ describe("async chain root attachment", () => {
 			mode: "single",
 			state: "complete",
 			startedAt: 1,
-			steps: [{ agent: "builder", status: "complete", sessionFile }],
+			steps: [{ agent: "worker", status: "complete", sessionFile }],
 		});
 		writeJson(importedRoot.resultPath, {
 			state: "complete",
 			success: true,
-			results: [{ agent: "builder", output: "root output", success: true, sessionFile }],
+			results: [{ agent: "worker", output: "root output", success: true, sessionFile }],
 		});
 
 		const result = await waitForImportedAsyncRoot(importedRoot, { pollIntervalMs: 1 });
@@ -55,7 +55,7 @@ describe("async chain root attachment", () => {
 			exitCode: result.exitCode,
 			sessionFile: result.sessionFile,
 		}, {
-			agent: "builder",
+			agent: "worker",
 			output: "root output",
 			exitCode: 0,
 			sessionFile,
@@ -69,7 +69,7 @@ describe("async chain root attachment", () => {
 			mode: "single",
 			state: "running",
 			startedAt: 1,
-			steps: [{ agent: "builder", status: "running" }],
+			steps: [{ agent: "worker", status: "running" }],
 		});
 
 		const waiting = waitForImportedAsyncRoot(importedRoot, { pollIntervalMs: 5 });
@@ -77,7 +77,7 @@ describe("async chain root attachment", () => {
 			writeJson(importedRoot.resultPath, {
 				state: "complete",
 				success: true,
-				results: [{ agent: "builder", output: "late root output", success: true }],
+				results: [{ agent: "worker", output: "late root output", success: true }],
 			});
 		}, 20);
 
@@ -95,13 +95,13 @@ describe("async chain root attachment", () => {
 			state: "failed",
 			startedAt: 1,
 			error: "root failed",
-			steps: [{ agent: "builder", status: "failed", error: "root failed" }],
+			steps: [{ agent: "worker", status: "failed", error: "root failed" }],
 		});
 		writeJson(importedRoot.resultPath, {
 			state: "failed",
 			success: false,
 			summary: "root failed",
-			results: [{ agent: "builder", output: "root failed", error: "root failed", success: false }],
+			results: [{ agent: "worker", output: "root failed", error: "root failed", success: false }],
 		});
 
 		const result = await waitForImportedAsyncRoot(importedRoot, { pollIntervalMs: 1 });
@@ -118,7 +118,7 @@ describe("async chain root attachment", () => {
 			mode: "single",
 			state: "complete",
 			startedAt: 1,
-			steps: [{ agent: "builder", status: "complete" }],
+			steps: [{ agent: "worker", status: "complete" }],
 		});
 
 		const result = await waitForImportedAsyncRoot(importedRoot, {
@@ -137,7 +137,7 @@ describe("async chain root attachment", () => {
 			mode: "single",
 			state: "running",
 			startedAt: 1,
-			steps: [{ agent: "builder", status: "running" }],
+			steps: [{ agent: "worker", status: "running" }],
 		});
 		let timedOut = false;
 		const waiting = waitForImportedAsyncRoot(importedRoot, {

package/dist/extensions/pi-subagents/test/unit/chain-serializer.test.ts

@@ -7,7 +7,7 @@ name: review-chain
 description: Review chain
 ---
 
-## commentator
+## reviewer
 output: report.md
 outputMode: file-only
 
@@ -28,7 +28,7 @@ name: review-chain
 description: Review chain
 ---
 
-## commentator
+## reviewer
 phase: Review
 label: correctness pass
 as: correctnessFindings
@@ -54,7 +54,7 @@ name: review-chain
 description: Review chain
 ---
 
-## commentator
+## reviewer
 toolBudget: {"soft":3,"hard":5,"block":["read","grep"]}
 
 Review the diff
@@ -71,12 +71,12 @@ name: review-chain
 description: Review chain
 ---
 
-## commentator
+## reviewer
 toolBudget: {"soft":6,"hard":5}
 
 Review the diff
 `, "project", "/tmp/review-chain.md"),
-			/toolBudget for step 'commentator'\.soft must be <= toolBudget for step 'commentator'\.hard/,
+			/toolBudget for step 'reviewer'\.soft must be <= toolBudget for step 'reviewer'\.hard/,
 		);
 	});
 
@@ -87,7 +87,7 @@ name: review-chain
 description: Review chain
 ---
 
-## commentator
+## reviewer
 outputSchema: {"type":"object"}
 
 Review the diff
@@ -102,10 +102,10 @@ Review the diff
 				name: "bad-dynamic-review",
 				description: "Bad dynamic targets",
 				chain: [
-					{ agent: "explorer", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
+					{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
 					{
 						expand: { from: { output: "targets", path: "/items" }, maxItems: 4 },
-						parallel: [{ agent: "commentator", task: "Review" }],
+						parallel: [{ agent: "reviewer", task: "Review" }],
 						collect: { as: "reviews" },
 					},
 				],
@@ -124,10 +124,10 @@ Review the diff
 			package: "code-analysis",
 			description: "Review dynamic targets",
 			chain: [
-				{ agent: "explorer", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
+				{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
 				{
 					expand: { from: { output: "targets", path: "/items" }, item: "target", key: "/path", maxItems: 4 },
-					parallel: { agent: "commentator", task: "Review {target.path}", outputSchema: { type: "object" } },
+					parallel: { agent: "reviewer", task: "Review {target.path}", outputSchema: { type: "object" } },
 					collect: { as: "reviews" },
 				},
 			],
@@ -146,10 +146,10 @@ Review the diff
 			name: "dynamic-review",
 			description: "Review dynamic targets",
 			chain: [
-				{ agent: "explorer", task: "Return targets", as: "targets", outputSchema: { type: "object" }, toolBudget: { hard: 4 } },
+				{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" }, toolBudget: { hard: 4 } },
 				{
 					expand: { from: { output: "targets", path: "/items" }, item: "target", key: "/path", maxItems: 4 },
-					parallel: { agent: "commentator", task: "Review {target.path}", outputSchema: { type: "object" }, toolBudget: { soft: 3, hard: 5 } },
+					parallel: { agent: "reviewer", task: "Review {target.path}", outputSchema: { type: "object" }, toolBudget: { soft: 3, hard: 5 } },
 					collect: { as: "reviews" },
 				},
 			],
@@ -164,7 +164,7 @@ Review the diff
 			() => parseJsonChain(JSON.stringify({
 				name: "bad-tool-budget",
 				description: "Bad tool budget",
-				chain: [{ agent: "builder", toolBudget: { hard: 0 } }],
+				chain: [{ agent: "worker", toolBudget: { hard: 0 } }],
 			}), "project", "/tmp/bad-tool-budget.chain.json"),
 			/step 1 toolBudget\.hard must be an integer >= 1/,
 		);
@@ -173,8 +173,8 @@ Review the diff
 				name: "bad-dynamic-tool-budget",
 				description: "Bad dynamic tool budget",
 				chain: [
-					{ agent: "explorer", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
-					{ expand: { from: { output: "targets", path: "/items" }, maxItems: 4 }, parallel: { agent: "builder", toolBudget: { hard: 3, block: [] } }, collect: { as: "reviews" } },
+					{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
+					{ expand: { from: { output: "targets", path: "/items" }, maxItems: 4 }, parallel: { agent: "worker", toolBudget: { hard: 3, block: [] } }, collect: { as: "reviews" } },
 				],
 			}), "project", "/tmp/bad-dynamic-tool-budget.chain.json"),
 			/step 2 dynamic template toolBudget\.block must contain at least one tool name/,
@@ -186,10 +186,10 @@ Review the diff
 			name: "dynamic-review",
 			description: "Review dynamic targets",
 			chain: [
-				{ agent: "explorer", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
+				{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
 				{
 					expand: { from: { output: "targets", path: "/items" }, item: "target", key: "/path", maxItems: 4 },
-					parallel: { agent: "commentator", task: "Review {target.path}", outputSchema: { type: "object" } },
+					parallel: { agent: "reviewer", task: "Review {target.path}", outputSchema: { type: "object" } },
 					collect: { as: "reviews" },
 				},
 			],
@@ -206,10 +206,10 @@ Review the diff
 			name: "accepted-chain",
 			description: "Chain with acceptance gates",
 			chain: [
-				{ agent: "builder", task: "Fix bug", acceptance: { level: "checked", evidence: ["changed-files", "commands-run"] } },
+				{ agent: "worker", task: "Fix bug", acceptance: { level: "checked", evidence: ["changed-files", "commands-run"] } },
 				{
 					parallel: [
-						{ agent: "commentator", task: "Review", acceptance: "attested" },
+						{ agent: "reviewer", task: "Review", acceptance: "attested" },
 					],
 				},
 			],
@@ -221,7 +221,7 @@ Review the diff
 			() => parseJsonChain(JSON.stringify({
 				name: "bad-acceptance",
 				description: "Bad acceptance",
-				chain: [{ agent: "builder", acceptance: { level: "none" } }],
+				chain: [{ agent: "worker", acceptance: { level: "none" } }],
 			}), "project", "/tmp/bad-acceptance.chain.json"),
 			/step 1 acceptance\.reason is required/,
 		);

package/dist/extensions/pi-subagents/test/unit/child-transcript.test.ts

@@ -21,7 +21,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "foreground",
 			runId: "run-1",
-			agent: "builder",
+			agent: "worker",
 			childIndex: 0,
 			cwd: "/repo",
 		});
@@ -35,7 +35,7 @@ describe("createChildTranscriptWriter", () => {
 		assert.equal(record.recordType, "message");
 		assert.equal(record.source, "foreground");
 		assert.equal(record.runId, "run-1");
-		assert.equal(record.agent, "builder");
+		assert.equal(record.agent, "worker");
 		assert.equal(record.childIndex, 0);
 		assert.equal(record.cwd, "/repo");
 		assert.equal(record.sourceEventType, "initial_prompt");
@@ -52,7 +52,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "async",
 			runId: "run-2",
-			agent: "commentator",
+			agent: "reviewer",
 			cwd: "/repo",
 		});
 		writer.writeInitialUserMessage("review it");
@@ -68,7 +68,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "async",
 			runId: "run-3",
-			agent: "builder",
+			agent: "worker",
 			childIndex: 1,
 			cwd: "/repo",
 		});
@@ -119,7 +119,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "foreground",
 			runId: "run-4",
-			agent: "builder",
+			agent: "worker",
 			cwd: "/repo",
 		});
 		writer.writeChildEvent({ type: "tool_execution_start", toolCallId: "call-1", toolName: "bash", args: { command: "ls" } });
@@ -152,7 +152,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "foreground",
 			runId: "run-bounded",
-			agent: "builder",
+			agent: "worker",
 			cwd: "/repo",
 		});
 		writer.writeChildEvent({ type: "tool_execution_start", toolCallId: "large", toolName: "bash", args: { command: "🧪".repeat(10_000) } });
@@ -178,7 +178,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "async",
 			runId: "run-5",
-			agent: "builder",
+			agent: "worker",
 			cwd: "/repo",
 		});
 		writer.writeStdoutLine("first line");
@@ -202,7 +202,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "foreground",
 			runId: "run-6",
-			agent: "builder",
+			agent: "worker",
 			cwd: "/repo",
 			maxBytes: 900,
 		});
@@ -254,7 +254,7 @@ describe("createChildTranscriptWriter", () => {
 			transcriptPath,
 			source: "foreground",
 			runId: "run-7",
-			agent: "builder",
+			agent: "worker",
 			cwd: "/repo",
 		});
 		writer.writeInitialUserMessage("should not be written");

package/dist/extensions/pi-subagents/test/unit/compaction-resume.test.ts

@@ -15,23 +15,53 @@ describe("async compaction resume", () => {
 			const handlers = new Map();
 			const events = { listeners: new Map(), on(name, handler) { this.listeners.set(name, handler); return () => this.listeners.delete(name); }, emit(name, payload) { this.listeners.get(name)?.(payload); } };
 			const sent = [];
+			const widgets = [];
+			let renders = 0;
 			const pi = new Proxy({
 				events,
 				on(name, handler) { handlers.set(name, handler); },
 				registerTool() {}, registerCommand() {}, registerShortcut() {}, registerMessageRenderer() {},
 				sendMessage(message, options) { sent.push({ message, options }); }, getSessionName() { return undefined; },
 			}, { get(target, prop) { return prop in target ? target[prop] : () => undefined; } });
-			const ctx = { cwd: process.cwd(), hasUI: true, ui: { setWidget() {}, requestRender() {}, theme: { fg(_name, text) { return text; }, bg(_name, text) { return text; }, bold(text) { return text; } } }, sessionManager: { getSessionId() { return "compact-session"; }, getSessionFile() { return null; }, getEntries() { return []; } }, modelRegistry: { getAvailable() { return []; } } };
+			const ctx = { cwd: process.cwd(), hasUI: true, ui: { setWidget(key, value) { widgets.push([key, value]); }, requestRender() { renders++; }, onTerminalInput() { return () => {}; }, getEditorText() { return ""; }, notify() {}, theme: { fg(_name, text) { return text; }, bg(_name, text) { return text; }, bold(text) { return text; } } }, sessionManager: { getSessionId() { return "compact-session"; }, getSessionFile() { return null; }, getEntries() { return []; } }, modelRegistry: { getAvailable() { return []; } } };
 			registerSubagentExtension(pi);
 			handlers.get("session_start")({}, ctx);
 			sent.length = 0;
-			events.emit("subagent:async-started", { id: "running", pid: 1, sessionId: "compact-session", mode: "single", agent: "builder", asyncDir: path.join(os.tmpdir(), "pi-compaction-test-running") });
-			handlers.get("session_compact")();
+			events.emit("subagent:async-started", { id: "running", pid: 1, sessionId: "compact-session", mode: "single", agent: "worker", asyncDir: path.join(os.tmpdir(), "pi-compaction-test-running") });
+			widgets.length = 0;
+			renders = 0;
+			handlers.get("session_before_compact")({ reason: "threshold", signal: new AbortController().signal });
+			const cleared = widgets.filter(([key, value]) => value === undefined).map(([key]) => key).sort();
+			if (JSON.stringify(cleared) !== JSON.stringify(["subagent-async", "subagent-fleet-status"])) throw new Error(JSON.stringify(widgets));
+			widgets.length = 0;
+			events.emit("subagent:async-complete", { id: "running", sessionId: "compact-session", agent: "worker", success: true, summary: "done" });
+			events.emit("subagent:async-started", { id: "running-2", pid: 2, sessionId: "compact-session", mode: "single", agent: "worker", asyncDir: path.join(os.tmpdir(), "pi-compaction-test-running-2") });
+			if (widgets.length !== 0 || renders !== 0) throw new Error("widgets repainted during compaction");
+			handlers.get("session_compact")({ reason: "threshold" });
+			if (widgets.length !== 0) throw new Error("widgets restored inside compaction hook");
+			handlers.get("agent_start")();
+			const restored = widgets.filter(([_, value]) => value !== undefined).map(([key]) => key).sort();
+			if (JSON.stringify(restored) !== JSON.stringify(["subagent-async", "subagent-fleet-status"])) throw new Error(JSON.stringify(widgets));
 			if (sent.length !== 1 || sent[0].options?.triggerTurn !== true || sent[0].message?.customType !== "subagent-compaction-resume") throw new Error(JSON.stringify(sent));
+
 			sent.length = 0;
-			events.emit("subagent:async-complete", { id: "running", sessionId: "compact-session", agent: "builder", success: true, summary: "done" });
-			handlers.get("session_compact")();
+			events.emit("subagent:async-complete", { id: "running-2", sessionId: "compact-session", agent: "worker", success: true, summary: "done" });
+			handlers.get("session_before_compact")({ reason: "threshold", signal: new AbortController().signal });
+			handlers.get("session_compact")({ reason: "threshold" });
+			handlers.get("agent_settled")();
 			if (sent.some((entry) => entry.message?.customType === "subagent-compaction-resume")) throw new Error("resumed without active work");
+
+			events.emit("subagent:async-started", { id: "running-3", pid: 3, sessionId: "compact-session", mode: "single", agent: "worker", asyncDir: path.join(os.tmpdir(), "pi-compaction-test-running-3") });
+			widgets.length = 0;
+			handlers.get("session_before_compact")({ reason: "overflow", signal: new AbortController().signal });
+			widgets.length = 0;
+			handlers.get("agent_settled")();
+			if (!widgets.some(([_, value]) => value !== undefined)) throw new Error("widgets did not recover after compaction failure");
+
+			widgets.length = 0;
+			handlers.get("session_before_compact")({ reason: "manual", signal: new AbortController().signal });
+			if (widgets.length !== 0) throw new Error("manual compaction changed widget state");
+			await handlers.get("session_shutdown")();
 		`;
 		const env = { ...process.env };
 		delete env.SELESAI_SUBAGENT_CHILD;

package/dist/extensions/pi-subagents/test/unit/completion-dedupe.test.ts

@@ -11,8 +11,8 @@ describe("buildCompletionKey", () => {
 	});
 
 	it("builds deterministic fallback key when id is missing", () => {
-		const first = buildCompletionKey({ agent: "commentator", timestamp: 123, taskIndex: 1, totalTasks: 2, success: true }, "x");
-		const second = buildCompletionKey({ agent: "commentator", timestamp: 123, taskIndex: 1, totalTasks: 2, success: true }, "x");
+		const first = buildCompletionKey({ agent: "reviewer", timestamp: 123, taskIndex: 1, totalTasks: 2, success: true }, "x");
+		const second = buildCompletionKey({ agent: "reviewer", timestamp: 123, taskIndex: 1, totalTasks: 2, success: true }, "x");
 		assert.equal(first, second);
 	});
 });

package/dist/extensions/pi-subagents/test/unit/completion-guard.test.ts

@@ -26,7 +26,7 @@ function assistantText(text: string): Message {
 
 test("implementation task with no mutation triggers the completion guard", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "builder",
+		agent: "worker",
 		task: "Implement the approved fix",
 		messages: [assistantText("Plan: update the files...")],
 	});
@@ -56,7 +56,7 @@ test("declared read-only builtin tools suppress implementation-word false positi
 test("read-only issue drafting tasks do not trigger on suggested fix wording", () => {
 	const task = "Draft GitHub issue for pi-subagents bug from current conversation. Include title, environment/context, reproduction steps, actual/expected, logs excerpt, suspected cause, suggested fix. Terse but complete. No tools needed.";
 	const result = evaluateCompletionMutationGuard({
-		agent: "explorer",
+		agent: "delegate",
 		task,
 		messages: [assistantText("Title: completionGuard false positive\n\nSuggested fix: model read-only intent.")],
 		tools: ["read", "grep", "find", "ls", "bash", "edit", "write", "contact_supervisor"],
@@ -67,9 +67,9 @@ test("read-only issue drafting tasks do not trigger on suggested fix wording", (
 		attemptedMutation: false,
 		triggered: false,
 	});
-	assert.equal(expectsImplementationMutation("builder", task), false);
+	assert.equal(expectsImplementationMutation("worker", task), false);
 	assert.equal(
-		expectsImplementationMutation("builder", "Draft GitHub issue for a bug. Include suspected cause and suggested fix."),
+		expectsImplementationMutation("worker", "Draft GitHub issue for a bug. Include suspected cause and suggested fix."),
 		false,
 	);
 });
@@ -89,9 +89,9 @@ test("omitted, bash, unknown, write, and MCP tool capabilities stay conservative
 	assert.equal(evaluateCompletionMutationGuard({ ...base, tools: ["read", "grep"], mcpDirectTools: ["github/search"] }).triggered, true);
 });
 
-test("builder with mutating-capable tools still triggers when no mutation is observed", () => {
+test("worker with mutating-capable tools still triggers when no mutation is observed", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "builder",
+		agent: "worker",
 		task: "Fix the test implementation",
 		messages: [assistantText("I will edit it next")],
 		tools: ["read", "edit"],
@@ -104,10 +104,10 @@ test("builder with mutating-capable tools still triggers when no mutation is obs
 	});
 });
 
-test("commentator review tasks with bash available do not require mutation", () => {
+test("oracle review tasks with bash available do not require mutation", () => {
 	const task = "Review prep findings and determine what to implement with playbooks instead of before.";
 	const result = evaluateCompletionMutationGuard({
-		agent: "commentator",
+		agent: "oracle",
 		task,
 		messages: [assistantText("Review complete with file-backed findings.")],
 		tools: ["read", "grep", "find", "ls", "bash", "intercom"],
@@ -121,57 +121,57 @@ test("commentator review tasks with bash available do not require mutation", ()
 });
 
 test("review-only, research, and framework output instructions do not expect mutation", () => {
-	assert.equal(expectsImplementationMutation("builder", "Review only: return findings, do not edit"), false);
-	assert.equal(expectsImplementationMutation("builder", "Do not edit files. Tell me how to fix the bug."), false);
-	assert.equal(expectsImplementationMutation("builder", "Review the diff and suggest fixes only. Do not edit files."), false);
-	assert.equal(expectsImplementationMutation("builder", "Implement this. Do not edit files outside this repo. Do not edit files."), false);
-	assert.equal(expectsImplementationMutation("builder", "Investigate why this failed"), false);
+	assert.equal(expectsImplementationMutation("worker", "Review only: return findings, do not edit"), false);
+	assert.equal(expectsImplementationMutation("worker", "Do not edit files. Tell me how to fix the bug."), false);
+	assert.equal(expectsImplementationMutation("worker", "Review the diff and suggest fixes only. Do not edit files."), false);
+	assert.equal(expectsImplementationMutation("worker", "Implement this. Do not edit files outside this repo. Do not edit files."), false);
+	assert.equal(expectsImplementationMutation("worker", "Investigate why this failed"), false);
 	assert.equal(expectsImplementationMutation("researcher", "Research the API behavior"), false);
 	assert.equal(expectsImplementationMutation("researcher", "Research this and patch the bug"), false);
-	assert.equal(expectsImplementationMutation("commentator", "Review this and fix any real issues"), false);
-	assert.equal(expectsImplementationMutation("commentator", "Review this and fix any real issues; regardless of findings, apply changes directly"), true);
-	assert.equal(expectsImplementationMutation("builder", "[Write to: /tmp/result.md]\n\nSummarize findings"), false);
-	assert.equal(expectsImplementationMutation("builder", "Write report"), false);
-	assert.equal(expectsImplementationMutation("builder", "Create a report"), false);
-	assert.equal(expectsImplementationMutation("builder", "Create a summary"), false);
-	assert.equal(expectsImplementationMutation("builder", "Add a report"), false);
-	assert.equal(expectsImplementationMutation("builder", "Update a summary"), false);
-	assert.equal(expectsImplementationMutation("builder", "Write to {chain_dir}"), false);
+	assert.equal(expectsImplementationMutation("reviewer", "Review this and fix any real issues"), false);
+	assert.equal(expectsImplementationMutation("reviewer", "Review this and fix any real issues; regardless of findings, apply changes directly"), true);
+	assert.equal(expectsImplementationMutation("worker", "[Write to: /tmp/result.md]\n\nSummarize findings"), false);
+	assert.equal(expectsImplementationMutation("worker", "Write report"), false);
+	assert.equal(expectsImplementationMutation("worker", "Create a report"), false);
+	assert.equal(expectsImplementationMutation("worker", "Create a summary"), false);
+	assert.equal(expectsImplementationMutation("worker", "Add a report"), false);
+	assert.equal(expectsImplementationMutation("worker", "Update a summary"), false);
+	assert.equal(expectsImplementationMutation("worker", "Write to {chain_dir}"), false);
 	assert.equal(
-		expectsImplementationMutation("builder", "Do async work\nUpdate progress at: /tmp/progress.md\n**Output:**\nWrite your findings to exactly this path: /tmp/out.md\nThis path is authoritative for this run.\nIgnore any other output filename or output path mentioned elsewhere."),
+		expectsImplementationMutation("worker", "Do async work\nUpdate progress at: /tmp/progress.md\n**Output:**\nWrite your findings to exactly this path: /tmp/out.md\nThis path is authoritative for this run.\nIgnore any other output filename or output path mentioned elsewhere."),
 		false,
 	);
 });
 
-test("builder implementation verbs win over investigative wording and scoped prohibitions", () => {
-	assert.equal(expectsImplementationMutation("builder", "Investigate why the builder did not edit files and fix it"), true);
-	assert.equal(expectsImplementationMutation("builder", "Do not modify tests; implement the fix"), true);
-	assert.equal(expectsImplementationMutation("builder", "Do not modify tests — implement the fix"), true);
-	assert.equal(expectsImplementationMutation("builder", "Research the current code path and patch the bug"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix the bug where no edits were made"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix lint"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix the build"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix TypeScript errors"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix CI"), true);
-	assert.equal(expectsImplementationMutation("builder", "Fix the failing test"), true);
-	assert.equal(expectsImplementationMutation("builder", "Patch the cold start test"), true);
-	assert.equal(expectsImplementationMutation("builder", "Implement the fix and return findings."), true);
+test("worker implementation verbs win over investigative wording and scoped prohibitions", () => {
+	assert.equal(expectsImplementationMutation("worker", "Investigate why the worker did not edit files and fix it"), true);
+	assert.equal(expectsImplementationMutation("worker", "Do not modify tests; implement the fix"), true);
+	assert.equal(expectsImplementationMutation("worker", "Do not modify tests — implement the fix"), true);
+	assert.equal(expectsImplementationMutation("worker", "Research the current code path and patch the bug"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix the bug where no edits were made"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix lint"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix the build"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix TypeScript errors"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix CI"), true);
+	assert.equal(expectsImplementationMutation("worker", "Fix the failing test"), true);
+	assert.equal(expectsImplementationMutation("worker", "Patch the cold start test"), true);
+	assert.equal(expectsImplementationMutation("worker", "Implement the fix and return findings."), true);
 });
 
-test("non-builder implementation tasks still expect mutation", () => {
-	assert.equal(expectsImplementationMutation("explorer", "Fix the bug where no edits were made"), true);
-	assert.equal(expectsImplementationMutation("explorer", "Apply the suggested fix to src/runs/shared/completion-guard.ts"), true);
-	assert.equal(expectsImplementationMutation("builder", "Draft a GitHub issue, then implement the fix"), true);
+test("non-worker implementation tasks still expect mutation", () => {
+	assert.equal(expectsImplementationMutation("delegate", "Fix the bug where no edits were made"), true);
+	assert.equal(expectsImplementationMutation("delegate", "Apply the suggested fix to src/runs/shared/completion-guard.ts"), true);
+	assert.equal(expectsImplementationMutation("worker", "Draft a GitHub issue, then implement the fix"), true);
 });
 
-test("builder edit intent covers common docs, config, and source tasks", () => {
-	assert.equal(expectsImplementationMutation("builder", "Update README to mention the native tool"), true);
-	assert.equal(expectsImplementationMutation("builder", "Remove share functionality and all Vercel references"), true);
-	assert.equal(expectsImplementationMutation("builder", "Replace the registered command with a render tool"), true);
-	assert.equal(expectsImplementationMutation("builder", "Create completion-guard.ts"), true);
-	assert.equal(expectsImplementationMutation("builder", "Add tests for the completion guard"), true);
-	assert.equal(expectsImplementationMutation("builder", "Implement the approved fixes. Do not edit files outside this repo."), true);
-	assert.equal(expectsImplementationMutation("builder", "Implement the fix. Do not edit unrelated files."), true);
+test("worker edit intent covers common docs, config, and source tasks", () => {
+	assert.equal(expectsImplementationMutation("worker", "Update README to mention the native tool"), true);
+	assert.equal(expectsImplementationMutation("worker", "Remove share functionality and all Vercel references"), true);
+	assert.equal(expectsImplementationMutation("worker", "Replace the registered command with a render tool"), true);
+	assert.equal(expectsImplementationMutation("worker", "Create completion-guard.ts"), true);
+	assert.equal(expectsImplementationMutation("worker", "Add tests for the completion guard"), true);
+	assert.equal(expectsImplementationMutation("worker", "Implement the approved fixes. Do not edit files outside this repo."), true);
+	assert.equal(expectsImplementationMutation("worker", "Implement the fix. Do not edit unrelated files."), true);
 });
 
 test("edit and write tool calls count as mutation attempts", () => {
@@ -225,7 +225,7 @@ test("read-only and quoted git commands do not count as mutation attempts", () =
 
 test("implementation task with mutation attempts does not trigger", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "builder",
+		agent: "worker",
 		task: "Fix the failing test",
 		messages: [assistantToolCall("edit", { path: "test.ts" })],
 	});
@@ -309,7 +309,7 @@ test("provider checkpoint with a changed commit counts as mutation evidence", ()
 		assert.equal(hasMutationToolCall([message]), true);
 		assert.equal(
 			evaluateCompletionMutationGuard({
-				agent: "builder",
+				agent: "worker",
 				task: "Edit the target source file",
 				messages: [message],
 			}).triggered,
@@ -327,7 +327,7 @@ test("unchanged provider checkpoint does not bypass the completion guard", () =>
 		assert.equal(hasMutationToolCall([message]), false);
 		assert.equal(
 			evaluateCompletionMutationGuard({
-				agent: "builder",
+				agent: "worker",
 				task: "Edit the target source file",
 				messages: [message],
 			}).triggered,
@@ -338,7 +338,7 @@ test("unchanged provider checkpoint does not bypass the completion guard", () =>
 
 test("implementation task with Cursor edit thinking does not trigger", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "builder",
+		agent: "worker",
 		task: "Edit docs/BACKEND_ARCHITECTURE.md",
 		messages: [
 			assistantThinking("Cursor edit: docs/BACKEND_ARCHITECTURE.md added 1 line\n"),