commandmate 0.44.1 → 0.44.3

package/dist/server/src/lib/detection/tools/frame.js

@@ -12,9 +12,13 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.STATUS_CHECK_LINE_COUNT = void 0;
 exports.normalizeFrame = normalizeFrame;
+exports.liveRegionOf = liveRegionOf;
+exports.withLiveRegion = withLiveRegion;
 const cli_patterns_1 = require("../cli-patterns");
 const tui_detection_frame_1 = require("../tui-detection-frame");
 const thinking_constants_1 = require("../../../config/thinking-constants");
+const live_region_specs_1 = require("./live-region-specs");
+const live_region_1 = require("./live-region");
 /**
  * Number of lines from the end to check for prompt and input indicators.
  *
@@ -23,8 +27,17 @@ const thinking_constants_1 = require("../../../config/thinking-constants");
  * existing importers are unaffected.
  */
 exports.STATUS_CHECK_LINE_COUNT = 15;
-/** Build the shared {@link NormalizedFrame} for one capture. */
-function normalizeFrame(output) {
+/**
+ * Build the shared {@link NormalizedFrame} for one capture.
+ *
+ * @param output - The capture. Pass it as captured (ANSI and box drawing
+ *   intact): the input-box markers of claude / Command Code / copilot / opencode
+ *   are rule rows that `stripBoxDrawing` blanks.
+ * @param tool - The CLI the capture came from. Names the live-region declaration
+ *   to apply (Issue #3183); omitted, the region is the whole frame — the reading
+ *   every rule had before.
+ */
+function normalizeFrame(output, tool) {
     const clean = (0, tui_detection_frame_1.normalizeTuiFrameForDetection)((0, cli_patterns_1.stripAnsi)(output));
     const lines = clean.split('\n');
     // Strip trailing empty lines (tmux terminal padding) before windowing.
@@ -40,8 +53,29 @@ function normalizeFrame(output) {
         clean,
         lines,
         contentLines,
+        liveRegion: (0, live_region_1.locateLiveRegion)(tool === undefined ? undefined : live_region_specs_1.LIVE_REGION_SPECS[tool], tool ?? null, {
+            raw: output,
+            contentLines,
+        }),
         lastLines: contentLines.slice(-exports.STATUS_CHECK_LINE_COUNT).join('\n'),
         // DR-003: Separate thinking detection window (5 lines) from prompt detection window (15 lines)
         thinkingLines: contentLines.slice(-thinking_constants_1.THINKING_TAIL_LINE_COUNT).join('\n'),
     };
 }
+/**
+ * The live region of `frame` as `tool` declares it (Issue #3183).
+ *
+ * `frame.liveRegion` when the frame was normalised for `tool`; recomputed
+ * otherwise — a caller that built the frame without naming the tool (older
+ * call sites, tests) still gets the tool's own reading, never a stale one.
+ */
+function liveRegionOf(frame, tool) {
+    if (frame.liveRegion.tool === tool)
+        return frame.liveRegion;
+    return (0, live_region_1.locateLiveRegion)(live_region_specs_1.LIVE_REGION_SPECS[tool], tool, { raw: frame.raw, contentLines: frame.contentLines });
+}
+/** `frame` with its live region located for `tool` (the same object when it already is). */
+function withLiveRegion(frame, tool) {
+    const liveRegion = liveRegionOf(frame, tool);
+    return liveRegion === frame.liveRegion ? frame : { ...frame, liveRegion };
+}

package/dist/server/src/lib/detection/tools/gemini/patterns.js

@@ -0,0 +1,29 @@
+"use strict";
+/**
+ * Gemini pattern constants moved out of `../../cli-patterns.ts`
+ * (Issue #3217). `cli-patterns.ts` re-exports every public name.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.GEMINI_THINKING_PATTERN = exports.GEMINI_PROMPT_PATTERN = void 0;
+/**
+ * Gemini interactive REPL prompt pattern
+ * Gemini CLI shows a `>` or `❯` prompt when waiting for user input in interactive mode.
+ *
+ * Two branches (Issue #386):
+ * - Branch 1: `^[>❯]\s*$` -- bare prompt character (empty input line)
+ * - Branch 2: `^\s*[>❯]\s+Type your message.*$` -- new-format prompt with placeholder text
+ *   (e.g., " >   Type your message or @path/to/file"). Leading whitespace is allowed
+ *   because tmux capture-pane output may include padding.
+ *
+ * Branch 2 requires "Type your message" after the indicator to avoid false positives
+ * on quoted response lines (e.g., "> some quoted text").
+ *
+ * @see CLAUDE_PROMPT_PATTERN for similar dual-format matching approach
+ */
+// [S4-5] /g flag prohibited: would make test() stateful
+exports.GEMINI_PROMPT_PATTERN = /^[>❯]\s*$|^\s*[>❯]\s+Type your message.*$/m;
+/**
+ * Gemini thinking/processing pattern
+ * Gemini CLI shows braille spinner characters and status text while processing.
+ */
+exports.GEMINI_THINKING_PATTERN = /[\u2800-\u28FF]|Thinking\.\.\./;

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

@@ -0,0 +1,31 @@
+"use strict";
+/**
+ * Every tool's live-region declaration, by tool (Issue #3183).
+ *
+ * A table of its own rather than a lookup through the detector registry:
+ * `normalizeFrame` reads it, and the detector modules import `normalizeFrame`,
+ * so going through `registry.ts` would close an import cycle. Each detector
+ * declares the same object as its `ToolDetectorSpec.liveRegion`, and
+ * `live-region-markers.test.ts` holds the two to identity.
+ *
+ * `gemini` and `vibe-local` have no measured frames and no declaration: their
+ * frames carry the whole-frame region, which is the pre-#3183 reading.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.LIVE_REGION_SPECS = void 0;
+const live_region_1 = require("./antigravity/live-region");
+const live_region_2 = require("./claude/live-region");
+const live_region_3 = require("./codex/live-region");
+const live_region_4 = require("./command-code/live-region");
+const live_region_5 = require("./copilot/live-region");
+const live_region_6 = require("./opencode/live-region");
+const live_region_7 = require("./opencode-v2/live-region");
+exports.LIVE_REGION_SPECS = {
+    claude: live_region_2.CLAUDE_LIVE_REGION,
+    codex: live_region_3.CODEX_LIVE_REGION,
+    antigravity: live_region_1.ANTIGRAVITY_LIVE_REGION,
+    'command-code': live_region_4.COMMAND_CODE_LIVE_REGION,
+    copilot: live_region_5.COPILOT_LIVE_REGION,
+    opencode: live_region_6.OPENCODE_LIVE_REGION,
+    'opencode-v2': live_region_7.OPENCODE_V2_LIVE_REGION,
+};

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

@@ -11,10 +11,12 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.opencodeStatusDetector = exports.VERIFIED_AGAINST = void 0;
 exports.readIdleEvidence = readIdleEvidence;
 const cli_patterns_1 = require("../../cli-patterns");
+const verdicts_1 = require("../verdicts");
 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 +83,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).
@@ -123,23 +127,11 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
         // `wait` still stops for it (`isSelectionListActive` → exit 10) and the UI
         // renders NavigationButtons, which send exactly the keys the strip takes.
         if (cli_patterns_1.OPENCODE_PERMISSION_PATTERN.test(frame.lastLines)) {
-            return {
-                status: 'waiting',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.OPENCODE_PERMISSION_PROMPT,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('waiting', status_reason_1.STATUS_REASON.OPENCODE_PERMISSION_PROMPT);
         }
         // A. Check footer for processing indicator ("esc interrupt" replaces "ctrl+t variants..." during processing)
         if (cli_patterns_1.OPENCODE_PROCESSING_INDICATOR.test(frame.lastLines)) {
-            return {
-                status: 'running',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.OPENCODE_PROCESSING_INDICATOR,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('running', status_reason_1.STATUS_REASON.OPENCODE_PROCESSING_INDICATOR);
         }
         const { candidates, lastIndex } = findContentEnd(frame);
         if (lastIndex < 0)
@@ -149,13 +141,7 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
             .slice(Math.max(0, lastIndex - thinking_constants_1.THINKING_TAIL_LINE_COUNT + 1), lastIndex + 1)
             .join('\n');
         if ((0, cli_patterns_1.detectThinking)('opencode', contentThinkingWindow)) {
-            return {
-                status: 'running',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.THINKING_INDICATOR,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('running', status_reason_1.STATUS_REASON.THINKING_INDICATOR);
         }
         // C. Check content area for selection list (Issue #473: fuzzy-search list detection)
         // Selection list header ("Select model"/"Select provider") may be far above the
@@ -164,13 +150,7 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
             .slice(Math.max(0, lastIndex - frame_1.STATUS_CHECK_LINE_COUNT + 1), lastIndex + 1)
             .join('\n');
         if (cli_patterns_1.OPENCODE_SELECTION_LIST_PATTERN.test(candidates.join('\n'))) {
-            return {
-                status: 'waiting',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.OPENCODE_SELECTION_LIST,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('waiting', status_reason_1.STATUS_REASON.OPENCODE_SELECTION_LIST);
         }
         // C2. A modal overlay painted over the transcript (Issue #2112).
         //
@@ -205,13 +185,7 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
         // production caller of `detectSessionStatus` passes the `capture-pane -e`
         // bytes (`lib/tmux/tmux.ts` always passes `-e`).
         if ((0, opencode_modal_overlay_1.detectOpenCodeModalOverlay)(frame.raw) !== null) {
-            return {
-                status: 'waiting',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.OPENCODE_MODAL_OVERLAY,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('waiting', status_reason_1.STATUS_REASON.OPENCODE_MODAL_OVERLAY);
         }
         // D. Check last few content lines for the finished-turn marker.
         //
@@ -224,13 +198,7 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
         // through to the heuristics rather than claiming a completion that never
         // happened, which is what design rule D1 asks for.
         if (cli_patterns_1.OPENCODE_TURN_COMPLETE_PATTERN.test(contentCheckWindow)) {
-            return {
-                status: 'ready',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.OPENCODE_RESPONSE_COMPLETE,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('ready', status_reason_1.STATUS_REASON.OPENCODE_RESPONSE_COMPLETE);
         }
         // E. Idle composer (Issue #473 found the row; Issue #1883 made it positive).
         //
@@ -254,13 +222,7 @@ exports.opencodeStatusDetector = (0, run_detection_1.createToolStatusDetector)({
             .slice(Math.max(0, lastIndex - frame_1.STATUS_CHECK_LINE_COUNT + 1), lastIndex + 1)
             .join('\n');
         if (cli_patterns_1.OPENCODE_IDLE_COMPOSER_PATTERN.test(composerWindow)) {
-            return {
-                status: 'ready',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.INPUT_PROMPT,
-                hasActivePrompt: false,
-                evidence: 'positive',
-            };
+            return (0, verdicts_1.positiveVerdict)('ready', status_reason_1.STATUS_REASON.INPUT_PROMPT);
         }
         return null;
     },

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,
+};