@selesai/code 0.8.3 → 0.8.4

package/dist/extensions/pi-subagents/test/unit/nested-control.test.ts

@@ -19,6 +19,7 @@ import {
 import { ASYNC_DIR, type SubagentState } from "../../src/shared/types.ts";
 
 const routeRoots: string[] = [];
+const fanoutListenerCleanupKey = "__piSubagentFanoutChildNestedControlInboxCleanups";
 const savedEnv = {
 	[SUBAGENT_CHILD_ENV]: process.env[SUBAGENT_CHILD_ENV],
 	[SUBAGENT_FANOUT_CHILD_ENV]: process.env[SUBAGENT_FANOUT_CHILD_ENV],
@@ -31,6 +32,15 @@ const savedEnv = {
 };
 
 afterEach(() => {
+	const globalStore = globalThis as Record<string, unknown>;
+	const listenerCleanups = globalStore[fanoutListenerCleanupKey];
+	if (listenerCleanups instanceof Map) {
+		for (const entry of listenerCleanups.values()) {
+			if (typeof entry === "function") entry();
+			else if (entry && typeof entry === "object" && typeof (entry as { cleanup?: unknown }).cleanup === "function") (entry as { cleanup: () => void }).cleanup();
+		}
+	}
+	delete globalStore[fanoutListenerCleanupKey];
 	for (const root of routeRoots.splice(0)) fs.rmSync(root, { recursive: true, force: true });
 	for (const [key, value] of Object.entries(savedEnv)) {
 		if (value === undefined) delete process.env[key];
@@ -88,7 +98,7 @@ function createNestedRun(id = "nested-live", state: "running" | "complete" | "fa
 		ts: 100,
 		parentRunId: "root-control",
 		parentStepIndex: 0,
-		child: { id, parentRunId: "root-control", parentStepIndex: 0, depth: 1, path: [{ runId: "root-control", stepIndex: 0 }], state, agent: "builder", ownerState: state === "running" ? "live" : "gone", ...extras },
+		child: { id, parentRunId: "root-control", parentStepIndex: 0, depth: 1, path: [{ runId: "root-control", stepIndex: 0 }], state, agent: "worker", ownerState: state === "running" ? "live" : "gone", ...extras },
 	});
 	return route;
 }
@@ -186,11 +196,52 @@ describe("nested control routing", () => {
 			const result = await createExecutor(state).execute("status", { action: "status", id: "root-control" }, new AbortController().signal, undefined, ctx(root));
 
 			assert.equal(result.isError, undefined);
+			assert.match(text(result), /^Status target: run root-control\nSpawn budget:/);
 			assert.match(text(result), /Run: root-control/);
-			assert.match(text(result), /↳ builder \[nested-foreground\] running/);
+			assert.match(text(result), /↳ worker \[nested-foreground\] running/);
 			assert.match(text(result), /Status: subagent\(\{ action: "status", id: "nested-foreground" \}\)/);
+
+			const transcript = await createExecutor(state).execute("transcript", { action: "status", id: "root-control", index: 0, view: "transcript" }, new AbortController().signal, undefined, ctx(root));
+			assert.equal(transcript.isError, undefined);
+			assert.match(text(transcript), /^Transcript target: run root-control · child 0\nSpawn budget:/);
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("labels dir-target status output", async () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-status-dir-label-"));
+		const runId = `status-dir-${Date.now().toString(36)}`;
+		const asyncDir = path.join(ASYNC_DIR, runId);
+		try {
+			fs.mkdirSync(asyncDir, { recursive: true });
+			fs.writeFileSync(path.join(asyncDir, "status.json"), JSON.stringify({
+				runId,
+				mode: "single",
+				state: "running",
+				startedAt: 1,
+				lastUpdate: 1,
+				cwd: root,
+				steps: [{ agent: "worker", status: "running" }],
+			}), "utf-8");
+
+			const state = createState();
+			state.foregroundControls.set("foreground-live", { runId: "foreground-live", mode: "single", startedAt: 1, updatedAt: 1, currentAgent: "worker", currentIndex: 0 });
+			state.lastForegroundControlId = "foreground-live";
+			const executor = createExecutor(state);
+			const result = await executor.execute("status-dir", { action: "status", dir: asyncDir }, new AbortController().signal, undefined, ctx(root));
+
+			assert.equal(result.isError, undefined);
+			assert.ok(text(result).startsWith(`Status target: dir ${asyncDir}\nSpawn budget:`), text(result));
+			assert.match(text(result), new RegExp(`Run: ${runId}`));
+			assert.doesNotMatch(text(result), /foreground-live/);
+
+			const transcript = await executor.execute("transcript-dir", { action: "status", dir: asyncDir, view: "transcript" }, new AbortController().signal, undefined, ctx(root));
+			assert.ok(text(transcript).startsWith(`Transcript target: dir ${asyncDir}\nSpawn budget:`), text(transcript));
+			assert.doesNotMatch(text(transcript), /Live foreground transcript/);
 		} finally {
 			fs.rmSync(root, { recursive: true, force: true });
+			fs.rmSync(asyncDir, { recursive: true, force: true });
 		}
 	});
 
@@ -318,7 +369,7 @@ describe("nested control routing", () => {
 		try {
 			const route = createNestedRun("nested-terminal-resume", "complete", { sessionFile: path.join(root, "missing-session.jsonl") });
 
-			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "builder", description: "Worker", prompt: "Do work" }])
+			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "worker", description: "Worker", prompt: "Do work" }])
 				.execute("resume", { action: "resume", id: "nested-terminal-resume", message: "continue" }, new AbortController().signal, undefined, ctx(root));
 
 			assert.equal(result.isError, true);
@@ -333,7 +384,7 @@ describe("nested control routing", () => {
 		try {
 			const route = createNestedRun("nested-stopped-resume", "stopped", { sessionFile: path.join(root, "missing-session.jsonl") });
 
-			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "builder", description: "Worker", prompt: "Do work" }])
+			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "worker", description: "Worker", prompt: "Do work" }])
 				.execute("resume", { action: "resume", id: "nested-stopped-resume", message: "continue" }, new AbortController().signal, undefined, ctx(root));
 
 			assert.equal(result.isError, true);
