@hank-warren/pi-ask-user-question 0.5.0 → 0.5.2

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @hank-warren/pi-ask-user-question
 
+## 0.5.2
+
+### Patch Changes
+
+- 3fbf632: Make backspace delete a whole character rather than half a surrogate pair. Both text-entry surfaces sliced UTF-16 code units, so one backspace after an emoji left a lone surrogate — an ill-formed string that was then submitted as an approval note or a free-text answer.
+
+  `pi-permission-selector` adds a code-point-aware `removeLastCharacter` export (additive; the frozen shared surface gains a name and loses none), and `pi-ask-user-question` uses it for the custom-answer field.
+
+- Updated dependencies [3fbf632]
+  - @hank-warren/pi-permission-selector@1.3.0
+
+## 0.5.1
+
+### Patch Changes
+
+- 464fe1e: fix: pasting into the ask-user-question custom-answer field was silently dropped
+
+  pi-tui re-wraps pastes in bracketed-paste markers (`\x1b[200~ … \x1b[201~`) before they reach `handleInput`. The questionnaire dialog's free-text field only accepted printable chunks, so a paste — which contains ESC — fell through every key predicate into the inert branch and vanished. The note editor was unaffected because `handleCommentKey` buffers its own pastes.
+
+  The bracketed-paste state machine inside `handleCommentKey` is now extracted as a new shared export, `consumePasteChunk` (pi-permission-selector minor), and the custom-answer field routes input through it: single-chunk and chunk-spanning pastes insert, multi-line pastes flatten to one line, and input trailing the end marker is re-dispatched as ordinary keys.
+
+- Updated dependencies [464fe1e]
+  - @hank-warren/pi-permission-selector@1.2.0
+
 ## 0.5.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hank-warren/pi-ask-user-question",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "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": [
@@ -46,7 +46,7 @@
     "CHANGELOG.md"
   ],
   "dependencies": {
-    "@hank-warren/pi-permission-selector": "^1.1.0"
+    "@hank-warren/pi-permission-selector": "^1.3.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",

package/view/dialog.ts

@@ -39,6 +39,7 @@
  */
 
 import {
+	consumePasteChunk,
 	isBackspaceKey,
 	isCharKey,
 	isEnterKey,
@@ -48,6 +49,7 @@ import {
 	isRightKey,
 	isShiftTabKey,
 	isTabKey,
+	removeLastCharacter,
 } from "@hank-warren/pi-permission-selector/keys.ts";
 import { OptionSelector, type SelectorOption } from "@hank-warren/pi-permission-selector/selector.ts";
 import {
@@ -99,6 +101,14 @@ const PREVIEW_MAX_ROWS = 12;
 interface TabState {
 	selector: OptionSelector;
 	customText?: string;
+	/**
+	 * In-progress bracketed paste aimed at the custom-answer field. pi-tui wraps
+	 * pastes in `\x1b[200~ … \x1b[201~` and the markers can span input chunks;
+	 * a paste chunk fails every key predicate, so without this buffer pasting
+	 * into the field was silently dropped (the note editor was unaffected —
+	 * handleCommentKey buffers its own pastes).
+	 */
+	pasteBuffer?: string;
 	pendingNotes?: string;
 	/**
 	 * Labels checked alongside the sentinel row on a multi-select question, held
@@ -371,10 +381,26 @@ export class QuestionnaireDialog {
 		// Free-text mode. Every check below MUST use a shared predicate — see
 		// rule 1. Esc unwinds to the option list rather than cancelling the
 		// dialog: one Esc never discards more than one layer.
+		//
+		// Bracketed pastes come first: a paste chunk contains ESC, so it would
+		// otherwise fall through every predicate into the inert branch and vanish.
+		const paste = consumePasteChunk(tab.pasteBuffer, keyData);
+		if (paste !== undefined) {
+			tab.pasteBuffer = paste.buffer;
+			if (paste.text !== undefined) {
+				tab.customText += paste.text;
+				this.repaint();
+			}
+			// Ordinary input after the end marker (e.g. a trailing Enter) goes back
+			// through normal key handling.
+			if (paste.rest) this.handleInput(paste.rest);
+			return;
+		}
 		if (isEscapeKey(keyData)) {
 			// Back to the option list. The selector still holds every tick, so a
 			// multi-select question is exactly as the user left it.
 			tab.customText = undefined;
+			tab.pasteBuffer = undefined;
 			tab.pendingNotes = undefined;
 			tab.pendingSelected = undefined;
 			this.repaint();
@@ -394,13 +420,16 @@ export class QuestionnaireDialog {
 				this.session.recordAnswer(text, { custom: true, notes: tab.pendingNotes });
 			}
 			tab.customText = undefined;
+			tab.pasteBuffer = undefined;
 			tab.pendingNotes = undefined;
 			tab.pendingSelected = undefined;
 			this.afterAnswer();
 			return;
 		}
 		if (isBackspaceKey(keyData)) {
-			tab.customText = tab.customText.slice(0, -1);
+			// Code-point aware: a UTF-16 slice would split a surrogate pair and
+			// leave a lone surrogate in the answer that reaches the model.
+			tab.customText = removeLastCharacter(tab.customText);
 			this.repaint();
 			return;
 		}