Skip to content

feat(auto-review): add opt-in classifier effort override - #93

Open
ItsAlbertZhang wants to merge 1 commit into
raine:mainfrom
ItsAlbertZhang:feat/auto-review-effort
Open

feat(auto-review): add opt-in classifier effort override#93
ItsAlbertZhang wants to merge 1 commit into
raine:mainfrom
ItsAlbertZhang:feat/auto-review-effort

Conversation

@ItsAlbertZhang

Copy link
Copy Markdown
Contributor

Summary

  • follow up on feat: allow a dedicated model for auto-review classifier requests #72 by complementing its auto-review classifier model configuration with a dedicated reasoning-effort override
  • add provider-neutral CCP_AUTO_REVIEW_EFFORT and autoReviewEffort settings
  • preserve all existing behavior by default; request behavior changes only when the override is explicitly enabled
  • apply the override only to detected non-streaming, tool-free auto-review requests, excluding token counting
  • let the classifier-specific setting take precedence over generic Codex effort configuration while retaining normal validation and compaction caps
  • document configuration precedence and add unit and smoke coverage for Codex and Kimi routing

Testing

  • cargo fmt --all -- --check
  • cargo test --all-targets --locked

🤖 Generated with Claude Code

Add CCP_AUTO_REVIEW_EFFORT and autoReviewEffort for detected
auto-review classifier requests.

Apply the configured effort before provider dispatch, preserving
exclusions for streaming, tool-using, and token-counting requests.
When applied to Codex, bypass the generic Codex effort override while
retaining request validation and compaction caps.

Document precedence and add configuration, routing, and smoke coverage.

Co-Authored-By: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant