pi-goal-list-loop-audit 0.38.26 → 0.38.28

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.38.28 — close redundant provenance rows in the active goal card (2026-09-08)
+
+### Fixed
+- The active card no longer shows a duplicate `handled turn` row when it is the same model as the primary model. When provenance is the final detail block, its last row now closes with `└─` instead of a dangling `│`/`├─`, so the card no longer implies missing rows. Failover-handled turns remain visible. Added focused regression coverage; `tsc` and the focused display suite pass.
+
+## 0.38.27 — selective port of PRs #45/#46: /loop pause + widget subtask count (2026-09-08)
+
+### Added
+- `/loop pause` soft-hold (ported from Bjynt's PR #46): parallel to `/goal pause` and `/glla pause` — clears the tick, holds iteration/best/history verbatim with `stopReason: "paused by user (/loop pause)"`, ledger `loop_paused`, no `finishLoopGit`, no queue advance; `/loop resume` picks it up via the extended `RESUMABLE_STOP`. Pinned by `tests/loop-pause.test.ts` (3 tests).
+- Widget task count: a closed parent covers its subtasks (ported from Bjynt's PR #45 — field symptom 6 real tasks displayed "2/24"). Pinned (7/12 + 1/4 cases).
+- Held with recorded rationale (`audit/PR-45-46-DISPOSITION-2026-09-08.md`): `auditTasks` per-task auditor (token-cost surface needs dedicated review), `bypassTriggered` abort shortening (round-trip untested + v0.38.24 abort-latch interaction unverified); dropped the malformed "best solution" auditor line (taste-judge scope creep). Full gate 1990 pass / 0 fail.
+
 ## 0.38.26 — approval-render store: spillway cap keeps undelivered renders (2026-09-08)
 
 ### Fixed

package/docs/INDEX.md

@@ -18,7 +18,7 @@ For shipped docs, the relevant entry points are:
   failback; v0.35.9 hardened cross-version npm tarball checks; v0.35.10
   handles multi-entry npm dry-run reports; v0.35.11 accepts both npm report
   shapes; v0.35.12 supports npm 12's keyed pack reports; v0.35.13 fixes stale-API recovery loops.
-  v0.35.14–v0.38.26 continue through the supervisor freeze (`/glla pause`),
+  v0.35.14–v0.38.28 continue through the supervisor freeze (`/glla pause`),
   load hold, auditor picker parity, Windows launch fix, zombie-watchdog
   subagent carve-out, due-wait backstop, the `/glla agents` visibility panel,
   durable state-root selection, blank-until-resume auditor context, frozen
@@ -28,7 +28,7 @@ For shipped docs, the relevant entry points are:
   packed-artifact release verification, and the 2026-09-07 display/lifecycle/
   settings audit pass (abort-latch send guards, ownership compare-and-swap,
   auditor inherit/clear parity) and the v0.38.25 post-objective summary
-  (canonical approval render, persist + replay, audit-goal counts line) — see CHANGELOG.md for the full trail.
+  (canonical approval render, persist + replay, audit-goal counts line), the v0.38.27 selective port of PRs #45/#46 (`/loop pause` soft-hold, widget subtask count), and the v0.38.28 provenance-row closure fix — see CHANGELOG.md for the full trail.
 - `../README.md` — what the plugin is, install, quickstart, and the
   architectural guarantee (drafting + confirm + detached auditor).
 - `../INSTALL.md` — source install / local development setup; the recommended

package/extensions/goal-loop-display.ts

@@ -294,7 +294,13 @@ function modelProvenanceLines(provenance: ModelProvenanceDisplay | undefined, wi
   const skipped = uniqueModelRefs(provenance.skippedForbiddenRefs);
   if (skipped.length > 0) lines.push(`skipped forbidden: ${truncate(skipped.join(", "), budget)}`);
   const handledTurn = typeof provenance.handledTurn === "string" ? provenance.handledTurn.trim() : "";
-  if (handledTurn) lines.push(`handled turn: ${truncate(handledTurn, budget)}`);
+  // A normal turn is handled by the configured primary model. Repeating the
+  // same ref immediately below it adds no information and, when it is the
+  // last row, leaves a misleading continuation glyph in the card. Keep the
+  // row only when a recovery/failover actually handled the turn.
+  if (handledTurn && handledTurn.toLowerCase() !== primary.toLowerCase()) {
+    lines.push(`handled turn: ${truncate(handledTurn, budget)}`);
+  }
   const handledAudit = typeof provenance.handledAudit === "string" ? provenance.handledAudit.trim() : "";
   if (handledAudit) {
     const source = provenance.handledAuditSource?.trim() ? modelSourceLabel(provenance.handledAuditSource) : "";
@@ -1273,8 +1279,15 @@ function countDone(g: Goal): number {
   let n = 0;
   const walk = (ts: Array<{ status: string; subtasks?: any[] }>) => {
     for (const t of ts) {
-      if (t.status === "complete") n++;
-      if (t.subtasks) walk(t.subtasks);
+      if (t.status === "complete") {
+        // Ported from Bjynt's PR #45: a closed parent covers its subtasks
+        // (field: 2 closed parents with pending subtasks displayed "2/24"
+        // for 6 real tasks). Mid-flight parents still count done subtasks
+        // independently via the walk below.
+        n += 1 + (t.subtasks?.length ?? 0);
+      } else if (t.subtasks) {
+        walk(t.subtasks);
+      }
     }
   };
   walk(g.taskList?.tasks ?? []);
@@ -1573,6 +1586,7 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
   // particular, never replace a configured forbidden ref with "none" — the
   // skipped line explains why it did not handle the turn.
   const provenance = modelProvenanceLines(extras?.modelProvenance, width);
+  const provenanceStart = lines.length;
   provenance.forEach((line, i) => {
     lines.push(`${i === 0 ? "├─" : "│ "} ${paint(theme, "dim", line)}`);
   });
@@ -1897,6 +1911,16 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
   // `/goal status` / `/glla` command hints were extension meta, not task
   // information — dropped. No queue, no footer at all.
   if (queue > 0) lines.push(`└─ ${paint(theme, "dim", `${queue} queued`)}`);
+  // v0.38.28: provenance rows can be the entire detail block when there is
+  // no recent action, pending task, or queue footer. Close that block with a
+  // terminator instead of leaving `├─`/`│` suggesting missing rows.
+  const provenanceEnd = provenanceStart + provenance.length;
+  if (provenance.length > 0 && provenanceEnd === lines.length) {
+    const last = lines[provenanceEnd - 1]!;
+    if (last.startsWith("├─ ") || last.startsWith("│ ")) {
+      lines[provenanceEnd - 1] = `└─ ${last.slice(3)}`;
+    }
+  }
   return lines;
 }
 

package/extensions/goal-loop.ts

@@ -1032,7 +1032,11 @@ async function cmdLoop(args: string, ctx: ExtensionContext): Promise<void> {
       // predicate it lands in never matched the prefix, so /loop resume
       // answered "No held loop to resume" and the preserved iteration,
       // best value, and history were unreachable without re-drafting.
-      !!r?.startsWith("stopped: automatic zero-stream abort");
+      !!r?.startsWith("stopped: automatic zero-stream abort") ||
+      // v0.38.27 (ported from Bjynt's PR #46): /loop pause is a soft-hold
+      // parallel to /goal pause. The loop stays resumable; finishLoopGit
+      // was skipped, so resume picks up iteration/best/history verbatim.
+      !!r?.startsWith("paused by user (/loop pause)");
     if (stored && !stored.active && RESUMABLE_STOP(stored.stopReason)) {
       // Branch-mode stop returns HEAD to originalBranch. Refuse a resume from
       // there rather than letting the next tick commit loop work to the
@@ -1255,6 +1259,39 @@ async function cmdLoop(args: string, ctx: ExtensionContext): Promise<void> {
     return;
   }
 
+  // v0.38.27 (ported from Bjynt's PR #46): soft-hold, parallel to /goal
+  // pause and /glla pause. The loop stops firing iterations but stays in
+  // a resumable held state — NO finishLoopGit, NO hard-stop ledger, NO
+  // list-queue advance. /loop resume picks it up via RESUMABLE_STOP with
+  // iteration, best value, and history preserved verbatim.
+  if (sub === "pause") {
+    if (!state.loop) {
+      ctx.ui.notify("No loop to pause.", "info");
+      return;
+    }
+    if (!state.loop.active) {
+      ctx.ui.notify(`Loop is not active (${state.loop.stopReason ?? "held"}). Use /loop resume to continue, or /loop status to inspect.`, "info");
+      return;
+    }
+    clearLoopTimer();
+    clearToolActivityState();
+    if (state.mainModelRecovery?.kind === "loop") {
+      clearMainModelRecoveryTimer();
+      state.mainModelRecovery = undefined;
+      flags.mainModelAbortForRecovery = false;
+      flags.continuationDispatchStoodDown = false;
+    }
+    state.loop = { ...state.loop, active: false, stopReason: "paused by user (/loop pause)" };
+    persistState(ctx);
+    appendLedger(ctx.cwd, "loop_paused", { reason: "user", iterations: state.loop.iteration, best: state.loop.bestValue });
+    ctx.ui.notify(
+      `Loop paused after ${state.loop.iteration} iterations. Best: ${state.loop.bestValue ?? "n/a"}.\nUse /loop resume to continue.`,
+      "info",
+    );
+    notifyExternal(ctx, `Loop paused by user after ${state.loop.iteration} iterations (best: ${state.loop.bestValue ?? "n/a"}). /loop resume to continue.`);
+    return;
+  }
+
   // v0.25.1: a CLEAN end — "completed: <reason>", distinct from
   // stuck/plateau/stopped-by-user. Additive: /loop stop is untouched.
   if (sub === "finish") {

package/extensions/loops/goal-activation.ts

@@ -867,6 +867,7 @@ export function registerGoalRuntime(pi: ExtensionAPI): void {
       ["polish", "alias of /loop refine"],
       ["stop", "end the loop (keeps the best state)"],
       ["cancel", "alias of /loop stop — end the loop"],
+      ["pause", "hold the active loop (soft-hold, resumable via /loop resume)"],
       ["finish", "end the loop cleanly: /loop finish [reason] → stopReason 'completed: <reason>'"],
     ]),
     handler: (args: string, ctx: ExtensionContext) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.38.26",
+  "version": "0.38.28",
   "description": "Mission control for autonomous pi: interview-drafted goals, an audited task queue, and forever-loops (metric, spec, project-audit) that run for hours. A detached extension-less auditor process re-verifies every completion with raw evidence without holding the main pi turn; confirmed drafts, decision pauses and consent gates keep you in charge.",
   "license": "AGPL-3.0-only",
   "author": "dracon",