Skip to content

fix(ci): grant review-tools caller the scopes its reusables require #1424

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
LucasSantana-Dev merged 1 commit into from
Jun 14, 2026
Merged

fix(ci): grant review-tools caller the scopes its reusables require #1424

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 9 additions & 3 deletions .github/workflows/review-tools.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,12 +31,18 @@ concurrency:
group: review-tools-${{ github.event.pull_request.number }}
cancel-in-progress: true

# Minimum permissions for both reusable workflows:
# - claude-review.yml needs to read PR contents and post review comments
# - danger.yml needs to read PR diff and post a comment with findings
# Minimum permissions for both reusable workflows. An explicit block sets
# every unlisted scope to `none`, and a called reusable workflow cannot
# request scopes the caller withheld — so the caller must grant the UNION of
# what the reusables declare, or the run fails at startup (see #1423):
# - claude-review.yml needs contents:read, pull-requests:write,
# issues:write, and id-token:write (OIDC)
# - danger.yml needs contents:read, pull-requests:write, issues:write
permissions:
contents: read
pull-requests: write
issues: write
id-token: write

jobs:
claude-review:
Expand Down
Loading