@vincemakes/kiso-tui 0.40.0 → 0.40.2

package/dist/compositor.d.ts

@@ -235,6 +235,10 @@ export declare class Body {
         count: number;
         first: number;
     } | null;
+    visiblePickWindow(): {
+        first: number;
+        size: number;
+    } | null;
     /** SIGWINCH: clear the OLD live area (recorded geometry, ED only —
      *  zero LF, zero \x1b[3J — the shell history untouched), then the
      *  full-redraw path at the NEW geometry (O(height), zero replay).
@@ -395,6 +399,12 @@ export declare class Dock {
         count: number;
         first: number;
     } | null;
+    /** B (review of this round): the pick panel's window for the frame last
+     *  drawn — what the digit keys must name. */
+    visiblePickWindow(): {
+        first: number;
+        size: number;
+    } | null;
     setStatus(text: string, hint?: string | null): void;
     setTail(tail: string): void;
     /** W21: bind the editor's panel state — the PanelSelect slot

package/dist/compositor.js

@@ -52,6 +52,7 @@ import { leadWidth } from "./width.js"; // W23: the ONE width authority (the edi
 // for four reads, so an ask can never render half as an approval.
 import { panelFrameOf, panelLeadOf, panelStatusOf } from "./ask-panel.js";
 import { MOUSE_OFF } from "./editor.js";
+import { pickWindowOf } from "./approval-panel.js";
 import { atPanelRows, bandHeader } from "./at-picker.js";
 // TUI2-R2 ②: the session picker's rows — the band's third occupant.
 import { sessionPickerRows } from "./session-picker.js";
@@ -1241,6 +1242,13 @@ export class Body {
     panelOptionRows() {
         return this.#panelRowSpan;
     }
+    /** B (review of this round): the pick panel's window for the frame last
+     *  drawn — the renderer's own value, not a second derivation. `null` when
+     *  no pick panel was on screen. */
+    #pickWin = null;
+    visiblePickWindow() {
+        return this.#pickWin;
+    }
     /** SIGWINCH: clear the OLD live area (recorded geometry, ED only —
      *  zero LF, zero \x1b[3J — the shell history untouched), then the
      *  full-redraw path at the NEW geometry (O(height), zero replay).
@@ -1853,6 +1861,7 @@ export class Body {
         // at when they clicked, which is this one.
         this.#panelRowSpan =
             live.panelSpan === null ? null : { top: liveTop + live.panelSpan.offset, count: live.panelSpan.count, first: live.panelSpan.first };
+        this.#pickWin = live.pickWin ?? null;
         // 5. the frame bytes.
         this.#paint(W, H, liveTop, liveLines, liveRowsTotal, chrome);
     }
@@ -1953,7 +1962,11 @@ export class Body {
             // fires on it. W22: the queue band sits below the panel — the
             // cap shrinks by it.
             const frame = panelFrameOf(panel, W, capped);
-            return { lines: frame.rows, panelSpan: frame.options };
+            // B (review of this round): the pick window THIS frame draws. The size
+            // depends on the budget, so the renderer is the only producer — the
+            // digit keys are handed this value instead of deriving a second one.
+            const pick = panel.pick ?? null;
+            return { lines: frame.rows, panelSpan: frame.options, pickWin: pick === null ? null : pickWindowOf(panel.view, pick.cursor, pick.phase, capped) };
         }
         return { lines: this.#liveProjection(W, ctx, cap), panelSpan: null };
     }
@@ -2869,6 +2882,11 @@ export class Dock {
     panelOptionRows() {
         return compositorRef?.panelOptionRows() ?? null;
     }
+    /** B (review of this round): the pick panel's window for the frame last
+     *  drawn — what the digit keys must name. */
+    visiblePickWindow() {
+        return compositorRef?.visiblePickWindow() ?? null;
+    }
     setStatus(text, hint) {
         compositorRef?.setStatus(text, hint ?? null);
     }

package/dist/editor.d.ts

