@selesai/code 0.13.12 → 0.13.14

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

@@ -3,7 +3,7 @@ import assert from "node:assert/strict";
 import * as fs from "node:fs";
 import * as path from "node:path";
 import type { MockPi } from "../support/helpers.ts";
-import { createEventBus, createMockPi, createTempDir, events, removeTempDir, tryImport } from "../support/helpers.ts";
+import { createEventBus, createMockPi, createTempDir, events, removeTempDir, resolveMockPiCallArgs, tryImport } from "../support/helpers.ts";
 import { discoverAgents } from "../../src/agents/agents.ts";
 import { ACTIVE_ASYNC_CAPACITY_DIR, acquireActiveAsyncCapacity, activeAsyncCapacitySessionKey } from "../../src/runs/background/active-async-capacity.ts";
 import { clearExclusions } from "../../src/runs/shared/model-exclusions.ts";
@@ -161,23 +161,23 @@ describe("fork context execution wiring", { skip: !available ? "subagent executo
 			.sort()
 			.at(-1);
 		assert.ok(callFile, "expected a recorded mock pi call");
-		return readRecordedArgs(callFile);
+		return readRecordedArgs(callFile, true);
 	}
 
 	function readAllCallArgs(): string[][] {
 		return fs.readdirSync(mockPi.dir)
 			.filter((name) => name.startsWith("call-") && name.endsWith(".json"))
 			.sort()
-			.map(readRecordedArgs);
+			.map((name) => readRecordedArgs(name));
 	}
 
-	function readRecordedArgs(callFile: string): string[] {
-		const payload = JSON.parse(fs.readFileSync(path.join(mockPi.dir, callFile), "utf-8"));
+	function readRecordedArgs(callFile: string, effective = false): string[] {
+		const payload = JSON.parse(fs.readFileSync(path.join(mockPi.dir, callFile), "utf-8")) as { args?: string[]; effectiveArgs?: string[] };
 		assert.equal(typeof payload, "object", "expected recorded args payload");
 		assert.notEqual(payload, null, "expected recorded args payload");
 		assert.ok("args" in payload, "expected recorded args payload");
 		assert.ok(Array.isArray(payload.args), "expected recorded args");
-		return payload.args;
+		return effective ? resolveMockPiCallArgs(payload) : payload.args;
 	}
 
 	function readSessionArgsFromCalls(): string[] {
@@ -192,14 +192,8 @@ describe("fork context execution wiring", { skip: !available ? "subagent executo
 			.filter((sessionFile): sessionFile is string => Boolean(sessionFile));
 	}
 
-	function readCallArgsForTask(taskText: string): string[] {
-		const args = readAllCallArgs().find((callArgs) => {
-			const prompt = callArgs.at(-1) ?? "";
-			return prompt.startsWith(`Task: ${taskText}\n`)
-				|| prompt.includes(`\n\nTask:\n${taskText}\n`);
-		});
-		assert.ok(args, `expected a recorded mock pi call for task '${taskText}'`);
-		return args;
+	function isTaskFileArg(arg: string): boolean {
+		return arg.startsWith("@") && arg.endsWith("task.md");
 	}
 
 	function readSessionArg(args: string[]): string {
@@ -313,8 +307,13 @@ 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"));
+		const args = readAllCallArgs()[0] ?? [];
+		const taskArg = args.at(-1) ?? "";
+		if (process.platform === "darwin") {
+			assert.ok(isTaskFileArg(taskArg));
+		} else {
+			assert.ok(taskArg.startsWith("Task: \n\n## Acceptance Contract"));
+		}
 	});
 
 	it("fails pruned fork model auth before child spawn", async () => {
@@ -1314,8 +1313,13 @@ 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`));
-		assert.ok(args, "expected a recorded mock pi call for this test task");
+		const args = readAllCallArgs()[0] ?? [];
+		const taskArg = args.at(-1) ?? "";
+		if (process.platform === "darwin") {
+			assert.ok(isTaskFileArg(taskArg));
+		} else {
+			assert.ok(taskArg.startsWith(`Task: ${task}\n\n## Acceptance Contract`));
+		}
 		const modelIndex = args.indexOf("--model");
 		assert.notEqual(modelIndex, -1);
 		assert.equal(args[modelIndex + 1], "anthropic/claude-haiku-4-5");

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

@@ -986,7 +986,6 @@ describe("intercom result delivery cutover", { skip: !available ? "executor not
 			assert.match(result.content[0]?.text ?? "", /Revived async subagent from/);
 			assert.match(result.content[0]?.text ?? "", /Do not run sleep timers or polling loops/);
 			assert.match(result.content[0]?.text ?? "", /Use bg_wait only/);
-			assert.doesNotMatch(result.content[0]?.text ?? "", /subagent_wait/);
 			assert.match(result.content[0]?.text ?? "", /Status if needed: subagent\(\{ action: "status"/);
 			assert.doesNotMatch(result.content[0]?.text ?? "", /Follow:/);
 			const revivedId = result.details?.asyncId;

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

@@ -307,9 +307,13 @@ describe("subagent async widget rendering", () => {
 		};
 
 		const text = buildWidgetLines([job], theme, 180).join("\n");
-		assert.match(text, /ci: CI checks · running · provider:local-tests · PR #1614/);
-		assert.match(text, /gate: Review gate · inconclusive · provider:opaque-provider · reason:stale-head · stale · out:gate.json/);
+		assert.match(text, /CI checks 1 active/);
+		assert.match(text, /Review gate 1 blocked/);
 		assert.doesNotMatch(text, /async subagent .*CI checks/);
+
+		const expanded = buildWidgetLines([job], theme, 180, true).join("\n");
+		assert.match(expanded, /ci: CI checks · running · provider:local-tests · PR #1614/);
+		assert.match(expanded, /gate: Review gate · inconclusive · provider:opaque-provider · reason:stale-head · stale · out:gate.json/);
 	});
 
 	it("keeps simple one-off async rows on the existing fallback projection", () => {
@@ -928,14 +932,14 @@ describe("subagent async widget rendering", () => {
 
 	it("keeps the active runs.lanes stage in focus before collapsed history", () => {
 		resetWidgetLayout();
-		withStdoutSize(60, 120, () => {
+		withStdoutSize(30, 120, () => {
 			const stageKeys = ["scope-scout", "red-tests", "label-helpers", "summary-title", "detail-row", "tiers-noise", "validation", "minimality-challenge", "fresh-review"];
 			const nodeIds = stageKeys.map((key) => `issue-1695.${key}`);
-			const statuses = stageKeys.map((_, index) => index < 3 ? "completed" : index === 3 ? "running" : "pending");
+			const statuses = stageKeys.map((_, index) => index < 7 ? "completed" : index === 7 ? "running" : "pending");
 			const workflowGraph = {
 				runId: "workflow-1695",
 				mode: "workflow",
-				phases: [{ title: "issue-1695 staged lane", nodeIds }],
+				phases: stageKeys.map((title, index) => ({ title, nodeIds: [nodeIds[index]!] })),
 				nodes: stageKeys.map((key, index) => ({
 					id: nodeIds[index],
 					kind: "step",
@@ -945,7 +949,7 @@ describe("subagent async widget rendering", () => {
 					flatIndex: index,
 					stepIndex: index,
 				})),
-				currentNodeId: nodeIds[3],
+				currentNodeId: nodeIds[7],
 			};
 			const job = {
 				asyncId: "workflow-1695",
@@ -953,11 +957,11 @@ describe("subagent async widget rendering", () => {
 				status: "running",
 				mode: "workflow",
 				agents: ["scout", "worker"],
-				currentStep: 3,
-				stepsTotal: 4,
+				currentStep: 7,
+				stepsTotal: stageKeys.length,
 				updatedAt: 200,
 				workflowGraph,
-				steps: stageKeys.slice(0, 4).map((key, index) => ({
+				steps: stageKeys.slice(0, 8).map((key, index) => ({
 					index,
 					agent: index === 0 ? "scout" : "worker",
 					status: statuses[index],
@@ -973,14 +977,13 @@ describe("subagent async widget rendering", () => {
 			renderWidget(ui.ctx as never, [job]);
 			const lines = renderWidgetLines(ui.widgets.at(-1));
 			const text = lines.join("\n");
-			const activeIndex = lines.findIndex((line) => line.includes("summary-title"));
-			const hiddenIndex = lines.findIndex((line) => /lines hidden/.test(line));
-			assert.ok(hiddenIndex >= 0, `collapsed history should expose a hidden-lines marker:\n${text}`);
-			assert.ok(activeIndex >= 0, `active stage should remain visible:\n${text}`);
-			assert.ok(activeIndex < hiddenIndex, `active stage must precede the hidden-lines marker:\n${text}`);
-			assert.match(text, /stage\s+4\/9/i);
-			assert.match(text, /summary-title task/);
-			assert.match(lines[activeIndex]!, /worker/);
+			assert.match(text, /7\/9 done · 1 active · 1 queued/);
+			assert.match(text, /minimality-challenge 1 active/);
+			assert.match(text, /fresh-review 1 queued/);
+			assert.match(text, /bottleneck · minimality-challenge/);
+			assert.match(text, /Press configured-expand-key for live detail/);
+			assert.doesNotMatch(text, /label-helpers/);
+			assert.doesNotMatch(text, /Step \d+\/9|task:|workspace:|out(?:put)?:/i);
 		});
 		resetWidgetLayout();
 	});
@@ -1018,7 +1021,7 @@ describe("subagent async widget rendering", () => {
 			stepsTotal: stages.length,
 			workflowGraph,
 			steps: stages,
-		}], theme, 220, false).join("\n");
+		}], theme, 220, true).join("\n");
 		const rowIndex = (label: string): number => text.split("\n").findIndex((line) => line.includes(label));
 		const completed = rowIndex("completed-history");
 		const failed = rowIndex("failed-stage");