@f5xc-salesdemos/pi-tui 19.29.3 → 19.29.5

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/pi-tui",
-	"version": "19.29.3",
+	"version": "19.29.5",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"homepage": "https://github.com/f5xc-salesdemos/xcsh",
 	"author": "Can Boluk",
@@ -37,8 +37,8 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@f5xc-salesdemos/pi-natives": "19.29.3",
-		"@f5xc-salesdemos/pi-utils": "19.29.3",
+		"@f5xc-salesdemos/pi-natives": "19.29.5",
+		"@f5xc-salesdemos/pi-utils": "19.29.5",
 		"marked": "^17.0"
 	},
 	"devDependencies": {

package/src/stdin-buffer.ts

@@ -224,8 +224,15 @@ function extractCompleteSequences(buffer: string): { sequences: string[]; remain
 
 export type StdinBufferOptions = {
 	/**
-	 * Maximum time to wait for sequence completion (default: 10ms)
-	 * After this time, the buffer is flushed even if incomplete
+	 * Maximum time to wait for sequence completion (default: 50ms)
+	 * After this time, the buffer is flushed even if incomplete.
+	 *
+	 * This is effectively the "ESC hold" window: a lone ESC (the Escape key)
+	 * registers after this delay, and a multi-byte escape sequence fragmented
+	 * across stdin reads (common under heavy render load) has this long to
+	 * reassemble before its tail would leak as individual printable characters.
+	 * 50ms is the standard ESC timeout — long enough to survive multi-frame
+	 * stalls, short enough to feel instant.
 	 */
 	timeout?: number;
 };
@@ -248,7 +255,7 @@ export class StdinBuffer extends EventEmitter<StdinBufferEventMap> {
 
 	constructor(options: StdinBufferOptions = {}) {
 		super();
-		this.#timeoutMs = options.timeout ?? 10;
+		this.#timeoutMs = options.timeout ?? 50;
 	}
 
 	process(data: string | Buffer): void {

package/src/terminal.ts

@@ -274,7 +274,10 @@ export class ProcessTerminal implements Terminal {
 	 * to handle the case where the response arrives split across multiple events.
 	 */
 	#setupStdinBuffer(): void {
-		this.#stdinBuffer = new StdinBuffer({ timeout: 10 });
+		// Use the default ESC-hold window (50ms) so escape sequences fragmented
+		// across stdin reads under render load reassemble instead of leaking their
+		// tail as editor text (e.g. modifyOtherKeys Ctrl+C \x1b[27;5;99~).
+		this.#stdinBuffer = new StdinBuffer();
 
 		// Kitty protocol response pattern: \x1b[?<flags>u
 		const kittyResponsePattern = /^\x1b\[\?(\d+)u$/;
@@ -503,6 +506,10 @@ export class ProcessTerminal implements Terminal {
 	}
 
 	async drainInput(maxMs = 1000, idleMs = 50): Promise<void> {
+		// Stop the periodic OSC 11 poll first. drainInput precedes relinquishing
+		// the TTY (exit, or a cooked-mode subprocess); a poll query fired now would
+		// have its response echoed as gibberish once raw mode is gone.
+		this.#stopOsc11Poll();
 		if (this.#kittyProtocolActive) {
 			// Disable Kitty keyboard protocol first so any late key releases
 			// do not generate new Kitty escape sequences.

package/src/tui.ts

@@ -580,6 +580,18 @@ export class TUI extends Container {
 		this.terminal.stop();
 	}
 
+	/**
+	 * Suspend the UI to hand the TTY to a cooked-mode subprocess (e.g. an
+	 * interactive `aws sso login`). Drains pending terminal input while still in
+	 * raw mode, then stops — so a late capability-probe response (notably the
+	 * periodic OSC 11 poll) cannot be echoed as gibberish during the cooked-mode
+	 * window. Pair with start() once the subprocess exits.
+	 */
+	async suspendForSubprocess(maxDrainMs = 150): Promise<void> {
+		await this.terminal.drainInput(maxDrainMs);
+		this.stop();
+	}
+
 	requestRender(force = false): void {
 		if (force) {
 			this.#previousLines = [];