@selesai/code 0.5.29 → 0.6.2

package/dist/extensions/pi-subagents/test/integration/chain-execution.test.ts

@@ -367,8 +367,9 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		assert.doesNotMatch(firstTaskArg, /\[Write to:|Write your findings to exactly this path/);
 		assert.ok(result.details.results[0]?.savedOutputPath);
 		assert.equal(fs.readFileSync(result.details.results[0].savedOutputPath, "utf-8"), "full chain output\nwith details");
-		assert.match(result.details.results[0]?.finalOutput ?? "", /Output saved to:/);
-		assert.doesNotMatch(result.details.results[0]?.finalOutput ?? "", /full chain output/);
+		assert.equal(result.details.results[0]?.finalOutput, undefined);
+		assert.match(result.details.results[0]?.outputReference?.message ?? "", /Output saved to:/);
+		assert.doesNotMatch(result.details.results[0]?.outputReference?.message ?? "", /full chain output/);
 		const secondTaskArg = readCallArgs(1).at(-1) ?? "";
 		assert.match(secondTaskArg, /Output saved to:/);
 		assert.match(secondTaskArg, /2 lines/);
@@ -404,7 +405,8 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		);
 
 		assert.ok(!result.isError, `chain should succeed: ${JSON.stringify(result.content)}`);
-		assert.match(result.details.results[0]?.finalOutput ?? "", /Output saved to:/);
+		assert.equal(result.details.results[0]?.finalOutput, undefined);
+		assert.match(result.details.results[0]?.outputReference?.message ?? "", /Output saved to:/);
 		assert.equal(result.details.results[0]?.acceptance?.status, "checked");
 		assert.ok(result.details.results[0]?.acceptance?.childReport);
 
@@ -630,10 +632,13 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 
 		assert.ok(!result.isError);
 		const step2Task = result.details.results[1].task;
+		// Reference-first handoff: {previous} carries the saved-output reference and
+		// the step reads the named path instead of re-inlined prose.
 		assert.ok(
-			step2Task.includes("MARKER_ABC_123"),
-			`step 2 task should contain step 1 output via {previous}: ${step2Task.slice(0, 200)}`,
+			step2Task.includes("Output saved to:"),
+			`step 2 task should contain the saved-output reference via {previous}: ${step2Task.slice(0, 200)}`,
 		);
+		assert.doesNotMatch(step2Task, /MARKER_ABC_123/);
 	});
 
 	it("passes named sequential outputs through {outputs.name}", async () => {
@@ -652,7 +657,8 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		);
 
 		assert.ok(!result.isError);
-		assert.match(readCallArgs(1).at(-1) ?? "", /CTX_123/);
+		assert.match(readCallArgs(1).at(-1) ?? "", /Output saved to: /);
+		assert.doesNotMatch(readCallArgs(1).at(-1) ?? "", /CTX_123/);
 		assert.equal(result.details.workflowGraph?.nodes[0]?.outputName, "contextOutput");
 	});
 
@@ -692,9 +698,10 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		assert.match(readCallArgs(1).at(-1) ?? "", /Review src\/a\.ts/);
 		assert.match(readCallArgs(2).at(-1) ?? "", /Review src\/b\.ts/);
 		assert.match(readCallArgs(3).at(-1) ?? "", /"key":"src\/a\.ts"/);
-		const collected = result.details.outputs?.reviews?.structured as Array<{ key: string; structured: unknown }>;
-		assert.deepEqual(collected.map((item) => item.key), ["src/a.ts", "src/b.ts"]);
-		assert.deepEqual(collected.map((item) => item.structured), [{ ok: "a" }, { ok: "b" }]);
+		// Terminal details strip chain outputs text/structured; bindings remain
+		// reference-first via {outputs.name} and the workflow graph keeps item keys.
+		assert.equal(result.details.outputs?.reviews?.text, "");
+		assert.equal(result.details.outputs?.reviews?.structured, undefined);
 		const dynamicNode = result.details.workflowGraph?.nodes[1];
 		assert.equal(dynamicNode?.kind, "dynamic-parallel-group");
 		assert.deepEqual(dynamicNode?.children?.map((child) => child.itemKey), ["src/a.ts", "src/b.ts"]);
