pi-goal-list-loop-audit 0.19.3 → 0.20.1

package/extensions/goal-loop-display.ts

@@ -17,9 +17,11 @@ export function fmtElapsed(ms: number): string {
   const s = Math.floor(ms / 1000);
   if (s < 60) return `${s}s`;
   const m = Math.floor(s / 60);
-  if (m < 60) return `${m}m`;
+  // Seconds stay visible up to the hour: the elapsed counter is the
+  // liveness signal — minute-only granularity looks frozen on a 1s tick.
+  if (m < 60) return `${m}m ${String(s % 60).padStart(2, "0")}s`;
   const h = Math.floor(m / 60);
-  return `${h}h${m % 60}m`;
+  return `${h}h ${String(m % 60).padStart(2, "0")}m`;
 }
 
 export function fmtTokens(n: number): string {
@@ -37,6 +39,18 @@ function sinceIso(iso: string): number {
   return Number.isFinite(t) ? Date.now() - t : 0;
 }
 
+// ---- semantic colors (optional; tests call without a theme → plain strings) ----
+
+export type DisplayColor = "accent" | "success" | "warning" | "error" | "muted" | "dim";
+export interface DisplayTheme {
+  fg(color: DisplayColor, text: string): string;
+}
+const paint = (theme: DisplayTheme | undefined, color: DisplayColor, text: string): string => (theme ? theme.fg(color, text) : text);
+
+/** Pause reasons that mean "something broke", not "waiting on the user". */
+const ERROR_PAUSE = /token limit|stalled|infra|auditor.*fail/i;
+const pauseIsError = (g: Goal): boolean => ERROR_PAUSE.test(g.pauseReason ?? "");
+
 // ---- status line (one-liner, always-on) ----
 
 export interface AuditDisplayProgress {
@@ -49,25 +63,28 @@ export interface AuditDisplayProgress {
  * One-line status for ctx.ui.setStatus("pi-glla", …).
  * Returns undefined when nothing is being supervised (clears the segment).
  */
-export function buildStatusText(state: State, audit?: AuditDisplayProgress | null, now = Date.now()): string | undefined {
+export function buildStatusText(state: State, audit?: AuditDisplayProgress | null, now = Date.now(), theme?: DisplayTheme): string | undefined {
   if (state.loop?.active) {
     const l = state.loop;
-    const arrow = l.direction === "min" ? "↓" : "↑";
-    return `glla: loop ${arrow} iter ${l.iteration}/${l.maxIterations} · best ${l.bestValue ?? "n/a"} · stall ${l.stallCount}/${l.plateauWindow}`;
+    const arrow = paint(theme, "accent", l.direction === "min" ? "↓" : "↑");
+    const stallText = `stall ${l.stallCount}/${l.plateauWindow}`;
+    const stall = l.stallCount >= l.plateauWindow - 1 ? paint(theme, "warning", stallText) : stallText;
+    return `glla: loop ${arrow} iter ${l.iteration}/${l.maxIterations} · best ${l.bestValue ?? "n/a"} · ${stall}`;
   }
   const g = state.goal;
   if (!g) return undefined;
   if (g.status === "auditing") {
     const tool = audit?.currentTool ? ` · ${audit.currentTool}` : "";
-    return `glla: auditing…${tool}`;
+    return `glla: ${paint(theme, "accent", "auditing…")}${tool}`;
   }
   if (g.status === "paused") {
-    return `glla: paused ⏸ ${truncate(g.pauseReason ?? "", 40)}`;
+    const label = `paused ⏸ ${truncate(g.pauseReason ?? "", 40)}`;
+    return `glla: ${paint(theme, pauseIsError(g) ? "error" : "warning", label)}`;
   }
   if (g.status === "active") {
     const queue = state.list?.length ? ` · list ${state.list.length}` : "";
     const tasks = g.taskList ? ` ${countDone(g)}/${countTotal(g)} tasks ·` : "";
-    return `glla: ${g.policy} ●${tasks} ${fmtElapsed(now - Date.parse(g.createdAt))}${queue}`;
+    return `glla: ${g.policy} ${paint(theme, "success", "●")}${tasks} ${fmtElapsed(now - Date.parse(g.createdAt))}${queue}`;
   }
   return undefined; // complete/aborted → clear
 }
@@ -102,52 +119,60 @@ function countTotal(g: Goal): number {
  * Widget lines for ctx.ui.setWidget("pi-glla", lines).
  * Returns undefined when nothing is worth showing.
  */
-export function buildWidgetLines(state: State, audit?: AuditDisplayProgress | null, now = Date.now()): string[] | undefined {
-  if (state.loop?.active) return loopLines(state.loop, now);
+export function buildWidgetLines(state: State, audit?: AuditDisplayProgress | null, now = Date.now(), theme?: DisplayTheme): string[] | undefined {
+  if (state.loop?.active) return loopLines(state.loop, now, theme);
   const g = state.goal;
   if (!g) return undefined;
   if (g.status === "complete" || g.status === "aborted") return undefined;
-  return goalLines(g, state, audit, now);
+  return goalLines(g, state, audit, now, theme);
 }
 
-function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | undefined, now: number): string[] {
-  const icon = g.status === "paused" ? "⏸" : g.status === "auditing" ? "⟡" : "◆";
+// Branch lines indent one space (pi-tasks convention): the tree sits under
+// the head glyph, text column consistent across widgets.
+function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | undefined, now: number, theme?: DisplayTheme): string[] {
+  const icon =
+    g.status === "paused"
+      ? paint(theme, pauseIsError(g) ? "error" : "warning", "⏸")
+      : g.status === "auditing"
+        ? paint(theme, "accent", "⟡")
+        : paint(theme, "success", "◆");
   const head = `${icon} ${truncate(g.objective.replace(/\s+/g, " "), 64)}`;
-  const lines = [head, `├─ ${statusLabel(g.status)} · ${fmtElapsed(now - Date.parse(g.createdAt))} · ${fmtTokens(g.usage?.tokensUsed ?? 0)}/${fmtTokens(g.usage?.tokensLimit ?? 10_000_000)} tok`];
+  const statusWord = g.status === "active" ? paint(theme, "success", "active") : g.status;
+  const tokens = paint(theme, "dim", `${fmtTokens(g.usage?.tokensUsed ?? 0)}/${fmtTokens(g.usage?.tokensLimit ?? 10_000_000)} tok`);
+  const lines = [head, ` ├─ ${statusWord} · ${fmtElapsed(now - Date.parse(g.createdAt))} · ${tokens}`];
   if (g.status === "auditing") {
-    lines.push(`├─ auditor: ${audit?.label ?? "running"}${audit?.currentTool ? ` · ${truncate(audit.currentTool, 30)}` : ""}`);
-    if (audit?.elapsedMs) lines.push(`└─ ${fmtElapsed(audit.elapsedMs)} in isolated session`);
-    else lines.push(`└─ isolated session, read-only tools`);
+    lines.push(` ├─ auditor: ${audit?.label ?? "running"}${audit?.currentTool ? ` · ${truncate(audit.currentTool, 30)}` : ""}`);
+    if (audit?.elapsedMs) lines.push(` └─ ${paint(theme, "dim", `${fmtElapsed(audit.elapsedMs)} in isolated session`)}`);
+    else lines.push(` └─ ${paint(theme, "dim", "isolated session, read-only tools")}`);
     return lines;
   }
   if (g.status === "paused" && g.pauseReason) {
-    lines.push(`├─ ${truncate(g.pauseReason, 60)}`);
-    if (g.pauseSuggestedAction) lines.push(`└─ ${truncate(g.pauseSuggestedAction, 60)}`);
+    lines.push(` ├─ ${paint(theme, pauseIsError(g) ? "error" : "warning", truncate(g.pauseReason, 60))}`);
+    if (g.pauseSuggestedAction) lines.push(` └─ ${paint(theme, "dim", truncate(g.pauseSuggestedAction, 60))}`);
     return lines;
   }
   const next = nextPending(g);
-  if (next) lines.push(`├─ next: ${truncate(next, 56)}`);
+  if (next) lines.push(` ├─ next: ${truncate(next, 56)}`);
   const queue = state.list?.length ?? 0;
-  lines.push(`└─ ${queue > 0 ? `list ${queue} · ` : ""}/goal status · /glla`);
+  lines.push(` └─ ${paint(theme, "dim", `${queue > 0 ? `list ${queue} · ` : ""}/goal status · /glla`)}`);
   return lines;
 }
 
-function loopLines(l: LoopState, now: number): string[] {
-  const arrow = l.direction === "min" ? "↓" : "↑";
+function loopLines(l: LoopState, now: number, theme?: DisplayTheme): string[] {
+  const arrow = paint(theme, "accent", l.direction === "min" ? "↓" : "↑");
+  const best = paint(theme, "success", `${l.bestValue ?? "n/a"}`);
+  const stallText = `stall ${l.stallCount}/${l.plateauWindow}`;
+  const stall = l.stallCount >= l.plateauWindow - 1 ? paint(theme, "warning", stallText) : stallText;
   const lines = [
-    `◆ ${truncate(l.target, 64)}`,
-    `├─ loop ${arrow} iter ${l.iteration}/${l.maxIterations} · ${fmtElapsed(now - Date.parse(l.startedAt))}`,
-    `├─ best ${l.bestValue ?? "n/a"} · last ${l.lastValue ?? "n/a"} · stall ${l.stallCount}/${l.plateauWindow}`,
-    `└─ ${truncate(l.measureCmd, 56)}`,
+    `${paint(theme, "accent", "◆")} ${truncate(l.target, 64)}`,
+    ` ├─ loop ${arrow} iter ${l.iteration}/${l.maxIterations} · ${fmtElapsed(now - Date.parse(l.startedAt))}`,
+    ` ├─ best ${best} · last ${l.lastValue ?? "n/a"} · ${stall}`,
+    ` └─ ${paint(theme, "dim", truncate(l.measureCmd, 56))}`,
   ];
-  if (l.branchName) lines.push(`   ⎇ ${truncate(l.branchName, 50)}`);
+  if (l.branchName) lines.push(`  ⎇ ${paint(theme, "muted", truncate(l.branchName, 50))}`);
   return lines;
 }
 
-function statusLabel(s: string): string {
-  return s === "active" ? "active" : s;
-}
-
 function nextPending(g: Goal): string | undefined {
   const tasks = g.taskList?.tasks ?? [];
   const queue = [...tasks];

package/extensions/goal-loop-forever.ts

@@ -60,7 +60,7 @@ export interface LoopState {
 export function loopBranchName(startedAtIso: string, target: string): string {
   const stamp = startedAtIso.replace(/[^0-9]/g, "").slice(0, 14);
   const slug = target.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 30) || "loop";
-  return `pi-gla-loop/${stamp}-${slug}`;
+  return `pi-glla-loop/${stamp}-${slug}`;
 }
 
 export const LOOP_DEFAULTS = {

package/extensions/loops/goal.ts

@@ -145,8 +145,9 @@ let uiTicker: NodeJS.Timeout | null = null;
 function refreshUI(ctx: ExtensionContext): void {
   if (!ctx.hasUI) return;
   try {
-    ctx.ui.setStatus("pi-glla", buildStatusText(state, latestAuditProgress));
-    ctx.ui.setWidget("pi-glla", buildWidgetLines(state, latestAuditProgress));
+    const theme = ctx.ui.theme as unknown as import("../goal-loop-display.js").DisplayTheme | undefined;
+    ctx.ui.setStatus("pi-glla", buildStatusText(state, latestAuditProgress, Date.now(), theme));
+    ctx.ui.setWidget("pi-glla", buildWidgetLines(state, latestAuditProgress, Date.now(), theme));
   } catch {
     // stale ctx — next event refreshes
   }
@@ -157,7 +158,7 @@ function startUITicker(): void {
   uiTicker = setInterval(() => {
     const ctx = freshCtx();
     if (ctx && isSupervising()) refreshUI(ctx);
-  }, 5_000);
+  }, 1_000);
   uiTicker.unref?.();
 }
 
@@ -1018,7 +1019,7 @@ async function runLoopTick(ctx: ExtensionContext, event?: any): Promise<void> {
   if (loop.branchName && outcome.kind === "continue") {
     if (outcome.improved) {
       await runGit(ctx, ["add", "-A"]);
-      const committed = await runGit(ctx, ["commit", "-m", `pi-gla-loop: iteration ${loop.iteration} (${loop.direction}=${loop.bestValue})`]);
+      const committed = await runGit(ctx, ["commit", "-m", `pi-glla-loop: iteration ${loop.iteration} (${loop.direction}=${loop.bestValue})`]);
       appendLedger(ctx.cwd, "loop_git", { action: "commit", iteration: loop.iteration, ok: committed.ok });
     } else {
       const reset = await runGit(ctx, ["reset", "--hard", "HEAD"]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.19.3",
+  "version": "0.20.1",
   "description": "Goal. Loop. Audit. Done. — a pi-coding-agent extension that supervises long-running work, with isolated auditor on each completion. Beat bamboozling by design: the auditor runs in a fresh session with no extensions, no skills, no editor — only the read tools needed to verify your goal.",
   "license": "MIT",
   "author": "dracon",