Skip to content

feat: ACNA-4515 add pr-reviewer workflow#101

Open
chsrimanaswi wants to merge 1 commit intomasterfrom
ACNA-4515-Add-pr-reviewer-workflow
Open

feat: ACNA-4515 add pr-reviewer workflow#101
chsrimanaswi wants to merge 1 commit intomasterfrom
ACNA-4515-Add-pr-reviewer-workflow

Conversation

@chsrimanaswi
Copy link
Copy Markdown

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 /review or /pr-reviewer comments 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 /review comments, posts inline suggestions and summary reviews via github-actions[bot].

Screenshots (if appropriate):

N/A

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

🤖 PR Reviewer

The workflow is well-structured with clear separation of gate-checking and review logic. The security model is sound — fork PRs are blocked, and comment-triggered reviews require admin/maintain permission. One moderate concern is that the gh api call for PR data is unquoted in a shell substitution and lacks error handling, and there is a minor risk of comment body injection if the body check were relaxed in the future, but as written the exact-match guards are sufficient.

LGTM! This PR looks good to merge.


💡 How to re-trigger

Comment /review or /pr-reviewer on this PR

Comment thread .github/workflows/pr-review.yml
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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