@@ -27,13 +27,20 @@ import { type PanelState, type PanelVerdict, type PanelView, type SaferAnswer }
 import { type AtItem, type AtMatch } from "./at-picker.js";
 import { type SessionCardView, type SessionPickState } from "./session-picker.js";
 /**
- * TUI2-R3v2 ② — the mouse-mode bytes, stated once.
+ * TUI2-R3v2 ②, corrected by the owner's ruling of 2026-09-21 (finding
+ * DC-56) — the mouse-mode bytes, stated once.
  *
  * ?1000 is the button-event report and ?1006 is the SGR encoding that
  * makes it parseable past column 95 (the legacy X10 encoding packs the
- * coordinate into one byte and simply breaks on a wide terminal). Both
- * go on together and come off together; a terminal left with either one
- * set is a terminal that prints escape bytes at the shell prompt.
+ * coordinate into one byte and simply breaks on a wide terminal). kiso
+ * still RESETS them — on entry (a previous process may have died with
+ * reporting left on) and on exit — but it no longer ENABLES them, because
+ * a terminal that is reporting stops scrolling its own scrollback: a
+ * surface that takes the mouse takes the history with it. kiso's
+ * transcript lives in that scrollback by design, and the first version of
+ * this feature locked it exactly where a person most wants to read back —
+ * under a blocking question. The history wins: no surface claims the
+ * mouse, and every surface keeps its keys.
  */
 /**
  * R5 — the viewer's key table, as a pure function of the input chunk.
@@ -45,7 +52,6 @@ import { type SessionCardView, type SessionPickState } from "./session-picker.js
  * different route.
  */
 export declare function viewerCommand(text: string): "up" | "down" | "toggle" | "all" | "pageUp" | "pageDown" | "home" | "end" | "close" | null;
-export declare const MOUSE_ON = "\u001B[?1000h\u001B[?1006h";
 export declare const MOUSE_OFF = "\u001B[?1000l\u001B[?1006l";
 export declare const PROMPT = "\u258C ";
 export declare const PROMPT_WIDTH: number;
@@ -221,6 +227,12 @@ export declare class Editor {
         count: number;
         first?: number;
     } | null) | null): void;
+    /** B: the pick window the last frame drew — the Dock's value, not the
+     *  editor's guess. Same pull-on-demand shape as `bindPanelRows`. */
+    bindPickWindow(fn: (() => {
+        first: number;
+        size: number;
+    } | null) | null): void;
     /** R3a — cross-session input history: seed the recall buffer and
      *  register the append sink. The cap and the adjacent-duplicate
      *  collapse are unchanged; the seed takes the TAIL of what the CLI

package/dist/editor.js

@@ -46,13 +46,20 @@ import { PickInput } from "./pick-input.js";
 // "you>" text is gone (the brick IS the prompt; the pipe path's readline
 // prompt keeps its own "you> " — v2a line mode, byte-for-byte).
 /**
- * TUI2-R3v2 ② — the mouse-mode bytes, stated once.
+ * TUI2-R3v2 ②, corrected by the owner's ruling of 2026-09-21 (finding
+ * DC-56) — the mouse-mode bytes, stated once.
  *
  * ?1000 is the button-event report and ?1006 is the SGR encoding that
  * makes it parseable past column 95 (the legacy X10 encoding packs the
- * coordinate into one byte and simply breaks on a wide terminal). Both
- * go on together and come off together; a terminal left with either one
- * set is a terminal that prints escape bytes at the shell prompt.
+ * coordinate into one byte and simply breaks on a wide terminal). kiso
+ * still RESETS them — on entry (a previous process may have died with
+ * reporting left on) and on exit — but it no longer ENABLES them, because
+ * a terminal that is reporting stops scrolling its own scrollback: a
+ * surface that takes the mouse takes the history with it. kiso's
+ * transcript lives in that scrollback by design, and the first version of
+ * this feature locked it exactly where a person most wants to read back —
+ * under a blocking question. The history wins: no surface claims the
+ * mouse, and every surface keeps its keys.
  */
 /**
  * R5 — the viewer's key table, as a pure function of the input chunk.
@@ -105,7 +112,6 @@ const ALT_WORD = new Map([
     ["\x7f", "killBack"],
     ["\x08", "killBack"],
 ]);
-export const MOUSE_ON = "\x1b[?1000h\x1b[?1006h";
 export const MOUSE_OFF = "\x1b[?1000l\x1b[?1006l";
 export const PROMPT = "▌ ";
 export const PROMPT_WIDTH = displayWidth(PROMPT);
@@ -314,12 +320,16 @@ export class Editor {
     /** TUI2-R3v2 ①: one-shot — a panel that just closed swallows the
      *  habitual trailing enter rather than submitting the restored draft. */
     #swallowEnter = false;
-    /** TUI2-R3v2 ②: whether SGR 1006 reporting is currently enabled. */
-    #mouseOn = false;
     /** TUI2-R3v2 ②: where the compositor put the panel's option rows this
      *  frame (absolute 1-based screen rows). The editor owns no geometry —
      *  it asks the surface that placed them. */
     #panelRows = null;
