commandmate 0.39.1 → 0.40.1

package/dist/server/src/lib/cmate-cli-tool-parser.js

@@ -23,12 +23,14 @@ exports.tokenizeCliToolColumn = tokenizeCliToolColumn;
 exports.parseCliToolColumn = parseCliToolColumn;
 exports.validateCopilotModelName = validateCopilotModelName;
 exports.validateAntigravityModelName = validateAntigravityModelName;
+exports.validateClaudeModelName = validateClaudeModelName;
 exports.validateOpencodeRunName = validateOpencodeRunName;
 exports.validateOpencodeTitle = validateOpencodeTitle;
 exports.parseAndValidateCliToolColumn = parseAndValidateCliToolColumn;
 exports.resolveScheduleCommandOptions = resolveScheduleCommandOptions;
 const copilot_constants_1 = require("../config/copilot-constants");
 const antigravity_constants_1 = require("../config/antigravity-constants");
+const claude_constants_1 = require("../config/claude-constants");
 const opencode_constants_1 = require("../config/opencode-constants");
 // =============================================================================
 // Constants
@@ -342,6 +344,35 @@ function validateAntigravityModelName(modelName) {
     }
     return { valid: true };
 }
+/**
+ * Validate a Claude model name using the reject approach (no sanitization).
+ * Issue #2771: `claude --model` takes an alias (`sonnet`, `opus`, `opus[1m]`) or
+ * a full id (`claude-sonnet-5`, `claude-opus-5[1m]`, a Bedrock / Vertex id). The
+ * brackets are why Copilot's pattern cannot be reused, and the display names
+ * Antigravity accepts ("Gemini 3.1 Pro (High)") are not something Claude takes.
+ *
+ * @param modelName - Model name to validate
+ * @returns Validation result with optional reason for rejection
+ */
+function validateClaudeModelName(modelName) {
+    // Control character rejection
+    if (/[\x00-\x1f\x7f]/.test(modelName)) {
+        return { valid: false, reason: 'Model name contains control characters' };
+    }
+    // Empty / whitespace-only rejection
+    if (modelName.trim() === '') {
+        return { valid: false, reason: 'Model name must not be empty' };
+    }
+    // Pattern validation (leading alphanumeric required, DR4-001)
+    if (!claude_constants_1.CLAUDE_MODEL_NAME_PATTERN.test(modelName)) {
+        return { valid: false, reason: 'Model name contains invalid characters' };
+    }
+    // Length validation
+    if (modelName.length > claude_constants_1.MAX_CLAUDE_MODEL_NAME_LENGTH) {
+        return { valid: false, reason: `Model name exceeds ${claude_constants_1.MAX_CLAUDE_MODEL_NAME_LENGTH} characters` };
+    }
+    return { valid: true };
+}
 /**
  * Validate an opencode `--agent` / `--variant` value (Issue #2044).
  *

package/dist/server/src/lib/detection/cli-patterns.js

@@ -4,8 +4,8 @@
  * Shared between response-poller.ts and API routes
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.COPILOT_TRANSCRIPT_DIVIDER_PATTERN = exports.COPILOT_USER_ECHO_PATTERN = exports.COPILOT_REASONING_HEADER_PATTERN = exports.COPILOT_BOX_ROW_PATTERN = exports.COPILOT_SEPARATOR_PATTERN = exports.COPILOT_IDLE_STATUS_PATTERN = exports.COPILOT_WORKING_STATUS_PATTERN = exports.COPILOT_THINKING_PATTERN = exports.COPILOT_PROMPT_PATTERN = exports.OPENCODE_SKIP_PATTERNS = exports.OPENCODE_SEPARATOR_PATTERN = exports.CLAUDE_SELECTION_LIST_FOOTER = exports.CLAUDE_MODEL_OVERLAY_FOOTER_PATTERN = exports.OPENCODE_SELECTION_LIST_PATTERN = exports.OPENCODE_FOOTER_STATUS_PATTERN = exports.OPENCODE_USER_ECHO_PATTERN = exports.OPENCODE_GUTTER_ROW_PATTERN = exports.OPENCODE_COMPOSER_BOTTOM_BORDER = exports.OPENCODE_PROCESSING_INDICATOR = exports.OPENCODE_PERMISSION_PATTERN = exports.OPENCODE_TURN_COMPLETE_PATTERN = exports.OPENCODE_RESPONSE_COMPLETE = exports.OPENCODE_LOADING_PATTERN = exports.OPENCODE_THINKING_PATTERN = exports.OPENCODE_PROMPT_AFTER_RESPONSE = exports.OPENCODE_IDLE_COMPOSER_PATTERN = exports.OPENCODE_PROMPT_PATTERN = exports.GEMINI_THINKING_PATTERN = exports.GEMINI_PROMPT_PATTERN = exports.MAX_PASTED_TEXT_RETRIES = exports.PASTED_TEXT_DETECT_DELAY = exports.PASTED_TEXT_PATTERN = exports.CODEX_USER_ECHO_PATTERN = exports.CODEX_STATUS_BAR_PATTERN = exports.CODEX_PAGER_FOOTER_PATTERN = exports.CODEX_FORM_SUBMIT_FOOTER_PATTERN = exports.CODEX_APPROVAL_FOOTER_PATTERN = exports.CODEX_SELECTION_LIST_PATTERN = exports.CODEX_SEPARATOR_PATTERN = exports.CODEX_HOOKS_REVIEW_ANCHORS = exports.CODEX_DIALOG_PATTERN = exports.CODEX_PROMPT_PATTERN = exports.CLAUDE_TRUST_DIALOG_PATTERN = exports.CLAUDE_SEPARATOR_PATTERN = exports.CLAUDE_PROMPT_PATTERN = exports.CODEX_INTERRUPT_HINT_PATTERN = exports.CODEX_THINKING_PATTERN = exports.CLAUDE_INTERRUPT_HINT_PATTERN = exports.CLAUDE_THINKING_PATTERN = exports.CLAUDE_SPINNER_CHARS = void 0;
-exports.CLAUDE_SESSION_ERROR_REGEX_PATTERNS = exports.CLAUDE_SESSION_ERROR_PATTERNS = exports.extractAnsiSequences = exports.stripAnsi = exports.COMMAND_CODE_SKIP_PATTERNS = exports.COMMAND_CODE_HOOK_NOTICE_PATTERN = exports.COMMAND_CODE_BANNER_PATTERNS = exports.COMMAND_CODE_MODE_INDICATOR_PATTERN = exports.COMMAND_CODE_COMPLETION_PATTERN = exports.COMMAND_CODE_RESPONSE_MARKER_PATTERN = exports.COMMAND_CODE_THINKING_PATTERN = exports.COMMAND_CODE_INTERRUPT_HINT_PATTERN = exports.COMMAND_CODE_SEPARATOR_PATTERN = exports.COMMAND_CODE_PROMPT_PATTERN = exports.COMMAND_CODE_SPINNER_CHARS = exports.ANTIGRAVITY_SKIP_PATTERNS = exports.ANTIGRAVITY_SURVEY_PATTERN = exports.ANTIGRAVITY_DIALOG_MAX_ROWS = exports.ANTIGRAVITY_DIALOG_BOUNDARY_PATTERN = exports.ANTIGRAVITY_SWITCH_MODEL_HEADER_PATTERN = exports.ANTIGRAVITY_NUMBERED_OPTION_PATTERN = exports.ANTIGRAVITY_NAVIGATE_FOOTER_PATTERN = exports.ANTIGRAVITY_SELECTION_LIST_PATTERN = exports.ANTIGRAVITY_SEPARATOR_PATTERN = exports.ANTIGRAVITY_THINKING_PATTERN = exports.ANTIGRAVITY_PROMPT_PATTERN = exports.ANTIGRAVITY_COMPOSER_MODE_BANNER_PATTERN = exports.ANTIGRAVITY_COMPOSER_MODE_BANNER_SOURCE = exports.VIBE_LOCAL_THINKING_PATTERN = exports.VIBE_LOCAL_PROMPT_PATTERN = exports.COPILOT_SKIP_PATTERNS = exports.COPILOT_FOLDER_TRUST_ANSWER_KEY = exports.COPILOT_FOLDER_TRUST_SESSION_OPTION_PATTERN = exports.COPILOT_FOLDER_TRUST_ANCHORS = exports.COPILOT_SELECTION_FOOTER_PATTERN = exports.COPILOT_BOOT_BANNER_ANCHORS = exports.COPILOT_TRANSCRIPT_CONTINUATION_PATTERN = exports.COPILOT_TOOL_ROW_PATTERN = exports.COPILOT_TOOL_VERBS = void 0;
+exports.COPILOT_USER_ECHO_PATTERN = exports.COPILOT_REASONING_HEADER_PATTERN = exports.COPILOT_BOX_ROW_PATTERN = exports.COPILOT_SEPARATOR_PATTERN = exports.COPILOT_IDLE_STATUS_PATTERN = exports.COPILOT_WORKING_STATUS_PATTERN = exports.COPILOT_THINKING_PATTERN = exports.COPILOT_PROMPT_PATTERN = exports.OPENCODE_SKIP_PATTERNS = exports.OPENCODE_SEPARATOR_PATTERN = exports.CLAUDE_SELECTION_LIST_FOOTER = exports.CLAUDE_MODEL_OVERLAY_FOOTER_PATTERN = exports.OPENCODE_SELECTION_LIST_PATTERN = exports.OPENCODE_FOOTER_STATUS_PATTERN = exports.OPENCODE_USER_ECHO_PATTERN = exports.OPENCODE_GUTTER_ROW_PATTERN = exports.OPENCODE_COMPOSER_BOTTOM_BORDER = exports.OPENCODE_PROCESSING_INDICATOR = exports.OPENCODE_PERMISSION_PATTERN = exports.OPENCODE_TURN_COMPLETE_PATTERN = exports.OPENCODE_RESPONSE_COMPLETE = exports.OPENCODE_LOADING_PATTERN = exports.OPENCODE_THINKING_PATTERN = exports.OPENCODE_PROMPT_AFTER_RESPONSE = exports.OPENCODE_IDLE_COMPOSER_PATTERN = exports.OPENCODE_PROMPT_PATTERN = exports.GEMINI_THINKING_PATTERN = exports.GEMINI_PROMPT_PATTERN = exports.MAX_PASTED_TEXT_RETRIES = exports.PASTED_TEXT_DETECT_DELAY = exports.PASTED_TEXT_PATTERN = exports.CODEX_USER_ECHO_PATTERN = exports.CODEX_TRAILED_STATUS_BAR_PATTERN = exports.CODEX_STATUS_BAR_PATTERN = exports.CODEX_PAGER_FOOTER_PATTERN = exports.CODEX_FORM_SUBMIT_FOOTER_PATTERN = exports.CODEX_APPROVAL_FOOTER_PATTERN = exports.CODEX_SELECTION_LIST_PATTERN = exports.CODEX_SEPARATOR_PATTERN = exports.CODEX_HOOKS_REVIEW_ANCHORS = exports.CODEX_DIALOG_PATTERN = exports.CODEX_PROMPT_PATTERN = exports.CLAUDE_TRUST_DIALOG_PATTERN = exports.CLAUDE_SEPARATOR_PATTERN = exports.CLAUDE_PROMPT_PATTERN = exports.CODEX_INTERRUPT_HINT_PATTERN = exports.CODEX_THINKING_PATTERN = exports.CLAUDE_INTERRUPT_HINT_PATTERN = exports.CLAUDE_THINKING_PATTERN = exports.CLAUDE_SPINNER_CHARS = void 0;
+exports.CLAUDE_SESSION_ERROR_REGEX_PATTERNS = exports.CLAUDE_SESSION_ERROR_PATTERNS = exports.extractAnsiSequences = exports.stripAnsi = exports.COMMAND_CODE_SKIP_PATTERNS = exports.COMMAND_CODE_HOOK_NOTICE_PATTERN = exports.COMMAND_CODE_BANNER_PATTERNS = exports.COMMAND_CODE_MODE_INDICATOR_PATTERN = exports.COMMAND_CODE_COMPLETION_PATTERN = exports.COMMAND_CODE_RESPONSE_MARKER_PATTERN = exports.COMMAND_CODE_THINKING_PATTERN = exports.COMMAND_CODE_INTERRUPT_HINT_PATTERN = exports.COMMAND_CODE_SEPARATOR_PATTERN = exports.COMMAND_CODE_PROMPT_PATTERN = exports.COMMAND_CODE_SPINNER_CHARS = exports.ANTIGRAVITY_SKIP_PATTERNS = exports.ANTIGRAVITY_SURVEY_PATTERN = exports.ANTIGRAVITY_DIALOG_MAX_ROWS = exports.ANTIGRAVITY_DIALOG_BOUNDARY_PATTERN = exports.ANTIGRAVITY_SWITCH_MODEL_HEADER_PATTERN = exports.ANTIGRAVITY_NUMBERED_OPTION_PATTERN = exports.ANTIGRAVITY_NAVIGATE_FOOTER_PATTERN = exports.ANTIGRAVITY_SELECTION_LIST_PATTERN = exports.ANTIGRAVITY_SEPARATOR_PATTERN = exports.ANTIGRAVITY_THINKING_PATTERN = exports.ANTIGRAVITY_PROMPT_PATTERN = exports.ANTIGRAVITY_COMPOSER_MODE_BANNER_PATTERN = exports.ANTIGRAVITY_COMPOSER_MODE_BANNER_SOURCE = exports.VIBE_LOCAL_THINKING_PATTERN = exports.VIBE_LOCAL_PROMPT_PATTERN = exports.COPILOT_SKIP_PATTERNS = exports.COPILOT_FOLDER_TRUST_ANSWER_KEY = exports.COPILOT_FOLDER_TRUST_SESSION_OPTION_PATTERN = exports.COPILOT_FOLDER_TRUST_ANCHORS = exports.COPILOT_SELECTION_FOOTER_PATTERN = exports.COPILOT_BOOT_BANNER_ANCHORS = exports.COPILOT_TRANSCRIPT_CONTINUATION_PATTERN = exports.COPILOT_TOOL_ROW_PATTERN = exports.COPILOT_TOOL_VERBS = exports.COPILOT_TRANSCRIPT_DIVIDER_PATTERN = void 0;
 exports.isCodexTurnActive = isCodexTurnActive;
 exports.findClaudeChromeStart = findClaudeChromeStart;
 exports.isCodexPromptReady = isCodexPromptReady;
@@ -629,6 +629,43 @@ exports.CODEX_PAGER_FOOTER_PATTERN = /(?:↑\/↓|pgup\/pgdn|home\/end)\s+to\s+(
  * line. No nested quantifiers (ReDoS-safe; adjacent greedy quantifiers only).
  */
 exports.CODEX_STATUS_BAR_PATTERN = /^\s*\S.*·\s*~?\/\S*\s*$/;
