commandmate 0.44.1 → 0.44.2

package/dist/server/src/lib/detection/tools/live-region.js

@@ -0,0 +1,204 @@
+"use strict";
+/**
+ * The live region: "where on this frame does the part the operator can act on
+ * begin?" (Issue #3183, `docs/design/3183-live-region-extraction.md`).
+ *
+ * Before this module every tool answered that question on its own — codex by
+ * the bottom-most `›` row (#892) and again by its SGR (#2841), agy by a `>`
+ * under the list (#2845 / #2851), Command Code by a footer at the very bottom
+ * (#2846), claude by the input box (#2847) — and a fix to one tool reached no
+ * other. Here the answer is computed once per frame, by `normalizeFrame`, from
+ * each tool's {@link LiveRegionSpec}; the status chain and the Auto-Yes path
+ * then apply the SAME two rules to it:
+ *
+ *  - {@link isQuotedNumberedPrompt}: a numbered list on a frame whose input box
+ *    is the bottom of the pane is conversation text, not a dialog;
+ *  - {@link vetoesDialog}: likewise for a tool's own dialog reading.
+ *
+ * This file holds the shared parts the per-tool declarations are built from and
+ * the two rules. It must stay free of the detector modules (`<tool>/detect.ts`)
+ * so `normalizeFrame` can use it without an import cycle.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.wholeFrameRegion = wholeFrameRegion;
+exports.locateLiveRegion = locateLiveRegion;
+exports.isQuotedNumberedPrompt = isQuotedNumberedPrompt;
+exports.vetoesDialog = vetoesDialog;
+exports.fencedComposer = fencedComposer;
+exports.bottomMostComposerRow = bottomMostComposerRow;
+exports.dialogTopFrom = dialogTopFrom;
+exports.ruleAboveOptionRun = ruleAboveOptionRun;
+exports.lastContentRow = lastContentRow;
+/** The whole frame — the reading every rule had before Issue #3183. */
+function wholeFrameRegion(tool, contentLines) {
+    return {
+        tool,
+        anchor: 'none',
+        startRow: 0,
+        composerAtBottom: false,
+        composerHidesDialogs: false,
+        lines: contentLines,
+    };
+}
+/**
+ * Locate the live region of one frame from one tool's declaration.
+ *
+ * The composer wins over the dialog top: a tool that draws its input box is,
+ * by every measurement in the design doc §7, not showing a dialog above it.
+ */
+function locateLiveRegion(spec, tool, rows) {
+    const { contentLines } = rows;
+    if (spec === undefined)
+        return wholeFrameRegion(tool, contentLines);
+    const composer = spec.composer.locate(rows);
+    if (composer !== null) {
+        return {
+            tool,
+            anchor: 'composer',
+            startRow: composer.start,
+            composerEndRow: composer.end ?? composer.start,
+            composerAtBottom: composer.atBottom ?? true,
+            composerHidesDialogs: spec.composerHidesDialogs,
+            lines: contentLines.slice(composer.start),
+        };
+    }
+    const dialog = spec.dialogTop?.locate(rows) ?? null;
+    if (dialog !== null) {
+        return {
+            tool,
+            anchor: 'dialog',
+            startRow: dialog.start,
+            composerAtBottom: false,
+            composerHidesDialogs: spec.composerHidesDialogs,
+            lines: contentLines.slice(dialog.start),
+        };
+    }
+    return wholeFrameRegion(tool, contentLines);
+}
+/**
+ * Is the numbered list the generic parser read a quotation? (Issue #3183)
+ *
+ * True when it is a `multiple_choice` reading on a frame whose input box is the
+ * bottom of the pane, for a tool whose every answerable screen replaces the
+ * input box ({@link LiveRegion.composerHidesDialogs}): the list then sits in
+ * the conversation above it — an agent's reply quoting a dialog, or a dialog
+ * left in the scrollback.
+ *
+ * For copilot and the opencode family a composer at the bottom does not prove
+ * that much (their pickers are drawn over it), and their numbered candidates
+ * stay with their own dialog rules — the Auto-Yes gate, which both enforce, and
+ * opencode-v2's `requireVouchedPrompt`.
+ *
+ * The one rule both the status chain (`run-detection.ts`) and the Auto-Yes
+ * path (`response-checker.ts`'s `detectPromptOnCleanFrame`) apply.
+ */
+function isQuotedNumberedPrompt(region, prompt) {
+    return (prompt.isPrompt &&
+        prompt.promptData?.type === 'multiple_choice' &&
+        region.composerAtBottom &&
+        region.composerHidesDialogs);
+}
+/**
+ * Should a tool's own dialog reading be discarded because the input box is the
+ * bottom of the pane? (Issue #3183)
+ *
+ * Numbered dialogs always are: every tool measured removes its input box while
+ * one is up (design doc §7). Arrow-key surfaces are only for tools whose every
+ * screen replaces the input box ({@link LiveRegion.composerHidesDialogs}):
+ * copilot and opencode draw their pickers OVER a live composer, so there a
+ * composer at the bottom says nothing about a `keys` overlay.
+ */
+function vetoesDialog(region, verdict) {
+    if (verdict === null || !region.composerAtBottom)
+        return false;
+    return verdict.answerMode === 'numbered' || region.composerHidesDialogs;
+}
+// ---------------------------------------------------------------------------
+// Shared parts the per-tool declarations are built from.
+// ---------------------------------------------------------------------------
+/**
+ * A composer fenced by rules and pinned to the bottom (claude, Command Code,
+ * copilot): the tool's own structural finder, which returns the index of the
+ * block's first row or -1. The block runs to the last content row (the status
+ * bar under the fence is part of it), so it is the bottom by construction.
+ */
+function fencedComposer(find) {
+    return {
+        locate({ contentLines }) {
+            const start = find(contentLines);
+            if (start < 0)
+                return null;
+            return { start, end: lastContentRow(contentLines), atBottom: true };
+        },
+    };
+}
+/**
+ * The bottom-most row `isComposerRow` accepts (agy's bare `>`, codex's `›`).
+ * `atBottom` decides whether the rows below it still hold something live.
+ */
+function bottomMostComposerRow(options) {
+    return {
+        locate(rows) {
+            const { contentLines } = rows;
+            for (let i = contentLines.length - 1; i >= 0; i--) {
+                if (options.isComposerRow(contentLines[i])) {
+                    return { start: i, end: i, atBottom: options.atBottom(rows, i) };
+                }
+            }
+            return null;
+        },
+    };
+}
+/** A dialog top found by a tool's own function over `contentLines` (-1 when absent). */
+function dialogTopFrom(find) {
+    return {
+        locate({ contentLines }) {
+            const start = find(contentLines);
+            return start < 0 ? null : { start };
+        },
+    };
+}
+/** A numbered option row, cursor glyph optional (`❯ 1. Yes`, `  2. No`, `› 3. …`). */
+const OPTION_ONE_ROW_PATTERN = /^\s*(?:[❯›>●]\s*)?1[.)]\s+\S/;
+/**
+ * The top of a dialog drawn as: rule, title, …, `1.` option, …, footer
+ * (claude, Command Code). From the bottom-most footer, up to the first option,
+ * up to the nearest rule. Searching from the OPTION rather than the footer is
+ * what keeps an AskUserQuestion picker's second, inner rule (between its
+ * options and its footer) from being taken for the top.
+ */
+function ruleAboveOptionRun(options) {
+    return dialogTopFrom(lines => {
+        let footer = -1;
+        for (let i = lines.length - 1; i >= 0; i--) {
+            if (options.footer(lines[i])) {
+                footer = i;
+                break;
+            }
+        }
+        if (footer < 0)
+            return -1;
+        const floor = Math.max(0, footer - options.maxRows);
+        let optionOne = -1;
+        for (let i = footer - 1; i >= floor; i--) {
+            if (OPTION_ONE_ROW_PATTERN.test(lines[i])) {
+                optionOne = i;
+                break;
+            }
+        }
+        if (optionOne < 0)
+            return -1;
+        for (let i = optionOne - 1; i >= floor; i--) {
+            if (options.rule.test(lines[i].trimEnd()))
+                return i;
+        }
+        return -1;
+    });
+}
+/** The last non-blank row's index, or -1. */
+function lastContentRow(lines) {
+    let i = lines.length - 1;
+    while (i >= 0 && lines[i].trim() === '')
+        i--;
+    return i;
+}

