@selesai/code 0.5.19 → 0.5.21

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

@@ -0,0 +1,575 @@
+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 { visibleWidth, type MarkdownTheme } from "@earendil-works/pi-tui";
+import { collectFleetSnapshot, openSubagentFleet, SubagentFleetComponent } from "../../src/tui/fleet.ts";
+import { FLEET_STATUS_WIDGET_KEY } from "../../src/tui/fleet-status.ts";
+import { getArtifactPaths, getArtifactsDir, getProjectArtifactsDir } from "../../src/shared/artifacts.ts";
+import type { SubagentState } from "../../src/shared/types.ts";
+
+function stateForTest(): SubagentState {
+	return {
+		baseCwd: process.cwd(),
+		currentSessionId: "session-current",
+		asyncJobs: new Map(),
+		foregroundRuns: new Map(),
+		foregroundControls: new Map(),
+		lastForegroundControlId: null,
+		cleanupTimers: new Map(),
+		lastUiContext: null,
+		poller: null,
+		completionSeen: new Map(),
+		watcher: null,
+		watcherRestartTimer: null,
+		resultFileCoalescer: { schedule: () => false, clear: () => {} },
+	};
+}
+
+function writeAsyncRun(root: string, input: {
+	id: string;
+	sessionId?: string;
+	state?: "running" | "complete";
+	agents?: string[];
+	contexts?: Array<"fresh" | "fork">;
+	output?: string;
+	transcript?: Array<Record<string, unknown>>;
+}): string {
+	const asyncDir = path.join(root, input.id);
+	fs.mkdirSync(asyncDir, { recursive: true });
+	const agents = input.agents ?? ["worker"];
+	if (input.output !== undefined) fs.writeFileSync(path.join(asyncDir, "output-0.log"), input.output, "utf-8");
+	const transcriptPath = input.transcript ? path.join(asyncDir, "transcript-0.jsonl") : undefined;
+	if (transcriptPath && input.transcript) fs.writeFileSync(transcriptPath, `${input.transcript.map((record) => JSON.stringify(record)).join("\n")}\n`, "utf-8");
+	fs.writeFileSync(path.join(asyncDir, "status.json"), JSON.stringify({
+		runId: input.id,
+		sessionId: input.sessionId ?? "session-current",
+		mode: agents.length > 1 ? "parallel" : "single",
+		state: input.state ?? "running",
+		startedAt: 100,
+		lastUpdate: 200,
+		currentStep: 0,
+		steps: agents.map((agent, index) => ({
+			agent,
+			...(input.contexts?.[index] ? { context: input.contexts[index] } : {}),
+			status: input.state === "complete" ? "complete" : index === 0 ? "running" : "pending",
+			startedAt: 100,
+			...(index === 0 ? { sessionFile: path.join(asyncDir, `${agent}.jsonl`), ...(transcriptPath ? { transcriptPath } : {}) } : {}),
+		})),
+		...(input.output !== undefined ? { outputFile: "output-0.log" } : {}),
+	}, null, 2));
+	return asyncDir;
+}
+
+const theme = {
+	fg: (_name: string, text: string) => text,
+	bold: (text: string) => text,
+};
+
+const markdownTheme: MarkdownTheme = {
+	heading: (text) => text,
+	link: (text) => text,
+	linkUrl: (text) => text,
+	code: (text) => text,
+	codeBlock: (text) => text,
+	codeBlockBorder: (text) => text,
+	quote: (text) => text,
+	quoteBorder: (text) => text,
+	hr: (text) => text,
+	listBullet: (text) => text,
+	bold: (text) => text,
+	italic: (text) => text,
+	strikethrough: (text) => text,
+	underline: (text) => text,
+};
+
+describe("native subagent fleet", () => {
+	it("collects current-session foreground and flattened async children", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-collect-"));
+		try {
+			writeAsyncRun(root, { id: "async-current", agents: ["worker", "reviewer"], output: "CURRENT OUTPUT" });
+			writeAsyncRun(root, { id: "async-other", sessionId: "session-other", output: "OTHER OUTPUT" });
+			const state = stateForTest();
+			state.foregroundControls.set("foreground-live", {
+				runId: "foreground-live",
+				mode: "chain",
+				startedAt: 10,
+				updatedAt: 30,
+				currentAgent: "scout",
+				currentIndex: 1,
+			});
+			state.foregroundRuns!.set("foreground-recent", {
+				runId: "foreground-recent",
+				mode: "single",
+				cwd: root,
+				sessionId: "session-current",
+				updatedAt: 20,
+				children: [{ agent: "planner", index: 0, status: "completed", finalOutput: "PLAN COMPLETE" }],
+			});
+			state.foregroundRuns!.set("foreground-other", {
+				runId: "foreground-other",
+				mode: "single",
+				cwd: root,
+				sessionId: "session-other",
+				updatedAt: 20,
+				children: [{ agent: "outsider", index: 0, status: "completed" }],
+			});
+
+			const snapshot = collectFleetSnapshot(state, { asyncDirRoot: root, resultsDir: path.join(root, "results") });
+			assert.deepEqual(snapshot.items.map((item) => item.key), [
+				"foreground-active:foreground-live:1",
+				"async:async-current:0",
+				"async:async-current:1",
+				"foreground-recent:foreground-recent:0",
+			]);
+			assert.equal(snapshot.error, undefined);
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("keeps every active async run ahead of the bounded recent-completion window", () => {
+		const state = stateForTest();
+		for (let index = 0; index < 22; index++) {
+			state.fleetJobs ??= new Map();
+			state.fleetJobs.set(`terminal-${index}`, {
+				asyncId: `terminal-${index}`,
+				asyncDir: path.join(os.tmpdir(), `missing-terminal-${index}`),
+				sessionId: "session-current",
+				status: "complete",
+				mode: "single",
+				agents: ["worker"],
+				startedAt: index,
+				updatedAt: index,
+			});
+		}
+		state.fleetJobs!.set("active-old", {
+			asyncId: "active-old",
+			asyncDir: path.join(os.tmpdir(), "missing-active-old"),
+			sessionId: "session-current",
+			status: "running",
+			mode: "single",
+			agents: ["scout"],
+			startedAt: 0,
+			updatedAt: 0,
+		});
+
+		const snapshot = collectFleetSnapshot(state);
+		assert.equal(snapshot.items.length, 21);
+		assert.equal(snapshot.items[0]?.runId, "active-old");
+		assert.equal(snapshot.items.find((item) => item.runId === "terminal-21")?.state, "complete");
+		assert.ok(!snapshot.items.some((item) => item.runId === "terminal-0"));
+	});
+
+	it("renders selectable transcript detail and completed artifact paths within terminal width", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-render-"));
+		try {
+			const asyncDir = writeAsyncRun(root, { id: "async-finished", state: "complete", contexts: ["fork"], output: "FINAL ASYNC OUTPUT" });
+			const state = stateForTest();
+			let closed = false;
+			let renderRequests = 0;
+			const tui = { terminal: { rows: 32, columns: 100 }, requestRender: () => { renderRequests++; } };
+			const component = new SubagentFleetComponent(
+				tui as never,
+				theme as never,
+				state,
+				() => { closed = true; },
+				{ asyncDirRoot: root, resultsDir: path.join(root, "results"), refreshMs: 60_000 },
+			);
+			try {
+				const lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("FINAL ASYNC OUTPUT")));
+				assert.ok(lines.some((line) => line.includes("output-0.log")));
+				assert.ok(lines.some((line) => line.includes("worker") && line.includes("[fork]")));
+				assert.ok(lines.some((line) => line.includes("worker.jsonl")));
+				for (const line of lines) assert.ok(visibleWidth(line) <= 100, `line exceeded width: ${line}`);
+				tui.terminal.rows = 10;
+				assert.ok(component.render(100).length <= 8, "short-terminal render should fit the overlay's 85% height cap");
+				component.handleInput("\x1b[6~");
+				component.handleInput("r");
+				assert.ok(renderRequests >= 2);
+				component.handleInput("\x1b");
+				assert.equal(closed, true);
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("renders structured tool activity and assistant Markdown when a child transcript is available", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-structured-"));
+		try {
+			const longResponse = `## Finding\n\n${Array.from({ length: 40 }, (_, index) => `detail ${index}`).join("\n")}\n\n\`\`\`ts\nconst fleet = true;\n\`\`\``;
+			writeAsyncRun(root, {
+				id: "async-structured",
+				state: "complete",
+				output: "RAW FALLBACK SHOULD NOT RENDER",
+				transcript: [
+					{ recordType: "message", sourceEventType: "initial_prompt", role: "user", text: "injected task" },
+					{ recordType: "tool_start", toolName: "read", argsPreview: "src/tui/fleet.ts" },
+					{ recordType: "tool_end", toolName: "read" },
+					{ recordType: "message", role: "toolResult", text: "very large tool payload", message: { toolName: "read", isError: false } },
+					{ recordType: "message", role: "assistant", model: "test-model", text: longResponse },
+				],
+			});
+			const state = stateForTest();
+			state.baseCwd = root;
+			const component = new SubagentFleetComponent(
+				{ terminal: { rows: 32, columns: 100 }, requestRender() {} } as never,
+				theme as never,
+				state,
+				() => {},
+				{ asyncDirRoot: root, resultsDir: path.join(root, "results"), refreshMs: 60_000, markdownTheme },
+			);
+			try {
+				let lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("Conversation") && line.includes("assistant response")));
+				assert.ok(lines.some((line) => line.includes("const fleet = true;")));
+				assert.ok(!lines.some((line) => line.includes("very large tool payload")));
+				assert.ok(!lines.some((line) => line.includes("RAW FALLBACK SHOULD NOT RENDER")));
+				const bottomLines = lines;
+				const realDateNow = Date.now;
+				const laterNow = realDateNow() + 2_000;
+				Date.now = () => laterNow;
+				try {
+					component.handleInput("K");
+					lines = component.render(100);
+					assert.notDeepEqual(lines, bottomLines, "Shift+K should scroll the conversation up by one line");
+					component.handleInput("J");
+					lines = component.render(100);
+					assert.deepEqual(lines, bottomLines, "Shift+J should scroll the conversation back down by one line");
+				} finally {
+					Date.now = realDateNow;
+				}
+				for (let page = 0; page < 4; page++) component.handleInput("\x1b[5~");
+				lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("Conversation") && line.includes("assistant response")), "the conversation header should remain pinned while scrolling");
+				assert.ok(lines.some((line) => line.includes("✓ read") && line.includes("src/tui/fleet.ts")), "page up should reveal compact tool activity");
+				assert.ok(lines.some((line) => line.includes("Assistant") && line.includes("test-model")), "page up should reveal the rendered assistant message header");
+				for (const renderWidth of [60, 80, 100]) {
+					for (const line of component.render(renderWidth)) {
+						assert.ok(visibleWidth(line) <= renderWidth, `line exceeded ${renderWidth} columns: ${line}`);
+					}
+				}
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("uses parent-tracked async state and cwd without rescanning status", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-async-cwd-"));
+		try {
+			const asyncDir = path.join(root, "async-custom-cwd");
+			const customCwd = path.join(root, "custom-cwd");
+			const artifactsRoot = getProjectArtifactsDir(customCwd);
+			fs.mkdirSync(asyncDir, { recursive: true });
+			fs.mkdirSync(artifactsRoot, { recursive: true });
+			const transcriptPath = path.join(artifactsRoot, "async-custom-cwd_worker_transcript.jsonl");
+			fs.writeFileSync(transcriptPath, `${JSON.stringify({ recordType: "message", role: "assistant", text: "Custom cwd async result" })}\n`, "utf-8");
+			fs.writeFileSync(path.join(asyncDir, "status.json"), "{in-flight status", "utf-8");
+			const state = stateForTest();
+			state.baseCwd = path.join(root, "parent-cwd");
+			state.asyncJobs.set("async-custom-cwd", {
+				asyncId: "async-custom-cwd",
+				asyncDir,
+				cwd: customCwd,
+				sessionId: "session-current",
+				status: "running",
+				mode: "single",
+				startedAt: 100,
+				updatedAt: 200,
+				steps: [{ agent: "worker", index: 0, status: "running", transcriptPath }],
+			});
+			const component = new SubagentFleetComponent(
+				{ terminal: { rows: 28, columns: 100 }, requestRender() {} } as never,
+				theme as never,
+				state,
+				() => {},
+				{ refreshMs: 60_000, markdownTheme },
+			);
+			try {
+				assert.ok(component.render(100).some((line) => line.includes("Custom cwd async result")));
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("toggles bounded tool output expansion with x and Ctrl+O", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-tool-toggle-"));
+		try {
+			writeAsyncRun(root, {
+				id: "async-tools",
+				state: "complete",
+				transcript: [
+					{ recordType: "tool_start", toolCallId: "read-1", toolName: "read", argsPreview: "src/a.ts", argsPayload: JSON.stringify({ path: "src/a.ts" }), ts: 1 },
+					{ recordType: "tool_end", toolCallId: "read-1", toolName: "read", ts: 2 },
+					{ recordType: "message", role: "toolResult", toolCallId: "read-1", toolName: "read", text: "alpha\nbeta\ngamma", isError: false, ts: 3 },
+				],
+			});
+			const state = stateForTest();
+			state.baseCwd = root;
+			let renderRequests = 0;
+			const component = new SubagentFleetComponent(
+				{ terminal: { rows: 32, columns: 100 }, requestRender() { renderRequests++; } } as never,
+				theme as never,
+				state,
+				() => {},
+				{ asyncDirRoot: root, resultsDir: path.join(root, "results"), refreshMs: 60_000, markdownTheme },
+			);
+			try {
+				let lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("alpha beta gamma") && line.includes("x to expand")));
+				component.handleInput("x");
+				lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("alpha") && !line.includes("beta")));
+				assert.ok(lines.some((line) => line.includes("x to collapse")));
+				component.handleInput("\x0f");
+				lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("alpha beta gamma") && line.includes("x to expand")));
+				fs.appendFileSync(
+					path.join(root, "async-tools", "transcript-0.jsonl"),
+					`${JSON.stringify({ recordType: "message", role: "assistant", text: "Cache refreshed after append", ts: 4 })}\n`,
+					"utf-8",
+				);
+				lines = component.render(100);
+				assert.ok(lines.some((line) => line.includes("Cache refreshed after append")));
+				assert.equal(renderRequests, 2);
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("renders the selected foreground parallel child's structured transcript", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-foreground-"));
+		try {
+			const state = stateForTest();
+			state.baseCwd = path.join(root, "parent-cwd");
+			const effectiveCwd = path.join(root, "effective-cwd");
+			const now = Date.now();
+			state.foregroundControls.set("foreground-live", {
+				runId: "foreground-live",
+				mode: "parallel",
+				startedAt: now - 1_000,
+				updatedAt: now,
+				cwd: effectiveCwd,
+				currentAgent: "reviewer",
+				currentIndex: 1,
+				description: "Review the active task",
+				activeChildren: new Map([
+					[0, { index: 0, agent: "worker", description: "Implement the active task", startedAt: now - 900, updatedAt: now - 100, tokens: 120 }],
+					[1, { index: 1, agent: "reviewer", description: "Review the active task", startedAt: now - 800, updatedAt: now, tokens: 240 }],
+				]),
+			});
+			const artifactsRoot = getProjectArtifactsDir(effectiveCwd);
+			fs.mkdirSync(artifactsRoot, { recursive: true });
+			const transcriptPath = getArtifactPaths(artifactsRoot, "foreground-live", "worker", 0).transcriptPath;
+			fs.writeFileSync(transcriptPath, `${JSON.stringify({ recordType: "message", role: "assistant", model: "test-model", text: "**Worker live result**" })}\n`, "utf-8");
+			const component = new SubagentFleetComponent(
+				{ terminal: { rows: 28, columns: 90 }, requestRender() {} } as never,
+				theme as never,
+				state,
+				() => {},
+				{ initialKey: "foreground-active:foreground-live:0", refreshMs: 60_000, markdownTheme },
+			);
+			try {
+				const lines = component.render(90);
+				assert.ok(lines.some((line) => line.includes("worker")));
+				assert.ok(lines.some((line) => line.includes("reviewer")));
+				assert.ok(lines.some((line) => line.includes("foreground · live")));
+				assert.ok(lines.some((line) => line.includes("Task") && line.includes("Implement the active task")));
+				assert.ok(lines.some((line) => line.includes("Conversation") && line.includes("assistant response")));
+				assert.ok(lines.some((line) => line.includes("Worker live result")));
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("renders configured session and temp transcripts for active foreground, completed foreground, and async children", () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-artifact-roots-"));
+		const createdTranscriptPaths: string[] = [];
+		try {
+			for (const preference of ["session", "temp"] as const) {
+				const baseCwd = path.join(root, `${preference}-cwd`);
+				const sessionFile = path.join(root, `${preference}-session`, "session.jsonl");
+				const artifactsRoot = getArtifactsDir(sessionFile, baseCwd, preference);
+				const prefix = `${path.basename(root)}-${preference}`;
+				fs.mkdirSync(artifactsRoot, { recursive: true });
+				const transcript = (runId: string, agent: string, index: number, text: string) => {
+					const transcriptPath = getArtifactPaths(artifactsRoot, runId, agent, index).transcriptPath;
+					fs.writeFileSync(transcriptPath, `${JSON.stringify({ recordType: "message", role: "assistant", text })}\n`, "utf-8");
+					createdTranscriptPaths.push(transcriptPath);
+					return transcriptPath;
+				};
+
+				const activeId = `${prefix}-active`;
+				const recentId = `${prefix}-recent`;
+				const asyncId = `${prefix}-async`;
+				transcript(activeId, "worker", 0, `${preference} active foreground transcript`);
+				const recentTranscript = transcript(recentId, "reviewer", 0, `${preference} completed foreground transcript`);
+				const asyncTranscript = transcript(asyncId, "scout", 0, `${preference} async transcript`);
+
+				const state = stateForTest();
+				state.baseCwd = baseCwd;
+				state.artifactDirPreference = preference;
+				state.parentSessionFile = sessionFile;
+				state.foregroundControls.set(activeId, {
+					runId: activeId,
+					mode: "single",
+					cwd: baseCwd,
+					startedAt: 100,
+					updatedAt: 300,
+					currentAgent: "worker",
+					currentIndex: 0,
+				});
+				state.foregroundRuns!.set(recentId, {
+					runId: recentId,
+					mode: "single",
+					cwd: baseCwd,
+					sessionId: "session-current",
+					updatedAt: 200,
+					children: [{ agent: "reviewer", index: 0, status: "completed", transcriptPath: recentTranscript }],
+				});
+				state.asyncJobs.set(asyncId, {
+					asyncId,
+					asyncDir: path.join(root, `${preference}-async-run`),
+					cwd: baseCwd,
+					sessionId: "session-current",
+					status: "complete",
+					mode: "single",
+					startedAt: 100,
+					updatedAt: 250,
+					steps: [{ agent: "scout", index: 0, status: "complete", transcriptPath: asyncTranscript }],
+				});
+
+				for (const [initialKey, expected] of [
+					[`foreground-active:${activeId}:0`, `${preference} active foreground transcript`],
+					[`foreground-recent:${recentId}:0`, `${preference} completed foreground transcript`],
+					[`async:${asyncId}:0`, `${preference} async transcript`],
+				] as const) {
+					const component = new SubagentFleetComponent(
+						{ terminal: { rows: 28, columns: 100 }, requestRender() {} } as never,
+						theme as never,
+						state,
+						() => {},
+						{ initialKey, refreshMs: 60_000, markdownTheme },
+					);
+					try {
+						assert.ok(component.render(100).some((line) => line.includes(expected)), `missing ${expected}`);
+					} finally {
+						component.dispose();
+					}
+				}
+			}
+		} finally {
+			for (const transcriptPath of createdTranscriptPaths) fs.rmSync(transcriptPath, { force: true });
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+
+	it("suppresses the status widget for the full inspector lifecycle", async () => {
+		const state = stateForTest();
+		let hidden = 0;
+		let observedOpen = false;
+		const ctx = {
+			hasUI: true,
+			ui: {
+				setWidget(key: string, content: unknown) {
+					assert.equal(key, FLEET_STATUS_WIDGET_KEY);
+					assert.equal(content, undefined);
+					hidden++;
+				},
+				async custom() {
+					observedOpen = state.fleetInspectorOpen === true;
+					throw new Error("overlay closed");
+				},
+			},
+		};
+
+		await assert.rejects(openSubagentFleet(ctx as never, state), /overlay closed/);
+		assert.equal(hidden, 1);
+		assert.equal(observedOpen, true);
+		assert.equal(state.fleetInspectorOpen, false);
+	});
+
+	it("opens the inspector with the FleetView-selected child focused", () => {
+		const state = stateForTest();
+		state.foregroundControls.set("run-worker", {
+			runId: "run-worker",
+			mode: "single",
+			startedAt: 10,
+			updatedAt: 20,
+			currentAgent: "worker",
+			currentIndex: 0,
+		});
+		state.foregroundControls.set("run-reviewer", {
+			runId: "run-reviewer",
+			mode: "single",
+			startedAt: 11,
+			updatedAt: 21,
+			currentAgent: "reviewer",
+			currentIndex: 0,
+		});
+		const component = new SubagentFleetComponent(
+			{ terminal: { rows: 28, columns: 90 }, requestRender() {} } as never,
+			theme as never,
+			state,
+			() => {},
+			{ initialKey: "foreground-active:run-worker:0", refreshMs: 60_000 },
+		);
+		try {
+			const selectedLine = component.render(90).find((line) => line.includes("›"));
+			assert.ok(selectedLine?.includes("run-work"), `unexpected selected row: ${selectedLine}`);
+		} finally {
+			component.dispose();
+		}
+	});
+
+	it("refreshes the roster while the overlay remains open", async () => {
+		const root = fs.mkdtempSync(path.join(os.tmpdir(), "pi-fleet-refresh-"));
+		try {
+			const state = stateForTest();
+			let renderRequests = 0;
+			const tui = { terminal: { rows: 28, columns: 90 }, requestRender: () => { renderRequests++; } };
+			const component = new SubagentFleetComponent(
+				tui as never,
+				theme as never,
+				state,
+				() => {},
+				{ asyncDirRoot: root, resultsDir: path.join(root, "results"), refreshMs: 10 },
+			);
+			try {
+				assert.ok(component.render(90).some((line) => line.includes("No tracked children")));
+				const initialOutput = Array.from({ length: 40 }, (_, index) => `output line ${index}`).join("\n");
+				const asyncDir = writeAsyncRun(root, { id: "appeared-live", output: initialOutput });
+				await new Promise((resolve) => setTimeout(resolve, 35));
+				let lines = component.render(90);
+				assert.ok(lines.some((line) => line.includes("appeared")));
+				assert.ok(lines.some((line) => line.includes("output line 39")));
+				fs.appendFileSync(path.join(asyncDir, "output-0.log"), "\nLATEST LIVE OUTPUT", "utf-8");
+				await new Promise((resolve) => setTimeout(resolve, 35));
+				lines = component.render(90);
+				assert.ok(lines.some((line) => line.includes("LATEST LIVE OUTPUT")), "live transcript should keep following new output");
+				assert.ok(renderRequests > 0);
+			} finally {
+				component.dispose();
+			}
+		} finally {
+			fs.rmSync(root, { recursive: true, force: true });
+		}
+	});
+});

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

@@ -0,0 +1,65 @@
+import assert from "node:assert/strict";
+import { describe, it } from "node:test";
+import type { AgentProgress, ForegroundRunControl } from "../../src/shared/types.ts";
+import { beginForegroundChild, finishForegroundChild, updateForegroundChild } from "../../src/runs/foreground/foreground-control.ts";
+
+function progress(index: number, agent: string, tokens: number): AgentProgress {
+	return {
+		index,
+		agent,
+		status: "running",
+		task: `${agent} task`,
+		recentTools: [],
+		recentOutput: [],
+		toolCount: index + 1,
+		tokens,
+		durationMs: 10,
+	};
+}
+
+describe("foreground child control", () => {
+	it("tracks concurrent children independently and promotes the latest active child", () => {
+		const control: ForegroundRunControl = {
+			runId: "parallel-run",
+			mode: "parallel",
+			startedAt: 1,
+			updatedAt: 1,
+			activeChildren: new Map(),
+		};
+		let firstInterrupts = 0;
+		let secondInterrupts = 0;
+		beginForegroundChild(control, {
+			index: 0,
+			agent: "reviewer",
+			description: "Review correctness",
+			interrupt: () => { firstInterrupts++; return true; },
+		});
+		beginForegroundChild(control, {
+			index: 1,
+			agent: "reviewer",
+			description: "Review quality",
+			interrupt: () => { secondInterrupts++; return true; },
+		});
+
+		assert.equal(control.activeChildren?.size, 2);
+		assert.equal(control.currentIndex, 1);
+		updateForegroundChild(control, 0, progress(0, "reviewer", 120));
+		assert.equal(control.currentIndex, 0);
+		assert.equal(control.tokens, 120);
+		assert.equal(control.activeChildren?.get(1)?.tokens, undefined);
+		assert.equal(control.interrupt?.(), true);
+		assert.equal(firstInterrupts, 1);
+		assert.equal(secondInterrupts, 0);
+
+		updateForegroundChild(control, 1, progress(1, "reviewer", 240));
+		finishForegroundChild(control, 1);
+		assert.equal(control.currentIndex, 0);
+		assert.equal(control.tokens, 120);
+		assert.deepEqual([...control.activeChildren!.keys()], [0]);
+
+		finishForegroundChild(control, 0);
+		assert.equal(control.activeChildren?.size, 0);
+		assert.equal(control.currentIndex, undefined);
+		assert.equal(control.interrupt, undefined);
+	});
+});