+/**
+ * Codex status bar with something drawn AFTER the path (Issue #2818) — the
+ * second shape `findCodexFooterBoundary` in `tools/codex/detect.ts` accepts.
+ *
+ * From codex 0.154.0 on, the bar stops ending in the path once the first turn
+ * has named the thread, and Plan mode adds a right-aligned badge:
+ *
+ * ```text
+ *   gpt-5.6-terra low · /private/var/…/repo · Run touch probe.txt
+ *   gpt-6-astra medium · ~/uat3-…/sandbox-repo                    Plan mode (shift+tab to cycle)
+ * ```
+ *
+ * {@link CODEX_STATUS_BAR_PATTERN} wants the path last, so every such frame had
+ * no boundary and fell to the detector's bar-independent branch D (#1150's
+ * safety net). That branch reads the 15-row tail, and on an idle frame the tail
+ * still holds the finished turn's `• Ran …` record — which is how an idle
+ * session read `running` (#2808's `idle-after-declined-approval.txt`; #2818
+ * reproduced it on a turn that simply ran one command and answered).
+ *
+ * Kept a SEPARATE pattern rather than a widened {@link CODEX_STATUS_BAR_PATTERN}
+ * so the change stays on the one reader it was measured for: that pattern is
+ * also the stripped-capture landmark of {@link findCodexChromeStart} and the
+ * value reader's first test in `model-info-extractor.ts`, neither of which
+ * this Issue measured.
+ *
+ * The shape is the one `CODEX_STATUS_BAR_WITH_TRAILER_PATTERN`
+ * (`model-info-extractor.ts`, #2592) reads values off, written out here rather
+ * than imported: that module's rule is that it must never be the reason this
+ * boundary moves. Head segment, `·`, a path, then EITHER a further `·`
+ * segment OR a column gap (two spaces) before right-aligned text. `[^·]*` puts
+ * the path right after the FIRST `·`, so codex's in-flight row (`• Working (…)
+ * · 1 background terminal running · /ps to view`) is not a bar — its first `·`
+ * is followed by a count, not a path.
+ *
+ * Single-line, no /g, no nested quantifiers (ReDoS-safe), as above.
+ */
+exports.CODEX_TRAILED_STATUS_BAR_PATTERN = /^\s*\S[^·]*·\s*~?\/\S*(?:\s*·[^\n]*|[^\S\n]{2,}\S[^\n]*)$/;
 /**
  * How far above the last non-blank row {@link findCodexChromeStart} looks for the
  * composer.
@@ -702,9 +739,11 @@ const CODEX_CHROME_GLYPH_ROW_PATTERN = /^›(\s|$)/;
  * the list silently stopped matching the thing it was written for. Issue #2310
  * measured what actually separates codex's three uses of `›` (U+203A), and it is
  * the SGR attributes, not the text: the composer glyph is bold (`ESC[1m›`), a
- * transcript echo is dim (`ESC[1;2m›`), a dialog option carries a coloured glyph
- * or a bold label. {@link readCodexGlyphRowKind} is that measurement, and this
- * reader is one of its callers.
+ * transcript echo is dim (`ESC[1;2m›`), a dialog option carries a bold label or
+ * a label drawn in the glyph's own colour (one span — since #2798 a coloured
+ * glyph alone is not enough, because 0.155.1 colours the composer's glyph too).
+ * {@link readCodexGlyphRowKind} is that measurement, and this reader is one of
+ * its callers.
  *
  * `-1` is returned for a frame whose bottom-most `›` is an option row: codex
  * replaces the composer with the dialog, so there is no chrome to trim and the

package/dist/server/src/lib/detection/model-info-extractor.js

@@ -62,7 +62,7 @@
  * @see Issue #1784, Phase 2 of the model/effort visibility series
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ANTIGRAVITY_MODEL_ID_EFFORT_PATTERN = exports.COMMAND_CODE_BANNER_MODELS_PATTERN = exports.OPENCODE_STEP_MODEL_PATTERN = exports.COPILOT_MODEL_CHANGE_PATTERN = exports.ANTIGRAVITY_BANNER_MODEL_PATTERN = exports.ANTIGRAVITY_STATUS_BAR_HINT_PATTERN = exports.CLAUDE_MODEL_SWITCH_PATTERN = exports.CLAUDE_STARTUP_BANNER_PATTERN = exports.CODEX_STATUS_BAR_WITH_TRAILER_PATTERN = exports.CODEX_FOOTER_MODEL_PATTERN = exports.REASONING_EFFORT_LEVELS = void 0;
+exports.ANTIGRAVITY_MODEL_ID_EFFORT_PATTERN = exports.COMMAND_CODE_BANNER_MODELS_PATTERN = exports.OPENCODE_STEP_MODEL_PATTERN = exports.COPILOT_MODEL_CHANGE_PATTERN = exports.ANTIGRAVITY_BANNER_MODEL_PATTERN = exports.ANTIGRAVITY_STATUS_BAR_HINT_PATTERN = exports.CLAUDE_MODEL_SWITCH_PATTERN = exports.CLAUDE_STARTUP_BANNER_PATTERN = exports.CODEX_STATUS_BAR_WITH_TRAILER_PATTERN = exports.CODEX_FOOTER_EFFORT_SLOT_PATTERN = exports.CODEX_FOOTER_MODEL_PATTERN = exports.REASONING_EFFORT_LEVELS = void 0;
 exports.resolveEffortToken = resolveEffortToken;
 exports.extractModelInfo = extractModelInfo;
 exports.deriveEffortFromModelId = deriveEffortFromModelId;
@@ -72,11 +72,27 @@ const cli_patterns_1 = require("./cli-patterns");
 /**
  * The reasoning-effort vocabulary these tools actually print.
  *
- * `minimal` and `xhigh` are the ends Codex exposes; Claude prints `xhigh` too.
+ * `minimal` and `xhigh` are the ends Codex exposed until 0.155; Claude prints
+ * `xhigh` too. Issue #2835 adds the three levels above `xhigh` that codex 0.155.1
+ * defines (`ReasoningEffort` in `codex-rs/protocol/src/openai_models.rs`, tag
+ * `rust-v0.155.1`: `max` / `ultra` / `persistent`, drawn in the status bar
+ * verbatim by `status_line_reasoning_effort_label`). Its `None` is drawn as
+ * `default` and is deliberately NOT a level — it names no effort. Command Code's
+ * `with max effort` banner resolves through the same list.
+ *
  * Anything outside this list is treated as "not an effort token", which is what
  * keeps a stray word at the end of a status bar from being published as one.
  */
