feat: ACNA-4515 add pr-reviewer workflow#63
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
🤖 PR Reviewer
The workflow is well-structured with a sensible permission gate and clear separation of concerns. Two moderate issues exist: the @main pin on the reusable workflow creates a supply-chain risk, and the gh api call for PR data is unquoted in a command substitution that feeds into jq, which is fine, but the DATA variable could be empty on API failure without error handling.
📝 3 suggestion(s) - Please review inline comments below.
💡 How to re-trigger
Comment /review or /pr-reviewer on this PR
Proposed changes not required
shazron
left a comment
There was a problem hiding this comment.
Update the required checks in Settings -> Branches. Remove the 18 checks, add the 22, 24 checks.
Description
Adds an AI-powered PR reviewer workflow that automatically reviews pull requests using Claude via AWS Bedrock. Triggers on PR open/reopen/synchronize and on
/reviewor/pr-reviewercomments by admins or maintainers.Related Issue
ACNA-4515
Motivation and Context
Reduces code review toil by providing automated first-pass reviews with inline suggestions. Part of a broader rollout across App Builder repos.
How Has This Been Tested?
Tested end-to-end in
adobe/generator-aio-app— workflow triggers correctly on PR events and/reviewcomments, posts inline suggestions and summary reviews viagithub-actions[bot].Screenshots (if appropriate):
N/A
Types of changes
Checklist: