pi-goal-list-loop-audit 0.38.69 → 0.38.71

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.38.71 — Decision cards lead with the action, plaque verbs deduped (2026-09-20)
+
+- Decision pause cards render the saved/action row before the numbered options, so Pi core tail truncation cuts late options and history instead of the resume path (UI survey finding 1, audit/UI-SURVEY-2026-09-20.md).
+- Durable/defer plaque bodies strip a leading Implement/Keep from the stored fix, fixing "Implement Implement Keep ..." doubling; ledger facts untouched (UI survey finding 3).
+
+## 0.38.70 — Paused action cards lead with the action (2026-09-20)
+
+- Paused cards render lifecycle/next, banner, reason, and suggested action before tally, provenance, and judgment, so Pi core tail truncation cuts history instead of the instruction (field 20260920_152744).
+- Judgment rides collapsed on action cards (header + selection; full plaques stay in `/goal status`).
+
 ## 0.38.69 — About and README refresh (2026-09-20)
 
 - Refined npm about line and README intro: tighter mission control wording, no em dashes.

package/docs/INDEX.md

@@ -16,7 +16,7 @@ Policy contracts and recent changes live in the `audit/` directory of the
   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.69 continue through the supervisor freeze (`/glla pause`),
+  v0.35.14–v0.38.71 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

package/extensions/goal-loop-core.ts

@@ -3145,6 +3145,24 @@ export function recommendDurableDeferChoice(input: DurableDeferRecommendationInp
   return input.durableBlocked === true ? "deferred" : "inline";
 }
 