-exports.REASONING_EFFORT_LEVELS = ['minimal', 'low', 'medium', 'high', 'xhigh'];
+exports.REASONING_EFFORT_LEVELS = [
+    'minimal',
+    'low',
+    'medium',
+    'high',
+    'xhigh',
+    'max',
+    'ultra',
+    'persistent',
+];
 /**
  * Longest string accepted as a model name.
  *
@@ -156,7 +172,23 @@ function resolveEffortToken(token, options = {}) {
  * optional `\s+<keyword>` are adjacent, never nested (ReDoS-safe, same rule
  * `cli-patterns.ts` documents).
  */
-exports.CODEX_FOOTER_MODEL_PATTERN = /^\s*([^\s·]+)(?:\s+(minimal|low|medium|high|xhigh)\b)?/i;
+exports.CODEX_FOOTER_MODEL_PATTERN = /^\s*([^\s·]+)(?:\s+(minimal|low|medium|high|xhigh|max|ultra|persistent)\b)?/i;
+/**
+ * The head of a Codex status bar that has a word where the effort goes
+ * (Issue #2835).
+ *
+ * codex draws `<model> <effort>` and, for an effort it has no name for, still
+ * draws a word there: `default` for `None` (`status_line_reasoning_effort_label`,
+ * codex 0.155.1; live in `tests/fixtures/agent-mode-2592/codex-default.txt`), or
+ * a model-defined value verbatim. A head whose second token is a word is
+ * therefore a bar that SHOWS an effort, even when {@link CODEX_FOOTER_MODEL_PATTERN}
+ * cannot name it. The legacy o4-mini head (`o4-mini            50% left`) has no
+ * such slot — its second token starts with a digit — and neither does a status
+ * line configured to show the model alone.
+ *
+ * No /g. No nested quantifiers: two adjacent runs and a lookahead.
+ */
+exports.CODEX_FOOTER_EFFORT_SLOT_PATTERN = /^\s*[^\s·]+\s+[A-Za-z][A-Za-z_-]*(?=\s|$)/;
 /**
  * A codex status bar with something AFTER the path (Issue #2592).
  *
@@ -202,7 +234,8 @@ function readCodexFooter(line) {
     const separator = line.indexOf('·');
     if (separator < 0)
         return null;
-    const match = exports.CODEX_FOOTER_MODEL_PATTERN.exec(line.slice(0, separator));
+    const head = line.slice(0, separator);
+    const match = exports.CODEX_FOOTER_MODEL_PATTERN.exec(head);
     if (!match)
         return null;
     const model = match[1];
@@ -219,6 +252,13 @@ function readCodexFooter(line) {
     // beside it is what separates chrome from prose.
     if (effort === null && !/\d/.test(model))
         return null;
+    // Issue #2835: a bar that shows an effort this module cannot name says the
+    // latched effort is no longer current. Publishing the old one would be a
+    // wrong value, which is worse than none — the case that found it was a
+    // session moved from `xhigh` to `max` while `max` was not yet a level.
+    if (effort === null && exports.CODEX_FOOTER_EFFORT_SLOT_PATTERN.test(head)) {
+        return { model, effort: null, effortUnreadable: true };
+    }
     return { model, effort };
 }
 // =============================================================================
@@ -612,11 +652,10 @@ function readOpencodeStepMarker(line) {
  *    footer reads `? for shortcuts · taste on`), so everything from the dot on
  *    is dropped, and the dot itself is optional.
  *  - **The effort is read when the banner prints one, through the shared
- *    vocabulary.** `low` / `high` resolve; `max` is not a
- *    {@link ReasoningEffort} and answers null rather than widening a list every
- *    other reader and the UI type depend on. The model half is published either
- *    way: the ` with … effort` frame delimits it cleanly, so nothing is
- *    half-read.
+ *    vocabulary.** `low` / `high` / `max` resolve (`max` since Issue #2835,
+ *    which added it to {@link REASONING_EFFORT_LEVELS} for codex; before that it
+ *    answered null). The model half is published either way: the
+ *    ` with … effort` frame delimits it cleanly, so nothing is half-read.
  *
  * ## After a `/model` switch (measured, 1.49.0)
  *

package/dist/server/src/lib/detection/selection-shape.js

@@ -64,10 +64,12 @@
  * written once.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.DISMISSABLE_PANEL_FOOTER_PATTERN = exports.COMMAND_CODE_SELECTION_LIST_FOOTER = exports.SET_AS_DEFAULT_FOOTER_PATTERN = exports.SESSION_SCOPE_FOOTER_PATTERN = exports.MAX_OPTION_NUMBER = exports.SELECTION_SHAPE_TAIL_LINE_COUNT = void 0;
+exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_FOOTER = exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_ROW = exports.COMMAND_CODE_PLAN_REVIEW_FOOTER = exports.DISMISSABLE_PANEL_FOOTER_PATTERN = exports.COMMAND_CODE_SELECTION_LIST_FOOTER = exports.SET_AS_DEFAULT_FOOTER_PATTERN = exports.SESSION_SCOPE_FOOTER_PATTERN = exports.MAX_OPTION_NUMBER = exports.SELECTION_SHAPE_TAIL_LINE_COUNT = void 0;
+exports.isCommandCodePlanApproveChoice = isCommandCodePlanApproveChoice;
 exports.hasDismissablePanelFooter = hasDismissablePanelFooter;
 exports.readCommandCodeQuestionRegion = readCommandCodeQuestionRegion;
 exports.hasCommandCodeQuestionChrome = hasCommandCodeQuestionChrome;
+exports.readCommandCodeReviewPage = readCommandCodeReviewPage;
 exports.extractCommandCodeSelectionListFrame = extractCommandCodeSelectionListFrame;
 exports.readSelectionListShape = readSelectionListShape;
 exports.shouldOfferOptionNumbers = shouldOfferOptionNumbers;
@@ -188,6 +190,98 @@ exports.COMMAND_CODE_SELECTION_LIST_FOOTER = /\benter\s+to\s+select\b\s*[·•]\
  * Code writes its picker hints in lower case; nothing rests on which it uses.
  */
 exports.DISMISSABLE_PANEL_FOOTER_PATTERN = /^\s*(?:press|hit|type)?\s*(?:<)?esc(?:ape)?(?:>)?\s+to\s+(?:close|dismiss|exit)\b[\s.·•]*$/im;
+/**
+ * The two footer rows of Command Code's plan review overlay (Issue #2761).
+ *
+ * Measured on 1.58.0 at 200x1000. The overlay is a rule, a `Plan review: <title>
+ * · <path> · v1` header, the plan with a line-number gutter, a second rule, a
+ * ` REVIEW ` badge and then, verbatim:
+ *
+ *     Approve ctrl+a   executes the plan
+ *     Cancel esc
+ *
+ * BOTH rows, adjacent, because each half alone is weak: `Cancel esc` is two
+ * words any picker might print, and `Approve` is a word an agent writes in a
+ * plan. Multi-line on purpose — it is tested against a joined tail
+ * (`NormalizedFrame.lastLines`, or the tail {@link readSelectionListShape}
+ * builds), never against a single row.
+ *
+ * Swept against every fixture in `tests/fixtures` and
+ * `tests/unit/lib/detection/fixtures` when this was written (300 files): the
+ * only match is the plan review capture itself.
+ *
+ * ## The `❯` either row may carry (Issue #2793)
+ *
+ * `↓` past the plan's last line moves the focus onto the ACTION list, and the
+ * focused action is drawn with a `❯ ` in front of it and two spaces before its
+ * key (`tests/fixtures/command-code-plan-review-2763/`, 1.58.0):
+ *
+ *     ❯ Approve  ctrl+a   executes the plan        ← `plan-review-action-focus-approve.txt`
+ *     Cancel esc
+ *
+ *     Approve ctrl+a   executes the plan · comments go along as notes
+ *     ❯ Cancel  esc                                ← `plan-review-long-scrolled-cancel-focused.txt`
+ *
+ * Without the optional `❯ ` this pattern missed both frames, and they went on to
+ * the composer check, where `COMMAND_CODE_PROMPT_PATTERN` (`^❯(\s*$|\s+\S)`)
+ * read the cursor row as the composer and published `ready` / `input_prompt`:
+ * `wait` exited 0 on a plan nobody had approved — #2754's `❯`-outside-the-list
+ * family. It is also the frame on which `Enter` EXECUTES the focused action, so
+ * missing it here would also have left the chat surface's `Enter` on screen
+ * (see {@link SelectionListShape.offersPlanApprove}). Only `❯`, and only as a
+ * prefix: `❯ ` + the row is the one measured spelling of a focused action.
+ * Re-swept over the same two directories with the widened pattern: every match
+ * is still a `command-code-plan-review-*` capture, which
+ * `command-code-plan-review-2793.test.ts` keeps pinned.
+ */
+exports.COMMAND_CODE_PLAN_REVIEW_FOOTER = /^\s*(?:❯ +)?Approve\s+ctrl\+a\b[^\n]*\n\s*(?:❯ +)?Cancel\s+esc\s*$/im;
+/**
+ * The radio row `ctrl+a` puts in place of the action list when comments are
+ * pending (Issue #2793).
+ *
+ * Measured on 1.58.0 (`tests/fixtures/command-code-plan-review-2763/`): with
+ * zero comments `ctrl+a` approves at once, and with one or more it first asks
+ * what to do with them. The three action rows are replaced by ONE row, and the
+ * selected side is `(•)`, the other `( )` — readable without the ANSI:
+ *
+ *     Approve (•) with 4 comments as notes ( ) original plan · discard comments
+ *
+ * `comments?` because the badge above it is measured in both numbers
+ * (`1 pending comment` / `4 pending comments`); the radio itself was captured
+ * with four.
+ */
+exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_ROW = /^\s*Approve\s+\([• ]\)\s+with\s+\d+\s+comments?\s+as\s+notes\s+\([• ]\)\s+original\s+plan\b/im;
+/**
+ * The hint bar under {@link COMMAND_CODE_PLAN_APPROVE_CHOICE_ROW}, verbatim
+ * `←/→ choose · enter confirm · esc back`.
+ *
+ * Anchored to the LAST row of the text it is tested against (no `m` flag: `$` is
+ * the end of the input), because that is where the overlay draws every one of
+ * its five measured hint bars, and because it is what keeps a transcript that
+ * QUOTES this screen from matching — the composer is always drawn below a
+ * transcript, so there the quote is never the last row.
+ */
+exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_FOOTER = /(?:^|\n)[ \t]*←\/→\s+choose\s*·\s*enter\s+confirm\s*·\s*esc\s+back\s*$/i;
+/**
+ * Whether this tail is the approve-with-comments confirmation (Issue #2793).
+ *
+ * BOTH rows, for the reason {@link COMMAND_CODE_PLAN_REVIEW_FOOTER} gives for
+ * its two: either half alone is a sentence something else could print. Not
+ * adjacent — `Editor exited with code 127` and a blank row sit between them on
+ * both captures — so each is tested on its own.
+ *
+ * Deliberately NOT folded into {@link COMMAND_CODE_PLAN_REVIEW_FOOTER}, and
+ * therefore not {@link SelectionListShape.offersPlanApprove}: on this screen
+ * `enter` is the documented CONFIRM of an approval the human already started
+ * with `ctrl+a`, so the chat surface must keep its `Enter`, and what `ctrl+a`
+ * does here was not measured.
+ *
+ * @param tail - the frame's last rows, ANSI stripped, last content row last
+ */
+function isCommandCodePlanApproveChoice(tail) {
+    return (exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_ROW.test(tail) &&
+        exports.COMMAND_CODE_PLAN_APPROVE_CHOICE_FOOTER.test(tail));
+}
 /**
  * How many rows from the end of the content the dismiss footer is looked for.
  *
@@ -276,8 +370,24 @@ const COMMAND_CODE_CURSOR_GLYPH = '❯';
 const COMMAND_CODE_TAB_SELECTED_MARKERS = '●◉⦿';
 /** The hollow half of {@link COMMAND_CODE_TAB_SELECTED_MARKERS}'s pair. */
 const COMMAND_CODE_TAB_UNSELECTED_MARKERS = '◯○◌⚪';
+/**
+ * The ANSWERED half, measured on 1.54.1 (Issue #2753).
+ *
+ * A multi-question `AskUserQuestion` keeps one tab per question and marks the
+ * ones already answered: `✔ Party size | ✔ Rental car | ● Update scope | ◯ Review`.
+ * Before this family existed the whole strip failed the cell test, the reader answered
+ * `none`, and the generic parser read the checkbox list under it as a single
+ * select — see the Issue for the capture.
+ *
+ * U+2714 and nothing else. `✓` (U+2713) and `☑` (U+2611) are the shapes a check
+ * mark could take; neither has been seen on a pane, and this reading is shared
+ * with `ChatSurface` and `extractCommandCodeSelectionListFrame`, which run for
+ * EVERY tool — so an unmeasured glyph here widens what other tools' frames can
+ * be claimed as this screen.
+ */
+const COMMAND_CODE_TAB_ANSWERED_MARKERS = '✔';
 /** One `<marker> <label>` cell of the tab strip. */
-const COMMAND_CODE_TAB_SEGMENT_PATTERN = new RegExp(`^\\s*([${COMMAND_CODE_TAB_SELECTED_MARKERS}${COMMAND_CODE_TAB_UNSELECTED_MARKERS}])\\s+\\S`);
+const COMMAND_CODE_TAB_SEGMENT_PATTERN = new RegExp(`^\\s*([${COMMAND_CODE_TAB_SELECTED_MARKERS}${COMMAND_CODE_TAB_UNSELECTED_MARKERS}${COMMAND_CODE_TAB_ANSWERED_MARKERS}])\\s+\\S`);
 /**
  * Is this row Command Code's question-screen tab strip?
  *
@@ -288,16 +398,19 @@ const COMMAND_CODE_TAB_SEGMENT_PATTERN = new RegExp(`^\\s*([${COMMAND_CODE_TAB_S
  *    the structure and the one prose never has in this position;
  *  - **every cell is `<marker> <label>`.** A row that is a strip for its first
  *    cell and prose for its second is prose;
- *  - **at least one filled marker and at least one hollow one.** A tab strip
+ *  - **at least one filled marker, and at least one other cell.** A tab strip
  *    says which tab has the screen. A row of identical bullets does not, and
- *    `● one | ● two` is exactly the list-with-a-pipe this rules out.
+ *    `● one | ● two` is exactly the list-with-a-pipe this rules out. The other
+ *    cell may be hollow (not answered yet) or the `✔` of an answered tab
+ *    (measured on 1.54.1 — Issue #2753); what may not be missing is the filled
+ *    one, so `✔ one | ✔ two` is still refused.
  */
 function isCommandCodeQuestionTabRow(line) {
     const segments = line.split('|');
     if (segments.length < 2)
         return false;
     let selected = 0;
-    let unselected = 0;
+    let others = 0;
     for (const segment of segments) {
         const match = COMMAND_CODE_TAB_SEGMENT_PATTERN.exec(segment);
         if (match === null)
@@ -305,9 +418,9 @@ function isCommandCodeQuestionTabRow(line) {
         if (COMMAND_CODE_TAB_SELECTED_MARKERS.includes(match[1]))
             selected += 1;
         else
-            unselected += 1;
+            others += 1;
     }
-    return selected > 0 && unselected > 0;
+    return selected > 0 && others > 0;
 }
 /**
  * Rows that mean the region spans more than the one dialog on screen now.
@@ -382,9 +495,13 @@ const COMMAND_CODE_QUESTION_REGION_REJECT_PATTERN = /\besc\s+to\s+interrupt\b|^[
  *  3. the options are a STRICT run `1.` … `N.`, 2 ≤ N ≤ {@link
  *     MAX_OPTION_NUMBER}. A gap, a repeat or a list that starts at `2` is not
  *     this screen;
- *  4. exactly one {@link COMMAND_CODE_CURSOR_GLYPH} in the region, and it is on
- *     an option row. Two cursors means two screens — typically a composer's own
- *     `❯` under the dialog — and none means the highlight is elsewhere;
+ *  4. exactly one {@link COMMAND_CODE_CURSOR_GLYPH} in the region. Two cursors
+ *     means two screens — typically a composer's own `❯` under the dialog — and
+ *     none means the highlight is elsewhere. Issue #2755 dropped the "and it is
+ *     on an option row" half: 1.54.1 parks the cursor on `Submit`, on `Next` and
+ *     on the `notes:` row, and refusing those frames published six measured
+ *     captures of a live question as finished turns. WHERE it sits is reported
+ *     as {@link CommandCodeQuestionRegion.cursorOnOptionRow} instead;
  *  5. no filter box, no `/model` footer, no dismiss-only footer and no other
  *     turn's UI in the region ({@link FILTER_INPUT_PATTERN},
  *     {@link COMMAND_CODE_SELECTION_LIST_FOOTER},
@@ -446,6 +563,7 @@ function readCommandCodeQuestionRegion(frame) {
         lastLineIndex: screen.lastLineIndex,
         tabLineIndex: screen.tabLineIndex,
         cursorLineIndex: screen.cursorLineIndex,
+        cursorOnOptionRow: screen.cursorOnOptionRow,
         optionCount: screen.numbers.length,
     };
 }
@@ -478,17 +596,126 @@ function readCommandCodeQuestionRegion(frame) {
  * Every structural condition stays: the last qualifying rule row, a genuine tab
  * strip as the first row under it ({@link isCommandCodeQuestionTabRow} — one
  * filled marker, one hollow, `|`-separated), a non-blank question body before
- * the first option, at least two numbered rows, at least one `❯` ON one of them,
+ * the first option, at least two numbered rows, at least one `❯` in the region,
  * and none of the pickers, panels or other-turn UI the other branches own. An
  * assistant answering in a numbered list carries none of that, and a frame with
- * one option or with the `❯` on a composer row is still `false` — the two cases
- * #2521 recorded as "the highlight is elsewhere".
+ * one option is still `false`.
+ *
+ * Issue #2755 no longer requires that `❯` to be ON a numbered row — see
+ * condition 4 of {@link readCommandCodeQuestionRegion} for why, and
+ * {@link CommandCodeQuestionRegion.cursorOnOptionRow} for what is reported
+ * instead.
  *
  * @param frame - a raw `capture-pane -p -e` frame, ANSI intact (CRLF tolerated)
  */
 function hasCommandCodeQuestionChrome(frame) {
     return scanCommandCodeQuestionScreen(frame) !== null;
 }
+/**
+ * The Review page's own three rows, measured on 1.54.1 (Issue #2755 §7).
+ *
+ * `Enter` on the `Submit` row does not send the answers — it opens a SECOND
+ * screen:
+ *
+ *     ✔ Update scope | ● Review
+ *
+ *     1. Which files should I update?
+ *        calc.js
+ *
+ *     ❯ 1. Submit
+ *       2. Cancel
+ *
+ *     ← to go back and edit
+ *
+ * ## Why this needs a reading of its own
+ *
+ * Because it is a real numbered list under a real tab strip, and the GENERIC
+ * parser answers it: a two-option `multiple_choice` whose default is `Submit`.
+ * The default answer on that payload does not pick anything — it COMMITS
+ * whatever the human has ticked so far, from a payload nobody meant to expose,
+ * which is #1928's shape one screen later. Auto-Yes fires on defaults.
+ *
+ * Words rather than structure, and deliberately: the two numbered rows are
+ * `Submit` and `Cancel`, which is the same shape as any other two-option
+ * question. What separates this page from a question is only what it SAYS, so
+ * all three rows are required together — the confirm pair AND the back hint —
+ * and the whole thing is anchored inside a recognised question region. A screen
+ * that carries two of the three keeps every verdict it had.
+ *
+ * The cursor glyph is optional on both numbered rows: it rests on `1. Submit`
+ * when the page is reached by `Enter`, and the Issue is explicit that a pattern
+ * decided on the bare word would miss it.
+ */
+const COMMAND_CODE_REVIEW_SUBMIT_ROW_PATTERN = /^[^\S\n]*(?:\u276F[^\S\n]*)?1[.)][^\S\n]+Submit[^\S\n]*$/im;
+/** The `Cancel` half of {@link COMMAND_CODE_REVIEW_SUBMIT_ROW_PATTERN}'s pair. */
+const COMMAND_CODE_REVIEW_CANCEL_ROW_PATTERN = /^[^\S\n]*(?:\u276F[^\S\n]*)?2[.)][^\S\n]+Cancel[^\S\n]*$/im;
+/** The footer only the Review page draws. */
+const COMMAND_CODE_REVIEW_BACK_HINT_PATTERN = /^[^\S\n]*\u2190[^\S\n]+to[^\S\n]+go[^\S\n]+back[^\S\n]+and[^\S\n]+edit[^\S\n]*$/im;
+/**
+ * The warning the Review page carries when it was reached with answers missing.
+ *
+ * Measured from the undocumented `d`, which ends a multi-select from any row:
+ * `⚠ You have not answered all questions` over a `No answer`. Issue #2755 §7
+ * requires that such a page is never read as "answered", which is why the
+ * predicate below reports it rather than hiding it.
+ */
+const COMMAND_CODE_REVIEW_UNANSWERED_PATTERN = /have\s+not\s+answered\s+all\s+questions/i;
+/**
+ * Read Command Code's `AskUserQuestion` Review page, if this frame is one
+ * (Issue #2755 §7).
+ *
+ * The same region {@link readCommandCodeQuestionRegion} works on — last rule
+ * row, tab strip as the first row under it — plus the three rows above. The
+ * question-shaped conditions are deliberately NOT applied: this page has no
+ * question body of its own on the `Submit`/`Cancel` spelling (the answers sit
+ * where the question would), and it draws `1.` twice, so the strict reading
+ * declines it for a numbering reason that says nothing about what it is.
+ *
+ * `null` for everything else, so no existing verdict moves except the two
+ * captures this is about.
+ *
+ * @param frame - a raw `capture-pane -p -e` frame, ANSI intact (CRLF tolerated)
+ */
+function readCommandCodeReviewPage(frame) {
+    if (!frame)
+        return null;
+    const lines = frame.replace(/\r\n/g, '\n').split('\n').map(ansi_1.stripAnsi);
+    let last = lines.length - 1;
+    while (last >= 0 && lines[last].trim() === '')
+        last -= 1;
+    if (last < 1)
+        return null;
+    let ruleLineIndex = -1;
+    for (let i = last - 1; i >= 0; i -= 1) {
+        const row = lines[i].trim();
+        if (row.length < COMMAND_CODE_RULE_MIN_COLUMNS)
+            continue;
+        if (!COMMAND_CODE_RULE_ROW_PATTERN.test(row))
+            continue;
+        ruleLineIndex = i;
+        break;
+    }
+    if (ruleLineIndex < 0)
+        return null;
+    const region = lines.slice(ruleLineIndex + 1, last + 1);
+    const tabOffset = region.findIndex((line) => line.trim() !== '');
+    if (tabOffset < 0)
+        return null;
+    if (!isCommandCodeQuestionTabRow(region[tabOffset]))
+        return null;
+    const body = region.join('\n');
+    if (!COMMAND_CODE_REVIEW_SUBMIT_ROW_PATTERN.test(body))
+        return null;
+    if (!COMMAND_CODE_REVIEW_CANCEL_ROW_PATTERN.test(body))
+        return null;
+    if (!COMMAND_CODE_REVIEW_BACK_HINT_PATTERN.test(body))
+        return null;
+    const submitRow = region.find((row) => COMMAND_CODE_REVIEW_SUBMIT_ROW_PATTERN.test(row));
+    return {
+        hasUnansweredWarning: COMMAND_CODE_REVIEW_UNANSWERED_PATTERN.test(body),
+        cursorOnSubmit: submitRow !== undefined && submitRow.includes(COMMAND_CODE_CURSOR_GLYPH),
+    };
+}
 /**
  * The structural half of the question-screen reading, shared by the strict
  * {@link readCommandCodeQuestionRegion} and the weaker
@@ -566,18 +793,38 @@ function scanCommandCodeQuestionScreen(frame) {
     if (!region.slice(tabOffset + 1, firstOptionOffset).some((line) => line.trim() !== '')) {
         return null;
     }
-    // 4. A cursor, on an option row.
+    // 4. A cursor, somewhere in the region.
+    //
+    // Issue #2755 removed the second half of this condition ("and it is on an
+    // option row"). 1.54.1 lets the `❯` leave the list — onto `Submit`, onto
+    // `Next`, onto the `notes:` input — and requiring it on a numbered row made
+    // six measured captures of a LIVE question fall through to the composer
+    // check, which answers `ready` off the dialog's own `❯` row and makes `wait`
+    // exit 0 on a pane that is asking a human a question (#2521's 偽完了, §8).
+    //
+    // The generalisation is deliberate and is stated in the Issue: the condition
+    // is "the `❯` is outside the list", NOT the word `Submit`. Adding the three
+    // measured labels would have to be re-opened for the fourth, and 1.54.1
+    // already draws three. Where the cursor sits is reported instead, in
+    // {@link CommandCodeQuestionScreen.cursorOnOptionRow}, and the reader decides
+    // what a screen it cannot answer is worth.
+    //
+    // Nothing else about the reading moved, so a frame that is not a question
+    // screen is refused by exactly the conditions that refused it before: the last
+    // rule row, a genuine tab strip as its first row, a question body, two or more
+    // numbered rows, and none of the pickers or panels the other branches own.
     let cursorCount = 0;
     let cursorOffset = -1;
+    let cursorOnOptionRow = false;
     for (let i = 0; i < region.length; i += 1) {
         const inRow = region[i].split(COMMAND_CODE_CURSOR_GLYPH).length - 1;
         if (inRow === 0)
             continue;
         cursorCount += inRow;
-        if (OPTION_ROW_PATTERN.test(region[i]))
-            cursorOffset = i;
+        cursorOffset = i;
+        cursorOnOptionRow = OPTION_ROW_PATTERN.test(region[i]);
     }
-    if (cursorCount === 0 || cursorOffset < 0)
+    if (cursorCount === 0)
         return null;
     return {
         ruleLineIndex,
@@ -587,6 +834,7 @@ function scanCommandCodeQuestionScreen(frame) {
         numbers,
         cursorCount,
         cursorLineIndex: ruleLineIndex + 1 + cursorOffset,
+        cursorOnOptionRow,
     };
 }
 /**
@@ -690,6 +938,7 @@ const EMPTY_SHAPE = {
     offersSessionScope: false,
     commitsDefaultOnEnter: false,
     hasFilterInput: false,
+    offersPlanApprove: false,
 };
 /** The last {@link SELECTION_SHAPE_TAIL_LINE_COUNT} rows that carry content. */
 function tailLines(frame) {
@@ -750,12 +999,13 @@ function readSelectionListShape(frame) {
         offersSessionScope: exports.SESSION_SCOPE_FOOTER_PATTERN.test(tail),
         commitsDefaultOnEnter: exports.SET_AS_DEFAULT_FOOTER_PATTERN.test(tail),
         hasFilterInput: FILTER_INPUT_PATTERN.test(tail),
+        offersPlanApprove: exports.COMMAND_CODE_PLAN_REVIEW_FOOTER.test(tail),
     };
 }
 /**
  * Whether the card may draw a `1`…`N` row for this shape.
  *
- * Two refusals, both measured rather than defensive:
+ * Three refusals, all measured rather than defensive:
  *
  *  - **a session-scope footer.** On claude's `/model` a number key commits AND
  *    rewrites the global default in one press (probed live on 2.1.260), so a
@@ -764,7 +1014,12 @@ function readSelectionListShape(frame) {
  *  - **a filter input.** copilot's `/model` and Command Code's picker put a
  *    focused search box on the dialog, where a `4` is four characters of a
  *    query and not the fourth model.
+ *  - **a plan review.** Every character typed there is a comment on the plan,
+ *    and the plan's own text is full of `1.` `2.` rows.
  */
 function shouldOfferOptionNumbers(shape) {
-    return shape.optionCount > 0 && !shape.offersSessionScope && !shape.hasFilterInput;
+    return (shape.optionCount > 0 &&
+        !shape.offersSessionScope &&
+        !shape.hasFilterInput &&
+        !shape.offersPlanApprove);
 }

package/dist/server/src/lib/detection/status-detector.js

@@ -60,6 +60,8 @@ exports.SELECTION_LIST_REASONS = new Set([
     // unclassified floor, whose controls are the answer characters and whose
     // answer characters go into the picker's search box.
     status_reason_1.STATUS_REASON.COMMAND_CODE_SELECTION_LIST,
+    // Issue #2761: Command Code's plan review. Arrow-driven, human-only.
+    status_reason_1.STATUS_REASON.COMMAND_CODE_PLAN_REVIEW,
     status_reason_1.STATUS_REASON.COPILOT_SELECTION_LIST,
     status_reason_1.STATUS_REASON.CODEX_SELECTION_LIST,
     // Issue #1017: Codex pager/edit-previous mode also drives NavigationButtons.