ci(claude-review): run auto review on Opus (claude-opus-4-8)#3693
Merged
Conversation
The review action currently runs on the default Sonnet 4.6. On PR #3643 (musb EP0 race) it posted "No issues found" while an Opus pass on the same diff surfaced substantive questions (ISR-boundary RXRDY lifetime, regression scope of the DATA-state split). Pin the reviewer to claude-opus-4-8 for higher-signal reviews; subagents keep their cheaper default models. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Pins the Claude Code Review GitHub Action to run the main review model as claude-opus-4-8 (while leaving subagents on cheaper defaults), to improve review depth for driver-level PRs.
Changes:
- Update
claude_argsfor theclaude-reviewworkflow to include--model claude-opus-4-8alongside--max-turns 50.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| # debugging. Revert to remove once done. | ||
| show_full_output: true | ||
| claude_args: '--max-turns 50' | ||
| claude_args: '--max-turns 50 --model claude-opus-4-8' |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Pin the Claude Code Review action's main model to
claude-opus-4-8viaclaude_args(was the action default, Sonnet 4.6). Subagents spawned by the review command keep their cheaper defaults (Haiku 4.5).Why
On #3643 (musb EP0 race fix) the Sonnet-driven auto review posted "No issues found", while an Opus pass over the same diff surfaced substantive review questions — ISR-boundary RXRDY/FIFO lifetime in the deferred-SETUP path, and the regression scope of the
PIPE0_STATE_DATAsplit affecting every MUSB control transfer. For driver-level PRs that depth is worth the tier bump.Cost
Opus 4.8 is $5/$25 per MTok vs Sonnet 4.6 at $3/$15 — a typical review run goes from roughly $0.30–1 to $1–3. Follows up on #3692 (
--max-turns 50).🤖 Generated with Claude Code