@selesai/code 0.6.3 → 0.8.0

package/dist/extensions/pi-subagents/test/unit/profiles.test.ts

@@ -4,6 +4,7 @@ import * as os from "node:os";
 import * as path from "node:path";
 import { afterEach, beforeEach, describe, it } from "node:test";
 
+import { discoverAgents } from "../../src/agents/agents.ts";
 import {
 	applySubagentProfile,
 	checkSubagentProfile,
@@ -49,26 +50,91 @@ describe("profiles helpers", () => {
 		assert.equal(fs.existsSync(getSubagentProfilesDir()), true);
 	});
 
-	it("applies a saved profile by replacing only settings.subagents", () => {
+	it("applies a saved profile while preserving unrelated subagent settings", () => {
 		const profilesDir = getSubagentProfilesDir();
 		fs.mkdirSync(profilesDir, { recursive: true });
 		fs.writeFileSync(path.join(profilesDir, "openai-codex.quota.json"), JSON.stringify({
 			subagents: {
 				agentOverrides: {
-					scout: { model: "openai-codex/gpt-5.3-codex-spark" },
+					explorer: {
+						model: "openai-codex/gpt-5.3-codex-spark",
+						thinking: "medium",
+						fallbackModels: ["openai-codex/gpt-5.4-mini"],
+					},
+					commentator: { thinking: false, fallbackModels: false },
 				},
 			},
 		}, null, 2));
 		const settingsPath = path.join(homeDir, ".selesai", "agent", "settings.json");
 		fs.mkdirSync(path.dirname(settingsPath), { recursive: true });
-		fs.writeFileSync(settingsPath, JSON.stringify({ defaultModel: "openai/gpt-5", subagents: { agentOverrides: { scout: { model: "old" } } } }, null, 2));
+		fs.writeFileSync(settingsPath, JSON.stringify({
+			defaultModel: "openai/gpt-5",
+			subagents: {
+				disableBuiltins: true,
+				modelScope: { enforce: true, allow: ["openai-codex/*"] },
+				agentOverrides: { explorer: { model: "old" }, stale: { model: "remove-me" } },
+			},
+		}, null, 2));
 
 		const result = applySubagentProfile("openai-codex.quota");
 		const written = JSON.parse(fs.readFileSync(settingsPath, "utf-8"));
 		assert.equal(result.settingsPath, settingsPath);
 		assert.equal(result.filePath.endsWith("openai-codex.quota.json"), true);
 		assert.equal(written.defaultModel, "openai/gpt-5");
-		assert.equal(written.subagents.agentOverrides.scout.model, "openai-codex/gpt-5.3-codex-spark");
+		assert.equal(written.subagents.disableBuiltins, true);
+		assert.deepEqual(written.subagents.modelScope, { enforce: true, allow: ["openai-codex/*"] });
+		assert.deepEqual(written.subagents.agentOverrides, {
+			explorer: {
+				model: "openai-codex/gpt-5.3-codex-spark",
+				thinking: "medium",
+				fallbackModels: ["openai-codex/gpt-5.4-mini"],
+			},
+			commentator: { thinking: false, fallbackModels: false },
+		});
+	});
+
+	it("applies profile models and thinking to user agents without frontmatter pins", () => {
+		const profilesDir = getSubagentProfilesDir();
+		fs.mkdirSync(profilesDir, { recursive: true });
+		fs.writeFileSync(path.join(profilesDir, "azure-fallback.json"), JSON.stringify({
+			subagents: {
+				disableBuiltins: true,
+				agentOverrides: {
+					builder: {
+						model: "bluebox-azure-openai/gpt-5_6-luna",
+						thinking: "high",
+						fallbackModels: ["bluebox-azure-openai/gpt-5_6-terra"],
+					},
+				},
+			},
+		}, null, 2));
+		const agentDir = path.join(homeDir, ".selesai", "agent", "agents");
+		fs.mkdirSync(agentDir, { recursive: true });
+		fs.writeFileSync(path.join(agentDir, "builder.md"), `---\nname: builder\ndescription: Profile-managed builder\n---\n\nDo work.\n`);
+
+		applySubagentProfile("azure-fallback");
+		const agents = discoverAgents(process.cwd(), "both").agents;
+		const builder = agents.find((agent) => agent.name === "builder");
+		assert.equal(builder?.source, "user");
+		assert.equal(builder?.model, "bluebox-azure-openai/gpt-5_6-luna");
+		assert.equal(builder?.thinking, "high");
+		assert.deepEqual(builder?.fallbackModels, ["bluebox-azure-openai/gpt-5_6-terra"]);
+		assert.equal(builder?.override?.scope, "user");
+		assert.equal(agents.some((agent) => agent.source === "builtin"), false);
+	});
+
+	it("rejects invalid profile fallback models", () => {
+		const profilesDir = getSubagentProfilesDir();
+		fs.mkdirSync(profilesDir, { recursive: true });
+		fs.writeFileSync(path.join(profilesDir, "invalid.json"), JSON.stringify({
+			subagents: {
+				agentOverrides: {
+					builder: { fallbackModels: ["openai/gpt-5", 42] },
+				},
+			},
+		}, null, 2));
+
+		assert.throws(() => applySubagentProfile("invalid"), /invalid fallbackModels.*array of strings or false/);
 	});
 
 	it("rejects profile and provider path traversal names", async () => {
@@ -244,8 +310,8 @@ describe("profiles helpers", () => {
 		fs.writeFileSync(path.join(profilesDir, "demo.json"), JSON.stringify({
 			subagents: {
 				agentOverrides: {
-					scout: { model: "openai-codex/gpt-5.3-codex-spark" },
-					worker: { model: "openai-codex/gpt-5.9" },
+					explorer: { model: "openai-codex/gpt-5.3-codex-spark" },
+					builder: { model: "openai-codex/gpt-5.9" },
 				},
 			},
 		}, null, 2));
@@ -262,13 +328,13 @@ describe("profiles helpers", () => {
 		const result = await checkSubagentProfile(pi, ctx as never, "demo");
 		assert.deepEqual(result.results, [
 			{
-				agent: "scout",
+				agent: "explorer",
 				model: "openai-codex/gpt-5.3-codex-spark",
 				inRegistry: true,
 				probe: { status: "ok", message: "OK" },
 			},
 			{
-				agent: "worker",
+				agent: "builder",
 				model: "openai-codex/gpt-5.9",
 				inRegistry: false,
 				probe: { status: "unavailable", message: "model unavailable" },
@@ -280,7 +346,7 @@ describe("profiles helpers", () => {
 		const profilesDir = getSubagentProfilesDir();
 		fs.mkdirSync(profilesDir, { recursive: true });
 		fs.writeFileSync(path.join(profilesDir, "demo.json"), JSON.stringify({
-			subagents: { agentOverrides: { scout: { model: "gpt-5.3-codex-spark:high" } } },
+			subagents: { agentOverrides: { explorer: { model: "gpt-5.3-codex-spark:high" } } },
 		}, null, 2));
 		const probedModels: unknown[] = [];
 		const pi = {

package/dist/extensions/pi-subagents/test/unit/prompt-template-bridge.test.ts

@@ -50,10 +50,10 @@ describe("prompt-template delegation bridge", () => {
 				executeCalls++;
 				onUpdate({
 					details: {
-						results: [{ agent: "worker", model: "openai/gpt-5-mini" }],
+						results: [{ agent: "builder", model: "openai/gpt-5-mini" }],
 						progress: [{
 							index: 0,
-							agent: "worker",
+							agent: "builder",
 							currentTool: "read",
 							currentToolArgs: "src/extension/index.ts",
 							recentOutput: ["line 1"],
@@ -78,7 +78,7 @@ describe("prompt-template delegation bridge", () => {
 
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r1",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -135,7 +135,7 @@ describe("prompt-template delegation bridge", () => {
 		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r-compact-tools",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -162,10 +162,10 @@ describe("prompt-template delegation bridge", () => {
 			execute: async (_requestId, _request, _signal, _ctx, onUpdate) => {
 				onUpdate({
 					details: {
-						results: [{ agent: "worker", model: "openai/gpt-5-mini" }],
+						results: [{ agent: "builder", model: "openai/gpt-5-mini" }],
 						progress: [{
 							index: 0,
-							agent: "worker",
+							agent: "builder",
 							recentOutput: ["line 1", 123 as unknown as string],
 						}],
 					},
@@ -178,7 +178,7 @@ describe("prompt-template delegation bridge", () => {
 		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r-malformed-output",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -210,7 +210,7 @@ describe("prompt-template delegation bridge", () => {
 		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r2",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -239,7 +239,7 @@ describe("prompt-template delegation bridge", () => {
 		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r3",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -270,7 +270,7 @@ describe("prompt-template delegation bridge", () => {
 
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r4",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -301,7 +301,7 @@ describe("prompt-template delegation bridge", () => {
 
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r5",
-			agent: "worker",
+			agent: "builder",
 			task: "do work",
 			context: "fresh",
 			model: "openai/gpt-5",
@@ -318,91 +318,27 @@ describe("prompt-template delegation bridge", () => {
 		bridge.dispose();
 	});
 
-	it("accepts tasks payloads and emits parallelResults", async () => {
+	it("rejects removed tasks and worktree payloads without executor dispatch", async () => {
 		const events = new FakeEvents();
-		let executeTasks: Array<{ agent: string; task: string; model?: string; cwd?: string }> | undefined;
+		let executeCalls = 0;
 		const bridge = registerPromptTemplateDelegationBridge({
 			events,
 			getContext: () => ({ cwd: "/repo" }),
-			execute: async (_requestId, request) => {
-				executeTasks = request.tasks;
-				return {
-					details: {
-						results: [
-							{ agent: "worker-a", messages: [{ role: "assistant", content: [{ type: "text", text: "a" }] }], exitCode: 0 },
-							{ agent: "worker-b", messages: [], exitCode: 1, error: "failed" },
-						],
-					},
-				};
-			},
+			execute: async () => { executeCalls++; return {}; },
 		});
 
-		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
+		const tasksResponse = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
 		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
 			requestId: "r6",
-			tasks: [
-				{ agent: "worker-a", task: "A", model: "openai/gpt-5", cwd: "/repo/a" },
-				{ agent: "worker-b", task: "B", model: "anthropic/claude-sonnet-4-20250514", cwd: "/repo/b" },
-			],
-			context: "fresh",
-			model: "openai/gpt-5",
-			cwd: "/repo",
-		});
-
-		const response = await responsePromise as {
-			isError: boolean;
-			parallelResults?: Array<{ agent: string; isError: boolean; errorText?: string }>;
-		};
-		assert.equal(Array.isArray(executeTasks), true);
-		assert.equal(executeTasks?.length, 2);
-		assert.equal(executeTasks?.[0]?.model, "openai/gpt-5");
-		assert.equal(executeTasks?.[1]?.model, "anthropic/claude-sonnet-4-20250514");
-		assert.equal(executeTasks?.[0]?.cwd, "/repo/a");
-		assert.equal(executeTasks?.[1]?.cwd, "/repo/b");
-		assert.equal(response.isError, false);
-		assert.equal(response.parallelResults?.[0]?.agent, "worker-a");
-		assert.equal(response.parallelResults?.[0]?.isError, false);
-		assert.equal(response.parallelResults?.[1]?.agent, "worker-b");
-		assert.equal(response.parallelResults?.[1]?.isError, true);
-		assert.equal(response.parallelResults?.[1]?.errorText, "failed");
-
-		bridge.dispose();
-	});
-
-	it("marks missing parallel task results as errors", async () => {
-		const events = new FakeEvents();
-		const bridge = registerPromptTemplateDelegationBridge({
-			events,
-			getContext: () => ({ cwd: "/repo" }),
-			execute: async () => ({
-				details: {
-					results: [{ agent: "worker-a", messages: [{ role: "assistant", content: [{ type: "text", text: "ok" }] }], exitCode: 0 }],
-				},
-			}),
-		});
-
-		const responsePromise = once(events, PROMPT_TEMPLATE_SUBAGENT_RESPONSE_EVENT);
-		events.emit(PROMPT_TEMPLATE_SUBAGENT_REQUEST_EVENT, {
-			requestId: "r7",
-			tasks: [
-				{ agent: "worker-a", task: "A" },
-				{ agent: "worker-b", task: "B" },
-			],
+			tasks: [{ agent: "builder-a", task: "A" }],
 			context: "fresh",
 			model: "openai/gpt-5",
 			cwd: "/repo",
 		});
-
-		const response = await responsePromise as {
-			isError: boolean;
-			parallelResults?: Array<{ agent: string; isError: boolean; errorText?: string }>;
-		};
-		assert.equal(response.isError, false);
-		assert.equal(response.parallelResults?.[0]?.isError, false);
-		assert.equal(response.parallelResults?.[1]?.agent, "worker-b");
-		assert.equal(response.parallelResults?.[1]?.isError, true);
-		assert.match(response.parallelResults?.[1]?.errorText ?? "", /missing result/i);
-
+		const response = await tasksResponse as { isError: boolean; errorText?: string };
+		assert.equal(response.isError, true);
+		assert.match(response.errorText ?? "", /removed.*workflowScript/i);
+		assert.equal(executeCalls, 0);
 		bridge.dispose();
 	});
 });

package/dist/extensions/pi-subagents/test/unit/prompt-workflows.test.ts

@@ -48,13 +48,13 @@ describe("prompt workflows", () => {
 	it("discovers project workflows over user workflows", () => {
 		writePrompt(path.join(agentDir, "prompts"), "native-test", `---
 description: User version
-subagent: reviewer
+subagent: commentator
 ---
 User body
 `);
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-test", `---
 description: Project version
-subagent: worker
+subagent: builder
 model: openai/gpt-5-mini
 ---
 Project body $1
@@ -63,14 +63,14 @@ Project body $1
 		const workflow = discoverPromptWorkflows(cwd).find((entry) => entry.name === "native-test");
 
 		assert.equal(workflow?.description, "Project version");
-		assert.equal(workflow?.agent, "worker");
+		assert.equal(workflow?.agent, "builder");
 		assert.equal(workflow?.model, "openai/gpt-5-mini");
 	});
 
 	it("runs a named workflow through native subagent execution", async () => {
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-run", `---
 description: Run native prompt
-subagent: reviewer
+subagent: commentator
 model: anthropic/claude-sonnet-4
 skill: deslop,typescript-code
 ---
@@ -87,28 +87,28 @@ Review $1 with $ARGUMENTS
 			run: async (params) => { runs.push(params); },
 		});
 
-		await commands.get("prompt-workflow")!.handler('native-run target --fork --worktree', makeCtx(cwd));
+		await commands.get("prompt-workflow")!.handler('native-run target --fork', makeCtx(cwd));
 
 		assert.equal(sent.length, 0);
 		assert.equal(runs.length, 1);
-		assert.equal(runs[0]?.agent, "reviewer");
+		assert.equal(runs[0]?.agent, "commentator");
 		assert.equal(runs[0]?.model, "anthropic/claude-sonnet-4");
 		assert.deepEqual(runs[0]?.skill, ["deslop", "typescript-code"]);
 		assert.equal(runs[0]?.context, "fork");
-		assert.equal(runs[0]?.worktree, true);
 		assert.equal(runs[0]?.task, "Review target with target");
 	});
 
-	it("runs prompt templates as a native chain", async () => {
+	it("runs declared prompt sequences through workflowScript", async () => {
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-analyze", `---
 description: Analyze
-subagent: scout
+subagent: explorer
+chain: native-analyze -> native-fix
 ---
 Analyze $@
 `);
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-fix", `---
 description: Fix
-subagent: worker
+subagent: builder
 ---
 Fix from {previous}: $@
 `);
@@ -122,13 +122,12 @@ Fix from {previous}: $@
 			run: async (params) => { runs.push(params); },
 		});
 
-		await commands.get("chain-prompts")!.handler("native-analyze -> native-fix -- bug report", makeCtx(cwd));
+		await commands.get("prompt-workflow")!.handler("native-analyze bug report", makeCtx(cwd));
 
 		assert.equal(runs.length, 1);
-		assert.equal(runs[0]?.chain?.length, 2);
-		assert.equal(runs[0]?.chain?.[0]?.agent, "scout");
-		assert.equal(runs[0]?.chain?.[0]?.task, "Analyze bug report");
-		assert.equal(runs[0]?.chain?.[1]?.agent, "worker");
-		assert.equal(runs[0]?.chain?.[1]?.task, "Fix from {previous}: bug report");
+		assert.match(runs[0]?.workflowScript ?? "", /runs\.run\("prompt-1-native-analyze"/);
+		assert.match(runs[0]?.workflowScript ?? "", /runs\.run\("prompt-2-native-fix"/);
+		assert.match(runs[0]?.workflowScript ?? "", /replaceAll\("\{previous\}"/);
+		assert.equal(commands.has("chain-prompts"), false);
 	});
 });

package/dist/extensions/pi-subagents/test/unit/render-helpers.test.ts

@@ -66,15 +66,15 @@ test("compact chain rendering uses workflow graph spans for dynamic fanout resul
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["scout", "expand:reviewer", "writer"],
+			chainAgents: ["explorer", "expand:commentator", "writer"],
 			totalSteps: 3,
-			results: [result("scout", "targets"), result("reviewer", "a"), result("reviewer", "b"), result("writer", "final")],
+			results: [result("explorer", "targets"), result("commentator", "a"), result("commentator", "b"), result("writer", "final")],
 			workflowGraph: {
 				runId: "render-dynamic",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -82,8 +82,8 @@ test("compact chain rendering uses workflow graph spans for dynamic fanout resul
 						status: "completed",
 						stepIndex: 1,
 						children: [
-							{ id: "step-1-item-a", kind: "agent", agent: "reviewer", label: "Review A", status: "completed", flatIndex: 1, stepIndex: 1 },
-							{ id: "step-1-item-b", kind: "agent", agent: "reviewer", label: "Review B", status: "completed", flatIndex: 2, stepIndex: 1 },
+							{ id: "step-1-item-a", kind: "agent", agent: "commentator", label: "Review A", status: "completed", flatIndex: 1, stepIndex: 1 },
+							{ id: "step-1-item-b", kind: "agent", agent: "commentator", label: "Review B", status: "completed", flatIndex: 2, stepIndex: 1 },
 						],
 						dynamic: { sourceOutput: "targets", sourcePath: "/items", itemName: "target", collectAs: "reviews" },
 					},
@@ -94,9 +94,9 @@ test("compact chain rendering uses workflow graph spans for dynamic fanout resul
 	}, { expanded: false }, theme as any);
 
 	const text = componentText(component);
-	assert.match(text, /Step 1: scout/);
-	assert.match(text, /Agent 1\/2: reviewer/);
-	assert.match(text, /Agent 2\/2: reviewer/);
+	assert.match(text, /Step 1: explorer/);
+	assert.match(text, /Agent 1\/2: commentator/);
+	assert.match(text, /Agent 2\/2: commentator/);
 	assert.match(text, /Step 3: writer/);
 });
 
@@ -105,15 +105,15 @@ test("compact chain rendering shows failed zero-child dynamic fanout groups", ()
 		content: [{ type: "text", text: "failed" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["scout", "expand:reviewer", "writer"],
+			chainAgents: ["explorer", "expand:commentator", "writer"],
 			totalSteps: 3,
-			results: [result("scout", "targets")],
+			results: [result("explorer", "targets")],
 			workflowGraph: {
 				runId: "render-empty-dynamic-failed",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -133,7 +133,7 @@ test("compact chain rendering shows failed zero-child dynamic fanout groups", ()
 	const text = componentText(component);
 	assert.match(text, /step 1\/3/);
 	assert.doesNotMatch(text, /step 3\/3/);
-	assert.match(text, /Step 1: scout/);
+	assert.match(text, /Step 1: explorer/);
 	assert.match(text, /Step 2: Review targets .* failed/);
 	assert.match(text, /No review targets materialized/);
 	assert.match(text, /Step 3: writer .* pending/);
@@ -144,15 +144,15 @@ test("expanded chain rendering uses workflow graph spans for dynamic fanout resu
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["scout", "expand:reviewer", "writer"],
+			chainAgents: ["explorer", "expand:commentator", "writer"],
 			totalSteps: 3,
-			results: [result("scout", "targets"), result("reviewer", "a"), result("reviewer", "b"), result("writer", "final")],
+			results: [result("explorer", "targets"), result("commentator", "a"), result("commentator", "b"), result("writer", "final")],
 			workflowGraph: {
 				runId: "render-dynamic-expanded",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -160,8 +160,8 @@ test("expanded chain rendering uses workflow graph spans for dynamic fanout resu
 						status: "completed",
 						stepIndex: 1,
 						children: [
-							{ id: "step-1-item-a", kind: "agent", agent: "reviewer", label: "Review A", status: "completed", flatIndex: 1, stepIndex: 1 },
-							{ id: "step-1-item-b", kind: "agent", agent: "reviewer", label: "Review B", status: "completed", flatIndex: 2, stepIndex: 1 },
+							{ id: "step-1-item-a", kind: "agent", agent: "commentator", label: "Review A", status: "completed", flatIndex: 1, stepIndex: 1 },
+							{ id: "step-1-item-b", kind: "agent", agent: "commentator", label: "Review B", status: "completed", flatIndex: 2, stepIndex: 1 },
 						],
 						dynamic: { sourceOutput: "targets", sourcePath: "/items", itemName: "target", collectAs: "reviews" },
 					},
@@ -172,9 +172,9 @@ test("expanded chain rendering uses workflow graph spans for dynamic fanout resu
 	}, { expanded: true }, theme as any);
 
 	const text = componentText(component);
-	assert.match(text, /Step 1: scout/);
-	assert.match(text, /Agent 1\/2: reviewer/);
-	assert.match(text, /Agent 2\/2: reviewer/);
+	assert.match(text, /Step 1: explorer/);
+	assert.match(text, /Agent 1\/2: commentator/);
+	assert.match(text, /Agent 2\/2: commentator/);
 	assert.match(text, /Step 3: writer/);
 });
 
@@ -183,7 +183,7 @@ test("compact multi-result rendering shows total cost in the header", () => {
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "parallel",
-			results: [result("scout", "a"), result("reviewer", "b")],
+			results: [result("explorer", "a"), result("commentator", "b")],
 			totalCost: { inputTokens: 30, outputTokens: 12, costUsd: 0.04 },
 		},
 	}, { expanded: false }, theme as any));
@@ -197,25 +197,25 @@ test("static sequential and static parallel chain rendering keep existing labels
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["scout", "writer"],
+			chainAgents: ["explorer", "writer"],
 			totalSteps: 2,
-			results: [result("scout", "a"), result("writer", "b")],
+			results: [result("explorer", "a"), result("writer", "b")],
 		},
 	}, { expanded: false }, theme as any));
-	assert.match(sequential, /Step 1: scout/);
+	assert.match(sequential, /Step 1: explorer/);
 	assert.match(sequential, /Step 2: writer/);
 
 	const parallel = componentText(renderSubagentResult({
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["scout", "[reviewer+auditor]", "writer"],
+			chainAgents: ["explorer", "[commentator+auditor]", "writer"],
 			totalSteps: 3,
-			results: [result("scout", "a"), result("reviewer", "b"), result("auditor", "c"), result("writer", "d")],
+			results: [result("explorer", "a"), result("commentator", "b"), result("auditor", "c"), result("writer", "d")],
 		},
 	}, { expanded: false }, theme as any));
-	assert.match(parallel, /Step 1: scout/);
-	assert.match(parallel, /Agent 1\/2: reviewer/);
+	assert.match(parallel, /Step 1: explorer/);
+	assert.match(parallel, /Agent 1\/2: commentator/);
 	assert.match(parallel, /Agent 2\/2: auditor/);
 	assert.match(parallel, /Step 3: writer/);
 });