comisai 1.0.45 → 1.0.46

package/node_modules/@comis/skills/dist/tools/builtin/terminal-driver/terminal-classifier.d.ts

@@ -1,29 +1,29 @@
 /**
- * The pure terminal state classifier (spec §4.3, the #1 milestone de-risk).
+ * The pure terminal state classifier.
  *
  * `classifyFrame(frame, history)` labels a SETTLED frame
  * `working | awaiting-input | exited | stuck` deterministically, from the render
  * snapshot (grid + REAL cursor) + a caller-supplied progress history. It is the
- * perception the worker (124-05/06) drives and the `session_status` tool surfaces.
+ * perception the worker drives and the `session_status` tool surfaces.
  *
- * The §4.3 decision tree (in priority order):
+ * The decision tree (in priority order):
  *   1. `!alive`            → `exited`         (PTY exit — nothing more can render)
  *   2. `!settled`          → `working`        (output still flowing / cursor advancing)
  *   3. settled + diff∅ + CURSOR PARKED        → `awaiting-input`  (a real prompt, `high`)
- *   3b. settled + diff∅ + DIALOG STRUCTURE    → `awaiting-input`  (CLASS-01 — a full-
+ *   3b. settled + diff∅ + DIALOG STRUCTURE    → `awaiting-input`  (a full-
  *                                              screen dialog whose cursor sits on a blank
  *                                              input line BELOW the prompt block, so the
  *                                              parked gate missed it; `medium`,
  *                                              `dialog_detected`)
  *   4. settled + no-progress > stuckMs:
- *        4a. + a DIALOG/PROMPT affordance (regardless of diff∅) → `awaiting-input` (LIVE-02 —
+ *        4a. + a DIALOG/PROMPT affordance (regardless of diff∅) → `awaiting-input` (
  *            a settled prompt the STALE backgrounded-drive anchor mis-diffed as changing, so
  *            diff∅ was false and 3/3b were skipped; `medium`, `dialog_detected`)
- *        4b. else                                               → `stuck`  (by PROGRESS, OPS-04)
+ *        4b. else                                               → `stuck`  (by PROGRESS, never wall-clock)
  *   5. else                → `working`        (settled but cursor NOT parked = a
  *                                              thinking/tool-use pause)
  *
- * The LOAD-BEARING gate (spec §4.3 risk table, severity HIGH): {@link isCursorParked}.
+ * The LOAD-BEARING gate: {@link isCursorParked}.
  * During generation an AI CLI renders output mid-screen — the cursor is NOT in the
  * input box, so a thinking/tool-use pause is read as `working`, NEVER as
  * `awaiting-input`. A false `awaiting-input` would wake a turn that fires a spurious
@@ -32,9 +32,9 @@
  * gate (cursor position relative to the rendered content) is primary; the optional
  * operator `hintPatterns` are a positive REINFORCEMENT only — they never OVERRIDE the
  * structure, so a prompt-injecting CLI cannot render a fake "(y/n)" mid-screen and be
- * read as a prompt (T-124-06).
+ * read as a prompt.
  *
- * Architecture invariants (binding — AGENTS.md / 124 house style, mirrors
+ * Architecture invariants (binding — AGENTS.md; mirrors
  * `terminal-caps.ts`):
  *   - PURE: a free function, NOT a factory. NO clock/timer reads (the debounce
  *     timing lives in the settle the worker drives; the classifier receives an
@@ -46,15 +46,14 @@
  *     from `terminal-render.js` — no platform runtime packages, no observability
  *     egress, no raw timer (the globals + infra-runtime-scope architecture gates).
  *
- * Determinism: a pure function of its inputs ⇒ the Task-3 fixture corpus pins it
- * (a `claude` version bump that shifts a render is caught as a failing corpus case,
- * spec §10.4).
+ * Determinism: a pure function of its inputs ⇒ the fixture corpus pins it
+ * (a `claude` version bump that shifts a render is caught as a failing corpus case).
  *
  * @module
  */
 import type { EmulatorSnapshot } from "./terminal-render.js";
 import type { PlatformPerception } from "./platforms/index.js";
