@xynogen/pix-ask 0.2.6 → 0.2.9

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-ask",
-	"version": "0.2.6",
+	"version": "0.2.9",
 	"description": "Pi tool — structured questionnaire UI (ask_user)",
 	"type": "module",
 	"main": "src/index.ts",
@@ -9,6 +9,7 @@
 	},
 	"files": [
 		"src",
+		"!src/**/*.test.ts",
 		"README.md",
 		"LICENSE"
 	],
@@ -34,8 +35,9 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"typebox": "^1.1.38",
-		"@xynogen/pix-pretty": "^1.7.9"
+		"@xynogen/pix-pretty": "^1.7.9",
+		"@xynogen/pix-runtime": "^0.1.1",
+		"typebox": "^1.1.38"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/components.ts

@@ -43,10 +43,7 @@ export class TabBar implements Component {
 			truncateToWidth(
 				t.fg("accent", "╭─") +
 					line +
-					t.fg(
-						"accent",
-						`${"─".repeat(Math.max(0, inner - line.length - 1))}╮`,
-					),
+					t.fg("accent", `${"─".repeat(Math.max(0, inner - line.length - 1))}╮`),
 				width,
 				"",
 			),

package/src/glyphs.ts

@@ -6,11 +6,10 @@ const RADIO_SELECTED = "◉";
 const RADIO_UNSELECTED = "○";
 
 /** Glyph + theme color token for a selection row. Caller applies theme.fg(color, glyph). */
-export function selectionGlyph(opts: {
-	multi: boolean;
-	selected: boolean;
-	checked: boolean;
-}): { glyph: string; color: string } {
+export function selectionGlyph(opts: { multi: boolean; selected: boolean; checked: boolean }): {
+	glyph: string;
+	color: string;
+} {
 	if (opts.multi) {
 		return opts.checked
 			? { glyph: CHECKBOX_CHECKED, color: "success" }
@@ -28,8 +27,7 @@ export function selectionGlyph(opts: {
  */
 export function checkboxGlyphs(): { checked: string; unchecked: string } {
 	const unicodeSafe =
-		visibleWidth(CHECKBOX_CHECKED) === 1 &&
-		visibleWidth(CHECKBOX_UNCHECKED) === 1;
+		visibleWidth(CHECKBOX_CHECKED) === 1 && visibleWidth(CHECKBOX_UNCHECKED) === 1;
 	return unicodeSafe
 		? { checked: CHECKBOX_CHECKED, unchecked: CHECKBOX_UNCHECKED }
 		: { checked: "[x]", unchecked: "[ ]" };

package/src/helpers.ts

@@ -20,15 +20,11 @@ export function safeMarkdownTheme(): MarkdownTheme | undefined {
 // ── Option / question helpers ──────────────────────────────────────────
 
 export function hasAnyPreview(q: QuestionData): boolean {
-	return q.options.some(
-		(o) => typeof o.preview === "string" && o.preview.length > 0,
-	);
+	return q.options.some((o) => typeof o.preview === "string" && o.preview.length > 0);
 }
 
 /** Which sentinel rows are auto-appended for a question. */
-export function sentinelsFor(
-	q: QuestionData,
-): Array<{ kind: string; label: string }> {
+export function sentinelsFor(q: QuestionData): Array<{ kind: string; label: string }> {
 	const out: Array<{ kind: string; label: string }> = [];
 	if (q.multiSelect) {
 		out.push({ kind: "next", label: SENTINEL_NEXT });
@@ -47,10 +43,7 @@ export function formatAnswerScalar(a: QuestionAnswer): string {
 	return a.answer ?? "(selected)";
 }
 
-export function buildResponseText(
-	answers: QuestionAnswer[],
-	questions: QuestionData[],
-): string {
+export function buildResponseText(answers: QuestionAnswer[], questions: QuestionData[]): string {
 	const segs: string[] = [];
 	for (const a of answers) {
 		const q = questions[a.questionIndex]?.question ?? `Q${a.questionIndex + 1}`;
@@ -58,9 +51,7 @@ export function buildResponseText(
 		if (a.preview) s += `. selected preview: ${a.preview}`;
 		segs.push(s);
 	}
-	return segs.length
-		? `User answered: ${segs.join(". ")}.`
-		: "User declined to answer questions.";
+	return segs.length ? `User answered: ${segs.join(". ")}.` : "User declined to answer questions.";
 }
 
 // ── Scroll indicator ───────────────────────────────────────────────────

package/src/index.ts

@@ -1,18 +1,11 @@
 import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
 import { Text } from "@earendil-works/pi-tui";
-
+import { once } from "@xynogen/pix-runtime/once";
 import { buildResponseText } from "./helpers.js";
-import { once } from "./once.ts";
 import { AskQuestionnaire } from "./questionnaire.js";
 import { rpcFallback } from "./rpc.js";
 import type { Params } from "./schema.js";
-import {
-	MAX_OPTIONS,
-	MAX_QUESTIONS,
-	MIN_OPTIONS,
-	ParamsSchema,
-	SENTINEL_FREEFORM,
-} from "./schema.js";
+import { MAX_OPTIONS, MAX_QUESTIONS, MIN_OPTIONS, ParamsSchema } from "./schema.js";
 import type { QuestionAnswer, QuestionnaireResult } from "./types.js";
 
 // ── Re-exports (consumed by tests and single-select-layout) ───────────
@@ -41,8 +34,6 @@ export default function registerAsk(pi: ExtensionAPI): void {
 			promptSnippet: `Ask the user up to ${MAX_QUESTIONS} structured questions (${MIN_OPTIONS}-${MAX_OPTIONS} options each) when requirements are ambiguous`,
 			promptGuidelines: [
 				`Use ask whenever the user's request is underspecified and you cannot proceed without concrete decisions — you can ask up to ${MAX_QUESTIONS} questions per invocation.`,
-				`Each question MUST have ${MIN_OPTIONS}-${MAX_OPTIONS} options. Every option requires a concise label (1-5 words) and a description explaining what the choice means or its trade-offs. The user can additionally type a custom answer ("${SENTINEL_FREEFORM}" row is appended automatically to single-select questions).`,
-				`Set multiSelect: true when multiple answers are valid; this suppresses the "${SENTINEL_FREEFORM}" row. Provide an options[].preview markdown string when an option benefits from richer side-by-side context (mockups, code snippets, diagrams, configs) — single-select only. NOTE: any non-empty preview on a single-select question ALSO suppresses the "${SENTINEL_FREEFORM}" row (no room in the side-by-side layout). If you recommend a specific option, make it the first option and append "(Recommended)" to its label.`,
 				"Do not stack multiple ask calls back-to-back — group all clarifying questions into one invocation.",
 			],
 			executionMode: "sequential",
@@ -60,9 +51,7 @@ export default function registerAsk(pi: ExtensionAPI): void {
 
 				if (!Array.isArray(typed.questions) || typed.questions.length === 0) {
 					return {
-						content: [
-							{ type: "text", text: "At least one question is required." },
-						],
+						content: [{ type: "text", text: "At least one question is required." }],
 						isError: true,
 						details: { answers: [], cancelled: true },
 					};
@@ -79,11 +68,9 @@ export default function registerAsk(pi: ExtensionAPI): void {
 				const result = await ctx.ui.custom<QuestionnaireResult | null>(
 					(tui, theme, keybindings, done) => {
 						if (signal) {
-							signal.addEventListener(
-								"abort",
-								() => done({ answers: [], cancelled: true }),
-								{ once: true },
-							);
+							signal.addEventListener("abort", () => done({ answers: [], cancelled: true }), {
+								once: true,
+							});
 						}
 						return new AskQuestionnaire(typed, tui, theme, keybindings, done);
 					},
@@ -92,9 +79,7 @@ export default function registerAsk(pi: ExtensionAPI): void {
 
 				if (!result || result.cancelled) {
 					return {
-						content: [
-							{ type: "text", text: "User cancelled the questionnaire" },
-						],
+						content: [{ type: "text", text: "User cancelled the questionnaire" }],
 						details: result ?? { answers: [], cancelled: true },
 					};
 				}
@@ -124,10 +109,7 @@ export default function registerAsk(pi: ExtensionAPI): void {
 					return new Text(theme.fg("warning", "Cancelled"), 0, 0);
 				}
 				const texts = details.answers.map((a) => {
-					const v =
-						a.kind === "multi"
-							? (a.selected ?? []).join(", ")
-							: (a.answer ?? "");
+					const v = a.kind === "multi" ? (a.selected ?? []).join(", ") : (a.answer ?? "");
 					return `${a.questionIndex + 1}: ${v}`;
 				});
 				return new Text(theme.fg("success", `✓ ${texts.join(" • ")}`), 0, 0);

package/src/questionnaire.ts

@@ -12,25 +12,32 @@ import {
 	truncateToWidth,
 	wrapTextWithAnsi,
 } from "@earendil-works/pi-tui";
+import { frameLines, modalWidth } from "@xynogen/pix-pretty/modal-frame";
 import { dim } from "./components.js";
-import { frameLines, modalWidth } from "./frame.js";
 import { checkboxGlyphs, selectionGlyph } from "./glyphs.js";
 import { safeMarkdownTheme, sentinelsFor } from "./helpers.js";
 import type { OptionData, Params, QuestionData } from "./schema.js";
-import {
-	SENTINEL_FREEFORM,
-	SENTINEL_NEXT,
-	SEPARATOR,
-	SPLIT_PANE_MIN_WIDTH,
-} from "./schema.js";
-import type {
-	AnswerKind,
-	QuestionAnswer,
-	QuestionnaireResult,
-} from "./types.js";
+import { SENTINEL_FREEFORM, SENTINEL_NEXT, SEPARATOR, SPLIT_PANE_MIN_WIDTH } from "./schema.js";
+import type { AnswerKind, QuestionAnswer, QuestionnaireResult } from "./types.js";
 
 // ── AskQuestionnaire ───────────────────────────────────────────────────
 
+function printableCharacter(data: string): string | undefined {
+	const decoded = decodeKittyPrintable(data) ?? data;
+	const characters = [...decoded];
+	if (characters.length !== 1) return undefined;
+	const codePoint = characters[0]?.codePointAt(0);
+	if (
+		codePoint === undefined ||
+		codePoint < 32 ||
+		codePoint === 127 ||
+		(codePoint >= 128 && codePoint <= 159)
+	) {
+		return undefined;
+	}
+	return characters[0];
+}
+
 export class AskQuestionnaire extends Container {
 	private params: Params;
 	private tui: TUI;
@@ -84,8 +91,7 @@ export class AskQuestionnaire extends Container {
 		label?: string;
 		option?: OptionData;
 	}> {
-		const items: Array<{ kind: string; label?: string; option?: OptionData }> =
-			[];
+		const items: Array<{ kind: string; label?: string; option?: OptionData }> = [];
 		for (const o of this.filteredOptions) {
 			items.push({ kind: "option", option: o });
 		}
@@ -137,9 +143,7 @@ export class AskQuestionnaire extends Container {
 		selected?: string[],
 		preview?: string,
 	): void {
-		this.answers = this.answers.filter(
-			(a) => a.questionIndex !== this.currentIndex,
-		);
+		this.answers = this.answers.filter((a) => a.questionIndex !== this.currentIndex);
 		this.answers.push({
 			questionIndex: this.currentIndex,
 			question: this.currentQ.question,
@@ -158,12 +162,7 @@ export class AskQuestionnaire extends Container {
 		}
 
 		if (item.kind === "option" && item.option) {
-			this.recordAnswer(
-				"option",
-				item.option.label,
-				undefined,
-				item.option.preview,
-			);
+			this.recordAnswer("option", item.option.label, undefined, item.option.preview);
 			this.nextQuestion();
 		} else if (item.kind === "other") {
 			this.inputMode = true;
@@ -204,9 +203,7 @@ export class AskQuestionnaire extends Container {
 	}
 
 	private restoreAnswerState(): void {
-		const prev = this.answers.find(
-			(a) => a.questionIndex === this.currentIndex,
-		);
+		const prev = this.answers.find((a) => a.questionIndex === this.currentIndex);
 		if (!prev) return;
 		const q = this.currentQ;
 		if (prev.kind === "multi") {
@@ -287,8 +284,7 @@ export class AskQuestionnaire extends Container {
 			matchesKey(data, Key.ctrl("k"))
 		) {
 			if (total > 0) {
-				this.selectedOptionIndex =
-					(this.selectedOptionIndex - 1 + total) % total;
+				this.selectedOptionIndex = (this.selectedOptionIndex - 1 + total) % total;
 				this.refresh();
 			}
 			return;
@@ -347,7 +343,10 @@ export class AskQuestionnaire extends Container {
 			return;
 		}
 
-		const numMatch = data.match(/^[1-9]$/);
+		// Decode CSI-u first: under Kitty flag 1 digits arrive as escape
+		// sequences, and the raw regex would miss them (digit would fall
+		// through into the search query instead of selecting an option).
+		const numMatch = (decodeKittyPrintable(data) ?? data).match(/^[1-9]$/);
 		if (numMatch && this.filteredOptions.length > 0) {
 			const idx = Number(numMatch[0]) - 1;
 			if (idx >= 0 && idx < this.filteredOptions.length) {
@@ -371,23 +370,13 @@ export class AskQuestionnaire extends Container {
 		}
 
 		if (!isMulti) {
-			const printable = decodeKittyPrintable(data);
+			// Accept Unicode text under both legacy and Kitty encodings, while
+			// rejecting C0, DEL, and C1 controls that corrupt filter queries.
+			const printable = printableCharacter(data);
 			if (printable !== undefined) {
 				this.searchQuery += printable;
 				this.selectedOptionIndex = 0;
 				this.refresh();
-				return;
-			}
-			const chars = [...data];
-			if (
-				chars.length === 1 &&
-				chars[0] &&
-				chars[0].charCodeAt(0) >= 32 &&
-				chars[0].charCodeAt(0) < 127
-			) {
-				this.searchQuery += chars[0];
-				this.selectedOptionIndex = 0;
-				this.refresh();
 			}
 		}
 	}
@@ -423,10 +412,7 @@ export class AskQuestionnaire extends Container {
 		const maxVisible = Math.min(total, 12);
 		const start = Math.max(
 			0,
-			Math.min(
-				this.selectedOptionIndex - Math.floor(maxVisible / 2),
-				total - maxVisible,
-			),
+			Math.min(this.selectedOptionIndex - Math.floor(maxVisible / 2), total - maxVisible),
 		);
 		const end = Math.min(start + maxVisible, total);
 
@@ -462,24 +448,18 @@ export class AskQuestionnaire extends Container {
 				const label = sel
 					? t.fg("accent", t.bold(SENTINEL_FREEFORM))
 					: t.fg("text", t.bold(SENTINEL_FREEFORM));
-				lines.push(
-					truncateToWidth(`${ptr} ${t.fg("dim", "✎")} ${label}`, inner, ""),
-				);
+				lines.push(truncateToWidth(`${ptr} ${t.fg("dim", "✎")} ${label}`, inner, ""));
 			} else if (item.kind === "next") {
 				const label = sel
 					? t.fg("accent", t.bold(SENTINEL_NEXT))
 					: t.fg("text", t.bold(SENTINEL_NEXT));
-				lines.push(
-					truncateToWidth(`${ptr} ${t.fg("dim", "→")} ${label}`, inner, ""),
-				);
+				lines.push(truncateToWidth(`${ptr} ${t.fg("dim", "→")} ${label}`, inner, ""));
 			}
 		}
 
 		if (start > 0 || end < total) {
 			const count =
-				this.filteredOptions.length > 0
-					? `${this.selectedOptionIndex + 1}/${total}`
-					: `${total}`;
+				this.filteredOptions.length > 0 ? `${this.selectedOptionIndex + 1}/${total}` : `${total}`;
 			lines.push(t.fg("dim", truncateToWidth(`  ${count}`, inner, "")));
 		}
 
@@ -496,19 +476,12 @@ export class AskQuestionnaire extends Container {
 		const mdWidth = Math.max(10, width);
 
 		if (this.mdTheme) {
-			const md = new Markdown(
-				`## ${item.option.label}\n\n${mdText}`,
-				0,
-				0,
-				this.mdTheme,
-			);
+			const md = new Markdown(`## ${item.option.label}\n\n${mdText}`, 0, 0, this.mdTheme);
 			return md.render(mdWidth);
 		}
 
 		const lines = wrapTextWithAnsi(mdText, mdWidth);
-		return lines.map((l) =>
-			truncateToWidth(this.theme.fg("muted", l), mdWidth, ""),
-		);
+		return lines.map((l) => truncateToWidth(this.theme.fg("muted", l), mdWidth, ""));
 	}
 
 	override render(termWidth: number): string[] {
@@ -520,9 +493,7 @@ export class AskQuestionnaire extends Container {
 		const t = this.theme;
 		const isMulti = !!this.currentQ.multiSelect;
 		const hasPreview =
-			!isMulti &&
-			this.selectedItem?.kind === "option" &&
-			!!this.selectedItem?.option?.preview;
+			!isMulti && this.selectedItem?.kind === "option" && !!this.selectedItem?.option?.preview;
 
 		const useSplit = hasPreview && width >= SPLIT_PANE_MIN_WIDTH;
 		const leftWidth = useSplit ? Math.floor((width - 2) * 0.45) : inner;
@@ -530,8 +501,7 @@ export class AskQuestionnaire extends Container {
 
 		const lines: string[] = [];
 
-		const row = (content: string): string =>
-			truncateToWidth(content, width, "");
+		const row = (content: string): string => truncateToWidth(content, width, "");
 
 		// Tab bar — rendered as the framed top edge (frameLines `top`).
 		let top: string | undefined;
@@ -554,10 +524,7 @@ export class AskQuestionnaire extends Container {
 		lines.push(row(`${chip}${prog}`));
 
 		// Question text
-		for (const w of wrapTextWithAnsi(
-			this.currentQ.question,
-			Math.max(10, inner),
-		)) {
+		for (const w of wrapTextWithAnsi(this.currentQ.question, Math.max(10, inner))) {
 			lines.push(row(t.fg("text", t.bold(w))));
 		}
 
@@ -601,28 +568,17 @@ export class AskQuestionnaire extends Container {
 		}
 
 		// Footer hints
-		const navHint =
-			this.params.questions.length > 1 ? "↑↓ nav • ←→ question" : "↑↓ nav";
+		const navHint = this.params.questions.length > 1 ? "↑↓ nav • ←→ question" : "↑↓ nav";
 		const hintParts = isMulti
-			? [
-					`${navHint} • space toggle • enter commit • esc clear`,
-					"ctrl+c cancel",
-				]
-			: [
-					`${navHint} • type filter • enter select • esc clear`,
-					"ctrl+c cancel",
-				];
+			? [`${navHint} • space toggle • enter commit • esc clear`, "ctrl+c cancel"]
+			: [`${navHint} • type filter • enter select • esc clear`, "ctrl+c cancel"];
 		lines.push(row(dim(t)(hintParts.join(" • "))));
 
 		return this.frame(mw, lines, top);
 	}
 
 	/** Wrap body lines in the rounded modal border at the given outer width. */
-	private frame(
-		outerWidth: number,
-		lines: string[],
-		top: string | undefined,
-	): string[] {
+	private frame(outerWidth: number, lines: string[], top: string | undefined): string[] {
 		const t = this.theme;
 		return frameLines({
 			width: outerWidth,

package/src/rpc.ts

@@ -21,9 +21,7 @@ export async function rpcFallback(
 		const header = q.header;
 
 		if (q.multiSelect) {
-			const lines = q.options.map(
-				(o, idx) => `${idx + 1}. ${o.label} — ${o.description}`,
-			);
+			const lines = q.options.map((o, idx) => `${idx + 1}. ${o.label} — ${o.description}`);
 			const raw = await ui.input(
 				`${header}: ${q.question}\n\n${lines.join("\n")}\n\nEnter numbers separated by commas:`,
 				"e.g. 1,3",
@@ -71,8 +69,7 @@ export async function rpcFallback(
 				});
 			} else {
 				const opt = q.options.find(
-					(o) =>
-						chosen === o.label || `${o.label} — ${o.description}` === chosen,
+					(o) => chosen === o.label || `${o.label} — ${o.description}` === chosen,
 				);
 				answers.push({
 					questionIndex: i,

package/src/schema.ts

@@ -27,8 +27,7 @@ export const OptionSchema = Type.Object({
 	}),
 	preview: Type.Optional(
 		Type.String({
-			description:
-				"Optional markdown preview for side-by-side layout (single-select only).",
+			description: "Optional markdown preview for side-by-side layout (single-select only).",
 		}),
 	),
 });
@@ -44,14 +43,12 @@ export const QuestionSchema = Type.Object({
 	options: Type.Array(OptionSchema, {
 		minItems: MIN_OPTIONS,
 		maxItems: MAX_OPTIONS,
-		description:
-			"2-4 options. 'Type something.' is auto-appended for single-select.",
+		description: "2-4 options. 'Type something.' is auto-appended for single-select.",
 	}),
 	multiSelect: Type.Optional(
 		Type.Boolean({
 			default: false,
-			description:
-				"Allow multiple selections. Suppresses 'Type something.' row.",
+			description: "Allow multiple selections. Suppresses 'Type something.' row.",
 		}),
 	),
 });

package/src/ask.test.ts

@@ -1,243 +0,0 @@
-/**
- * ask.test.ts — tests for the ask questionnaire tool
- *
- * Tests cover pure functions (schema validation, sentinel logic, answer
- * formatting). TUI components are not tested here.
- */
-
-import { describe, expect, test } from "bun:test";
-import {
-	buildResponseText,
-	formatAnswerScalar,
-	hasAnyPreview,
-	type OptionData,
-	type QuestionData,
-	sentinelsFor,
-} from "./index.ts";
-
-// ── Fixtures ──────────────────────────────────────────────────────────
-
-const opt = (
-	label: string,
-	description = "Test option",
-	preview?: string,
-): OptionData => ({
-	label,
-	description,
-	...(preview ? { preview } : {}),
-});
-
-const qSingle: QuestionData = {
-	question: "Which approach?",
-	header: "Approach",
-	options: [
-		opt("REST", "Traditional REST API"),
-		opt("GraphQL", "Query language for APIs"),
-	],
-};
-
-const qMulti: QuestionData = {
-	question: "Which features?",
-	header: "Features",
-	options: [
-		opt("Auth", "User authentication"),
-		opt("Search", "Full text search"),
-		opt("Export", "Data export"),
-	],
-	multiSelect: true,
-};
-
-const qWithPreview: QuestionData = {
-	question: "Pick a component?",
-	header: "Component",
-	options: [
-		opt("Button", "Clickable button", "<Button>Primary</Button>"),
-		opt("Card", "Container card", "<Card><Content/></Card>"),
-	],
-};
-
-const qSingleNoPreview: QuestionData = {
-	question: "Color?",
-	header: "Color",
-	options: [opt("Red", "Ruby red"), opt("Blue", "Ocean blue")],
-};
-
-// ── hasAnyPreview ─────────────────────────────────────────────────────
-
-describe("hasAnyPreview", () => {
-	test("returns false when no option has preview", () => {
-		expect(hasAnyPreview(qSingle)).toBe(false);
-		expect(hasAnyPreview(qMulti)).toBe(false);
-	});
-
-	test("returns true when at least one option has preview", () => {
-		expect(hasAnyPreview(qWithPreview)).toBe(true);
-	});
-
-	test("returns false for empty options", () => {
-		const q: QuestionData = { question: "?", header: "X", options: [] };
-		expect(hasAnyPreview(q)).toBe(false);
-	});
-});
-
-// ── sentinelsFor ──────────────────────────────────────────────────────
-
-describe("sentinelsFor", () => {
-	test('single-select without preview appends "Type something."', () => {
-		const r = sentinelsFor(qSingleNoPreview);
-		expect(r).toHaveLength(1);
-		expect(r[0]?.kind).toBe("other");
-		expect(r[0]?.label).toBe("Type something.");
-	});
-
-	test('single-select with preview appends nothing (only "Chat about this" is separate)', () => {
-		const r = sentinelsFor(qWithPreview);
-		expect(r).toHaveLength(0);
-	});
-
-	test('multi-select appends "Next"', () => {
-		const r = sentinelsFor(qMulti);
-		expect(r).toHaveLength(1);
-		expect(r[0]?.kind).toBe("next");
-		expect(r[0]?.label).toBe("Next");
-	});
-
-	test("multi-select never appends Type something.", () => {
-		const r = sentinelsFor({ ...qMulti, multiSelect: true });
-		expect(r.every((s) => s.kind !== "other")).toBe(true);
-	});
-
-	test("empty options still gets freeform sentinel (no preview = single-select)", () => {
-		const r = sentinelsFor({ question: "?", header: "X", options: [] });
-		expect(r).toHaveLength(1);
-		expect(r[0]?.kind).toBe("other");
-	});
-});
-
-// ── formatAnswerScalar ────────────────────────────────────────────────
-
-describe("formatAnswerScalar", () => {
-	test("option kind returns the answer string", () => {
-		const a = {
-			questionIndex: 0,
-			question: "Q",
-			kind: "option" as const,
-			answer: "REST",
-		};
-		expect(formatAnswerScalar(a)).toBe("REST");
-	});
-
-	test("multi kind joins selected with comma", () => {
-		const a = {
-			questionIndex: 0,
-			question: "Q",
-			kind: "multi" as const,
-			answer: null,
-			selected: ["Auth", "Search"],
-		};
-		expect(formatAnswerScalar(a)).toBe("Auth, Search");
-	});
-
-	test("custom kind returns the typed text", () => {
-		const a = {
-			questionIndex: 0,
-			question: "Q",
-			kind: "custom" as const,
-			answer: "my custom answer",
-		};
-		expect(formatAnswerScalar(a)).toBe("my custom answer");
-	});
-
-	test("chat kind returns (chat)", () => {
-		const a = {
-			questionIndex: 0,
-			question: "Q",
-			kind: "chat" as const,
-			answer: null,
-		};
-		expect(formatAnswerScalar(a)).toBe("(chat)");
-	});
-});
-
-// ── buildResponseText ─────────────────────────────────────────────────
-
-describe("buildResponseText", () => {
-	test("formats single answer", () => {
-		const answers = [
-			{
-				questionIndex: 0,
-				question: "Which approach?",
-				kind: "option" as const,
-				answer: "REST",
-			},
-		];
-		const text = buildResponseText(answers, [qSingle]);
-		expect(text).toContain("REST");
-		expect(text).toContain("Which approach?");
-	});
-
-	test("formats multi-select answer", () => {
-		const answers = [
-			{
-				questionIndex: 0,
-				question: "Which features?",
-				kind: "multi" as const,
-				answer: null,
-				selected: ["Auth", "Search"],
-			},
-		];
-		const text = buildResponseText(answers, [qMulti]);
-		expect(text).toContain("Auth, Search");
-		expect(text).toContain("Which features?");
-	});
-
-	test("includes preview in response when present", () => {
-		const answers = [
-			{
-				questionIndex: 0,
-				question: "Pick a component?",
-				kind: "option" as const,
-				answer: "Button",
-				preview: "<Button>Primary</Button>",
-			},
-		];
-		const text = buildResponseText(answers, [qWithPreview]);
-		expect(text).toContain("preview: <Button>Primary</Button>");
-	});
-
-	test("formats multiple answers", () => {
-		const qs = [qSingle, qMulti];
-		const answers = [
-			{
-				questionIndex: 0,
-				question: "Which approach?",
-				kind: "option" as const,
-				answer: "GraphQL",
-			},
-			{
-				questionIndex: 1,
-				question: "Which features?",
-				kind: "multi" as const,
-				answer: null,
-				selected: ["Export"],
-			},
-		];
-		const text = buildResponseText(answers, qs);
-		expect(text).toContain("GraphQL");
-		expect(text).toContain("Export");
-	});
-
-	test("shows declined message when no answers", () => {
-		const text = buildResponseText([], [qSingle]);
-		expect(text).toContain("declined");
-	});
-});
-
-// ── Tool registration shape ─────────────────────────────────────────
-
-describe("registerAsk", () => {
-	test("exports a default function", async () => {
-		const mod = await import("./index.ts");
-		expect(typeof mod.default).toBe("function");
-	});
-});

package/src/frame.test.ts

@@ -1,39 +0,0 @@
-import { expect, test } from "bun:test";
-import { visibleWidth } from "@earendil-works/pi-tui";
-import { frameLines, modalWidth } from "./frame.js";
-
-const noColor = (s: string) => s;
-
-test("modalWidth clamps to [40, 96] with 4-col margin", () => {
-	expect(modalWidth(200)).toBe(96);
-	expect(modalWidth(50)).toBe(46);
-	expect(modalWidth(10)).toBe(40);
-});
-
-test("frameLines draws rounded border with uniform width", () => {
-	const out = frameLines({
-		width: 40,
-		lines: ["hello", "world"],
-		color: noColor,
-	});
-	const first = out[0] ?? "";
-	const last = out[out.length - 1] ?? "";
-	expect(first.startsWith("╭")).toBe(true);
-	expect(first.endsWith("╮")).toBe(true);
-	expect(last.startsWith("╰")).toBe(true);
-	expect(last.endsWith("╯")).toBe(true);
-	for (const line of out) {
-		expect(visibleWidth(line)).toBe(40);
-	}
-});
-
-test("frameLines pads ANSI-colored input by visible width", () => {
-	const out = frameLines({
-		width: 40,
-		lines: ["\x1b[31mhi\x1b[0m"],
-		color: noColor,
-	});
-	for (const line of out) {
-		expect(visibleWidth(line)).toBe(40);
-	}
-});

package/src/frame.ts

@@ -1,73 +0,0 @@
-/**
- * Modal frame primitives — inlined from pix-pretty/modal-frame to avoid
- * cross-package subpath imports that break under Node CJS require.
- */
-
-import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
-
-// ── Constants ─────────────────────────────────────────────────────────────────
-
-const MIN_WIDTH = 40;
-const MAX_WIDTH = 96;
-const MARGIN = 4;
-/** 2 border cols + 2 padding spaces */
-const CHROME = 4;
-
-// ── Width ─────────────────────────────────────────────────────────────────────
-
-/** Clamp terminal width to a sane modal width (40–96 cols). */
-export function modalWidth(termWidth: number): number {
-	return Math.min(MAX_WIDTH, Math.max(MIN_WIDTH, termWidth - MARGIN));
-}
-
-// ── Frame ─────────────────────────────────────────────────────────────────────
-
-export interface FrameOptions {
-	width: number;
-	lines: string[];
-	/** Color function for border glyphs — e.g. `(s) => theme.fg("accent", s)` */
-	color: (s: string) => string;
-	/** Background fill function — e.g. `(s) => theme.bg("customMessageBg", s)` */
-	bg?: (s: string) => string;
-	/** Optional pre-styled string rendered as the first content row (tab bar etc.) */
-	top?: string;
-}
-
-/**
- * Render a rounded modal box.
- *
- * Returns an array of full-width ANSI strings:
- *   ╭──────────────────╮
- *   │ [top row]        │   ← only if top is set
- *   │ content line 1   │
- *   │ content line 2   │
- *   ╰──────────────────╯
- */
-export function frameLines(opts: FrameOptions): string[] {
-	const { width, lines, color, top } = opts;
-	const bg = opts.bg ?? ((s: string) => s);
-	const inner = Math.max(1, width - CHROME);
-	const dashes = "─".repeat(width - 2);
-
-	const SENTINEL = "\x00";
-	const bgOpen = bg(SENTINEL).split(SENTINEL)[0] ?? "";
-	const reassert = (s: string): string =>
-		bgOpen
-			? s.replace(/\x1b\[([0-9;]*)m/g, (seq, p: string) =>
-					p === "0" || p.split(";").includes("49") ? `${seq}${bgOpen}` : seq,
-				)
-			: s;
-
-	const row = (content: string): string => {
-		const pad = inner - visibleWidth(content);
-		const padded =
-			pad > 0 ? content + " ".repeat(pad) : truncateToWidth(content, inner);
-		return bg(`${color("│")} ${reassert(padded)} ${color("│")}`);
-	};
-
-	const out: string[] = [bg(color(`╭${dashes}╮`))];
-	if (top !== undefined) out.push(row(top));
-	for (const line of lines) out.push(row(line));
-	out.push(bg(color(`╰${dashes}╯`)));
-	return out;
-}

package/src/glyphs.test.ts

@@ -1,48 +0,0 @@
-import { describe, expect, test } from "bun:test";
-import { visibleWidth } from "@earendil-works/pi-tui";
-import { checkboxGlyphs, selectionGlyph } from "./glyphs.js";
-
-describe("selectionGlyph", () => {
-	test("multi checked → ▣ / success", () => {
-		expect(
-			selectionGlyph({ multi: true, selected: false, checked: true }),
-		).toEqual({
-			glyph: "▣",
-			color: "success",
-		});
-	});
-
-	test("multi unchecked → ☐ / dim", () => {
-		expect(
-			selectionGlyph({ multi: true, selected: true, checked: false }),
-		).toEqual({
-			glyph: "☐",
-			color: "dim",
-		});
-	});
-
-	test("radio selected → ◉ / accent", () => {
-		expect(
-			selectionGlyph({ multi: false, selected: true, checked: false }),
-		).toEqual({
-			glyph: "◉",
-			color: "accent",
-		});
-	});
-
-	test("radio unselected → ○ / dim", () => {
-		expect(
-			selectionGlyph({ multi: false, selected: false, checked: false }),
-		).toEqual({
-			glyph: "○",
-			color: "dim",
-		});
-	});
-});
-
-describe("checkboxGlyphs", () => {
-	test("checked and unchecked have equal display width", () => {
-		const { checked, unchecked } = checkboxGlyphs();
-		expect(visibleWidth(checked)).toBe(visibleWidth(unchecked));
-	});
-});

package/src/once.ts

@@ -1,27 +0,0 @@
-/**
- * Per-instance idempotency guard for extension activation.
- *
- * pix-core (the meta-package) invokes this package's factory, and a standalone
- * install makes Pi invoke it again — sometimes against the SAME `pi`. We must
- * dedupe that. But Pi rebuilds the extension runtime on /new, /resume, /fork,
- * and /reload, handing the factory a BRAND-NEW `pi`; that must re-register.
- *
- * Keying the registry on the `pi` instance satisfies both: same instance =>
- * skip, new instance => run. The registry lives on globalThis because jiti
- * (`moduleCache: false`) re-evaluates this module on every load pass, so a
- * module-scoped WeakMap would not be shared between the aggregator pass and the
- * standalone pass within a single session.
- */
-export function once(pi: object, key: string, fn: () => void): void {
-	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
-	if (!g.__pixOnce) g.__pixOnce = new WeakMap<object, Set<string>>();
-	const registry = g.__pixOnce;
-	let loaded = registry.get(pi);
-	if (!loaded) {
-		loaded = new Set<string>();
-		registry.set(pi, loaded);
-	}
-	if (loaded.has(key)) return;
-	loaded.add(key);
-	fn();
-}