@fyeeme/pi-ask-user-lite 2.0.1 → 2.0.3

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [2.0.3] - 2026-10-01
+
+### Changed
+
+- `ask_user` now registers with pi 0.99 `exposure: "model-only"`: declared to the model, unreachable from codemode scripts and nested tool execution (ask-the-user is a model-level decision). No `outputSchema` is declared — model-only tools have no programmatic consumer of structured results. Model-facing behavior is unchanged.
+- Peer dependency floor raised to `@earendil-works/pi-coding-agent >= 0.99.0`; dev toolchain pinned to 0.99.2.
+
+## [2.0.2] - 2026-09-09
+
+### Fixed
+
+- `ask_user` no longer crashes in RPC mode: `ctx.ui.custom()` is TUI-only (it resolves to `undefined` under RPC), so RPC hosts now degrade to native `select`/`input` dialogs — multi-select via a "Done selecting" pick loop, "Other" via `input`, and timeout auto-selecting the recommended options.
+
 ## [2.0.1] - 2026-08-27
 
 First release under the `@fyeeme/pi-ask-user-lite` name; the entries below describe the 2.0.0 feature set published under the old `@fyeeme/pi-ask-user` name (code is identical apart from the rename).

package/index.ts

@@ -15,7 +15,7 @@
  * `/tree` inspection.
  */
 
-import type { ExtensionAPI, ExtensionCommandContext, ExtensionUIContext, ToolDefinition } from "@earendil-works/pi-coding-agent";
+import type { ExtensionAPI, ExtensionCommandContext, ExtensionToolContext, ExtensionUIContext, ToolDefinition } from "@earendil-works/pi-coding-agent";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { Editor, type EditorTheme, Key, matchesKey, Text, type TUI, visibleWidth, wrapTextWithAnsi } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
@@ -658,6 +658,80 @@ async function runAskDialog(
 	return { kind: action.kind, timedOut: action.timedOut === true, state };
 }
 
