umputun / ralphex / 26265222277

83%

Build Type

push

github

Committed by web-flow

Commit Message

Add first-class codex executor mode (#350)

* docs: add implementation plan for --codex first-class mode

Plan for promoting CodexExecutor from external-review-only to a
peer of ClaudeExecutor across task, review, and finalize phases.
Adds --codex and --pass-claude-md flags, role-named Executors
struct (Task/Review/External/Custom), codex-flavored review prompts,
and additive -c flag overrides that preserve user's ~/.codex config.

13 tasks. Survived smells pre-check, plan-review pass, and an
interactive revdiff round that pivoted away from the original
CODEX_HOME temp-directory approach.

* feat(codex): add first-class --codex executor mode

Adds --codex CLI flag and matching `executor = codex` config option that
runs the full pipeline (task, internal review, finalize) through the codex
CLI instead of Claude Code. Motivated by the June 2026 Anthropic billing
split that moves unattended ralphex runs off the Claude Max plan; gives
OpenAI/codex-plan users a single flag to keep the full pipeline on their
existing subscription.

Highlights:
- New `--codex` and `--pass-claude-md` CLI flags plus matching `executor`
  and `pass_claude_md` config keys, with mutex validation against
  `--external-only`, `--codex-only`, and `--external-review-tool != none`.
- Role-named executor wiring: `Executors` struct switches from
  `Claude`/`ReviewClaude`/`Codex` to `Task`/`Review`/`External`/`Custom`.
  Under `--codex` the constructor wires the same shared `CodexExecutor`
  instance into both `Task` and `Review` so any phase can use
  `{{agent:...}}` expansions.
- `CodexExecutor` gains `MultiAgent`, `PassClaudeMd`, `IdleTimeout`,
  `Sandbox`-with-mode-aware-default, additive `-c` flag overrides, an
  idle-timeout machinery via `touchReader` + `time.AfterFunc`, and
  `finalError` extraction so non-zero exits surface a useful diagnostic.
- Shared review prompts: `{{agent:<name>}}` expander reads
  `cfg.AppConfig.Executor` and emits the executor-appropriate invocation
  (Ta... (continued)

Coverage Stats

649 of 712 new or added lines in 9 files covered. (91.15%)

21 existing lines in 4 files now uncovered.

7426 of 8926 relevant lines covered (83.2%)

232.18 hits per line