Fix Claude Code Review workflow permissions#111
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the Claude Code Review GitHub Actions caller workflow to explicitly grant the permissions required by the reusable workflow in shakacode/.github, avoiding failures due to default “none” permissions on nested jobs.
Changes:
- Add a job-level
permissionsblock (contents read, pull-requests write, issues write, id-token write) to the reusable workflow caller.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| uses: shakacode/.github/.github/workflows/claude-code-review.yml@main | ||
| permissions: | ||
| contents: read | ||
| pull-requests: write | ||
| issues: write |
There was a problem hiding this comment.
With the added pull-requests: write / issues: write permissions, referencing the reusable workflow via @main increases supply-chain risk (the called workflow can change without review). Consider pinning uses: to an immutable ref (tag or commit SHA) and updating intentionally when needed.
Summary
permissionsblock to the Claude Code Review caller workflowshakacode/.githubneedsissues: write,pull-requests: write, andid-token: write, but callers must explicitly grant these permissionsTest plan
🤖 Generated with Claude Code