pi-goal-list-loop-audit 0.38.25 → 0.38.27

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 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
+- Post-tag review of the v0.38.25 store: the sidecar cap could trim undelivered renders past 20 entries (non-positive slice budget). The cap now applies to delivered history only — undelivered renders are never dropped (self-draining: any user command replays them). Same 7 behavioral pins green; full gate 1986 pass / 0 fail.
+
 ## 0.38.25 — post-objective summary: canonical approval render, persist + replay, audit-goal counts (2026-09-07)
 
 ### 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.25 continue through the supervisor freeze (`/glla pause`),
+  v0.35.14–v0.38.27 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) and the v0.38.27 selective port of PRs #45/#46 (`/loop pause` soft-hold, widget subtask count) — 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/approval-render-store.ts

@@ -47,7 +47,7 @@ function isValidRender(entry: unknown): entry is PendingApprovalRender {
     && (e.deliveredAt === undefined || typeof e.deliveredAt === "string");
 }
 
-function readRenders(cwd: string): PendingApprovalRender[] | undefined {
+function readRenders(cwd: string): PendingApprovalRender[] {
   let raw: string;
   try {
     raw = fs.readFileSync(approvalRenderStorePath(cwd), "utf-8");
@@ -91,7 +91,7 @@ export function persistApprovalRender(cwd: string, render: {
   delivered: boolean;
 }): boolean {
   const at = nowIso();
-  const existing = readRenders(cwd) ?? [];
+  const existing = readRenders(cwd);
   const entry: PendingApprovalRender = {
     goalId: render.goalId,
     objective: render.objective.slice(0, 300),
@@ -99,11 +99,14 @@ export function persistApprovalRender(cwd: string, render: {
     createdAt: at,
     ...(render.delivered ? { deliveredAt: at } : {}),
   };
+  // The cap trims DELIVERED history only: undelivered renders are never
+  // dropped (each is ~1KB and any user command replays them, so the
+  // undelivered tail is self-draining in practice).
   const undelivered = existing.filter((e) => !e.deliveredAt);
-  const delivered = [...existing.filter((e) => e.deliveredAt), ...(render.delivered ? [entry] : [])]
-    .slice(-(MAX_STORED_RENDERS - undelivered.length - (render.delivered ? 0 : 1)));
-  const next = [...undelivered, ...(render.delivered ? [] : [entry]), ...delivered];
-  if (!writeRenders(cwd, next.slice(-MAX_STORED_RENDERS))) return false;
+  const delivered = [...existing.filter((e) => e.deliveredAt), ...(render.delivered ? [entry] : [])];
+  const deliveredBudget = Math.max(0, MAX_STORED_RENDERS - undelivered.length - (render.delivered ? 0 : 1));
+  const next = [...undelivered, ...(render.delivered ? [] : [entry]), ...delivered.slice(-deliveredBudget)];
+  if (!writeRenders(cwd, next)) return false;
   appendLedger(cwd, "terminal_approval_render_persisted", {
     goalId: render.goalId,
     delivered: render.delivered,
@@ -118,7 +121,7 @@ export function persistApprovalRender(cwd: string, render: {
  * throws. */
 export function replayUndeliveredApprovalRenders(ctx: { cwd: string; ui: { notify: (message: string, type?: "info" | "warning" | "error") => void } }): number {
   const renders = readRenders(ctx.cwd);
-  if (!renders || renders.length === 0) return 0;
+  if (renders.length === 0) return 0;
   const pending = renders.filter((e) => !e.deliveredAt).slice(0, MAX_REPLAY_PER_CONTACT);
   if (pending.length === 0) return 0;
   const at = nowIso();

package/extensions/goal-loop-display.ts

@@ -1273,8 +1273,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 ?? []);

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.25",
+  "version": "0.38.27",
   "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",