@selesai/code 0.5.28 → 0.6.1

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

@@ -17,6 +17,7 @@ import {
 	type SubagentDelegationV2Response,
 } from "../../src/api/delegation.ts";
 import { parseSubagentDelegationRequest } from "../../src/slash/delegation-request.ts";
+import { toSubagentDelegationExecutionParams } from "../../src/slash/delegation-adapters.ts";
 import {
 	registerPromptTemplateDelegationBridge,
 	type PromptTemplateBridgeEvents,
@@ -213,7 +214,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 +224,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 +239,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 +251,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);
@@ -506,6 +509,29 @@ describe("public subagent delegation contract", () => {
 		assert.deepEqual(responses, []);
 	});
 
+	it("resolves an omitted v1 outputMode to explicit inline so response.output stays populated", () => {
+		const minimal: SubagentDelegationRequest = {
+			version: 1,
+			requestId: "adapter-default-1",
+			agent: "reviewer",
+			task: "Review evidence",
+			context: "fresh",
+			cwd: "/repo",
+		};
+		// Omitted outputMode on a strict v1 request keeps the legacy full-text
+		// contract: explicit inline, so `SubagentDelegationResponse.output` stays
+		// populated instead of degrading to the reference-first file-only default.
+		const params = toSubagentDelegationExecutionParams(minimal);
+		assert.equal(params.outputMode, "inline");
+		assert.equal(params.output, undefined);
+
+		// Explicit v1 modes are forwarded exactly; v2 stays unaffected (output: false).
+		assert.equal(toSubagentDelegationExecutionParams(request).outputMode, "file-only");
+		const inline = toSubagentDelegationExecutionParams({ ...request, outputMode: "inline" });
+		assert.equal(inline.outputMode, "inline");
+		assert.equal(toSubagentDelegationExecutionParams({ ...request, outputMode: undefined }).outputMode, "inline");
+	});
+
 	it("runs one v1 request through the existing executor and returns structured metadata", async () => {
 		const events = new FakeEvents();
 		let executeCalls = 0;

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 },

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

@@ -1,6 +1,6 @@
 import assert from "node:assert/strict";
 import { describe, it } from "node:test";
-import { Editor, visibleWidth } from "@earendil-works/pi-tui";
+import { Editor, type EditorComponent, visibleWidth } from "@earendil-works/pi-tui";
 import type { ExtensionContext } from "@selesai/code";
 import type { SubagentState } from "../../src/shared/types.ts";
 import { collectFleetSnapshot } from "../../src/tui/fleet.ts";
@@ -10,6 +10,7 @@ import {
 	collectFleetStatusEntries,
 	formatFleetElapsed,
 	formatFleetTokens,
+	resolveFleetViewPlacement,
 } from "../../src/tui/fleet-status.ts";
 
 function stateForTest(): SubagentState {
@@ -45,6 +46,13 @@ describe("below-editor subagent FleetView", () => {
 		assert.equal(formatFleetTokens(1_250_000), "↓ 1.3M tokens");
 	});
 
+	it("resolves configured FleetView placement with a below-editor fallback", () => {
+		assert.equal(resolveFleetViewPlacement(undefined), "belowEditor");
+		assert.equal(resolveFleetViewPlacement("belowEditor"), "belowEditor");
+		assert.equal(resolveFleetViewPlacement("aboveEditor"), "aboveEditor");
+		assert.equal(resolveFleetViewPlacement("side"), "belowEditor");
+	});
+
 	it("renders main plus active children below the editor and bounds every line", () => {
 		const state = stateForTest();
 		const now = Date.now();
@@ -56,6 +64,7 @@ describe("below-editor subagent FleetView", () => {
 				updatedAt: now,
 				currentAgent: `worker-${index}`,
 				description: index === 0 ? "Inspect\nmodule 0" : `Inspect module ${index}`,
+				...(index === 0 ? { model: "anthropic/fable-5", thinking: "low" } : {}),
 				tokens: index === 0 ? 13_100 : index,
 			});
 		}
@@ -85,7 +94,7 @@ describe("below-editor subagent FleetView", () => {
 			const component = widgetFactory!({ requestRender() {} }, theme);
 			const lines = component.render(80);
 			assert.ok(lines.some((line) => line.includes("⏺ main")));
-			assert.ok(lines.some((line) => line.includes("worker-0") && line.includes("Inspect module 0")));
+			assert.ok(lines.some((line) => line.includes("worker-0 (fable-5 · thinking low)") && line.includes("Inspect module 0")));
 			assert.ok(lines.some((line) => line.includes("11s · ↓ 13.1k tokens")));
 			assert.ok(lines.some((line) => line.includes("↓ 2 more")));
 			for (const line of lines) assert.ok(visibleWidth(line) <= 80, `line exceeded width: ${line}`);
@@ -94,6 +103,143 @@ describe("below-editor subagent FleetView", () => {
 		}
 	});
 
+	it("registers above the editor when configured", () => {
+		const state = stateForTest();
+		state.foregroundControls.set("run-worker", {
+			runId: "run-worker",
+			mode: "single",
+			startedAt: 10,
+			updatedAt: 20,
+			currentAgent: "worker",
+		});
+		let placement: string | undefined;
+		const ctx = {
+			hasUI: true,
+			ui: {
+				setWidget(_key: string, content: unknown, options?: { placement?: string }) {
+					if (content) placement = options?.placement;
+				},
+				onTerminalInput() { return () => {}; },
+				getEditorText() { return ""; },
+				notify() {},
+				theme,
+			},
+		} as unknown as ExtensionContext;
+		const fleet = new SubagentFleetStatus(state, () => {}, { refreshMs: 60_000, placement: "aboveEditor" });
+		try {
+			fleet.setContext(ctx);
+			assert.equal(placement, "aboveEditor");
+		} finally {
+			fleet.dispose();
+		}
+	});
+
+	it("stops refreshing when the captured extension context becomes stale", () => {
+		const state = stateForTest();
+		state.foregroundControls.set("run-worker", {
+			runId: "run-worker",
+			mode: "single",
+			startedAt: 10,
+			updatedAt: 20,
+			currentAgent: "worker",
+		});
+		let stale = false;
+		let contextReads = 0;
+		let inputUnsubscribes = 0;
+		let widgetRemovals = 0;
+		const ctx = {
+			get hasUI() {
+				contextReads++;
+				if (stale) {
+					throw new Error("This extension ctx is stale after session replacement or reload.");
+				}
+				return true;
+			},
+			ui: {
+				setWidget(_key: string, content: unknown) {
+					if (content === undefined) widgetRemovals++;
+				},
+				onTerminalInput() { return () => { inputUnsubscribes++; }; },
+				getEditorText() { return ""; },
+				notify() {},
+				theme,
+			},
+		} as unknown as ExtensionContext;
+		const fleet = new SubagentFleetStatus(state, () => {}, { refreshMs: 60_000 });
+		try {
+			fleet.setContext(ctx);
+			stale = true;
+			assert.doesNotThrow(() => fleet.refresh());
+			assert.equal(inputUnsubscribes, 1);
+			assert.equal(widgetRemovals, 1);
+			assert.equal((fleet as unknown as { timer?: unknown }).timer, undefined);
+			const readsAfterStaleRefresh = contextReads;
+			fleet.refresh();
+			assert.equal(contextReads, readsAfterStaleRefresh, "later refreshes must not reuse the stale context");
+		} finally {
+			fleet.dispose();
+		}
+	});
+
+	it("does not swallow unrelated widget cleanup errors", () => {
+		const state = stateForTest();
+		state.foregroundControls.set("run-worker", {
+			runId: "run-worker",
+			mode: "single",
+			startedAt: 10,
+			updatedAt: 20,
+			currentAgent: "worker",
+		});
+		const ctx = {
+			hasUI: true,
+			ui: {
+				setWidget(_key: string, content: unknown) {
+					if (content === undefined) throw new Error("widget cleanup failed");
+				},
+				onTerminalInput() { return () => {}; },
+				getEditorText() { return ""; },
+				notify() {},
+				theme,
+			},
+		} as unknown as ExtensionContext;
+		const fleet = new SubagentFleetStatus(state, () => {}, { refreshMs: 60_000 });
+		fleet.setContext(ctx);
+		assert.throws(() => fleet.dispose(), /widget cleanup failed/);
+	});
+
+	it("preserves multiple unrelated UI cleanup errors", () => {
+		const state = stateForTest();
+		state.foregroundControls.set("run-worker", {
+			runId: "run-worker",
+			mode: "single",
+			startedAt: 10,
+			updatedAt: 20,
+			currentAgent: "worker",
+		});
+		const ctx = {
+			hasUI: true,
+			ui: {
+				setWidget(_key: string, content: unknown) {
+					if (content === undefined) throw new Error("widget cleanup failed");
+				},
+				onTerminalInput() {
+					return () => { throw new Error("input cleanup failed"); };
+				},
+				getEditorText() { return ""; },
+				notify() {},
+				theme,
+			},
+		} as unknown as ExtensionContext;
+		const fleet = new SubagentFleetStatus(state, () => {}, { refreshMs: 60_000 });
+		fleet.setContext(ctx);
+		assert.throws(
+			() => fleet.dispose(),
+			(error: unknown) => error instanceof AggregateError
+				&& error.errors.map(String).join("\n").includes("input cleanup failed")
+				&& error.errors.map(String).join("\n").includes("widget cleanup failed"),
+		);
+	});
+
 	it("keeps widget ownership through invalidation so an empty refresh removes it", () => {
 		const state = stateForTest();
 		state.foregroundControls.set("run-worker", {
@@ -268,8 +414,8 @@ describe("below-editor subagent FleetView", () => {
 			startedAt: 100,
 			updatedAt: 200,
 			steps: [
-				{ agent: "reviewer", index: 0, status: "running", startedAt: 120, tokens: { input: 4_000, output: 200, total: 4_200 } },
-				{ agent: "worker", index: 1, status: "complete", tokens: { input: 100, output: 20, total: 120 } },
+				{ agent: "reviewer", index: 0, status: "running", description: "Review only authentication", startedAt: 120, model: "openai/gpt-5", thinking: "medium", tokens: { input: 4_000, output: 200, total: 4_200 } },
+				{ agent: "worker", index: 1, status: "running", description: "Implement only billing", startedAt: 121, tokens: { input: 100, output: 20, total: 120 } },
 			],
 		});
 		const fleet = new SubagentFleetStatus(state, () => {}, { refreshMs: 60_000 });
@@ -288,9 +434,10 @@ describe("below-editor subagent FleetView", () => {
 		try {
 			fleet.setContext(ctx);
 			const lines = widgetFactory!({ requestRender() {} }, theme).render(100);
-			assert.ok(lines.some((line) => line.includes("reviewer") && line.includes("Review the authentication changes")));
+			assert.ok(lines.some((line) => line.includes("reviewer (gpt-5 · thinking medium)") && line.includes("Review only authentication")));
+			assert.ok(lines.some((line) => line.includes("worker") && line.includes("Implement only billing")));
+			assert.ok(lines.every((line) => !line.includes("Review the authentication changes")), "per-child descriptions should replace the run-level fallback when present");
 			assert.ok(lines.some((line) => line.includes("↓ 4.2k tokens")));
-			assert.ok(!lines.some((line) => line.includes("worker")), "completed async children should leave the status fleet");
 		} finally {
 			fleet.dispose();
 		}
@@ -331,11 +478,32 @@ describe("below-editor subagent FleetView", () => {
 
 			assert.equal(inputHandler!("\x1b[B"), undefined, "non-empty editor should retain Down");
 			editorText = "";
-			tui.focusedComponent = {} as Editor;
+			tui.focusedComponent = {
+				render() { return []; },
+				invalidate() {},
+				handleInput() {},
+			} as unknown as Editor;
 			assert.equal(inputHandler!("\x1b[B"), undefined, "non-editor focus should retain Down");
+
+			const crossModuleCustomEditor = {
+				render() { return []; },
+				invalidate() {},
+				handleInput() {},
+				getText() { return ""; },
+				setText() {},
+			} satisfies EditorComponent;
+			assert.equal(crossModuleCustomEditor instanceof Editor, false, "regression setup must cross the instanceof boundary");
+			tui.focusedComponent = crossModuleCustomEditor as unknown as Editor;
+			assert.equal(inputHandler!("j"), undefined, "inactive FleetView should retain printable navigation keys");
+			assert.equal(inputHandler!("k"), undefined, "inactive FleetView should retain printable navigation keys");
+			assert.deepEqual(inputHandler!("\x1b[B"), { consume: true }, "custom editors should activate FleetView across jiti boundaries");
+			assert.deepEqual(inputHandler!("j"), { consume: true }, "active FleetView should navigate down with j");
+			assert.ok(component.render(100).some((line) => line.includes("⏺ worker")));
+			assert.deepEqual(inputHandler!("k"), { consume: true }, "active FleetView should navigate up with k");
+			assert.ok(component.render(100).some((line) => line.includes("⏺ main")));
+
 			tui.focusedComponent = Object.create(Editor.prototype) as Editor;
 			assert.deepEqual(inputHandler!("\x1b[B"), { consume: true });
-			assert.deepEqual(inputHandler!("\x1b[B"), { consume: true });
 			assert.ok(component.render(100).some((line) => line.includes("⏺ worker")));
 			assert.deepEqual(inputHandler!("\r"), { consume: true });
 			await Promise.resolve();