@@ -354,7 +405,7 @@ describe("nested control routing", () => {
 			fs.writeFileSync(attackerSessionFile, "");
 			const route = createNestedRun("nested-untrusted-resume", "complete", { sessionFile: attackerSessionFile });
 
-			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "builder", description: "Worker", prompt: "Do work" }])
+			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "worker", description: "Worker", prompt: "Do work" }])
 				.execute("resume", { action: "resume", id: "nested-untrusted-resume", message: "continue" }, new AbortController().signal, undefined, ctx(root, parentSessionFile));
 
 			assert.equal(result.isError, true);
@@ -374,7 +425,7 @@ describe("nested control routing", () => {
 			fs.writeFileSync(siblingSessionFile, "");
 			const route = createNestedRun("nested-sibling-resume", "complete", { sessionFile: siblingSessionFile });
 
-			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "builder", description: "Worker", prompt: "Do work" }])
+			const result = await createExecutor(stateWithNestedRoute(route), [{ name: "worker", description: "Worker", prompt: "Do work" }])
 				.execute("resume", { action: "resume", id: "nested-sibling-resume", message: "continue" }, new AbortController().signal, undefined, ctx(root, parentSessionFile));
 
 			assert.equal(result.isError, true);
@@ -395,8 +446,8 @@ describe("nested control routing", () => {
 				modelRegistry: { getAvailable() { throw new Error("model registry exploded"); } },
 			};
 
-			const result = await createExecutor(createState(), [{ name: "builder", description: "Worker", prompt: "Do work" }])
-				.execute("run", { agent: "builder", task: "go" }, new AbortController().signal, undefined, throwingCtx);
+			const result = await createExecutor(createState(), [{ name: "worker", description: "Worker", prompt: "Do work" }])
+				.execute("run", { agent: "worker", task: "go" }, new AbortController().signal, undefined, throwingCtx);
 
 			assert.equal(result.isError, true);
 			assert.match(text(result), /model registry exploded/);
@@ -485,6 +536,69 @@ describe("nested control routing", () => {
 		}
 	});
 
