@xynogen/pix-pretty 1.7.3 → 1.7.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-pretty",
-	"version": "1.7.3",
+	"version": "1.7.5",
 	"description": "Enhanced tool output rendering with syntax highlighting, file icons, tree views, FFF search, and paste chip formatting",
 	"type": "module",
 	"main": "src/index.ts",

package/src/gate-overlay.test.ts

@@ -117,6 +117,23 @@ describe("showOverlay — confirm mode", () => {
 		expect(joined).toContain("MY TITLE");
 		expect(joined).toContain("body-line-x");
 	});
+
+	test("wraps a long body command instead of truncating it", async () => {
+		// A command far wider than any modal width — must survive in full, wrapped.
+		const longCmd = `echo ${"pix-gate-installed-or-linked ".repeat(8)}done`;
+		let captured: string[] = [];
+		await showOverlay(
+			makeUI((comp) => {
+				captured = comp.render(80);
+				comp.handleInput(ENTER);
+			}),
+			{ mode: "confirm", title: "T", body: [longCmd], timeoutMs: 0 },
+		);
+		// Every whitespace-delimited token of the command appears somewhere in the
+		// frame — nothing was dropped by truncation.
+		const joined = captured.join("\n");
+		for (const tok of longCmd.split(" ")) expect(joined).toContain(tok);
+	});
 });
 
 describe("showOverlay — sudo mode", () => {

package/src/gate-overlay.ts

@@ -14,12 +14,17 @@
  *   - Single source of truth for the overlay look across pix-gate and pix-sudo.
  */
 
-import { Input, type SelectItem, SelectList } from "@earendil-works/pi-tui";
+import {
+	Input,
+	type SelectItem,
+	SelectList,
+	wrapTextWithAnsi,
+} from "@earendil-works/pi-tui";
 import { frameLines, modalWidth, selectListTheme } from "./modal-frame.js";
 
 // ── Types ─────────────────────────────────────────────────────────────────────
 
-export type OverlayAction = "approved" | "denied" | "timeout";
+export type OverlayAction = "approved" | "denied" | "timeout"; // ponytail: timeout kept for back-compat; never emitted now
 
 export interface OverlayResult {
 	action: OverlayAction;
@@ -40,7 +45,7 @@ interface BaseConfig {
 	title: string;
 	/** Optional body lines under the title. */
 	body?: string[];
-	/** Auto-deny after this many ms. 0 = no timeout. Default 30_000. */
+	/** @deprecated No-op — timeout removed. Dialog waits indefinitely for user input. */
 	timeoutMs?: number;
 	/**
 	 * Choices shown in the SelectList.
@@ -96,10 +101,6 @@ export interface OverlayUI {
 
 // ── Constants ─────────────────────────────────────────────────────────────────
 
-const SECOND_MS = 1_000;
-const COUNTDOWN_WARN_S = 5;
-const DEFAULT_TIMEOUT_MS = 30_000;
-
 const DEFAULT_CHOICES: OverlayChoice[] = [
 	{ value: "yes", label: "Allow", description: "Proceed" },
 	{ value: "no", label: "Deny", description: "Block" },
@@ -143,12 +144,15 @@ function buildLines(opts: {
 	const inner = width - 4; // CHROME = 2 border + 2 padding
 	const lines: string[] = [];
 
-	// Title
-	lines.push(theme.fg(accent, theme.bold(config.title)));
+	// Title — wrap so a long reason/command isn't truncated by the frame.
+	for (const t of wrapTextWithAnsi(config.title, inner)) {
+		lines.push(theme.fg(accent, theme.bold(t)));
+	}
 
-	// Body
+	// Body — wrap each line so long commands wrap instead of getting cut off.
 	for (const line of config.body ?? []) {
-		lines.push(theme.fg("text", line));
+		const wrapped = line === "" ? [""] : wrapTextWithAnsi(line, inner);
+		for (const w of wrapped) lines.push(theme.fg("text", w));
 	}
 
 	// Divider after title/body
@@ -214,17 +218,10 @@ export function showOverlay(
 	config: OverlayConfig,
 ): Promise<OverlayResult> {
 	const accent = config.accent ?? "accent";
-	const timeoutMs = config.timeoutMs ?? DEFAULT_TIMEOUT_MS;
 	const choices = config.choices ?? DEFAULT_CHOICES;
 	const approveVal = config.approveValue ?? "yes";
 
 	return new Promise((resolve) => {
-		const controller = new AbortController();
-		const timer =
-			timeoutMs > 0
-				? setTimeout(() => controller.abort(), timeoutMs)
-				: undefined;
-
 		ui.custom<OverlayResult>(
 			(tui, theme, _kb, done) => {
 				type Stage = "select" | "password";
@@ -245,35 +242,8 @@ export function showOverlay(
 				);
 				const maskedInput = new MaskedInput();
 
-				// ── countdown ───────────────────────────────────────────────────
-				let ticker: ReturnType<typeof setInterval> | undefined;
-				if (timeoutMs > 0) {
-					const deadlineMs = Date.now() + timeoutMs;
-					const updateCountdown = () => {
-						const remaining = Math.max(
-							0,
-							Math.ceil((deadlineMs - Date.now()) / SECOND_MS),
-						);
-						countdownLine =
-							theme.fg("dim", "Auto-deny in ") +
-							theme.fg(
-								remaining <= COUNTDOWN_WARN_S ? accent : "muted",
-								`${remaining}s`,
-							);
-					};
-					updateCountdown();
-					ticker = setInterval(() => {
-						updateCountdown();
-						tui.requestRender();
-					}, SECOND_MS);
-				}
-
 				// ── finish ───────────────────────────────────────────────────────
-				const finish = (result: OverlayResult) => {
-					if (timer !== undefined) clearTimeout(timer);
-					if (ticker !== undefined) clearInterval(ticker);
-					done(result);
-				};
+				const finish = (result: OverlayResult) => done(result);
 
 				// ── event wiring ─────────────────────────────────────────────────
 				selectList.onSelect = (item) => {
@@ -292,10 +262,6 @@ export function showOverlay(
 					finish({ action: "approved", password: pw });
 				maskedInput.onEscape = () => finish({ action: "denied" });
 
-				controller.signal.addEventListener("abort", () =>
-					finish({ action: "timeout" }),
-				);
-
 				// ── component interface ──────────────────────────────────────────
 				return {
 					render: (w) => {
@@ -327,8 +293,7 @@ export function showOverlay(
 			},
 			{ overlay: true },
 		).then((result) => {
-			if (timer !== undefined) clearTimeout(timer);
-			resolve(result ?? { action: "timeout" });
+			resolve(result ?? { action: "denied" });
 		});
 	});
 }