umputun / agterm / 29182352001

98%

push

github

feat: drive Codex agent-status from lifecycle hooks (#194)

Codex agent-status came only from codex-notify.sh, which guessed "waiting on
you" by keyword-matching the turn's final assistant message. It misfired both
ways: a real approval prompt fired no notification (notify runs only on
agent-turn-complete, so blocked never showed when it mattered), and an ordinary
turn whose final message merely mentioned "permission"/"approval" flipped to a
stuck blocked.

Codex now exposes lifecycle hooks, so the installer merges six of them into
~/.codex/config.toml (SessionStart->idle, UserPromptSubmit/PreToolUse/PostToolUse
->active, PermissionRequest->blocked, Stop->completed). PermissionRequest turns
the row blocked the moment Codex asks; Stop always sets completed with no text
guessing. codex-notify.sh is removed, and codex drops out of the shell
integration's default AGTERM_AGENT_RE since its hooks now drive finer per-turn
state, like Claude Code.

The config.toml merge parses the file with TOMLDecoder (pure-Swift, TOML 1.1
compliant) before touching it: it skips a config that already defines its own
hooks or doesn't parse (surfacing the block for a manual add), strips the old
notify line only when its parsed value points at codex-notify.sh, and otherwise
appends a marker-guarded block that preserves the user's comments and layout.
Both the Codex and Claude write paths now distinguish an absent config from one
that exists-but-unreadable, so a read failure no longer overwrites the file
without a backup.

Fix #193

70 of 71 new or added lines in 1 file covered. (98.59%)

4923 of 5035 relevant lines covered (97.78%)

5971303.48 hits per line