Skip to content

v1.17.1 — codex_approval pattern + stuck-detector coverage doc#54

Merged
amariichi merged 2 commits into
mainfrom
codex-approval-pattern
May 23, 2026
Merged

v1.17.1 — codex_approval pattern + stuck-detector coverage doc#54
amariichi merged 2 commits into
mainfrom
codex-approval-pattern

Conversation

@amariichi
Copy link
Copy Markdown
Owner

Summary

  • Add codex_approval regex to DEFAULT_STUCK_PATTERNS for Codex's
    "Would you like to run the following command?" shell-approval modal.
    v1.17.0 only caught Claude's "Do you want to proceed?" wording; that
    also covers Antigravity (same phrase) but missed Codex entirely.
  • Document the full pattern coverage (which CLI each regex matches, and
    what is intentionally not caught) in the minimum-headroom-ops skill.
  • Bump 1.17.0 → 1.17.1 across the six hardcoded sites.

Test plan

  • npm test — 359/359 pass locally (2 new tests added).
  • CI green on this PR.

🤖 Generated with Claude Code

amariichi and others added 2 commits May 23, 2026 20:48
v1.17.0 shipped the detector with one approval-modal pattern keyed on
Claude Code's "Do you want to proceed?" wording. Antigravity happens to
use the same phrase, so it was already covered, but Codex uses a
different opening ("Would you like to run the following command?") and
was silently missed.

Add a second approval pattern, `codex_approval`, with the Codex-specific
regex. Keep `claude_approval` as the existing id (it also matches agy,
which we now document).

Update the minimum-headroom-ops skill with a coverage table that lists
every shipping pattern, which CLI it matches, and what is intentionally
left out — so operators know when to extend `DEFAULT_STUCK_PATTERNS`
versus when to fall back to `agent.pane_snapshot` directly.

Tests:

- New: codex_approval pattern matches the Codex shell-command approval
  modal (sample taken from a real codex pane).
- New: claude_approval also catches the Antigravity permission modal
  (documents the existing behavior).
- Updated: DEFAULT_STUCK_PATTERNS id list now includes codex_approval.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Patch release for the codex_approval pattern addition. Six sites move
together: package.json, mcp-server/dist/index.js SERVER_VERSION,
tts-worker/pyproject.toml, asr-worker/pyproject.toml, and both uv.lock
files (refreshed via `uv lock`).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@amariichi amariichi merged commit e2dcd59 into main May 23, 2026
1 check passed
@amariichi amariichi deleted the codex-approval-pattern branch May 23, 2026 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant