@oh-my-pi/pi-tui 8.12.10 → 8.13.0

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-tui",
-	"version": "8.12.10",
+	"version": "8.13.0",
 	"description": "Terminal User Interface library with differential rendering for efficient text-based applications",
 	"type": "module",
 	"main": "./src/index.ts",
@@ -47,7 +47,7 @@
 		"bun": ">=1.3.7"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-natives": "8.12.10",
+		"@oh-my-pi/pi-natives": "8.13.0",
 		"@types/mime-types": "^3.0.1",
 		"chalk": "^5.6.2",
 		"marked": "^17.0.1",

package/src/terminal-image.ts

@@ -1,9 +1,10 @@
 export type ImageProtocol = "kitty" | "iterm2" | null;
 
 export interface TerminalCapabilities {
-	images: ImageProtocol;
-	trueColor: boolean;
-	hyperlinks: boolean;
+	readonly images: ImageProtocol;
+	readonly trueColor: boolean;
+	readonly hyperlinks: boolean;
+	containsImage(line: string): boolean;
 }
 
 export interface CellDimensions {
@@ -35,37 +36,72 @@ export function setCellDimensions(dims: CellDimensions): void {
 	cellDimensions = dims;
 }
 
+const kBaseCaps: TerminalCapabilities = {
+	images: null,
+	trueColor: false,
+	hyperlinks: true,
+	containsImage() {
+		return false;
+	},
+};
+
+function createTerminalCaps(parts: Partial<TerminalCapabilities>): TerminalCapabilities {
+	return Object.freeze({ ...kBaseCaps, ...parts });
+}
+
+const kKittyCaps = createTerminalCaps({
+	images: "kitty",
+	trueColor: true,
+	hyperlinks: true,
+	containsImage(line: string) {
+		return line.includes("\x1b_G");
+	},
+});
+const kGhosttyCaps = kKittyCaps;
+const kWeztermCaps = kKittyCaps;
+
+const kIterm2Caps = createTerminalCaps({
+	images: "iterm2",
+	trueColor: true,
+	hyperlinks: true,
+	containsImage(line: string) {
+		return line.includes("\x1b]1337;File=");
+	},
+});
+
+const kTrueColorCaps = createTerminalCaps({
+	...kBaseCaps,
+	trueColor: true,
+});
+
+const kVscodeCaps = kTrueColorCaps;
+const kAlacrittyCaps = kTrueColorCaps;
+
 export function detectCapabilities(): TerminalCapabilities {
 	const termProgram = process.env.TERM_PROGRAM?.toLowerCase() || "";
 	const term = process.env.TERM?.toLowerCase() || "";
 	const colorTerm = process.env.COLORTERM?.toLowerCase() || "";
 
 	if (process.env.KITTY_WINDOW_ID || termProgram === "kitty") {
-		return { images: "kitty", trueColor: true, hyperlinks: true };
+		return kKittyCaps;
 	}
-
 	if (termProgram === "ghostty" || term.includes("ghostty") || process.env.GHOSTTY_RESOURCES_DIR) {
-		return { images: "kitty", trueColor: true, hyperlinks: true };
+		return kGhosttyCaps;
 	}
-
 	if (process.env.WEZTERM_PANE || termProgram === "wezterm") {
-		return { images: "kitty", trueColor: true, hyperlinks: true };
+		return kWeztermCaps;
 	}
-
 	if (process.env.ITERM_SESSION_ID || termProgram === "iterm.app") {
-		return { images: "iterm2", trueColor: true, hyperlinks: true };
+		return kIterm2Caps;
 	}
-
 	if (termProgram === "vscode") {
-		return { images: null, trueColor: true, hyperlinks: true };
+		return kVscodeCaps;
 	}
-
 	if (termProgram === "alacritty") {
-		return { images: null, trueColor: true, hyperlinks: true };
+		return kAlacrittyCaps;
 	}
-
 	const trueColor = colorTerm === "truecolor" || colorTerm === "24bit";
-	return { images: null, trueColor, hyperlinks: true };
+	return trueColor ? kTrueColorCaps : kBaseCaps;
 }
 
 export function getCapabilities(): TerminalCapabilities {

package/src/tui.ts

@@ -541,8 +541,8 @@ export class TUI extends Container {
 		return result;
 	}
 
-	private containsImage(line: string): boolean {
-		return line.includes("\x1b_G") || line.includes("\x1b]1337;File=");
+	static containsImage(line: string): boolean {
+		return getCapabilities().containsImage(line);
 	}
 
 	/**
@@ -764,7 +764,13 @@ export class TUI extends Container {
 
 	private applyLineResets(lines: string[]): string[] {
 		const reset = TUI.SEGMENT_RESET;
-		return lines.map(line => (this.containsImage(line) ? line : line + reset));
+		for (let i = 0; i < lines.length; i++) {
+			const line = lines[i];
+			if (!TUI.containsImage(line)) {
+				lines[i] = line + reset;
+			}
+		}
+		return lines;
 	}
 
 	/**
@@ -839,7 +845,7 @@ export class TUI extends Container {
 		overlayWidth: number,
 		totalWidth: number,
 	): string {
-		if (this.containsImage(baseLine)) return baseLine;
+		if (TUI.containsImage(baseLine)) return baseLine;
 
 		// Single pass through baseLine extracts both before and after segments
 		const afterStart = startCol + overlayWidth;
@@ -1081,7 +1087,7 @@ export class TUI extends Container {
 			if (i > firstChanged) buffer += "\r\n";
 			buffer += "\x1b[2K"; // Clear current line
 			const line = newLines[i];
-			const isImageLine = this.containsImage(line);
+			const isImageLine = TUI.containsImage(line);
 			if (!isImageLine && visibleWidth(line) > width) {
 				// Log all lines to crash file for debugging
 				const crashLogPath = path.join(os.homedir(), ".omp", "agent", "omp-crash.log");