Skip to content

Fix workflow permissions for PR commenting#3

Merged
corylanou merged 1 commit into
mainfrom
fix/workflow-permissions
Aug 12, 2025
Merged

Fix workflow permissions for PR commenting#3
corylanou merged 1 commit into
mainfrom
fix/workflow-permissions

Conversation

@corylanou
Copy link
Copy Markdown
Contributor

Summary

  • Add explicit permissions to the manual integration test workflow to enable PR commenting
  • Fixes "Resource not accessible by integration" error when workflow attempts to post test results

Changes

  • Added workflow-level permissions:
    • contents: read - Required to checkout code
    • issues: write - Required to comment on issues
    • pull-requests: write - Required to comment on PRs

Problem

The manual integration test workflow was failing in the test summary job with:

RequestError [HttpError]: Resource not accessible by integration

This occurred when the workflow tried to post comments on PRs using the GitHub API.

Solution

By explicitly declaring the required permissions at the workflow level, we grant the necessary access for the GITHUB_TOKEN to interact with issues and pull requests.

Test Plan

  • Run the manual integration workflow with a PR number
  • Verify the workflow can successfully post comments to the PR
  • Confirm all other workflow steps continue to function correctly

🤖 Generated with Claude Code

Add explicit permissions to the manual integration test workflow to
allow commenting on pull requests and issues. This fixes the
"Resource not accessible by integration" error when the workflow
attempts to post test results to a PR.

Permissions added:
- contents: read (to checkout code)
- issues: write (to comment on issues)
- pull-requests: write (to comment on PRs)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@corylanou corylanou merged commit 3b4c500 into main Aug 12, 2025
9 checks passed
@corylanou corylanou deleted the fix/workflow-permissions branch August 12, 2025 20:38
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