@selesai/code 0.5.27 → 0.5.29

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

@@ -109,6 +109,14 @@ describe("async resume lookup", () => {
 			});
 			assert.throws(() => resolveAsyncResumeTarget({ id: "run-descriptor" }, { asyncDirRoot: asyncRoot, resultsDir }), /capabilityCeiling sources/);
 
+			writeJson(path.join(asyncDir, "recovery-descriptor.json"), {
+				...descriptor,
+				launchContractDigest: "launch-contract-digest",
+			});
+			const valid = resolveAsyncResumeTarget({ id: "run-descriptor" }, { asyncDirRoot: asyncRoot, resultsDir });
+			assert.equal(valid.launchContractDigest, "launch-contract-digest");
+			assert.equal(valid.recoveryDescriptor?.launchContractDigest, "launch-contract-digest");
+
 			writeJson(path.join(asyncDir, "recovery-descriptor.json"), { ...descriptor, sourceRunId: "another-run" });
 			assert.throws(() => resolveAsyncResumeTarget({ id: "run-descriptor" }, { asyncDirRoot: asyncRoot, resultsDir }), /different source run/);
 

package/dist/extensions/pi-subagents/test/unit/capability-ceiling-agent-allowlist.test.ts

@@ -0,0 +1,102 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import { handleList } from "../../src/agents/agent-management.ts";
+import type { AgentConfig } from "../../src/agents/agents.ts";
+import { resolveSubagentLaunchContract } from "../../src/api/preflight.ts";
+import {
+	decodeSubagentCapabilityCeiling,
+	encodeSubagentCapabilityCeiling,
+	intersectSubagentCapabilityCeilings,
+	parseSubagentCapabilityCeiling,
+	registerSubagentCapabilityCeiling,
+} from "../../src/api/capability-ceiling.ts";
+import { runSync } from "../../src/runs/foreground/execution.ts";
+import { buildPiArgs } from "../../src/runs/shared/pi-args.ts";
+
+function agent(name: string): AgentConfig {
+	return {
+		name,
+		description: `${name} agent`,
+		systemPrompt: `${name} prompt`,
+		systemPromptMode: "replace",
+		inheritProjectContext: false,
+		inheritSkills: false,
+		source: "project",
+		filePath: `/tmp/${name}.md`,
+	};
+}
+
+describe("capability ceiling agent allowlist", () => {
+	it("parses, round-trips, and intersects allowedAgents", () => {
+		const parsed = parseSubagentCapabilityCeiling({ version: 1, allowedAgents: ["worker", "reviewer", "worker"], denyExtensions: false, sources: ["plan"] });
+		assert.deepEqual(parsed.allowedAgents, ["reviewer", "worker"]);
+		assert.deepEqual(decodeSubagentCapabilityCeiling(encodeSubagentCapabilityCeiling(parsed)), parsed);
+
+		assert.deepEqual(intersectSubagentCapabilityCeilings(
+			{ version: 1, allowedAgents: ["worker", "reviewer"], denyExtensions: false, sources: ["outer"] },
+			{ version: 1, allowedAgents: ["reviewer", "scout"], denyExtensions: true, sources: ["inner"] },
+		), {
+			version: 1,
+			allowedAgents: ["reviewer"],
+			denyExtensions: true,
+			sources: ["inner", "outer"],
+		});
+
+		assert.deepEqual(intersectSubagentCapabilityCeilings(
+			{ version: 1, allowedTools: ["read"], denyExtensions: false, sources: ["tools-only"] },
+			{ version: 1, allowedAgents: [], denyExtensions: false, sources: ["none"] },
+		)?.allowedAgents, []);
+	});
+
+	it("marks non-allowlisted agents as restricted in list output", () => {
+		const sessionId = `allowlist-list-${Date.now()}-${Math.random()}`;
+		const handle = registerSubagentCapabilityCeiling({ sessionId, source: "plan-mode", ceiling: { allowedAgents: ["commentator"] } });
+		try {
+			const result = handleList({}, { cwd: process.cwd(), currentSessionId: sessionId, modelRegistry: { getAvailable: () => [] } });
+			const text = result.content[0]?.text ?? "";
+			assert.match(text, /Executable agents:/);
+			assert.match(text, /- commentator /);
+			assert.match(text, /Restricted agents \(not executable in this session; capability ceiling: plan-mode\):/);
+			assert.match(text, /- builder /);
+		} finally {
+			handle.dispose();
+		}
+	});
+
+	it("rejects a non-allowlisted agent in preflight launch resolution", async () => {
+		const result = await resolveSubagentLaunchContract({
+			agent: "builder",
+			cwd: process.cwd(),
+			capabilityCeiling: { version: 1, allowedAgents: ["commentator"], denyExtensions: false, sources: ["plan-mode"] },
+		});
+		assert.equal(result.ok, false);
+		assert.equal(result.code, "restricted_agent");
+		assert.match(result.message, /does not allow agent 'builder'/);
+		assert.match(result.message, /Allowed agents: commentator/);
+	});
+
+	it("rejects a non-allowlisted foreground launch before spawning", async () => {
+		const result = await runSync(process.cwd(), [agent("builder"), agent("commentator")], "builder", "Do work", {
+			capabilityCeiling: { version: 1, allowedAgents: ["commentator"], denyExtensions: false, sources: ["plan-mode"] },
+		});
+		assert.equal(result.exitCode, 1);
+		assert.match(result.error ?? "", /does not allow agent 'builder'/);
+		assert.deepEqual(result.capabilityCeiling?.allowedAgents, ["commentator"]);
+	});
+
+	it("includes allowedAgents in propagated launch env and audit metadata", () => {
+		const { env, capabilityAudit } = buildPiArgs({
+			baseArgs: [],
+			task: "Review",
+			sessionEnabled: false,
+			inheritProjectContext: false,
+			inheritSkills: false,
+			childAgentName: "commentator",
+			capabilityCeiling: { version: 1, allowedAgents: ["commentator"], allowedTools: ["read"], denyExtensions: true, sources: ["plan-mode"] },
+		});
+		assert.equal(capabilityAudit?.agentAllowed, true);
+		assert.deepEqual(capabilityAudit?.agentRestrictionSources, ["plan-mode"]);
+		assert.ok(env.SELESAI_SUBAGENT_CAPABILITY_CEILING_V1);
+		assert.deepEqual(decodeSubagentCapabilityCeiling(env.SELESAI_SUBAGENT_CAPABILITY_CEILING_V1)?.allowedAgents, ["commentator"]);
+	});
+});

