Fix Claude Code Review workflow permissions#696
Conversation
🚀 Quick Review App CommandsWelcome! Here are the commands you can use in this PR:
|
WalkthroughA permissions block is added to the GitHub Actions workflow configuration, enabling the workflow to read repository contents and write to pull requests, issues, and the OIDC token provider for enhanced security control. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
No actionable comments were generated in the recent review. 🎉 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 |
Greptile OverviewGreptile SummaryAdded explicit Changes:
The change follows GitHub Actions' security model where caller workflows must explicitly grant permissions to reusable workflows. The permissions are appropriate for a code review bot that needs to read repository content and write comments to PRs and issues. Confidence Score: 5/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant PR as Pull Request Event
participant Caller as claude-code-review.yml
participant Reusable as shakacode/.github workflow
participant GitHub as GitHub API
PR->>Caller: Trigger (opened/synchronize)
Note over Caller: Grants permissions:<br/>contents: read<br/>pull-requests: write<br/>issues: write<br/>id-token: write
Caller->>Reusable: Call reusable workflow
Note over Reusable: Receives granted permissions
Reusable->>GitHub: Write PR comments
Reusable->>GitHub: Write issue comments
Reusable->>GitHub: Authenticate with id-token
GitHub-->>Reusable: Success
Reusable-->>Caller: Complete
Caller-->>PR: Workflow complete
|
|
✅ Review app for PR #696 was successfully deleted |
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
Summary by CodeRabbit