Skip to content

Add auto-assign reviewers workflow - #1

Merged
carochacs merged 5 commits into
mainfrom
claude/auto-assign-reviewers-workflow-q66f32
Aug 13, 2026
Merged

Add auto-assign reviewers workflow#1
carochacs merged 5 commits into
mainfrom
claude/auto-assign-reviewers-workflow-q66f32

Conversation

@carochacs

Copy link
Copy Markdown
Collaborator

Summary

Adds GitHub Actions workflow to automatically assign reviewers to pull requests.

Changes

  • Configures kentaro-m/auto-assign-action@v2.0.2 workflow
  • Automatically assigns carochacs as reviewer on all opened/reopened PRs
  • Sets PR creator as assignee using addAssignees: author
  • Workflow triggers on PR opened/reopened events

Generated by Claude Code

claude added 4 commits August 13, 2026 14:16
- Configures kentaro-m/auto-assign-action@v2.0.2
- Automatically assigns carochacs as reviewer on all PRs
- Sets PR creator as assignee

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011RQnPatXgn3E1gkQVcQAiD
Ensures reviewers are assigned when draft PRs are marked as ready for review.
…with: parameters

The kentaro-m/auto-assign-action@v2.0.2 reads config from .github/auto_assign.yml
and doesn't accept the with: parameters we were passing. This commit:
- Creates .github/auto_assign.yml with proper configuration
- Removes invalid with: parameters from workflow
- Workflow now uses config file instead of inline parameters

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

This PR's workflow fails on every pull request event: kentaro-m/auto-assign-action@v2.0.2 ignores the with: inputs and loads its configuration from .github/auto_assign.yml, which doesn't exist in this repo — the job errors out with a 404 Not Found before assigning anything.

Reviewed changes

  • Auto-assign reviewers workflow — new .github/workflows/auto-assign-reviewers.yml runs kentaro-m/auto-assign-action@v2.0.2 on pull_request opened/reopened/ready_for_review, intending to request review from carochacs and assign the PR author.

Pullfrog  | Fix all ➔Fix 👍s ➔View workflow run | Using Big Pickle (free) | 𝕏

Comment on lines +15 to +19
with:
addReviewers: 'true'
addAssignees: 'author'
reviewers: 'carochacs'
numberOfReviewers: '0'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These four inputs are ignored: kentaro-m/auto-assign-action@v2.0.2 declares only repo-token and configuration-path in its action.yml, and its run.ts loads configuration exclusively from .github/auto_assign.yml. That file doesn't exist in this repo, so every PR run fails with a 404 Not Found before any assignment happens.

with:
addReviewers: 'true'
addAssignees: 'author'
reviewers: 'carochacs'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Requesting carochacs as a reviewer will never take effect: the action filters the PR author out of the reviewer candidate list (chooseUsers in src/utils.ts), and GitHub rejects review requests to the author outright (see kentaro-m/auto-assign-action#11). With carochacs authoring every PR here, this reviewer assignment is inert — the workflow will only ever assign the author as assignee.

@carochacs carochacs self-assigned this Aug 13, 2026
…-assign-action

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011RQnPatXgn3E1gkQVcQAiD

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

The config fix is correct — .github/auto_assign.yml now exists and the invalid with: block is gone, resolving the previously-blocking config-load failure. One gap remains: the addAssignees: author step calls POST /issues/{n}/assignees, which GitHub's endpoint→permission mapping places under the issues permission, not pull-requests. With only pull-requests: write granted, that call is rejected; since the action catches and logs the error as a warning, the job still passes while silently never assigning the author.

Reviewed changes (delta since prior review 773e4b0)

  • Add config file, drop invalid inputs — new .github/auto_assign.yml (addReviewers: true, addAssignees: author, reviewers: [carochacs], numberOfReviewers: 0) and removal of the four undeclared with: keys from .github/workflows/auto-assign-reviewers.yml.

Pullfrog  | Fix all ➔Fix 👍s ➔View workflow run | Using Big Pickle (free) | 𝕏

Comment thread .github/auto_assign.yml
@@ -0,0 +1,6 @@
# Auto-assign configuration for kentaro-m/auto-assign-action
addReviewers: true
addAssignees: author

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

addAssignees: author maps to POST /issues/{issue_number}/assignees, which GitHub's endpoint→permission list assigns to the issues permission rather than pull-requests. This job grants only pull-requests: write, so the assignee call is rejected with Resource not accessible by integration — the action logs it as a warning and the job still passes, so the author-assignment half silently never happens. Add issues: write to the job permissions: block if the assignee behavior is wanted.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ The permissions fix is complete and correct — contents: read (the action fetches the config file via the contents API at the head ref), issues: write (assignee endpoint), and pull-requests: write (reviewer endpoint) are exactly the three scopes kentaro-m/auto-assign-action@v2.0.2 needs. No new issues in this commit.

Reviewed changes (delta since prior review 619194a)

  • Grant full permission set — job permissions: now adds contents: read and issues: write alongside pull-requests: write.

Pullfrog  | Fix it ➔View workflow run | Using Big Pickle (free) | 𝕏

@pullfrog

pullfrog Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review 4928149941 contains 0 review threads — the review body is a clean bill for the permissions fix (contents: read, issues: write, pull-requests: write), stating "No new issues in this commit." Verified the PR branch is clean at the reviewed commit 3ed3dbd; no code changes were required, so nothing was committed or pushed.

Task list (1/7 completed)
  • Checkout PR branch
  • Fetch review comments (review 4928149941)
  • Verify and address each comment
  • Test changes and review diff
  • Commit and push changes
  • Reply and resolve review threads
  • Report progress

Pullfrog  | View workflow run | via Pullfrog | Using Big Pickle (free) | 𝕏

@get-flashbacks get-flashbacks deleted a comment from coderabbitai Bot Aug 13, 2026
@carochacs
carochacs merged commit c3b9a2c into main Aug 13, 2026
2 checks passed
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