commandmate 0.47.0 → 0.48.0

package/dist/server/src/lib/detection/prompt-detect-multiple-choice.js

@@ -1,19 +1,25 @@
 "use strict";
 /** Multiple choice prompt detection logic extracted from prompt-detector.ts (Issue #575). */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.APPROVAL_TARGET_MAX_LOOKBACK = void 0;
+exports.joinApprovalTarget = exports.findApprovalContextStart = exports.findApprovalContextRange = exports.APPROVAL_TARGET_MAX_LOOKBACK = void 0;
 exports.findClaudeTaskPanelLines = findClaudeTaskPanelLines;
 exports.findClaudeChrome = findClaudeChrome;
 exports.maskClaudeChrome = maskClaudeChrome;
 exports.isTypedTextFieldOption = isTypedTextFieldOption;
-exports.findApprovalContextStart = findApprovalContextStart;
-exports.joinApprovalTarget = joinApprovalTarget;
 exports.buildMultipleChoiceResult = buildMultipleChoiceResult;
 exports.detectMultipleChoicePrompt = detectMultipleChoicePrompt;
 const tui_detection_frame_1 = require("./tui-detection-frame");
 const cli_patterns_1 = require("./cli-patterns");
+const patterns_1 = require("./tools/claude/patterns");
 const prompt_view_1 = require("../../lib/session/prompt-view");
+const option_recommendation_1 = require("./option-recommendation");
 const picker_chrome_1 = require("./tools/claude/picker-chrome");
+const prompt_detect_multiple_choice_approval_context_1 = require("./prompt-detect-multiple-choice-approval-context");
+var prompt_detect_multiple_choice_approval_context_2 = require("./prompt-detect-multiple-choice-approval-context");
+Object.defineProperty(exports, "APPROVAL_TARGET_MAX_LOOKBACK", { enumerable: true, get: function () { return prompt_detect_multiple_choice_approval_context_2.APPROVAL_TARGET_MAX_LOOKBACK; } });
+Object.defineProperty(exports, "findApprovalContextRange", { enumerable: true, get: function () { return prompt_detect_multiple_choice_approval_context_2.findApprovalContextRange; } });
+Object.defineProperty(exports, "findApprovalContextStart", { enumerable: true, get: function () { return prompt_detect_multiple_choice_approval_context_2.findApprovalContextStart; } });
+Object.defineProperty(exports, "joinApprovalTarget", { enumerable: true, get: function () { return prompt_detect_multiple_choice_approval_context_2.joinApprovalTarget; } });
 // ============================================================================
 // Constants
 // ============================================================================
@@ -220,13 +226,6 @@ function maskClaudeChrome(lines, chrome) {
  * Anchored, no nested quantifiers — ReDoS safe (S4-001).
  */
 const PANE_WRAPPED_LABEL_PATTERN = /^\s{3,}\S/;
-/**
- * Pattern for separator lines (horizontal rules).
- * Matches lines consisting only of dash (-) or em-dash (─) characters.
- * Used to skip separator lines in question extraction and non-option line handling.
- * Anchored at both ends -- ReDoS safe (S4-001).
- */
-const SEPARATOR_LINE_PATTERN = /^[-─]+$/;
 /**
  * How many lines above `questionEndIndex` the question text is collected from.
  * Also the point the Issue #1699 approval-context scan starts walking up from,
@@ -552,7 +551,7 @@ function findQuestionLineInRange(lines, startIndex, scanRange, lowerBound) {
     for (let i = startIndex - 1; i >= scanLimit; i--) {
         const candidateLine = lines[i]?.trim() ?? '';
         // Skip empty lines and separator lines (horizontal rules)
-        if (!candidateLine || SEPARATOR_LINE_PATTERN.test(candidateLine))
+        if (!candidateLine || prompt_detect_multiple_choice_approval_context_1.SEPARATOR_LINE_PATTERN.test(candidateLine))
             continue;
         if (isQuestionLikeLine(candidateLine)) {
             return true;
@@ -649,26 +648,86 @@ function isContinuationLine(rawLine, line) {
     }
     return false;
 }
+/**
+ * The user's own message echo (Issue #3832): claude's `❯ msg`, codex's `› msg`.
+ * The cursor of an option row (`❯ 1.`) is not one. `>` is left out on purpose:
+ * codex's trust dialog opens with `> You are in <dir>` and a here-document's
+ * body carries `> ` rows, so a `>` row may be the prompt's own.
+ */
+const USER_ECHO_PATTERN = /^[❯›]\s(?!\s*\d+[.)]\s)/;
+/**
+ * codex's end-of-turn row (Issue #3832): `Worked for 17s • 19:30`, or the rule
+ * form `─ Worked for 1m 2s ─`. claude's is {@link CLAUDE_TURN_COMPLETE_PATTERN}.
+ */
+const CODEX_TURN_COMPLETE_PATTERN = /^─*\s*Worked for\s+(?:\d+h\s*)?(?:\d+m\s*)?\d+s\b/;
+/** True when the nearest non-blank row above `row` (down to `floor`) ends a finished turn. */
+function followsFinishedTurn(lines, row, floor) {
+    for (let i = row - 1; i >= floor; i--) {
+        const line = lines[i];
+        if (line === undefined || line.trim() === '')
+            continue;
+        return patterns_1.CLAUDE_TURN_COMPLETE_PATTERN.test(line) || CODEX_TURN_COMPLETE_PATTERN.test(line.trim());
+    }
+    return false;
+}
+/**
+ * The lowest row in `[from, questionEndIndex)` that is the user's message echo
+ * (Issue #3832), or -1.
+ *
+ * A `❯ ` / `› ` row is taken for the echo only when the screen leaves no doubt:
+ * drawn left of the dialog's frame (claude's echo at column 0 against its frame
+ * at 1, codex's at 0 against 2), and right under the previous turn's end-of-turn
+ * row (`✻ Baked for 3s`, `Worked for 17s`). A body row that happens to start
+ * with the glyph — at the frame's column, or under the command it belongs to —
+ * is not one, so the rows above it stay in `question` and the deny surface is
+ * taken wide.
+ */
+function findUserEchoRow(lines, questionEndIndex, from, panelEnd, floor) {
+    const frameIndent = (0, prompt_detect_multiple_choice_approval_context_1.panelFrameIndent)(lines, questionEndIndex, panelEnd);
+    for (let i = questionEndIndex - 1; i >= from; i--) {
+        const line = lines[i];
+        if (line === undefined || line.trim() === '')
+            continue;
+        if ((0, prompt_detect_multiple_choice_approval_context_1.indentOf)(line) < frameIndent && USER_ECHO_PATTERN.test(line.trim())
+            && followsFinishedTurn(lines, i, floor)) {
+            return i;
+        }
+    }
+    return -1;
+}
 /**
  * Extract question text from the lines around questionEndIndex.
  * Collects non-empty, non-separator lines from up to 5 lines before questionEndIndex
  * through questionEndIndex itself, joining them with spaces.
  *
+ * [Issue #3832] Not above the user's message echo ({@link findUserEchoRow}):
+ * that message opened the current turn, so the rows above it — the previous
+ * turn's `⏺ IMPL_COMPLETED` / `✻ Cogitated …`, codex's `• HTTP/2 200` /
+ * `Worked for 17s` on the measured screens — are not the prompt's and used to
+ * reach the deny surface through `question`. The echo row itself stays. No
+ * other row bounds the window: a `⏺` / `●` / `•` row may be the call the prompt
+ * asks about (codex's `• Calling …`, agy's `● Bash(…)`), and `question` is what
+ * keeps such a row in the deny surface when `approvalTarget` cuts at it.
+ *
  * @param lines - Array of output lines
  * @param questionEndIndex - Index of the last line before options, or -1 if not found
  * @param floor - First row the question may start at. Issue #2486: one below an
  *   AskUserQuestion picker's tab row, so neither that row nor the transcript
  *   above it becomes part of the question; 0 when there is none.
+ * @param panelEnd - Exclusive end of the dialog's own rows, which set the frame
+ *   column the echo is drawn at or left of (Issue #3832)
  * @returns Extracted question text, or generic fallback if questionEndIndex is -1
  */
-function extractQuestionText(lines, questionEndIndex, floor = 0) {
+function extractQuestionText(lines, questionEndIndex, floor = 0, panelEnd = questionEndIndex + 1) {
     if (questionEndIndex < 0) {
         return 'Please select an option:';
     }
+    const windowStart = Math.max(floor, questionEndIndex - QUESTION_CONTEXT_LINES);
+    const start = Math.max(windowStart, findUserEchoRow(lines, questionEndIndex, windowStart, panelEnd, floor));
     const questionLines = [];
-    for (let i = Math.max(floor, questionEndIndex - QUESTION_CONTEXT_LINES); i <= questionEndIndex; i++) {
+    for (let i = start; i <= questionEndIndex; i++) {
         const line = lines[i].trim();
-        if (line && !SEPARATOR_LINE_PATTERN.test(line)) {
+        if (line && !prompt_detect_multiple_choice_approval_context_1.SEPARATOR_LINE_PATTERN.test(line)) {
             questionLines.push(line);
         }
     }
@@ -694,79 +753,6 @@ function extractInstructionText(lines, questionEndIndex, effectiveEnd) {
     const joined = blockLines.join('\n').trim();
     return joined.length > 0 ? joined : undefined;
 }
-/**
- * Lines that mean "the previous turn ended here" when scanning upward.
- *
- * Every supported CLI renders its transcript as marker-prefixed rows — Claude's
- * tool-use `⏺` and result `⎿`, Codex's activity `•`, its approval receipts `✔`
- * / `✗`, the user's own `›` / `❯` / `>` echo, and its output continuation `└`.
- * The lines a prompt panel puts above its question (a header such as "Bash
- * command", the command itself, a one-line description) never carry one, which
- * is what makes this usable as the panel's upper edge after `stripBoxDrawing`
- * has erased the real box border.
- *
- * Horizontal rules count too: Codex draws one between turns.
- *
- * Anchored at the start of the trimmed line — ReDoS safe (no repetition).
- */
-const TURN_BOUNDARY_PATTERN = /^[⏺⎿●○•✔✓✖✗✘›»❯└├>]/;
-/**
- * How far above the question the approval context may reach (Issue #1699).
- *
- * The boundary marker above is the actual fix; this cap only exists so a frame
- * that happens to contain no marker at all (a bare shell `(y/n)` prompt under
- * plain command output) cannot drag an unbounded scrollback window into the
- * deny surface the way the old fixed 19-line lookback did. Sized for the
- * tallest real permission panel measured: header + blank + a wrapped multi-line
- * command + description + blank.
- */
-exports.APPROVAL_TARGET_MAX_LOOKBACK = 12;
-/** True when this line belongs to a previous turn rather than the prompt panel. */
-function isTurnBoundaryLine(rawLine) {
-    if (rawLine === undefined)
-        return false;
-    const line = rawLine.trim();
-    if (line === '')
-        return false;
-    return TURN_BOUNDARY_PATTERN.test(line) || SEPARATOR_LINE_PATTERN.test(line);
-}
-/**
- * First line index that still belongs to *this* prompt, scanning upward.
- *
- * Walks up from `questionLineIndex - 1` and stops just below the nearest turn
- * boundary; with no boundary in reach it falls back to the lookback cap.
- * Everything above the returned index is a previous turn and must never reach a
- * deny pattern — that poisoning is Issue #1699.
- *
- * Shared by both detector paths: multiple_choice passes its `questionEndIndex`,
- * yes_no the line its pattern matched on.
- *
- * @param lines - Full frame, already ANSI/box stripped
- * @param questionLineIndex - Line the prompt's own question sits on
- * @returns Index of the first line of the current prompt's own block
- */
-function findApprovalContextStart(lines, questionLineIndex) {
-    const floor = Math.max(0, questionLineIndex - exports.APPROVAL_TARGET_MAX_LOOKBACK);
-    for (let i = questionLineIndex - 1; i >= floor; i--) {
-        if (isTurnBoundaryLine(lines[i])) {
-            return i + 1;
-        }
-    }
-    return floor;
-}
-/**
- * Join a slice of the frame into an approval-target string.
- *
- * @returns The trimmed block, or undefined when it is empty
- */
-function joinApprovalTarget(lines, start, end) {
-    const joined = lines
-        .slice(start, end)
-        .map(l => l.trimEnd())
-        .join('\n')
-        .trim();
-    return joined.length > 0 ? joined : undefined;
-}
 /**
  * Extract what this multiple-choice prompt is asking approval for (Issue #1699).
  *
@@ -782,8 +768,9 @@ function joinApprovalTarget(lines, start, end) {
  *
  * Residual, deliberately left alone: `question` itself is built from up to
  * {@link QUESTION_CONTEXT_LINES} lines above `questionEndIndex` and is also part
- * of the deny surface. Narrowing that would change prompt dedup keys and what
- * the UI renders, and it is bounded by the options below it either way.
+ * of the deny surface. Issue #3832 only stops it at the user's message echo
+ * (see {@link extractQuestionText}); a previous turn below no echo still
+ * reaches it, which over-suppresses rather than under-suppresses.
  *
  * [Issue #2486] One exception to both: neither reaches above an AskUserQuestion
  * picker's tab row. Everything up there is the transcript — the Bash call Claude
@@ -800,7 +787,19 @@ function extractApprovalTarget(lines, questionEndIndex, effectiveEnd, floor = 0)
     if (questionEndIndex < 0) {
         return undefined;
     }
-    return joinApprovalTarget(lines, Math.max(floor, findApprovalContextStart(lines, questionEndIndex)), effectiveEnd);
+    const range = (0, prompt_detect_multiple_choice_approval_context_1.findApprovalContextRange)(lines, questionEndIndex, effectiveEnd);
+    // Issue #3881: the picker's tab row is a boundary of its own, so a scan it
+    // cuts short is complete whatever the scan said about the rows above.
+    const bounded = floor > 0 && floor >= range.start;
+    const start = Math.max(floor, range.start);
+    return {
+        approvalTarget: (0, prompt_detect_multiple_choice_approval_context_1.joinApprovalTarget)(lines, start, effectiveEnd),
+        approvalContext: {
+            startRow: start,
+            endRow: effectiveEnd,
+            completeness: bounded ? 'complete' : range.completeness,
+        },
+    };
 }
 /**
  * Build the final PromptDetectionResult for a multiple choice prompt.
@@ -819,7 +818,7 @@ function extractApprovalTarget(lines, questionEndIndex, effectiveEnd, floor = 0)
  * @returns PromptDetectionResult with isPrompt: true and multiple_choice data
  */
 function buildMultipleChoiceResult(question, collectedOptions, context, output, truncateRawContentFn, submitMode, isAskUserQuestion) {
-    const { instructionText, approvalTarget } = context ?? {};
+    const { instructionText, approvalTarget, approvalContext } = context ?? {};
     return {
         isPrompt: true,
         promptData: {
@@ -837,11 +836,16 @@ function buildMultipleChoiceResult(question, collectedOptions, context, output,
             status: 'pending',
             instructionText,
             ...(approvalTarget !== undefined ? { approvalTarget } : {}),
+            // Issue #3881 (H-3): only the case the resolver must act on.
+            ...(approvalTarget !== undefined && approvalContext?.completeness === 'truncated'
+                ? { approvalTargetCompleteness: 'truncated' }
+                : {}),
             ...(submitMode ? { submitMode } : {}),
             ...(isAskUserQuestion ? { isAskUserQuestion: true } : {}),
         },
         cleanContent: question.trim(),
         rawContent: truncateRawContentFn(output.trim()), // Issue #235: complete prompt output (truncated) [MF-001]
+        ...(approvalContext !== undefined ? { approvalContext } : {}),
     };
 }
 // ============================================================================
@@ -893,6 +897,61 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
     // C-003: Use ?? true for readability instead of !== false double negation
     const requireDefault = options?.requireDefaultIndicator ?? true;
     const lines = output.split('\n');
+    const { scanStart, effectiveEnd, hasAskUserQuestionFooter } = findScanWindow(lines);
+    const scanWindow = lines.slice(scanStart, effectiveEnd);
+    if (hasModelOverlayFooter(scanWindow)) {
+        return noPromptResult(output);
+    }
+    const { hasConfirmationFooter, hasNumberFooter } = findConfirmationFooter(scanWindow);
+    // ==========================================================================
+    // Pass 1: Check for ❯ indicator existence in scan window
+    // If no ❯ lines found and requireDefault is true, there is no multiple_choice prompt.
+    // When requireDefault is false, skip this gate entirely to allow ❯-less detection.
+    // ==========================================================================
+    if (requireDefault) {
+        const hasDefaultLine = hasDefaultOptionLine(scanWindow);
+        if (!hasDefaultLine && !hasConfirmationFooter) {
+            return noPromptResult(output);
+        }
+    }
+    const scan = collectOptions(lines, scanStart, effectiveEnd);
+    if (scan === null) {
+        return noPromptResult(output);
+    }
+    const { collectedOptions, optionRows, questionEndIndex } = scan;
+    if (!passesOptionValidation(collectedOptions, requireDefault, hasConfirmationFooter, questionEndIndex, lines, scanStart)) {
+        return noPromptResult(output);
+    }
+    // [Issue #2486] An AskUserQuestion picker's tab row is the dialog's top edge:
+    // the row is chrome and everything above it is transcript. Found once so
+    // `question` and `approvalTarget` start at the same row; 0 on a frame with no
+    // tab row, which leaves every other prompt exactly as it was. `instructionText`
+    // keeps its wide window — it is the human's view, and on a preview picker the
+    // pane it carries is what the question asks the human to look at.
+    const pickerTop = questionEndIndex < 0
+        ? 0
+        : (0, picker_chrome_1.findAskUserQuestionTabRow)(lines, questionEndIndex, prompt_detect_multiple_choice_approval_context_1.APPROVAL_TARGET_MAX_LOOKBACK) + 1;
+    const question = extractQuestionText(lines, questionEndIndex, pickerTop, effectiveEnd);
+    const instructionText = extractInstructionText(lines, questionEndIndex, effectiveEnd);
+    // Issue #1699: the same block, cut at the previous turn's boundary. Display
+    // keeps the wide window; deny patterns are judged against this one.
+    const approvalBlock = extractApprovalTarget(lines, questionEndIndex, effectiveEnd, pickerTop);
+    // Issue #616: Determine submitMode from confirmation footer (detected in single-pass above)
+    const submitMode = hasConfirmationFooter
+        ? (hasNumberFooter ? 'answer_only' : 'answer_then_enter')
+        : undefined;
+    const result = buildMultipleChoiceResult(question, collectedOptions, { instructionText, ...approvalBlock }, output, truncateRawContentFn, submitMode, hasAskUserQuestionFooter);
+    // [Issue #3863] Only the AskUserQuestion picker: the one this parser reads
+    // as a question, whose rows are a title and an indented description.
+    if (!hasAskUserQuestionFooter)
+        return result;
+    const paneColumns = (0, option_recommendation_1.lowerBoundPaneColumns)(lines);
+    return {
+        ...result,
+        optionRecommendations: collectedOptions.map((option, index) => readPickerOptionRecommendation(option.label, optionRows[index], paneColumns)),
+    };
+}
+function findScanWindow(lines) {
     // Strip trailing empty lines (tmux terminal padding) before computing scan window.
     // tmux buffers often end with many empty padding lines that would shift the
     // scan window away from the actual prompt content.
@@ -926,7 +985,9 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
             break;
         }
     }
-    const scanWindow = lines.slice(scanStart, effectiveEnd);
+    return { scanStart, effectiveEnd, hasAskUserQuestionFooter };
+}
+function hasModelOverlayFooter(scanWindow) {
     // [Issue #1495] Claude `/model` local-settings overlay guard.
     // The `/model` overlay renders a ❯-marked numbered model list ("1. Default …
     // 5. Haiku") under a "Select model" header, which the option-collection logic
@@ -942,9 +1003,12 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
     // selection list via CLAUDE_SELECTION_LIST_FOOTER (NavigationButtons + ESC hatch).
     for (const rawLine of scanWindow) {
         if (cli_patterns_1.CLAUDE_MODEL_OVERLAY_FOOTER_PATTERN.test(rawLine)) {
-            return noPromptResult(output);
+            return true;
         }
     }
+    return false;
+}
+function findConfirmationFooter(scanWindow) {
     // Single-pass footer detection: identify both confirmation footer presence and
     // specific "press number to confirm" variant in one iteration (Issue #616).
     let hasConfirmationFooter = false;
@@ -959,31 +1023,36 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
             break; // Both patterns are subsets of CONFIRMATION_FOOTER; one match suffices
         }
     }
-    // ==========================================================================
-    // Pass 1: Check for ❯ indicator existence in scan window
-    // If no ❯ lines found and requireDefault is true, there is no multiple_choice prompt.
-    // When requireDefault is false, skip this gate entirely to allow ❯-less detection.
-    // ==========================================================================
-    if (requireDefault) {
-        let hasDefaultLine = false;
-        for (const rawLine of scanWindow) {
-            const line = rawLine.trim();
-            if (DEFAULT_OPTION_PATTERN.test(line)) {
-                hasDefaultLine = true;
-                break;
-            }
-        }
-        if (!hasDefaultLine && !hasConfirmationFooter) {
-            return noPromptResult(output);
+    return { hasConfirmationFooter, hasNumberFooter };
+}
+function hasDefaultOptionLine(scanWindow) {
+    for (const rawLine of scanWindow) {
+        const line = rawLine.trim();
+        if (DEFAULT_OPTION_PATTERN.test(line)) {
+            return true;
         }
     }
+    return false;
+}
+function collectOptions(lines, scanStart, effectiveEnd) {
     // ==========================================================================
     // Pass 2: Collect options (executed when Pass 1 passes or is skipped)
     // Scan from end to find options, using both patterns.
     // ==========================================================================
-    const collectedOptions = [];
-    let questionEndIndex = -1;
-    let continuationLineCount = 0;
+    const state = {
+        collectedOptions: [],
+        // [Issue #3863] Per collected option (same order): the option row as drawn
+        // and the rows under it, for telling its title from its description.
+        optionRows: [],
+        // Continuation rows met since the option below was collected — the rows
+        // UNDER the option the reverse scan reaches next.
+        rowsBelow: [],
+        questionEndIndex: -1,
+        continuationLineCount: 0,
+        // [Issue #2486] Wrapped label rows beside a preview pane, held for the option
+        // ABOVE them, which the reverse scan has not reached yet.
+        paneWrap: [],
+    };
     // [Issue #1708] Claude's bottom-anchored task panel is an overlay, not part of
     // the prompt frame. Resolved as a block up front because the reverse scan meets
     // its rows before its header and so cannot recognise them one at a time.
@@ -992,16 +1061,13 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
     // rows the scan likewise meets before the top border that identifies it — and
     // some of which it shares with the options column.
     const chrome = findClaudeChrome(lines, scanStart, effectiveEnd);
-    // [Issue #2486] Wrapped label rows beside a preview pane, held for the option
-    // ABOVE them, which the reverse scan has not reached yet.
-    let paneWrap = [];
     for (let i = effectiveEnd - 1; i >= scanStart; i--) {
         // [Issue #1708] Chrome rows are metadata. Same handling as the
         // COLLAPSED_OUTPUT / SUMMARY guards below: skip without ending the scan, so
         // the options rendered ABOVE the panel (or beside the pane) are still
         // reachable.
         if (chrome.rows.has(i)) {
-            continuationLineCount++;
+            state.continuationLineCount++;
             continue;
         }
         // [Issue #2486] A row the preview pane shares with the options column is
@@ -1011,143 +1077,180 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
         // Original indentation preserved; only the pane is cut off.
         const rawLine = paneCut === undefined ? lines[i] : lines[i].slice(0, paneCut).trimEnd();
         const line = rawLine.trim();
-        // [Issue #2486] See PANE_WRAPPED_LABEL_PATTERN: the tail of the label above.
-        if (paneCut !== undefined && PANE_WRAPPED_LABEL_PATTERN.test(rawLine)) {
-            paneWrap.unshift(line);
-            continuationLineCount++;
-            continue;
-        }
-        // Collapsed preview markers like "[… 12 lines] ctrl + a view all" are
-        // metadata, not selectable options. Skip them before option parsing to
-        // avoid misclassifying "12" as an option number.
-        if (COLLAPSED_OUTPUT_PATTERN.test(line)) {
-            continuationLineCount++;
+        if (isMetadataRow(state, rawLine, line, paneCut)) {
             continue;
         }
-        // [Issue #704] Claude v2.1.142 renders summary lines like "… +1 pending"
-        // BELOW the real options. Without this guard, NORMAL_OPTION_PATTERN below
-        // would match the summary line as `option N / label='pending'`, which
-        // would then poison isValidPrecedingOption() and cause every real option
-        // above (1./2./3.) to be rejected — resulting in collectedOptions.length<2
-        // and a spurious no_prompt verdict. Treat the summary line as metadata
-        // (same continuation handling as COLLAPSED_OUTPUT_PATTERN above) so the
-        // reverse scan continues into the actual option block.
-        //
-        // Defense-in-depth: works alongside the S2 effectiveEnd trim above. Either
-        // S1 or S2 alone is sufficient to fix Issue #704; keeping both protects
-        // against future Claude renderings where the footer line text changes
-        // (S1 still catches) or the summary line wording changes (S2 still
-        // catches via footer trim).
-        if (SUMMARY_LINE_PATTERN.test(line)) {
-            continuationLineCount++;
+        if (tryCollectOption(state, rawLine, line, paneCut)) {
             continue;
         }
-        // Try DEFAULT_OPTION_PATTERN first (❯ indicator)
-        const defaultMatch = line.match(DEFAULT_OPTION_PATTERN);
-        if (defaultMatch) {
-            const number = parseInt(defaultMatch[1], 10);
-            if (number <= 20 && isValidPrecedingOption(number, collectedOptions)) {
-                // [Issue #2486] A label that wrapped beside a preview pane is whole again.
-                const label = [defaultMatch[2].trim(), ...paneWrap].join(' ');
-                paneWrap = [];
-                collectedOptions.unshift({ number, label, isDefault: true });
-                continuationLineCount = 0;
-                continue;
-            }
-        }
-        // Try NORMAL_OPTION_PATTERN (no ❯ indicator)
-        const normalMatch = line.match(NORMAL_OPTION_PATTERN);
-        if (normalMatch) {
-            const number = parseInt(normalMatch[1], 10);
-            // Skip unreasonably large option numbers (e.g., "[… 373 lines]" from Codex
-            // collapsed output matches as option 373). CLI prompts never exceed 20 options.
-            if (number > 20) {
-                // Treat as non-option line
-            }
-            else if (!isValidPrecedingOption(number, collectedOptions)) {
-                // Treat as non-option line (e.g., diff line numbers from Codex approval prompts)
-            }
-            else {
-                const label = [normalMatch[2].trim(), ...paneWrap].join(' ');
-                paneWrap = [];
-                collectedOptions.unshift({ number, label, isDefault: false });
-                continuationLineCount = 0;
-                continue;
-            }
-        }
         // [Issue #287 Bug3] User input prompt barrier:
         // When no options have been collected yet and the line starts with ❯ (U+276F)
         // but did NOT match DEFAULT_OPTION_PATTERN above, this line is a Claude Code
         // user input prompt (e.g., "❯ 1", "❯ /command") or idle prompt ("❯").
         // Anything above this line in the scrollback is historical conversation text,
         // not an active prompt. Stop scanning to prevent false positives.
-        if (collectedOptions.length === 0 && (line.startsWith('\u276F') || line.startsWith('\u25CF') || line.startsWith('\u203A'))) {
-            return noPromptResult(output);
+        if (state.collectedOptions.length === 0 && (line.startsWith('❯') || line.startsWith('●') || line.startsWith('›'))) {
+            return null;
+        }
+        // [Issue #3863] An indented row under the bottom option: its wrapped title
+        // or its description. Only remembered; the scan goes on as before.
+        if (state.collectedOptions.length === 0 && line && /^\s{3,}/.test(rawLine)) {
+            state.rowsBelow.unshift(rawLine);
         }
         // Non-option line handling
-        if (collectedOptions.length > 0 && line && !SEPARATOR_LINE_PATTERN.test(line)) {
-            // [Issue #460] For deeply indented lines (4+ leading spaces), check
-            // continuation BEFORE question keywords. These are option description
-            // lines (e.g., "     Let Gemini CLI decide the best model...") that may
-            // contain keywords like "decide" or "select" but are NOT question lines.
-            // The 2-space "  Select model" case (Issue #256) is excluded by the 4+ threshold.
-            const isDeepIndent = /^\s{4,}/.test(rawLine);
-            if (isDeepIndent && isContinuationLine(rawLine, line)) {
-                continuationLineCount++;
-                if (continuationLineCount > MAX_DEEP_INDENT_CONTINUATION_LINES) {
-                    questionEndIndex = i;
-                    break;
-                }
-                continue;
-            }
-            // [MF-001 / Issue #256] Check if line is a question-like line BEFORE
-            // continuation check. This preserves isContinuationLine()'s SRP by not
-            // mixing question detection into it. Without this pre-check, indented
-            // question lines (e.g., "  Select model") could be misclassified as
-            // continuation lines by isContinuationLine()'s hasLeadingSpaces check.
-            //
-            // [SF-S4-003] Both this pre-check and SEC-001b upward scan use the same
-            // isQuestionLikeLine() function intentionally (DRY). If a question line is
-            // caught here, SEC-001b upward scan is not needed (questionEndIndex line
-            // itself passes isQuestionLikeLine()).
-            if (isQuestionLikeLine(line)) {
-                questionEndIndex = i;
+        if (state.collectedOptions.length > 0 && line && !prompt_detect_multiple_choice_approval_context_1.SEPARATOR_LINE_PATTERN.test(line)) {
+            if (scanNonOptionLine(state, rawLine, line, i) === 'break') {
                 break;
             }
-            // Check if this is a continuation line (indented line between options,
-            // or path/filename fragments from terminal width wrapping - Issue #181)
-            if (isContinuationLine(rawLine, line)) {
-                continuationLineCount++;
-                const maxContinuationLines = /^\s{4,}/.test(rawLine)
-                    ? MAX_DEEP_INDENT_CONTINUATION_LINES
-                    : MAX_CONTINUATION_LINES;
-                // Issue #372: Codex TUI indents all output with 2 spaces, causing
-                // every line to match isContinuationLine(). Limit the scan distance
-                // to prevent traversing into body text where numbered lists would be
-                // collected as false options.
-                if (continuationLineCount > maxContinuationLines) {
-                    questionEndIndex = i;
-                    break;
-                }
-                // Skip continuation lines and continue scanning for more options
-                continue;
-            }
-            // Found a non-empty, non-separator line before options - likely the question
-            questionEndIndex = i;
-            break;
         }
     }
+    return {
+        collectedOptions: state.collectedOptions,
+        optionRows: state.optionRows,
+        questionEndIndex: state.questionEndIndex,
+    };
+}
+/** Rows the reverse scan skips as metadata without ending the scan. */
+function isMetadataRow(state, rawLine, line, paneCut) {
+    // [Issue #2486] See PANE_WRAPPED_LABEL_PATTERN: the tail of the label above.
+    if (paneCut !== undefined && PANE_WRAPPED_LABEL_PATTERN.test(rawLine)) {
+        state.paneWrap.unshift(line);
+        state.continuationLineCount++;
+        return true;
+    }
+    // Collapsed preview markers like "[… 12 lines] ctrl + a view all" are
+    // metadata, not selectable options. Skip them before option parsing to
+    // avoid misclassifying "12" as an option number.
+    if (COLLAPSED_OUTPUT_PATTERN.test(line)) {
+        state.continuationLineCount++;
+        return true;
+    }
+    // [Issue #704] Claude v2.1.142 renders summary lines like "… +1 pending"
+    // BELOW the real options. Without this guard, NORMAL_OPTION_PATTERN below
+    // would match the summary line as `option N / label='pending'`, which
+    // would then poison isValidPrecedingOption() and cause every real option
+    // above (1./2./3.) to be rejected — resulting in collectedOptions.length<2
+    // and a spurious no_prompt verdict. Treat the summary line as metadata
+    // (same continuation handling as COLLAPSED_OUTPUT_PATTERN above) so the
+    // reverse scan continues into the actual option block.
+    //
+    // Defense-in-depth: works alongside the S2 effectiveEnd trim above. Either
+    // S1 or S2 alone is sufficient to fix Issue #704; keeping both protects
+    // against future Claude renderings where the footer line text changes
+    // (S1 still catches) or the summary line wording changes (S2 still
+    // catches via footer trim).
+    if (SUMMARY_LINE_PATTERN.test(line)) {
+        state.continuationLineCount++;
+        return true;
+    }
+    return false;
+}
+/** Collects the row as an option when either option pattern accepts it. */
+function tryCollectOption(state, rawLine, line, paneCut) {
+    // Try DEFAULT_OPTION_PATTERN first (❯ indicator)
+    const defaultMatch = line.match(DEFAULT_OPTION_PATTERN);
+    if (defaultMatch) {
+        const number = parseInt(defaultMatch[1], 10);
+        if (number <= 20 && isValidPrecedingOption(number, state.collectedOptions)) {
+            // [Issue #2486] A label that wrapped beside a preview pane is whole again.
+            collectOption(state, rawLine, number, defaultMatch[2], true, paneCut);
+            return true;
+        }
+    }
+    // Try NORMAL_OPTION_PATTERN (no ❯ indicator)
+    const normalMatch = line.match(NORMAL_OPTION_PATTERN);
+    if (normalMatch) {
+        const number = parseInt(normalMatch[1], 10);
+        // Skip unreasonably large option numbers (e.g., "[… 373 lines]" from Codex
+        // collapsed output matches as option 373). CLI prompts never exceed 20 options.
+        if (number > 20) {
+            // Treat as non-option line
+        }
+        else if (!isValidPrecedingOption(number, state.collectedOptions)) {
+            // Treat as non-option line (e.g., diff line numbers from Codex approval prompts)
+        }
+        else {
+            collectOption(state, rawLine, number, normalMatch[2], false, paneCut);
+            return true;
+        }
+    }
+    return false;
+}
+function collectOption(state, rawLine, number, matchedLabel, isDefault, paneCut) {
+    const label = [matchedLabel.trim(), ...state.paneWrap].join(' ');
+    state.optionRows.unshift({ row: rawLine, besidePane: paneCut !== undefined || state.paneWrap.length > 0, rowsBelow: state.rowsBelow });
+    state.rowsBelow = [];
+    state.paneWrap = [];
+    state.collectedOptions.unshift({ number, label, isDefault });
+    state.continuationLineCount = 0;
+}
+/** A non-empty, non-separator, non-option row above collected options. */
+function scanNonOptionLine(state, rawLine, line, i) {
+    // [Issue #460] For deeply indented lines (4+ leading spaces), check
+    // continuation BEFORE question keywords. These are option description
+    // lines (e.g., "     Let Gemini CLI decide the best model...") that may
+    // contain keywords like "decide" or "select" but are NOT question lines.
+    // The 2-space "  Select model" case (Issue #256) is excluded by the 4+ threshold.
+    const isDeepIndent = /^\s{4,}/.test(rawLine);
+    if (isDeepIndent && isContinuationLine(rawLine, line)) {
+        state.rowsBelow.unshift(rawLine);
+        state.continuationLineCount++;
+        if (state.continuationLineCount > MAX_DEEP_INDENT_CONTINUATION_LINES) {
+            state.questionEndIndex = i;
+            return 'break';
+        }
+        return 'continue';
+    }
+    // [MF-001 / Issue #256] Check if line is a question-like line BEFORE
+    // continuation check. This preserves isContinuationLine()'s SRP by not
+    // mixing question detection into it. Without this pre-check, indented
+    // question lines (e.g., "  Select model") could be misclassified as
+    // continuation lines by isContinuationLine()'s hasLeadingSpaces check.
+    //
+    // [SF-S4-003] Both this pre-check and SEC-001b upward scan use the same
+    // isQuestionLikeLine() function intentionally (DRY). If a question line is
+    // caught here, SEC-001b upward scan is not needed (questionEndIndex line
+    // itself passes isQuestionLikeLine()).
+    if (isQuestionLikeLine(line)) {
+        state.questionEndIndex = i;
+        return 'break';
+    }
+    // Check if this is a continuation line (indented line between options,
+    // or path/filename fragments from terminal width wrapping - Issue #181)
+    if (isContinuationLine(rawLine, line)) {
+        state.rowsBelow.unshift(rawLine);
+        state.continuationLineCount++;
+        const maxContinuationLines = /^\s{4,}/.test(rawLine)
+            ? MAX_DEEP_INDENT_CONTINUATION_LINES
+            : MAX_CONTINUATION_LINES;
+        // Issue #372: Codex TUI indents all output with 2 spaces, causing
+        // every line to match isContinuationLine(). Limit the scan distance
+        // to prevent traversing into body text where numbered lists would be
+        // collected as false options.
+        if (state.continuationLineCount > maxContinuationLines) {
+            state.questionEndIndex = i;
+            return 'break';
+        }
+        // Skip continuation lines and continue scanning for more options
+        return 'continue';
+    }
+    // Found a non-empty, non-separator line before options - likely the question
+    state.questionEndIndex = i;
+    return 'break';
+}
+/** Layers 3–5: whether the collected options make a prompt. */
+function passesOptionValidation(collectedOptions, requireDefault, hasConfirmationFooter, questionEndIndex, lines, scanStart) {
     // Layer 3: Consecutive number validation (defensive measure)
     const optionNumbers = collectedOptions.map(opt => opt.number);
     if (!isConsecutiveFromOne(optionNumbers)) {
-        return noPromptResult(output);
+        return false;
     }
     // Layer 4: Must have at least 2 options. When requireDefault is true,
     // also require at least one option with ❯ indicator.
     const hasDefaultIndicator = collectedOptions.some(opt => opt.isDefault);
     const allowMissingDefaultIndicator = requireDefault && hasConfirmationFooter;
     if (collectedOptions.length < 2 || (requireDefault && !hasDefaultIndicator && !allowMissingDefaultIndicator)) {
-        return noPromptResult(output);
+        return false;
     }
     // Layer 5 [SEC-001]: Enhanced question line validation for requireDefaultIndicator=false.
     // When requireDefault is false, apply stricter validation to prevent false positives
@@ -1162,46 +1265,71 @@ function detectMultipleChoicePrompt(output, options, truncateRawContentFn) {
     // (「実装方針を選んでください」) be detected. Indicator-less frames (#193 capture
     // artifacts) still go through SEC-001a/b unchanged.
     if (!requireDefault && !hasDefaultIndicator) {
-        // SEC-001a: No question line found (questionEndIndex === -1) - reject.
-        // Prevents generic question fallback from triggering Auto-Yes
-        // on plain numbered lists that happen to be consecutive from 1.
-        if (questionEndIndex === -1) {
-            return noPromptResult(output);
-        }
-        // SEC-001b: Question line exists but is not actually a question/selection request.
-        // Validates that the question line contains a question mark or a selection keyword
-        // with colon, distinguishing "Select an option:" from "Recommendations:".
-        //
-        // [Issue #256] Enhanced with upward scan via findQuestionLineInRange() (SF-003).
-        // When questionEndIndex line itself is not a question-like line, scan upward
-        // within QUESTION_SCAN_RANGE to find a question line above it. This handles:
-        // - Multi-line question wrapping where ? is on a line above questionEndIndex
-        // - Model selection prompts where "Select model" is above description lines
-        const questionLine = lines[questionEndIndex]?.trim() ?? '';
-        if (!isQuestionLikeLine(questionLine)) {
-            // Upward scan: look for a question-like line above questionEndIndex
-            if (!findQuestionLineInRange(lines, questionEndIndex, QUESTION_SCAN_RANGE, scanStart)) {
-                return noPromptResult(output);
-            }
+        return passesQuestionLineValidation(questionEndIndex, lines, scanStart);
+    }
+    return true;
+}
+function passesQuestionLineValidation(questionEndIndex, lines, scanStart) {
+    // SEC-001a: No question line found (questionEndIndex === -1) - reject.
+    // Prevents generic question fallback from triggering Auto-Yes
+    // on plain numbered lists that happen to be consecutive from 1.
+    if (questionEndIndex === -1) {
+        return false;
+    }
+    // SEC-001b: Question line exists but is not actually a question/selection request.
+    // Validates that the question line contains a question mark or a selection keyword
+    // with colon, distinguishing "Select an option:" from "Recommendations:".
+    //
+    // [Issue #256] Enhanced with upward scan via findQuestionLineInRange() (SF-003).
+    // When questionEndIndex line itself is not a question-like line, scan upward
+    // within QUESTION_SCAN_RANGE to find a question line above it. This handles:
+    // - Multi-line question wrapping where ? is on a line above questionEndIndex
+    // - Model selection prompts where "Select model" is above description lines
+    const questionLine = lines[questionEndIndex]?.trim() ?? '';
+    if (!isQuestionLikeLine(questionLine)) {
+        // Upward scan: look for a question-like line above questionEndIndex
+        if (!findQuestionLineInRange(lines, questionEndIndex, QUESTION_SCAN_RANGE, scanStart)) {
+            return false;
         }
     }
-    // [Issue #2486] An AskUserQuestion picker's tab row is the dialog's top edge:
-    // the row is chrome and everything above it is transcript. Found once so
-    // `question` and `approvalTarget` start at the same row; 0 on a frame with no
-    // tab row, which leaves every other prompt exactly as it was. `instructionText`
-    // keeps its wide window — it is the human's view, and on a preview picker the
-    // pane it carries is what the question asks the human to look at.
-    const pickerTop = questionEndIndex < 0
-        ? 0
-        : (0, picker_chrome_1.findAskUserQuestionTabRow)(lines, questionEndIndex, exports.APPROVAL_TARGET_MAX_LOOKBACK) + 1;
-    const question = extractQuestionText(lines, questionEndIndex, pickerTop);
-    const instructionText = extractInstructionText(lines, questionEndIndex, effectiveEnd);
-    // Issue #1699: the same block, cut at the previous turn's boundary. Display
-    // keeps the wide window; deny patterns are judged against this one.
-    const approvalTarget = extractApprovalTarget(lines, questionEndIndex, effectiveEnd, pickerTop);
-    // Issue #616: Determine submitMode from confirmation footer (detected in single-pass above)
-    const submitMode = hasConfirmationFooter
-        ? (hasNumberFooter ? 'answer_only' : 'answer_then_enter')
-        : undefined;
-    return buildMultipleChoiceResult(question, collectedOptions, { instructionText, approvalTarget }, output, truncateRawContentFn, submitMode, hasAskUserQuestionFooter);
+    return true;
+}
+/**
+ * [Issue #3863] What an AskUserQuestion option's title says about being
+ * recommended (`option-recommendation`).
+ *
+ * The picker draws a title row and its description indented under it; a title
+ * too long for the pane wraps to the same indent, so the rows alone do not
+ * say which they are. Read:
+ *
+ * - no row under it: the label is the title;
+ * - beside a preview pane (#2486), where the column draws no descriptions:
+ *   the label — its wrapped rows already joined — is the title, as long as no
+ *   other row is under it;
+ * - the row under it could not be the title's wrap (its first word would have
+ *   fit on the option row, `cannotBeWrapOf`): the label is the title;
+ * - the row under it may be the wrap and holds nothing but a marker: the title
+ *   wrapped right before its marker — and ends there only if nothing after
+ *   the marker continues it: no further row, or a row that could not be the
+ *   marker row's wrap (its first word would have fit after the marker). A row
+ *   that may still be the title wrapping on (a word too long to fit after the
+ *   marker) leaves the title's end, and so the marker's place, unknown;
+ * - otherwise `unknown`.
+ */
+function readPickerOptionRecommendation(label, reading, paneColumns) {
+    if (reading === undefined)
+        return 'unknown';
+    const [next] = reading.rowsBelow;
+    if (next === undefined)
+        return (0, option_recommendation_1.isRecommendedTitle)(label);
+    if (reading.besidePane)
+        return 'unknown';
+    if ((0, option_recommendation_1.cannotBeWrapOf)(reading.row, next, paneColumns))
+        return (0, option_recommendation_1.isRecommendedTitle)(label);
+    if (!(0, option_recommendation_1.isRecommendationMarker)(next))
+        return 'unknown';
+    const after = reading.rowsBelow[1];
+    if (after === undefined || (0, option_recommendation_1.cannotBeWrapOf)(next, after, paneColumns))
+        return true;
+    return 'unknown';
 }