@xynogen/pix-ask 0.2.2 → 0.2.5

package/README.md

@@ -4,7 +4,7 @@ Pi tool — structured questionnaire UI (`ask_user`).
 
 ## What it does
 
-Registers the `ask_user` tool in Pi. When the agent needs to resolve ambiguous requirements, it presents up to 4 structured multiple-choice questions in a TUI dialog. Each question requires 2–4 options with labels and descriptions; supports multi-select and markdown side-by-side previews for richer context. Single-select questions auto-append a "Type something." row for free-form input and always include "Chat about this" as an escape hatch. In non-interactive (RPC/JSON) mode, falls back to text-based prompts. The agent uses this tool before proceeding rather than guessing at intent.
+Registers the `ask_user` tool in Pi. When the agent needs to resolve ambiguous requirements, it presents up to 4 structured multiple-choice questions in a TUI dialog. Each question requires 2–4 options with labels and descriptions; supports multi-select and markdown side-by-side previews for richer context. Single-select questions (without a preview) auto-append a "Type something." row for free-form input; multi-select questions append a "Next" row to advance. Single-select questions with a `preview` skip the free-form row to make room for the side-by-side layout. In non-interactive (RPC/JSON) mode, falls back to text-based prompts. The agent uses this tool before proceeding rather than guessing at intent.
 
 ## Install
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@xynogen/pix-ask",
-	"version": "0.2.2",
-	"description": "Pi tool \u2014 structured questionnaire UI (ask_user)",
+	"version": "0.2.5",
+	"description": "Pi tool — structured questionnaire UI (ask_user)",
 	"type": "module",
 	"main": "src/index.ts",
 	"scripts": {
@@ -35,7 +35,7 @@
 	},
 	"dependencies": {
 		"typebox": "^1.1.38",
-		"@xynogen/pix-pretty": "*"
+		"@xynogen/pix-pretty": "^1.7.9"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*",

package/src/once.ts

@@ -14,7 +14,8 @@
  */
 export function once(pi: object, key: string, fn: () => void): void {
 	const g = globalThis as { __pixOnce?: WeakMap<object, Set<string>> };
-	const registry = (g.__pixOnce ??= new 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>();

package/src/questionnaire.ts

@@ -65,7 +65,9 @@ export class AskQuestionnaire extends Container {
 	// ── Accessors ──────────────────────────────────────────────────────
 
 	private get currentQ(): QuestionData {
-		return this.params.questions[this.currentIndex]!;
+		const q = this.params.questions[this.currentIndex];
+		if (!q) throw new Error("currentIndex out of bounds");
+		return q;
 	}
 
 	private get filteredOptions(): OptionData[] {
@@ -209,7 +211,7 @@ export class AskQuestionnaire extends Container {
 		const q = this.currentQ;
 		if (prev.kind === "multi") {
 			for (let i = 0; i < q.options.length; i++) {
-				if (prev.selected?.includes(q.options[i]!.label)) {
+				if (prev.selected?.includes(q.options[i]?.label ?? "")) {
 					this.multiChecked.add(i);
 				}
 			}
@@ -354,7 +356,8 @@ export class AskQuestionnaire extends Container {
 					this.selectedOptionIndex = Math.min(idx, this.totalItems - 1);
 					this.refresh();
 				} else {
-					const opt = this.filteredOptions[idx]!;
+					const opt = this.filteredOptions[idx];
+					if (!opt) return;
 					this.recordAnswer("option", opt.label, undefined, opt.preview);
 					this.nextQuestion();
 				}
@@ -434,7 +437,8 @@ export class AskQuestionnaire extends Container {
 		const pad = " ".repeat(LABEL_COL);
 
 		for (let i = start; i < end; i++) {
-			const item = items[i]!;
+			const item = items[i];
+			if (!item) continue;
 			const sel = i === this.selectedOptionIndex;
 			const ptr = sel ? t.fg("accent", "→") : " ";
 

package/src/rpc.ts

@@ -6,14 +6,18 @@ import type { QuestionAnswer, QuestionnaireResult } from "./types.js";
 // Used when ctx.hasUI is false (headless / JSON / print mode).
 
 export async function rpcFallback(
-	ui: { select: Function; input: Function },
+	ui: {
+		select(title: string, options: string[]): Promise<string | undefined>;
+		input(title: string, placeholder?: string): Promise<string | undefined>;
+	},
 	params: Params,
 ): Promise<QuestionnaireResult> {
 	const answers: QuestionAnswer[] = [];
 	let cancelled = false;
 
 	for (let i = 0; i < params.questions.length; i++) {
-		const q = params.questions[i]!;
+		const q = params.questions[i];
+		if (!q) break;
 		const header = q.header;
 
 		if (q.multiSelect) {
@@ -69,7 +73,7 @@ export async function rpcFallback(
 				const opt = q.options.find(
 					(o) =>
 						chosen === o.label || `${o.label} — ${o.description}` === chosen,
-				)!;
+				);
 				answers.push({
 					questionIndex: i,
 					question: q.question,