+// ---------------------------------------------------------------------------
+// RPC fallback: native select/input dialogs (no ctx.ui.custom — RPC hosts
+// resolve custom() to undefined; Mode Behavior doc requires a ctx.mode ===
+// "tui" guard around custom()). Mirrors omp's legacy per-question selector:
+// single-select per pick, a "Done selecting" terminal option for multi
+// questions, free-text Other via input, and timeout auto-selection of the
+// recommended options for every unanswered question.
+// ---------------------------------------------------------------------------
+
+type RpcAskUi = Pick<ExtensionUIContext, "select" | "input">;
+
+const RPC_DONE_LABEL = "Done selecting";
+/** Window after the deadline within which an `undefined` pick is attributed
+ *  to the UI-enforced timeout rather than a user Escape (omp heuristic). */
+const RPC_TIMEOUT_TOLERANCE_MS = 500;
+
+async function runAskDialogRpc(
+	ui: RpcAskUi,
+	questions: AskQuestion[],
+	options_: { deadline?: number; signal?: AbortSignal },
+): Promise<DialogOutcome> {
+	const answers: Array<{ selectedOptions: string[]; customInput?: string } | undefined> = questions.map(() => undefined);
+	const notes: Array<string | undefined> = questions.map(() => undefined);
+	const state: DialogState = { index: 0, cursors: [], checked: [], answers, notes };
+	const remainingMs = () => (options_.deadline === undefined ? undefined : Math.max(0, options_.deadline - Date.now()));
+	const timedOutAt = () => options_.deadline !== undefined && Date.now() >= options_.deadline - RPC_TIMEOUT_TOLERANCE_MS;
+	const dialogOpts = () => ({ timeout: remainingMs(), signal: options_.signal });
+
+	for (const [index, q] of questions.entries()) {
+		state.index = index;
+		const labels = addRecommendedSuffix(q.options, q.multi ? undefined : q.recommended);
+		const tail = q.multi ? [RPC_DONE_LABEL, OTHER_OPTION, CHAT_OPTION] : [OTHER_OPTION, CHAT_OPTION];
+		const selected = new Set<string>();
+
+		for (;;) {
+			const choice = await ui.select(q.question, [...labels, ...tail], dialogOpts());
+			if (choice === undefined) {
+				if (timedOutAt()) {
+					return { kind: "submit", timedOut: true, state };
+				}
+				return { kind: "cancel", timedOut: false, state };
+			}
+			if (choice === CHAT_OPTION) {
+				return { kind: "chat", timedOut: false, state };
+			}
+			if (choice === OTHER_OPTION) {
+				const custom = await ui.input(`Your answer for "${q.header ?? q.question}":`, undefined, dialogOpts());
+				if (custom === undefined) {
+					if (timedOutAt()) return { kind: "submit", timedOut: true, state };
+					return { kind: "cancel", timedOut: false, state };
+				}
+				if (custom.length > 0) {
+					answers[index] = { selectedOptions: [], customInput: custom }; // omp: re-selecting replaces
+					break;
+				}
+				continue; // empty submit declines the custom input — back to the rows
+			}
+			if (q.multi) {
+				if (choice === RPC_DONE_LABEL) {
+					answers[index] = { selectedOptions: [...selected] };
+					break;
+				}
+				const label = stripRecommendedSuffix(choice);
+				if (selected.has(label)) selected.delete(label);
+				else selected.add(label);
+				continue; // keep picking until Done
+			}
+			answers[index] = { selectedOptions: [stripRecommendedSuffix(choice)] };
+			break;
+		}
+	}
+	return { kind: "submit", timedOut: false, state };
+}
+
 function buildResults(questions: AskQuestion[], state: DialogState, timedOut: boolean): QuestionResult[] {
 	return questions.map((q, i) => {
 		const answer = state.answers[i];
@@ -687,7 +761,12 @@ const askUserTool: ToolDefinition<typeof AskParamsSchema, AskUserDetails> = {
 			"Ask the user one or more clarifying questions with selectable options. " +
 			"Use when choices have materially different tradeoffs the user must decide. " +
 			"The user can always provide free-form input via an automatic 'Other' option.",
-		promptSnippet: "Ask the user structured questions with options during execution",
+		// pi 0.99: model-only — declared to the model, unreachable from codemode
+	// scripts / nested tool execution (ask-the-user is a model-level decision;
+	// spec: ask-user-exposure). No outputSchema: model-only tools have no
+	// programmatic consumer of structuredContent.
+	exposure: "model-only",
+	promptSnippet: "Ask the user structured questions with options during execution",
 		promptGuidelines: [
 			"Use ask_user only after exhausting repo conventions, configs, and docs; reserve it for decisions whose options have materially different tradeoffs.",
 		],
@@ -725,7 +804,12 @@ const askUserTool: ToolDefinition<typeof AskParamsSchema, AskUserDetails> = {
 			const deadline =
 				typeof params.timeoutSeconds === "number" && params.timeoutSeconds > 0 ? Date.now() + params.timeoutSeconds * 1000 : undefined;
 
-			const outcome = await runAskDialog(ctx.ui, questions, { deadline, signal });
+			// ctx.ui.custom() is TUI-only (Mode Behavior): RPC hosts resolve it to
+			// undefined, so degrade to native select/input dialogs there.
+			const outcome =
+				ctx.mode === "tui"
+					? await runAskDialog(ctx.ui, questions, { deadline, signal })
+					: await runAskDialogRpc(ctx.ui, questions, { deadline, signal });
 
 			if (outcome.kind === "chat") {
 				const questionTexts = questions.map((q) => q.question).join("\n");
@@ -889,7 +973,10 @@ async function runAskDemo(ctx: ExtensionCommandContext): Promise<void> {
 	const run = async (label: string, params: AskParams): Promise<string> => {
 		ctx.ui.notify(`ask-demo ${label} — follow the dialog instructions`, "info");
 		try {
-			const result = await askUserTool.execute(`demo-${label}`, params, undefined, undefined, ctx);
+			// pi 0.99 split tool and command contexts into sibling branches; the demo
+			// only exercises ExtensionContext members (ui/abort), so the cast is safe.
+			const toolCtx = ctx as unknown as ExtensionToolContext;
+			const result = await askUserTool.execute(`demo-${label}`, params, undefined, undefined, toolCtx);
 			const text = result.content[0]?.type === "text" ? result.content[0].text : "";
 			ctx.ui.notify(`ask-demo ${label} → ${text.split("\n").filter(Boolean).join(" | ").slice(0, 200)}`, "info");
 			return text;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@fyeeme/pi-ask-user-lite",
-	"version": "2.0.1",
+	"version": "2.0.3",
 	"description": "Structured ask_user tool for pi — lets the LLM surface clarifying questions with options, multi-select, recommended defaults, and free-form Other input.",
 	"type": "module",
 	"license": "MIT",
@@ -43,13 +43,13 @@
 		"typecheck": "tsc"
 	},
 	"peerDependencies": {
-		"@earendil-works/pi-coding-agent": ">=0.84.1",
-		"@earendil-works/pi-tui": ">=0.84.1",
+		"@earendil-works/pi-coding-agent": ">=0.99.0",
+		"@earendil-works/pi-tui": ">=0.99.0",
 		"typebox": ">=1.0.0"
 	},
 	"devDependencies": {
-		"@earendil-works/pi-coding-agent": "0.84.1",
-		"@earendil-works/pi-tui": "0.84.1",
+		"@earendil-works/pi-coding-agent": "0.99.2",
+		"@earendil-works/pi-tui": "0.99.2",
 		"@types/node": "22.19.19",
 		"typebox": "1.1.38",
 		"typescript": "5.9.3",