@xynogen/pix-ask 0.2.3 → 0.2.6

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@xynogen/pix-ask",
-	"version": "0.2.3",
-	"description": "Pi tool \u2014 structured questionnaire UI (ask_user)",
+	"version": "0.2.6",
+	"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/index.ts

@@ -106,7 +106,7 @@ export default function registerAsk(pi: ExtensionAPI): void {
 			renderCall(args, theme) {
 				const questions = Array.isArray(args.questions) ? args.questions : [];
 				const count = questions.length;
-				const firstQ = (questions[0]?.question ?? "") as string;
+				const firstQ = questions[0]?.question ?? "";
 				let text = theme.fg("toolTitle", theme.bold(`ask (${count}) `));
 				text += theme.fg("muted", firstQ);
 				if (count > 1) text += theme.fg("dim", ` +${count - 1} more`);

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[] {
@@ -170,7 +172,7 @@ export class AskQuestionnaire extends Container {
 		} else if (item.kind === "next") {
 			const selected = Array.from(this.multiChecked)
 				.sort((a, b) => a - b)
-				.map((i) => this.currentQ.options[i]?.label);
+				.map((i) => this.currentQ.options[i]?.label ?? "");
 			if (selected.length === 0) {
 				this.cancel();
 				return;
@@ -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) {
@@ -32,7 +36,7 @@ export async function rpcFallback(
 				.split(",")
 				.map((s) => Number(s.trim()))
 				.filter((n) => n >= 1 && n <= q.options.length);
-			const selected = indices.map((n) => q.options[n - 1]?.label);
+			const selected = indices.map((n) => q.options[n - 1]?.label ?? "");
 			if (selected.length > 0) {
 				answers.push({
 					questionIndex: i,
@@ -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,