xiaotime 0.5.5 → 0.5.7

package/dist/__tests__/ui_state_listener_lifecycle.test.js

@@ -1,152 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-/**
- * Stdin keypress listener lifecycle tests.
- *
- * SPRINT-CLI-INPUT-AND-TOOL-HARDENING-1 (#7299) PR2.
- *
- * The bug being guarded: each `_makeRl` call attaches a fresh readline
- * `Interface` with `terminal: true`, which in turn registers a keypress
- * listener on `process.stdin`. Node's `rl.close()` is documented to
- * detach that listener, but the cleanup is unreliable when stdin's raw
- * mode was toggled mid-cycle (the approval-menu path does that). The
- * pre-fix accumulation: every `promptApprovalMenu` cycle left one extra
- * keypress listener attached, and every active listener echoes
- * keystrokes independently. Stephan observed 6-8x character
- * multiplication after a session with several approval-required tool
- * calls 2026-05-25.
- *
- * The fix in `_makeRl` strips leftover `keypress` listeners before
- * `readline.createInterface` attaches the new one — Node's
- * `emitKeypressEvents` re-bootstraps on the next createInterface, so
- * the keypress event infrastructure is preserved.
- *
- * These tests pin the listener-count contract: after any number of rl
- * recreations, the count of `keypress` listeners on `process.stdin`
- * stays at exactly 1 (the live rl's). Each test drives at least one
- * full approval-menu cycle via the no-TTY fall-through path that
- * `approval_menu.test.ts` already exercises (vitest's stdin is not a
- * TTY, so `setRawMode` throws and the menu returns `(no TTY)` —
- * `_makeRl` still fires in the `finally` block, which is the
- * load-bearing path for this guard).
- */
-const vitest_1 = require("vitest");
-const ui_state_js_1 = require("../ui_state.js");
-/** Listener types `_makeRl` strips, by stream. Each
- * readline.createInterface({ terminal: true }) registers one of each
- * on the indicated stream. `resize` lives on stdout (readline tracks
- * terminal width from the output stream); the other three on stdin.
- * Keep these in sync with `_makeRl`. */
-const STDIN_EVENTS = ["keypress", "end", "error"];
-const STDOUT_EVENTS = ["resize"];
-function _stripAll() {
-    for (const ev of STDIN_EVENTS)
-        process.stdin.removeAllListeners(ev);
-    for (const ev of STDOUT_EVENTS)
-        process.stdout.removeAllListeners(ev);
-}
-(0, vitest_1.beforeEach)(() => {
-    // Suppress menu output; we only care about listener counts.
-    vitest_1.vi.spyOn(process.stdout, "write").mockImplementation(() => true);
-});
-(0, vitest_1.afterEach)(() => {
-    vitest_1.vi.restoreAllMocks();
-    // Clean slate for the next test — otherwise an accidental leak
-    // here would surface as a failure in an unrelated downstream test.
-    _stripAll();
-});
-(0, vitest_1.describe)("stdin keypress listener lifecycle — ISC-7299-003", () => {
-    (0, vitest_1.test)("single SessionUI instantiation attaches exactly one keypress listener", () => {
-        process.stdin.removeAllListeners("keypress");
-        const ui = new ui_state_js_1.SessionUI();
-        // _makeRl ran once in the constructor → one new keypress listener.
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        ui.getRl().close();
-    });
-    (0, vitest_1.test)("count stays at 1 after one approval-menu cycle", async () => {
-        process.stdin.removeAllListeners("keypress");
-        const ui = new ui_state_js_1.SessionUI();
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        // promptApprovalMenu bails on no-TTY but the finally block still
-        // runs `_makeRl`, so this exercises the recreate path. Pre-fix,
-        // this would leave the previous rl's keypress listener attached
-        // and the count would jump to 2.
-        await ui.promptApprovalMenu();
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        ui.getRl().close();
-    });
-    (0, vitest_1.test)("count stays at 1 after five sequential approval-menu cycles", async () => {
-        // The user-observed bug: 6-8x character echo after several tool
-        // calls. This test replays the exact accumulation path and asserts
-        // it does not grow.
-        process.stdin.removeAllListeners("keypress");
-        const ui = new ui_state_js_1.SessionUI();
-        for (let i = 0; i < 5; i++) {
-            await ui.promptApprovalMenu();
-            // After EACH cycle the count must still be 1 — not 2, not 6.
-            (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        }
-        ui.getRl().close();
-    });
-    (0, vitest_1.test)("end / error / resize listeners also stay bounded across cycles", async () => {
-        // Adjacent leaks — same root cause as the keypress leak
-        // (rl.close() doesn't reliably detach on setRawMode-toggled
-        // stdin), different event types. `end` / `error` accumulate on
-        // stdin; `resize` accumulates on stdout (readline tracks terminal
-        // width from the output stream). Pre-fix these crossed Node's
-        // default MaxListeners (10) after ~11 recreations and logged
-        // MaxListenersExceededWarning to stderr. Post-fix the strip in
-        // `_makeRl` keeps each at ≤ a single-digit ceiling across an
-        // unbounded number of cycles.
-        _stripAll();
-        const ui = new ui_state_js_1.SessionUI();
-        for (let i = 0; i < 15; i++) {
-            await ui.promptApprovalMenu();
-        }
-        // Each event type ends up with the live rl's single listener.
-        // Cap the assertion at 2 to leave headroom for Node-version-
-        // specific listener-shape changes — the bug was unbounded growth,
-        // not a precise count. Pre-fix this would be ~16 per event type.
-        for (const ev of STDIN_EVENTS) {
-            (0, vitest_1.expect)(process.stdin.listenerCount(ev)).toBeLessThanOrEqual(2);
-        }
-        for (const ev of STDOUT_EVENTS) {
-            (0, vitest_1.expect)(process.stdout.listenerCount(ev)).toBeLessThanOrEqual(2);
-        }
-        ui.getRl().close();
-    });
-    (0, vitest_1.test)("count returns to 0 after the live rl closes", () => {
-        process.stdin.removeAllListeners("keypress");
-        const ui = new ui_state_js_1.SessionUI();
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        ui.getRl().close();
-        // Node's rl.close() detaches its own keypress listener cleanly in
-        // the normal (no-raw-mode-toggle) shutdown path. This is the
-        // baseline that the bug breaks — keypress count should land at 0
-        // after close in normal teardown.
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(0);
-    });
-});
-(0, vitest_1.describe)("character-multiplication regression — ISC-7299-004", () => {
-    // Direct integration of the multiplication scenario: drive 5
-    // approval cycles, then assert that a single emitted keypress would
-    // be handled by exactly one listener. We can't observe stdout-echo
-    // counts directly under vitest (no real TTY), but the listener
-    // count IS the proximate cause — N listeners produces N+1 echo.
-    // This test wraps the listener-count guarantee in the user-visible
-    // failure-mode framing so a future regression is read as the
-    // multiplication bug it would surface as in production.
-    (0, vitest_1.test)("after 5 simulated failed tool-approval cycles, exactly one keypress handler is active", async () => {
-        process.stdin.removeAllListeners("keypress");
-        const ui = new ui_state_js_1.SessionUI();
-        for (let i = 0; i < 5; i++) {
-            await ui.promptApprovalMenu();
-        }
-        // Pre-fix observed at this point: 6 listeners → user types 'a',
-        // terminal renders 'aaaaaa'. Post-fix: 1 listener → 'a' renders
-        // exactly once.
-        (0, vitest_1.expect)(process.stdin.listenerCount("keypress")).toBe(1);
-        ui.getRl().close();
-    });
-});
-//# sourceMappingURL=ui_state_listener_lifecycle.test.js.map

package/dist/__tests__/ui_state_listener_lifecycle.test.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"ui_state_listener_lifecycle.test.js","sourceRoot":"","sources":["../../src/__tests__/ui_state_listener_lifecycle.test.ts"],"names":[],"mappings":";;AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA6BG;AACH,mCAA2E;AAC3E,gDAA2C;AAE3C;;;;wCAIwC;AACxC,MAAM,YAAY,GAAG,CAAC,UAAU,EAAE,KAAK,EAAE,OAAO,CAAU,CAAC;AAC3D,MAAM,aAAa,GAAG,CAAC,QAAQ,CAAU,CAAC;AAE1C,SAAS,SAAS;IAChB,KAAK,MAAM,EAAE,IAAI,YAAY;QAAE,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,EAAE,CAAC,CAAC;IACpE,KAAK,MAAM,EAAE,IAAI,aAAa;QAAE,OAAO,CAAC,MAAM,CAAC,kBAAkB,CAAC,EAAE,CAAC,CAAC;AACxE,CAAC;AAED,IAAA,mBAAU,EAAC,GAAG,EAAE;IACd,4DAA4D;IAC5D,WAAE,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,CAAC;AACnE,CAAC,CAAC,CAAC;AAEH,IAAA,kBAAS,EAAC,GAAG,EAAE;IACb,WAAE,CAAC,eAAe,EAAE,CAAC;IACrB,+DAA+D;IAC/D,mEAAmE;IACnE,SAAS,EAAE,CAAC;AACd,CAAC,CAAC,CAAC;AAEH,IAAA,iBAAQ,EAAC,kDAAkD,EAAE,GAAG,EAAE;IAChE,IAAA,aAAI,EAAC,uEAAuE,EAAE,GAAG,EAAE;QACjF,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,UAAU,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,mEAAmE;QACnE,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,IAAA,aAAI,EAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAChE,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,UAAU,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAExD,iEAAiE;QACjE,gEAAgE;QAChE,gEAAgE;QAChE,iCAAiC;QACjC,MAAM,EAAE,CAAC,kBAAkB,EAAE,CAAC;QAE9B,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,IAAA,aAAI,EAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC7E,gEAAgE;QAChE,mEAAmE;QACnE,oBAAoB;QACpB,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,UAAU,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC3B,MAAM,EAAE,CAAC,kBAAkB,EAAE,CAAC;YAC9B,6DAA6D;YAC7D,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC1D,CAAC;QACD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,IAAA,aAAI,EAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAChF,wDAAwD;QACxD,4DAA4D;QAC5D,+DAA+D;QAC/D,kEAAkE;QAClE,8DAA8D;QAC9D,6DAA6D;QAC7D,+DAA+D;QAC/D,6DAA6D;QAC7D,8BAA8B;QAC9B,SAAS,EAAE,CAAC;QACZ,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC;YAC5B,MAAM,EAAE,CAAC,kBAAkB,EAAE,CAAC;QAChC,CAAC;QACD,8DAA8D;QAC9D,6DAA6D;QAC7D,kEAAkE;QAClE,iEAAiE;QACjE,KAAK,MAAM,EAAE,IAAI,YAAY,EAAE,CAAC;YAC9B,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;QACjE,CAAC;QACD,KAAK,MAAM,EAAE,IAAI,aAAa,EAAE,CAAC;YAC/B,IAAA,eAAM,EAAC,OAAO,CAAC,MAAM,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,CAAC;QAClE,CAAC;QACD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;IAEH,IAAA,aAAI,EAAC,6CAA6C,EAAE,GAAG,EAAE;QACvD,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,UAAU,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;QACnB,kEAAkE;QAClE,6DAA6D;QAC7D,iEAAiE;QACjE,kCAAkC;QAClC,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1D,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,IAAA,iBAAQ,EAAC,oDAAoD,EAAE,GAAG,EAAE;IAClE,6DAA6D;IAC7D,oEAAoE;IACpE,mEAAmE;IACnE,+DAA+D;IAC/D,gEAAgE;IAChE,mEAAmE;IACnE,6DAA6D;IAC7D,wDAAwD;IAExD,IAAA,aAAI,EAAC,uFAAuF,EAAE,KAAK,IAAI,EAAE;QACvG,OAAO,CAAC,KAAK,CAAC,kBAAkB,CAAC,UAAU,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,uBAAS,EAAE,CAAC;QAC3B,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC3B,MAAM,EAAE,CAAC,kBAAkB,EAAE,CAAC;QAChC,CAAC;QACD,gEAAgE;QAChE,gEAAgE;QAChE,gBAAgB;QAChB,IAAA,eAAM,EAAC,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACxD,EAAE,CAAC,KAAK,EAAE,CAAC,KAAK,EAAE,CAAC;IACrB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}