pi-usereq 0.49.0 → 0.51.0

package/pi.dev-src/pi-mono/tui/test/bug-regression-isimageline-startswith-bug.test.ts

@@ -1,237 +0,0 @@
-/**
- * Bug regression test for isImageLine() crash scenario
- *
- * Bug: When isImageLine() used startsWith() and terminal doesn't support images,
- * it would return false for lines containing image escape sequences, causing TUI to
- * crash with "Rendered line exceeds terminal width" error.
- *
- * Fix: Changed to use includes() to detect escape sequences anywhere in the line.
- *
- * This test demonstrates:
- * 1. The bug scenario with the old implementation
- * 2. That the fix works correctly
- */
-
-import assert from "node:assert";
-import { describe, it } from "node:test";
-
-describe("Bug regression: isImageLine() crash with image escape sequences", () => {
-	describe("Bug scenario: Terminal without image support", () => {
-		it("old implementation would return false, causing crash", () => {
-			/**
-			 * OLD IMPLEMENTATION (buggy):
-			 * ```typescript
-			 * export function isImageLine(line: string): boolean {
-			 *   const prefix = getImageEscapePrefix();
-			 *   return prefix !== null && line.startsWith(prefix);
-			 * }
-			 * ```
-			 *
-			 * When terminal doesn't support images:
-			 * - getImageEscapePrefix() returns null
-			 * - isImageLine() returns false even for lines containing image sequences
-			 * - TUI performs width check on line containing 300KB+ of base64 data
-			 * - Crash: "Rendered line exceeds terminal width (304401 > 115)"
-			 */
-
-			// Simulate old implementation behavior
-			const oldIsImageLine = (line: string, imageEscapePrefix: string | null): boolean => {
-				return imageEscapePrefix !== null && line.startsWith(imageEscapePrefix);
-			};
-
-			// When terminal doesn't support images, prefix is null
-			const terminalWithoutImageSupport = null;
-
-			// Line containing image escape sequence with text before it (common bug scenario)
-			const lineWithImageSequence =
-				"Read image file [image/jpeg]\x1b]1337;File=size=800,600;inline=1:base64data...\x07";
-
-			// Old implementation would return false (BUG!)
-			const oldResult = oldIsImageLine(lineWithImageSequence, terminalWithoutImageSupport);
-			assert.strictEqual(
-				oldResult,
-				false,
-				"Bug: old implementation returns false for line containing image sequence when terminal has no image support",
-			);
-		});
-
-		it("new implementation returns true correctly", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			// Line containing image escape sequence with text before it
-			const lineWithImageSequence =
-				"Read image file [image/jpeg]\x1b]1337;File=size=800,600;inline=1:base64data...\x07";
-
-			// New implementation should return true (FIX!)
-			const newResult = isImageLine(lineWithImageSequence);
-			assert.strictEqual(newResult, true, "Fix: new implementation returns true for line containing image sequence");
-		});
-
-		it("new implementation detects Kitty sequences in any position", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			const scenarios = [
-				"At start: \x1b_Ga=T,f=100,data...\x1b\\",
-				"Prefix \x1b_Ga=T,data...\x1b\\",
-				"Suffix text \x1b_Ga=T,data...\x1b\\ suffix",
-				"Middle \x1b_Ga=T,data...\x1b\\ more text",
-				// Very long line (simulating 300KB+ crash scenario)
-				`Text before \x1b_Ga=T,f=100${"A".repeat(300000)} text after`,
-			];
-
-			for (const line of scenarios) {
-				assert.strictEqual(isImageLine(line), true, `Should detect Kitty sequence in: ${line.slice(0, 50)}...`);
-			}
-		});
-
-		it("new implementation detects iTerm2 sequences in any position", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			const scenarios = [
-				"At start: \x1b]1337;File=size=100,100:base64...\x07",
-				"Prefix \x1b]1337;File=inline=1:data==\x07",
-				"Suffix text \x1b]1337;File=inline=1:data==\x07 suffix",
-				"Middle \x1b]1337;File=inline=1:data==\x07 more text",
-				// Very long line (simulating 304KB crash scenario)
-				`Text before \x1b]1337;File=size=800,600;inline=1:${"B".repeat(300000)} text after`,
-			];
-
-			for (const line of scenarios) {
-				assert.strictEqual(isImageLine(line), true, `Should detect iTerm2 sequence in: ${line.slice(0, 50)}...`);
-			}
-		});
-	});
-
-	describe("Integration: Tool execution scenario", () => {
-		/**
-		 * This simulates what happens when the `read` tool reads an image file.
-		 * The tool result contains both text and image content:
-		 *
-		 * ```typescript
-		 * {
-		 *   content: [
-		 *     { type: "text", text: "Read image file [image/jpeg]\n800x600" },
-		 *     { type: "image", data: "base64...", mimeType: "image/jpeg" }
-		 *   ]
-		 * }
-		 * ```
-		 *
-		 * When this is rendered, the image component creates escape sequences.
-		 * If isImageLine() doesn't detect them, TUI crashes.
-		 */
-
-		it("detects image sequences in read tool output", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			// Simulate output when read tool processes an image
-			// The line might have text from the read result plus the image escape sequence
-			const toolOutputLine = "Read image file [image/jpeg]\x1b]1337;File=size=800,600;inline=1:base64image...\x07";
-
-			assert.strictEqual(isImageLine(toolOutputLine), true, "Should detect image sequence in tool output line");
-		});
-
-		it("detects Kitty sequences from Image component", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			// Kitty image component creates multi-line output with escape sequences
-			const kittyLine = "\x1b_Ga=T,f=100,t=f,d=base64data...\x1b\\\x1b_Gm=i=1;\x1b\\";
-
-			assert.strictEqual(isImageLine(kittyLine), true, "Should detect Kitty image component output");
-		});
-
-		it("handles ANSI codes before image sequences", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			// Line might have styling (error, warning, etc.) before image data
-			const lines = [
-				"\x1b[31mError\x1b[0m: \x1b]1337;File=inline=1:base64==\x07",
-				"\x1b[33mWarning\x1b[0m: \x1b_Ga=T,data...\x1b\\",
-				"\x1b[1mBold\x1b[0m \x1b]1337;File=:base64==\x07\x1b[0m",
-			];
-
-			for (const line of lines) {
-				assert.strictEqual(
-					isImageLine(line),
-					true,
-					`Should detect image sequence after ANSI codes: ${line.slice(0, 30)}...`,
-				);
-			}
-		});
-	});
-
-	describe("Crash scenario simulation", () => {
-		it("does NOT crash on very long lines with image sequences", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			/**
-			 * Simulate the exact crash scenario:
-			 * - Line is 304,401 characters (the crash log showed 58649 > 115)
-			 * - Contains image escape sequence somewhere in the middle
-			 * - Old implementation would return false, causing TUI to do width check
-			 * - New implementation returns true, skipping width check (preventing crash)
-			 */
-
-			const base64Char = "A".repeat(100);
-			const iterm2Sequence = "\x1b]1337;File=size=800,600;inline=1:";
-
-			// Build a line that would cause the crash
-			const crashLine =
-				"Output: " +
-				iterm2Sequence +
-				base64Char.repeat(3040) + // ~304,000 chars
-				" end of output";
-
-			// Verify line is very long
-			assert(crashLine.length > 300000, "Test line should be > 300KB");
-
-			// New implementation should detect it (prevents crash)
-			const detected = isImageLine(crashLine);
-			assert.strictEqual(detected, true, "Should detect image sequence in very long line, preventing TUI crash");
-		});
-
-		it("handles lines exactly matching crash log dimensions", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			/**
-			 * Crash log showed: line 58649 chars wide, terminal width 115
-			 * Let's create a line with similar characteristics
-			 */
-
-			const targetWidth = 58649;
-			const prefix = "Text";
-			const sequence = "\x1b_Ga=T,f=100";
-			const suffix = "End";
-			const padding = "A".repeat(targetWidth - prefix.length - sequence.length - suffix.length);
-			const line = `${prefix}${sequence}${padding}${suffix}`;
-
-			assert.strictEqual(line.length, 58649);
-			assert.strictEqual(isImageLine(line), true, "Should detect image sequence in 58649-char line");
-		});
-	});
-
-	describe("Negative cases: Don't false positive", () => {
-		it("does not detect images in regular long text", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			// Very long line WITHOUT image sequences
-			const longText = "A".repeat(100000);
-
-			assert.strictEqual(isImageLine(longText), false, "Should not detect images in plain long text");
-		});
-
-		it("does not detect images in lines with file paths", async () => {
-			const { isImageLine } = await import("../src/terminal-image.js");
-
-			const filePaths = [
-				"/path/to/1337/image.jpg",
-				"/usr/local/bin/File_converter",
-				"~/Documents/1337File_backup.png",
-				"./_G_test_file.txt",
-			];
-
-			for (const path of filePaths) {
-				assert.strictEqual(isImageLine(path), false, `Should not falsely detect image sequence in path: ${path}`);
-			}
-		});
-	});
-});

