Skip to content

Fix Claude Code Review workflow permissions#212

Merged
justin808 merged 1 commit intomasterfrom
jg/fix-claude-review-perms
Feb 11, 2026
Merged

Fix Claude Code Review workflow permissions#212
justin808 merged 1 commit intomasterfrom
jg/fix-claude-review-perms

Conversation

@justin808
Copy link
Member

@justin808 justin808 commented Feb 9, 2026

Summary

  • Add permissions block to the Claude Code Review caller workflow
  • The reusable workflow in shakacode/.github needs issues: write, pull-requests: write, and id-token: write, but callers must explicitly grant these permissions
  • Without this, the workflow fails with: "The nested job is requesting permissions but is only allowed none"

Test plan

  • Verify the Claude Code Review workflow runs successfully on a new PR

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated internal CI/CD workflow permissions to support code review automation.

Note: This release contains no user-facing changes.

@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Walkthrough

A permissions block was added to the claude-review GitHub Actions workflow job, granting read access to contents, write access to pull-requests and issues, and write access to id-token. No functional changes to existing workflow logic.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Permissions
.github/workflows/claude-code-review.yml
Added permissions block to the claude-review job granting: contents (read), pull-requests (write), issues (write), and id-token (write).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A rabbit hops through workflows divine,
Granting permissions, line by line,
Read the contents, write with care,
To pull-requests and issues everywhere!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix Claude Code Review workflow permissions' directly and clearly summarizes the main change: adding a permissions block to fix the Claude Code Review workflow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

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

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.

❤️ Share

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

@justin808 justin808 merged commit 7fc2e9b into master Feb 11, 2026
5 of 6 checks passed
@justin808 justin808 deleted the jg/fix-claude-review-perms branch February 11, 2026 02:25
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.

1 participant

Comments