Skip to content

Use centralized Claude Code Review workflow#110

Merged
justin808 merged 1 commit intomainfrom
jg/use-centralized-review-workflow
Feb 9, 2026
Merged

Use centralized Claude Code Review workflow#110
justin808 merged 1 commit intomainfrom
jg/use-centralized-review-workflow

Conversation

@justin808
Copy link
Member

Summary

  • Replaces inline Claude Code Review workflow with a call to the shared reusable workflow in shakacode/.github
  • Removes hardcoded model — uses Claude Code's default (currently Opus 4.6)
  • Future changes only need one update in shakacode/.github

Dependencies

Test plan

🤖 Generated with Claude Code

Replace inline review config with a call to the shared reusable
workflow. Removes hardcoded model so reviews use Claude Code's
default (currently Opus 4.6).

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Copilot AI review requested due to automatic review settings February 9, 2026 01:54
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Warning

Rate limit exceeded

@justin808 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 12 minutes and 17 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jg/use-centralized-review-workflow

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR centralizes the “Claude Code Review” GitHub Action by replacing the repo’s inline workflow steps with a call to the shared reusable workflow in shakacode/.github.

Changes:

  • Replaced the local claude-code-review job implementation with a reusable workflow invocation.
  • Removed the previously hardcoded model/args configuration in favor of the shared workflow defaults.

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

# or https://docs.claude.com/en/docs/claude-code/sdk#command-line for available options
claude_args: '--model claude-sonnet-4-5-20250929 --allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'

uses: shakacode/.github/.github/workflows/claude-code-review.yml@main
Copy link

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

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

The reusable workflow is referenced via @main, which is mutable and can change behavior (or be compromised) without any change in this repo. Pin to an immutable ref (a tag like vX.Y.Z or a commit SHA) so runs are reproducible and reduce supply-chain risk—especially since this PR already depends on shakacode/.github#4 being merged.

Suggested change
uses: shakacode/.github/.github/workflows/claude-code-review.yml@main
uses: shakacode/.github/.github/workflows/claude-code-review.yml@v1

Copilot uses AI. Check for mistakes.
Comment on lines 8 to +11
claude-review:
# Optional: Filter by PR author
# if: |
# github.event.pull_request.user.login == 'external-contributor' ||
# github.event.pull_request.user.login == 'new-developer' ||
# github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'

runs-on: ubuntu-latest
permissions:
contents: read
pull-requests: read
issues: read
id-token: write

steps:
- name: Checkout repository
uses: actions/checkout@v4
with:
fetch-depth: 1

- name: Run Claude Code Review
id: claude-review
uses: anthropics/claude-code-action@v1
with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
prompt: |
REPO: ${{ github.repository }}
PR NUMBER: ${{ github.event.pull_request.number }}

Please review this pull request and provide feedback on:
- Code quality and best practices
- Potential bugs or issues
- Performance considerations
- Security concerns
- Test coverage

Use the repository's CLAUDE.md for guidance on style and conventions. Be constructive and helpful in your feedback.

Use `gh pr comment` with your Bash tool to leave your review as a comment on the PR.

# See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
# or https://docs.claude.com/en/docs/claude-code/sdk#command-line for available options
claude_args: '--model claude-sonnet-4-5-20250929 --allowed-tools "Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)"'

uses: shakacode/.github/.github/workflows/claude-code-review.yml@main
secrets:
CLAUDE_CODE_OAUTH_TOKEN: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
Copy link

Copilot AI Feb 9, 2026

Choose a reason for hiding this comment

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

This workflow no longer declares permissions for the job. In GitHub Actions, the called reusable workflow cannot elevate GITHUB_TOKEN permissions beyond what the caller grants, and the defaults can vary by repo/org settings. Consider explicitly setting the minimal required permissions here (similar to .github/workflows/claude.yml lines 21-26) to make the workflow deterministic and avoid unexpected failures when commenting on PRs or reading issues.

Copilot uses AI. Check for mistakes.
@justin808 justin808 merged commit d9e808f into main Feb 9, 2026
7 of 8 checks passed
@justin808 justin808 deleted the jg/use-centralized-review-workflow branch February 9, 2026 02:40
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.

2 participants