Skip to content

ci: pin default Claude review model to claude-opus-4-7#78

Merged
Troublor merged 1 commit into
mainfrom
williamaaroncheung/ci/pin-review-model
Jul 8, 2026
Merged

ci: pin default Claude review model to claude-opus-4-7#78
Troublor merged 1 commit into
mainfrom
williamaaroncheung/ci/pin-review-model

Conversation

@Troublor

@Troublor Troublor commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • The claude-pr-review composite action's model input defaulted to the opus alias, which resolves to whatever the latest Opus model is at run time (currently Opus 4.8). Every consuming repo (mega-reth, mega-evm, mega-e2e, mega-agents, mega-devops, dist-docs, watchdog-rpc, documentation) inherits this default.
  • This pins the default to claude-opus-4-7 so PR reviews run on a fixed, predictable model instead of silently shifting when the alias advances to a new Opus release.
  • Callers can still override via the model input, or set it empty to fall through to the claude-code-action / CLI default.

Test plan

  • Change is limited to the default: value and description of the model input in .github/actions/claude-pr-review/action.yml; the composite action logic is unchanged.
  • The value is passed straight through to claude --model; claude-opus-4-7 is a valid model id (hyphenated form the CLI expects).
  • Verifiable on the next PR review run: the review agent job should invoke --model claude-opus-4-7.

This PR was generated by an automated agent.

The model input defaulted to the 'opus' alias, which resolves to whatever
the latest Opus is at run time (currently Opus 4.8). Pin it to
claude-opus-4-7 so PR reviews run on a fixed, predictable model instead of
silently moving when the alias advances.
@Troublor Troublor added the agent label Jul 8, 2026

@claude claude Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Clean

  • Summary: reviewed the one-file change in .github/actions/claude-pr-review/action.yml that pins the model input default from the moving opus alias to claude-opus-4-7 and refreshes the description.
  • Correctness: claude-opus-4-7 is the valid hyphenated model id passed straight through to claude --model; override path (including passing opus to opt back into the moving alias) and empty-string fall-through are preserved.
  • Design: pinning a moving alias is the right call — reviews stay reproducible when the alias advances, and the change is scoped narrowly to the default plus its description.
  • Repo conventions: PR title is Conventional Commits and the branch follows the {developer}/{category}/{description} pattern from CLAUDE.md.
  • No new actionable findings.
  • Still open from earlier reviews: none.

@Troublor Troublor merged commit cea80db into main Jul 8, 2026
6 of 7 checks passed
@Troublor Troublor deleted the williamaaroncheung/ci/pin-review-model branch July 8, 2026 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant