@f5xc-salesdemos/pi-tui 18.40.2 → 18.41.1

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@f5xc-salesdemos/pi-tui",
-	"version": "18.40.2",
+	"version": "18.41.1",
 	"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": "18.40.2",
-		"@f5xc-salesdemos/pi-utils": "18.40.2",
+		"@f5xc-salesdemos/pi-natives": "18.41.1",
+		"@f5xc-salesdemos/pi-utils": "18.41.1",
 		"marked": "^17.0"
 	},
 	"devDependencies": {

package/src/tui.ts

@@ -3,7 +3,7 @@
  */
 import * as fs from "node:fs";
 import * as path from "node:path";
-import { $flag, getCrashLogPath, getDebugLogPath } from "@f5xc-salesdemos/pi-utils";
+import { $flag, getDebugLogPath } from "@f5xc-salesdemos/pi-utils";
 import { isKeyRelease, matchesKey } from "./keys";
 import type { Terminal } from "./terminal";
 import { ImageProtocol, setCellDimensions, setTerminalImageProtocol, TERMINAL } from "./terminal-capabilities";
@@ -996,6 +996,14 @@ export class TUI extends Container {
 		// Render all components to get new lines
 		let newLines = this.render(width);
 
+		// Clamp any oversized lines before dirty-checking so the truncated form
+		// matches what we store in #previousLines, preventing perpetual repaints.
+		for (let i = 0; i < newLines.length; i++) {
+			if (!TERMINAL.isImageLine(newLines[i]) && visibleWidth(newLines[i]) > width) {
+				newLines[i] = sliceByColumn(newLines[i], 0, width, true);
+			}
+		}
+
 		// Composite overlays into the rendered lines (before differential compare)
 		if (this.overlayStack.length > 0) {
 			newLines = this.#compositeOverlays(newLines, width, height);
@@ -1195,36 +1203,7 @@ export class TUI extends Container {
 			if (i > firstChanged) buffer += "\r\n";
 			buffer += "\x1b[2K"; // Clear current line
 			const line = newLines[i];
-			const isImage = TERMINAL.isImageLine(line);
-			if (!isImage && visibleWidth(line) > width) {
-				// Log all lines to crash file for debugging
-				const crashLogPath = getCrashLogPath();
-				const crashData = [
-					`Crash at ${new Date().toISOString()}`,
-					`Terminal width: ${width}`,
-					`Line ${i} visible width: ${visibleWidth(line)}`,
-					"",
-					"=== All rendered lines ===",
-					...newLines.map((l, idx) => `[${idx}] (w=${visibleWidth(l)}) ${l}`),
-					"",
-				].join("\n");
-				fs.mkdirSync(path.dirname(crashLogPath), { recursive: true });
-				fs.writeFileSync(crashLogPath, crashData);
-
-				// Clean up terminal state before throwing
-				this.stop();
-
-				const errorMsg = [
-					`Rendered line ${i} exceeds terminal width (${visibleWidth(line)} > ${width}).`,
-					"",
-					"This is likely caused by a custom TUI component not truncating its output.",
-					"Use visibleWidth() to measure and truncateToWidth() to truncate lines.",
-					"",
-					`Debug log written to: ${crashLogPath}`,
-				].join("\n");
-				throw new Error(errorMsg);
-			}
-			buffer += isImage ? line : line + SEGMENT_RESET;
+			buffer += TERMINAL.isImageLine(line) ? line : line + SEGMENT_RESET;
 		}
 
 		// Track where cursor ended up after rendering