Skip to content

fix(examples): update workflow permissions for proper operation#1399

Open
anishesg wants to merge 1 commit into
anthropics:mainfrom
proudhare:fix/ph-issue-1383
Open

fix(examples): update workflow permissions for proper operation#1399
anishesg wants to merge 1 commit into
anthropics:mainfrom
proudhare:fix/ph-issue-1383

Conversation

@anishesg

@anishesg anishesg commented Jun 9, 2026

Copy link
Copy Markdown

The example workflow files had insufficient permissions that would prevent the action from functioning correctly. The workflows need write access to contents, pull-requests, and issues to perform code reviews and post comments. Updated pr-review-comprehensive.yml, pr-review-filtered-authors.yml, and pr-review-filtered-paths.yml to change permissions from read to write where needed. Also added a new code-review.yml example that demonstrates using the /code-review command with proper permissions configured from the start.

Fixes #1383

The example workflow files had insufficient permissions that would prevent the action from functioning correctly. The workflows need write access to contents, pull-requests, and issues to perform code reviews and post comments. Updated pr-review-comprehensive.yml, pr-review-filtered-authors.yml, and pr-review-filtered-paths.yml to change permissions from read to write where needed. Also added a new code-review.yml example that demonstrates using the /code-review command with proper permissions configured from the start.

Signed-off-by: anish <anishesg@users.noreply.github.com>
@ashwin-ant

Copy link
Copy Markdown
Collaborator

This shouldn't be needed if you're using the Claude GitHub app since we fetch a separate app token inside the action itself.

@anishesg

anishesg commented Jun 9, 2026

Copy link
Copy Markdown
Author

ah got it, removed that line. didn't realize the app token was already being fetched in the action

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.

/install-github-app generates no-op workflows: missing --comment + read-only permissions

2 participants