umputun / ralphex / 26264210846

83%

master: 83%

Build Type

Pull #350

github

Committed by umputun

Commit Message

fix(review-loop): iteration 1 addressed 13 findings

- [major] [prompts.go] prepend review-context lead-in in formatAgentExpansion so spawned review agents get the branch diff and changed-file pointers
- [minor] [codex.go] move tailCancel after wait() so the rollout tailer drains a fully-flushed file
- [minor] [progress.go] hold writeMu around the construction-time restart/header write
- [minor] [codex.go] group processStderr streaming params into stderrStreamOpts struct
- [minor] [codex.go] group headerEmitted with unexported state, after exported config fields
- [minor] [codex.go] log a diagnostic when the codex rollout file is missing or unopenable
- [minor] [codex.go] correct codex-as-claude wrapper script path in godoc
- [minor] [codex.go] reset idle timer on any rollout bytes, before display filtering
- [doc] [custom-providers.md] correct idle-timeout scope: external codex review in default-claude mode is not affected
- [doc] [CLAUDE.md] startup banner is driven by codex_model/codex_reasoning_effort config, not --task-model
- [test] [main_test.go] cover populated and empty codex detail lines in TestPrintStartupInfo
- [test] [progress_test.go] add race-checked concurrent-producer test for writeMu serialization
- [test] [codex_test.go] add table-driven TestCodexExecutor_finalError

Pull Request Pull Request #350: Add first-class codex executor mode

Coverage Stats

643 of 706 new or added lines in 9 files covered. (91.08%)

21 existing lines in 4 files now uncovered.

7421 of 8921 relevant lines covered (83.19%)

232.14 hits per line