Skip to content

Fix Claude Code Review workflow permissions - #92

Closed
EdiWeeks wants to merge 1 commit into
T-Minus-15:mainfrom
EdiWeeks:fix-claude-review-workflow
Closed

EdiWeeks wants to merge 1 commit into
T-Minus-15:mainfrom
EdiWeeks:fix-claude-review-workflow

Conversation

@EdiWeeks

Copy link
Copy Markdown
Contributor

Summary

  • Add github_token explicitly to bypass OIDC requirement for fork PRs
  • Change pull-requests permission from read to write so the action can post review comments

This fixes the CI/CD failure on PR #91 and other PRs from forks, where OIDC token authentication fails because GitHub restricts OIDC tokens for fork PRs for security reasons.

Test plan

🤖 Generated with Claude Code

- Add github_token explicitly to bypass OIDC requirement for fork PRs
- Change pull-requests permission to write so action can post comments

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@BenGWeeks
BenGWeeks marked this pull request as draft January 16, 2026 10:27
@EdiWeeks EdiWeeks closed this Jan 16, 2026
@EdiWeeks
EdiWeeks deleted the fix-claude-review-workflow branch January 16, 2026 15:19
@BenGWeeks

Copy link
Copy Markdown
Collaborator

Why is this closed @EdiWeeks?

@EdiWeeks

EdiWeeks commented Jan 16, 2026 •

Copy link
Copy Markdown
Contributor Author

The fix was applied to #90 and #91 PRs, so workflows are fixed. @BenGWeeks

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.

2 participants