@hank-warren/pi-ask-user-question 0.6.0 → 0.6.1

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # @hank-warren/pi-ask-user-question
 
+## 0.6.1
+
+### Patch Changes
+
+- 1035138: Report a waiting question to Herdr as `blocked`.
+
+  `ask_user_question` and Plan Mode's `plan_mode_question` now emit the same
+  `herdr:blocked` event pi-auto-permissions emits for an approval prompt, labelled
+  `question` / `plan question` and cleared in a `finally`, so Herdr's pi
+  integration shows a session waiting on a question as `blocked` rather than
+  `working`. A supervising agent in another pane can wait on that state and answer
+  the dialog. No-op outside Herdr.
+
 ## 0.6.0
 
 ### Minor Changes

package/README.md

@@ -123,6 +123,10 @@ pi.events.on("hank:ask-user:prompt", ({ questions }) => {
 Channel names are immutable and payloads are append-only — see
 [`events.ts`](./events.ts) for the full stability policy.
 
+## Changelog
+
+See [CHANGELOG.md](CHANGELOG.md) for release history.
+
 ## License
 
 MIT

package/ask-user-question.ts

@@ -65,9 +65,23 @@ function emitPrompt(pi: ExtensionAPI, params: AskUserParams): void {
 	pi.events.emit(ASK_USER_PROMPT_EVENT, payload);
 }
 
+/**
+ * Label Herdr shows for a pane waiting on this dialog. A supervising agent
+ * reads it to tell a question from an Auto Permissions prompt (whose label
+ * is the gate name, e.g. `shell command`).
+ */
+export const HERDR_BLOCKED_LABEL = "question";
+
 function emitBlocked(pi: ExtensionAPI, active: boolean): void {
 	const payload: AskUserBlockedEventPayload = { active };
 	pi.events.emit(ASK_USER_BLOCKED_EVENT, payload);
+	// Herdr's pi integration turns `herdr:blocked` into `agent_status: "blocked"`,
+	// which is how an orchestrator in another pane learns this session is
+	// waiting on a human rather than working. Same contract pi-auto-permissions
+	// uses (`setHerdrBlocked`); duplicated rather than imported, because a
+	// questionnaire must not pull in the permissions engine. No-op outside Herdr.
+	if (process.env.HERDR_ENV !== "1") return;
+	pi.events.emit("herdr:blocked", active ? { active: true, label: HERDR_BLOCKED_LABEL } : { active: false });
 }
 
 export function registerTool(pi: ExtensionAPI): void {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hank-warren/pi-ask-user-question",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "Structured questionnaire tool for Pi with numbered options, digit hotkeys and Tab-to-comment, composed from the shared permission-selector component.",
   "type": "module",
   "keywords": [
@@ -24,7 +24,7 @@
   },
   "homepage": "https://github.com/hank-warren/pi-extensions/tree/main/packages/pi-ask-user-question#readme",
   "engines": {
-    "node": ">=18.0.0"
+    "node": ">=22.19.0"
   },
   "pi": {
     "extensions": [

package/questionnaire.ts

@@ -25,7 +25,7 @@ import {
 	type QuestionnaireResult,
 } from "./tool/schema.ts";
 
-export interface SelectableRow {
+interface SelectableRow {
 	value: string;
 	label: string;
 	description?: string;

package/tool/envelope.ts

@@ -14,7 +14,7 @@ export const DECLINE_MESSAGE = "User declined to answer questions";
 export const ENVELOPE_PREFIX = "User has answered your questions:";
 export const ENVELOPE_SUFFIX = "You can now continue with the user's answers in mind.";
 
-export interface ToolResult {
+interface ToolResult {
 	content: Array<{ type: "text"; text: string }>;
 	details: QuestionnaireResult;
 }
@@ -28,7 +28,7 @@ export function buildToolResult(text: string, details: QuestionnaireResult): Too
  * preview and the user's note appended when present. Order and wording match
  * rpiv's envelope and are pinned by tests.
  */
-export function buildAnswerSegment(a: QuestionAnswer): string {
+function buildAnswerSegment(a: QuestionAnswer): string {
 	const parts: string[] = [`"${a.question}"="${a.answer}"`];
 	if (a.preview && a.preview.length > 0) parts.push(`selected preview: ${a.preview}`);
 	if (a.notes && a.notes.length > 0) parts.push(`user notes: ${a.notes}`);

package/tool/schema.ts

@@ -12,13 +12,13 @@ import { Type } from "typebox";
 export const TOOL_NAME = "ask_user_question";
 
 export const MIN_OPTIONS = 2;
-export const MAX_OPTIONS = 4;
+const MAX_OPTIONS = 4;
 /**
  * Multi-select questions get a larger cap. Checkboxes are a shortlist UI, not a
  * pick-one UI, so six is where a list stops fitting comfortably above the input
  * dock — not where it stops being a decision.
  */
-export const MAX_MULTI_OPTIONS = 6;
+const MAX_MULTI_OPTIONS = 6;
 export const MIN_QUESTIONS = 1;
 export const MAX_QUESTIONS = 4;
 
@@ -42,7 +42,7 @@ export const CUSTOM_ANSWER_VALUE = "\u0000custom-answer";
 /** Separator joining a multi-select answer's labels into `answer` (spec §5.4). */
 export const MULTI_SELECT_JOIN = ", ";
 
-export const OptionSchema = Type.Object({
+const OptionSchema = Type.Object({
 	label: Type.String({
 		description:
 			"The display text for this option that the user will see and select. Aim for 1-5 words, but there is no hard limit: a longer label is fine when it genuinely helps, and long labels wrap in the dialog rather than being rejected.",
@@ -59,7 +59,7 @@ export const OptionSchema = Type.Object({
 	),
 });
 
-export const QuestionSchema = Type.Object({
+const QuestionSchema = Type.Object({
 	question: Type.String({
 		description:
 			"The complete question to ask the user. Should be clear, specific, and end with a question mark.",
@@ -91,13 +91,13 @@ export const QuestionParamsSchema = Type.Object({
 	}),
 });
 
-export interface OptionParams {
+interface OptionParams {
 	label: string;
 	description: string;
 	preview?: string;
 }
 
-export interface QuestionParams {
+interface QuestionParams {
 	question: string;
 	header: string;
 	/** Checkbox mode: the user may check several options. Default false. */

package/tool/validate.ts

@@ -24,7 +24,7 @@ import {
 	RESERVED_LABELS,
 } from "./schema.ts";
 
-export type ValidationCode =
+type ValidationCode =
 	| "bad_question_count"
 	| "bad_option_count"
 	| "header_too_long"
@@ -33,7 +33,7 @@ export type ValidationCode =
 	| "duplicate_label"
 	| "empty_question";
 
-export interface ValidationError {
+interface ValidationError {
 	code: ValidationCode;
 	message: string;
 }

package/view/dialog.ts

@@ -64,11 +64,11 @@ import type { QuestionnaireSession } from "../questionnaire.ts";
 import type { QuestionnaireResult } from "../tool/schema.ts";
 
 /** Structural subset of pi's Theme used here; keeps the view unit-testable. */
-export interface DialogTheme {
+interface DialogTheme {
 	fg(role: string, text: string): string;
 }
 
-export interface DialogOptions {
+interface DialogOptions {
 	session: QuestionnaireSession;
 	theme?: DialogTheme;
 	/** Called exactly once with the final outcome. */