@zeldrisho/pi-gate 0.7.2 → 0.7.4

package/CHANGELOG.md

@@ -7,6 +7,19 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.7.4] - 2026-11-01
+
+### Changed
+
+- Shorten approval timeout and cancellation notices.
+- Remove the package's Node.js engine restriction; the current Node.js LTS release is the tested runtime.
+
+## [0.7.3] - 2026-10-28
+
+### Changed
+
+- Show the approval timeout and configured confirmation keys, and allow horizontal scrolling of long command lines.
+
 ## [0.7.2] - 2026-10-25
 
 ### Changed
@@ -112,7 +125,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - support `prompt`, `block`, and `allow` actions, with longest-pattern-wins resolution
 - block rather than auto-approve in non-UI modes
 
-[Unreleased]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.2...HEAD
+[Unreleased]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.4...HEAD
+[0.7.4]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.3...pi-gate-v0.7.4
+[0.7.3]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.2...pi-gate-v0.7.3
 [0.7.2]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.1...pi-gate-v0.7.2
 [0.7.1]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.7.0...pi-gate-v0.7.1
 [0.7.0]: https://github.com/zeldrisho/pi-packages/compare/pi-gate-v0.6.1...pi-gate-v0.7.0

package/README.md

@@ -46,9 +46,7 @@ pi-gate is a guard rail, not a sandbox. Complex shell syntax, pipelines, redirec
 
 The built-in `bash` tool is gated, and `write`/`edit` calls targeting the gate config require approval. Prompt dialogs offer `Allow` and `Deny`; approvals are never remembered. To stop repeated prompts, add a specific `allow` rule to `gate.json`.
 
-Approval dialogs are serialized for concurrent tool calls; cancellation, shutdown, reload, session replacement, or a policy change while a dialog is open denies the pending call. Calls queued before shutdown or session replacement are also denied.
-
-The matched command text is highlighted in the TUI approval dialog. Long commands are truncated in approval dialogs, but an approved command executes in full. Denials, policy blocks, dismissals, and timeouts stop the turn; the agent is told to ask before retrying a denial or proceeding past a block, while a timeout asks whether to retry approval. Other allowed calls in a parallel batch may still run. In non-interactive modes (`-p`, JSON), commands requiring approval are denied.
+The approval dialog highlights the matched command text. Long commands are truncated in the dialog but execute in full if approved. Denials, blocks, dismissals, and timeouts stop the turn; other allowed calls in a parallel batch may still run. In non-interactive modes (`-p`, JSON), commands requiring approval are denied.
 
 In Herdr-managed panes, install Herdr's Pi integration with `herdr integration install pi` to report pending approvals as `blocked`. Herdr is optional.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zeldrisho/pi-gate",
-  "version": "0.7.2",
+  "version": "0.7.4",
   "description": "Pi extension that gates bash commands and protects its configuration from tool edits",
   "keywords": [
     "command-blocking",
@@ -36,16 +36,13 @@
     "@earendil-works/pi-tui": "^1.4.0",
     "typebox": "^1.4.2",
     "typescript": "^7.0.2",
-    "vite": "npm:@voidzero-dev/vite-plus-core@1.3.0",
-    "vite-plus": "1.3.0"
+    "vite": "npm:@voidzero-dev/vite-plus-core@1.4.0",
+    "vite-plus": "1.4.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "*",
     "@earendil-works/pi-tui": "*"
   },