+/** v0.38.71 (field UI-SURVEY-2026-09-20): plaque bodies compose a verb
+ * around the stored fix ("Implement X now" / "Keep X as the follow-up"),
+ * so a fix that already starts with that verb doubled ("Implement
+ * Implement Keep ..."). Strip one leading Implement/Keep; the composed
+ * verb carries the meaning either way. */
+function stripPlaqueVerb(fix: string): string {
+  // Loop twice: fixes arrive as bare phrases ("pin plaque ordering") or
+  // imperative stacks ("Implement Keep ...") — one pass leaves
+  // "Implement Keep ..." doubled against the composed verb.
+  let out = fix;
+  for (let i = 0; i < 2; i++) {
+    const next = out.replace(/^(implement|keep)\s+/i, "");
+    if (next === out) break;
+    out = next;
+  }
+  return out;
+}
+
 /**
  * Build the ordered recommendation plaques consumed by prompt and UI
  * projections. The returned order is driven by DURABLE_DEFER_PLAQUE_ORDER so
@@ -3152,7 +3170,7 @@ export function recommendDurableDeferChoice(input: DurableDeferRecommendationInp
  */
 export function buildDurableDeferRecommendation(input: DurableDeferRecommendationInput): DurableDeferRecommendation {
   const facts = normalizeDurableDeferRecommendationInput(input);
-  const durableFix = facts.durableFix || "the durable root-cause fix";
+  const durableFix = stripPlaqueVerb(facts.durableFix || "the durable root-cause fix");
   const deferCount = facts.deferRecommendations.length;
   const choice = recommendDurableDeferChoice(facts);
   const plaques: Record<DurableDeferPlaqueKind, DurableDeferPlaque> = {

package/extensions/goal-loop-display.ts

@@ -1112,6 +1112,21 @@ function pausedNextTransition(g: Goal, state: State, now: number): string {
   }
 }
 
+/** v0.38.70 (field 20260920_152744): close a paused action card with its
+ * deferred history. The pause body already ended on a `└─` closer; reopen
+ * it to `├─` and hang the tally/provenance/collapsed-judgment rows after
+ * the action, closing on the last history row. Empty history is a no-op,
+ * so cards with nothing deferred keep their exact shape. */
+function closePausedWithHistory(lines: string[], history: string[], theme?: DisplayTheme): string[] {
+  if (history.length === 0) return lines;
+  const tail = lines[lines.length - 1];
+  if (tail !== undefined) lines[lines.length - 1] = tail.replace(/^└─/, "├─");
+  history.forEach((h, i) => {
+    lines.push(`${i === history.length - 1 ? "└─" : "├─"} ${paint(theme, "dim", h)}`);
+  });
+  return lines;
+}
+
 function pausedLifecycleLines(g: Goal, state: State, extras: WidgetExtras | undefined, now: number): [string, string] {
   const queued = state.list?.length ?? 0;
   return [
@@ -1932,9 +1947,20 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
   // v0.38.8: durable verdict tally as a first-class card row — the widget
   // is the glance surface, and stored verdicts are the progress evidence
   // when no auditor is live. Silent when history is empty.
+  // v0.38.70 (field 20260920_152744): a paused action card must lead with
+  // the action, not history. Pi core truncates the widget tail (~10 rows),
+  // so judgment plaques above the blocked banner pushed the suggested action
+  // below the cut — "... (widget truncated)" with no action visible.
+  // Tally, provenance, and judgment ride AFTER the pause closer instead;
+  // judgment rides collapsed (header + selection; full plaques stay in
+  // /goal status). Interrupted/attention cards keep the existing order.
+  const deferHistory = g.status === "paused" && !!g.pauseReason && !interrupted && !attention;
+  const deferredHistory: string[] = [];
   const headTally = formatVerdictTallySegment(auditorVerdictTally(g.auditHistory, now), now);
   if (headTally) {
-    lines.push(`├─ ${paint(theme, "dim", `audits: ${truncate(headTally, Math.max(20, (width ?? 80) - 12))}`)}`);
+    const tallyRow = `audits: ${truncate(headTally, Math.max(20, (width ?? 80) - 12))}`;
+    if (deferHistory) deferredHistory.push(tallyRow);
+    else lines.push(`├─ ${paint(theme, "dim", tallyRow)}`);
   }
   if (g.repairTarget) {
     lines.push(`├─ ${paint(theme, "warning", `REPLAN REQUIRED · original target: ${truncate(g.repairTarget.objective.replace(/\s+/g, " "), Math.max(30, (width ?? 80) - 28))}`)}`);
@@ -1967,12 +1993,22 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
     ? []
     : modelProvenanceLines(extras?.modelProvenance, width);
   const provenanceStart = lines.length;
-  provenance.forEach((line, i) => {
-    lines.push(`${i === 0 ? "├─" : "│ "} ${paint(theme, "dim", line)}`);
-  });
+  if (deferHistory) {
+    provenance.forEach((line) => deferredHistory.push(line));
+  } else {
+    provenance.forEach((line, i) => {
+      lines.push(`${i === 0 ? "├─" : "│ "} ${paint(theme, "dim", line)}`);
+    });
+  }
   if (extras?.durableDeferRecommendation) {
     const judgment = buildDurableDeferDecisionLines(extras.durableDeferRecommendation, width);
-    judgment.forEach((line) => lines.push(`├─ ${paint(theme, "dim", line)}`));
+    // Collapsed on action cards: header + selection only. Plaque bodies
+    // stay in /goal status; the glance card must fit the action.
+    const collapsed = deferHistory && judgment.length > 2
+      ? [judgment[0]!, judgment[judgment.length - 1]!]
+      : judgment;
+    if (deferHistory) deferredHistory.push(...collapsed);
+    else judgment.forEach((line) => lines.push(`├─ ${paint(theme, "dim", line)}`));
   }
   if (interrupted) {
     const resumeCmd = isList ? "/list resume" : "/goal resume";
@@ -2031,7 +2067,7 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
       lines.push(`├─ ${paint(theme, "dim", lifecycle)}`);
       lines.push(`│  ${paint(theme, "dim", transition)}`);
       lines.push(`└─ ${paint(theme, "warning", sanitizeProviderDisplayText(g.pauseSuggestedAction ?? `The claim is safe; ${isList ? "/list resume" : "/goal resume"} starts exactly one fresh auditor.`))}`);
-      return lines;
+      return closePausedWithHistory(lines, deferredHistory, theme);
     }
     const [lifecycle, transition] = pausedLifecycleLines(g, state, extras, now);
     lines.push(`├─ ${paint(theme, "dim", lifecycle)}`);
@@ -2100,16 +2136,9 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
         lines.push(`${i === 0 ? "├─" : "│ "} ${paint(theme, reasonPaint, w)}`);
       });
     }
-    // v0.28.22: decision options — one numbered line each (Claude Code /
-    // muselinn-Ask convention), the recommended one accented and flagged.
-    if (kind === "decision" && g.pauseOptions && g.pauseOptions.length > 0) {
-      g.pauseOptions.slice(0, 6).forEach((opt, i) => {
-        const rec = g.pauseRecommended === i + 1;
-        const text = `${i + 1}. ${truncate(opt, budget - 4)}${rec ? " ◂ recommended" : ""}`;
-        lines.push(`│  ${paint(theme, rec ? "accent" : "dim", text)}`);
-      });
-      if (g.pauseOptions.length > 6) lines.push(`│  ${paint(theme, "dim", `… and ${g.pauseOptions.length - 6} more`)}`);
-    }
+    const decisionOptions = kind === "decision" && g.pauseOptions && g.pauseOptions.length > 0
+      ? g.pauseOptions
+      : undefined;
     // v0.28.22: wait countdown — moved into the auto-retrying line above
     // (v0.34.64). The old separate `resumes X — or /goal resume now` line
     // implied manual rescue was the path; autoResume + the recovery-cleared
@@ -2128,7 +2157,7 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
       // the card here, closing the previous row as the footer.
       const tail = lines[lines.length - 1];
       if (tail !== undefined) lines[lines.length - 1] = tail.replace(/^├─|^│\s*/, "└─ ");
-      return lines;
+      return closePausedWithHistory(lines, deferredHistory, theme);
     }
     const spent: string[] = [];
     const tokUsed = g.usage?.tokensUsed ?? 0;
