umputun / ralphex / 26300107587

83%

master: 83%

Build Type

Pull #357

github

Committed by umputun

Commit Message

feat(codex): per-phase model selection for codex executor

--task-model/--review-model (and config task_model/review_model) now
select the codex model and effort per phase instead of collapsing into
one. buildCodexExecutors builds a separate review CodexExecutor when the
resolved review model/effort differs from task, mirroring
buildClaudeExecutors; otherwise the task executor handles task, review,
and finalize.

ResolveCodexModelEffort resolves a model[:effort] spec against
codex_model/codex_reasoning_effort as the defaults - an empty spec keeps
the defaults, each populated half overrides its default. The claude-only
max effort is reported and dropped; the startup banner emits the warning
and shows a review model/effort line when review differs from task.

Replaces the earlier single-field applyCodexModelOverride approach: the
flags no longer collapse onto cfg.CodexModel, so tasks and reviews can
run on different codex models.

Related to #356

Pull Request Pull Request #357: feat(codex): per-phase model selection for codex executor

Coverage Stats

63 of 81 new or added lines in 2 files covered. (77.78%)

83 existing lines in 1 file now uncovered.

7487 of 8990 relevant lines covered (83.28%)

230.91 hits per line