package/pi.dev-src/pi-mono/tui/test/chat-simple.ts

@@ -1,129 +0,0 @@
-/**
- * Simple chat interface demo using tui.ts
- */
-
-import chalk from "chalk";
-import { CombinedAutocompleteProvider } from "../src/autocomplete.js";
-import { Editor } from "../src/components/editor.js";
-import { Loader } from "../src/components/loader.js";
-import { Markdown } from "../src/components/markdown.js";
-import { Text } from "../src/components/text.js";
-import { ProcessTerminal } from "../src/terminal.js";
-import { TUI } from "../src/tui.js";
-import { defaultEditorTheme, defaultMarkdownTheme } from "./test-themes.js";
-
-// Create terminal
-const terminal = new ProcessTerminal();
-
-// Create TUI
-const tui = new TUI(terminal);
-
-// Create chat container with some initial messages
-tui.addChild(
-	new Text("Welcome to Simple Chat!\n\nType your messages below. Type '/' for commands. Press Ctrl+C to exit."),
-);
-
-// Create editor with autocomplete
-const editor = new Editor(tui, defaultEditorTheme);
-
-// Set up autocomplete provider with slash commands and file completion
-const autocompleteProvider = new CombinedAutocompleteProvider(
-	[
-		{ name: "delete", description: "Delete the last message" },
-		{ name: "clear", description: "Clear all messages" },
-	],
-	process.cwd(),
-);
-editor.setAutocompleteProvider(autocompleteProvider);
-
-tui.addChild(editor);
-
-// Focus the editor
-tui.setFocus(editor);
-
-// Track if we're waiting for bot response
-let isResponding = false;
-
-// Handle message submission
-editor.onSubmit = (value: string) => {
-	// Prevent submission if already responding
-	if (isResponding) {
-		return;
-	}
-
-	const trimmed = value.trim();
-
-	// Handle slash commands
-	if (trimmed === "/delete") {
-		const children = tui.children;
-		// Remove component before editor (if there are any besides the initial text)
-		if (children.length > 3) {
-			// children[0] = "Welcome to Simple Chat!"
-			// children[1] = "Type your messages below..."
-			// children[2...n-1] = messages
-			// children[n] = editor
-			children.splice(children.length - 2, 1);
-		}
-		tui.requestRender();
-		return;
-	}
-
-	if (trimmed === "/clear") {
-		const children = tui.children;
-		// Remove all messages but keep the welcome text and editor
-		children.splice(2, children.length - 3);
-		tui.requestRender();
-		return;
-	}
-
-	if (trimmed) {
-		isResponding = true;
-		editor.disableSubmit = true;
-
-		const userMessage = new Markdown(value, 1, 1, defaultMarkdownTheme);
-
-		const children = tui.children;
-		children.splice(children.length - 1, 0, userMessage);
-
-		const loader = new Loader(
-			tui,
-			(s) => chalk.cyan(s),
-			(s) => chalk.dim(s),
-			"Thinking...",
-		);
-		children.splice(children.length - 1, 0, loader);
-
-		tui.requestRender();
-
-		setTimeout(() => {
-			tui.removeChild(loader);
-
-			// Simulate a response
-			const responses = [
-				"That's interesting! Tell me more.",
-				"I see what you mean.",
-				"Fascinating perspective!",
-				"Could you elaborate on that?",
-				"That makes sense to me.",
-				"I hadn't thought of it that way.",
-				"Great point!",
-				"Thanks for sharing that.",
-			];
-			const randomResponse = responses[Math.floor(Math.random() * responses.length)];
-
-			// Add assistant message with no background (transparent)
-			const botMessage = new Markdown(randomResponse, 1, 1, defaultMarkdownTheme);
-			children.splice(children.length - 1, 0, botMessage);
-
-			// Re-enable submit
-			isResponding = false;
-			editor.disableSubmit = false;
-
-			// Request render
-			tui.requestRender();
-		}, 1000);
-	}
-};
-
-// Start the TUI
-tui.start();