Skip to content

NAS-144265 / 28.0.0-BETA.1 / Default the review model to claude-opus-5-5 - #20

Merged
william-gr merged 2 commits into
masterfrom
default-opus-5-5
Oct 7, 2026
Merged

william-gr merged 2 commits into
masterfrom
default-opus-5-5

Conversation

@william-gr

Copy link
Copy Markdown
Contributor

claude-opus-5 pins Opus 5 rather than tracking the latest Opus. Switch the model default to Opus 5.5 (cheaper, newer). Note Opus 5.5 defaults to medium effort.

claude-opus-5 pins Opus 5, not the latest Opus.
Opus 5.5 needs Claude Code 2.1.280 or newer; v1.0.187 bundles 2.1.224.
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

No findings. A person should review this before merge, because it changes a default that callers will see.

Human review needed: in .github/workflows/claude-review.yml:63, the model input default changes from claude-opus-5 to claude-opus-5-5. Every caller that does not set model is on @master, so all of them move to the new model when this merges. That changes their review cost and, as the PR description says, the default effort level.

What I checked:

  • Model default. The input default and the README table (README.md:197) match. Nothing else in the repo still refers to claude-opus-5. The value still reaches the action only through --model ${{ inputs.model }} in claude_args, so nothing else needs to change.
  • Action pin. anthropics/claude-code-action moves from v1.0.187 to v1.0.244. It is still pinned to an exact version, and the bump has its own commit, so it is clearly intentional. No other file names the old version. The version list at README.md:242 is history about how callers drifted apart, not the current pin.
  • CI. check-ticket is failing because the PR title and body do not reference a ticket. That comes from the PR metadata, not from this diff.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a human review. No findings.

Nothing blocks, but this change is one a person should decide on:

  • .github/workflows/claude-review.yml:63 changes the model input default from claude-opus-5 to claude-opus-5-5, which every caller on @master that leaves model unset will pick up

@william-gr william-gr added the jira label Oct 7, 2026
@bugclerk bugclerk changed the title Default the review model to claude-opus-5-5 NAS-144265 / 28.0.0-BETA.1 / Default the review model to claude-opus-5-5 Oct 7, 2026
@bugclerk

bugclerk commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

@aervin
aervin self-requested a review October 7, 2026 15:21
@william-gr
william-gr merged commit c075b5a into master Oct 7, 2026
7 of 8 checks passed
@william-gr
william-gr deleted the default-opus-5-5 branch October 7, 2026 15:21
@bugclerk

bugclerk commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.

@iXsystems iXsystems locked as resolved and limited conversation to collaborators Oct 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants