umputun / ralphex / 24538255464

83%

Build Type

push

github

Committed by web-flow

Commit Message

feat: rename claude_model to task_model, address unmerged #272 review findings (#284)

* feat: rename claude_model to task_model, address unmerged #272 review findings

- rename claude_model -> task_model and --claude-model -> --task-model for symmetry with review_model/--review-model
- pass task model to plan mode (was missing)
- strip duplicate --model flags from claude_args to prevent CLI errors
- fix typo in pkg/config/defaults/config: review_model section had `# claude_model =` example
- add processor test for ReviewClaude executor separation (nil fallback + distinct review executor)
- add ReviewModel merge test, normalize --flag=value form in docs
- update docs (README, CLAUDE.md, llms.txt, custom-providers.md)
- update review-claude tests for post-codex-skip behavior added in #282

picks up unmerged follow-up commits from feat/claude-model-config branch

* fix: address copilot review on PR 284

- stripFlag now handles --flag=value and bare --flag forms (was only handling --flag value space-form), prevents duplicate/orphaned --model in claude_args
- add stripFlag tests for equals form, bare flag, repeated occurrences, prefix-only false-match
- doc wording: clarify that task_model/review_model are passed as --model to the configured claude_command and may be ignored or implemented by custom wrappers

Coverage Stats

27 of 34 new or added lines in 5 files covered. (79.41%)

7 existing lines in 3 files now uncovered.

6403 of 7832 relevant lines covered (81.75%)

209.1 hits per line