feat: ACNA-4515 add pr-reviewer workflow - #77
Conversation
There was a problem hiding this comment.
🤖 PR Reviewer
The workflow is well-structured with clear gate logic and good security practices (permission checks, fork exclusion, env var injection). Two minor issues worth addressing: the gh api call for PR data is unquoted in a command substitution context which could cause word-splitting issues, and pinning the reusable workflow to @main is a supply-chain risk.
📝 3 suggestion(s) - Please review inline comments below.
💡 How to re-trigger
Comment /review or /pr-reviewer on this PR
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Proposed changes not required
There was a problem hiding this comment.
🤖 PR Reviewer
The workflow is generally well-structured with good security practices like using environment variables instead of inline expressions in shell scripts. However, three issues from the previous review remain unaddressed: the DATA variable approach, defensive quoting, and the @main pin for the reusable workflow which is a supply-chain security risk.
🔄 3 re-raised suggestion(s) from previous review
💡 How to re-trigger
Comment /review or /pr-reviewer on this PR
proposed changes not required
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: