pi-goal-list-loop-audit 0.15.0 → 0.16.0

package/README.md

@@ -33,6 +33,7 @@ Four top-level commands, that's all:
 /goal                              # drafting: agent grills, you Confirm
 /goal "audit the repo"             # no contract clause → agent grills you first (propose is gated on it)
 /goal "Step 1. Step 2. Done when: tests pass."   # has contract → starts now
+/goal start "fix the flaky login test"          # explicit skip-draft: starts now, no interview (auditor infers the contract)
 /goal status                       # show state
 /goal pause                        # pause
 /goal resume                       # resume
@@ -69,7 +70,9 @@ your objectives can start with any verb.)
 
 Drafting rules: **no-args drafts, args-without-a-`Done when:`-clause get
 grilled by the agent (proposing is mechanically blocked until you have
-replied at least once), args-with-a-clause start instantly, a file path is
+replied at least once — typed chat or an answered `ask_user_question` dialog
+both count), args-with-a-clause start instantly, `/goal start` skips the
+interview by explicit command, a file path is
 bulk direct.** A
 sisyphus-style plan file (checklists, bullets, numbered, plain lines) imports
 as-is — headings become nothing, items become goals. And the drafter itself

package/extensions/goal-loop-core.ts

@@ -158,10 +158,10 @@ export interface Goal {
 export type GoalRoute =
   | { kind: "draft" }
   | { kind: "set"; text: string }
-  | { kind: "sub"; name: "status" | "pause" | "resume" | "cancel" | "tweak" | "archive"; rest: string };
+  | { kind: "sub"; name: "status" | "pause" | "resume" | "cancel" | "tweak" | "archive" | "start"; rest: string };
 
 const GOAL_EXACT_SUBS = new Set(["status", "pause", "resume", "cancel"]);
-const GOAL_ARG_SUBS = new Set(["tweak", "archive"]);
+const GOAL_ARG_SUBS = new Set(["tweak", "archive", "start"]);
 
 export function routeGoalArgs(raw: string): GoalRoute {
   const trimmed = raw.trim();
@@ -173,7 +173,7 @@ export function routeGoalArgs(raw: string): GoalRoute {
     return { kind: "sub", name: first as "status" | "pause" | "resume" | "cancel", rest: "" };
   }
   if (GOAL_ARG_SUBS.has(first)) {
-    return { kind: "sub", name: first as "tweak" | "archive", rest };
+    return { kind: "sub", name: first as "tweak" | "archive" | "start", rest };
   }
   return { kind: "set", text: trimmed };
 }
@@ -275,9 +275,29 @@ export function buildSeedGrillMessage(tmpl: string, seed: string, tool: string):
  * mechanism guarantees an interview HAPPENED; question quality is the
  * model's job (shaped by buildSeedGrillMessage).
  */
-export function draftProposalBlock(userReplies: number): string | null {
+export function draftProposalBlock(userReplies: number, blockedAttempts = 0): string | null {
   if (userReplies > 0) return null;
-  return "INTERVIEW FIRST — you have not received a single user reply since drafting started. Ask the user ONE sharp question about their objective (seed-specific, with a recommended default; challenge non-answers by offering concrete options), wait for the answer, and only then call the propose tool again. The Confirm dialog stays closed until the user has actually been heard.";
+  const base = "INTERVIEW FIRST — you have not received a single user reply since drafting started. Ask the user ONE sharp question about their objective (seed-specific, with a recommended default; challenge non-answers by offering concrete options), wait for the answer, and only then call the propose tool again. The Confirm dialog stays closed until the user has actually been heard.";
+  // v0.15.1 escape hatch: typed chat replies AND answered ask_user_question
+  // dialogs both count. If we have blocked 3+ proposals, the replies are
+  // arriving through a path this plugin cannot see — hand the user a manual
+  // unlock instead of manufacturing yet another interview round.
+  if (blockedAttempts >= 3) {
+    return base + " NOTE: proposals have been blocked repeatedly despite interviewing — the reply counter may not see your channel. Tell the user plainly: 'type any chat message (e.g. \"go on\") to unlock the Confirm dialog', wait for it, then propose again. Do NOT ask another interview question first.";
+  }
+  return base;
+}
+
+/**
+ * v0.15.1: an ask_user_question tool result counts as a user reply during
+ * drafting — dialog answers arrive as tool results, not chat messages.
+ * Answered = not cancelled (Esc) with at least one answer recorded.
+ */
+export function askUserQuestionAnswered(toolName: string, details: unknown): boolean {
+  if (toolName !== "ask_user_question") return false;
+  if (!details || typeof details !== "object") return false;
+  const d = details as { answers?: unknown; cancelled?: unknown };
+  return d.cancelled === false && Array.isArray(d.answers) && d.answers.length > 0;
 }
 
 /**

package/extensions/loops/goal.ts

@@ -39,6 +39,7 @@ import {
   takeAt,
   goalArgsNeedDrafting,
   buildSeedGrillMessage,
+  askUserQuestionAnswered,
   draftProposalBlock,
   type TaskProposal,
   validateTaskProposal,
@@ -110,6 +111,7 @@ let draftingTarget: "goal" | "list" | "loop" | null = null;
 // v0.14.0 drafting floor: user replies counted while drafting; the injected
 // seed prompt itself arrives as a user message — skip exactly that one.
 let draftingUserReplies = 0;
+let draftingBlockedProposals = 0; // v0.15.1: stuck-gate escape hatch
 let draftingSeedInFlight = false;
 
 // Dedup set for token accounting (agent_end may replay seen messages).
@@ -432,7 +434,7 @@ async function startDrafting(ctx: ExtensionContext, target: "goal" | "list" | "l
   const [file, label, tool] = prompts[target]!;
   ctx.ui.notify(
     seed
-      ? `${label}: the objective has no "Done when:" clause — the agent will grill you about it first (nothing activates until you confirm).`
+      ? `${label}: the objective has no "Done when:" clause — the agent will grill you about it first (nothing activates until you confirm). Skip the interview entirely: /goal start <objective>.`
       : `${label} started. The agent will grill until the contract is concrete, then ${tool} opens a Confirm dialog. No work begins before confirmation.`,
     "info",
   );
@@ -458,6 +460,7 @@ async function startDrafting(ctx: ExtensionContext, target: "goal" | "list" | "l
   try {
     extensionApi?.sendUserMessage(tmpl, { deliverAs: ctx.isIdle() ? "followUp" : "steer" });
     draftingUserReplies = 0;
+    draftingBlockedProposals = 0;
     draftingSeedInFlight = true; // our injected prompt also arrives as a user message — don't count it
   } catch {
     draftingTarget = null;
@@ -478,6 +481,16 @@ async function cmdGoal(args: string, ctx: ExtensionContext): Promise<void> {
     if (route.name === "cancel") return cmdCancel(ctx);
     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
+    // immediately, no interview, no "Done when:" heuristic. Symmetric
+    // with /loop start. The auditor infers the contract from the objective.
+    if (route.name === "start") {
+      if (!route.rest) {
+        ctx.ui.notify("Usage: /goal start <objective> — activates immediately, skipping the drafting interview. (Without start, an objective needs a 'Done when:' clause or it gets drafted first.)", "warning");
+        return;
+      }
+      return cmdSet(route.rest, ctx, true);
+    }
   }
   return cmdSet(route.kind === "set" ? route.text : "", ctx);
 }
@@ -486,7 +499,7 @@ async function cmdGoal(args: string, ctx: ExtensionContext): Promise<void> {
 // /goal: bypass drafting, start now (the only entry in v0.1.0)
 // =================================================================
 
-async function cmdSet(args: string, ctx: ExtensionContext): Promise<void> {
+async function cmdSet(args: string, ctx: ExtensionContext, skipDraft = false): Promise<void> {
   let raw = args.trim();
   // Users naturally quote the objective ("/goal \"do X\""); strip one layer of
   // surrounding matching quotes so they don't leak into the goal text.
@@ -504,7 +517,8 @@ async function cmdSet(args: string, ctx: ExtensionContext): Promise<void> {
   // v0.11.0: a contract-less objective gets drafted, not activated raw —
   // the pi-goal-x lesson: arg + Enter is worse than a 5-minute draft.
   // Include an explicit "Done when: …" clause to activate instantly.
-  if (goalArgsNeedDrafting(raw)) {
+  // v0.16.0: /goal start bypasses this by explicit user command.
+  if (!skipDraft && goalArgsNeedDrafting(raw)) {
     await startDrafting(ctx, "goal", raw);
     return;
   }
@@ -1420,7 +1434,9 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
         };
       }
       // v0.14.0: the interview floor — no Confirm until the user replied.
-      const block = draftProposalBlock(draftingUserReplies);
+      // v0.14.0: the interview floor — no Confirm until the user replied.
+      if (draftingUserReplies === 0) draftingBlockedProposals++;
+      const block = draftProposalBlock(draftingUserReplies, draftingBlockedProposals);
       if (block) {
         return { content: [{ type: "text", text: block }], details: {} };
       }
@@ -1525,7 +1541,8 @@ function registerAgentTools(pi: any, ctx: ExtensionContext): void {
         };
       }
       // v0.14.0: the interview floor — no Confirm until the user replied.
-      const loopBlock = draftProposalBlock(draftingUserReplies);
+      if (draftingUserReplies === 0) draftingBlockedProposals++;
+      const loopBlock = draftProposalBlock(draftingUserReplies, draftingBlockedProposals);
       if (loopBlock) {
         return { content: [{ type: "text", text: loopBlock }], details: {} };
       }
@@ -2143,7 +2160,7 @@ export default function (pi: ExtensionAPI): void {
   //   /loop  — the metric loop (draft|start|status|stop)
   //   /gla   — the settings UI (+ scriptable key=value)
   pi.registerCommand("goal", {
-    description: "Set/draft a goal, or /goal status|pause|resume|cancel|tweak <text>|archive. Objectives without a 'Done when:' clause are grilled into a contract first (nothing activates until you confirm); include the clause to start instantly.",
+    description: "Set/draft a goal, or /goal status|pause|resume|cancel|tweak <text>|archive|start <objective>. Objectives without a 'Done when:' clause are grilled into a contract first; include the clause or use /goal start to skip the interview and activate instantly.",
     handler: (args: string, ctx: ExtensionContext) => { rememberCtx(ctx); return cmdGoal(args, ctx); },
   });
   const settingsHandler = (args: string, ctx: ExtensionContext) => { rememberCtx(ctx); return cmdSettings(args, ctx); };
@@ -2188,6 +2205,15 @@ export default function (pi: ExtensionAPI): void {
     draftingUserReplies++;
   });
 
+  // v0.15.1: ask_user_question answers arrive as tool results, not chat
+  // messages — count answered (non-cancelled) questionnaires as replies too.
+  pi.on("tool_result", async (event: any) => {
+    if (draftingTarget === null) return;
+    if (askUserQuestionAnswered(String(event?.toolName ?? ""), event?.details)) {
+      draftingUserReplies++;
+    }
+  });
+
   pi.on("session_start", async (_event: any, ctx: ExtensionContext) => {
     rememberCtx(ctx);
     state = readState(ctx.cwd);

package/package.json

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