@@ -2139,16 +2168,35 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
     const savedLine = hasTelemetry
       ? `saved — ${spent.join(" · ")} · resumes exactly here`
       : `awaiting first turn — resumes exactly here`;
+    const optionsFollow = decisionOptions !== undefined;
     if (g.pauseSuggestedAction) {
       lines.push(`├─ ${paint(theme, "dim", truncate(savedLine, budget))}`);
       const wrapped = wrap(sanitizeProviderDisplayText(g.pauseSuggestedAction), budget, 3);
       // v0.28.22: for ACTION NEEDED pauses the action is the point — pop it.
       const actionPaint = kind === "error" ? "warning" : "dim";
-      wrapped.forEach((w, i) => lines.push(`${i === wrapped.length - 1 ? "└─" : "│ "} ${paint(theme, actionPaint, w)}`));
+      // v0.38.71 (field UI-SURVEY-2026-09-20 finding 1): the action precedes
+      // the options — a 7-option decision card runs 16 rows and Pi core cuts
+      // the tail, which used to hold the action. Options survive in full in
+      // the ask_user_question dialog; the card keeps the resume path visible.
+      wrapped.forEach((w, i) => lines.push(`${i === wrapped.length - 1 && !optionsFollow ? "└─" : "│ "} ${paint(theme, actionPaint, w)}`));
     } else {
-      lines.push(`└─ ${paint(theme, "dim", truncate(savedLine, budget))}`);
+      lines.push(`${optionsFollow ? "├─" : "└─"} ${paint(theme, "dim", truncate(savedLine, budget))}`);
     }
-    return lines;
+    // v0.28.22: decision options — one numbered line each (Claude Code /
+    // muselinn-Ask convention), the recommended one accented and flagged.
+    if (decisionOptions) {
+      decisionOptions.slice(0, 6).forEach((opt, i) => {
+        const rec = g.pauseRecommended === i + 1;
+        const text = `${i + 1}. ${truncate(opt, budget - 4)}${rec ? " ◂ recommended" : ""}`;
+        lines.push(`│  ${paint(theme, rec ? "accent" : "dim", text)}`);
+      });
+      if (decisionOptions.length > 6) lines.push(`│  ${paint(theme, "dim", `… and ${decisionOptions.length - 6} more`)}`);
+      // Close the options block; the history helper reopens it when deferred
+      // rows follow.
+      const lastOpt = lines[lines.length - 1];
+      if (lastOpt !== undefined) lines[lines.length - 1] = lastOpt.replace(/^│  /, "└─ ");
+    }
+    return closePausedWithHistory(lines, deferredHistory, theme);
   }
   if (attention) {
     const budget = budgetFor(width, 3, 60);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "pi-goal-list-loop-audit",
-	"version": "0.38.69",
+	"version": "0.38.71",
 	"description": "Mission control for long running pi work: interview drafted goals, an audited task queue, and metric or audit loops that run for hours. Every completion is rechecked by a detached auditor with raw evidence, while Confirm drafts, decision pauses, and consent gates keep you in charge.",
 	"license": "AGPL-3.0-only",
 	"author": "dracon",