@oh-my-pi/pi-tui 1.338.0 → 2.0.1337

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-tui",
-	"version": "1.338.0",
+	"version": "2.0.1337",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"type": "module",
 	"main": "src/index.ts",

package/src/autocomplete.ts

@@ -1,6 +1,6 @@
-import { readdirSync, statSync } from "fs";
-import { homedir } from "os";
-import { basename, dirname, join } from "path";
+import { readdirSync, statSync } from "node:fs";
+import { homedir } from "node:os";
+import { basename, dirname, join } from "node:path";
 
 // Use fd to walk directory tree (fast, respects .gitignore)
 function walkDirectoryWithFd(
@@ -387,7 +387,8 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 						const fullPath = join(searchDir, entry.name);
 						isDirectory = statSync(fullPath).isDirectory();
 					} catch {
-						// Broken symlink or permission error - treat as file
+						// Broken symlink, file deleted between readdir and stat, or permission error
+						continue;
 					}
 				}
 
@@ -459,7 +460,7 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 			});
 
 			return suggestions;
-		} catch (_e) {
+		} catch {
 			// Directory doesn't exist or not accessible
 			return [];
 		}
@@ -527,6 +528,7 @@ export class CombinedAutocompleteProvider implements AutocompleteProvider {
 
 			return suggestions;
 		} catch {
+			// Directory doesn't exist or not accessible
 			return [];
 		}
 	}

package/src/components/box.ts

@@ -1,5 +1,5 @@
-import type { Component } from "../tui.js";
-import { applyBackgroundToLine, visibleWidth } from "../utils.js";
+import type { Component } from "../tui";
+import { applyBackgroundToLine, visibleWidth } from "../utils";
 
 /**
  * Box component - a container that applies padding and background to all children

package/src/components/cancellable-loader.ts

@@ -1,5 +1,5 @@
-import { isEscape } from "../keys.js";
-import { Loader } from "./loader.js";
+import { isEscape } from "../keys";
+import { Loader } from "./loader";
 
 /**
  * Loader that can be cancelled with Escape.

package/src/components/editor.ts

@@ -1,4 +1,4 @@
-import type { AutocompleteProvider, CombinedAutocompleteProvider } from "../autocomplete.js";
+import type { AutocompleteProvider, CombinedAutocompleteProvider } from "../autocomplete";
 import {
 	isAltBackspace,
 	isAltEnter,
@@ -24,10 +24,10 @@ import {
 	isHome,
 	isShiftEnter,
 	isTab,
-} from "../keys.js";
-import type { Component } from "../tui.js";
-import { getSegmenter, isPunctuationChar, isWhitespaceChar, visibleWidth } from "../utils.js";
-import { SelectList, type SelectListTheme } from "./select-list.js";
+} from "../keys";
+import type { Component } from "../tui";
+import { getSegmenter, isPunctuationChar, isWhitespaceChar, visibleWidth } from "../utils";
+import { SelectList, type SelectListTheme } from "./select-list";
 
 const segmenter = getSegmenter();
 

package/src/components/image.ts

@@ -4,8 +4,8 @@ import {
 	type ImageDimensions,
 	imageFallback,
 	renderImage,
-} from "../terminal-image.js";
-import type { Component } from "../tui.js";
+} from "../terminal-image";
+import type { Component } from "../tui";
 
 export interface ImageTheme {
 	fallbackColor: (str: string) => string;

package/src/components/input.ts

@@ -14,9 +14,9 @@ import {
 	isCtrlW,
 	isDelete,
 	isEnter,
-} from "../keys.js";
-import type { Component } from "../tui.js";
-import { getSegmenter, isPunctuationChar, isWhitespaceChar, visibleWidth } from "../utils.js";
+} from "../keys";
+import type { Component } from "../tui";
+import { getSegmenter, isPunctuationChar, isWhitespaceChar, visibleWidth } from "../utils";
 
 const segmenter = getSegmenter();
 

package/src/components/loader.ts

@@ -1,5 +1,5 @@
-import type { TUI } from "../tui.js";
-import { Text } from "./text.js";
+import type { TUI } from "../tui";
+import { Text } from "./text";
 
 /**
  * Loader component that updates every 80ms with spinning animation

package/src/components/markdown.ts

@@ -1,6 +1,6 @@
 import { marked, type Token } from "marked";
-import type { Component } from "../tui.js";
-import { applyBackgroundToLine, visibleWidth, wrapTextWithAnsi } from "../utils.js";
+import type { Component } from "../tui";
+import { applyBackgroundToLine, visibleWidth, wrapTextWithAnsi } from "../utils";
 
 /**
  * Default text styling for markdown content.

package/src/components/select-list.ts

@@ -1,6 +1,6 @@
-import { isArrowDown, isArrowUp, isCtrlC, isEnter, isEscape } from "../keys.js";
-import type { Component } from "../tui.js";
-import { truncateToWidth } from "../utils.js";
+import { isArrowDown, isArrowUp, isCtrlC, isEnter, isEscape } from "../keys";
+import type { Component } from "../tui";
+import { truncateToWidth } from "../utils";
 
 export interface SelectItem {
 	value: string;

package/src/components/settings-list.ts

@@ -1,6 +1,6 @@
-import { isArrowDown, isArrowUp, isCtrlC, isEnter, isEscape } from "../keys.js";
-import type { Component } from "../tui.js";
-import { truncateToWidth, visibleWidth } from "../utils.js";
+import { isArrowDown, isArrowUp, isCtrlC, isEnter, isEscape } from "../keys";
+import type { Component } from "../tui";
+import { truncateToWidth, visibleWidth } from "../utils";
 
 export interface SettingItem {
 	/** Unique identifier for this setting */

