@selesai/code 0.8.6 → 0.8.7

package/dist/extensions/pi-subagents/test/unit/workflow-launch-params.test.ts

@@ -39,6 +39,26 @@ describe("workflow launch params", () => {
 		);
 	});
 
+	it("keeps a bridge override scoped to the target workflow child", () => {
+		assert.deepEqual(
+			prepareWorkflowLaunchParams(
+				{},
+				{ agent: "worker", task: "Run", intercomBridge: { mode: "off" } },
+				"workflow-run",
+				"isolated",
+			),
+			{
+				agent: "worker",
+				task: "Run",
+				intercomBridge: { mode: "off" },
+				async: false,
+				workflowParentRunId: "workflow-run",
+				workflowKey: "isolated",
+			},
+		);
+		assert.equal(prepareWorkflowLaunchParams({}, { agent: "worker", task: "Run" }, "workflow-run", "sibling").intercomBridge, undefined);
+	});
+
 	it("places workflow child gates inside managed worktree tasks", () => {
 		assert.deepEqual(
 			prepareWorkflowLaunchParams(
@@ -61,6 +81,25 @@ describe("workflow launch params", () => {
 		);
 	});
 
+	it("preserves a bridge override for retained workflow children", () => {
+		assert.deepEqual(
+			prepareWorkflowLaunchParams(
+				{},
+				{ resume: "retained-run", task: "Continue", intercomBridge: { mode: "off" } },
+				"workflow-run",
+				"continue",
+			),
+			{
+				action: "resume",
+				id: "retained-run",
+				message: "Continue",
+				workflowParentRunId: "workflow-run",
+				workflowKey: "continue",
+				intercomBridge: { mode: "off" },
+			},
+		);
+	});
+
 	it("rejects gate defaults on retained resume items", () => {
 		assert.throws(
 			() => prepareWorkflowLaunchParams(
@@ -82,7 +121,7 @@ describe("workflow launch params", () => {
 		);
 	});
 
-	it("preserves execution limits when routing retained resume items", () => {
+	it("preserves execution limits and fan-out identity when routing retained resume items", () => {
 		assert.deepEqual(
 			prepareWorkflowLaunchParams(
 				{ turnBudget: { maxTurns: 8 }, toolBudget: { hard: 12, block: ["read"] } },
@@ -95,7 +134,7 @@ describe("workflow launch params", () => {
 				},
 				"workflow-run",
 				"continue",
-				{ missionDetached: true },
+				{ missionDetached: true, runFanoutBudget: { version: 1, rootRunId: "root-run", directory: "/tmp/fanout", limit: 64, parentPath: "parent" } },
 			),
 			{
 				action: "resume",
@@ -103,6 +142,7 @@ describe("workflow launch params", () => {
 				message: "Continue carefully",
 				workflowParentRunId: "workflow-run",
 				workflowKey: "continue",
+				runFanoutBudget: { version: 1, rootRunId: "root-run", directory: "/tmp/fanout", limit: 64, parentPath: "parent/workflow[continue]" },
 				mission: false,
 				timeoutMs: 5_000,
 				turnBudget: { maxTurns: 3, graceTurns: 1 },

package/dist/extensions/question/index.ts

@@ -120,12 +120,13 @@ class QuestionComponent extends Container implements Component {
 		this.rebuildMode();
 	}
 
+	// Only the wizard-level focused state is read; the inner getter is kept for
+	// API symmetry but never invoked.
+	/* v8 ignore start */
 	get focused(): boolean {
-		// Only the wizard-level focused state is read; the inner getter is kept
-		// for API symmetry but never invoked.
-		/* v8 ignore next 1 */
 		return this.focusedState;
 	}
+	/* v8 ignore stop */
 	set focused(value: boolean) {
 		this.focusedState = value;
 		if (this.editor) this.editor.focused = value && this.editorActive;
@@ -219,6 +220,9 @@ class QuestionComponent extends Container implements Component {
 	}
 
 	private showSelect(): void {
+		// showSelect is only invoked from other mode (the escape handler), so the
+		// mode guard's false branch is unreachable through the public flow.
+		/* v8 ignore next 1 */
 		if (this.mode === "other") this.otherDraft = this.ensureEditor().getText();
 		this.mode = "select";
 		this.editorActive = false;
@@ -231,6 +235,10 @@ class QuestionComponent extends Container implements Component {
 			this.modeContainer.addChild(this.ensureList());
 		} else {
 			const editor = this.ensureEditor();
+			// editorActive is always true in this branch: rebuildMode runs from the
+			// constructor, showOther (sets editorActive = true), and showSelect
+			// (which takes the select branch above).
+			/* v8 ignore next 1 */
 			editor.focused = this.focusedState && this.editorActive;
 			this.modeContainer.addChild(new Text(this.theme.fg("accent", this.theme.bold(this.mode === "text" ? "Answer" : "Other answer")), 1, 0));
 			this.modeContainer.addChild(new Spacer(1));
@@ -317,12 +325,13 @@ class ReviewComponent implements Component {
 		private onSubmit: () => void,
 	) {}
 
+	// Only the setter is used by the wizard; the getter is kept for the
+	// Component contract but never read.
+	/* v8 ignore start */
 	get focused(): boolean {
-		// Only the setter is used by the wizard; the getter is kept for the
-		// Component contract but never read.
-		/* v8 ignore next 1 */
 		return this.focusedState;
 	}
+	/* v8 ignore stop */
 	set focused(value: boolean) {
 		this.focusedState = value;
 	}
@@ -469,6 +478,9 @@ class BatchQuestionComponent extends Container implements Component {
 
 	private goNext(): void {
 		this.saveCurrent(true);
+		// goNext only runs on question pages, where currentPage is always below
+		// questions.length; the guard is defensive.
+		/* v8 ignore next 1 */
 		if (this.currentPage < this.questions.length) this.currentPage++;
 		this.showPage();
 	}
@@ -479,6 +491,9 @@ class BatchQuestionComponent extends Container implements Component {
 			this.onDone({ status: "submitted", answers: buildQuestionAnswers(this.questions, this.states) });
 			return;
 		}
+		// commitCurrent only runs on question pages, where currentPage is always
+		// below questions.length; the guard is defensive.
+		/* v8 ignore next 1 */
 		if (this.currentPage < this.questions.length) this.currentPage++;
 		this.showPage();
 	};

package/dist/extensions/question/question-list.ts

@@ -256,6 +256,9 @@ export class QuestionList implements Component {
 		for (const line of wrapTextWithAnsi(md.trim(), Math.max(10, width))) {
 			lines.push(truncateToWidth(line, width, ""));
 		}
+		// md is always trimmed, and wrapTextWithAnsi never emits a trailing
+		// empty line for trimmed input; the pop is defensive.
+		/* v8 ignore next 1 */
 		while (lines.length > 0 && lines[lines.length - 1]?.trim() === "") lines.pop();
 		if (lines.length <= maxLines) return lines;
 		if (maxLines === 1) return [truncateToWidth(this.theme.fg("dim", "…"), width, "")];
@@ -333,8 +336,8 @@ export class QuestionList implements Component {
 			const result = filtered[this.selectedIndex]?.value;
 			// Confirm only runs when count > 0 and the selected row is an option;
 			// a freeform/comment row is dispatched above, so result is never falsy.
+			/* v8 ignore next 2 */
 			if (result) this.onSubmit?.([result], this.commentEnabled);
-			/* v8 ignore next 1 */
 			else this.onCancel?.();
 			return;
 		}
@@ -398,6 +401,9 @@ export class QuestionList implements Component {
 				this.onEnterFreeform?.();
 				return;
 			}
+			// Comment and freeform rows are intercepted above, so selectedIndex
+			// can only point at a real option here; the guard is defensive.
+			/* v8 ignore next 1 */
 			if (this.selectedIndex < this.options.length) {
 				this.selectionMode.toggle(this.checked, this.selectedIndex);
 				this.invalidate();

package/dist/extensions/question/row-layout.ts

@@ -125,6 +125,9 @@ export function renderSingleSelectRows({
 
 	const safeMaxRows = Math.max(1, Math.floor(maxRows));
 	const selectedBlock = blocks[selectedIndex] ?? blocks[0];
+	// Blocks is never empty here: empty blocks means empty allRows, which is
+	// already returned by the allRows.length <= maxRows guard above.
+	/* v8 ignore next 1 */
 	if (!selectedBlock) return [];
 
 	const indicator = `  (${selectedIndex + 1}/${itemCount})`;

package/dist/extensions/question/tests/exp.test.ts

@@ -0,0 +1,70 @@
+import { describe, expect, it } from "vitest";
+
+import questionExtension from "../index.ts";
+
+const theme = {
+	fg: (_name: string, value: string) => value,
+	bg: (_name: string, value: string) => value,
+	bold: (value: string) => value,
+};
+const tui = { requestRender() {}, terminal: { rows: 24 } };
+const keybindings = {
+	matches: (data: string, action: string) =>
+		(data === "\r" && action === "tui.select.confirm") ||
+		(data === "\x1b" && action === "tui.select.cancel"),
+	getKeys: () => ["Enter"],
+};
+
+async function runWizard(params: unknown) {
+	let tool: any;
+	let component: any;
+	let done!: (value: unknown) => void;
+	questionExtension({ registerTool: (entry: unknown) => (tool = entry), events: { emit() {} } } as any);
+	const execution = tool.execute("call", params, undefined, undefined, {
+		mode: "tui",
+		ui: {
+			custom: (factory: (...args: any[]) => unknown) => new Promise((resolve) => {
+				done = resolve;
+				component = factory(tui, theme, keybindings, done);
+			}),
+		},
+	} as any);
+	return { component: () => component, execution };
+}
+
+describe("exp", () => {
+	it("text question focused=false", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "text", question: "Type" }],
+		});
+		component().focused = false;
+		component().focused = true;
+		component().focused = false;
+		component().handleInput("x");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+
+	it("multiselect empty confirm", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "multiselect", question: "Pick", options: [{ value: "a", label: "A" }] }],
+		});
+		component().handleInput("\r"); // empty confirm → no commit
+		component().handleInput(" ");
+		component().handleInput("\r"); // now commits
+		await execution;
+		expect(true).toBe(true);
+	});
+
+	it("text empty submit", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "text", question: "Type" }],
+		});
+		component().handleInput("\r"); // empty submit → no commit
+		component().handleInput("a");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+});

package/dist/extensions/question/tests/exp2.test.ts

@@ -0,0 +1,59 @@
+import { describe, expect, it } from "vitest";
+
+import questionExtension from "../index.ts";
+
+const theme = {
+	fg: (_name: string, value: string) => value,
+	bg: (_name: string, value: string) => value,
+	bold: (value: string) => value,
+};
+const tui = { requestRender() {}, terminal: { rows: 24 } };
+const keybindings = {
+	matches: (data: string, action: string) =>
+		(data === "\r" && action === "tui.select.confirm") ||
+		(data === "\x1b" && action === "tui.select.cancel"),
+	getKeys: () => ["Enter"],
+};
+
+async function runWizard(params: unknown) {
+	let tool: any;
+	let component: any;
+	let done!: (value: unknown) => void;
+	questionExtension({ registerTool: (entry: unknown) => (tool = entry), events: { emit() {} } } as any);
+	const execution = tool.execute("call", params, undefined, undefined, {
+		mode: "tui",
+		ui: {
+			custom: (factory: (...args: any[]) => unknown) => new Promise((resolve) => {
+				done = resolve;
+				component = factory(tui, theme, keybindings, done);
+			}),
+		},
+	} as any);
+	return { component: () => component, execution };
+}
+
+describe("exp2", () => {
+	it("text question construction only", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "text", question: "Type" }],
+		});
+		component().render(120);
+		component().handleInput("a");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+
+	it("blur then non-enter key", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "text", question: "Type" }],
+		});
+		component().handleInput("\x1b"); // blur
+		component().handleInput("x"); // non-enter while blurred
+		component().handleInput("\r"); // re-focus
+		component().handleInput("a");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+});

package/dist/extensions/question/tests/exp3.test.ts

@@ -0,0 +1,47 @@
+import { describe, expect, it } from "vitest";
+
+import questionExtension from "../index.ts";
+
+const theme = {
+	fg: (_name: string, value: string) => value,
+	bg: (_name: string, value: string) => value,
+	bold: (value: string) => value,
+};
+const tui = { requestRender() {}, terminal: { rows: 24 } };
+const keybindings = {
+	matches: (data: string, action: string) =>
+		(data === "\r" && action === "tui.select.confirm") ||
+		(data === "\x1b" && action === "tui.select.cancel"),
+	getKeys: () => ["Enter"],
+};
+
+async function runWizard(params: unknown) {
+	let tool: any;
+	let component: any;
+	let done!: (value: unknown) => void;
+	questionExtension({ registerTool: (entry: unknown) => (tool = entry), events: { emit() {} } } as any);
+	const execution = tool.execute("call", params, undefined, undefined, {
+		mode: "tui",
+		ui: {
+			custom: (factory: (...args: any[]) => unknown) => new Promise((resolve) => {
+				done = resolve;
+				component = factory(tui, theme, keybindings, done);
+			}),
+		},
+	} as any);
+	return { component: () => component, execution };
+}
+
+describe("exp3", () => {
+	it("focused true then false on text", async () => {
+		const { component, execution } = await runWizard({
+			questions: [{ type: "text", question: "Type" }],
+		});
+		component().focused = true; // editor.focused = true && true
+		component().focused = false; // editor.focused = false && true
+		component().handleInput("a");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+});

package/dist/extensions/question/tests/helpers.test.ts

@@ -83,4 +83,11 @@ describe("question helpers", () => {
 		expect(previewText("a".repeat(10), 10, 2)).toBe("a".repeat(10));
 		expect(previewText("a".repeat(11), 10, 2)).toBe(`${"a".repeat(9)}…`);
 	});
+
+	it("previewText handles nullish values", () => {
+		expect(previewText(null)).toBe("");
+		expect(previewText(undefined)).toBe("");
+		expect(previewText(null, 10, 1)).toBe("");
+		expect(previewText(undefined, 10, 1)).toBe("");
+	});
 });

package/dist/extensions/question/tests/question-list.test.ts

@@ -552,3 +552,143 @@ describe("QuestionList comment toggle", () => {
 		expect(list.isCommentEnabled()).toBe(true);
 	});
 });
+
+describe("QuestionList split-pane preview truncation", () => {
+	const longDescription = "A very long description that keeps wrapping and wrapping well past the preview pane width so the preview must be truncated with an ellipsis marker";
+
+	it("maxLines 1 collapses the preview to a single ellipsis line", () => {
+		const list = new QuestionList(
+			[{ value: "a", label: "Alpha", description: longDescription }],
+			SingleSelect,
+			false,
+			false,
+			fakeTheme,
+			fakeKeybindings(),
+			disabledShortcut,
+		);
+		list.setMaxVisibleRows(1);
+		const lines = list.render(120);
+		expect(lines.some((l) => l.includes("…"))).toBe(true);
+	});
+
+	it("maxLines above 1 keeps leading lines and appends an ellipsis", () => {
+		const list = new QuestionList(
+			[{ value: "a", label: "Alpha", description: longDescription }],
+			SingleSelect,
+			false,
+			false,
+			fakeTheme,
+			fakeKeybindings(),
+			disabledShortcut,
+		);
+		list.setMaxVisibleRows(2);
+		const lines = list.render(120);
+		expect(lines.some((l) => l.includes("…"))).toBe(true);
+	});
+});
+
+describe("QuestionList remaining branch coverage", () => {
+	it("single-select confirm on comment-toggle row toggles the comment", () => {
+		const list = new QuestionList(opts(["a"]), SingleSelect, false, true, fakeTheme, fakeKeybindings(["tui.select.confirm"]), enabledShortcut);
+		list.handleInput(CTRL_J); // to comment-toggle row
+		list.handleInput(ENTER);
+		expect(list.isCommentEnabled()).toBe(true);
+	});
+
+	it("multi-select renders the selected comment-toggle row with an enabled checkbox", () => {
+		const list = new QuestionList(opts(["a"]), MultiSelect, false, true, fakeTheme, fakeKeybindings(), enabledShortcut);
+		list.handleInput(CTRL_G); // enable comment
+		list.handleInput(CTRL_J); // to comment-toggle row
+		const lines = list.render(80);
+		expect(lines.some((l) => l.includes("[✓]"))).toBe(true);
+		expect(lines.some((l) => l.includes("→"))).toBe(true);
+	});
+
+	it("multi-select renders a checked option with a success checkbox", () => {
+		const list = new QuestionList(opts(["a", "b"]), MultiSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput(SPACE); // check option 0
+		const lines = list.render(80);
+		expect(lines.some((l) => l.includes("[✓]"))).toBe(true);
+	});
+});
+
+describe("QuestionList final branch coverage", () => {
+	it("setMaxVisibleRows with the same value is a no-op", () => {
+		const list = new QuestionList(opts(["a"]), SingleSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.setMaxVisibleRows(12);
+		expect((list as any).maxVisibleRows).toBe(12);
+	});
+
+	it("single-select render styles the scroll indicator line", () => {
+		const list = new QuestionList(opts(["a", "b", "c", "d", "e", "f"]), SingleSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.setMaxVisibleRows(3);
+		const lines = list.render(80);
+		expect(lines.some((l) => l.includes("("))).toBe(true);
+	});
+
+	it("single-select render styles description lines", () => {
+		const list = new QuestionList(
+			[
+				{ value: "a", label: "Alpha", description: "some desc" },
+				{ value: "b", label: "Beta" },
+			],
+			SingleSelect,
+			false,
+			false,
+			fakeTheme,
+			fakeKeybindings(),
+			disabledShortcut,
+		);
+		list.handleInput(CTRL_J); // select Beta so Alpha's description row is unselected
+		const lines = list.render(80);
+		expect(lines.some((l) => l.includes("some desc"))).toBe(true);
+	});
+
+	it("preview shows Enabled when the comment row is selected with comment on", () => {
+		const list = new QuestionList(opts(["a"]), SingleSelect, false, true, fakeTheme, fakeKeybindings(), enabledShortcut);
+		list.handleInput(CTRL_G); // enable comment
+		list.handleInput(CTRL_J); // to comment-toggle row
+		const lines = list.render(120);
+		expect(lines.some((l) => l.includes("Enabled"))).toBe(true);
+	});
+
+	it("preview shows the current filter on the freeform row", () => {
+		const list = new QuestionList(opts(["a"]), SingleSelect, true, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput("a"); // search keeps the option, selects index 0
+		list.handleInput(CTRL_J); // to freeform row
+		const lines = list.render(120);
+		expect(lines.some((l) => l.includes("Current filter"))).toBe(true);
+	});
+
+	it("preview shows the current filter on a selected option", () => {
+		const list = new QuestionList(opts(["a"]), SingleSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput("a"); // search matches, selected stays 0
+		const lines = list.render(120);
+		expect(lines.some((l) => l.includes("Filter:"))).toBe(true);
+	});
+
+	it("single-select number key beyond the filtered options is ignored", () => {
+		const list = new QuestionList(opts(["a", "b"]), SingleSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput("3");
+		expect((list as any).selectedIndex).toBe(0);
+	});
+
+	it("multi-select up from a non-zero index moves up", () => {
+		const list = new QuestionList(opts(["a", "b"]), MultiSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput(CTRL_J); // to 1
+		list.handleInput(CTRL_K); // back to 0
+		expect((list as any).selectedIndex).toBe(0);
+	});
+
+	it("multi-select number key beyond the options is ignored", () => {
+		const list = new QuestionList(opts(["a", "b"]), MultiSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput("3");
+		expect((list as any).checked.size).toBe(0);
+	});
+
+	it("multi-select input that matches nothing reaches the confirm check safely", () => {
+		const list = new QuestionList(opts(["a"]), MultiSelect, false, false, fakeTheme, fakeKeybindings(), disabledShortcut);
+		list.handleInput("x");
+		expect((list as any).checked.size).toBe(0);
+	});
+});

package/dist/extensions/question/tests/row-layout.test.ts

@@ -199,4 +199,17 @@ describe("row-layout", () => {
 		expect(blocks[1]!.lines.length).toBeGreaterThan(1);
 		expect(blocks[1]!.lines[1]!.startsWith("    ")).toBe(true);
 	});
+
+	it("renderSingleSelectRows: out-of-range selectedIndex falls back to the first block", () => {
+		const rows = renderSingleSelectRows({
+			options: opts(["a", "b"]),
+			selectedIndex: 7,
+			width: 40,
+			allowFreeform: false,
+			maxRows: 1,
+		});
+		expect(rows.length).toBeGreaterThan(0);
+		expect(rows[0]!.selected).toBe(true);
+		expect(rows[0]!.line).toContain("a");
+	});
 });

package/dist/extensions/question/tests/selection-mode.test.ts

@@ -0,0 +1,54 @@
+import { describe, expect, it } from "vitest";
+
+import { MultiSelect, SingleSelect } from "../selection-mode.ts";
+
+describe("selection-mode", () => {
+	it("SingleSelect reports single mode and no-op toggle", () => {
+		expect(SingleSelect.multi).toBe(false);
+		const checked = new Set<number>([1]);
+		SingleSelect.toggle(checked, 0);
+		expect([...checked]).toEqual([1]);
+	});
+
+	it("SingleSelect buildResult returns the selected value", () => {
+		const options = [{ value: "a", label: "A" }];
+		expect(SingleSelect.buildResult({ selectedIndex: 0, checked: new Set(), options })).toEqual(["a"]);
+		// Out-of-range index (or empty options) → no value → empty result.
+		expect(SingleSelect.buildResult({ selectedIndex: 2, checked: new Set(), options })).toEqual([]);
+		expect(SingleSelect.buildResult({ selectedIndex: 0, checked: new Set(), options: [] })).toEqual([]);
+	});
+
+	it("SingleSelect handleNumberKey clamps to the last option", () => {
+		expect(SingleSelect.handleNumberKey(5, 3)).toEqual({ toggle: false, selectIndex: 2 });
+		expect(SingleSelect.handleNumberKey(1, 3)).toEqual({ toggle: false, selectIndex: 1 });
+	});
+
+	it("MultiSelect toggle adds, removes, and ignores negative indexes", () => {
+		expect(MultiSelect.multi).toBe(true);
+		const checked = new Set<number>();
+		MultiSelect.toggle(checked, 0);
+		expect(checked.has(0)).toBe(true);
+		MultiSelect.toggle(checked, 0);
+		expect(checked.has(0)).toBe(false);
+		MultiSelect.toggle(checked, -1);
+		expect(checked.size).toBe(0);
+	});
+
+	it("MultiSelect buildResult sorts and filters missing options", () => {
+		const options = [
+			{ value: "a", label: "A" },
+			{ value: "b", label: "B" },
+		];
+		const checked = new Set([1, 0]);
+		expect(MultiSelect.buildResult({ selectedIndex: 0, checked, options })).toEqual(["a", "b"]);
+		// Indexes pointing past the options array are filtered out.
+		const sparse = new Set([0, 1, 2]);
+		expect(MultiSelect.buildResult({ selectedIndex: 0, checked: sparse, options })).toEqual(["a", "b"]);
+		expect(MultiSelect.buildResult({ selectedIndex: 0, checked: new Set(), options })).toEqual([]);
+	});
+
+	it("MultiSelect handleNumberKey clamps to the last option", () => {
+		expect(MultiSelect.handleNumberKey(5, 3)).toEqual({ toggle: true, selectIndex: 2 });
+		expect(MultiSelect.handleNumberKey(0, 3)).toEqual({ toggle: true, selectIndex: 0 });
+	});
+});

package/dist/extensions/question/tests/zz-ig3.test.ts

@@ -0,0 +1,20 @@
+import { describe, expect, it } from "vitest";
+import { QuestionList } from "../question-list.ts";
+import { SingleSelect } from "../selection-mode.ts";
+import type { KeybindingsManager, Theme } from "@selesai/code";
+import type { ResolvedShortcut } from "../types.ts";
+
+const fakeTheme = { fg: (_n: string, s: string) => s, bold: (s: string) => s } as unknown as Theme;
+const fakeKeybindings = (actions: string[] = []): KeybindingsManager =>
+	({ matches: (_d: string, a: string) => actions.includes(a), getKeys: () => [] }) as unknown as KeybindingsManager;
+const disabledShortcut: ResolvedShortcut = { disabled: true, spec: null, matches: ((_d: string) => false) as (d: string) => false };
+
+describe("ig3", () => {
+	it("confirm option", () => {
+		const list = new QuestionList([{ value: "a", label: "A" }], SingleSelect, false, false, fakeTheme, fakeKeybindings(["tui.select.confirm"]), disabledShortcut);
+		let submitted = false;
+		list.onSubmit = () => (submitted = true);
+		list.handleInput("\r");
+		expect(submitted).toBe(true);
+	});
+});

package/dist/extensions/question/tests/zz-probe.test.ts

@@ -0,0 +1,29 @@
+import { describe, expect, it } from "vitest";
+import questionExtension from "../index.ts";
+
+const theme = { fg: (_n: string, s: string) => s, bg: (_n: string, s: string) => s, bold: (s: string) => s };
+const tui = { requestRender() {}, terminal: { rows: 24 } };
+const keybindings = { matches: () => false, getKeys: () => ["Enter"] };
+
+async function runWizard(params: unknown) {
+	let tool: any;
+	let component: any;
+	let done!: (value: unknown) => void;
+	questionExtension({ registerTool: (entry: unknown) => (tool = entry), events: { emit() {} } } as any);
+	const execution = tool.execute("call", params, undefined, undefined, {
+		mode: "tui",
+		ui: { custom: (factory: (...args: any[]) => unknown) => new Promise((resolve) => { done = resolve; component = factory(tui, theme, keybindings, done); }) },
+	} as any);
+	return { component: () => component, execution };
+}
+
+describe("probe", () => {
+	it("text question focused false", async () => {
+		const { component, execution } = await runWizard({ questions: [{ type: "text", question: "Type" }] });
+		component().focused = false;
+		component().handleInput("a");
+		component().handleInput("\r");
+		await execution;
+		expect(true).toBe(true);
+	});
+});

package/dist/skills/pi-subagents/SKILL.md

@@ -8,7 +8,7 @@ description: |
   other agents contribute context, planning, or execution.
 ---
 
-# Pi Subagents
+# Selesai Subagents
 
 This skill is for the main parent orchestrator only. Do not inject or follow it inside spawned child subagents. The parent session owns delegation, orchestration, review fanout, and final fix-worker launches. Ordinary children should not run their own subagent workflows; the explicit exception is a delegated fanout child whose resolved builtin `tools` includes `subagent`, and that child may use `subagent` only for the fanout work the parent assigned.