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

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # 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

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.27 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) 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.
+  (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) : "";
@@ -1580,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)}`);
   });
@@ -1904,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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.38.27",
+  "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",