@@ -868,7 +875,7 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 					{ agent: "scout", task: "Return targets", as: "targets", outputSchema: { type: "object" } },
 					{
 						expand: { from: { output: "targets", path: "/items" }, key: "/path", maxItems: 4 },
-						parallel: { agent: "reviewer", task: "Review {item.path}", outputMode: "file-only" },
+						parallel: { agent: "reviewer", task: "Review {item.path}", output: false, outputMode: "file-only" },
 						collect: { as: "reviews" },
 					},
 				],
@@ -906,7 +913,8 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 
 		assert.ok(!result.isError, `chain should succeed: ${JSON.stringify(result.content)}`);
 		assert.equal(mockPi.callCount(), 2);
-		assert.deepEqual(result.details.outputs?.reviews?.structured, []);
+		assert.equal(result.details.outputs?.reviews?.text, "");
+		assert.equal(result.details.outputs?.reviews?.structured, undefined);
 		assert.equal(result.details.workflowGraph?.nodes[1]?.status, "completed");
 		assert.deepEqual(result.details.workflowGraph?.nodes[1]?.children, []);
 	});
@@ -1123,7 +1131,9 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		assert.equal(result.details.results.length, 2);
 		assert.equal(result.details.results[0]?.acceptance?.status, "rejected");
 		assert.equal(result.details.results[0]?.exitCode, 0);
-		assert.match(result.details.results[1]?.finalOutput ?? "", /Step 2 ran/);
+		assert.equal(result.details.results[1]?.finalOutput, undefined);
+		assert.ok(result.details.results[1]?.savedOutputPath);
+		assert.equal(fs.readFileSync(result.details.results[1]!.savedOutputPath!, "utf-8"), "Step 2 ran");
 	});
 
 	it("agent contract v1 chain can gate progression on acceptance", async () => {
@@ -1195,7 +1205,8 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 		);
 
 		const finalTaskArg = readCallArgs(chainLength - 1).at(-1) ?? "";
-		assert.match(finalTaskArg, /step-38-output/);
+		assert.match(finalTaskArg, /Output saved to: /);
+		assert.doesNotMatch(finalTaskArg, /step-38-output/);
 		assert.doesNotMatch(finalTaskArg, /step-37-output/);
 		assert.match(result.content[0]?.text ?? "", /40 steps/);
 	});