package/dist/server/src/lib/detection/tools/opencode/detect.js

@@ -15,6 +15,7 @@ const status_reason_1 = require("../../status-reason");
 const opencode_modal_overlay_1 = require("../../opencode-modal-overlay");
 const prompt_1 = require("./prompt");
 const frame_1 = require("../frame");
+const live_region_1 = require("./live-region");
 const run_detection_1 = require("../run-detection");
 const verified_against_1 = require("../verified-against");
 const thinking_constants_1 = require("../../../../config/thinking-constants");
@@ -81,6 +82,8 @@ function readIdleEvidence(frame) {
 exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
     tool: 'opencode',
     verifiedAgainst: exports.VERIFIED_AGAINST,
+    // Issue #3183: where the live part of the frame begins (`./live-region.ts`).
+    liveRegion: live_region_1.OPENCODE_LIVE_REGION,
     afterThinking(frame) {
         // 2.5. OpenCode status detection (Issue #379)
         // OpenCode TUI layout: content area (top) | empty padding (~150 lines) | footer status bar (~6 lines at bottom).

package/dist/server/src/lib/detection/tools/opencode/live-region.js

@@ -0,0 +1,24 @@
+"use strict";
+/**
+ * opencode (v1)'s live region (Issue #3183; design doc §7 "opencode").
+ *
+ * - **composer** — the gutter block above `╹▀▀…` and the `ctrl+p` footer.
+ *   Measured: `opencode-agent-health-3021/quoted-dialog-reply-done.txt`
+ *   L192-L199 (1.18.33).
+ * - **dialog top** — the `┃  △ Permission required` row. Measured:
+ *   `opencode-live-1893/permission-bash.txt` L192 / L198 (1.18.21).
+ *
+ * `composerHidesDialogs: false`: opencode draws its pickers and the command
+ * palette over a live composer, so a composer at the bottom vetoes only the
+ * numbered readings.
+ *
+ * @module lib/detection/tools/opencode/live-region
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.OPENCODE_LIVE_REGION = void 0;
+const opencode_gutter_live_region_1 = require("../opencode-gutter-live-region");
+exports.OPENCODE_LIVE_REGION = {
+    composer: opencode_gutter_live_region_1.OPENCODE_GUTTER_COMPOSER,
+    dialogTop: (0, opencode_gutter_live_region_1.opencodeGutterDialogTop)(/△ Permission required/),
+    composerHidesDialogs: false,
+};

package/dist/server/src/lib/detection/tools/opencode-gutter-live-region.js

@@ -0,0 +1,72 @@
+"use strict";
+/**
+ * The live-region parts the two opencode generations share (Issue #3183;
+ * design doc §7 "opencode" / "opencode-v2").
+ *
+ * Both draw the composer as a run of `┃` gutter rows ending in the model bar,
+ * then a `╹▀▀…` rule, then a footer carrying `ctrl+p` (or v1's `ctrl+t`). Both
+ * REMOVE that footer and rule while an approval strip or a question form is up
+ * (`opencode-live-1893/permission-bash.txt`, `opencode-v2-dialogs-2984/
+ * permission.txt` / `question.txt` end on a `┃` row), so finding them is
+ * finding the composer.
+ *
+ * The user's own turns are drawn on the same gutter
+ * (`opencode-agent-health-3021/quoted-dialog-reply-done.txt` L17-L24 holds a
+ * quoted `Allow once   Allow always   Reject` strip inside one), which is why the
+ * composer is the gutter run directly above the rule and nothing higher.
+ *
+ * @module lib/detection/tools/opencode-gutter-live-region
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.OPENCODE_GUTTER_COMPOSER = void 0;
+exports.opencodeGutterDialogTop = opencodeGutterDialogTop;
+const live_region_1 = require("./live-region");
+/** How far above the last row the keybinding footer may sit (v1 1.18.33 wraps it over 3 rows). */
+const FOOTER_MAX_ROWS = 4;
+/** How far above the footer the `╹▀` rule may sit. */
+const RULE_MAX_ROWS = 3;
+/** How far above the bottom a dialog's title may sit. */
+const DIALOG_MAX_ROWS = 60;
+const FOOTER_PATTERN = /ctrl\+[tp]\b/;
+const RULE_PATTERN = /^\s*╹▀+/;
+const GUTTER_PATTERN = /^\s*[┃│]/;
+exports.OPENCODE_GUTTER_COMPOSER = {
+    locate({ contentLines }) {
+        const last = (0, live_region_1.lastContentRow)(contentLines);
+        let footer = -1;
+        for (let i = last; i >= Math.max(0, last - FOOTER_MAX_ROWS); i--) {
+            if (FOOTER_PATTERN.test(contentLines[i])) {
+                footer = i;
+                break;
+            }
+        }
+        if (footer < 0)
+            return null;
+        let rule = -1;
+        for (let i = footer - 1; i >= Math.max(0, footer - RULE_MAX_ROWS); i--) {
+            if (RULE_PATTERN.test(contentLines[i])) {
+                rule = i;
+                break;
+            }
+        }
+        if (rule < 0)
+            return null;
+        let start = rule;
+        while (start - 1 >= 0 && GUTTER_PATTERN.test(contentLines[start - 1]))
+            start--;
+        return { start, end: last, atBottom: true };
+    },
+};
+/** The bottom-most gutter row `title` matches, within the last rows of the pane. */
+function opencodeGutterDialogTop(title) {
+    return {
+        locate({ contentLines }) {
+            const last = (0, live_region_1.lastContentRow)(contentLines);
+            for (let i = last; i >= Math.max(0, last - DIALOG_MAX_ROWS); i--) {
+                if (GUTTER_PATTERN.test(contentLines[i]) && title.test(contentLines[i]))
+                    return { start: i };
+            }
+            return null;
+        },
+    };
+}

package/dist/server/src/lib/detection/tools/opencode-v2/detect.js

@@ -48,6 +48,7 @@ exports.opencodeV2StatusDetector = exports.OPENCODE_V2_PERMISSION_OPTIONS = expo
 exports.endsWithTurnComplete = endsWithTurnComplete;
 const cli_patterns_1 = require("../../cli-patterns");
 const status_reason_1 = require("../../status-reason");
+const live_region_1 = require("./live-region");
 const run_detection_1 = require("../run-detection");
 const verified_against_1 = require("../verified-against");
 exports.VERIFIED_AGAINST = verified_against_1.OPENCODE_V2_VERIFIED_AGAINST;
@@ -141,6 +142,8 @@ function endsWithTurnComplete(frame) {
 exports.opencodeV2StatusDetector = (0, run_detection_1.createToolStatusDetector)({
     tool: 'opencode-v2',
     verifiedAgainst: exports.VERIFIED_AGAINST,
+    // Issue #3183: where the live part of the frame begins (`./live-region.ts`).
+    liveRegion: live_region_1.OPENCODE_V2_LIVE_REGION,
     // Issue #2991: every v2 dialog is recognised by `beforePrompt` below, so a
     // numbered list that reaches the generic parser is text in a reply (the UAT
     // frame: `❯ 1. Yes / 2. No` quoted in the answer). Unvouched by

package/dist/server/src/lib/detection/tools/opencode-v2/live-region.js

@@ -0,0 +1,24 @@
+"use strict";
+/**
+ * opencode-v2's live region (Issue #3183; design doc §7 "opencode-v2").
+ *
+ * - **composer** — the gutter block above `╹▀▀…` and the `ctrl+p commands`
+ *   footer. Measured: `opencode-v2-dialogs-2984/quoted-dialog-reply.txt`
+ *   L194-L199 (2.0.18), under the quotation at L27-L29.
+ * - **dialog top** — the `┃  △ Permission required` row or the question form's
+ *   `┃  Questions` row. Measured: `opencode-v2-dialogs-2984/permission.txt`
+ *   L186, `question.txt` L190.
+ *
+ * `composerHidesDialogs: false`, as for v1: v2's dialog-title overlays
+ * (`commands.txt`) are drawn over a live composer.
+ *
+ * @module lib/detection/tools/opencode-v2/live-region
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.OPENCODE_V2_LIVE_REGION = void 0;
+const opencode_gutter_live_region_1 = require("../opencode-gutter-live-region");
+exports.OPENCODE_V2_LIVE_REGION = {
+    composer: opencode_gutter_live_region_1.OPENCODE_GUTTER_COMPOSER,
+    dialogTop: (0, opencode_gutter_live_region_1.opencodeGutterDialogTop)(/△ Permission required|┃\s*Questions\s*$/),
+    composerHidesDialogs: false,
+};

package/dist/server/src/lib/detection/tools/run-detection.js

@@ -32,6 +32,8 @@ exports.runToolDetection = runToolDetection;
 exports.createToolStatusDetector = createToolStatusDetector;
 const cli_patterns_1 = require("../cli-patterns");
 const prompt_detector_1 = require("../prompt-detector");
+const frame_1 = require("./frame");
+const live_region_1 = require("./live-region");
 const status_reason_1 = require("../status-reason");
 const detection_evidence_config_1 = require("../../../config/detection-evidence-config");
 const idle_evidence_observation_1 = require("../idle-evidence-observation");
@@ -77,7 +79,10 @@ function resolveIdleEvidence(spec, frame) {
     return mode === 'observe' ? 'positive' : evidence;
 }
 /** Run the whole chain for one tool over one frame. */
-function runToolDetection(spec, frame, context = {}) {
+function runToolDetection(spec, input, context = {}) {
+    // Issue #3183: every branch below reads the tool's own live region, whoever
+    // built the frame.
+    const frame = (0, frame_1.withLiveRegion)(input, spec.tool);
     // 0.x — the tool's own pre-prompt branches. A picker, a pager or a status bar
     // whose body the generic parser would misread as a numbered dialog.
     const early = spec.beforePrompt?.(frame) ?? null;
@@ -93,7 +98,14 @@ function runToolDetection(spec, frame, context = {}) {
         : (0, cli_patterns_1.stripBoxDrawing)(frame.lastLines);
     let promptDetection = (0, prompt_detector_1.detectPrompt)(promptInput, promptOptions);
     if (promptDetection.isPrompt) {
-        if (spec.isStalePrompt?.(frame, promptDetection)) {
+        if ((0, live_region_1.isQuotedNumberedPrompt)(frame.liveRegion, promptDetection)) {
+            // Issue #3183: the input box is the bottom of the pane, so the numbered
+            // list sits in the conversation above it — a reply quoting a dialog
+            // (#2845 / #2991) or a dialog left in the scrollback. The same rule the
+            // Auto-Yes path applies (`detectPromptOnCleanFrame`), on the same region.
+            promptDetection = { ...promptDetection, isPrompt: false, promptData: undefined };
+        }
+        else if (spec.isStalePrompt?.(frame, promptDetection)) {
             // Issue #1160: an ALREADY-ANSWERED block still inside the scan window.
             // Neutralise it so Auto-Yes and the sidebar never act on a dead prompt,
             // and let the chain continue to the tool's running/idle branches.
@@ -205,7 +217,17 @@ function createToolStatusDetector(spec) {
         // caller which of the two it got, because the Auto-Yes gate must suppress
         // for "this tool has rules and this frame is not a dialog" and must NOT
         // suppress for "nobody has measured this tool yet".
-        detectDialog: spec.detectDialog ?? (() => null),
+        //
+        // Issue #3183: wrapped so every tool's reading gets the live-region veto in
+        // one place — a dialog on a frame whose input box is the bottom of the pane
+        // is a quotation, whatever the tool's own rule matched.
+        detectDialog: spec.detectDialog
+            ? (input) => {
+                const frame = (0, frame_1.withLiveRegion)(input, spec.tool);
+                const verdict = spec.detectDialog(frame);
+                return (0, live_region_1.vetoesDialog)(frame.liveRegion, verdict) ? null : verdict;
+            }
+            : () => null,
         hasDialogRules: spec.detectDialog !== undefined,
     };
 }

package/dist/server/src/lib/polling/auto-yes-dialog-gate.js

@@ -61,6 +61,7 @@ exports.evaluateDialogPresence = evaluateDialogPresence;
 exports.judgePromptResponse = judgePromptResponse;
 exports.assessPromptAnswerability = assessPromptAnswerability;
 const frame_1 = require("../../lib/detection/tools/frame");
+const live_region_1 = require("../../lib/detection/tools/live-region");
 const registry_1 = require("../../lib/detection/tools/registry");
 const prompt_detector_1 = require("../../lib/detection/prompt-detector");
 const cli_patterns_1 = require("../../lib/detection/cli-patterns");
@@ -203,7 +204,9 @@ const GATED_PROMPT_TYPE = 'multiple_choice';
  *   `captureAndCleanOutput` produced: ANSI **and** box drawing already removed.
  *   Every `detectDialog` implementation is written against that spelling as well
  *   as the untouched one, so the same frame reaches the same verdict on both
- *   paths.
+ *   paths. Or (Issue #3183) the `NormalizedFrame` the poller built ONCE from
+ *   the raw capture for this tick — the same object its prompt reading used, so
+ *   both read one live region; that is what the poller passes.
  * @param env - Environment to read the kill switch from; injectable for tests
  */
 function evaluateAutoYesDialogGate(cliToolId, promptType, cleanOutput, env = process.env) {
@@ -237,7 +240,9 @@ function judgeDialogPresence(cliToolId, promptType, frame, mode) {
     if (promptType !== GATED_PROMPT_TYPE || !detector.hasDialogRules || mode === 'legacy') {
         return { present: true, dialog: null, mode, gated: false };
     }
-    const dialog = detector.detectDialog((0, frame_1.normalizeFrame)(frame));
+    // Issue #3183: normalised for the tool, so the detector reads the tool's live
+    // region (and its veto) — never a whole-frame default.
+    const dialog = detector.detectDialog(typeof frame === 'string' ? (0, frame_1.normalizeFrame)(frame, cliToolId) : frame);
     return { present: dialog !== null, dialog, mode, gated: true };
 }
 /**
@@ -388,10 +393,17 @@ function assessPromptAnswerability(cliToolId, frame) {
         };
     }
     const isCommandCodeQuestion = commandCodeQuestion.kind === 'prompt';
-    const promptCheck = (commandCodeQuestion.kind === 'prompt' ? commandCodeQuestion.prompt : null)
+    const candidate = (commandCodeQuestion.kind === 'prompt' ? commandCodeQuestion.prompt : null)
         ?? toolDialog
         ?? (0, prompt_detector_1.detectPrompt)(cleanOutput, (0, cli_patterns_1.buildDetectPromptOptions)(cliToolId));
-    const presence = evaluateDialogPresence(cliToolId, promptCheck.promptData?.type, frame);
+    // Issue #3183: one frame, normalised once from the capture, for the
+    // quotation veto and the presence gate alike — the reading the status chain
+    // and the Auto-Yes poller apply, so Send is offered exactly when they agree.
+    const normalized = (0, frame_1.normalizeFrame)(frame, cliToolId);
+    const promptCheck = (0, live_region_1.isQuotedNumberedPrompt)(normalized.liveRegion, candidate)
+        ? { isPrompt: false, cleanContent: cleanOutput }
+        : candidate;
+    const presence = evaluateDialogPresence(cliToolId, promptCheck.promptData?.type, normalized);
     return {
         promptCheck,
         isCommandCodeQuestion,

package/dist/server/src/lib/polling/response-checker.js

@@ -18,8 +18,8 @@ const db_1 = require("../../lib/db");
 const ws_server_1 = require("../../lib/ws-server");
 const prompt_detector_1 = require("../../lib/detection/prompt-detector");
 const dialog_1 = require("../../lib/detection/tools/antigravity/dialog");
-const detect_1 = require("../../lib/detection/tools/antigravity/detect");
 const frame_1 = require("../../lib/detection/tools/frame");
+const live_region_1 = require("../../lib/detection/tools/live-region");
 const dialog_2 = require("../../lib/detection/tools/command-code/dialog");
 const conversation_logger_1 = require("../../lib/conversation-logger");
 const types_1 = require("../../lib/cli-tools/types");
@@ -234,9 +234,14 @@ function buildPromptExtractionResult(lines, lastCapturedLine, totalLines, buffer
  *   then handed back as no prompt unless agy asked CommandMate about a tool call
  *   for that instance a moment ago. Omitted, the reading is not gated — which is
  *   what every caller that only DISPLAYS the frame wants.
+ * @param frame - the SAME tick's `normalizeFrame(raw, cliToolId)`, when the
+ *   caller built one (Issue #3183). The Auto-Yes poller builds it once and hands
+ *   the same object to the dialog gate, so the quotation veto below and the gate
+ *   read one live region. Omitted, it is built here from `rawFrame` (or, failing
+ *   that, `cleanOutput`).
  * @returns PromptDetectionResult with isPrompt, promptData, and cleanContent
  */
-function detectPromptOnCleanFrame(cleanOutput, cliToolId, precomputedLines, rawFrame, receiptScope) {
+function detectPromptOnCleanFrame(cleanOutput, cliToolId, precomputedLines, rawFrame, receiptScope, frame) {
     // Issue #2364: agy's `↑/↓ Navigate` dialogs are read by agy's own reader
     // before the generic pass, on the same spelling `tools/antigravity/detect.ts`
     // reads them on. The generic multiple-choice parser takes one row per option
@@ -287,25 +292,25 @@ function detectPromptOnCleanFrame(cleanOutput, cliToolId, precomputedLines, rawF
     }
     const promptOptions = (0, cli_patterns_1.buildDetectPromptOptions)(cliToolId);
     const result = (0, prompt_detector_1.detectPrompt)(cleanOutput, precomputedLines ? { ...promptOptions, precomputedLines } : promptOptions);
-    // Issue #2851: agy's reader above declines a dialog quoted in a reply (#2845),
-    // and this generic pass then read the quotation itself — `Do you want to
-    // proceed?` and its four options, above a live `>` composer — as an answerable
-    // `multiple_choice`. agy's Auto-Yes gate row is `legacy`, so nothing after this
-    // point judges the frame again and the poller answered a dialog nobody had
-    // opened. The status side declines the same candidate with the same predicate
-    // (`isStalePrompt`, #2845): agy draws its numbered screens in place of the
-    // composer or below it, never above it, so a composer under the list's last
-    // row makes the list a quotation.
+    // Issue #3183 (agy-only before, #2851): a numbered list on a frame whose
+    // input box is the bottom of the pane is a quotation — a reply quoting a
+    // dialog, or one left in the scrollback — and is not a prompt to answer. The
+    // status side declines the same candidate with the same rule on the same
+    // region (`run-detection.ts`), so `waiting` and an automatic answer cannot
+    // disagree about one frame. Every tool, not only agy: #2851 reached Auto-Yes
+    // because agy's `>` is not one of the glyphs the generic parser stops at, and
+    // opencode's gutter is not either.
     //
-    // `rawFrame` when the caller has it, and `cleanOutput` otherwise. The status
-    // side normalises the capture as captured and the predicate then strips box
-    // drawing exactly once; handing it `cleanOutput` — already stripped once, and
-    // `stripBoxDrawing` is not idempotent — would strip a second time and could
-    // read different rows.
-    if (cliToolId === 'antigravity' &&
-        result.isPrompt &&
-        (0, detect_1.isAntigravityQuotedNumberedList)((0, frame_1.normalizeFrame)(rawFrame ?? cleanOutput), result)) {
-        return { isPrompt: false, cleanContent: cleanOutput.trim() };
+    // `rawFrame` when there is no prebuilt frame: the input-box markers are rule
+    // rows that the cleaned spelling has blanked. `stripBoxDrawing` is not
+    // idempotent, so the cleaned string is never cleaned again here.
+    if (result.isPrompt) {
+        const liveRegion = frame !== undefined
+            ? (0, frame_1.liveRegionOf)(frame, cliToolId)
+            : (0, frame_1.normalizeFrame)(rawFrame ?? cleanOutput, cliToolId).liveRegion;
+        if ((0, live_region_1.isQuotedNumberedPrompt)(liveRegion, result)) {
+            return { isPrompt: false, cleanContent: cleanOutput.trim() };
+        }
     }
     // Issue #2849: the same withholding on this exit. agy's reader answering `null`
     // does not mean the frame is no dialog — the generic pass above reads shapes

package/dist/server/src/lib/prompt-answer-sender.js

@@ -294,7 +294,7 @@ async function assertAnswerModeAcceptsNumber(params) {
     const frame = await readGuardFrame(params);
     if (frame === null)
         return null;
-    const dialog = detector.detectDialog((0, frame_1.normalizeFrame)(frame));
+    const dialog = detector.detectDialog((0, frame_1.normalizeFrame)(frame, params.cliToolId));
     if (dialog === null)
         return null;
     if (dialog.answerMode === TEXT_ANSWERABLE_ANSWER_MODE)

package/dist/server/src/lib/realtime/terminal-broadcast.js

@@ -110,6 +110,9 @@ function emitTerminalSnapshot(worktreeId, cliToolId, payload, instanceId) {
         // `JSON.stringify` (in `broadcast`) already drops from the wire on its own.
         // Defaulting it would turn "not judged" into a judgement.
         promptAnswerable: payload.promptAnswerable,
+        // Issue #3184: derived from `promptData` by the builder; `?? null` like
+        // `promptData` itself, since both say "no prompt" the same way.
+        promptView: payload.promptView ?? null,
         isSelectionListActive: payload.isSelectionListActive ?? false,
         isPagerActive: payload.isPagerActive ?? false,
         isDismissablePanelActive: payload.isDismissablePanelActive ?? false,

package/dist/server/src/lib/relay/relay-delivery.js

@@ -222,6 +222,8 @@ function findWorkerReply(db, worker, since) {
 /** The options of B's open dialog, in the shape the notice lists them. */
 function promptOptionsOf(message) {
     const data = message.promptData;
+    // #3184: type narrowing only — the stored row's screen `options` are listed;
+    // a stored degraded record never carries a decision id, so it has none.
     if (!(0, models_1.isAnswerablePromptData)(data))
         return [];
     if (data.type === 'yes_no') {
@@ -383,6 +385,7 @@ function notifyRelayPromptWaiting(worker) {
             return;
         const locale = (0, model_change_push_notifier_1.resolveReadersLocale)(db);
         const options = promptOptionsOf(prompt);
+        // #3184: type narrowing only, to read `question` off the closed union.
         const question = (0, models_1.isAnswerablePromptData)(prompt.promptData)
             ? prompt.promptData.question
             : (prompt.promptData?.question ?? prompt.content);

package/dist/server/src/lib/relay/relay-triggers.js

@@ -75,6 +75,8 @@ exports.areRelayTriggersActive = areRelayTriggersActive;
 const waiting_episode_state_1 = require("../../lib/session/waiting-episode-state");
 const relay_policy_1 = require("../../lib/relay/relay-policy");
 const logger_1 = require("../../lib/logger");
+const prompt_waiting_composition_1 = require("../../lib/session/prompt-waiting-composition");
+const registry_1 = require("../../lib/hooks/sources/registry");
 const logger = (0, logger_1.createLogger)('relay-triggers');
 /**
  * Tell the relays that this instance finished a turn.
@@ -151,6 +153,13 @@ function startRelayTriggers() {
         // another session as a question it might report to a human.
         if (!transition.waiting || transition.kind !== 'prompt')
             return;
+        // Issue #3184 (design §6-2): a wait answered over the agent's API is
+        // published as `prompt` for the dots and the push body, but the relay is
+        // not raised for it — its notice lists the stored row's screen options,
+        // and such a wait has none to list. Relaying it is a separate Issue.
+        if ((0, prompt_waiting_composition_1.isApiAnswerableWaitNow)(transition.worktreeId, transition.cliToolId, transition.instanceId, (0, registry_1.getAgentEventSource)(transition.cliToolId).capabilities.eventIdentity)) {
+            return;
+        }
         onRelayPromptWaiting(transition.worktreeId, transition.cliToolId, transition.instanceId);
     });
     const timer = setInterval(() => void runRelayMaintenance(), relay_policy_1.RELAY_PUMP_INTERVAL_MS);

package/dist/server/src/lib/resource-cleanup.js

@@ -22,6 +22,7 @@ exports.stopResourceCleanup = stopResourceCleanup;
 const child_process_1 = require("child_process");
 const fs_1 = require("fs");
 const auto_yes_manager_1 = require("./polling/auto-yes-manager");
+const auto_yes_lifecycle_1 = require("./auto-yes-lifecycle");
 const path_validator_1 = require("./security/path-validator");
 const schedule_manager_1 = require("./schedule-manager");
 const timer_manager_1 = require("./timer-manager");
@@ -174,18 +175,19 @@ function cleanupOrphanedMapEntries() {
         // If DB query returned nothing, skip cleanup to avoid false positives
         return result;
     }
-    // Cleanup autoYesStates (Issue #525: composite keys)
+    // Cleanup autoYesStates (Issue #525: composite keys). Issue #3184: through the
+    // `orphan-swept` row of the Auto-Yes lifecycle table.
     const autoYesStateKeys = (0, auto_yes_manager_1.getAutoYesStateCompositeKeys)();
     for (const compositeKey of autoYesStateKeys) {
         const worktreeId = (0, auto_yes_manager_1.extractWorktreeId)(compositeKey);
         // [SEC4-MF-001] Validate extracted worktreeId
         if (!(0, path_validator_1.isValidWorktreeId)(worktreeId)) {
-            (0, auto_yes_manager_1.deleteAutoYesState)(compositeKey);
+            (0, auto_yes_lifecycle_1.applyAutoYesStateRule)('orphan-swept', { scope: 'key', compositeKey });
             result.deletedAutoYesStateIds.push(compositeKey);
             continue;
         }
         if (!validWorktreeIds.has(worktreeId)) {
-            (0, auto_yes_manager_1.deleteAutoYesState)(compositeKey);
+            (0, auto_yes_lifecycle_1.applyAutoYesStateRule)('orphan-swept', { scope: 'key', compositeKey });
             result.deletedAutoYesStateIds.push(compositeKey);
         }
     }
@@ -194,7 +196,7 @@ function cleanupOrphanedMapEntries() {
     for (const compositeKey of autoYesPollerKeys) {
         const worktreeId = (0, auto_yes_manager_1.extractWorktreeId)(compositeKey);
         if (!(0, path_validator_1.isValidWorktreeId)(worktreeId) || !validWorktreeIds.has(worktreeId)) {
-            (0, auto_yes_manager_1.stopAutoYesPolling)(compositeKey);
+            (0, auto_yes_lifecycle_1.stopAutoYesPollersFor)('orphan-swept', { scope: 'key', compositeKey });
             result.deletedAutoYesPollerIds.push(compositeKey);
         }
     }