Skip to content

Grant issues:write so the PR Review workflow can start - #3975

Merged
Mpdreamz merged 1 commit into
mainfrom
fix/pr-review
Aug 31, 2026
Merged

Grant issues:write so the PR Review workflow can start#3975
Mpdreamz merged 1 commit into
mainfrom
fix/pr-review

Conversation

@Mpdreamz

Copy link
Copy Markdown
Member

The PR Review workflow has failed to start on every pull request since it was added — GitHub rejects it before any job runs.

Affects: Automation

Why

The workflow's permissions: block grants only issues: read, but the reusable workflow it calls has conclusion and safe_outputs jobs that request issues: write for failure and no-op run tracking. GitHub validates that a caller's permissions cover every nested job's request, and rejects the whole run as a permission escalation when they don't. All 100 most recent runs of pr-review.yml ended in startup_failure with no jobs ever created.

What

Workflow permissions

.github/workflows/pr-review.yml now grants issues: write instead of issues: read, matching the grant other Elastic repos already use successfully for the same reusable workflow (elastic/ai-github-actions/.github/workflows/gh-aw-pr-review.lock.yml@v0).

Made with Cursor

The PR Review workflow granted only issues:read, but its reusable
workflow's conclusion and safe_outputs jobs request issues:write for
failure and no-op run tracking. GitHub rejects the permission
escalation before any job starts, so every PR Review run since the
workflow was added has failed with a startup error.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@Mpdreamz
Mpdreamz requested a review from a team as a code owner August 31, 2026 13:22
@Mpdreamz Mpdreamz added the bug label Aug 31, 2026
@Mpdreamz
Mpdreamz requested a review from reakaleek August 31, 2026 13:22
@Mpdreamz
Mpdreamz merged commit da1becf into main Aug 31, 2026
34 checks passed
@Mpdreamz
Mpdreamz deleted the fix/pr-review branch August 31, 2026 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants