Skip to content

feat(blocked-prompt): relay Codex tool approval dialogs to the phone - #32

Merged
tak-bro merged 3 commits into
mainfrom
feat/codex-approval
Nov 1, 2026
Merged

tak-bro merged 3 commits into
mainfrom
feat/codex-approval

Conversation

@tak-bro

@tak-bro tak-bro commented Nov 1, 2026

Copy link
Copy Markdown
Member

Summary

  • Read Codex (codex-cli 0.166.0) tool approval dialogs — shell command and file edit — into the sealed prompt summary so the phone shows the card and answers (parseCodexDialog, READERS.codex).
  • Answers are up/down + enter (already accepted by the iOS/Android decoders; no app change). Enter on "No" cancels exactly like esc (measured on a live codex 0.166.0).
  • New bundled rule codex-blocked-tool-approval (question + confirm footer) so the session reads blocked; idle guards verified to refuse the screen on their own. Bench rows for the three captures; README line updated.
  • Backlog F27 / C154 (Codex part).

Acceptance

AC-1..8 PASS (src/blocked-prompt-codex.test.ts, src/harness-bench.test.ts); AC-9: full vitest 5633 passed, 8 failed = src/http-proxy.test.ts, same on main.

Ordering / human to-do

  • Bundle version and PUBLISHED_FLOOR move to 2026.11.01.1. The hwip server manifest must get the same rule + version and be deployed before this merges/publishes (stale-ignore otherwise). Mirror PR is a separate assignment.
  • Not verified: MCP / permission-grant approval wording (reads unknown, not idle); Codex versions after 0.166.0.
  • Overlap: cli#30 (notify --auto) touches no shared file.

🤖 Generated with Claude Code

Parse codex-cli 0.166.0 shell-command and file-edit approvals into the sealed
summary (command or destination, reason, numbered answers) answered with
up/down + enter, which lands on No the way esc does.
Add the codex-blocked-tool-approval rule (question + confirm footer),
bench rows for the three captured dialogs, bundle version 2026.11.01.1
with the matching published floor, and the README line.
@tak-bro tak-bro self-assigned this Nov 1, 2026
@tak-bro
tak-bro marked this pull request as ready for review November 1, 2026 02:09
@tak-bro
tak-bro merged commit 2808854 into main Nov 1, 2026
4 checks passed
@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 0.1.16 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant