@xynogen/pix-pretty 1.7.4 → 1.7.6

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-pretty",
-	"version": "1.7.4",
+	"version": "1.7.6",
 	"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", () => {
@@ -161,3 +178,56 @@ describe("showOverlay — sudo mode", () => {
 		expect(joined).toContain("●");
 	});
 });
+
+// ── Auto-deny timer (dead-man's switch) ───────────────────────────────────────
+//
+// Timer-aware mock: unlike makeUI, this keeps the promise pending and resolves
+// only when `done` fires — so a real setInterval expiry can drive the result.
+// `onReady` gets the live component to optionally feed input before expiry.
+function makeTimerUI(onReady?: (comp: Wired) => void): OverlayUI {
+	return {
+		custom: <T>(
+			cb: (
+				tui: { requestRender(): void },
+				th: typeof theme,
+				kb: unknown,
+				done: (v: T) => void,
+			) => Wired,
+		): Promise<T | undefined> =>
+			new Promise((resolve) => {
+				const comp = cb({ requestRender: () => {} }, theme, undefined, (v) =>
+					resolve(v),
+				);
+				comp.render(80);
+				onReady?.(comp);
+			}),
+	};
+}
+
+describe("showOverlay — auto-deny timer", () => {
+	test("expires to timeout when left untouched", async () => {
+		const result = await showOverlay(makeTimerUI(), {
+			mode: "confirm",
+			title: "T",
+			timeoutMs: 1000, // ceil → 1s, fires on first tick
+		});
+		expect(result.action).toBe("timeout");
+	});
+
+	test("first keypress cancels the timer (no auto-deny)", async () => {
+		let live: Wired | undefined;
+		const pending = showOverlay(
+			makeTimerUI((comp) => {
+				live = comp;
+				comp.handleInput(DOWN); // any key — cancels the dead-man's switch
+			}),
+			{ mode: "confirm", title: "T", timeoutMs: 1000 },
+		);
+		// Wait well past the 1s window. A live timer would have resolved "timeout";
+		// since the keypress cancelled it, the promise is still pending here.
+		await new Promise((r) => setTimeout(r, 1300));
+		live?.handleInput(ENTER); // now deny explicitly
+		const result = await pending;
+		expect(result.action).toBe("denied");
+	});
+});

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"; // ponytail: timeout kept for back-compat; never emitted now
+export type OverlayAction = "approved" | "denied" | "timeout";
 
 export interface OverlayResult {
 	action: OverlayAction;
@@ -40,7 +45,11 @@ interface BaseConfig {
 	title: string;
 	/** Optional body lines under the title. */
 	body?: string[];
-	/** @deprecated No-op — timeout removed. Dialog waits indefinitely for user input. */
+	/**
+	 * Auto-deny after this many ms of NO user input (dead-man's switch). The
+	 * first keypress cancels the timer and the dialog then waits indefinitely.
+	 * 0 or omitted = no timer (wait forever). Resolves with action "timeout".
+	 */
 	timeoutMs?: number;
 	/**
 	 * Choices shown in the SelectList.
@@ -139,12 +148,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
@@ -220,6 +232,18 @@ export function showOverlay(
 				let stage: Stage = "select";
 				let countdownLine: string | undefined;
 
+				// Dead-man's-switch timer: counts down only while untouched. The
+				// first keypress cancels it (user is present → let them decide). If
+				// it expires with no input, auto-deny so the agent isn't stuck.
+				const timeoutMs = config.timeoutMs ?? 0;
+				let remaining = Math.ceil(timeoutMs / 1000);
+				let timer: ReturnType<typeof setInterval> | undefined;
+				const cancelTimer = () => {
+					if (timer) clearInterval(timer);
+					timer = undefined;
+					countdownLine = undefined;
+				};
+
 				// ── components ──────────────────────────────────────────────────
 				const selectItems: SelectItem[] = choices.map((c) => ({
 					value: c.value,
@@ -235,7 +259,24 @@ export function showOverlay(
 				const maskedInput = new MaskedInput();
 
 				// ── finish ───────────────────────────────────────────────────────
-				const finish = (result: OverlayResult) => done(result);
+				const finish = (result: OverlayResult) => {
+					cancelTimer();
+					done(result);
+				};
+
+				// Arm the dead-man's switch (only when a timeout was requested).
+				if (timeoutMs > 0) {
+					countdownLine = theme.fg("dim", `auto-deny in ${remaining}s`);
+					timer = setInterval(() => {
+						remaining -= 1;
+						if (remaining <= 0) {
+							finish({ action: "timeout" });
+							return;
+						}
+						countdownLine = theme.fg("dim", `auto-deny in ${remaining}s`);
+						tui.requestRender();
+					}, 1000);
+				}
 
 				// ── event wiring ─────────────────────────────────────────────────
 				selectList.onSelect = (item) => {
@@ -277,6 +318,7 @@ export function showOverlay(
 					},
 					invalidate: () => {},
 					handleInput: (data) => {
+						cancelTimer(); // user is present — stop the auto-deny countdown
 						if (stage === "select") selectList.handleInput(data);
 						else maskedInput.handleInput(data);
 						tui.requestRender();