umputun / ralphex / 25178291348

83%

Build Type

push

github

Committed by web-flow

Commit Message

feat: add provider override CLI flags (#314)

* gemini creates Agents.md automatically

* add plan: claude-command-external-review-tool-flags

* feat: add provider override cli flags

* feat: add provider override CLI tests

* feat: document provider override flags

* feat: verify provider flag acceptance criteria

* feat: update provider flag plan lifecycle

* fix: address code review findings

* refactor: drop underscore CLI aliases for provider override flags

Removes hidden --claude_command, --claude_args, --external_review_tool,
and --custom_review_script aliases. They were the only underscore-spelled
CLI flags in ralphex and didn't justify the alias fields, *Set bools,
conflict validation, and test matrix needed to support them. The hyphen
flags and the underscored config keys are unchanged.

* refactor: stop silently flipping codex_enabled on CLI external-review override

A CLI flag mutating an unrelated config field is surprising. Push the
explicitness signal down instead: processor.Config gains an
ExternalReviewToolSet flag, Runner.externalReviewTool() honors an
explicit choice over the legacy codex_enabled=false back-compat path,
and applyCLIOverrides only writes the field the user actually set.

Legacy behavior unchanged: a config with only codex_enabled=false (no
explicit external_review_tool from CLI) still resolves to "none".

Coverage Stats

29 of 29 new or added lines in 3 files covered. (100.0%)

6 existing lines in 3 files now uncovered.

6810 of 8243 relevant lines covered (82.62%)

220.92 hits per line