@selesai/code 0.12.0 → 0.13.1

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

@@ -39,13 +39,13 @@ function job(child: NestedRunSummary): AsyncJobState {
 
 describe("nested widget rendering", () => {
 	it("renders a bounded collapsed tree and full child rows when expanded", () => {
-		const child = nested("nested-reviewer", "root-run", "running", { currentTool: "read", model: "gpt-5.6-luna:medium", thinking: "medium" });
+		const child = nested("nested-reviewer", "root-run", "running", { sessionName: "  nested-reviewer: Review nested run  ", currentTool: "read", model: "gpt-5.6-luna:medium", thinking: "medium" });
 		const collapsed = buildWidgetLines([job(child)], theme as any, 120, false).join("\n");
-		assert.match(collapsed, /↳ └─ \[\d{2}:\d{2}:\d{2}\] . nested-reviewer · running · gpt-5.6-luna · thinking medium · read/);
+		assert.match(collapsed, /↳ └─ \[\d{2}:\d{2}:\d{2}\] . nested-reviewer: Review nested run · running · gpt-5.6-luna · thinking medium · read/);
 		assert.equal((collapsed.match(/thinking medium/g) ?? []).length, 1);
 
 		const expanded = buildWidgetLines([job(child)], theme as any, 120, true).join("\n");
-		assert.match(expanded, /↳ \[\d{2}:\d{2}:\d{2}\] . nested-reviewer · running · gpt-5.6-luna · thinking medium · read/);
+		assert.match(expanded, /↳ \[\d{2}:\d{2}:\d{2}\] . nested-reviewer: Review nested run · running · gpt-5.6-luna · thinking medium · read/);
 
 		const epoch = buildWidgetLines([job(nested("epoch", "root-run", "running", { lastUpdate: 0, startedAt: 0 }))], theme as any, 120, false).join("\n");
 		assert.match(epoch, /↳ └─ \[\d{2}:\d{2}:\d{2}\] . epoch · running/);
@@ -122,7 +122,7 @@ describe("nested widget rendering", () => {
 	it("timestamps every nested lifecycle state and completed steps", () => {
 		const states: NestedRunSummary["state"][] = ["queued", "running", "complete", "failed", "paused", "stopped"];
 		const root = nested("matrix-root", "root-run", "running", {
-			steps: [{ agent: "completed-step", status: "completed", endedAt: 2_000 }],
+			steps: [{ agent: "completed-step", sessionName: "  completed-step: Finalize report  ", status: "completed", endedAt: 2_000 }],
 			children: states.map((state, index) => nested(`child-${state}`, "matrix-root", state, {
 				parentStepIndex: undefined,
 				startedAt: 1_000 + index,
@@ -133,7 +133,7 @@ describe("nested widget rendering", () => {
 		for (const state of states) {
 			assert.match(expanded, new RegExp(`\\[\\d{2}:\\d{2}:\\d{2}\\] . child-${state} · ${state}`));
 		}
-		assert.match(expanded, /\[\d{2}:\d{2}:\d{2}\] . completed-step · completed/);
+		assert.match(expanded, /\[\d{2}:\d{2}:\d{2}\] . completed-step: Finalize report · completed/);
 	});
 
 	it("keeps event-time timestamps stable instead of advancing with wall time", async () => {
@@ -147,6 +147,8 @@ describe("nested widget rendering", () => {
 	it("rerenders when only nested state changes", () => {
 		const first = job(nested("nested-reviewer", "root-run", "running"));
 		const second = job(nested("nested-reviewer", "root-run", "complete"));
+		const renamed = job(nested("nested-reviewer", "root-run", "running", { sessionName: "nested-reviewer: Review nested run" }));
 		assert.notEqual(widgetRenderKey(first), widgetRenderKey(second));
+		assert.notEqual(widgetRenderKey(first), widgetRenderKey(renamed));
 	});
 });

package/dist/extensions/pi-subagents/test/unit/workflow-chat-progress.test.ts

@@ -297,6 +297,112 @@ describe("workflow chat progress rendering", () => {
 		assert.doesNotMatch(text, /failed\s+detaches/);
 	});
 
+	it("seeds declared preflight lanes as planned before workflow trace entries", () => {
+		const preflight = {
+			version: 1 as const,
+			coverage: "partial" as const,
+			lanes: [
+				{ key: "writer", mode: "mutation" as const, claims: ["src/workflows"] },
+				{ key: "review", mode: "review" as const, expectedOutput: "review.md" },
+			],
+		};
+		assert.deepEqual(buildWorkflowChatProgressRows([], preflight).map((row) => ({ key: row.key, state: row.state })), [
+			{ key: "writer", state: "planned" },
+			{ key: "review", state: "planned" },
+		]);
+		const text = componentText(renderSubagentResult({
+			content: [{ type: "text", text: "Workflow running." }],
+			details: {
+				mode: "workflow",
+				runId: "wf_planned",
+				results: [],
+				preflight,
+				chatProgress: { mode: "live-card", repoRelation: "same", repoLabel: "pi-subagents" },
+				workflow: { trace: [], emits: [], console: [] },
+			},
+		}, { expanded: false }, theme as any));
+		assert.match(text, /planned\s+writer/);
+		assert.match(text, /planned\s+review/);
+		assert.doesNotMatch(text, /mode:mutation/);
+		assert.doesNotMatch(text, /expected:review\.md/);
+		assert.doesNotMatch(text, /waiting for workflow child launches/);
+
+		const expanded = componentText(renderSubagentResult({
+			content: [{ type: "text", text: "Workflow running." }],
+			details: {
+				mode: "workflow",
+				runId: "wf_planned",
+				results: [],
+				preflight,
+				chatProgress: { mode: "live-card", repoRelation: "same", repoLabel: "pi-subagents" },
+				workflow: { trace: [], emits: [], console: [] },
+			},
+		}, { expanded: true }, theme as any));
+		assert.match(expanded, /mode:mutation/);
+		assert.match(expanded, /expected:review\.md/);
+	});
+
+	it("uses the compact plan preview for collapsed workflow launch output", () => {
+		const preflight = {
+			version: 1 as const,
+			coverage: "complete" as const,
+			lanes: [{ key: "writer", mode: "mutation" as const, decision: "Implement the change" }],
+		};
+		const output = "Preflight: v1 · complete · 1 lane\n  key | mode | decision | claims | expected output | independence\n  writer | mutation | Implement the change | — | — | —\n\nAsync workflow [wf_launch] started.";
+		const result = {
+			content: [{ type: "text" as const, text: output }],
+			details: { mode: "workflow" as const, runId: "wf_launch", results: [], preflight },
+		};
+		const compact = componentText(renderSubagentResult(result, { expanded: false }, theme as any));
+		assert.match(compact, /Plan: 1 lane · Implement the change/);
+		assert.doesNotMatch(compact, /Preflight: v1/);
+		assert.doesNotMatch(compact, /key \| mode \| decision/);
+
+		const expanded = componentText(renderSubagentResult(result, { expanded: true }, theme as any));
+		assert.match(expanded, /Preflight: v1 · complete · 1 lane/);
+		assert.match(expanded, /key \| mode \| decision/);
+	});
+
+	it("keeps each expanded preflight warning visible as a bounded row", () => {
+		const result = {
+			content: [{ type: "text" as const, text: "Workflow running." }],
+			details: {
+				mode: "workflow" as const,
+				runId: "wf_warning_rows",
+				results: [],
+				preflight: { version: 1 as const, coverage: "complete" as const, lanes: [{ key: "writer", mode: "mutation" as const }] },
+				chatProgress: { mode: "live-card" as const, repoRelation: "same" as const, repoLabel: "pi-subagents" },
+				workflow: {
+					trace: [{ operation: "run" as const, key: "writer", state: "started" as const }],
+					emits: [],
+					console: [],
+					preflightWarnings: [
+						`Preflight advisory: first warning ${"x".repeat(120)}`,
+						`Preflight advisory: second warning ${"y".repeat(120)}`,
+					],
+				},
+			},
+		};
+		const originalColumns = Object.getOwnPropertyDescriptor(process.stdout, "columns");
+		Object.defineProperty(process.stdout, "columns", { configurable: true, value: 56 });
+		try {
+			const expanded = renderSubagentResult(result, { expanded: true }, theme as any).render(56);
+			assert.ok(expanded.some((line) => line.includes("Preflight warnings:")));
+			const warningRows = expanded.filter((line) => line.includes("- Preflight advisory"));
+			assert.equal(warningRows.length, 2);
+			assert.match(warningRows[0]!, /first warning/);
+			assert.match(warningRows[1]!, /second warning/);
+			assert.ok(warningRows.every((line) => line.trimEnd().length <= 52));
+
+			const compact = renderSubagentResult(result, { expanded: false }, theme as any).render(56);
+			assert.equal(compact.filter((line) => line.includes("Plan note:")).length, 1);
+			assert.doesNotMatch(compact.join("\n"), /first warning|second warning/);
+		} finally {
+			if (originalColumns) Object.defineProperty(process.stdout, "columns", originalColumns);
+			else delete (process.stdout as { columns?: number }).columns;
+		}
+	});
+
 	it("keeps mixed detached and failed workflow traces failed", () => {
 		const text = componentText(renderSubagentResult({
 			content: [{ type: "text", text: "Workflow failed: child failed after a detached sibling." }],