@xynogen/pix-pretty 1.7.2 → 1.7.4

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@xynogen/pix-pretty",
-	"version": "1.7.2",
+	"version": "1.7.4",
 	"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.ts

@@ -19,7 +19,7 @@ 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 +40,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 +96,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" },
@@ -214,17 +210,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 +234,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 +254,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 +285,7 @@ export function showOverlay(
 			},
 			{ overlay: true },
 		).then((result) => {
-			if (timer !== undefined) clearTimeout(timer);
-			resolve(result ?? { action: "timeout" });
+			resolve(result ?? { action: "denied" });
 		});
 	});
 }

package/src/progress.ts

@@ -13,7 +13,7 @@
  *   p.close();                             // releases input, removes overlay
  */
 
-import { Box, Text } from "@earendil-works/pi-tui";
+import { frameLines, modalWidth } from "./modal-frame.js";
 
 interface ProgressTheme {
 	fg(color: string, text: string): string;
@@ -66,31 +66,15 @@ export function openProgress(
 	ui.custom<void>(
 		(tui, theme, _kb, done) => {
 			let frame = 0;
-			const titleText = new Text(theme.fg(accent, theme.bold(title)), 1, 0);
-			const statusText = new Text("", 1, 0);
-
-			const render = () => {
-				statusText.setText(
-					`${theme.fg(accent, SPINNER[frame])} ${theme.fg("muted", labelValue)}`,
-				);
-			};
-
 			let labelValue = "Working…";
-			render();
 
 			const ticker = setInterval(() => {
 				frame = (frame + 1) % SPINNER.length;
-				render();
 				tui.requestRender();
 			}, SPINNER_INTERVAL_MS);
 
-			const container = new Box(2, 1, (s) => theme.bg("customMessageBg", s));
-			container.addChild(titleText);
-			container.addChild(statusText);
-
 			setLabelImpl = (label: string) => {
 				labelValue = label;
-				render();
 				tui.requestRender();
 			};
 			closeImpl = () => {
@@ -99,8 +83,19 @@ export function openProgress(
 			};
 
 			return {
-				render: (w: number) => container.render(w),
-				invalidate: () => container.invalidate(),
+				render: (w: number) => {
+					const mw = modalWidth(w);
+					return frameLines({
+						width: mw,
+						lines: [
+							theme.fg(accent, theme.bold(title)),
+							`${theme.fg(accent, SPINNER[frame])} ${theme.fg("muted", labelValue)}`,
+						],
+						color: (s) => theme.fg(accent, s),
+						bg: (s) => theme.bg("customMessageBg", s),
+					});
+				},
+				invalidate: () => {},
 				// Swallow every keystroke: a focused overlay owns input, so nothing
 				// reaches the editor while the update runs.
 				handleInput: () => {},