Skip to content

Update Claude review to latest models - #2440

Merged
umangyadav merged 1 commit into
developfrom
update-claude-models
Jul 28, 2026
Merged

Update Claude review to latest models#2440
umangyadav merged 1 commit into
developfrom
update-claude-models

Conversation

@umangyadav

Copy link
Copy Markdown
Member

Summary

  • update the automated review model from Claude Opus 4.7 to Opus 5
  • update the default Sonnet model from 4.6 to Sonnet 5
  • retain Haiku 4.5 as the latest Haiku model

Test plan

  • Run git diff --check
  • Run .github/scripts/tests/test_sanitize.sh (171 tests passed)
  • Verify the workflow YAML has no linter diagnostics

Made with Cursor

Use Opus 5 and Sonnet 5 so automated reviews run on the current model generation.

Co-authored-by: Cursor <cursoragent@cursor.com>
@umangyadav
umangyadav requested a review from causten as a code owner July 28, 2026 16:33
@umangyadav
umangyadav requested a review from justinrosner July 28, 2026 16:33
@umangyadav umangyadav self-assigned this Jul 28, 2026
@umangyadav
umangyadav requested a review from Copilot July 28, 2026 16:33
@rocmlir-pr-reviewer rocmlir-pr-reviewer Bot added the modifies-ci-paths PR modifies the Claude review CI security perimeter; audit before applying claude-review label Jul 28, 2026
@rocmlir-pr-reviewer

Copy link
Copy Markdown

⚠️ This PR modifies the Claude-review CI security perimeter

The following files in this PR control whether and how the
claude-review workflow protects its LLM Gateway secrets at
runtime (see .github/workflows/CLAUDE_AUTO_REVIEW.md):

  • .github/workflows/claude_auto_review.yml

Before applying the claude-review label on this PR, please:

  1. Audit the diff in these paths line-by-line. A malicious or
    accidental change could disable the --allowedTools
    restriction, the overlay step, the sanitizer, or the
    review/post job split -- any of which would expose the
    secrets in env to the PR-modified workflow.
  2. If the changes are legitimate and you want a Claude review,
    do NOT apply the claude-review label. Instead, run via
    Actions → Claude Auto Review → Run workflow and enter
    this PR's number. The dispatch path runs from the trusted,
    code-owner-approved version of the workflow on
    develop, so a malicious PR-side
    modification cannot affect the run.

(This banner is automated. The modifies-ci-paths label
will be removed automatically if a future push removes the
perimeter modifications. The Layer-3 in-workflow guard will
additionally fail the claude-review label-triggered run
on this PR if the label is applied while perimeter changes
are present.)

@umangyadav
umangyadav merged commit c08a598 into develop Jul 28, 2026
9 checks passed
@umangyadav
umangyadav deleted the update-claude-models branch July 28, 2026 16:35

Copilot AI 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.

Pull request overview

Updates the GitHub Actions workflow that runs automated Claude-based PR reviews by switching the configured gateway model identifiers to the latest Opus and Sonnet versions.

Changes:

  • Update ANTHROPIC_MODEL and ANTHROPIC_DEFAULT_OPUS_MODEL from Claude-Opus-4.7 to Claude-Opus-5.
  • Update ANTHROPIC_DEFAULT_SONNET_MODEL from Claude-Sonnet-4.6 to Claude-Sonnet-5.
  • Keep ANTHROPIC_DEFAULT_HAIKU_MODEL at Claude-Haiku-4.5.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifies-ci-paths PR modifies the Claude review CI security perimeter; audit before applying claude-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants