commandmate 0.44.1 → 0.44.3

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

@@ -47,7 +47,9 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.opencodeV2StatusDetector = exports.OPENCODE_V2_PERMISSION_OPTIONS = exports.OPENCODE_V2_TURN_COMPLETE_PATTERN = exports.OPENCODE_V2_QUESTION_PATTERN = exports.OPENCODE_V2_PERMISSION_PATTERN = exports.VERIFIED_AGAINST = void 0;
 exports.endsWithTurnComplete = endsWithTurnComplete;
 const cli_patterns_1 = require("../../cli-patterns");
+const verdicts_1 = require("../verdicts");
 const status_reason_1 = require("../../status-reason");
+const live_region_1 = require("./live-region");
 const run_detection_1 = require("../run-detection");
 const verified_against_1 = require("../verified-against");
 exports.VERIFIED_AGAINST = verified_against_1.OPENCODE_V2_VERIFIED_AGAINST;
@@ -141,6 +143,8 @@ function endsWithTurnComplete(frame) {
 exports.opencodeV2StatusDetector = (0, run_detection_1.createToolStatusDetector)({
     tool: 'opencode-v2',
     verifiedAgainst: exports.VERIFIED_AGAINST,
+    // Issue #3183: where the live part of the frame begins (`./live-region.ts`).
+    liveRegion: live_region_1.OPENCODE_V2_LIVE_REGION,
     // Issue #2991: every v2 dialog is recognised by `beforePrompt` below, so a
     // numbered list that reaches the generic parser is text in a reply (the UAT
     // frame: `❯ 1. Yes / 2. No` quoted in the answer). Unvouched by
@@ -157,22 +161,10 @@ exports.opencodeV2StatusDetector = (0, run_detection_1.createToolStatusDetector)
     // with no Enter — see `detectDialog` below.
     beforePrompt(frame) {
         if (exports.OPENCODE_V2_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);
         }
         if (exports.OPENCODE_V2_QUESTION_PATTERN.test(frame.lastLines)) {
-            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);
         }
         // Issue #2971: a dialog opened over the composer keeps the footer (and
         // often the placeholder) on screen, so every rule below read it as `ready`
@@ -182,50 +174,26 @@ exports.opencodeV2StatusDetector = (0, run_detection_1.createToolStatusDetector)
         // overlay, driven by ↑↓ + Enter and closed with `esc` — no number answers
         // it, hence `hasActivePrompt: false`.
         if ((0, cli_patterns_1.findOpencodeV2DialogTitle)((0, cli_patterns_1.stripAnsi)(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);
         }
         return null;
     },
     afterThinking(frame) {
         if ((0, cli_patterns_1.detectThinking)('opencode-v2', frame.lastLines)) {
-            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);
         }
         // Issue #2965: the completion row as the last transcript row, with no
         // `esc interrupt` (ruled out just above) — a finished turn, whether or not
         // the footer made it into the capture.
         if (endsWithTurnComplete(frame)) {
-            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);
         }
         // The placeholder against the ANSI-stripped frame with its box drawing
         // intact (the pattern anchors on the `┃` gutter). The footer is on every
         // frame, and the running hint was ruled out just above.
         if (cli_patterns_1.OPENCODE_V2_IDLE_COMPOSER_PATTERN.test((0, cli_patterns_1.stripAnsi)(frame.raw)) ||
             cli_patterns_1.OPENCODE_V2_FOOTER_PATTERN.test(frame.lastLines)) {
-            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-v2/live-region.js

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

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

@@ -0,0 +1,63 @@
+"use strict";
+/**
+ * OpenCode V2 pattern constants moved out of `../../cli-patterns.ts`
+ * (Issue #3217 I-3). `cli-patterns.ts` re-exports every public name.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.OPENCODE_V2_SKIP_PATTERNS = exports.OPENCODE_V2_THINKING_PATTERN = exports.OPENCODE_V2_FOOTER_PATTERN = exports.OPENCODE_V2_IDLE_COMPOSER_PATTERN = void 0;
+exports.isOpencodeV2ComposerVisible = isOpencodeV2ComposerVisible;
+const pasted_text_1 = require("../../shared/pasted-text");
+const patterns_1 = require("../opencode/patterns");
+// =============================================================================
+// OpenCode V2 (`opencode2`, Issue #2934)
+// =============================================================================
+//
+// A separate tool id from `opencode` (Epic #2370, decision 1). Its TUI is drawn
+// by the same toolkit and many rows look alike, but v1's constants are v1's:
+// they are left exactly as they are, and v2 names its own here so a change on
+// either side cannot move the other. Phase 1 reads only what the send path and
+// the fallback status reader need; the approval dialog (`Always allow`, the
+// reverse of v1's `Allow always`), the completion row and the session tabs are
+// Phase 3's.
+/**
+ * OpenCode V2's empty composer: the input box's `┃` gutter with the
+ * `Ask anything…` placeholder on the same row (2.0.18 draws U+2026; the ASCII
+ * `...` is accepted too, as for v1 since #2915).
+ *
+ * Measured on 2.0.18 at 80x200 (2026-09-28):
+ * `   ┃  Ask anything… "Fix broken tests"`. Match against the ANSI-stripped
+ * frame BEFORE `stripBoxDrawing`, which removes the gutter this anchors on.
+ */
+exports.OPENCODE_V2_IDLE_COMPOSER_PATTERN = /^[^\S\n]*[\u2502\u2503][^\S\n]*Ask anything(?:\.\.\.|\u2026)/m;
+/**
+ * OpenCode V2's footer, which ends in `ctrl+p commands` on every frame the
+ * TUI draws (before a turn: `<path>:<branch>  shift+tab agents  ctrl+p commands`;
+ * after one: `<path>:<branch>  8.8K (1%)  ctrl+p commands`, 2.0.18).
+ *
+ * Proof that the TUI — and so its composer — is on screen, NOT that it is idle:
+ * a running turn keeps the footer.
+ */
+exports.OPENCODE_V2_FOOTER_PATTERN = /ctrl\+p commands/;
+/**
+ * OpenCode V2 working indicator: the footer's `esc interrupt` hint, drawn only
+ * while a turn is running (2.0.18).
+ */
+exports.OPENCODE_V2_THINKING_PATTERN = /esc interrupt/;
+/**
+ * Whether OpenCode V2's composer is on screen, so a send may type into it
+ * (Issue #2934, D6): the gutter-anchored placeholder, or the footer.
+ *
+ * @param text - ANSI-stripped capture, box drawing intact
+ */
+function isOpencodeV2ComposerVisible(text) {
+    return exports.OPENCODE_V2_IDLE_COMPOSER_PATTERN.test(text) || exports.OPENCODE_V2_FOOTER_PATTERN.test(text);
+}
+/** OpenCode V2 rows that are chrome, never reply content. */
+exports.OPENCODE_V2_SKIP_PATTERNS = [
+    patterns_1.OPENCODE_SEPARATOR_PATTERN,
+    exports.OPENCODE_V2_FOOTER_PATTERN,
+    exports.OPENCODE_V2_THINKING_PATTERN,
+    /Ask anything(?:\.\.\.|\u2026)/,
+    /^Build\s+·/,
+    pasted_text_1.PASTED_TEXT_PATTERN,
+];

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

@@ -32,6 +32,9 @@ exports.runToolDetection = runToolDetection;
 exports.createToolStatusDetector = createToolStatusDetector;
 const cli_patterns_1 = require("../cli-patterns");
 const prompt_detector_1 = require("../prompt-detector");
+const frame_1 = require("./frame");
+const live_region_1 = require("./live-region");
+const verdicts_1 = require("./verdicts");
 const status_reason_1 = require("../status-reason");
 const detection_evidence_config_1 = require("../../../config/detection-evidence-config");
 const idle_evidence_observation_1 = require("../idle-evidence-observation");
@@ -43,20 +46,6 @@ const idle_evidence_observation_1 = require("../idle-evidence-observation");
  * which is `running` with no evidence, not `ready`.
  */
 const STALE_OUTPUT_THRESHOLD_MS = 5000;
-/**
- * Which tools hand the FULL frame to `detectPrompt` instead of the 15-line tail.
- *
- * Their multiple-choice prompts with descriptions can exceed 15 lines: Codex
- * approval prompts with long file lists, Claude "Yes, and don't ask again for:
- * git commit -m …" options that embed full commit messages. `detectPrompt`
- * applies its own 50-line window internally.
- */
-const FULL_FRAME_PROMPT_TOOLS = new Set([
-    'opencode',
-    'codex',
-    'claude',
-    'copilot',
-]);
 /**
  * Resolve the `evidence` for a `ready` / `input_prompt` verdict from the generic
  * composer check (§4 D1 決定 1 item 2).
@@ -77,7 +66,10 @@ function resolveIdleEvidence(spec, frame) {
     return mode === 'observe' ? 'positive' : evidence;
 }
 /** Run the whole chain for one tool over one frame. */
-function runToolDetection(spec, frame, context = {}) {
+function runToolDetection(spec, input, context = {}) {
+    // Issue #3183: every branch below reads the tool's own live region, whoever
+    // built the frame.
+    const frame = (0, frame_1.withLiveRegion)(input, spec.tool);
     // 0.x — the tool's own pre-prompt branches. A picker, a pager or a status bar
     // whose body the generic parser would misread as a numbered dialog.
     const early = spec.beforePrompt?.(frame) ?? null;
@@ -88,12 +80,19 @@ function runToolDetection(spec, frame, context = {}) {
     // Gemini wraps prompts in box-drawing characters (╭╮╰╯│─) which prevent
     // detectPrompt() from recognizing the prompt content.
     const promptOptions = (0, cli_patterns_1.buildDetectPromptOptions)(spec.tool);
-    const promptInput = FULL_FRAME_PROMPT_TOOLS.has(spec.tool)
+    const promptInput = (0, cli_patterns_1.usesFullFramePrompt)(spec.tool)
         ? (0, cli_patterns_1.stripBoxDrawing)(frame.clean)
         : (0, cli_patterns_1.stripBoxDrawing)(frame.lastLines);
     let promptDetection = (0, prompt_detector_1.detectPrompt)(promptInput, promptOptions);
     if (promptDetection.isPrompt) {
-        if (spec.isStalePrompt?.(frame, promptDetection)) {
+        if ((0, live_region_1.isQuotedNumberedPrompt)(frame.liveRegion, promptDetection)) {
+            // Issue #3183: the input box is the bottom of the pane, so the numbered
+            // list sits in the conversation above it — a reply quoting a dialog
+            // (#2845 / #2991) or a dialog left in the scrollback. The same rule the
+            // Auto-Yes path applies (`detectPromptOnCleanFrame`), on the same region.
+            promptDetection = { ...promptDetection, isPrompt: false, promptData: undefined };
+        }
+        else if (spec.isStalePrompt?.(frame, promptDetection)) {
             // Issue #1160: an ALREADY-ANSWERED block still inside the scan window.
             // Neutralise it so Auto-Yes and the sidebar never act on a dead prompt,
             // and let the chain continue to the tool's running/idle branches.
@@ -109,14 +108,7 @@ function runToolDetection(spec, frame, context = {}) {
             promptDetection = { ...promptDetection, isPrompt: false, promptData: undefined };
         }
         else {
-            return {
-                status: 'waiting',
-                confidence: 'high',
-                reason: status_reason_1.STATUS_REASON.PROMPT_DETECTED,
-                hasActivePrompt: true,
-                evidence: 'positive',
-                promptDetection,
-            };
+            return (0, verdicts_1.activePromptVerdict)(status_reason_1.STATUS_REASON.PROMPT_DETECTED, promptDetection);
         }
     }
     // 1.5 — tool branches that must sit between prompt detection and the thinking
@@ -127,14 +119,7 @@ function runToolDetection(spec, frame, context = {}) {
     // 2. Thinking indicator detection — THINKING_TAIL_LINE_COUNT window (narrower).
     // CLI tool is actively processing (shows spinner, "Planning...", etc.)
     if ((0, cli_patterns_1.detectThinking)(spec.tool, frame.thinkingLines)) {
-        return {
-            status: 'running',
-            confidence: 'high',
-            reason: status_reason_1.STATUS_REASON.THINKING_INDICATOR,
-            hasActivePrompt: false,
-            evidence: 'positive',
-            promptDetection,
-        };
+        return (0, verdicts_1.positiveVerdictWithPrompt)('running', status_reason_1.STATUS_REASON.THINKING_INDICATOR, promptDetection);
     }
     // 2.x — the tool's own running and completion markers.
     const afterThinking = spec.afterThinking?.(frame, promptDetection) ?? null;
@@ -169,14 +154,7 @@ function runToolDetection(spec, frame, context = {}) {
     if (context.lastOutputTimestamp) {
         const elapsed = Date.now() - context.lastOutputTimestamp.getTime();
         if (elapsed > STALE_OUTPUT_THRESHOLD_MS) {
-            return {
-                status: 'running',
-                confidence: 'low',
-                reason: status_reason_1.STATUS_REASON.NO_RECENT_OUTPUT,
-                hasActivePrompt: false,
-                evidence: 'none',
-                promptDetection,
-            };
+            return (0, verdicts_1.unreadVerdictWithPrompt)(status_reason_1.STATUS_REASON.NO_RECENT_OUTPUT, promptDetection);
         }
     }
     // 5. Floor. `running` with no evidence, as before — the safe default when the
@@ -186,14 +164,7 @@ function runToolDetection(spec, frame, context = {}) {
     // rules looked at this frame and found nothing, which is a statement about the
     // rules and an instruction to capture the frame as a fixture. `default` stays
     // the answer for a tool that has no chain of its own, where nothing looked.
-    return {
-        status: 'running',
-        confidence: 'low',
-        reason: spec.unreadableReason ?? status_reason_1.STATUS_REASON.DEFAULT,
-        hasActivePrompt: false,
-        evidence: 'none',
-        promptDetection,
-    };
+    return (0, verdicts_1.unreadVerdictWithPrompt)(spec.unreadableReason ?? status_reason_1.STATUS_REASON.DEFAULT, promptDetection);
 }
 /** Bind the shared chain around one tool's declarations. */
 function createToolStatusDetector(spec) {
@@ -205,7 +176,17 @@ function createToolStatusDetector(spec) {
         // caller which of the two it got, because the Auto-Yes gate must suppress
         // for "this tool has rules and this frame is not a dialog" and must NOT
         // suppress for "nobody has measured this tool yet".
-        detectDialog: spec.detectDialog ?? (() => null),
+        //
+        // Issue #3183: wrapped so every tool's reading gets the live-region veto in
+        // one place — a dialog on a frame whose input box is the bottom of the pane
+        // is a quotation, whatever the tool's own rule matched.
+        detectDialog: spec.detectDialog
+            ? (input) => {
+                const frame = (0, frame_1.withLiveRegion)(input, spec.tool);
+                const verdict = spec.detectDialog(frame);
+                return (0, live_region_1.vetoesDialog)(frame.liveRegion, verdict) ? null : verdict;
+            }
+            : () => null,
         hasDialogRules: spec.detectDialog !== undefined,
     };
 }

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

@@ -0,0 +1,59 @@
+"use strict";
+/**
+ * Small builders for the status verdicts a tool detector returns (Issue #3219).
+ *
+ * They only assemble the `ToolStatusVerdict` object literal the detectors used to
+ * spell out by hand. Each builder produces exactly the fields its call sites
+ * wrote: the `promptDetection` key is present only for the builders that take
+ * one, and is never added as `undefined` where the original omitted it.
+ *
+ * Types and small functions only: no tool file and no `cli-patterns` import.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.positiveVerdict = positiveVerdict;
+exports.positiveVerdictWithPrompt = positiveVerdictWithPrompt;
+exports.activePromptVerdict = activePromptVerdict;
+exports.unreadVerdictWithPrompt = unreadVerdictWithPrompt;
+/** A verdict resting on something the detector positively recognised, no prompt of its own. */
+function positiveVerdict(status, reason) {
+    return {
+        status,
+        confidence: 'high',
+        reason,
+        hasActivePrompt: false,
+        evidence: 'positive',
+    };
+}
+/** {@link positiveVerdict} that also carries the branch's own `promptDetection` key. */
+function positiveVerdictWithPrompt(status, reason, promptDetection) {
+    return {
+        status,
+        confidence: 'high',
+        reason,
+        hasActivePrompt: false,
+        evidence: 'positive',
+        promptDetection,
+    };
+}
+/** `waiting` on a positively detected prompt (`hasActivePrompt: true`). */
+function activePromptVerdict(reason, promptDetection) {
+    return {
+        status: 'waiting',
+        confidence: 'high',
+        reason,
+        hasActivePrompt: true,
+        evidence: 'positive',
+        promptDetection,
+    };
+}
+/** `running` with low confidence and no evidence, carrying the `promptDetection` key. */
+function unreadVerdictWithPrompt(reason, promptDetection) {
+    return {
+        status: 'running',
+        confidence: 'low',
+        reason,
+        hasActivePrompt: false,
+        evidence: 'none',
+        promptDetection,
+    };
+}

package/dist/server/src/lib/detection/tools/vibe-local/patterns.js

@@ -0,0 +1,20 @@
+"use strict";
+/**
+ * Vibe Local 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.VIBE_LOCAL_THINKING_PATTERN = exports.VIBE_LOCAL_PROMPT_PATTERN = void 0;
+/**
+ * Vibe Local prompt pattern
+ * vibe-local (vibe-coder) shows `ctx:N% ❯` prompt when waiting for user input.
+ * The prompt line includes a context usage percentage prefix.
+ * Examples: "ctx:9% ❯", "ctx:30% ❯", "ctx:9% ❯ /model"
+ */
+exports.VIBE_LOCAL_PROMPT_PATTERN = /ctx:\d+%\s*[>❯]/m;
+/**
+ * Vibe Local thinking/processing pattern
+ * vibe-local shows spinner characters and status text while processing.
+ * Matches braille spinners, "Thinking", and tool execution indicators.
+ */
+exports.VIBE_LOCAL_THINKING_PATTERN = /[\u2800-\u28FF]|Thinking|⠋|⠙|⠹|⠸|⠼|⠴|⠦|⠧|⠇|⠏|Running|Executing/;

package/dist/server/src/lib/detection/truncate-raw-content.js

@@ -0,0 +1,38 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.RAW_CONTENT_MAX_CHARS = exports.RAW_CONTENT_MAX_LINES = void 0;
+exports.truncateRawContent = truncateRawContent;
+/**
+ * Maximum number of lines to retain in rawContent.
+ * Tail lines are preserved (instruction text typically appears just before the prompt).
+ * @see truncateRawContent
+ */
+exports.RAW_CONTENT_MAX_LINES = 200;
+/**
+ * Maximum number of characters to retain in rawContent.
+ * Tail characters are preserved.
+ * @see truncateRawContent
+ */
+exports.RAW_CONTENT_MAX_CHARS = 5000;
+/**
+ * Truncate raw content to fit within size limits.
+ * Preserves the tail (end) of the content since instruction text
+ * typically appears just before the prompt at the end of output.
+ *
+ * Security: No regular expressions used -- no ReDoS risk. [SF-S4-002]
+ * String.split('\n') and String.slice() are literal string operations only.
+ *
+ * @param content - The content to truncate
+ * @returns Truncated content (last RAW_CONTENT_MAX_LINES lines, max RAW_CONTENT_MAX_CHARS characters)
+ */
+function truncateRawContent(content) {
+    const lines = content.split('\n');
+    const truncatedLines = lines.length > exports.RAW_CONTENT_MAX_LINES
+        ? lines.slice(-exports.RAW_CONTENT_MAX_LINES)
+        : lines;
+    let result = truncatedLines.join('\n');
+    if (result.length > exports.RAW_CONTENT_MAX_CHARS) {
+        result = result.slice(-exports.RAW_CONTENT_MAX_CHARS);
+    }
+    return result;
+}

package/dist/server/src/lib/detection/turn-abandoned.js

@@ -0,0 +1,66 @@
+"use strict";
+/**
+ * Per tool, whether the screen may end a turn the agent's hooks opened
+ * (Issue #3337).
+ *
+ * The turn model lets three polls of a finished-looking frame close a turn as
+ * `scraper_evidence` (#1930). For codex 0.160.0 that closed live turns: frames
+ * of a running turn read `ready` (`tests/fixtures/codex-mid-turn-3337/`), and a
+ * reply sent on that `ready` interrupted the turn. So codex's hook turns end at
+ * the agent's own `Stop` — except on a frame that shows the turn was
+ * interrupted, after which no `Stop` comes.
+ *
+ * Only codex is listed, because only codex is measured to need it. Reading
+ * "this turn was abandoned" off a frame has to be complete to be safe — a form
+ * the reader misses leaves the turn `running` until the 30-minute stale bound —
+ * and Claude Code showed two forms in one afternoon (`⎿  Interrupted`, and an
+ * Esc early in the turn that puts the prompt back in the composer with no marker
+ * at all, `tests/fixtures/claude-interrupted-3337/`). A tool not listed keeps
+ * the #1930 behaviour: the screen closes its turn, hooks or not.
+ *
+ * Read by `lib/session/hook-turn-hold`, for both `capture --json` and the
+ * relay's readiness check.
+ *
+ * @module lib/detection/turn-abandoned
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.screenMayCloseHookTurn = screenMayCloseHookTurn;
+exports.screenShowsAbandonedHookTurn = screenShowsAbandonedHookTurn;
+const ansi_1 = require("./ansi");
+const patterns_1 = require("./tools/codex/patterns");
+/**
+ * The tools whose hook turns end at the agent's `Stop` rather than at the
+ * screen. Absent means "the screen may close it", as before #3337.
+ */
+const HOOK_TURN_SCREEN_POLICIES = {
+    // Measured on 0.160.0: mid-turn frames read `ready`; an Esc fires no `Stop`
+    // and leaves `■ Conversation interrupted` above the composer.
+    codex: { showsAbandonedTurn: patterns_1.isCodexTurnInterruptedFrame },
+};
+/**
+ * Whether the screen may end a turn this tool's hooks opened.
+ *
+ * True for a tool with no policy, and for a listed tool whose frame shows an
+ * abandoned turn.
+ *
+ * @param output - The pane as captured (ANSI is stripped here)
+ */
+function screenMayCloseHookTurn(cliToolId, output) {
+    const policy = HOOK_TURN_SCREEN_POLICIES[cliToolId];
+    if (!policy)
+        return true;
+    return policy.showsAbandonedTurn((0, ansi_1.stripAnsi)(output).split('\n'));
+}
+/**
+ * Whether the frame shows that a hook turn of a listed tool was abandoned
+ * (Issue #3377) — the same reading {@link screenMayCloseHookTurn} closes on,
+ * false for a tool with no policy.
+ *
+ * @param output - The pane as captured (ANSI is stripped here)
+ */
+function screenShowsAbandonedHookTurn(cliToolId, output) {
+    const policy = HOOK_TURN_SCREEN_POLICIES[cliToolId];
+    if (!policy)
+        return false;
+    return policy.showsAbandonedTurn((0, ansi_1.stripAnsi)(output).split('\n'));
+}

package/dist/server/src/lib/detection/unclassified-frame-tracker.js

@@ -20,6 +20,7 @@ exports.UNCLASSIFIED_RECORD_DWELL_MS = void 0;
 exports.observeUnclassifiedFrame = observeUnclassifiedFrame;
 exports.resetUnclassifiedFrameTracking = resetUnclassifiedFrameTracking;
 exports.unclassifiedFrameRunCount = unclassifiedFrameRunCount;
+const global_state_1 = require("../global-state");
 /**
  * How long a frame must stay unclassified before it is worth recording.
  *
@@ -29,8 +30,7 @@ exports.unclassifiedFrameRunCount = unclassifiedFrameRunCount;
  * stall the other never saw.
  */
 exports.UNCLASSIFIED_RECORD_DWELL_MS = 60_000;
-const runs = globalThis.__unclassifiedFrameRuns ??
-    (globalThis.__unclassifiedFrameRuns = new Map());
+const runs = (0, global_state_1.getOrInitGlobal)('__unclassifiedFrameRuns', () => new Map());
 /**
  * How long an unobserved run survives before it is dropped.
  *

package/dist/server/src/lib/global-state.js

@@ -0,0 +1,16 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.getOrInitGlobal = getOrInitGlobal;
+/**
+ * Process-wide state helper for the `globalThis.__*` cache pattern.
+ *
+ * Returns the value already stored on `globalThis[key]`, or runs `init` once and
+ * stores its result. `init` runs only when the slot is `null`/`undefined`, at the
+ * moment this function is called (so callers keep their own init timing).
+ *
+ * `key` and the initial value are typed from each file's `declare global`
+ * declaration. This module imports nothing so any layer (server, CLI) can use it.
+ */
+function getOrInitGlobal(key, init) {
+    return (globalThis[key] ??= init());
+}

package/dist/server/src/lib/hooks/agent-session-telemetry.js

@@ -92,6 +92,7 @@ exports.refreshAgentSessionContextUsage = refreshAgentSessionContextUsage;
 const auto_yes_state_1 = require("../../lib/auto-yes-state");
 const client_1 = require("./sources/opencode/client");
 const ports_1 = require("./sources/opencode/ports");
+const global_state_1 = require("../global-state");
 /**
  * Longest session title kept.
  *
@@ -109,8 +110,7 @@ exports.MAX_AGENT_SESSION_TITLE_LENGTH = 200;
  */
 exports.MAX_AGENT_SESSION_NAME_LENGTH = 120;
 /** compositeKey -> the most recent session description for that instance. */
-const records = globalThis.__agentSessionTelemetry ??
-    (globalThis.__agentSessionTelemetry = new Map());
+const records = (0, global_state_1.getOrInitGlobal)('__agentSessionTelemetry', () => new Map());
 /** Record what the agent last said about this instance's session. */
 function recordAgentSessionTelemetry(target, record) {
     records.set((0, auto_yes_state_1.buildCompositeKey)(target.worktreeId, target.cliToolId, target.instanceId), record);
@@ -238,11 +238,9 @@ function readOpencodeSessionInfo(value, at) {
     };
 }
 /** compositeKey -> the last context occupancy measured for that instance. */
-const contextUsages = globalThis.__agentSessionContextUsage ??
-    (globalThis.__agentSessionContextUsage = new Map());
+const contextUsages = (0, global_state_1.getOrInitGlobal)('__agentSessionContextUsage', () => new Map());
 /** compositeKeys with a refresh in flight, so a poll storm makes one request. */
-const contextRefreshes = globalThis.__agentSessionContextRefreshes ??
-    (globalThis.__agentSessionContextRefreshes = new Set());
+const contextRefreshes = (0, global_state_1.getOrInitGlobal)('__agentSessionContextRefreshes', () => new Set());
 /**
  * The percentage opencode's own footer shows, by opencode's own rule.
  *

package/dist/server/src/lib/hooks/permission-decision-state.js

@@ -35,9 +35,9 @@ exports.recordPermissionDecision = recordPermissionDecision;
 exports.getLastPermissionDecision = getLastPermissionDecision;
 exports.clearPermissionDecisions = clearPermissionDecisions;
 const auto_yes_state_1 = require("../../lib/auto-yes-state");
+const global_state_1 = require("../global-state");
 /** compositeKey -> the most recent adjudication for that session. */
-const lastDecisions = globalThis.__permissionDecisions ??
-    (globalThis.__permissionDecisions = new Map());
+const lastDecisions = (0, global_state_1.getOrInitGlobal)('__permissionDecisions', () => new Map());
 /**
  * Record how one approval was adjudicated.
  *