+	it("cleans the prior listener on reload and processes a resume request once", async () => {
+		const route = createNestedRoute("root-reload-listener");
+		routeRoots.push(path.dirname(route.eventSink));
+		setNestedRouteEnv(route, "root-reload-listener");
+		process.env[SUBAGENT_CHILD_ENV] = "1";
+		process.env[SUBAGENT_FANOUT_CHILD_ENV] = "1";
+		const registrations: string[] = [];
+		const activeIntervals = new Set<ReturnType<typeof setInterval>>();
+		const originalSetInterval = globalThis.setInterval;
+		const originalClearInterval = globalThis.clearInterval;
+		globalThis.setInterval = ((...args: Parameters<typeof setInterval>) => {
+			const interval = originalSetInterval(...args);
+			activeIntervals.add(interval);
+			return interval;
+		}) as typeof setInterval;
+		globalThis.clearInterval = ((interval: ReturnType<typeof setInterval>) => {
+			activeIntervals.delete(interval);
+			return originalClearInterval(interval);
+		}) as typeof clearInterval;
+		try {
+			const makePi = () => ({
+				events: { emit() {}, on() { return () => {}; } },
+				registerTool() { registrations.push("subagent"); },
+				getSessionName() { return "child"; },
+			}) as any;
+			const firstPi = makePi();
+
+			registerFanoutChildSubagentExtension(firstPi);
+			registerFanoutChildSubagentExtension(firstPi);
+			registerFanoutChildSubagentExtension(makePi());
+			assert.equal(activeIntervals.size, 1);
+
+			const unrelatedRoute = createNestedRoute("root-unrelated-listener");
+			routeRoots.push(path.dirname(unrelatedRoute.eventSink));
+			setNestedRouteEnv(unrelatedRoute, "root-unrelated-listener");
+			registerFanoutChildSubagentExtension(makePi());
+			assert.equal(activeIntervals.size, 2);
+
+			delete process.env[SUBAGENT_PARENT_EVENT_SINK_ENV];
+			delete process.env[SUBAGENT_PARENT_CONTROL_INBOX_ENV];
+			delete process.env[SUBAGENT_PARENT_ROOT_RUN_ID_ENV];
+			delete process.env[SUBAGENT_PARENT_CAPABILITY_TOKEN_ENV];
+			registerFanoutChildSubagentExtension(makePi());
+			assert.equal(activeIntervals.size, 2);
+
+			setNestedRouteEnv(route, "root-reload-listener");
+			writeNestedControlRequest(route, {
+				ts: Date.now(),
+				requestId: "reload-resume-once",
+				targetRunId: "missing-run",
+				action: "resume",
+				message: "continue",
+			});
+
+			await waitFor(() => readNestedControlResults(route).length === 1);
+			assert.equal(registrations.length, 4);
+			assert.equal(readNestedControlResults(route)[0]?.requestId, "reload-resume-once");
+		} finally {
+			globalThis.setInterval = originalSetInterval;
+			globalThis.clearInterval = originalClearInterval;
+		}
+	});
+
 	it("negatively acknowledges ownerless fanout child control requests and removes them", async () => {
 		const route = createNestedRoute("root-ownerless");
 		routeRoots.push(path.dirname(route.eventSink));

package/dist/extensions/pi-subagents/test/unit/nested-events.test.ts

@@ -64,8 +64,8 @@ function child(id: string, state: "queued" | "running" | "complete" | "failed" |
 		path: [{ runId: parentRunId, stepIndex: 1 }],
 		mode: "single" as const,
 		state,
-		agent: "commentator",
-		agents: ["commentator"],
+		agent: "reviewer",
+		agents: ["reviewer"],
 		startedAt: 10,
 		lastUpdate: ts,
 		steps: [{ agent: "leaf", status: state === "running" ? "running" as const : "complete" as const }],
@@ -355,7 +355,7 @@ describe("nested event parsing and projection", () => {
 				...child("nested-model-bounds", "running", 100),
 				model: "m".repeat(600),
 				thinking: "turbo",
-				steps: [{ agent: "leaf", status: "running", model: "builder", thinking: "xhigh" }],
+				steps: [{ agent: "leaf", status: "running", model: "worker", thinking: "xhigh" }],
 			},
 		});
 		const summary = projectNestedEvents(route).children[0]!;
@@ -371,13 +371,13 @@ describe("nested event parsing and projection", () => {
 			state: "running",
 			startedAt: 1,
 			steps: [
-				{ agent: "builder", status: "running", model: "provider/builder", thinking: "high" },
+				{ agent: "worker", status: "running", model: "provider/worker", thinking: "high" },
 			],
 		}, "/tmp/child-run", { id: "child-run", parentRunId: "parent-run", depth: 1, mode: "single", ts: 2 });
 
-		assert.equal(summary.model, "provider/builder");
+		assert.equal(summary.model, "provider/worker");
 		assert.equal(summary.thinking, "high");
-		assert.equal(summary.steps?.[0]?.model, "provider/builder");
+		assert.equal(summary.steps?.[0]?.model, "provider/worker");
 		assert.equal(summary.steps?.[0]?.thinking, "high");
 	});
 