-  "engines": {
-    "node": ">=24"
-  },
   "pi": {
     "extensions": [
       "./src/index.ts"

package/src/approval-prompt.ts

@@ -1,5 +1,13 @@
+import { keyHint, rawKeyHint } from "@earendil-works/pi-coding-agent";
 import type { Component } from "@earendil-works/pi-tui";
-import { truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
+import {
+  Key,
+  matchesKey,
+  sliceByColumn,
+  truncateToWidth,
+  visibleWidth,
+  wrapTextWithAnsi,
+} from "@earendil-works/pi-tui";
 
 const ACTIONS = ["Allow", "Deny"] as const;
 
@@ -31,8 +39,12 @@ export function createApprovalPrompt(
   done: (result: Action | undefined) => void,
   command: string,
   rule: string,
+  deadline?: number,
 ): Component {
   let selectedIndex = 0;
+  let horizontalOffset = 0;
+  let maxHorizontalOffset = 0;
+  let pageWidth = 1;
 
   const invalidate = () => {};
 
@@ -55,11 +67,26 @@ export function createApprovalPrompt(
   const component: Component = {
     render(width: number): string[] {
       const available = innerWidth(width);
+
+      const longestLine = Math.max(
+        0,
+        ...command.split("\n").map((line) => visibleWidth(line.replace(/[»«]/gu, ""))),
+      );
+
+      pageWidth = longestLine > available ? Math.max(1, available - 2) : available;
+      maxHorizontalOffset = longestLine > available ? longestLine - pageWidth : 0;
+      horizontalOffset = Math.min(horizontalOffset, maxHorizontalOffset);
       const border = theme.style("─".repeat(available + 4), { fg: "border" });
 
       const lines = [
         border,
-        panelLine(theme.bold("pi-gate: allow this command?"), width, "accent"),
+        panelLine(
+          theme.bold(
+            `${deadline === undefined ? "" : `(${Math.max(0, Math.ceil((deadline - Date.now()) / 1_000))}s) `}pi-gate: allow this command?`,
+          ),
+          width,
+          "accent",
+        ),
         panelLine("Review the requested command or file change:", width, "muted"),
         panelLine("", width),
         ...command.split("\n").map((line) => {
@@ -86,10 +113,15 @@ export function createApprovalPrompt(
             .join("");
 
           const visible = line.replace(/[»«]/gu, "");
-          const content = truncateToWidth(visible, contentWidth, "…");
-          const padding = " ".repeat(Math.max(0, contentWidth - visibleWidth(content)));
+          const content = sliceByColumn(rendered, horizontalOffset, pageWidth, true);
+          const prefix = horizontalOffset > 0 && visibleWidth(visible) > 0 ? "…" : "";
+          const suffix = visibleWidth(visible) > horizontalOffset + pageWidth ? "…" : "";
+
+          const padding = " ".repeat(
+            Math.max(0, contentWidth - visibleWidth(prefix + content + suffix)),
+          );
 
-          return `  ${rendered}${theme.style(padding, { fg: "toolOutput" })}  `;
+          return `  ${theme.style(prefix, { fg: "toolOutput" })}${content}${theme.style(suffix + padding, { fg: "toolOutput" })}  `;
         }),
         panelLine("", width),
         panelLine(`Matched rule: ${rule}`, width, "muted"),
@@ -105,14 +137,21 @@ export function createApprovalPrompt(
           });
         }),
         panelLine("", width),
-        panelLine("↑↓ move · enter confirm · esc cancel", width, "muted"),
+        ...wrapTextWithAnsi(
+          `${rawKeyHint("↑↓", "navigate")}  ${maxHorizontalOffset > 0 ? `${rawKeyHint("←→", "scroll command")}  ` : ""}${keyHint("tui.select.confirm", "select")}  ${keyHint("tui.select.cancel", "cancel")}`,
+          available,
+        ).map((line) => `  ${line}  `),
         border,
       ];
 
       return lines.map((line) => truncateToWidth(line, width));
     },
     handleInput(data: string): void {
-      if (keybindings.matches(data, "tui.select.up") || data === "k") {
+      if (matchesKey(data, Key.left)) {
+        horizontalOffset = Math.max(0, horizontalOffset - pageWidth);
+      } else if (matchesKey(data, Key.right)) {
+        horizontalOffset = Math.min(maxHorizontalOffset, horizontalOffset + pageWidth);
+      } else if (keybindings.matches(data, "tui.select.up") || data === "k") {
         selectedIndex = Math.max(0, selectedIndex - 1);
       } else if (keybindings.matches(data, "tui.select.down") || data === "j") {
         selectedIndex = Math.min(ACTIONS.length - 1, selectedIndex + 1);

package/src/index.ts

@@ -66,6 +66,7 @@ async function selectApproval(
   let uiDone: ((value: ApprovalAction | undefined) => void) | undefined;
   let settled = false;
   let timer: ReturnType<typeof setTimeout> | undefined;
+  let countdown: ReturnType<typeof setInterval> | undefined;
   let removeAbortListener: (() => void) | undefined;
 
   return new Promise<ApprovalAction | undefined>((resolve, reject) => {
@@ -74,6 +75,8 @@ async function selectApproval(
       settled = true;
 
       if (timer !== undefined) clearTimeout(timer);
+
+      if (countdown !== undefined) clearInterval(countdown);
       removeAbortListener?.();
       resolve(value);
       uiDone?.(value);
@@ -82,13 +85,15 @@ async function selectApproval(
     const onAbort = () => finish(undefined);
 
     try {
+      const deadline = Date.now() + timeoutMs;
       timer = setTimeout(() => finish(undefined), timeoutMs);
 
       const uiPromise = ctx.ui.custom<ApprovalAction | undefined>(
         (tui, theme, keybindings, done) => {
           uiDone = done;
+          countdown = setInterval(() => tui.requestRender(), 1_000);
 
-          return createApprovalPrompt(tui, theme, keybindings, done, command, rule);
+          return createApprovalPrompt(tui, theme, keybindings, done, command, rule, deadline);
         },
       );
 
@@ -97,6 +102,8 @@ async function selectApproval(
         .catch(reject)
         .finally(() => {
           if (timer !== undefined) clearTimeout(timer);
+
+          if (countdown !== undefined) clearInterval(countdown);
           removeAbortListener?.();
         });
 
@@ -110,6 +117,8 @@ async function selectApproval(
       settled = true;
 
       if (timer !== undefined) clearTimeout(timer);
+
+      if (countdown !== undefined) clearInterval(countdown);
       removeAbortListener?.();
       reject(error);
     }
@@ -129,7 +138,7 @@ function reasonRuleName(pattern: string): string {
 
 /** Formats an approval timeout with the matched rule, elapsed limit in seconds, and retry guidance. */
 function timeoutReason(pattern: string, timeoutMs: number): string {
-  return `pi-gate: no response to rule "${reasonRuleName(pattern)}" within ${timeoutMs / 1_000}s; command not run. Ask the user whether to retry approval.`;
+  return `pi-gate: approval timed out after ${timeoutMs / 1_000}s (rule "${reasonRuleName(pattern)}"); command not run. Ask the user before retrying.`;
 }
 
 function reportHerdrBlocked(pi: ExtensionAPI, ctx: { mode?: string }, active: boolean): void {
@@ -333,7 +342,7 @@ export default function piGate(pi: ExtensionAPI): void {
         return {
           block: true,
           reason:
-            "pi-gate: approval cancelled or policy changed; command not run. Ask the user whether to retry.",
+            "pi-gate: approval cancelled or policy changed; command not run. Ask the user before retrying.",
           terminate: true,
         };
       }