@hank-warren/pi-auto-permissions 0.2.0 → 0.3.0

package/NOTICE.md

@@ -4,4 +4,4 @@ This package is a fork of [`@ogulcancelik/pi-auto-permissions`](https://github.c
 
 The original work is Copyright (c) 2026 Can Celik and is used under the MIT License included in [`LICENSE`](LICENSE).
 
-Fork-specific changes are Copyright (c) 2026 Hank Warren and are released under the same MIT License. They include the `reviewEvidence.userAnswerTools` dialog-answer evidence feature, proposed upstream in [ogulcancelik/pi-extensions#24](https://github.com/ogulcancelik/pi-extensions/issues/24) / [PR #25](https://github.com/ogulcancelik/pi-extensions/pull/25), and the prompted-review evaluation log with explicit user feedback.
+Fork-specific changes are Copyright (c) 2026 Hank Warren and are released under the same MIT License. They include the `reviewEvidence.userAnswerTools` dialog-answer evidence feature, proposed upstream in [ogulcancelik/pi-extensions#24](https://github.com/ogulcancelik/pi-extensions/issues/24) / [PR #25](https://github.com/ogulcancelik/pi-extensions/pull/25), the prompted-review evaluation log with explicit user feedback, and the single-line animated review status widget.

package/README.md

@@ -235,7 +235,13 @@ The path defaults to `usage.jsonl` beside the Auto Permissions config and resolv
 
 ## Review display
 
-The default UI shows guardian progress in a temporary widget below the editor. Configure it with:
+The default UI shows guardian progress as a single animated status line in a temporary widget below the editor:
+
+```text
+auto permissions · Git commit · ✶ waiting for openai-codex/gpt-5.6-luna
+```
+
+A sparkle spinner (`✶ ✸ ✻ ✽`) cycles while the guardian is reviewing and resolves to `✓ approved`, `↻ revision requested`, `? waiting for your approval`, or `✗ blocked`. The guardian's reason, when present, appears on a dim second line; the command itself is not repeated because it is already visible in the Bash tool box. Configure the widget with:
 
 ```json
 {

package/index.ts

@@ -17,6 +17,13 @@ import {
 } from "./evaluation-log.js";
 import { findGates, type Gate } from "./gates.js";
 import { appendUsageRecord, buildUsageLogRecord } from "./usage-log.js";
+import {
+  reviewFrameIntervalMs,
+  reviewStatusFrame,
+  reviewStatusLines,
+  type ReviewDisplayState,
+  type ReviewLinePalette,
+} from "./widget-status.js";
 import {
   buildReviewEnvelope,
   collectReviewEvidence,
@@ -29,7 +36,6 @@ const WIDGET_KEY = "auto-permissions";
 const PROJECT_CONFIG_DIR_NAME = (PiCodingAgent as { CONFIG_DIR_NAME?: string }).CONFIG_DIR_NAME ?? ".pi";
 
 type BlockResult = { block: true; reason: string };
-type ReviewDisplayState = "waiting" | "approved" | "revise" | "ask_user" | "blocked";
 type ReviewTarget = { toolName: string; toolCallId?: string };
 type ReviewRow = {
   gate: Gate;
@@ -204,6 +210,7 @@ export default function autoPermissionsExtension(pi: ExtensionAPI) {
   let bashRendererRegistered = false;
   let bashRendererSourcePath: string | undefined;
   let clearWidgetTimer: ReturnType<typeof setTimeout> | undefined;
+  let widgetAnimTimer: ReturnType<typeof setInterval> | undefined;
   const reviewRows = new Map<string, ReviewRow>();
   const reviewRowInvalidators = new Map<string, () => void>();
   let reviewerLineage: ReviewerLineage | undefined;
@@ -473,6 +480,8 @@ export default function autoPermissionsExtension(pi: ExtensionAPI) {
   function clearReviewWidget(ctx: ExtensionContext): void {
     if (clearWidgetTimer) clearTimeout(clearWidgetTimer);
     clearWidgetTimer = undefined;
+    if (widgetAnimTimer) clearInterval(widgetAnimTimer);
+    widgetAnimTimer = undefined;
     if (ctx.mode === "tui") ctx.ui.setWidget(WIDGET_KEY, undefined);
   }
 
@@ -491,7 +500,7 @@ export default function autoPermissionsExtension(pi: ExtensionAPI) {
     ctx: ExtensionContext,
     config: AutoPermissionsConfig,
     gate: Gate,
-    command: string,
+    _command: string,
     target: ReviewTarget,
     state: ReviewDisplayState,
     detail?: string,
@@ -512,27 +521,45 @@ export default function autoPermissionsExtension(pi: ExtensionAPI) {
 
     if (clearWidgetTimer) clearTimeout(clearWidgetTimer);
     clearWidgetTimer = undefined;
-    ctx.ui.setWidget(WIDGET_KEY, (_tui, theme) => ({
-      render(width: number): string[] {
-        const status = state === "waiting"
-          ? theme.fg("warning", `◌ waiting for ${reviewer}`)
-          : state === "approved"
-            ? theme.fg("success", "✓ approved")
-            : state === "revise"
-              ? theme.fg("warning", "↻ revision requested")
-              : state === "ask_user"
-                ? theme.fg("accent", "? waiting for your approval")
-                : theme.fg("error", "✗ blocked");
-        const lines = [
-          `${theme.fg("accent", theme.bold("auto permissions"))} ${theme.fg("muted", `· ${gate.label}`)}`,
-          `  ${status}`,
-          theme.fg("dim", `  $ ${command}`),
-        ];
-        if (detail) lines.push(theme.fg("muted", `  ${detail}`));
-        return lines.map((line) => truncateToWidth(line, Math.max(1, width)));
-      },
-      invalidate() {},
-    }), { placement: "belowEditor" });
+    if (widgetAnimTimer) clearInterval(widgetAnimTimer);
+    widgetAnimTimer = undefined;
+    ctx.ui.setWidget(WIDGET_KEY, (tui, theme) => {
+      const palette: ReviewLinePalette = {
+        header: (text) => theme.fg("accent", theme.bold(text)),
+        muted: (text) => theme.fg("muted", text),
+        warning: (text) => theme.fg("warning", text),
+        success: (text) => theme.fg("success", text),
+        accent: (text) => theme.fg("accent", text),
+        error: (text) => theme.fg("error", text),
+      };
+      let frameIndex = 0;
+      let timer: ReturnType<typeof setInterval> | undefined;
+      const stopTimer = () => {
+        if (timer) clearInterval(timer);
+        if (widgetAnimTimer === timer) widgetAnimTimer = undefined;
+        timer = undefined;
+      };
+      const intervalMs = reviewFrameIntervalMs(state);
+      if (intervalMs !== undefined) {
+        timer = setInterval(() => {
+          frameIndex++;
+          if (reviewStatusFrame(state, reviewer, frameIndex).done) stopTimer();
+          tui.requestRender();
+        }, intervalMs);
+        timer.unref?.();
+        widgetAnimTimer = timer;
+      }
+      return {
+        render(width: number): string[] {
+          return reviewStatusLines(state, gate.label, reviewer, frameIndex, detail, palette)
+            .map((line) => truncateToWidth(line, Math.max(1, width)));
+        },
+        invalidate() {},
+        dispose() {
+          stopTimer();
+        },
+      };
+    }, { placement: "belowEditor" });
 
     if (autoClear) {
       clearWidgetTimer = setTimeout(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hank-warren/pi-auto-permissions",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "Fork of @ogulcancelik/pi-auto-permissions with dialog-answer user evidence: context-aware Bash permissions for Pi with automated guardian review.",
   "type": "module",
   "keywords": [
@@ -44,6 +44,7 @@
     "gates.ts",
     "review.ts",
     "usage-log.ts",
+    "widget-status.ts",
     "README.md",
     "LICENSE"
   ],

package/widget-status.ts

@@ -0,0 +1,89 @@
+/**
+ * Pure helpers for the below-editor review status widget: single-line status
+ * formatting plus frame progression for the waiting sparkle spinner. Kept
+ * free of TUI dependencies so they are unit-testable.
+ */
+
+export type ReviewDisplayState = "waiting" | "approved" | "revise" | "ask_user" | "blocked";
+
+export type ReviewStatusTone = "warning" | "success" | "accent" | "error";
+
+export interface ReviewStatusFrame {
+  glyph: string;
+  label: string;
+  tone: ReviewStatusTone;
+  /** True once no further frames are needed (state is static). */
+  done: boolean;
+}
+
+export const WAITING_FRAMES = ["✶", "✸", "✻", "✽"] as const;
+export const WAITING_FRAME_INTERVAL_MS = 120;
+
+export function reviewStatusFrame(
+  state: ReviewDisplayState,
+  reviewer: string,
+  frameIndex: number,
+): ReviewStatusFrame {
+  const index = Math.max(0, Math.floor(frameIndex));
+  switch (state) {
+    case "waiting":
+      return {
+        glyph: WAITING_FRAMES[index % WAITING_FRAMES.length],
+        label: `waiting for ${reviewer}`,
+        tone: "warning",
+        done: false,
+      };
+    case "approved":
+      return { glyph: "✓", label: "approved", tone: "success", done: true };
+    case "revise":
+      return { glyph: "↻", label: "revision requested", tone: "warning", done: true };
+    case "ask_user":
+      return { glyph: "?", label: "waiting for your approval", tone: "accent", done: true };
+    case "blocked":
+      return { glyph: "✗", label: "blocked", tone: "error", done: true };
+  }
+}
+
+export function reviewFrameIntervalMs(state: ReviewDisplayState): number | undefined {
+  return state === "waiting" ? WAITING_FRAME_INTERVAL_MS : undefined;
+}
+
+export interface ReviewLinePalette {
+  header: (text: string) => string;
+  muted: (text: string) => string;
+  warning: (text: string) => string;
+  success: (text: string) => string;
+  accent: (text: string) => string;
+  error: (text: string) => string;
+}
+
+export const PLAIN_PALETTE: ReviewLinePalette = {
+  header: (text) => text,
+  muted: (text) => text,
+  warning: (text) => text,
+  success: (text) => text,
+  accent: (text) => text,
+  error: (text) => text,
+};
+
+/**
+ * Render the widget content: one status line, plus a dim detail line only
+ * when a guardian reason is present. The command is intentionally omitted —
+ * it is already visible in the bash tool box above.
+ */
+export function reviewStatusLines(
+  state: ReviewDisplayState,
+  gateLabel: string,
+  reviewer: string,
+  frameIndex: number,
+  detail: string | undefined,
+  palette: ReviewLinePalette,
+): string[] {
+  const frame = reviewStatusFrame(state, reviewer, frameIndex);
+  const status = palette[frame.tone](`${frame.glyph} ${frame.label}`);
+  const lines = [
+    `${palette.header("auto permissions")} ${palette.muted(`· ${gateLabel} ·`)} ${status}`,
+  ];
+  if (detail) lines.push(palette.muted(`  ${detail}`));
+  return lines;
+}