package/src/components/spacer.ts

@@ -1,4 +1,4 @@
-import type { Component } from "../tui.js";
+import type { Component } from "../tui";
 
 /**
  * Spacer component that renders empty lines

package/src/components/tab-bar.ts

@@ -9,8 +9,8 @@
  * - Shift+Tab / Arrow Left: Previous tab (wraps around)
  */
 
-import { isArrowLeft, isArrowRight, isShiftTab, isTab } from "../keys.js";
-import type { Component } from "../tui.js";
+import { isArrowLeft, isArrowRight, isShiftTab, isTab } from "../keys";
+import type { Component } from "../tui";
 
 /** Tab definition */
 export interface Tab {

package/src/components/text.ts

@@ -1,5 +1,5 @@
-import type { Component } from "../tui.js";
-import { applyBackgroundToLine, visibleWidth, wrapTextWithAnsi } from "../utils.js";
+import type { Component } from "../tui";
+import { applyBackgroundToLine, visibleWidth, wrapTextWithAnsi } from "../utils";
 
 /**
  * Text component - displays multi-line text with word wrapping
@@ -22,6 +22,10 @@ export class Text implements Component {
 		this.customBgFn = customBgFn;
 	}
 
+	getText(): string {
+		return this.text;
+	}
+
 	setText(text: string): void {
 		this.text = text;
 		this.cachedText = undefined;

package/src/components/truncated-text.ts

@@ -1,5 +1,5 @@
-import type { Component } from "../tui.js";
-import { truncateToWidth, visibleWidth } from "../utils.js";
+import type { Component } from "../tui";
+import { truncateToWidth, visibleWidth } from "../utils";
 
 /**
  * Text component that truncates to fit viewport width

package/src/index.ts

@@ -6,21 +6,21 @@ export {
 	type AutocompleteProvider,
 	CombinedAutocompleteProvider,
 	type SlashCommand,
-} from "./autocomplete.js";
+} from "./autocomplete";
 // Components
-export { Box } from "./components/box.js";
-export { CancellableLoader } from "./components/cancellable-loader.js";
-export { Editor, type EditorTheme } from "./components/editor.js";
-export { Image, type ImageOptions, type ImageTheme } from "./components/image.js";
-export { Input } from "./components/input.js";
-export { Loader } from "./components/loader.js";
-export { type DefaultTextStyle, Markdown, type MarkdownTheme } from "./components/markdown.js";
-export { type SelectItem, SelectList, type SelectListTheme } from "./components/select-list.js";
-export { type SettingItem, SettingsList, type SettingsListTheme } from "./components/settings-list.js";
-export { Spacer } from "./components/spacer.js";
-export { type Tab, TabBar, type TabBarTheme } from "./components/tab-bar.js";
-export { Text } from "./components/text.js";
-export { TruncatedText } from "./components/truncated-text.js";
+export { Box } from "./components/box";
+export { CancellableLoader } from "./components/cancellable-loader";
+export { Editor, type EditorTheme } from "./components/editor";
+export { Image, type ImageOptions, type ImageTheme } from "./components/image";
+export { Input } from "./components/input";
+export { Loader } from "./components/loader";
+export { type DefaultTextStyle, Markdown, type MarkdownTheme } from "./components/markdown";
+export { type SelectItem, SelectList, type SelectListTheme } from "./components/select-list";
+export { type SettingItem, SettingsList, type SettingsListTheme } from "./components/settings-list";
+export { Spacer } from "./components/spacer";
+export { type Tab, TabBar, type TabBarTheme } from "./components/tab-bar";
+export { Text } from "./components/text";
+export { TruncatedText } from "./components/truncated-text";
 // Kitty keyboard protocol helpers
 export {
 	isAltBackspace,
@@ -60,9 +60,9 @@ export {
 	isShiftTab,
 	isTab,
 	Keys,
-} from "./keys.js";
+} from "./keys";
 // Terminal interface and implementations
-export { emergencyTerminalRestore, ProcessTerminal, type Terminal } from "./terminal.js";
+export { emergencyTerminalRestore, ProcessTerminal, type Terminal } from "./terminal";
 // Terminal image support
 export {
 	type CellDimensions,
@@ -85,7 +85,7 @@ export {
 	resetCapabilitiesCache,
 	setCellDimensions,
 	type TerminalCapabilities,
-} from "./terminal-image.js";
-export { type Component, Container, TUI } from "./tui.js";
+} from "./terminal-image";
+export { type Component, Container, TUI } from "./tui";
 // Utilities
-export { truncateToWidth, visibleWidth, wrapTextWithAnsi } from "./utils.js";
+export { truncateToWidth, visibleWidth, wrapTextWithAnsi } from "./utils";

package/src/tui.ts

@@ -5,10 +5,10 @@
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
-import { isShiftCtrlD } from "./keys.js";
-import type { Terminal } from "./terminal.js";
-import { getCapabilities, setCellDimensions } from "./terminal-image.js";
-import { visibleWidth } from "./utils.js";
+import { isShiftCtrlD } from "./keys";
+import type { Terminal } from "./terminal";
+import { getCapabilities, setCellDimensions } from "./terminal-image";
+import { visibleWidth } from "./utils";
 
 /**
  * Component interface - all components must implement this
@@ -86,6 +86,7 @@ export class TUI extends Container {
 	private cursorRow = 0; // Track where cursor is (0-indexed, relative to our first line)
 	private inputBuffer = ""; // Buffer for parsing terminal responses
 	private cellSizeQueryPending = false;
+	private inputQueue: string[] = []; // Queue input during cell size query to avoid interleaving
 
 	constructor(terminal: Terminal) {
 		super();
@@ -142,9 +143,24 @@ export class TUI extends Container {
 			this.inputBuffer += data;
 			const filtered = this.parseCellSizeResponse();
 			if (filtered.length === 0) return;
-			data = filtered;
+			if (filtered.length > 0) {
+				this.inputQueue.push(filtered);
+			}
+			// Process queued input after cell size response completes
+			if (!this.cellSizeQueryPending && this.inputQueue.length > 0) {
+				const queued = this.inputQueue;
+				this.inputQueue = [];
+				for (const item of queued) {
+					this.processInput(item);
+				}
+			}
+			return;
 		}
 
+		this.processInput(data);
+	}
+
+	private processInput(data: string): void {
 		// Global debug key handler (Shift+Ctrl+D)
 		if (isShiftCtrlD(data) && this.onDebug) {
 			this.onDebug();
@@ -169,16 +185,17 @@ export class TUI extends Container {
 			const heightPx = parseInt(match[1], 10);
 			const widthPx = parseInt(match[2], 10);
 
+			// Remove the response from buffer first
+			this.inputBuffer = this.inputBuffer.replace(responsePattern, "");
+			this.cellSizeQueryPending = false;
+
 			if (heightPx > 0 && widthPx > 0) {
 				setCellDimensions({ widthPx, heightPx });
 				// Invalidate all components so images re-render with correct dimensions
+				// This is safe now because cellSizeQueryPending=false prevents race with render
 				this.invalidate();
 				this.requestRender();
 			}
-
-			// Remove the response from buffer
-			this.inputBuffer = this.inputBuffer.replace(responsePattern, "");
-			this.cellSizeQueryPending = false;
 		}
 
 		// Check if we have a partial cell size response starting (wait for more data)
@@ -206,8 +223,11 @@ export class TUI extends Container {
 	}
 
 	private doRender(): void {
+		// Capture terminal dimensions at start to ensure consistency throughout render
 		const width = this.terminal.columns;
 		const height = this.terminal.rows;
+		// Snapshot cursor position at start of render for consistent viewport calculations
+		const currentCursorRow = this.cursorRow;
 
 		// Render all components to get new lines
 		const newLines = this.render(width);
@@ -267,10 +287,10 @@ export class TUI extends Container {
 		}
 
 		// Check if firstChanged is outside the viewport
-		// cursorRow is the line where cursor is (0-indexed)
-		// Viewport shows lines from (cursorRow - height + 1) to cursorRow
+		// Use snapshotted cursor position for consistent viewport calculation
+		// Viewport shows lines from (currentCursorRow - height + 1) to currentCursorRow
 		// If firstChanged < viewportTop, we need full re-render
-		const viewportTop = this.cursorRow - height + 1;
+		const viewportTop = currentCursorRow - height + 1;
 		if (firstChanged < viewportTop) {
 			// First change is above viewport - need full re-render
 			let buffer = "\x1b[?2026h"; // Begin synchronized output
@@ -291,8 +311,8 @@ export class TUI extends Container {
 		// Build buffer with all updates wrapped in synchronized output
 		let buffer = "\x1b[?2026h"; // Begin synchronized output
 
-		// Move cursor to first changed line
-		const lineDiff = firstChanged - this.cursorRow;
+		// Move cursor to first changed line using snapshotted position
+		const lineDiff = firstChanged - currentCursorRow;
 		if (lineDiff > 0) {
 			buffer += `\x1b[${lineDiff}B`; // Move down
 		} else if (lineDiff < 0) {
@@ -323,7 +343,9 @@ export class TUI extends Container {
 				try {
 					fs.mkdirSync(path.dirname(crashLogPath), { recursive: true });
 					fs.writeFileSync(crashLogPath, crashData);
-				} catch {}
+				} catch {
+					// Ignore - crash log is best-effort
+				}
 				throw new Error(`Rendered line ${i} exceeds terminal width. Debug log written to ${crashLogPath}`);
 			}
 			buffer += line;