pi-goal-list-loop-audit 0.28.21 → 0.28.23

package/README.md

@@ -38,6 +38,7 @@ Five top-level commands — `/goal`, `/list`, `/loop`, `/glla`, `/review`:
 /goal pause                        # pause
 /goal resume                       # resume
 /goal cancel                       # abort
+/goal decide                       # re-open the decision picker (v0.28.23)
 /goal tweak "<new objective>"      # edit in place (Confirm dialog)
 /goal archive                      # archived goals, newest first
 /glla                               # settings UI table · /glla key=value · /glla stats · /glla audits [N|full] · /glla postaudit · /glla autoaccept=on

package/extensions/goal-loop-core.ts

@@ -156,6 +156,16 @@ export interface Goal {
   stopReason?: string;
   pauseReason?: string;
   pauseSuggestedAction?: string;
+  /** v0.28.22: pause classification — drives the widget/status rendering
+   * (a decision pause, an operational failure, a time-gated wait, and a
+   * generic block must not look alike). Undefined = legacy flat card. */
+  pauseKind?: "decision" | "error" | "wait" | "blocked";
+  /** v0.28.22: decision pauses — the options the user picks between. */
+  pauseOptions?: string[];
+  /** v0.28.22: 1-based index into pauseOptions the agent recommends. */
+  pauseRecommended?: number;
+  /** v0.28.22: ISO time a wait-pause becomes resumable (countdown shown). */
+  pauseResumeAt?: string;
   /** v0.28.1 (S1/S2): stale-handle interrupt marker. Set INSTEAD of pausing
    * when pi invalidates the extension handle mid-goal — the goal stays
    * active so a fresh session auto-resumes it via the restore gate. Cleared
@@ -193,9 +203,9 @@ export interface Goal {
 export type GoalRoute =
   | { kind: "draft" }
   | { kind: "set"; text: string }
-  | { kind: "sub"; name: "status" | "pause" | "resume" | "cancel" | "tweak" | "archive" | "start"; rest: string };
+  | { kind: "sub"; name: "status" | "pause" | "resume" | "cancel" | "decide" | "tweak" | "archive" | "start"; rest: string };
 
-const GOAL_EXACT_SUBS = new Set(["status", "pause", "resume", "cancel"]);
+const GOAL_EXACT_SUBS = new Set(["status", "pause", "resume", "cancel", "decide"]);
 const GOAL_ARG_SUBS = new Set(["tweak", "archive", "start"]);
 
 export function routeGoalArgs(raw: string): GoalRoute {
@@ -205,7 +215,7 @@ export function routeGoalArgs(raw: string): GoalRoute {
   const first = (space === -1 ? trimmed : trimmed.slice(0, space)).toLowerCase();
   const rest = space === -1 ? "" : trimmed.slice(space + 1).trim();
   if (GOAL_EXACT_SUBS.has(first) && rest === "") {
-    return { kind: "sub", name: first as "status" | "pause" | "resume" | "cancel", rest: "" };
+    return { kind: "sub", name: first as "status" | "pause" | "resume" | "cancel" | "decide", rest: "" };
   }
   if (GOAL_ARG_SUBS.has(first)) {
     return { kind: "sub", name: first as "tweak" | "archive" | "start", rest };

package/extensions/goal-loop-display.ts

@@ -107,6 +107,18 @@ const paint = (theme: DisplayTheme | undefined, color: DisplayColor, text: strin
 const ERROR_PAUSE = /token limit|stalled|infra|auditor.*fail/i;
 const pauseIsError = (g: Goal): boolean => ERROR_PAUSE.test(g.pauseReason ?? "");
 
+/** v0.28.22: the rendering class of a pause — declared kind wins; legacy
+ * pauses (no kind) fall back to the error-regex so old states still
+ * classify sensibly. */
+type PauseKind = "decision" | "error" | "wait" | "blocked";
+const pauseKind = (g: Goal): PauseKind | undefined => g.pauseKind ?? (pauseIsError(g) ? "error" : undefined);
+
+/** v0.28.22: "06:40 UTC" from an ISO string (wait-pause countdown). */
+const shortClock = (iso: string): string => {
+  const d = new Date(iso);
+  return Number.isNaN(d.getTime()) ? iso.slice(0, 16) : d.toISOString().slice(11, 16) + " UTC";
+};
+
 // ---- status line (one-liner, always-on) ----
 
 export interface AuditDisplayProgress {
@@ -150,6 +162,13 @@ export function buildStatusText(state: State, audit?: AuditDisplayProgress | nul
     return `glla: ${paint(theme, "accent", "auditing…")}${tool}${heldSuffix}`;
   }
   if (g.status === "paused") {
+    // v0.28.22: the status line names the ACTIONABILITY, not the reason —
+    // "decision needed" / "action needed" / "waiting" tell you at a glance
+    // whether the session needs you. Legacy pauses keep the reason dump.
+    const kind = pauseKind(g);
+    if (kind === "decision") return `glla: ${g.policy} ${paint(theme, "accent", "⏸ decision needed")}${heldSuffix}`;
+    if (kind === "error") return `glla: ${g.policy} ${paint(theme, "error", `⏸ action needed — ${truncate(g.pauseReason ?? "", 30)}`)}${heldSuffix}`;
+    if (kind === "wait") return `glla: ${g.policy} ${paint(theme, "dim", `⏳ waiting${g.pauseResumeAt ? ` · resumes ${shortClock(g.pauseResumeAt)}` : ""}`)}${heldSuffix}`;
     const label = `${g.policy} paused ⏸ ${truncate(g.pauseReason ?? "", 40)}`;
     return `glla: ${paint(theme, pauseIsError(g) ? "error" : "warning", label)}${heldSuffix}`;
   }
@@ -274,14 +293,38 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
     return lines;
   }
   if (g.status === "paused" && g.pauseReason) {
-    const isErr = pauseIsError(g);
+    const kind = pauseKind(g);
+    const isErr = kind === "error";
     const budget = budgetFor(width, 3, 60);
-    // v0.27.1: wrap reason + suggested action over up to 3 lines each
-    // (see wrap()); before, both were truncated at ~60 chars and the actual
-    // question in a decision-pause never reached the user.
-    wrap(g.pauseReason, budget, 3).forEach((w, i) => {
-      lines.push(`${i === 0 ? "├─" : "│ "} ${paint(theme, isErr ? "error" : "warning", w)}`);
+    // v0.28.22: actionability banner — a decision pause, an operational
+    // failure, and a time-gated wait must not look alike (user report:
+    // "if something actionable is going on it can be hard to tell").
+    if (kind === "decision") lines.push(`├─ ${paint(theme, "accent", "decision needed — your call unblocks this")}`);
+    else if (kind === "error") lines.push(`├─ ${paint(theme, "error", "action needed — this won't fix itself")}`);
+    else if (kind === "wait") lines.push(`├─ ${paint(theme, "dim", "waiting — nothing for you to do")}`);
+    // v0.27.1: wrap reason + suggested action (see wrap()). v0.28.22:
+    // decision/wait reasons cap at 2 lines — the options/countdown below
+    // carry the actionable content; error reasons keep 3.
+    const reasonPaint = isErr ? "error" : kind === "wait" ? "dim" : "warning";
+    wrap(g.pauseReason, budget, kind === "decision" || kind === "wait" ? 2 : 3).forEach((w, i) => {
+      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`)}`);
+    }
+    // v0.28.22: wait countdown — when the pause lifts on its own.
+    if (kind === "wait" && g.pauseResumeAt) {
+      const ms = Date.parse(g.pauseResumeAt) - now;
+      const when = Number.isNaN(ms) ? g.pauseResumeAt : ms <= 0 ? "now" : `${shortClock(g.pauseResumeAt)} (in ${fmtElapsed(ms)})`;
+      lines.push(`├─ ${paint(theme, "dim", `resumes ${when} — or /goal resume now`)}`);
+    }
     // v0.27.1: what survives the pause — the first question at a pause is
     // "did I lose the work?". Answer it on the card.
     // v0.27.9: when the goal has no telemetry yet (restored-in-fresh-session
@@ -300,7 +343,9 @@ function goalLines(g: Goal, state: State, audit: AuditDisplayProgress | null | u
     if (g.pauseSuggestedAction) {
       lines.push(`├─ ${paint(theme, "dim", truncate(savedLine, budget))}`);
       const wrapped = wrap(g.pauseSuggestedAction, budget, 3);
-      wrapped.forEach((w, i) => lines.push(`${i === wrapped.length - 1 ? "└─" : "│ "} ${paint(theme, "dim", w)}`));
+      // 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)}`));
     } else {
       lines.push(`└─ ${paint(theme, "dim", truncate(savedLine, budget))}`);
     }

package/extensions/goal-settings.ts

@@ -33,6 +33,10 @@ export interface Settings {
   /** on → restored goals/loops/lists auto-resume even in fresh sessions
    * (unattended rigs). Default off: restore holds until /goal resume. */
   autoResume?: boolean;
+  /** v0.28.23: off → decision pauses don't pop the select() picker (the
+   * widget card still shows the options; /goal decide opens it on demand).
+   * Default on; unattended rigs have no UI so this never fires there. */
+  decisionPopup?: boolean;
   /** v0.28.14: what happens to stale carryover (paused goal, waiting list,
    * held loop from before this session) when NEW work activates.
    * pause (default) = leave it + ONE summary; clear = drop it all honestly;
@@ -156,6 +160,7 @@ export const SETTINGS_KEYS: Array<keyof Settings> = [
   "tokenLimit",
   "wedgeAlertMinutes",
   "autoResume",
+  "decisionPopup",
   "carryover",
   "autoAcceptDrafts",
   "auditCap",

package/extensions/loops/goal.ts

@@ -498,6 +498,7 @@ function escalateSendRearmStorm(ctx: ExtensionContext, kind: "continuation" | "l
   if (state.goal && state.goal.status === "active") {
     updateGoal({
       status: "paused",
+      pauseKind: "error",
       pauseReason: `send-retry storm: ${mins}m of 50ms re-arms — the session never went idle for the continuation`,
       pauseSuggestedAction: "The session never went idle for the send (wedged queue or permanently busy). Restart pi, then /goal resume.",
     }, ctx);
@@ -521,6 +522,7 @@ function escalateStallNow(ctx: ExtensionContext, threshold: number): boolean {
   if (state.goal && state.goal.status === "active") {
     updateGoal({
       status: "paused",
+      pauseKind: "error",
       pauseReason: `stalled: ${threshold} continuation refires landed no turn`,
       pauseSuggestedAction: "The continuation chain is broken in this process (wedged message queue or stale API). Restart pi, then /goal resume.",
     }, ctx);
@@ -1136,6 +1138,13 @@ async function cmdGoal(args: string, ctx: ExtensionContext): Promise<void> {
     if (route.name === "pause") return cmdPause(ctx);
     if (route.name === "resume") return cmdResume(ctx);
     if (route.name === "cancel") return cmdCancel(ctx);
+    // v0.28.23: re-open the decision picker for a decision pause (the
+    // popup auto-opens when the pause lands; this is the on-demand path).
+    if (route.name === "decide") {
+      const shown = await showDecisionPrompt(ctx);
+      if (!shown) ctx.ui.notify("No pending decision — the goal isn't paused on a choice (or no UI).", "info");
+      return;
+    }
     if (route.name === "tweak") return cmdTweak(route.rest, ctx);
     if (route.name === "archive") return cmdGoals(ctx);
     // v0.16.0: /goal start <objective> — explicit skip-draft. Activates
@@ -1259,7 +1268,7 @@ async function cmdResume(ctx: ExtensionContext): Promise<void> {
   const usage = state.goal.usage
     ? { tokensUsed: state.goal.usage.tokensUsed, tokensLimit: freshLimit }
     : undefined;
-  updateGoal({ status: "active", pauseReason: undefined, pauseSuggestedAction: undefined, ...(staleEntry ? { interruptedAt: nowIso(), interruptedReason: "resumed in a stale session" } : {}), ...(usage ? { usage } : {}) }, ctx);
+  updateGoal({ status: "active", pauseReason: undefined, pauseSuggestedAction: undefined, pauseKind: undefined, pauseOptions: undefined, pauseRecommended: undefined, pauseResumeAt: undefined, ...(staleEntry ? { interruptedAt: nowIso(), interruptedReason: "resumed in a stale session" } : {}), ...(usage ? { usage } : {}) }, ctx);
   if (staleEntry) return;
   // v0.22.5: say what was resumed — with a non-empty list this also resumes
   // the queue (the active goal IS the list's head item).
@@ -1289,6 +1298,68 @@ async function cmdCancel(ctx: ExtensionContext): Promise<void> {
   ctx.ui.notify(`Goal aborted.${isLoopActive() ? " A loop is still active — /loop stop ends it." : ""}`, "info");
 }
 
+// ---- v0.28.23: decision picker popup ----
+// A decision pause is ACTIONABLE — the widget card summarizes (and
+// truncates) it, but picking from a truncated wall was the user's
+// complaint. Borrow Claude Code / muselinn-Ask: a real select() modal
+// with the FULL option text, pick → act. Escape leaves the card as the
+// fallback; /goal decide re-opens the picker at any time.
+
+let decisionPromptOpen = false;
+
+/** True when the goal is paused on a user decision with options. */
+function pendingDecision(): Goal | null {
+  const g = state.goal;
+  return g && g.status === "paused" && g.pauseKind === "decision" && g.pauseOptions && g.pauseOptions.length > 0 ? g : null;
+}
+
+/** Open the decision picker for the current decision pause. Returns true
+ * when a picker was shown (false → caller notifies "no pending decision"). */
+async function showDecisionPrompt(ctx: ExtensionContext): Promise<boolean> {
+  const g = pendingDecision();
+  if (!g || !ctx.hasUI || decisionPromptOpen) return false;
+  decisionPromptOpen = true;
+  try {
+    const title = `Decision needed — ${g.objective.replace(/\s+/g, " ").slice(0, 72)}${g.pauseReason ? ` · ${g.pauseReason.slice(0, 80)}` : ""}`;
+    const options = g.pauseOptions!.map((o, i) => (g.pauseRecommended === i + 1 ? `${o}  (recommended)` : o));
+    const pick = await ctx.ui.select(title, options);
+    if (!pick) return true; // Escape — the widget card remains the fallback
+    const idx = options.indexOf(pick);
+    const label = g.pauseOptions![idx] ?? pick.replace(/ {2}\(recommended\)$/, "");
+    // Executable options — "Label (/goal cancel)" — RUN the command.
+    // Placeholder commands (…/<arg>) fall through to the message path.
+    const cmdMatch = label.match(/\(\/(goal|list|loop) ([a-z]+)\)\s*$/);
+    if (cmdMatch && !label.includes("…") && !label.includes("<")) {
+      const [, group, verb] = cmdMatch;
+      if (group === "goal" && verb === "resume") await cmdResume(ctx);
+      else if (group === "goal" && verb === "cancel") await cmdCancel(ctx);
+      else if (group === "loop" && verb === "stop") await cmdLoop("stop", ctx);
+      else if (group === "loop" && verb === "resume") await cmdLoop("resume", ctx);
+      else {
+        extensionApi?.sendUserMessage(`Decision for the paused goal "${g.objective}": ${label} — continue on this path.`);
+        await cmdResume(ctx);
+      }
+      return true;
+    }
+    // Content choice — deliver to the agent, then resume.
+    extensionApi?.sendUserMessage(`Decision for the paused goal "${g.objective}": ${label} — continue on this path.`);
+    await cmdResume(ctx);
+    return true;
+  } finally {
+    decisionPromptOpen = false;
+  }
+}
+
+/** Pop the picker after a decision pause lands — deferred so the current
+ * turn finishes first (pi serializes dialogs). No-ops without a UI, when
+ * disabled (/glla decisionpopup=off), or when one is already open. */
+function maybeDecisionPopup(ctx: ExtensionContext): void {
+  if (!ctx.hasUI || loadSettings(ctx.cwd).decisionPopup === false) return;
+  setTimeout(() => {
+    void showDecisionPrompt(ctx).catch(() => {});
+  }, 600);
+}
+
 async function cmdGoals(ctx: ExtensionContext): Promise<void> {
   const dir = archiveDir(ctx.cwd);
   if (!fs.existsSync(dir)) {
@@ -2050,10 +2121,10 @@ async function cmdLoop(args: string, ctx: ExtensionContext): Promise<void> {
   const sub = (parts[0] ?? "").toLowerCase();
   const rest = args.trim().slice(sub.length).trim();
 
-  if (!sub) {
-    // /loop with no args → resume a held loop if one is waiting; otherwise
-    // draft the loop config (metric design is the whole game for a
-    // long-running loop; never start one blind).
+  if (!sub || sub === "resume") {
+    // /loop with no args (or /loop resume, v0.28.22) → resume a held loop
+    // if one is waiting; otherwise draft the loop config (metric design is
+    // the whole game for a long-running loop; never start one blind).
     if (isLoopActive()) {
       ctx.ui.notify("A loop is already active — /loop status to inspect, /loop stop to end it.", "info");
       return;
@@ -2063,7 +2134,7 @@ async function cmdLoop(args: string, ctx: ExtensionContext): Promise<void> {
       // v0.28.14: one-active-thing — a held loop must not resume over an
       // active goal/list-item (this was the last unguarded stacking path).
       if (state.goal && state.goal.status === "active") {
-        ctx.ui.notify("A goal is active — the held loop stays held. /goal pause or /goal cancel it first, then /loop to resume.", "warning");
+        ctx.ui.notify("A goal is active — the held loop stays held. /goal pause or /goal cancel it first, then /loop resume.", "warning");
         return;
       }
       state.loop = { ...stored, active: true, stopReason: undefined };
@@ -2075,6 +2146,10 @@ async function cmdLoop(args: string, ctx: ExtensionContext): Promise<void> {
       );
       return;
     }
+    if (sub === "resume") {
+      ctx.ui.notify("No held loop to resume. /loop to draft one, or /loop start \"<target>\" for an infinite metricless loop.", "info");
+      return;
+    }
     await startDrafting(ctx, "loop");
     return;
   }
@@ -2440,10 +2515,14 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
         updateGoal({
           status: "paused",
           auditHistory: history,
+          pauseKind: "decision",
+          pauseOptions: ["Tweak the objective — /goal tweak <new text>", "Cancel the goal (/goal cancel)"],
+          pauseRecommended: 1,
           pauseReason: `auditor verdict: IMPOSSIBLE — ${reason}`,
           pauseSuggestedAction: "The auditor says this goal can never be satisfied as stated. /goal tweak the objective (or /goal cancel), then /goal resume.",
         }, ctx);
         ctx.ui.notify(`Auditor: goal IMPOSSIBLE — ${reason}. Goal paused; /goal tweak or /goal cancel, then /goal resume.`, "warning");
+        maybeDecisionPopup(ctx);
         appendLedger(ctx.cwd, "goal_paused", { reason: `auditor impossible: ${reason}` });
         notifyExternal(ctx, `Goal paused (auditor: impossible): ${reason.slice(0, 120)}`);
         return {
@@ -2472,6 +2551,8 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
             status: "paused",
             auditHistory: history,
             auditInfraStreak: undefined, // quota reached the auditor — infra streak broken
+            pauseKind: "wait",
+            pauseResumeAt: new Date(Date.now() + quota.retryAfterSec * 1000).toISOString(),
             pauseReason: `auditor quota: ${result.error}`,
             pauseSuggestedAction: `Quota auto-retry in ${retryMin}m — or /goal resume to retry now`,
           }, ctx);
@@ -2506,6 +2587,7 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
             status: "paused",
             auditHistory: history,
             auditInfraStreak: infraStreak,
+            pauseKind: "error",
             pauseReason: `auditor infrastructure failed ${infraStreak}× in a row — the auditor model is likely broken (last: ${result.error.slice(0, 120)})`,
             pauseSuggestedAction: "Fix the auditor model (/glla model=provider/id) or restart pi, then /goal resume. Your work was NOT judged.",
           }, ctx);
@@ -2612,10 +2694,14 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
         updateGoal({
           status: "paused",
           auditHistory: history,
+          pauseKind: "decision",
+          pauseOptions: ["Fix the disapproval gap, then continue (/goal resume)", "Tweak the objective — /goal tweak <new text>", "Cancel the goal (/goal cancel)"],
+          pauseRecommended: 1,
           pauseReason: `auditor disapproved ${trailingDisapprovals}× consecutively (cap ${auditCap})`,
           pauseSuggestedAction: "Read the audit history (/goal status), fix the actual gap or /goal tweak the objective, then /goal resume. Raise the cap with /glla auditcap=N.",
         }, ctx);
         ctx.ui.notify(`Goal paused: auditor disapproved ${trailingDisapprovals}× consecutively (cap ${auditCap}). /goal status for the reports; /goal resume to continue.`, "warning");
+          maybeDecisionPopup(ctx);
         appendLedger(ctx.cwd, "goal_paused", { reason: `disapproval cap: ${trailingDisapprovals} consecutive (cap ${auditCap})` });
         notifyExternal(ctx, `Goal paused: ${trailingDisapprovals} consecutive auditor disapprovals`);
         return {
@@ -2646,21 +2732,30 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
   pi.registerTool(defineTool({
     name: "pause_goal",
     label: "Pause goal",
-    description: "Pause the active goal with a reason and suggested action. Use when blocked on user input or unable to make progress.",
+    description: "Pause the active goal with a reason and suggested action. Use when blocked on user input or unable to make progress. When the user must CHOOSE between options, pass kind=\"decision\" with the options list (recommended = 1-based index of the best one) — decision pauses render as a prominent DECISION NEEDED card. Time-gated waits (retry at a specific time) use kind=\"wait\" with resumeAt (ISO). Operational failures use kind=\"error\".",
     parameters: Type.Object({
       reason: Type.String({ description: "Why the work is paused" }),
       suggestedAction: Type.Optional(Type.String({ description: "What the user should do next" })),
+      kind: Type.Optional(Type.Union([Type.Literal("decision"), Type.Literal("error"), Type.Literal("wait"), Type.Literal("blocked")], { description: "Pause class: decision (user picks an option), error (operational failure), wait (time-gated), blocked (generic)" })),
+      options: Type.Optional(Type.Array(Type.String(), { description: "For kind=decision: the options the user picks between (one line each)" })),
+      recommended: Type.Optional(Type.Number({ description: "For kind=decision: 1-based index of the recommended option" })),
+      resumeAt: Type.Optional(Type.String({ description: "For kind=wait: ISO time the pause lifts (countdown is shown)" })),
     }),
     async execute(_id, params, _signal, _onUpdate, execCtx) {
       const foreign1 = foreignToolGuard(execCtx);
       if (foreign1) return { content: [{ type: "text", text: foreign1 }], details: {} };
-      const p = params as { reason: string; suggestedAction?: string };
+      const p = params as { reason: string; suggestedAction?: string; kind?: "decision" | "error" | "wait" | "blocked"; options?: string[]; recommended?: number; resumeAt?: string };
       if (!state.goal) return { content: [{ type: "text", text: "No active goal." }], details: {} };
       updateGoal({
         status: "paused",
         pauseReason: p.reason,
         pauseSuggestedAction: p.suggestedAction,
+        pauseKind: p.kind,
+        pauseOptions: p.kind === "decision" && p.options && p.options.length > 0 ? p.options : undefined,
+        pauseRecommended: p.kind === "decision" && p.recommended && p.recommended >= 1 ? Math.floor(p.recommended) : undefined,
+        pauseResumeAt: p.kind === "wait" && p.resumeAt ? p.resumeAt : undefined,
       }, ctx);
+      if (p.kind === "decision" && p.options && p.options.length > 0) maybeDecisionPopup(ctx);
       // v0.27.1: surface the FULL pause contract — reason AND suggested
       // action. Before, the action only appeared in /goal status and the
       // widget truncated both at ~60 chars, so decision-pauses ("choose a
@@ -3386,6 +3481,14 @@ export async function handleSettingChoice(id: string, ctx: ExtensionContext): Pr
       if (v) saveSettings("global", ctx.cwd, { autoAcceptDrafts: v.startsWith("on") ? true : undefined });
       return;
     }
+    case "decisionPopup": {
+      const v = await ctx.ui.select("Decision popup (v0.28.23 — decision pauses pop the select() picker)", [
+        "on — a decision pause opens the picker; the widget card is the Escape fallback",
+        "off — widget card only; /goal decide opens the picker on demand",
+      ]);
+      if (v) saveSettings("global", ctx.cwd, { decisionPopup: v.startsWith("off") ? false : undefined });
+      return;
+    }
     case "aggressiveMode": {
       const v = await ctx.ui.select("Aggressive mode (flips DEFAULTS toward keep-going — explicit per-key settings still win)", [
         "off — current behavior: pause at the audit cap, wedge alerts on, manual resume",
@@ -3941,6 +4044,16 @@ async function cmdSettings(args: string, ctx: ExtensionContext): Promise<void> {
       } else {
         ctx.ui.notify(`autoresume must be on or off, got: ${value}`, "warning");
       }
+    } else if (key === "decisionpopup") {
+      if (["on", "true", "1", "yes"].includes(value)) {
+        patch.decisionPopup = true;
+        changed = true;
+      } else if (["off", "false", "0", "no"].includes(value)) {
+        patch.decisionPopup = false;
+        changed = true;
+      } else {
+        ctx.ui.notify(`decisionpopup must be on or off, got: ${value}`, "warning");
+      }
     } else if (key === "carryover") {
       if (["resume", "pause", "clear"].includes(value)) {
         patch.carryover = value as "resume" | "pause" | "clear";
@@ -4072,7 +4185,7 @@ async function cmdSettings(args: string, ctx: ExtensionContext): Promise<void> {
     }
   }
   if (!changed) {
-    ctx.ui.notify("Nothing changed. Use key=value (model, thinking, notify, tokenlimit, autoresume, carryover, auditcap, auditfeedbackchars, aggressivemode, quotaretryminutes, stuckmax), optionally prefixed with 'project'.", "info");
+    ctx.ui.notify("Nothing changed. Use key=value (model, thinking, notify, tokenlimit, autoresume, decisionpopup, carryover, auditcap, auditfeedbackchars, aggressivemode, quotaretryminutes, stuckmax), optionally prefixed with 'project'.", "info");
     return;
   }
   saveSettings(scope, ctx.cwd, patch);
@@ -4206,6 +4319,7 @@ export default function (pi: ExtensionAPI): void {
       ["notify=", "desktop push command: /glla notify='notify-send pi \"$1\"'"],
       ["tokenlimit=", "per-goal token budget (0 = off): /glla tokenlimit=2000000"],
       ["autoresume=", "default: hold when a session is loaded, auto-resume on reload/fork; on: always auto-resume; off: never"],
+      ["decisionpopup=", "on|off: decision pauses pop the select() picker (default on; the widget card always lists the options, /goal decide reopens the picker)"],
       ["auditcap=", "N: pause goal after N consecutive auditor disapprovals (default 5, 0 = unlimited)"],
       ["auditfeedbackchars=", "cap on executor-visible disapproval report chars (0 = full report, the default)"],
       ["aggressivemode=", "on: keep-going defaults — autoResume, cap 10, stuck 10, wedge off, quota auto-retry, cap→TODOs"],
@@ -4462,7 +4576,7 @@ export default function (pi: ExtensionAPI): void {
         state.loop = { ...l, active: false, stopReason: HELD_ON_RESTORE };
         persistState(ctx);
         ctx.ui.notify(
-          `Loop held on restore: ${l.target.slice(0, 60)} — /loop to resume, /glla autoresume=on to auto-resume on session load in this project.`,
+          `Loop held on restore: ${l.target.slice(0, 60)} — /loop resume to continue, /glla autoresume=on to auto-resume on session load in this project.`,
           "info",
         );
       }
@@ -4492,6 +4606,7 @@ export default function (pi: ExtensionAPI): void {
         const resumeHint = `${resumeCmd} to continue${queued > 0 ? ` (+${queued} waiting in the list)` : ""} · /glla autoresume=on to auto-resume in this project`;
         updateGoal({
           status: "paused",
+          pauseKind: "blocked",
           pauseReason: "restored on session load — held for explicit resume",
           pauseSuggestedAction: resumeHint,
         }, ctx);
@@ -4522,6 +4637,9 @@ export default function (pi: ExtensionAPI): void {
     if (state.loop && state.goal && state.goal.status === "active") {
       updateGoal({
         status: "paused",
+        pauseKind: "decision",
+        pauseOptions: ["Stop the loop, then resume the goal (/loop stop)", "Cancel the goal (/goal cancel) — the loop keeps running"],
+        pauseRecommended: 1,
         pauseReason: "held on session load — the loop owns the active slot (one active thing at a time)",
         pauseSuggestedAction: "/loop to work the loop, or /loop stop then /goal resume to work the goal",
       }, ctx);
@@ -4529,6 +4647,7 @@ export default function (pi: ExtensionAPI): void {
         `Goal [${state.goal.id}] held — a loop also exists; one active thing at a time. /loop to resume the loop, or /loop stop then /goal resume.`,
         "info",
       );
+      maybeDecisionPopup(ctx);
     }
     // Always paint on session load (v0.22.1): the branches above only reach
     // refreshUI via persistState, so a goal that was ALREADY paused (or any
@@ -4617,10 +4736,14 @@ export default function (pi: ExtensionAPI): void {
         if (state.goal) {
           updateGoal({
             status: "paused",
+            pauseKind: "decision",
+            pauseOptions: ["Retry — /goal resume", "Tweak the objective — /goal tweak <new text>", "Cancel the goal (/goal cancel)"],
+            pauseRecommended: 1,
             pauseReason: `stalled: ${HEARTBEAT_MAX_NUDGES} consecutive unproductive turns (no tools, short or repetitive)`,
             pauseSuggestedAction: "Inspect the goal — /goal resume to retry, /goal tweak to narrow it, /goal cancel to abort.",
           }, ctx);
           ctx.ui.notify(`Goal paused: stalled (${HEARTBEAT_MAX_NUDGES} unproductive turns).`, "warning");
+          maybeDecisionPopup(ctx);
           notifyExternal(ctx, "Goal paused: stalled (no tool calls).");
           return;
         }
@@ -4664,6 +4787,7 @@ export default function (pi: ExtensionAPI): void {
         updateGoal({
           usage: { tokensUsed: used, tokensLimit: limit },
           status: "paused",
+          pauseKind: "error",
           pauseReason: `token limit exceeded (${used.toLocaleString()} > ${limit.toLocaleString()})`,
           pauseSuggestedAction: "/glla tokenlimit=<n> to raise the cap (or 0 to disable), then /goal resume",
         }, ctx);
@@ -4687,6 +4811,8 @@ export default function (pi: ExtensionAPI): void {
         const reason = `5 consecutive errors${detail}`;
         updateGoal({
           status: "paused",
+          pauseKind: "wait",
+          pauseResumeAt: new Date(Date.now() + 60_000).toISOString(),
           pauseReason: reason,
           pauseSuggestedAction: "Transient provider flake? The goal auto-resumes once in 60s if still paused for this reason — or /goal resume now.",
         }, ctx);
@@ -4714,6 +4840,7 @@ export default function (pi: ExtensionAPI): void {
       if (consecutiveAbortIterations >= 5) {
         updateGoal({
           status: "paused",
+          pauseKind: "blocked",
           pauseReason: "5 consecutive aborts (user interrupted)",
           pauseSuggestedAction: "You interrupted 5 turns in a row — the goal stays paused until you /goal resume (or /goal cancel).",
         }, ctx);

package/extensions/settings-menu.ts

@@ -129,7 +129,16 @@ export function buildSettingsRows(
       valueText: show("autoResume", "default"),
       sourceText: src("autoResume"),
       description:
-        "on: resume on session load too · off: never · default: hold on load, resume on reload/fork",
+        "on: resume on session load too · off: never · default: hold on EVERY load — explicit resume (v0.28.21)",
+    },
+    {
+      id: "decisionPopup",
+      section: "keep-going",
+      label: "Decision popup",
+      valueText: show("decisionPopup", "on"),
+      sourceText: src("decisionPopup"),
+      description:
+        "on: decision pauses pop the select() picker · off: widget card only — /goal decide reopens the picker (v0.28.23)",
     },
     {
       id: "autoAcceptDrafts",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-goal-list-loop-audit",
-  "version": "0.28.21",
+  "version": "0.28.23",
   "description": "Goal. Loop. Audit. Done. \u2014 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 \u2014 only the read tools needed to verify your goal.",
   "license": "MIT",
   "author": "dracon",

package/schemas/goal.schema.json

@@ -46,6 +46,10 @@
     "stopReason": { "type": "string" },
     "pauseReason": { "type": "string" },
     "pauseSuggestedAction": { "type": "string" },
+    "pauseKind": { "type": "string", "enum": ["decision", "error", "wait", "blocked"] },
+    "pauseOptions": { "type": "array", "items": { "type": "string" } },
+    "pauseRecommended": { "type": "number" },
+    "pauseResumeAt": { "type": "string" },
     "interruptedAt": { "type": "string" },
     "interruptedReason": { "type": "string" },
     "auditInfraStreak": { "type": "number" },