@selesai/code 0.8.2 → 0.8.4

package/dist/extensions/pi-subagents/test/unit/prompt-workflows.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 { discoverPromptWorkflows, registerPromptWorkflowCommands } from "../../src/slash/prompt-workflows.ts";
+import { renderWorkflowPrompt } from "../../src/shared/prompt-resources.ts";
 import type { SubagentParamsLike } from "../../src/runs/foreground/subagent-executor.ts";
 
 const originalAgentDir = process.env.SELESAI_CODING_AGENT_DIR;
@@ -48,13 +49,13 @@ describe("prompt workflows", () => {
 	it("discovers project workflows over user workflows", () => {
 		writePrompt(path.join(agentDir, "prompts"), "native-test", `---
 description: User version
-subagent: commentator
+subagent: reviewer
 ---
 User body
 `);
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-test", `---
 description: Project version
-subagent: builder
+subagent: worker
 model: openai/gpt-5-mini
 ---
 Project body $1
@@ -63,14 +64,36 @@ Project body $1
 		const workflow = discoverPromptWorkflows(cwd).find((entry) => entry.name === "native-test");
 
 		assert.equal(workflow?.description, "Project version");
-		assert.equal(workflow?.agent, "builder");
+		assert.equal(workflow?.agent, "worker");
 		assert.equal(workflow?.model, "openai/gpt-5-mini");
 	});
 
+	it("renders explicitly scoped prompt fragments with simple variables", () => {
+		writePrompt(path.join(agentDir, "prompts"), "writer-followup", `---
+description: User follow-up
+---
+User pass {{pass}}: {{previous}} {{missing}}
+`);
+		writePrompt(path.join(cwd, ".selesai", "prompts"), "writer-followup", "Project pass {{pass}}: {{previous}}");
+
+		assert.equal(renderWorkflowPrompt("user:writer-followup", { pass: 1, previous: "draft" }, cwd), "User pass 1: draft {{missing}}");
+		assert.equal(renderWorkflowPrompt("project:writer-followup", { pass: 2, previous: "revision" }, cwd), "Project pass 2: revision");
+		assert.match(renderWorkflowPrompt("package:review-loop", undefined, cwd), /^Run a parent-orchestrated review loop/);
+		assert.throws(() => renderWorkflowPrompt("writer-followup", {}, cwd), /package:<name>.*user:<name>.*project:<name>/);
+		assert.throws(() => renderWorkflowPrompt("project:../secret", {}, cwd), /ref must use/);
+		try {
+			fs.symlinkSync(path.join(agentDir, "prompts", "writer-followup.md"), path.join(cwd, ".selesai", "prompts", "linked.md"));
+			assert.throws(() => renderWorkflowPrompt("project:linked", {}, cwd), /not a regular file/);
+		} catch (error) {
+			if (!["EPERM", "EACCES", "ENOSYS"].includes((error as NodeJS.ErrnoException).code ?? "")) throw error;
+		}
+		assert.throws(() => renderWorkflowPrompt("project:writer-followup", { previous: { nested: true } }, cwd), /string, number, or boolean/);
+	});
+
 	it("runs a named workflow through native subagent execution", async () => {
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-run", `---
 description: Run native prompt
-subagent: commentator
+subagent: reviewer
 model: anthropic/claude-sonnet-4
 skill: deslop,typescript-code
 ---
@@ -91,24 +114,35 @@ Review $1 with $ARGUMENTS
 
 		assert.equal(sent.length, 0);
 		assert.equal(runs.length, 1);
-		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]?.task, "Review target with target");
+		const params = runs[0];
+		assert.equal(params?.agent, undefined);
+		assert.equal(params?.task, undefined);
+		assert.equal(params?.clarify, undefined);
+		assert.equal(params?.model, undefined);
+		assert.equal(params?.skill, undefined);
+		assert.equal(params?.context, undefined);
+		assert.equal(params?.agentScope, "both");
+		assert.equal(params?.async, false);
+		const script = params?.workflowScript ?? "";
+		assert.match(script, /runs\.run\("prompt-1-native-run"/);
+		assert.match(script, /"agent":"reviewer"/);
+		assert.match(script, /"model":"anthropic\/claude-sonnet-4"/);
+		assert.match(script, /"skill":\["deslop","typescript-code"\]/);
+		assert.match(script, /"context":"fork"/);
+		assert.match(script, /Review target with target/);
 	});
 
 	it("runs declared prompt sequences through workflowScript", async () => {
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-analyze", `---
 description: Analyze
-subagent: explorer
+subagent: scout
 chain: native-analyze -> native-fix
 ---
 Analyze $@
 `);
 		writePrompt(path.join(cwd, ".selesai", "prompts"), "native-fix", `---
 description: Fix
-subagent: builder
+subagent: worker
 ---
 Fix from {previous}: $@
 `);
@@ -125,9 +159,18 @@ Fix from {previous}: $@
 		await commands.get("prompt-workflow")!.handler("native-analyze bug report", makeCtx(cwd));
 
 		assert.equal(runs.length, 1);
-		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);
+		const params = runs[0];
+		assert.equal(params?.agent, undefined);
+		assert.equal(params?.task, undefined);
+		assert.equal(params?.clarify, undefined);
+		assert.equal(params?.agentScope, "both");
+		assert.equal(params?.async, false);
+		assert.match(params?.workflowScript ?? "", /runs\.run\("prompt-1-native-analyze"/);
+		assert.match(params?.workflowScript ?? "", /runs\.run\("prompt-2-native-fix"/);
+		assert.match(params?.workflowScript ?? "", /replaceAll\("\{previous\}"/);
+		assert.equal(commands.has("chain-prompts"), true);
+		assert.equal(commands.has("chain"), true);
+		assert.equal(commands.has("parallel"), true);
+		assert.equal(commands.has("run-chain"), true);
 	});
 });

package/dist/extensions/pi-subagents/test/unit/public-execution.test.ts

@@ -0,0 +1,40 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import { normalizePublicSubagentExecution } from "../../src/extension/public-execution.ts";
+
+describe("public subagent execution normalization", () => {
+	it("accepts all four execution modes, management, and direct-field schedules", () => {
+		assert.deepEqual(normalizePublicSubagentExecution({ workflowScript: "return 1" }), { ok: true, params: { workflowScript: "return 1" } });
+		assert.deepEqual(normalizePublicSubagentExecution({ agent: "worker", task: "work" }), { ok: true, params: { agent: "worker", task: "work" } });
+		assert.deepEqual(normalizePublicSubagentExecution({ tasks: [{ agent: "worker" }] }), { ok: true, params: { tasks: [{ agent: "worker" }] } });
+		assert.deepEqual(normalizePublicSubagentExecution({ chain: [{ agent: "worker" }] }), { ok: true, params: { chain: [{ agent: "worker" }] } });
+		assert.deepEqual(normalizePublicSubagentExecution({ action: " list " }), { ok: true, params: { action: "list" } });
+		assert.deepEqual(
+			normalizePublicSubagentExecution({ action: " schedule.create ", every: "1h", workflowScript: "return 1" }),
+			{ ok: true, params: { action: "schedule.create", every: "1h", workflowScript: "return 1" } },
+		);
+		assert.deepEqual(
+			normalizePublicSubagentExecution({ action: "schedule.create", agent: "worker", task: "work" }),
+			{ ok: true, params: { action: "schedule.create", agent: "worker", task: "work" } },
+		);
+	});
+
+	it("rejects malformed public execution shapes", () => {
+		for (const params of [
+			{ action: " " },
+			{ action: "single" },
+			{ action: "parallel" },
+			{ action: "chain" },
+			{ parallel: [{ agent: "worker" }] },
+			{ concurrency: 2 },
+			{ clarify: true, workflowScript: "return 1" },
+			{ resume: "retained-run", workflowScript: "return 1" },
+			{},
+			{ workflowScript: " " },
+			{ action: "status", workflowScript: "return 1" },
+			{ action: "schedule.create", every: "1h" },
+		] as const) {
+			assert.equal(normalizePublicSubagentExecution(params).ok, false, JSON.stringify(params));
+		}
+	});
+});

package/dist/extensions/pi-subagents/test/unit/reads-resolution.test.ts

@@ -0,0 +1,66 @@
+import { describe, test, before, after } from "node:test";
+import * as assert from "node:assert";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import { expandHomePath, resolveChainPath } from "../../src/shared/settings.ts";
+
+const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-reads-resolution-"));
+const homeDir = path.join(tmpDir, "home");
+const chainDir = path.join(tmpDir, "chain");
+const originalHome = process.env.HOME;
+const originalUserProfile = process.env.USERPROFILE;
+
+before(() => {
+	process.env.HOME = homeDir;
+	process.env.USERPROFILE = homeDir;
+	assert.equal(os.homedir(), homeDir);
+});
+
+after(() => {
+	if (originalHome === undefined) delete process.env.HOME;
+	else process.env.HOME = originalHome;
+	if (originalUserProfile === undefined) delete process.env.USERPROFILE;
+	else process.env.USERPROFILE = originalUserProfile;
+});
+
+describe("reads path resolution", () => {
+	describe("expandHomePath", () => {
+		test("expands bare ~ to the home directory", () => {
+			assert.equal(expandHomePath("~"), homeDir);
+		});
+
+		test("expands ~/ to the home directory", () => {
+			assert.equal(expandHomePath("~/"), homeDir);
+		});
+
+		test("expands ~/file to <home>/file", () => {
+			assert.equal(expandHomePath("~/.zprofile"), path.join(homeDir, ".zprofile"));
+		});
+
+		test("leaves absolute paths unchanged", () => {
+			assert.equal(expandHomePath("/etc/hosts"), "/etc/hosts");
+		});
+
+		test("leaves repository-relative paths unchanged", () => {
+			assert.equal(expandHomePath("docs/x.md"), "docs/x.md");
+		});
+	});
+
+	describe("resolveChainPath", () => {
+		test("expands ~ before resolving", () => {
+			assert.equal(
+				resolveChainPath("~/.zprofile", chainDir),
+				path.join(homeDir, ".zprofile"),
+			);
+		});
+
+		test("passes absolute paths through", () => {
+			assert.equal(resolveChainPath("/etc/hosts", chainDir), "/etc/hosts");
+		});
+
+		test("prepends chainDir to repository-relative paths", () => {
+			assert.equal(resolveChainPath("docs/x.md", chainDir), path.join(chainDir, "docs/x.md"));
+		});
+	});
+});

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

@@ -61,20 +61,39 @@ test("truncLine respects grapheme display width", () => {
 	assert.equal(visibleWidth(rendered), 5);
 });
 
+test("truncLine emits no marker at zero width", () => {
+	assert.equal(truncLine("abcdef", 0), "");
+});
+
+test("multiline rendering omits two-column graphemes at one-column width", () => {
+	const originalColumns = Object.getOwnPropertyDescriptor(process.stdout, "columns");
+	Object.defineProperty(process.stdout, "columns", { configurable: true, value: 5 });
+	try {
+		const rendered = componentText(renderSubagentResult({
+			content: [{ type: "text", text: "a🙂b\n🙂" }],
+		}, { expanded: true }, theme as any));
+		assert.deepEqual(rendered.split("\n"), ["a", "b"]);
+		for (const line of rendered.split("\n")) assert.ok(visibleWidth(line) <= 1);
+	} finally {
+		if (originalColumns) Object.defineProperty(process.stdout, "columns", originalColumns);
+		else delete (process.stdout as { columns?: number }).columns;
+	}
+});
+
 test("compact chain rendering uses workflow graph spans for dynamic fanout results", () => {
 	const component = renderSubagentResult({
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["explorer", "expand:commentator", "writer"],
+			chainAgents: ["scout", "expand:reviewer", "writer"],
 			totalSteps: 3,
-			results: [result("explorer", "targets"), result("commentator", "a"), result("commentator", "b"), result("writer", "final")],
+			results: [result("scout", "targets"), result("reviewer", "a"), result("reviewer", "b"), result("writer", "final")],
 			workflowGraph: {
 				runId: "render-dynamic",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -82,8 +101,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: "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 },
+							{ 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 },
 						],
 						dynamic: { sourceOutput: "targets", sourcePath: "/items", itemName: "target", collectAs: "reviews" },
 					},
@@ -94,9 +113,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: explorer/);
-	assert.match(text, /Agent 1\/2: commentator/);
-	assert.match(text, /Agent 2\/2: commentator/);
+	assert.match(text, /Step 1: scout/);
+	assert.match(text, /Agent 1\/2: reviewer/);
+	assert.match(text, /Agent 2\/2: reviewer/);
 	assert.match(text, /Step 3: writer/);
 });
 
@@ -105,15 +124,15 @@ test("compact chain rendering shows failed zero-child dynamic fanout groups", ()
 		content: [{ type: "text", text: "failed" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["explorer", "expand:commentator", "writer"],
+			chainAgents: ["scout", "expand:reviewer", "writer"],
 			totalSteps: 3,
-			results: [result("explorer", "targets")],
+			results: [result("scout", "targets")],
 			workflowGraph: {
 				runId: "render-empty-dynamic-failed",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -133,7 +152,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: explorer/);
+	assert.match(text, /Step 1: scout/);
 	assert.match(text, /Step 2: Review targets .* failed/);
 	assert.match(text, /No review targets materialized/);
 	assert.match(text, /Step 3: writer .* pending/);
@@ -144,15 +163,15 @@ test("expanded chain rendering uses workflow graph spans for dynamic fanout resu
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["explorer", "expand:commentator", "writer"],
+			chainAgents: ["scout", "expand:reviewer", "writer"],
 			totalSteps: 3,
-			results: [result("explorer", "targets"), result("commentator", "a"), result("commentator", "b"), result("writer", "final")],
+			results: [result("scout", "targets"), result("reviewer", "a"), result("reviewer", "b"), result("writer", "final")],
 			workflowGraph: {
 				runId: "render-dynamic-expanded",
 				mode: "chain",
 				phases: [],
 				nodes: [
-					{ id: "step-0", kind: "step", agent: "explorer", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
+					{ id: "step-0", kind: "step", agent: "scout", label: "Scout", status: "completed", flatIndex: 0, stepIndex: 0 },
 					{
 						id: "step-1",
 						kind: "dynamic-parallel-group",
@@ -160,8 +179,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: "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 },
+							{ 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 },
 						],
 						dynamic: { sourceOutput: "targets", sourcePath: "/items", itemName: "target", collectAs: "reviews" },
 					},
@@ -172,9 +191,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: explorer/);
-	assert.match(text, /Agent 1\/2: commentator/);
-	assert.match(text, /Agent 2\/2: commentator/);
+	assert.match(text, /Step 1: scout/);
+	assert.match(text, /Agent 1\/2: reviewer/);
+	assert.match(text, /Agent 2\/2: reviewer/);
 	assert.match(text, /Step 3: writer/);
 });
 
@@ -183,7 +202,7 @@ test("compact multi-result rendering shows total cost in the header", () => {
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "parallel",
-			results: [result("explorer", "a"), result("commentator", "b")],
+			results: [result("scout", "a"), result("reviewer", "b")],
 			totalCost: { inputTokens: 30, outputTokens: 12, costUsd: 0.04 },
 		},
 	}, { expanded: false }, theme as any));
@@ -197,25 +216,25 @@ test("static sequential and static parallel chain rendering keep existing labels
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["explorer", "writer"],
+			chainAgents: ["scout", "writer"],
 			totalSteps: 2,
-			results: [result("explorer", "a"), result("writer", "b")],
+			results: [result("scout", "a"), result("writer", "b")],
 		},
 	}, { expanded: false }, theme as any));
-	assert.match(sequential, /Step 1: explorer/);
+	assert.match(sequential, /Step 1: scout/);
 	assert.match(sequential, /Step 2: writer/);
 
 	const parallel = componentText(renderSubagentResult({
 		content: [{ type: "text", text: "done" }],
 		details: {
 			mode: "chain",
-			chainAgents: ["explorer", "[commentator+auditor]", "writer"],
+			chainAgents: ["scout", "[reviewer+auditor]", "writer"],
 			totalSteps: 3,
-			results: [result("explorer", "a"), result("commentator", "b"), result("auditor", "c"), result("writer", "d")],
+			results: [result("scout", "a"), result("reviewer", "b"), result("auditor", "c"), result("writer", "d")],
 		},
 	}, { expanded: false }, theme as any));
-	assert.match(parallel, /Step 1: explorer/);
-	assert.match(parallel, /Agent 1\/2: commentator/);
+	assert.match(parallel, /Step 1: scout/);
+	assert.match(parallel, /Agent 1\/2: reviewer/);
 	assert.match(parallel, /Agent 2\/2: auditor/);
 	assert.match(parallel, /Step 3: writer/);
 });

package/dist/extensions/pi-subagents/test/unit/result-intercom.test.ts

@@ -21,15 +21,15 @@ describe("result intercom formatter", () => {
 			chainSteps: 4,
 			children: [
 				{
-					agent: "commentator-a",
+					agent: "reviewer-a",
 					status: "completed",
 					summary: "Completed checks",
 					artifactPath: "/tmp/a.md",
 					sessionPath: "/tmp/a-session.jsonl",
-					intercomTarget: "subagent-commentator-a-run-123-1",
+					intercomTarget: "subagent-reviewer-a-run-123-1",
 				},
 				{
-					agent: "commentator-b",
+					agent: "reviewer-b",
 					status: "failed",
 					summary: "Failed checks",
 					artifactPath: "/tmp/b.md",
@@ -48,9 +48,9 @@ describe("result intercom formatter", () => {
 		assert.match(payload.message, /Outputs: 2 unknown \(semantic adequacy unassessed\)/);
 		assert.match(payload.message, /Chain steps: 4/);
 		assert.match(payload.message, /Intercom targets below identify child sessions used while they were running/);
-		assert.match(payload.message, /1\. commentator-a — process completed · output unknown/);
-		assert.match(payload.message, /Run intercom target: subagent-commentator-a-run-123-1/);
-		assert.match(payload.message, /2\. commentator-b — process failed · output unknown/);
+		assert.match(payload.message, /1\. reviewer-a — process completed · output unknown/);
+		assert.match(payload.message, /Run intercom target: subagent-reviewer-a-run-123-1/);
+		assert.match(payload.message, /2\. reviewer-b — process failed · output unknown/);
 		assert.match(payload.message, /Output artifact: \/tmp\/a\.md/);
 		assert.match(payload.message, /Session: \/tmp\/a-session\.jsonl/);
 	});
@@ -66,7 +66,7 @@ describe("result intercom formatter", () => {
 				mode: "single",
 				source: "async",
 				asyncId: "run-single",
-				children: [{ agent: "builder", status: "completed", summary: "done", sessionPath }],
+				children: [{ agent: "worker", status: "completed", summary: "done", sessionPath }],
 			});
 
 			assert.match(payload.message, /Revive: subagent\(\{ action: "resume", id: "run-single", message: "\.\.\." \}\)/);
@@ -109,7 +109,7 @@ describe("result intercom formatter", () => {
 			mode: "single",
 			source: "async",
 			asyncId: "run-missing-session",
-			children: [{ agent: "builder", status: "failed", summary: "failed", sessionPath: path.join(os.tmpdir(), "missing-pi-session.jsonl") }],
+			children: [{ agent: "worker", status: "failed", summary: "failed", sessionPath: path.join(os.tmpdir(), "missing-pi-session.jsonl") }],
 		});
 
 		assert.match(payload.message, /Resume: unavailable; no child session file was persisted/);
@@ -132,7 +132,7 @@ describe("result intercom formatter", () => {
 				depth: 1,
 				path: [{ runId: "root-run", stepIndex: 1 }],
 				state: "complete",
-				agent: "commentator",
+				agent: "reviewer",
 				model: "provider/gpt-5.6-luna:medium",
 				thinking: "medium",
 				sessionFile: path.join(os.tmpdir(), "nested-a.jsonl"),
@@ -166,7 +166,7 @@ describe("result intercom formatter", () => {
 		assert.equal(Object.hasOwn(grandchild ?? {}, "controlInbox"), false);
 		assert.equal(Object.hasOwn(grandchild ?? {}, "capabilityToken"), false);
 		assert.match(payload.message, /Nested subagents:/);
-		assert.match(payload.message, /↳ commentator — complete \[nested-a\]/);
+		assert.match(payload.message, /↳ reviewer — complete \[nested-a\]/);
 	});
 
 	it("separates process failure from output presence and gives salvage guidance", () => {
@@ -197,7 +197,7 @@ describe("result intercom formatter", () => {
 			runId: "run-bound",
 			mode: "single",
 			source: "foreground",
-			children: [{ agent: "builder", status: "completed", summary: longSummary }],
+			children: [{ agent: "worker", status: "completed", summary: longSummary }],
 		});
 		assert.equal(payload.children[0]!.summary, longSummary);
 		assert.match(payload.message, new RegExp(`${"x".repeat(2000)}\\n${"y".repeat(2000)}`));
@@ -232,7 +232,7 @@ describe("result intercom formatter", () => {
 		const stripped = stripDetailsOutputsForIntercomReceipt({
 			mode: "single",
 			results: [{
-				agent: "builder",
+				agent: "worker",
 				task: "Task",
 				exitCode: 0,
 				messages: [{ role: "assistant", content: [{ type: "text", text: "full" }] } as never],

package/dist/extensions/pi-subagents/test/unit/retained-children.test.ts

@@ -0,0 +1,57 @@
+import assert from "node:assert/strict";
+import * as fs from "node:fs";
+import * as os from "node:os";
+import * as path from "node:path";
+import { describe, it } from "node:test";
+import { listRetainedChildren } from "../../src/runs/background/retained-children.ts";
+
+function writeRetainedRun(root: string, index: number, sessionId = "parent-a", parentWorkflowRunId: string | null = "workflow-a"): void {
+	const runId = `child-${index}`;
+	const sessionFile = path.join(root, "sessions", `${runId}.jsonl`);
+	fs.mkdirSync(path.dirname(sessionFile), { recursive: true });
+	fs.writeFileSync(sessionFile, "", "utf-8");
+	const asyncDir = path.join(root, "runs", runId);
+	fs.mkdirSync(asyncDir, { recursive: true });
+	fs.writeFileSync(path.join(asyncDir, "status.json"), JSON.stringify({
+		runId,
+		sessionId,
+		mode: "single",
+		state: "complete",
+		startedAt: index,
+		endedAt: 1_000 + index,
+		lastUpdate: 1_000 + index,
+		...(parentWorkflowRunId ? { parentWorkflowRunId } : {}),
+		steps: [{
+			agent: "worker",
+			status: "complete",
+			description: `  Task ${index}  with   spacing ${"x".repeat(140)}  `,
+			endedAt: 1_000 + index,
+			sessionFile,
+			tokens: { input: index, output: index + 1, total: index * 2 + 1 },
+		}],
+	}), "utf-8");
+}
+
+describe("retained child roster", () => {
+	it("returns only the newest 10 completed workflow children owned by the parent session", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-retained-children-"));
+		try {
+			for (let index = 0; index < 12; index++) writeRetainedRun(root, index);
+			writeRetainedRun(root, 50, "parent-b");
+			writeRetainedRun(root, 51, "parent-a", null);
+
+			const children = listRetainedChildren(path.join(root, "runs"), "parent-a");
+
+			assert.equal(children.length, 10);
+			assert.deepEqual(children.map((child) => child.runId), Array.from({ length: 10 }, (_, offset) => `child-${11 - offset}`));
+			assert.equal(children[0]?.agent, "worker");
+			assert.equal(children[0]?.completedAt, 1_011);
+			assert.ok((children[0]?.taskSummary.length ?? 0) <= 120);
+			assert.equal(children[0]?.taskSummary.startsWith("Task 11 with spacing"), true);
+			assert.deepEqual(children[0]?.tokenTotals, { input: 11, output: 12, total: 23 });
+			assert.equal(children.some((child) => child.runId === "child-50" || child.runId === "child-51"), false);
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+});