+    /** B (review of this round): the pick panel's window, published by the
+     *  surface that DREW it — the frame's budget decides the size, so the
+     *  editor PULLS it here instead of letting the input layer re-derive one
+     *  (two derivations are two sizes, and on a short terminal the digits named
+     *  rows nobody could see). */
+    #pickWin = null;
     #lineCb = null;
     #pendingLines = []; // submits before onLine is wired (startup) — never dropped
     #sigintCb = null;
@@ -525,7 +535,6 @@ export class Editor {
             // the human has no way to ask for it back.
             process.stdin.setRawMode(true);
             process.stdout.write(MOUSE_OFF);
-            this.#mouseOn = false;
             process.stdout.write("\x1b[?2004h");
             process.stdin.on("data", this.#onData);
         }
@@ -1038,7 +1047,6 @@ export class Editor {
         // chance this process gets, and emitting six harmless bytes twice
         // is not a cost worth reasoning about.
         process.stdout.write(MOUSE_OFF);
-        this.#mouseOn = false;
         process.stdout.write("\x1b[?2004l"); // bracketed paste OFF
         // REL-0161: the hardware cursor was hidden for the session's whole
         // life (the compositor's entry reset); this is the one place kiso
@@ -1050,30 +1058,20 @@ export class Editor {
         this.#closedResolve();
     }
     /**
-     * TUI2-R3v2 ② — mouse reporting follows the SELECTION SURFACES and
-     * nothing else.
+     * TUI2-R3v2 ②, corrected by the owner's ruling of 2026-09-21 (finding
+     * DC-56) — the bands still tell their host when a selection surface
+     * opens or closes, and the host's answer is now "leave the terminal
+     * alone".
      *
-     * While it is on, the terminal's own text selection changes behaviour
-     * (shift+drag still selects on every terminal that matters, but plain
-     * drag-to-copy does not), so leaving it on for the whole session would
-     * tax every copy-paste in the product to pay for a gesture that only
-     * means something while a list is up. It goes on when one opens and
-     * off when it closes — and both calls are idempotent, because the
-     * surfaces nest (a panel can open over a picker) and the bytes must
-     * not depend on the order they unwind in.
+     * The bands' contract is unchanged — a band reports that it has taken
+     * the composer — but the mouse is not a thing kiso hands out: reporting
+     * on means the terminal stops scrolling its own scrollback, and every
+     * band kiso has (the approval/ask/pick panel, the session picker, the @
+     * picker) is a keyboard surface — digits, arrows, enter, esc, a filter.
+     * The call sites stay so a future surface that DOES want the mouse has
+     * one place to ask, and the ruling is written at that place.
      */
-    #setMouse(on) {
-        if (this.#mouseOn === on)
-            return;
-        this.#mouseOn = on;
-        if (this.#entered)
-            process.stdout.write(on ? MOUSE_ON : MOUSE_OFF);
-    }
-    /** The surfaces that own a selection — the approval/ask/pick panel, the
-     *  session picker and the @ picker. Any one of them up = reporting on. */
-    #syncMouse() {
-        this.#setMouse(this.#panelInput.up() || this.#pickInput.up() || this.#atUp());
-    }
+    #syncMouse() { }
     /** OR-11 (a) — the lead the composer's row is DRAWN with, which is not
      *  always the brick. The CLI binds the compositor's input lead as `""`
      *  (a prompt character is a third thing saying "input lives here", and
@@ -1751,6 +1749,13 @@ export class Editor {
      * and context-menu everywhere else and would mean "approve" here.
      * The stakes are a side effect the human did not ask for, and an
      * ambiguous mouse event is not consent.
+     *
+     * 2026-09-21 (finding DC-56): kiso no longer ENABLES reporting, so on a
+     * terminal this path is reached only when something else left the mouse
+     * on — and then a click still answers the panel. The wheel stays
+     * dropped either way (a scroll past a panel is not a choice), which is
+     * the other half of why the settled rule is "no surface takes the
+     * mouse": a terminal that is reporting cannot scroll its own history.
      */
     #mouseEvent(params, press) {
         if (!press)
@@ -1771,6 +1776,11 @@ export class Editor {
     bindPanelRows(fn) {
         this.#panelRows = fn;
     }
+    /** B: the pick window the last frame drew — the Dock's value, not the
+     *  editor's guess. Same pull-on-demand shape as `bindPanelRows`. */
+    bindPickWindow(fn) {
+        this.#pickWin = fn;
+    }
     /** TMUX-F1 ②: `run` identical arrow sequences arrived in ONE read. Every
      *  surface but one gets every press — a wheel over the transcript viewer,
      *  a panel, a multi-row draft scrolls, as a wheel should. The HISTORY
@@ -2000,6 +2010,7 @@ export class Editor {
             reflow: () => this.#reflow(),
             render: () => this.#onRender(),
             syncMouse: () => this.#syncMouse(),
+            pickWindow: () => this.#pickWin?.() ?? null,
             closeBands: () => {
                 this.#menuOpen = false;
                 this.#menuSel = 0;

package/dist/panel-input.d.ts

@@ -32,6 +32,14 @@ export interface BandHost {
     line(): string;
     /** The buffer with its paste capsules expanded — the text that would leave the editor. */
     expandPastes(line: string): string;
+    /** The pick panel's window as the RENDERER drew it (review of this round):
+     *  the size depends on the frame's budget, so the digit keys must be handed
+     *  the value instead of deriving it again from PICK_MAX. Absent → the input
+     *  layer falls back to its own derivation. */
+    pickWindow?(): {
+        first: number;
+        size: number;
+    } | null;
     /** Empty the buffer: chars, cursor and the ↑↓ goal column. */
     clear(): void;
     /** Type one code point at the cursor. */

package/dist/panel-input.js

@@ -16,7 +16,7 @@
  * and the mouse hit. The bodies are the same; only the buffer access
  * goes through the host.
  */
-import { PICK_MAX, panelOptions, saferDegradedNote, startLevel, stepLevel, } from "./approval-panel.js";
+import { PICK_MAX, panelOptions, pickWindow, saferDegradedNote, startLevel, stepLevel, } from "./approval-panel.js";
 import { askCommitCustom, askKey, askOnCustomRow, askStart } from "./ask-panel.js";
 export class PanelInput {
     host;
@@ -206,7 +206,11 @@ export class PanelInput {
         if (panel === null)
             return false;
         if (panel.pick !== null && panel.pick.phase === "options") {
-            const n = Math.min(panel.view.pick.options.length, PICK_MAX);
+            // DC-58: EVERY option is reachable by ↑↓ — the window follows the
+            // cursor (`pickWindow`) and PICK_MAX is only how many rows one
+            // screen holds. This bound used to be `min(count, PICK_MAX)`, which
+            // is what left a 60-profile config with nine reachable rows.
+            const n = panel.view.pick.options.length;
             const cur = panel.pick.cursor;
             const next = dir === "up" ? Math.max(0, cur - 1) : Math.min(Math.max(0, n - 1), cur + 1);
             // OR-7: the second axis belongs to the highlighted option, so it
@@ -448,9 +452,18 @@ export class PanelInput {
         const panel = this.#panel;
         if (panel === null || panel.pick === null)
             return;
-        if (index < 0 || index >= Math.min(panel.view.pick.options.length, PICK_MAX))
+        const count = panel.view.pick.options.length;
+        // DC-58: a digit names the row ON SCREEN. The window's origin is what
+        // turns a visible position into an option index — the same
+        // `pickWindow` the renderer drew the rows with, so what the row says
+        // and what the key does cannot drift apart.
+        const { first, size } = this.host.pickWindow?.() ?? pickWindow(panel.pick.cursor, count, PICK_MAX);
+        if (index < 0 || index >= size)
             return;
-        panel.pick = { cursor: index, phase: "options", level: startLevel(panel.view.pick.options[index]) };
+        const target = first + index;
+        if (target >= count)
+            return;
+        panel.pick = { cursor: target, phase: "options", level: startLevel(panel.view.pick.options[target]) };
         this.host.render();
     }
     #pickPanelEnter() {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vincemakes/kiso-tui",
-	"version": "0.40.0",
+	"version": "0.40.2",
 	"description": "kiso tui \u2014 the pure terminal layer (cell renderer, dock, raw editor, diff, palette). Zero runtime dependencies: input is data, output is bytes.",
 	"type": "module",
 	"license": "MIT",
@@ -35,6 +35,6 @@
 	},
 	"homepage": "https://github.com/vincemakes/kiso/tree/main/packages/tui#readme",
 	"dependencies": {
-		"@vincemakes/kiso-tui-cells": "0.40.0"
+		"@vincemakes/kiso-tui-cells": "0.40.2"
 	}
 }