Skip to content

Commit a1bfa81

Browse files
sjnimsclaude
andauthored
fix(ci): add permissions block to pr-checks.yml orchestrator (#36)
## Summary - Added required `permissions:` block to `pr-checks.yml` orchestrator workflow - Enables reusable workflow calls to access necessary GitHub token permissions ## Problem Fixes #35 The PR Checks orchestrator workflow fails with `startup_failure` because reusable workflows can only access permissions explicitly granted by the calling workflow. Without a `permissions:` block, all permissions default to `none`. **Error:** "The nested job 'validate' is requesting 'actions: read, issues: write, pull-requests: write, id-token: write', but is only allowed 'actions: none, issues: none, pull-requests: none, id-token: none'." ## Solution Added a `permissions:` block with the union of all permissions required by: - `component-validation.yml` - `version-check.yml` - `claude-pr-review.yml` ```yaml permissions: contents: read pull-requests: write issues: write id-token: write actions: read ``` ### Alternatives Considered - Adding permissions to each job individually - rejected because workflow-level permissions are cleaner and reusable workflows inherit from the caller ## Changes - `.github/workflows/pr-checks.yml`: Added `permissions:` block after `concurrency:` ## Testing - [x] actionlint validation passes - [x] Workflow syntax is valid - [ ] PR Checks workflow runs successfully (will verify when this PR triggers) --- 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>
1 parent 95c87cf commit a1bfa81

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

.github/workflows/pr-checks.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,15 @@ concurrency:
1111
group: pr-checks-${{ github.event.pull_request.number }}
1212
cancel-in-progress: true
1313

14+
# Permissions required by reusable workflows
15+
# Union of: component-validation, version-check, claude-pr-review
16+
permissions:
17+
contents: read
18+
pull-requests: write
19+
issues: write
20+
id-token: write
21+
actions: read
22+
1423
jobs:
1524
# Detect what files changed to determine which checks to run
1625
changes:

0 commit comments

Comments
 (0)