@@ -389,7 +389,7 @@ describe("nested event parsing and projection", () => {
 			startedAt: 1,
 			processTerminal: { version: 1, state: "bogus", runId: "child-run", runnerProcessInstanceId: "runner-1" } as never,
 			steps: [{
-				agent: "builder",
+				agent: "worker",
 				status: "complete",
 				processTerminal: { version: 1, state: "observed", runId: "wrong-run", runnerProcessInstanceId: "runner-1" } as never,
 			}],
@@ -409,7 +409,7 @@ describe("nested event parsing and projection", () => {
 			startedAt: 1,
 			runtimeAcknowledgedExtensions: { version: 1, source: "child-runtime", ids: ["/Users/alice/.secret-extension", "ok-ext", "x".repeat(5000)], omitted: 0 } as never,
 			steps: [{
-				agent: "builder",
+				agent: "worker",
 				status: "complete",
 				runtimeAcknowledgedExtensions: { version: 1, source: "child-runtime", ids: ["C:/Users/alice/secret"], omitted: 0 } as never,
 			}],

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

@@ -99,7 +99,7 @@ function createFakeClock() {
 function completionResult(overrides: Record<string, unknown> = {}) {
 	return {
 		id: `notify-${Math.random().toString(36).slice(2)}`,
-		agent: "builder",
+		agent: "worker",
 		success: true,
 		summary: "Done",
 		exitCode: 0,
@@ -115,7 +115,7 @@ describe("registerSubagentNotify", () => {
 
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-empty-1",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary: "",
 			exitCode: 0,
@@ -127,7 +127,7 @@ describe("registerSubagentNotify", () => {
 		assert.deepEqual(sent[0], {
 			message: {
 				customType: "subagent-notify",
-				content: "Background task completed: **builder**\n\n(no output)",
+				content: "Background task completed: **worker**\n\n(no output)",
 				display: false,
 			},
 			options: { triggerTurn: true },
@@ -169,7 +169,7 @@ describe("registerSubagentNotify", () => {
 			id: "foreground-run:0",
 			runId: "foreground-run",
 			source: "foreground",
-			agent: "commentator",
+			agent: "reviewer",
 			success: true,
 			summary: "Recovered final review",
 			exitCode: 0,
@@ -181,7 +181,7 @@ describe("registerSubagentNotify", () => {
 		assert.deepEqual(sent[0], {
 			message: {
 				customType: "subagent-notify",
-				content: "Detached foreground task completed: **commentator**\n\nRecovered final review",
+				content: "Detached foreground task completed: **reviewer**\n\nRecovered final review",
 				display: true,
 			},
 			options: { triggerTurn: true },
@@ -193,7 +193,7 @@ describe("registerSubagentNotify", () => {
 		events.emit(SUBAGENT_FOREGROUND_COMPLETE_EVENT, {
 			id: "foreground-run:0",
 			source: "foreground",
-			agent: "commentator",
+			agent: "reviewer",
 			success: true,
 			summary: "Recovered final review",
 			timestamp: 123,
@@ -208,7 +208,7 @@ describe("registerSubagentNotify", () => {
 
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-summary-1",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary,
 			exitCode: 0,
@@ -222,7 +222,7 @@ describe("registerSubagentNotify", () => {
 		assert.deepEqual(sent[0], {
 			message: {
 				customType: "subagent-notify",
-				content: `Background task completed: **builder** (2/3)\n\n${summary}`,
+				content: `Background task completed: **worker** (2/3)\n\n${summary}`,
 				display: false,
 			},
 			options: { triggerTurn: true },
@@ -234,7 +234,7 @@ describe("registerSubagentNotify", () => {
 
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-path-1",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary: "Done",
 			exitCode: 0,
@@ -246,7 +246,7 @@ describe("registerSubagentNotify", () => {
 		assert.deepEqual(sent, [{
 			message: {
 				customType: "subagent-notify",
-				content: "Background task completed: **builder**\n\nDone\n\nSession file: /tmp/session.jsonl",
+				content: "Background task completed: **worker**\n\nDone\n\nSession file: /tmp/session.jsonl",
 				display: false,
 			},
 			options: { triggerTurn: true },
@@ -258,7 +258,7 @@ describe("registerSubagentNotify", () => {
 
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-paused-1",
-			agent: "builder",
+			agent: "worker",
 			success: false,
 			state: "paused",
 			summary: "Paused after interrupt. Waiting for explicit next action.",
@@ -270,7 +270,7 @@ describe("registerSubagentNotify", () => {
 		assert.deepEqual(sent[0], {
 			message: {
 				customType: "subagent-notify",
-				content: "Background task paused: **builder**\n\nPaused after interrupt. Waiting for explicit next action.",
+				content: "Background task paused: **worker**\n\nPaused after interrupt. Waiting for explicit next action.",
 				display: true,
 			},
 			options: { triggerTurn: true },
@@ -282,7 +282,7 @@ describe("registerSubagentNotify", () => {
 
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-other-session",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary: "Other done",
 			timestamp: 100,
@@ -290,7 +290,7 @@ describe("registerSubagentNotify", () => {
 		});
 		events.emit(SUBAGENT_ASYNC_COMPLETE_EVENT, {
 			id: "notify-sessionless",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary: "Legacy cwd-scoped done",
 			timestamp: 101,
@@ -407,16 +407,16 @@ describe("registerSubagentNotify", () => {
 describe("completion formatting helpers", () => {
 	it("formats and parses a parallel handoff without folding it into the result preview", () => {
 		const content = formatSingleCompletion({
-			agent: "builder",
+			agent: "worker",
 			status: "completed",
 			resultPreview: "Done",
 			handoffPath: "/tmp/run/handoff.json",
 			sessionLabel: "Session file",
 			sessionValue: "/tmp/session.jsonl",
 		});
-		assert.equal(content, "Background task completed: **builder**\n\nDone\n\nParallel handoff: /tmp/run/handoff.json\n\nSession file: /tmp/session.jsonl");
+		assert.equal(content, "Background task completed: **worker**\n\nDone\n\nParallel handoff: /tmp/run/handoff.json\n\nSession file: /tmp/session.jsonl");
 		assert.deepEqual(parseSubagentNotifyContent(content), {
-			agent: "builder",
+			agent: "worker",
 			status: "completed",
 			resultPreview: "Done",
 			handoffPath: "/tmp/run/handoff.json",
@@ -425,7 +425,7 @@ describe("completion formatting helpers", () => {
 		});
 		assert.equal(buildCompletionDetails({
 			id: "run",
-			agent: "builder",
+			agent: "worker",
 			success: true,
 			summary: "Done",
 			parallelHandoff: { path: "/tmp/run/handoff.json" },
@@ -434,32 +434,32 @@ describe("completion formatting helpers", () => {
 
 	it("formatSingleCompletion mirrors the in-handler single message shape", () => {
 		const content = formatSingleCompletion({
-			agent: "builder",
+			agent: "worker",
 			status: "completed",
 			taskInfo: " (2/3)",
 			resultPreview: "Done",
 			sessionLabel: "Session file",
 			sessionValue: "/tmp/session.jsonl",
 		});
-		assert.equal(content, "Background task completed: **builder** (2/3)\n\nDone\n\nSession file: /tmp/session.jsonl");
+		assert.equal(content, "Background task completed: **worker** (2/3)\n\nDone\n\nSession file: /tmp/session.jsonl");
 	});
 
 	it("parses detached foreground notification content for the custom renderer", () => {
 		const content = formatSingleCompletion({
-			agent: "commentator",
+			agent: "reviewer",
 			status: "failed",
 			source: "foreground",
 			resultPreview: "Acceptance rejected",
 			sessionLabel: "Session file",
-			sessionValue: "/tmp/commentator.jsonl",
+			sessionValue: "/tmp/reviewer.jsonl",
 		});
 		assert.deepEqual(parseSubagentNotifyContent(content), {
-			agent: "commentator",
+			agent: "reviewer",
 			status: "failed",
 			source: "foreground",
 			resultPreview: "Acceptance rejected",
 			sessionLabel: "session file",
-			sessionValue: "/tmp/commentator.jsonl",
+			sessionValue: "/tmp/reviewer.jsonl",
 		});
 	});
 

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

@@ -26,9 +26,8 @@ const expectedHostPeerRanges = {
 const expectedHostDevVersions = {
 	"@earendil-works/pi-agent-core": "0.81.0",
 	"@earendil-works/pi-ai": "0.81.0",
-	"@selesai/code": "0.8.2",
 	"@earendil-works/pi-tui": "0.81.0",
-} satisfies Record<(typeof hostPeerPackages)[number], string>;
+} satisfies Record<Exclude<(typeof hostPeerPackages)[number], "@selesai/code">, string>;
 
 function collectSourceFiles(dir: string): string[] {
 	const files: string[] = [];
@@ -64,6 +63,7 @@ test("published extension APIs use supported package entrypoints", async () => {
 		"./intercom-bridge": "./src/api/intercom-bridge.ts",
 		"./pi-args": "./src/api/pi-args.ts",
 		"./shared-types": "./src/api/shared-types.ts",
+		"./project-panes": "./src/api/project-panes.ts",
 	});
 	const backgroundWork = await import("pi-subagents/background-work");
 	assert.equal(backgroundWork.BACKGROUND_WORK_PROTOCOL_VERSION, 1);
@@ -90,6 +90,11 @@ test("published extension APIs use supported package entrypoints", async () => {
 	assert.equal(typeof sharedTypes.wrapForkTask, "function");
 	assert.equal(typeof sharedTypes.DEFAULT_FORK_PREAMBLE, "string");
 	assert.equal("TEMP_ROOT_DIR" in sharedTypes, false);
+	const projectPanes = await import("pi-subagents/project-panes");
+	assert.equal(projectPanes.PROJECT_PANES_API_VERSION, 1);
+	assert.equal(typeof projectPanes.openProjectPane, "function");
+	assert.equal(typeof projectPanes.getProjectPaneStatus, "function");
+	assert.equal(typeof projectPanes.closeProjectPane, "function");
 });
 
 test("direct @earendil-works runtime imports are declared for CI installs", () => {
@@ -118,6 +123,10 @@ test("direct dependency declarations are exact version pins", () => {
 
 	for (const section of ["dependencies", "devDependencies"] as const) {
 		for (const [name, version] of Object.entries<string>(packageJson[section] ?? {})) {
+			if (name === "@selesai/code") {
+				assert.equal(version, "file:./test/fixtures/pi-coding-agent-shim");
+				continue;
+			}
 			assert.match(version, exactVersionPattern, `${section}.${name} should use an exact version`);
 		}
 	}
@@ -147,6 +156,11 @@ test("host-owned development packages use the supported SDK baseline", () => {
 	for (const [name, version] of Object.entries(expectedHostDevVersions)) {
 		assert.equal(packageJson.devDependencies?.[name], version, `${name} should use ${version}`);
 	}
+	assert.equal(
+		packageJson.devDependencies?.["@selesai/code"],
+		"file:./test/fixtures/pi-coding-agent-shim",
+		"pi-coding-agent should use the local type/runtime shim until upstream Pi no longer pins vulnerable Undici",
+	);
 });
 
 test("old pi package scope is not used by source or tests", () => {

package/dist/extensions/pi-subagents/test/unit/parallel-handoff.test.ts

@@ -89,16 +89,16 @@ describe("parallel handoff", () => {
 				stepIndex: 2,
 				flatStartIndex: 3,
 				setup: setup("/repo", "base-1"),
-				diffs: [diff(dir, 0, "builder", true)],
+				diffs: [diff(dir, 0, "worker", true)],
 				cleanup: cleanup(),
 				results: [{
-					agent: "builder",
+					agent: "worker",
 					status: "completed",
 					summary: "implemented",
 					outputPath: "/artifacts/output.md",
 					structuredOutput: { ok: true },
 					structuredOutputPath: "/artifacts/structured.json",
-					sessionPath: "/sessions/builder.jsonl",
+					sessionPath: "/sessions/worker.jsonl",
 				}],
 				now: 100,
 			});
@@ -120,9 +120,9 @@ describe("parallel handoff", () => {
 				stepIndex: 1,
 				flatStartIndex: 1,
 				setup: setup("/repo", "base-1"),
-				diffs: [diff(dir, 1, "commentator", false)],
+				diffs: [diff(dir, 1, "reviewer", false)],
 				cleanup: cleanup("partial"),
-				results: [{ agent: "commentator", status: "failed", summary: "blocked" }],
+				results: [{ agent: "reviewer", status: "failed", summary: "blocked" }],
 				now: 200,
 			});
 			assert.equal(second.groupCount, 2);
@@ -189,9 +189,9 @@ describe("parallel handoff", () => {
 				stepIndex: 0,
 				flatStartIndex: 0,
 				setup: setup("/repo", "base-1"),
-				diffs: [diff(dir, 0, "builder", false)],
+				diffs: [diff(dir, 0, "worker", false)],
 				cleanup: cleanup(),
-				results: [{ agent: "builder", status: "completed" as const, summary: "done" }],
+				results: [{ agent: "worker", status: "completed" as const, summary: "done" }],
 			};
 			writeParallelHandoffGroup({ ...common, runId: "run-1" });
 			assert.throws(

package/dist/extensions/pi-subagents/test/unit/parallel-utils.test.ts

@@ -49,11 +49,11 @@ describe("flattenSteps", () => {
 
 	it("expands parallel groups into individual steps", () => {
 		const steps: RunnerStep[] = [
-			{ agent: "explorer", task: "find info" },
+			{ agent: "scout", task: "find info" },
 			{
 				parallel: [
-					{ agent: "commentator-a", task: "review part 1" },
-					{ agent: "commentator-b", task: "review part 2" },
+					{ agent: "reviewer-a", task: "review part 1" },
+					{ agent: "reviewer-b", task: "review part 2" },
 				],
 			},
 			{ agent: "summarizer", task: "combine" },
@@ -62,7 +62,7 @@ describe("flattenSteps", () => {
 		assert.equal(flat.length, 4);
 		assert.deepEqual(
 			flat.map((s) => s.agent),
-			["explorer", "commentator-a", "commentator-b", "summarizer"],
+			["scout", "reviewer-a", "reviewer-b", "summarizer"],
 		);
 	});
 
@@ -149,8 +149,8 @@ describe("mapConcurrent", () => {
 		// All workers should start nearly simultaneously
 		const d1 = startTimes[1]! - startTimes[0]!;
 		const d2 = startTimes[2]! - startTimes[0]!;
-		assert.ok(d1 < 20, `builder 1 should start immediately, got ${d1}ms delay`);
-		assert.ok(d2 < 20, `builder 2 should start immediately, got ${d2}ms delay`);
+		assert.ok(d1 < 20, `worker 1 should start immediately, got ${d1}ms delay`);
+		assert.ok(d2 < 20, `worker 2 should start immediately, got ${d2}ms delay`);
 	});
 
 	it("notifies scheduling settlement only after workers outliving an early rejection stop", async () => {
@@ -222,12 +222,12 @@ describe("DEFAULT_GLOBAL_CONCURRENCY_LIMIT", () => {
 describe("aggregateParallelOutputs", () => {
 	it("aggregates successful outputs with headers", () => {
 		const result = aggregateParallelOutputs([
-			{ agent: "commentator-a", output: "Looks good", exitCode: 0 },
-			{ agent: "commentator-b", output: "Needs fixes", exitCode: 0 },
+			{ agent: "reviewer-a", output: "Looks good", exitCode: 0 },
+			{ agent: "reviewer-b", output: "Needs fixes", exitCode: 0 },
 		]);
-		assert.ok(result.includes("=== Parallel Task 1 (commentator-a) ==="));
+		assert.ok(result.includes("=== Parallel Task 1 (reviewer-a) ==="));
 		assert.ok(result.includes("Looks good"));
-		assert.ok(result.includes("=== Parallel Task 2 (commentator-b) ==="));
+		assert.ok(result.includes("=== Parallel Task 2 (reviewer-b) ==="));
 		assert.ok(result.includes("Needs fixes"));
 	});
 

package/dist/extensions/pi-subagents/test/unit/path-handling.test.ts

@@ -96,7 +96,7 @@ describe("path.join vs template string concatenation", () => {
 
 	it("parallel subdir naming should use path.join", () => {
 		// settings.ts:302,306 pattern: `${subdir}/${task.output}`
-		const subdir = "parallel-0/0-_code-commentator";
+		const subdir = "parallel-0/0-_code-reviewer";
 		const output = "review.md";
 
 		const templateResult = `${subdir}/${output}`;
@@ -104,10 +104,10 @@ describe("path.join vs template string concatenation", () => {
 
 		// Both produce forward slashes here (subdir itself uses /).
 		// But if subdir comes from path.join on Windows, it would have backslashes.
-		const windowsSubdir = path.join("parallel-0", "0-_code-commentator");
+		const windowsSubdir = path.join("parallel-0", "0-_code-reviewer");
 		const windowsJoin = path.join(windowsSubdir, output);
 		// Consistent: all native separators
-		assert.equal(windowsJoin, path.join("parallel-0", "0-_code-commentator", output));
+		assert.equal(windowsJoin, path.join("parallel-0", "0-_code-reviewer", output));
 	});
 });