@@ -1284,7 +1295,9 @@ describe("chain execution — sequential", { skip: !available ? "pi packages not
 			);
 
 			assert.ok(!result.isError);
-			assert.deepEqual(JSON.parse(result.details.results[0].finalOutput ?? "{}"), {
+			const depthChild = result.details.results[0];
+			assert.ok(depthChild.savedOutputPath);
+			assert.deepEqual(JSON.parse(fs.readFileSync(depthChild.savedOutputPath, "utf-8")), {
 				SELESAI_SUBAGENT_DEPTH: "1",
 				SELESAI_SUBAGENT_MAX_DEPTH: "1",
 			});
@@ -1474,8 +1487,9 @@ describe("chain execution — parallel steps", { skip: !available ? "pi packages
 
 		assert.ok(!result.isError);
 		const finalTask = readCallArgs(2).at(-1) ?? "";
-		assert.match(finalTask, /Alpha named output/);
-		assert.match(finalTask, /Beta named output/);
+		assert.match(finalTask, /Output saved to: /);
+		assert.doesNotMatch(finalTask, /Alpha named output/);
+		assert.doesNotMatch(finalTask, /Beta named output/);
 	});
 
 	it("funnels an initial parallel step through one agent, then fans the funnel output back out", async () => {
@@ -1512,13 +1526,14 @@ describe("chain execution — parallel steps", { skip: !available ? "pi packages
 		assert.equal(result.details.totalSteps, 3);
 		const funnelTask = readCallArgsMatching("Synthesize:").at(-1) ?? "";
 		assert.match(funnelTask, /=== Parallel Task 1 \(scout-a\) ===/);
-		assert.match(funnelTask, /Scout A findings/);
 		assert.match(funnelTask, /=== Parallel Task 2 \(scout-b\) ===/);
-		assert.match(funnelTask, /Scout B findings/);
+		assert.match(funnelTask, /Output saved to: /);
+		assert.doesNotMatch(funnelTask, /Scout A findings/);
+		assert.doesNotMatch(funnelTask, /Scout B findings/);
 		const fanoutTaskA = readCallArgsMatching("Review funnel A:").at(-1) ?? "";
 		const fanoutTaskB = readCallArgsMatching("Review funnel B:").at(-1) ?? "";
-		assert.match(fanoutTaskA, /Review funnel A:\nFunnel synthesis/);
-		assert.match(fanoutTaskB, /Review funnel B:\nFunnel synthesis/);
+		assert.match(fanoutTaskA, /Review funnel A:\nOutput saved to: /);
+		assert.match(fanoutTaskB, /Review funnel B:\nOutput saved to: /);
 		assert.equal(result.details.workflowGraph?.nodes[0]?.kind, "parallel-group");
 		assert.equal(result.details.workflowGraph?.nodes[1]?.kind, "step");
 		assert.equal(result.details.workflowGraph?.nodes[2]?.kind, "parallel-group");
@@ -1560,7 +1575,7 @@ describe("chain execution — parallel steps", { skip: !available ? "pi packages
 			makeChainParams(
 				[{
 					parallel: [
-						{ agent: "reviewer-a", task: "Review A", outputMode: "file-only" },
+						{ agent: "reviewer-a", task: "Review A", output: false, outputMode: "file-only" },
 						{ agent: "reviewer-b", task: "Review B", output: "b.md" },
 					],
 				}],

package/dist/extensions/pi-subagents/test/integration/fork-context-execution.test.ts

@@ -310,7 +310,8 @@ describe("fork context execution wiring", { skip: !available ? "subagent executo
 
 		assert.equal(result.isError, undefined);
 		const args = readCallArgs();
-		assert.ok((args.at(-1) ?? "").startsWith("Task: \n\n## Acceptance Contract"));
+		assert.ok((args.at(-1) ?? "").startsWith("Task: \n\n---\n**Output:**"));
+		assert.match(args.at(-1) ?? "", /## Acceptance Contract/);
 	});
 
 	it("does not treat top-level agent as single mode when tasks are present", async () => {
@@ -327,7 +328,8 @@ describe("fork context execution wiring", { skip: !available ? "subagent executo
 
 		assert.equal(result.isError, undefined);
 		const args = readCallArgs();
-		assert.ok((args.at(-1) ?? "").startsWith("Task: parallel task\n\n## Acceptance Contract"));
+		assert.ok((args.at(-1) ?? "").startsWith("Task: parallel task\n\n---\n**Output:**"));
+		assert.match(args.at(-1) ?? "", /## Acceptance Contract/);
 	});
 
 	it("uses agent defaultContext fork when launch context is omitted", async () => {
@@ -1827,7 +1829,7 @@ describe("fork context execution wiring", { skip: !available ? "subagent executo
 		);
 
 		assert.equal(result.isError, undefined);
-		const args = readAllCallArgs().find((callArgs) => (callArgs.at(-1) ?? "").startsWith(`Task: ${task}\n\n## Acceptance Contract`));
+		const args = readAllCallArgs().find((callArgs) => (callArgs.at(-1) ?? "").startsWith(`Task: ${task}\n\n---\n**Output:**`));
 		assert.ok(args, "expected a recorded mock pi call for this test task");
 		const modelIndex = args.indexOf("--model");
 		assert.notEqual(modelIndex, -1);

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

@@ -227,10 +227,13 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 		assert.match(result.content[0]?.text ?? "", /Delivered single subagent result via intercom\./);
 		assert.doesNotMatch(result.content[0]?.text ?? "", /Full child output from worker/);
 		assert.equal(result.details?.results?.[0]?.finalOutput, undefined);
-		assert.match(String(payload.message ?? ""), /Full child output from worker/);
+		// Reference-first intercom delivery: the child summary is the saved-output
+		// reference (error/status first for failed children), never raw child prose.
+		assert.match(String(payload.message ?? ""), /Output saved to: /);
+		assert.doesNotMatch(String(payload.message ?? ""), /Full child output from worker/);
 	});
 
-	it("falls back to legacy foreground output when the bridge is inactive", async () => {
+	it("returns the native saved-output receipt when the bridge is inactive", async () => {
 		mockPi.onCall({ output: "Legacy foreground output" });
 		const { executor, events } = makeExecutor({ bridgeMode: "off" });
 
@@ -243,10 +246,14 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 		);
 
 		assert.equal(events.emitted.some((entry) => entry.channel === "subagent:result-intercom"), false);
-		assert.match(result.content[0]?.text ?? "", /Legacy foreground output/);
+		assert.match(result.content[0]?.text ?? "", /Output saved to: /);
+		assert.doesNotMatch(result.content[0]?.text ?? "", /Legacy foreground output/);
+		const runId = result.details?.runId;
+		assert.ok(runId);
+		assert.equal(fs.readFileSync(path.join(tempDir, ".pi-subagents", "artifacts", "outputs", runId, "result.md"), "utf-8"), "Legacy foreground output");
 	});
 
-	it("keeps native foreground output without attempting external grouped delivery when disabled", async () => {
+	it("keeps the native saved-output receipt without attempting external grouped delivery when disabled", async () => {
 		mockPi.onCall({ output: "Native foreground output" });
 		const { executor, events } = makeExecutor({ resultDelivery: false });
 
@@ -259,10 +266,11 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 		);
 
 		assert.equal(events.emitted.some((entry) => entry.channel === "subagent:result-intercom"), false);
-		assert.match(result.content[0]?.text ?? "", /Native foreground output/);
+		assert.match(result.content[0]?.text ?? "", /Output saved to: /);
+		assert.doesNotMatch(result.content[0]?.text ?? "", /Native foreground output/);
 	});
 
-	it("falls back to legacy foreground output when grouped delivery is not acknowledged", async () => {
+	it("falls back to the compact native saved-output receipt when grouped delivery is not acknowledged", async () => {
 		mockPi.onCall({ output: "Unacknowledged foreground output" });
 		const { executor, events } = makeExecutor({ acknowledgeResults: false });
 
@@ -275,7 +283,8 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 		);
 
 		assert.equal(events.emitted.some((entry) => entry.channel === "subagent:result-intercom"), true);
-		assert.match(result.content[0]?.text ?? "", /Unacknowledged foreground output/);
+		assert.match(result.content[0]?.text ?? "", /Output saved to: /);
+		assert.doesNotMatch(result.content[0]?.text ?? "", /Unacknowledged foreground output/);
 	});
 
 	it("top-level parallel runs emit one grouped event containing all children", async () => {
@@ -1250,7 +1259,10 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 		const payload = completion.payload as { success?: boolean; summary?: string };
 		assert.equal(payload.success, false);
 		assert.match(payload.summary ?? "", /Acceptance rejected/);
-		assert.match(payload.summary ?? "", /final answer with rejected acceptance evidence/);
+		// Reference-first foreground completion: the recovery summary carries the
+		// saved-output reference (with error/status), never raw child prose.
+		assert.match(payload.summary ?? "", /Output saved to: /);
+		assert.doesNotMatch(payload.summary ?? "", /final answer with rejected acceptance evidence/);
 
 		const status = await executor.execute(
 			"foreground-detached-failed-status",

package/dist/extensions/pi-subagents/test/integration/parallel-execution.test.ts

@@ -287,7 +287,9 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 			groups: Array<{ children: Array<{ agent: string; summary: string; patch: { path: string } }>; cleanup: { state: string; tasks: Array<{ path: string; worktreeRemoved: boolean; branchRemoved: boolean }> } }>;
 		};
 		assert.equal(handoff.groups[0]!.children[0]!.agent, "echo");
-		assert.equal(handoff.groups[0]!.children[0]!.summary, "Worktree task complete");
+		// Reference-first handoff: the summary carries the saved-output reference.
+		assert.match(handoff.groups[0]!.children[0]!.summary, /Output saved to: /);
+		assert.doesNotMatch(handoff.groups[0]!.children[0]!.summary, /Worktree task complete/);
 		assert.equal(fs.existsSync(handoff.groups[0]!.children[0]!.patch.path), true);
 		assert.ok(result.details?.runId);
 		assert.ok(handoff.groups[0]!.children[0]!.patch.path.includes(`${path.sep}worktree-diffs${path.sep}${result.details.runId}${path.sep}`));
@@ -351,7 +353,8 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 
 			assert.equal(result.isError, undefined);
 			assert.equal(result.details?.mode, "parallel");
-			assert.match(result.content[0]?.text ?? "", new RegExp(`${action} alias finished`));
+			assert.match(result.content[0]?.text ?? "", /Output saved to: /);
+			assert.doesNotMatch(result.content[0]?.text ?? "", new RegExp(`${action} alias finished`));
 		}
 	});
 
@@ -465,7 +468,9 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 		assert.equal(result.details?.results?.[0]?.timedOut, true);
 		assert.equal(result.details?.results?.[0]?.error, "Subagent timed out after 300ms.");
 		assert.equal(result.details?.results?.[1]?.exitCode, 0);
-		assert.equal(result.details?.results?.[1]?.finalOutput, "fast done");
+		assert.equal(result.details?.results?.[1]?.finalOutput, undefined);
+		assert.ok(result.details?.results?.[1]?.savedOutputPath);
+		assert.equal(fs.readFileSync(result.details?.results?.[1]?.savedOutputPath as string, "utf-8"), "fast done");
 		assert.match(result.content[0]?.text ?? "", /1\/2 succeeded/);
 		assert.match(result.content[0]?.text ?? "", /TIMED OUT: Subagent timed out after 300ms\./);
 	});
@@ -490,8 +495,9 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 		assert.match(text, /Output saved to:/);
 		assert.match(text, /2 lines/);
 		assert.doesNotMatch(text, /Parallel full report/);
-		assert.match(result.details?.results?.[0]?.finalOutput ?? "", /Output saved to:/);
-		assert.doesNotMatch(result.details?.results?.[0]?.finalOutput ?? "", /Parallel full report/);
+		assert.equal(result.details?.results?.[0]?.finalOutput, undefined);
+		assert.match(result.details?.results?.[0]?.outputReference?.message ?? "", /Output saved to:/);
+		assert.doesNotMatch(result.details?.results?.[0]?.outputReference?.message ?? "", /Parallel full report/);
 		assert.equal(fs.readFileSync(outputPath, "utf-8"), "Parallel full report\nwith details");
 	});
 
@@ -500,7 +506,7 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 
 		const result = await executor.execute(
 			"parallel-file-only-missing-output",
-			{ tasks: [{ agent: "echo", task: "Write report", outputMode: "file-only" }] },
+			{ tasks: [{ agent: "echo", task: "Write report", output: false, outputMode: "file-only" }] },
 			new AbortController().signal,
 			undefined,
 			makeMinimalCtx(tempDir),
@@ -604,7 +610,8 @@ describe("parallel agent execution", { skip: !piAvailable ? "pi packages not ava
 
 Inspect
 
-## Acceptance Contract`));
+---\n**Output:**`));
+		assert.match(taskArg, /## Acceptance Contract/);
 	});
 
 	it("top-level parallel defaultProgress uses isolated run storage", { skip: !createSubagentExecutor ? "executor not importable" : undefined }, async () => {

package/dist/extensions/pi-subagents/test/integration/render-fork-badge.test.ts

@@ -1,4 +1,7 @@
 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 { keyText } from "@selesai/code";
 
@@ -852,4 +855,228 @@ describe("renderSubagentResult fork indicator", () => {
 		assert.match(text, /Step 1: scout/);
 		assert.doesNotMatch(text, /parallel group:/);
 	});
+
+	it("keeps settled expanded file-only results reference-first without re-inlining saved child prose", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPath = path.join(tempDir, "child-output.md");
+			const childProse = Array.from({ length: 40 }, (_, index) => `SENSITIVE FULL CHILD OUTPUT line ${index + 1}`).join("\n");
+			fs.writeFileSync(outputPath, childProse, "utf-8");
+			const bytes = Buffer.byteLength(childProse, "utf-8");
+			const referenceMessage = `Output saved to: ${outputPath} (${bytes} B, 40 lines). Read this file if needed.`;
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "single",
+					results: [{
+						agent: "writer",
+						task: "Write the report",
+						exitCode: 0,
+						messages: [],
+						usage: emptyUsage,
+						outputMode: "file-only",
+						savedOutputPath: outputPath,
+						outputReference: { path: outputPath, bytes, lines: 40, message: referenceMessage },
+						sessionFile: path.join(tempDir, "session.jsonl"),
+						artifactPaths: { outputPath, dir: tempDir },
+				}],
+				},
+			}, { expanded: true }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.doesNotMatch(text, /SENSITIVE FULL CHILD OUTPUT/);
+			assert.match(text, /Output saved to:/);
+			assert.match(text, new RegExp(outputPath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")));
+			assert.match(text, /Session:/);
+			assert.match(text, /Artifacts:/);
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
+	it("keeps explicit inline output and live progress rendering in expanded results", () => {
+		const inline = renderSubagentResult!({
+			content: [{ type: "text", text: "done" }],
+			details: {
+				mode: "single",
+				results: [{
+					agent: "writer",
+					task: "Inline task",
+					exitCode: 0,
+					messages: [],
+					usage: emptyUsage,
+					outputMode: "inline",
+					finalOutput: "FULL INLINE RESULT TEXT",
+				}],
+			},
+		}, { expanded: true }, theme).render(200).join("\n");
+		assert.match(inline, /FULL INLINE RESULT TEXT/);
+
+		const running = renderSubagentResult!({
+			content: [{ type: "text", text: "running" }],
+			details: {
+				mode: "single",
+				results: [{
+					agent: "reviewer",
+					task: "review",
+					exitCode: 0,
+					messages: [],
+					usage: emptyUsage,
+					progress: { index: 0, agent: "reviewer", status: "running", task: "review", recentTools: [], recentOutput: [], toolCount: 1, tokens: 42, durationMs: 3_000 },
+				}],
+			},
+		}, { expanded: true }, theme).render(200).join("\n");
+		assert.match(running, /running/);
+		assert.match(running, /for live detail/);
+	});
+
+	it("synthesizes a path-only output reference for legacy savedOutputPath results without reading the file", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPath = path.join(tempDir, "legacy-child-output.md");
+			// The file does not exist on disk: the renderer must never read it.
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "single",
+					results: [{
+						agent: "writer",
+						task: "Write the report",
+						exitCode: 0,
+						messages: [],
+						usage: emptyUsage,
+						savedOutputPath: outputPath,
+					}],
+				},
+			}, { expanded: true }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.match(text, new RegExp(`Output saved to: ${outputPath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`));
+			assert.match(text, /Read this file if needed/);
+			assert.doesNotMatch(text, /\(\d+ B, \d+ lines\)/);
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
+	it("keeps foreign savedOutputPath results reference-first even when they carry inline text", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPath = path.join(tempDir, "foreign-child-output.md");
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "single",
+					results: [{
+						agent: "writer",
+						task: "Write the report",
+						exitCode: 0,
+						messages: [],
+						usage: emptyUsage,
+						savedOutputPath: outputPath,
+						finalOutput: "SENSITIVE FOREIGN CHILD PROSE that must not be inlined",
+					}],
+				},
+			}, { expanded: true }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.doesNotMatch(text, /SENSITIVE FOREIGN CHILD PROSE/);
+			assert.match(text, new RegExp(`Output saved to: ${outputPath.replace(/[.*+?^${}()|[\]\\]/g, "\\$&")}`));
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
+	it("keeps compact file-only results reference-first with the saved-output reference", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPath = path.join(tempDir, "child-output.md");
+			const childProse = Array.from({ length: 40 }, (_, index) => `SENSITIVE FULL CHILD OUTPUT line ${index + 1}`).join("\n");
+			fs.writeFileSync(outputPath, childProse, "utf-8");
+			const bytes = Buffer.byteLength(childProse, "utf-8");
+			const referenceMessage = `Output saved to: ${outputPath} (${bytes} B, 40 lines). Read this file if needed.`;
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "single",
+					results: [{
+						agent: "writer",
+						task: "Write the report",
+						exitCode: 0,
+						messages: [],
+						usage: emptyUsage,
+						outputMode: "file-only",
+						savedOutputPath: outputPath,
+						outputReference: { path: outputPath, bytes, lines: 40, message: referenceMessage },
+					}],
+				},
+			}, { expanded: false }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.doesNotMatch(text, /SENSITIVE FULL CHILD OUTPUT/);
+			assert.match(text, /Output saved to:/);
+			assert.doesNotMatch(text, /no text output/);
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
+	it("keeps compact multi file-only results reference-first and success-marked", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPathA = path.join(tempDir, "a.md");
+			const outputPathB = path.join(tempDir, "b.md");
+			fs.writeFileSync(outputPathA, "SENSITIVE CHILD OUTPUT A", "utf-8");
+			fs.writeFileSync(outputPathB, "SENSITIVE CHILD OUTPUT B", "utf-8");
+			const refA = { path: outputPathA, bytes: 24, lines: 1, message: `Output saved to: ${outputPathA} (24 B, 1 line). Read this file if needed.` };
+			const refB = { path: outputPathB, bytes: 24, lines: 1, message: `Output saved to: ${outputPathB} (24 B, 1 line). Read this file if needed.` };
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "parallel",
+					totalSteps: 2,
+					results: [
+						{ agent: "scout", task: "scan", exitCode: 0, messages: [], usage: emptyUsage, outputMode: "file-only", savedOutputPath: outputPathA, outputReference: refA },
+						{ agent: "reviewer", task: "review", exitCode: 0, messages: [], usage: emptyUsage, outputMode: "file-only", savedOutputPath: outputPathB, outputReference: refB },
+					],
+				},
+			}, { expanded: false }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.doesNotMatch(text, /SENSITIVE CHILD OUTPUT/);
+			assert.doesNotMatch(text, /no text output/);
+			assert.match(text, /^✓ parallel/);
+			assert.match(text, /Agent 1\/2: scout/);
+			assert.match(text, /Agent 2\/2: reviewer/);
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
+
+	it("marks legacy compact multi savedOutputPath results as done without inlining child prose", () => {
+		const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), "pi-subagents-render-"));
+		try {
+			const outputPath = path.join(tempDir, "legacy.md");
+			const widget = renderSubagentResult!({
+				content: [{ type: "text", text: "done" }],
+				details: {
+					mode: "parallel",
+					totalSteps: 2,
+					results: [
+						{ agent: "scout", task: "scan", exitCode: 0, messages: [], usage: emptyUsage, savedOutputPath: outputPath, finalOutput: "LEGACY SENSITIVE PROSE" },
+						{ agent: "writer", task: "Write the report", exitCode: 0, messages: [], usage: emptyUsage, savedOutputPath: outputPath, finalOutput: "MORE LEGACY SENSITIVE PROSE" },
+					],
+				},
+			}, { expanded: false }, theme);
+
+			const text = widget.render(200).join("\n");
+			assert.doesNotMatch(text, /LEGACY SENSITIVE PROSE|MORE LEGACY SENSITIVE PROSE/);
+			assert.doesNotMatch(text, /no text output/);
+			assert.match(text, /^✓ parallel/);
+			assert.match(text, /Agent 1\/2: scout/);
+			assert.match(text, /Agent 2\/2: writer/);
+		} finally {
+			fs.rmSync(tempDir, { recursive: true, force: true });
+		}
+	});
 });