-/** The four mutually-exclusive session states (spec §4.3). */
+/** The four mutually-exclusive session states. */
 export type ClassifierState = "working" | "awaiting-input" | "exited" | "stuck";
 /**
  * The frame the classifier reads — composes the render {@link EmulatorSnapshot}
@@ -81,27 +80,27 @@ export interface ClassifierFrame {
     /**
      * OPTIONAL operator-configured prompt cues (e.g. `"❯"`, `"(y/n)"`). A positive
      * REINFORCEMENT for the parked gate only — never a hardcoded or screen-derived
-     * trust signal, and never enough to OVERRIDE the structural cursor-position test
-     * (T-124-06). Absent ⇒ the gate is purely structural.
+     * trust signal, and never enough to OVERRIDE the structural cursor-position test.
+     * Absent ⇒ the gate is purely structural.
      */
     hintPatterns?: readonly string[];
     /**
      * OPTIONAL selected-platform perception (the `TerminalPlatformProfile.perception` for the
-     * session's operator-declared allowId, fed by the worker — v2.26 CLASSIFY-01). The classifier
-     * stays the SOLE owner of `activity` (D4): these patterns FEED the generic decision —
+     * session's operator-declared allowId, fed by the worker). The classifier
+     * stays the SOLE owner of the activity decision: these patterns FEED the generic decision —
      * `workingLine` biases a settled-unparked frame WITH recent progress to `working` (the Codex
      * `Working (Ns)` / Claude spinner case); `menuOrPicker` + `promptAffordance` feed the structural
-     * dialog detector (the D5 v2.11 menu fix + LIVE-02 idle-`❯`). `turnEnd` is populated but reserved
-     * for the §6-v2 structured-perception layer (NOT routed into the activity decision — it would
+     * dialog detector (the full-screen-menu + idle-`❯` cases). `turnEnd` is populated but reserved
+     * for a future structured-perception layer (NOT routed into the activity decision — it would
      * over-fire on Claude's per-tool-action `⏺` bullet). Absent ⇒ the purely generic path,
-     * byte-identical to today (INV-1).
+     * byte-identical to the profile-free behavior.
      */
     perception?: PlatformPerception;
     /**
-     * OPTIONAL selected-platform dialog `detect` patterns (`profile.dialogs[].detect`, v2.26
-     * DIALOG-01) — the trust-gate / permission-prompt / approval-overlay signatures. The dialog
+     * OPTIONAL selected-platform dialog `detect` patterns (`profile.dialogs[].detect`)
+     * — the trust-gate / permission-prompt / approval-overlay signatures. The dialog
      * detector consumes these as awaiting-input cues (a profile dialog IS awaiting-input), additive
-     * to the generic structural detection. Absent ⇒ the generic path (INV-1). Detection only; the
+     * to the generic structural detection. Absent ⇒ the generic path. Detection only; the
      * safe-only auto-answer (`decideAutoAnswer`) makes the keystroke decision downstream.
      */
     dialogDetects?: readonly RegExp[];
@@ -110,7 +109,7 @@ export interface ClassifierFrame {
  * The progress history the caller supplies (the classifier is pure — it does NOT
  * read a clock). `noProgressMs` is how long the screen has shown no progress
  * (output/cursor/diff), measured by the worker against its injected clock; `stuckMs`
- * is the operator's stuck threshold (OPS-04). Stuck is by PROGRESS, never elapsed
+ * is the operator's stuck threshold. Stuck is by PROGRESS, never elapsed
  * wall-clock of the session.
  */
 export interface FrameHistory {
@@ -145,7 +144,7 @@ export interface Classification {
  *     (`lastNonBlankRow <= PARK_ROW_TOLERANCE`), a cursor sitting ABOVE the last
  *     non-blank row (content still below it) parks ONLY if an operator `hintPattern`
  *     matches — closing the no-op hole where a tiny mid-generation frame would
- *     otherwise spuriously park (WR-04). A cursor genuinely at the bottom row is fine.
+ *     otherwise spuriously park. A cursor genuinely at the bottom row is fine.
  *
  * `hintPatterns` can only REINFORCE the line-has-text leg; they cannot satisfy the
  * row/column structure on their own, so a fake mid-screen "(y/n)" is rejected.
@@ -165,8 +164,8 @@ export declare function isCursorParked(cursor: {
     y: number;
 }, screen: string, _cols: number, _rows: number, hintPatterns?: readonly string[]): boolean;
 /**
- * Classify a frame into `working | awaiting-input | exited | stuck` per the spec
- * §4.3 decision tree. Pure, total, never throws — biases to the SAFE direction
+ * Classify a frame into `working | awaiting-input | exited | stuck` per the
+ * decision tree above. Pure, total, never throws — biases to the SAFE direction
  * (`working`) whenever the prompt structure is not unambiguous.
  *
  * @param frame - The settle/diff signals + the render snapshot.

package/node_modules/@comis/skills/dist/tools/builtin/terminal-driver/terminal-classifier.js

@@ -1,30 +1,30 @@
 // SPDX-License-Identifier: Apache-2.0
 /**
- * The pure terminal state classifier (spec §4.3, the #1 milestone de-risk).
+ * The pure terminal state classifier.
  *
  * `classifyFrame(frame, history)` labels a SETTLED frame
  * `working | awaiting-input | exited | stuck` deterministically, from the render
  * snapshot (grid + REAL cursor) + a caller-supplied progress history. It is the
- * perception the worker (124-05/06) drives and the `session_status` tool surfaces.
+ * perception the worker drives and the `session_status` tool surfaces.
  *
- * The §4.3 decision tree (in priority order):
+ * The decision tree (in priority order):
  *   1. `!alive`            → `exited`         (PTY exit — nothing more can render)
  *   2. `!settled`          → `working`        (output still flowing / cursor advancing)
  *   3. settled + diff∅ + CURSOR PARKED        → `awaiting-input`  (a real prompt, `high`)
- *   3b. settled + diff∅ + DIALOG STRUCTURE    → `awaiting-input`  (CLASS-01 — a full-
+ *   3b. settled + diff∅ + DIALOG STRUCTURE    → `awaiting-input`  (a full-
  *                                              screen dialog whose cursor sits on a blank
  *                                              input line BELOW the prompt block, so the
  *                                              parked gate missed it; `medium`,
  *                                              `dialog_detected`)
  *   4. settled + no-progress > stuckMs:
- *        4a. + a DIALOG/PROMPT affordance (regardless of diff∅) → `awaiting-input` (LIVE-02 —
+ *        4a. + a DIALOG/PROMPT affordance (regardless of diff∅) → `awaiting-input` (
  *            a settled prompt the STALE backgrounded-drive anchor mis-diffed as changing, so
  *            diff∅ was false and 3/3b were skipped; `medium`, `dialog_detected`)
- *        4b. else                                               → `stuck`  (by PROGRESS, OPS-04)
+ *        4b. else                                               → `stuck`  (by PROGRESS, never wall-clock)
  *   5. else                → `working`        (settled but cursor NOT parked = a
  *                                              thinking/tool-use pause)
  *
- * The LOAD-BEARING gate (spec §4.3 risk table, severity HIGH): {@link isCursorParked}.
+ * The LOAD-BEARING gate: {@link isCursorParked}.
  * During generation an AI CLI renders output mid-screen — the cursor is NOT in the
  * input box, so a thinking/tool-use pause is read as `working`, NEVER as
  * `awaiting-input`. A false `awaiting-input` would wake a turn that fires a spurious
@@ -33,9 +33,9 @@
  * gate (cursor position relative to the rendered content) is primary; the optional
  * operator `hintPatterns` are a positive REINFORCEMENT only — they never OVERRIDE the
  * structure, so a prompt-injecting CLI cannot render a fake "(y/n)" mid-screen and be
- * read as a prompt (T-124-06).
+ * read as a prompt.
  *
- * Architecture invariants (binding — AGENTS.md / 124 house style, mirrors
+ * Architecture invariants (binding — AGENTS.md; mirrors
  * `terminal-caps.ts`):
  *   - PURE: a free function, NOT a factory. NO clock/timer reads (the debounce
  *     timing lives in the settle the worker drives; the classifier receives an
@@ -47,9 +47,8 @@
  *     from `terminal-render.js` — no platform runtime packages, no observability
  *     egress, no raw timer (the globals + infra-runtime-scope architecture gates).
  *
- * Determinism: a pure function of its inputs ⇒ the Task-3 fixture corpus pins it
- * (a `claude` version bump that shifts a render is caught as a failing corpus case,
- * spec §10.4).
+ * Determinism: a pure function of its inputs ⇒ the fixture corpus pins it
+ * (a `claude` version bump that shifts a render is caught as a failing corpus case).
  *
  * @module
  */
@@ -86,7 +85,7 @@ const PARK_ROW_TOLERANCE = 1;
  *     (`lastNonBlankRow <= PARK_ROW_TOLERANCE`), a cursor sitting ABOVE the last
  *     non-blank row (content still below it) parks ONLY if an operator `hintPattern`
  *     matches — closing the no-op hole where a tiny mid-generation frame would
- *     otherwise spuriously park (WR-04). A cursor genuinely at the bottom row is fine.
+ *     otherwise spuriously park. A cursor genuinely at the bottom row is fine.
  *
  * `hintPatterns` can only REINFORCE the line-has-text leg; they cannot satisfy the
  * row/column structure on their own, so a fake mid-screen "(y/n)" is rejected.
@@ -126,7 +125,7 @@ export function isCursorParked(cursor, screen, _cols, _rows, hintPatterns = [])
     const cursorLine = lines[cursor.y] ?? "";
     const lineHasText = cursorLine.trim().length > 0;
     const hintMatches = hintPatterns.some((p) => p.length > 0 && cursorLine.includes(p));
-    // WR-04: the lower-bound "cursor mid-screen ABOVE content" rejection above is
+    // The lower-bound "cursor mid-screen ABOVE content" rejection above is
     // VACUOUS on a short screen — when `lastNonBlankRow <= PARK_ROW_TOLERANCE` the
     // threshold `lastNonBlankRow - PARK_ROW_TOLERANCE` is ≤ 0, so a cursor sitting
     // ABOVE the last non-blank row (the thinking-pause shape: content still rendered
@@ -151,15 +150,15 @@ export function isCursorParked(cursor, screen, _cols, _rows, hintPatterns = [])
     return true;
 }
 // ---------------------------------------------------------------------------
-// The classifier (the §4.3 decision tree)
+// The classifier (the decision tree)
 // ---------------------------------------------------------------------------
 /** True iff any pattern matches the text — a selected-platform perception list. Empty/undefined ⇒ false. */
 function matchesAnyPattern(text, patterns) {
     return patterns !== undefined && patterns.some((re) => re.test(text));
 }
 /**
- * Classify a frame into `working | awaiting-input | exited | stuck` per the spec
- * §4.3 decision tree. Pure, total, never throws — biases to the SAFE direction
+ * Classify a frame into `working | awaiting-input | exited | stuck` per the
+ * decision tree above. Pure, total, never throws — biases to the SAFE direction
  * (`working`) whenever the prompt structure is not unambiguous.
  *
  * @param frame - The settle/diff signals + the render snapshot.
@@ -167,15 +166,15 @@ function matchesAnyPattern(text, patterns) {
  * @returns The typed {@link Classification}.
  */
 export function classifyFrame(frame, history) {
-    // CLASSIFY-01: the selected platform profile's awaiting-input affordance patterns (or none — the
+    // The selected platform profile's awaiting-input affordance patterns (or none — the
     // generic path). `menuOrPicker` + `promptAffordance` FEED the structural dialog detector; the
-    // classifier remains the sole owner of `activity` (D4). `turnEnd` is deliberately EXCLUDED here
-    // (review WR-01): Claude's `⏺` turn bullet is also its per-tool-action bullet, so feeding it would
+    // classifier remains the sole owner of the activity decision. `turnEnd` is deliberately EXCLUDED
+    // here: Claude's `⏺` turn bullet is also its per-tool-action bullet, so feeding it would
     // over-fire awaiting-input on a mid-turn pause — the idle `❯` (promptAffordance) is the real cue;
-    // `turnEnd` stays populated for the §6-v2 structured-perception layer. Empty when no profile (INV-1).
+    // `turnEnd` stays populated for a future structured-perception layer. Empty when no profile.
     const perceptionAffordances = [
         ...(frame.perception ? [...(frame.perception.menuOrPicker ?? []), ...(frame.perception.promptAffordance ?? [])] : []),
-        ...(frame.dialogDetects ?? []), // DIALOG-01: a profile dialog (trust/permission/approval) IS awaiting-input
+        ...(frame.dialogDetects ?? []), // a profile dialog (trust/permission/approval) IS awaiting-input
     ];
     // 1. PTY exit — terminal; nothing more can render.
     if (!frame.alive) {
@@ -193,20 +192,20 @@ export function classifyFrame(frame, history) {
     if (frame.diffEmpty && parked) {
         return { state: "awaiting-input", confidence: "high", reason: "settled_cursor_parked" };
     }
-    // 3.5. CLASSIFY-01: a SELECTED-platform working-line indicator (Claude spinner glyph+gerund /
+    // 3.5. A SELECTED-platform working-line indicator (Claude spinner glyph+gerund /
     //      Codex `Working (Ns)`) on a settled-but-UNPARKED frame that has made progress WITHIN the
     //      stuck window means the CLI is mid-work — a render that briefly stopped, NOT a prompt or a
     //      hang. Bias to `working` (pre-empts the dialog branch below). GATED on
-    //      `noProgressMs <= stuckMs` (review WR-02): a frame static for the WHOLE stuck window is hung
+    //      `noProgressMs <= stuckMs`: a frame static for the WHOLE stuck window is hung
     //      regardless of a leftover spinner glyph — letting it fall through to the stuck branch closes
     //      the hang-suppression hole (the daemon backstop derives `stuck` from this verdict and has no
     //      independent wall-clock timeout). A genuinely parked prompt already won at step 3; with no
-    //      profile this is a no-op (INV-1).
+    //      profile this is a no-op.
     if (history.noProgressMs <= history.stuckMs &&
         matchesAnyPattern(frame.snapshot.screen, frame.perception?.workingLine)) {
         return { state: "working", confidence: "medium", reason: "working_line" };
     }
-    // 3b. CLASS-01: a settled, diff∅ frame whose STRUCTURE is unmistakably a full-screen
+    // 3b. A settled, diff∅ frame whose STRUCTURE is unmistakably a full-screen
     //     dialog/menu — even though the cursor is NOT parked. This is the documented
     //     claude-2.1.x shape: the prompt block (a box / an enumerated menu / a selector)
     //     renders ABOVE and the cursor sits on a blank input line BELOW it, so
@@ -214,28 +213,28 @@ export function classifyFrame(frame, history) {
     //     to `stuck`. The predicate is pure + structural + CLI-agnostic; `hintPatterns`
     //     reinforce a borderline selector only. Confidence is `medium` (the structural
     //     certainty of a parked cursor is `high`; this is the heuristic dialog branch).
-    //     No new classifier state — reuses `awaiting-input`. SEC-12 escalate-always still
-    //     gates the actual answer downstream (a dialog_detected frame routes through the
-    //     same decideAutoAnswer the wake-turn calls — I4 no-bypass).
+    //     No new classifier state — reuses `awaiting-input`. The escalate-always gate still
+    //     vets the actual answer downstream (a dialog_detected frame routes through the
+    //     same decideAutoAnswer the wake-turn calls — no bypass).
     if (frame.diffEmpty && detectsFullScreenDialog(frame.snapshot, frame.hintPatterns, perceptionAffordances)) {
         return { state: "awaiting-input", confidence: "medium", reason: "dialog_detected" };
     }
     // 4. Settled, no progress past the stuck window. Before declaring a hang, re-check the
-    //    interactive-affordance STRUCTURE — INDEPENDENT of `diffEmpty` (LIVE-02). The `diffEmpty`
+    //    interactive-affordance STRUCTURE — INDEPENDENT of `diffEmpty`. The `diffEmpty`
     //    gate on the awaiting-input branches (3/3b) keys on the attention emitter's edge-trigger
     //    anchor, which goes STALE for a backgrounded, idle drive: the worker runs settles only on
-    //    OUTPUT, so once a drive is promoted (DRIVE-02) and the CLI falls quiet, no settle
+    //    OUTPUT, so once a drive is promoted and the CLI falls quiet, no settle
     //    re-advances `lastClassifiedSnapshot`/`lastProgressMs`. The liveness backstop's
-    //    point-in-time `status` query (LIVE-01) then diffs the CURRENT settled prompt against that
+    //    point-in-time `status` query then diffs the CURRENT settled prompt against that
     //    stale baseline → `diffEmpty=false` → branches 3/3b are skipped → the frame falls here and
-    //    is mislabeled `stuck`, and the backstop re-escalates it every tick (real-VPS 2026-06-16:
+    //    is mislabeled `stuck`, and the backstop re-escalates it every tick (observed live:
     //    claude's idle `❯` input box with a status footer BELOW the cursor — isCursorParked missed
     //    it (footer below) AND the stale-anchor diff made diffEmpty=false). But noProgressMs >
     //    stuckMs PROVES the screen has been static for the WHOLE window, so a detected
     //    dialog/prompt/selector affordance is a SETTLED prompt awaiting input, NOT a hang (a
     //    still-GENERATING CLI emits output → progress → never reaches this branch). Only a static
-    //    frame with NO affordance is genuinely stuck. SEC-12 escalate-always still gates the answer
-    //    downstream (this routes through the SAME decideAutoAnswer as step 3b — I4 no-bypass).
+    //    frame with NO affordance is genuinely stuck. The escalate-always gate still vets the answer
+    //    downstream (this routes through the SAME decideAutoAnswer as step 3b — no bypass).
     if (history.noProgressMs > history.stuckMs) {
         if (detectsFullScreenDialog(frame.snapshot, frame.hintPatterns, perceptionAffordances)) {
             return { state: "awaiting-input", confidence: "medium", reason: "dialog_detected" };

package/node_modules/@comis/skills/dist/tools/builtin/terminal-driver/terminal-dialog-detector.d.ts

@@ -1,18 +1,17 @@
 /**
- * The pure structural full-screen-dialog predicate (CLASS-01; design §4 Phase A,
- * §7.1.3 STRUCTURAL-ONLY).
+ * The pure structural full-screen-dialog predicate (STRUCTURAL-ONLY).
  *
  * `detectsFullScreenDialog(snapshot, hintPatterns?)` answers ONE question over the
  * rendered grid: is this frame, by STRUCTURE alone, an interactive full-screen dialog
  * (a boxed prompt, an enumerated option menu, or a genuine selector affordance)? It
- * exists to close the documented claude-2.1.x misread (`project_v211_classifier_claude_menus_stuck`):
+ * exists to close the documented claude-2.1.x misread:
  * a full-screen permission/menu dialog whose cursor sits on a blank input line BELOW
  * the prompt block makes {@link isCursorParked} (correctly) return `false`, so
  * {@link classifyFrame} used to fall through to `stuck`. This predicate lets the
  * classifier read such a settled, diff-empty frame as `awaiting-input` instead.
  *
- * CLI-AGNOSTIC by design (§7.1.3): it is a STRUCTURAL test, NOT a per-CLI pattern
- * table (that brittleness is exactly what the CLASS-02 fixture corpus guards against).
+ * CLI-AGNOSTIC by design: it is a STRUCTURAL test, NOT a per-CLI pattern
+ * table (that brittleness is exactly what the fixture corpus guards against).
  * It fires `true` when ANY ONE strong structural cue is present over the rows:
  *   - a box-drawing run (`╭╮╰╯│─…`, an ASCII `+--+` corner/edge, or a PREDOMINANTLY
  *     border-fill `|----|` row — NOT an arbitrary `| prose | prose |` markdown row), OR
@@ -22,7 +21,7 @@
  *   - a genuine selector affordance: a `❯` caret, or a `(y/n)`/`[Y/n]` confirmation
  *     token that is a STANDALONE END-OF-LINE affordance (not a substring mid-prose).
  *
- * TIGHT by design (RESEARCH Pitfall 2 / T-163-02 — the severity-HIGH failure; MR-01):
+ * TIGHT by design — an over-broad match is the severity-HIGH failure mode:
  * mere indentation, bullets (`●`/`⎿`), a stray `>` quote, a markdown table row
  * (`| col | col |`), a numbered prose list that keeps generating, or a `(y/n)` token
  * buried mid-sentence in generation prose must NOT match — else a completed/thinking
@@ -31,17 +30,17 @@
  *
  * `hintPatterns` (the operator-configured cues, NEVER model/screen-derived) REINFORCE a
  * BORDERLINE selector match only — a hintPattern present on prose with no structural cue
- * does NOT force `true`. Structure is primary (T-124-06): a prompt-injecting CLI cannot
+ * does NOT force `true`. Structure is primary: a prompt-injecting CLI cannot
  * phish a keystroke by echoing the operator's cue mid-generation.
  *
  * Caller contract: {@link classifyFrame} gates this on `frame.diffEmpty` and reaches it
  * only AFTER `isCursorParked` returned `false`, so a STATIC settled old menu IS
  * `awaiting-input` BY DESIGN (a settled menu is awaiting input); "hung" means NO
  * structure (no box/menu/selector), which still falls through to the `stuck`-by-progress
- * branch. PARK_ROW_TOLERANCE / isCursorParked are NOT touched — CLASS-01 ADDS a branch,
- * it never weakens the load-bearing gate.
+ * branch. PARK_ROW_TOLERANCE / isCursorParked are NOT touched — this predicate ADDS a
+ * branch, it never weakens the load-bearing gate.
  *
- * Architecture invariants (binding — AGENTS.md / 124 house style, mirrors
+ * Architecture invariants (binding — AGENTS.md; mirrors
  * `terminal-classifier.ts` / `terminal-auto-answer.ts`):
  *   - PURE: a free function, NOT a factory. NO clock/timer reads, NO module-global
  *     mutable state, NO I/O.
@@ -56,7 +55,7 @@
  */
 import type { EmulatorSnapshot } from "./terminal-render.js";
 /**
- * Is this frame, by STRUCTURE, a full-screen interactive dialog? — CLASS-01.
+ * Is this frame, by STRUCTURE, a full-screen interactive dialog?
  *
  * Fires `true` iff ANY strong structural cue is present over the rendered rows:
  * a box-drawing/ASCII-border row, OR ≥2 enumerated option rows, OR a genuine selector
@@ -67,11 +66,11 @@ import type { EmulatorSnapshot } from "./terminal-render.js";
  *   `snapshot.screen` (split on `"\n"`) is read; the cursor is the caller's concern
  *   (this branch is reached only when the cursor is NOT parked).
  * @param hintPatterns - Optional operator prompt cues (reinforcement only — never
- *   enough to satisfy structure alone; preserves T-124-06).
+ *   enough to satisfy structure alone; structure stays primary).
  * @param perceptionAffordances - Optional SELECTED-platform perception patterns
  *   (`menuOrPicker`/`promptAffordance`/`turnEnd` from the profile, fed by the classifier).
  *   A rendered line matching any is a dialog cue — developer-authored + operator-selected
- *   by allowId (not screen-derived, T-124-06-safe); empty ⇒ the generic structural path.
+ *   by allowId (not screen-derived, so unphishable); empty ⇒ the generic structural path.
  * @returns `true` iff the rendered structure is unmistakably a dialog/menu.
  */
 export declare function detectsFullScreenDialog(snapshot: EmulatorSnapshot, hintPatterns?: readonly string[], perceptionAffordances?: readonly RegExp[]): boolean;

package/node_modules/@comis/skills/dist/tools/builtin/terminal-driver/terminal-dialog-detector.js

@@ -1,19 +1,18 @@
 // SPDX-License-Identifier: Apache-2.0
 /**
- * The pure structural full-screen-dialog predicate (CLASS-01; design §4 Phase A,
- * §7.1.3 STRUCTURAL-ONLY).
+ * The pure structural full-screen-dialog predicate (STRUCTURAL-ONLY).
  *
  * `detectsFullScreenDialog(snapshot, hintPatterns?)` answers ONE question over the
  * rendered grid: is this frame, by STRUCTURE alone, an interactive full-screen dialog
  * (a boxed prompt, an enumerated option menu, or a genuine selector affordance)? It
- * exists to close the documented claude-2.1.x misread (`project_v211_classifier_claude_menus_stuck`):
+ * exists to close the documented claude-2.1.x misread:
  * a full-screen permission/menu dialog whose cursor sits on a blank input line BELOW
  * the prompt block makes {@link isCursorParked} (correctly) return `false`, so
  * {@link classifyFrame} used to fall through to `stuck`. This predicate lets the
  * classifier read such a settled, diff-empty frame as `awaiting-input` instead.
  *
- * CLI-AGNOSTIC by design (§7.1.3): it is a STRUCTURAL test, NOT a per-CLI pattern
- * table (that brittleness is exactly what the CLASS-02 fixture corpus guards against).
+ * CLI-AGNOSTIC by design: it is a STRUCTURAL test, NOT a per-CLI pattern
+ * table (that brittleness is exactly what the fixture corpus guards against).
  * It fires `true` when ANY ONE strong structural cue is present over the rows:
  *   - a box-drawing run (`╭╮╰╯│─…`, an ASCII `+--+` corner/edge, or a PREDOMINANTLY
  *     border-fill `|----|` row — NOT an arbitrary `| prose | prose |` markdown row), OR
@@ -23,7 +22,7 @@
  *   - a genuine selector affordance: a `❯` caret, or a `(y/n)`/`[Y/n]` confirmation
  *     token that is a STANDALONE END-OF-LINE affordance (not a substring mid-prose).
  *
- * TIGHT by design (RESEARCH Pitfall 2 / T-163-02 — the severity-HIGH failure; MR-01):
+ * TIGHT by design — an over-broad match is the severity-HIGH failure mode:
  * mere indentation, bullets (`●`/`⎿`), a stray `>` quote, a markdown table row
  * (`| col | col |`), a numbered prose list that keeps generating, or a `(y/n)` token
  * buried mid-sentence in generation prose must NOT match — else a completed/thinking
@@ -32,17 +31,17 @@
  *
  * `hintPatterns` (the operator-configured cues, NEVER model/screen-derived) REINFORCE a
  * BORDERLINE selector match only — a hintPattern present on prose with no structural cue
- * does NOT force `true`. Structure is primary (T-124-06): a prompt-injecting CLI cannot
+ * does NOT force `true`. Structure is primary: a prompt-injecting CLI cannot
  * phish a keystroke by echoing the operator's cue mid-generation.
  *
  * Caller contract: {@link classifyFrame} gates this on `frame.diffEmpty` and reaches it
  * only AFTER `isCursorParked` returned `false`, so a STATIC settled old menu IS
  * `awaiting-input` BY DESIGN (a settled menu is awaiting input); "hung" means NO
  * structure (no box/menu/selector), which still falls through to the `stuck`-by-progress
- * branch. PARK_ROW_TOLERANCE / isCursorParked are NOT touched — CLASS-01 ADDS a branch,
- * it never weakens the load-bearing gate.
+ * branch. PARK_ROW_TOLERANCE / isCursorParked are NOT touched — this predicate ADDS a
+ * branch, it never weakens the load-bearing gate.
  *
- * Architecture invariants (binding — AGENTS.md / 124 house style, mirrors
+ * Architecture invariants (binding — AGENTS.md; mirrors
  * `terminal-classifier.ts` / `terminal-auto-answer.ts`):
  *   - PURE: a free function, NOT a factory. NO clock/timer reads, NO module-global
  *     mutable state, NO I/O.
@@ -56,7 +55,7 @@
  * @module
  */
 // ---------------------------------------------------------------------------
-// Structural-cue matchers — STRONG cues only (tight by design, T-163-02).
+// Structural-cue matchers — STRONG cues only (tight by design).
 // ---------------------------------------------------------------------------
 /**
  * A bordered/boxed region: any Unicode box-drawing glyph OR an ASCII border row. A box
@@ -67,7 +66,7 @@ const BOX_DRAWING = /[╭╮╰╯│─└┌┐┘├┤┬┴┼]/;
  * An ASCII border row: a `+` with `-`/`=` runs (`+----+` corners/edges), OR a `|`-bounded
  * row that is PREDOMINANTLY BORDER FILL — only border glyphs (`-`/`=`/`+`/`|`) and spaces
  * between the outer pipes (a `|------|` rule or a blank `|      |` box edge). The second
- * alternative is INTENTIONALLY NOT `^\s*\|.*\|\s*$` (MR-01): that earlier form matched
+ * alternative is INTENTIONALLY NOT `^\s*\|.*\|\s*$`: that broader form matched
  * EVERY pipe-bounded row — so a markdown table row (`| Option | Cost |`) or a single
  * pipe-bounded ascii-art line (`|  A --> B  |`) in generation output was misread as
  * dialog chrome. A real box's `+---+` top/bottom border still fires on the first
@@ -80,7 +79,7 @@ const ASCII_BORDER = /(?:\+[-=]{2,}\+)|(?:^\s*\|[-=+|\s]*\|\s*$)/;
  * Anchored at line start (after optional indent) so a `step 1.` mid-sentence in prose
  * does NOT match — only a genuine list item. The caller requires ≥2 such rows AND that
  * they be the TRAILING structure (no prose below the last option) before treating them
- * as a menu — a numbered prose list that continues with a sentence is NOT a menu (MR-01).
+ * as a menu — a numbered prose list that continues with a sentence is NOT a menu.
  */
 // eslint-disable-next-line security/detect-unsafe-regex -- linear; no nested/overlapping quantifier (single anchored `\s*` + an unambiguous alternation + a lone `\s+\S`; verified <0.2ms on 100k pathological input — not ReDoS).
 const ENUMERATOR = /^\s*(?:[❯>]\s*)?(?:\d+[.)]|\[\d+\]|\([a-z]\))\s+\S/;
@@ -88,8 +87,8 @@ const ENUMERATOR = /^\s*(?:[❯>]\s*)?(?:\d+[.)]|\[\d+\]|\([a-z]\))\s+\S/;
  * A genuine selector affordance: a `❯` caret at a word boundary (`(?:^|\s)❯`), OR a
  * `(y/n)`/`(yes/no)`/`[y/n]`/`[Y/n]` confirmation token that is a STANDALONE END-OF-LINE
  * affordance (the token, then only an optional trailing prompt char `?`/`:`/`>` and
- * whitespace, then end-of-line). The end-of-line anchor is load-bearing (MR-01/MR-02):
- * the earlier alternatives had NO position anchor, so a `(yes/no)` / `[y/n]` token buried
+ * whitespace, then end-of-line). The end-of-line anchor is load-bearing:
+ * an unanchored alternative would let a `(yes/no)` / `[y/n]` token buried
  * mid-sentence in generation prose matched anywhere on the line and was misread as a
  * prompt. A real confirmation prompt always carries the token as the trailing affordance
  * (`Overwrite? (y/n)`), so anchoring it to end-of-line keeps every genuine prompt while
@@ -130,7 +129,7 @@ function isHintAffordanceLine(trimmed, hintPatterns) {
 // The predicate
 // ---------------------------------------------------------------------------
 /**
- * Is this frame, by STRUCTURE, a full-screen interactive dialog? — CLASS-01.
+ * Is this frame, by STRUCTURE, a full-screen interactive dialog?
  *
  * Fires `true` iff ANY strong structural cue is present over the rendered rows:
  * a box-drawing/ASCII-border row, OR ≥2 enumerated option rows, OR a genuine selector
@@ -141,11 +140,11 @@ function isHintAffordanceLine(trimmed, hintPatterns) {
  *   `snapshot.screen` (split on `"\n"`) is read; the cursor is the caller's concern
  *   (this branch is reached only when the cursor is NOT parked).
  * @param hintPatterns - Optional operator prompt cues (reinforcement only — never
- *   enough to satisfy structure alone; preserves T-124-06).
+ *   enough to satisfy structure alone; structure stays primary).
  * @param perceptionAffordances - Optional SELECTED-platform perception patterns
  *   (`menuOrPicker`/`promptAffordance`/`turnEnd` from the profile, fed by the classifier).
  *   A rendered line matching any is a dialog cue — developer-authored + operator-selected
- *   by allowId (not screen-derived, T-124-06-safe); empty ⇒ the generic structural path.
+ *   by allowId (not screen-derived, so unphishable); empty ⇒ the generic structural path.
  * @returns `true` iff the rendered structure is unmistakably a dialog/menu.
  */
 export function detectsFullScreenDialog(snapshot, hintPatterns = [], perceptionAffordances = []) {
@@ -172,7 +171,7 @@ export function detectsFullScreenDialog(snapshot, hintPatterns = [], perceptionA
         // an immediate cue.
         if (SELECTOR.test(line))
             sawSelector = true;
-        // v2.26 CLASSIFY-01/D5: a SELECTED platform profile's perception affordance (a
+        // A SELECTED platform profile's perception affordance (a
         // `menuOrPicker`/`promptAffordance`/`turnEnd` pattern — e.g. Claude's `Select Model`
         // picker or `Esc to cancel`) the generic structural cues miss. These are
         // developer-authored + operator-selected-by-allowId (not screen/model-derived, so a
@@ -182,14 +181,14 @@ export function detectsFullScreenDialog(snapshot, hintPatterns = [], perceptionA
             sawPerceptionAffordance = true;
         }
         // Count enumerated option rows (a single one is prose, "step 1."). The ≥2 check +
-        // the trailing-structure test below are what promote them to a menu (MR-01). Capture
+        // the trailing-structure test below are what promote them to a menu. Capture
         // the tail after each hit so the last enumerator's tail is what survives the loop.
         if (ENUMERATOR.test(line)) {
             enumeratorRows++;
             rowsAfterLastEnumerator = lines.slice(lineIndex);
         }
         // hintPatterns REINFORCE only — and ONLY as a borderline selector affordance, never
-        // as a free substring in prose (T-124-06: a CLI must not phish a keystroke by echoing
+        // as a free substring in prose (a CLI must not phish a keystroke by echoing
         // the operator's cue mid-generation). A real affordance carries the cue at the END of
         // a SHORT prompt line (e.g. "Overwrite file? proceed?"), so reinforce iff the matched
         // operator cue sits at the trailing affordance position of a short line — never buried
@@ -200,7 +199,7 @@ export function detectsFullScreenDialog(snapshot, hintPatterns = [], perceptionA
     }
     // ≥2 line-start enumerators are a MENU only when the options are the TRAILING
     // structure — i.e. nothing but blank rows, more enumerators, or border rows appear
-    // BELOW the last option (MR-01). A *completed* response that ends in a NUMBERED PROSE
+    // BELOW the last option. A *completed* response that ends in a NUMBERED PROSE
     // LIST keeps generating prose after the items ("…and that completes it"), so a
     // non-enumerator prose line below the last enumerated row demotes it back to prose.
     // (A boxed menu already returned `true` above on its `+---+` border, so this only
@@ -216,8 +215,8 @@ export function detectsFullScreenDialog(snapshot, hintPatterns = [], perceptionA
  * Are `rowsBelow` (the rows strictly AFTER the last enumerated option) all non-prose —
  * i.e. blank, another enumerator, or a box/ASCII border? `true` means the enumerated
  * options are the TRAILING structure of the frame (a real menu); `false` means
- * generation prose continues past the last option (a numbered prose list, NOT a menu —
- * MR-01). Pure; never throws (an empty tail — the options ARE the last rows — yields
+ * generation prose continues past the last option (a numbered prose list, NOT a
+ * menu). Pure; never throws (an empty tail — the options ARE the last rows — yields
  * `true`).
  */
 function isTrailingStructure(rowsBelow) {