@crouton-kit/humanloop 0.3.14 → 0.3.16

package/dist/cli.js

@@ -212,7 +212,6 @@ const REQUEST_SCHEMA = {
                                 id: { type: 'string' },
                                 label: { type: 'string' },
                                 description: { type: 'string' },
-                                shortcut: { type: 'string', description: 'Single char. Auto-assigned if absent.' },
                             },
                         },
                     },

package/dist/inbox/deck-schema.d.ts

@@ -4,7 +4,6 @@ export declare const interactionOptionSchema: z.ZodObject<{
     id: z.ZodString;
     label: z.ZodString;
     description: z.ZodOptional<z.ZodString>;
-    shortcut: z.ZodOptional<z.ZodString>;
 }, z.core.$strip>;
 export declare const preAnswerSchema: z.ZodObject<{
     selectedOptionId: z.ZodOptional<z.ZodString>;
@@ -30,7 +29,6 @@ export declare const deckSchema: z.ZodObject<{
             id: z.ZodString;
             label: z.ZodString;
             description: z.ZodOptional<z.ZodString>;
-            shortcut: z.ZodOptional<z.ZodString>;
         }, z.core.$strip>>;
         multiSelect: z.ZodOptional<z.ZodBoolean>;
         allowFreetext: z.ZodOptional<z.ZodBoolean>;

package/dist/inbox/deck-schema.js

@@ -8,7 +8,10 @@ export const interactionOptionSchema = z.object({
     id: z.string().min(1),
     label: z.string().min(1),
     description: z.string().optional(),
-    shortcut: z.string().optional(),
+    // No author-settable `shortcut`: option shortcuts are auto-assigned by the
+    // TUI (assignShortcuts). zod strips unknown keys, so any author-supplied
+    // `shortcut` is silently dropped here — this is the enforcement boundary that
+    // stops a deck from shadowing a reserved key (e.g. `c` = comment).
 });
 export const preAnswerSchema = z.object({
     selectedOptionId: z.string().optional(),

package/dist/tui/input.js

@@ -34,6 +34,10 @@ export function handleKeypress(input, key, state, render, exit) {
         exit();
         return;
     }
+    // Clear any transient hint from the previous keypress. Handlers below may set
+    // a fresh one (e.g. an empty multi-select Enter), so it survives exactly one
+    // render cycle.
+    state.hint = undefined;
     if (state.inputMode) {
         handleInputMode(input, key, state, render);
         checkAutoExit(state, exit);
@@ -53,9 +57,21 @@ export function handleKeypress(input, key, state, render, exit) {
     }
 }
 function checkAutoExit(state, exit) {
-    if (state.phase === 'final' && state.responses.size >= state.interactions.length) {
-        exit();
-    }
+    if (state.phase !== 'final')
+        return;
+    if (state.responses.size < state.interactions.length)
+        return;
+    // Multi-select commits route THROUGH the Summary/confirm screen instead of
+    // auto-exiting on the first Enter: the commit advances the deck to `final`,
+    // but the human must press Enter again (handleFinal) to actually submit.
+    // The interaction that pushed us into `final` is still at currentIndex — the
+    // advance helpers leave it there when they fall through to `final` — so a
+    // multi-select there means "just confirmed a set, await deliberate submit".
+    // Single-select keeps its submit-on-pick fast path (auto-exit below).
+    const justCommitted = state.interactions[state.currentIndex];
+    if (justCommitted?.multiSelect === true)
+        return;
+    exit();
 }
 // ── Overview ─────────────────────────────────────────────────────────────────
 function handleOverview(input, key, state, render, exit) {
@@ -225,6 +241,15 @@ function handleInteractionAction(input, key, state, interaction, render) {
     // and advances; single-select picks that one option.
     if (key.return && state.selectedAction < opts.length) {
         if (interaction.multiSelect) {
+            const checked = state.responses.get(interaction.id)?.selectedOptionIds ?? [];
+            if (checked.length === 0) {
+                // Accidental Enter with nothing toggled is a no-op: don't finalize or
+                // advance. A deliberate empty finish is still reachable via `q` →
+                // overview → finish (partial); freetext-only via the [c] row.
+                state.hint = 'Select at least one option (space to toggle), or q to skip';
+                render();
+                return;
+            }
             commitMulti(state, interaction);
             advanceToNextUnanswered(state);
             render();
@@ -301,6 +326,15 @@ function handleInputMode(input, key, state, render) {
         render();
         return;
     }
+    // Ctrl-U: delete to the start of the line (readline "unix-line-discard").
+    // iTerm2 maps Cmd+Backspace to a bare 0x15, which arrives here as ctrl+'u'.
+    // The buffer is end-anchored (no mid-string cursor), so "to line start" is
+    // the entire buffer.
+    if (key.ctrl && input === 'u') {
+        mode.buffer = '';
+        render();
+        return;
+    }
     if (key.backspace) {
         const chars = [...mode.buffer];
         chars.pop();

package/dist/tui/render.js

@@ -322,6 +322,12 @@ export function renderItemReview(state, cols, rows) {
     else {
         postLines.push(...renderActions(interaction, state.selectedAction, maxW, response));
     }
+    // Transient hint (e.g. an empty multi-select Enter that was rejected). Sits
+    // just above the footer; cleared on the next keypress.
+    if (state.hint !== undefined && state.hint.length > 0) {
+        postLines.push('');
+        postLines.push(`  ${YELLOW}${sanitize(state.hint)}${RESET}`);
+    }
     // Window the body
     const reservedRows = preLines.length + postLines.length + 1; // +1 for footer
     const bodyHeight = Math.max(1, rows - reservedRows);

package/dist/tui/terminal.js

@@ -27,7 +27,12 @@ export function parseKeypress(data) {
     }
     // Alt+Backspace: terminals send ESC followed by DEL/BS. Must precede the
     // bare-ESC check so the two-byte sequence isn't swallowed as plain escape.
-    if (str === '\x1b\x7f' || str === '\x1b\b') {
+    // iTerm2 (and many readline configs) instead map Option/Alt+Backspace to a
+    // bare Ctrl-W (0x17, the "word-erase" byte) with no ESC prefix — fold it into
+    // the same meta+backspace key so the word-delete path fires either way. This
+    // must also precede the generic Ctrl-<letter> branch below, which would
+    // otherwise turn 0x17 into a literal 'w' in the buffer.
+    if (str === '\x1b\x7f' || str === '\x1b\b' || str === '\x17') {
         key.meta = true;
         key.backspace = true;
         return { input: '', key };

package/dist/types.d.ts

@@ -4,6 +4,9 @@ export interface InteractionOption {
     id: string;
     label: string;
     description?: string;
+    /** Auto-assigned by the TUI (assignShortcuts) — NOT author-settable. Omitted
+     *  from the deck schema so a deck can't create a shortcut that shadows a
+     *  reserved key (e.g. `c` = comment, which would submit+close on single-select). */
     shortcut?: string;
 }
 /**
@@ -118,6 +121,9 @@ export interface TuiState {
     selectedAction: number;
     detailExpanded: boolean;
     scrollOffset: number;
+    /** Transient one-line notice shown in item-review (e.g. an empty multi-select
+     *  Enter that was rejected). Cleared on the next keypress. */
+    hint?: string;
     persist?: () => void;
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@crouton-kit/humanloop",
-  "version": "0.3.14",
+  "version": "0.3.16",
   "description": "Human-in-the-loop decision TUI — agents write questions, humans answer them",
   "type": "module",
   "main": "dist/index.js",