package/dist/extensions/pi-subagents/test/unit/capability-ceiling.test.ts

@@ -33,7 +33,7 @@ describe("subagent capability ceiling", () => {
 	});
 
 	it("rejects malformed policy and disposed updates", () => {
-		assert.throws(() => registerSubagentCapabilityCeiling({ sessionId: "x", source: "x", ceiling: {} }), /allowedTools or denyExtensions/);
+		assert.throws(() => registerSubagentCapabilityCeiling({ sessionId: "x", source: "x", ceiling: {} }), /allowedTools, allowedAgents, or denyExtensions/);
 		const handle = registerSubagentCapabilityCeiling({ sessionId: "disposed", source: "test", ceiling: { denyExtensions: true } });
 		handle.dispose();
 		assert.throws(() => handle.update({ allowedTools: ["read"] }), /disposed/);

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

@@ -198,6 +198,12 @@ describe("chain append requests", () => {
 			"reviewer:pending",
 			"auditor:pending",
 		]);
+		// Appended steps must carry their own bounded task description for fleet display.
+		assert.deepEqual(status.steps?.slice(1).map((step) => step.description), [
+			"Use {previous}",
+			"Use {previous}",
+			"Use {previous}",
+		]);
 		assert.deepEqual(status.parallelGroups, [{ start: 2, count: 2, stepIndex: 2 }]);
 		assert.equal(status.workflowGraph?.nodes[1]?.id, "step-1");
 		assert.equal(status.workflowGraph?.nodes[2]?.kind, "parallel-group");

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

@@ -285,7 +285,7 @@ describe("registered subagent tool prepareArguments", { skip: !schemasAvailable
 		const output = execFileSync(
 			process.execPath,
 			[
-				"--experimental-transform-types",
+				"--experimental-strip-types",
 				"--import",
 				"./test/support/register-loader.mjs",
 				"--input-type=module",

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: "worker",
+		agent: "builder",
 		task: "Implement the approved fix",
 		messages: [assistantText("Plan: update the files...")],
 	});
@@ -67,9 +67,9 @@ test("read-only issue drafting tasks do not trigger on suggested fix wording", (
 		attemptedMutation: false,
 		triggered: false,
 	});
-	assert.equal(expectsImplementationMutation("worker", task), false);
+	assert.equal(expectsImplementationMutation("builder", task), false);
 	assert.equal(
-		expectsImplementationMutation("worker", "Draft GitHub issue for a bug. Include suspected cause and suggested fix."),
+		expectsImplementationMutation("builder", "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("worker with mutating-capable tools still triggers when no mutation is observed", () => {
+test("builder with mutating-capable tools still triggers when no mutation is observed", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "worker",
+		agent: "builder",
 		task: "Fix the test implementation",
 		messages: [assistantText("I will edit it next")],
 		tools: ["read", "edit"],
@@ -104,10 +104,10 @@ test("worker with mutating-capable tools still triggers when no mutation is obse
 	});
 });
 
-test("oracle review tasks with bash available do not require mutation", () => {
+test("commentator 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: "oracle",
+		agent: "commentator",
 		task,
 		messages: [assistantText("Review complete with file-backed findings.")],
 		tools: ["read", "grep", "find", "ls", "bash", "intercom"],
@@ -121,57 +121,57 @@ test("oracle review tasks with bash available do not require mutation", () => {
 });
 
 test("review-only, research, and framework output instructions do not expect mutation", () => {
-	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("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("researcher", "Research the API behavior"), false);
 	assert.equal(expectsImplementationMutation("researcher", "Research this and patch the bug"), 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("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("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."),
+		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."),
 		false,
 	);
 });
 
-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("builder implementation verbs win over investigative wording and scoped prohibitions", () => {
+	assert.equal(expectsImplementationMutation("builder", "Investigate why the worker 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("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);
+	assert.equal(expectsImplementationMutation("builder", "Draft a GitHub issue, then implement the fix"), 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("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("edit and write tool calls count as mutation attempts", () => {
@@ -195,7 +195,7 @@ test("obvious mutating bash commands count as mutation attempts", () => {
 
 test("implementation task with mutation attempts does not trigger", () => {
 	const result = evaluateCompletionMutationGuard({
-		agent: "worker",
+		agent: "builder",
 		task: "Fix the failing test",
 		messages: [assistantToolCall("edit", { path: "test.ts" })],
 	});
@@ -279,7 +279,7 @@ test("provider checkpoint with a changed commit counts as mutation evidence", ()
 		assert.equal(hasMutationToolCall([message]), true);
 		assert.equal(
 			evaluateCompletionMutationGuard({
-				agent: "worker",
+				agent: "builder",
 				task: "Edit the target source file",
 				messages: [message],
 			}).triggered,
@@ -297,7 +297,7 @@ test("unchanged provider checkpoint does not bypass the completion guard", () =>
 		assert.equal(hasMutationToolCall([message]), false);
 		assert.equal(
 			evaluateCompletionMutationGuard({
-				agent: "worker",
+				agent: "builder",
 				task: "Edit the target source file",
 				messages: [message],
 			}).triggered,
@@ -308,7 +308,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: "worker",
+		agent: "builder",
 		task: "Edit docs/BACKEND_ARCHITECTURE.md",
 		messages: [
 			assistantThinking("Cursor edit: docs/BACKEND_ARCHITECTURE.md added 1 line\n"),

package/dist/extensions/pi-subagents/test/unit/config-dir-runtime.test.ts

@@ -65,6 +65,29 @@ describe("config directory resolution", () => {
 		}
 	});
 
+	it("invalidates cached runtime resolution when the package root changes", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-config-cache-"));
+		try {
+			const firstRoot = path.join(tempDir, "first");
+			const secondRoot = path.join(tempDir, "second");
+			for (const [root, configDir] of [[firstRoot, ".first-pi"], [secondRoot, ".second-pi"]] as const) {
+				fs.mkdirSync(root, { recursive: true });
+				fs.writeFileSync(path.join(root, "package.json"), JSON.stringify({
+					name: "@selesai/code",
+					piConfig: { configDir },
+				}), "utf-8");
+			}
+
+			process.env[SELESAI_CODING_AGENT_PACKAGE_ROOT_ENV] = firstRoot;
+			assert.equal(getConfigDirName(), ".first-pi");
+			assert.equal(getConfigDirName(), ".first-pi");
+			process.env[SELESAI_CODING_AGENT_PACKAGE_ROOT_ENV] = secondRoot;
+			assert.equal(getConfigDirName(), ".second-pi");
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
 	it("canonicalizes watcher paths and preserves the original path when native realpath fails", () => {
 		assert.equal(resolveWatchPath("C:\\SHORT~1\\watch", () => "C:\\Long Path\\watch"), "C:\\Long Path\\watch");
 		assert.equal(resolveWatchPath("C:\\SHORT~1\\watch", () => { throw new Error("missing"); }), "C:\\SHORT~1\\watch");

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

@@ -148,6 +148,33 @@ describe("control channel: request file", () => {
 		}
 	});
 
+	it("keeps steer requests for retry when the inbox cannot be scanned", () => {
+		const asyncDir = tmpAsyncDir("pi-control-steer-scan-retry-");
+		try {
+			requestAsyncSteer(asyncDir, { message: "retry me", id: "retry", ts: 1 });
+			let failScan = true;
+			const fsImpl = {
+				existsSync: fs.existsSync,
+				readdirSync: ((target: fs.PathLike) => {
+					if (failScan) {
+						failScan = false;
+						throw Object.assign(new Error("file table overflow"), { code: "ENFILE" });
+					}
+					return fs.readdirSync(target);
+				}) as typeof fs.readdirSync,
+				readFileSync: fs.readFileSync,
+				rmSync: fs.rmSync,
+			};
+
+			assert.deepEqual(consumeSteerRequests(asyncDir, fsImpl), []);
+			assert.deepEqual(consumeSteerRequests(asyncDir, fsImpl), [
+				{ type: "steer", id: "retry", ts: 1, message: "retry me" },
+			]);
+		} finally {
+			cleanup(asyncDir);
+		}
+	});
+
 	it("does not deliver a steer request if another consumer removed it first", () => {
 		const asyncDir = tmpAsyncDir("pi-control-steer-concurrent-");
 		try {

package/dist/extensions/pi-subagents/test/unit/delegation-api.test.ts

@@ -213,7 +213,7 @@ describe("public subagent delegation contract", () => {
 			},
 			executeVersioned: async (_id, params, _signal, _ctx, onUpdate) => {
 				observedParams = params as unknown as Record<string, unknown>;
-				onUpdate({ details: { mode: "single", results: [{ agent: "reviewer", model: "openai/gpt-5", thinking: "high" }], progress: [{ currentTool: "read" }] } });
+				onUpdate({ details: { mode: "single", runId: "run-v2", results: [{ agent: "reviewer", model: "openai/gpt-5", thinking: "high" }], progress: [{ currentTool: "read" }] } });
 				return {
 					details: {
 						mode: "single",
@@ -223,6 +223,7 @@ describe("public subagent delegation contract", () => {
 							exitCode: 0,
 							model: "openai/gpt-5",
 							thinking: "high",
+							launchContractDigest: "launch-contract-digest",
 							finalOutput: '{"looks":"json"}',
 							usage: { input: 2, output: 3, cacheRead: 4, cacheWrite: 5, cost: 0.01, turns: 2 },
 							progressSummary: { toolCount: 6, tokens: 5, durationMs: 7 },
@@ -237,7 +238,7 @@ describe("public subagent delegation contract", () => {
 		const responsePromise = once(events, SUBAGENT_DELEGATION_RESPONSE_EVENT);
 		events.emit(SUBAGENT_DELEGATION_REQUEST_EVENT, textRequest);
 		assert.deepEqual(await startedPromise, { version: 2, requestId: "attempt-1", ownerRunId: "owner-1", nodeId: "node-1" });
-		assert.deepEqual(await updatePromise, { version: 2, requestId: "attempt-1", ownerRunId: "owner-1", nodeId: "node-1", currentTool: "read", model: "openai/gpt-5" });
+		assert.deepEqual(await updatePromise, { version: 2, requestId: "attempt-1", ownerRunId: "owner-1", nodeId: "node-1", runId: "run-v2", currentTool: "read", model: "openai/gpt-5" });
 		assert.deepEqual(await responsePromise, {
 			version: 2,
 			requestId: "attempt-1",
@@ -249,6 +250,7 @@ describe("public subagent delegation contract", () => {
 			model: "openai/gpt-5",
 			thinking: "high",
 			exitCode: 0,
+			launchContractDigest: "launch-contract-digest",
 			result: { kind: "text", text: '{"looks":"json"}' },
 			usage: { input: 2, output: 3, cacheRead: 4, cacheWrite: 5, cost: 0.01, turns: 2, toolCalls: 6, durationMs: 7 },
 		} satisfies SubagentDelegationV2Response);

package/dist/extensions/pi-subagents/test/unit/dynamic-fanout.test.ts

@@ -200,6 +200,7 @@ describe("dynamic fanout helpers", () => {
 		const result = (agent: string, structuredOutput: unknown): SingleResult => ({
 			agent,
 			task: "t",
+			index: 0,
 			exitCode: 0,
 			messages: [],
 			usage: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, cost: 0, turns: 0 },