Skip to content

Add PR review agent#399

Closed
mehtarac wants to merge 0 commit intostrands-agents:mainfrom
mehtarac:main
Closed

Add PR review agent#399
mehtarac wants to merge 0 commit intostrands-agents:mainfrom
mehtarac:main

Conversation

@mehtarac
Copy link
Member

@mehtarac mehtarac commented Jan 13, 2026

Description

This PR adds an Agent as a github action workflow. This agent can be used following a /strands review command, which triggers the action to start the review process. During the review:

  1. Agent sets the the review environment reading the PR description, current branch, and identifies the type of change.
  2. Identifies the linked issue, checks for existing review comments and uses a new tool, get_pr_files to review changed files and scope.
  3. Reviews repo guidelines from CONTRIBUTING.md and AGENTS.md
  4. Does a code analysis for
  • File and directory strucuture, naming conventions, and breaking changes
  • Language best practices
  • Testing review
  • Documentation review
  1. Generates review comments
  2. Posts review comments

Related Issues

#1

Documentation PR

Type of Change

New feature

Testing

How have you tested the change?
Tested in forked repo on a sample PR: mehtarac#1

  • I ran npm run check

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mehtarac mehtarac marked this pull request as draft January 13, 2026 15:34
@mehtarac mehtarac marked this pull request as ready for review January 13, 2026 15:48
@mehtarac mehtarac changed the title pull request Add PR review agent Jan 13, 2026
Unshure

This comment was marked as resolved.

@Unshure
Copy link
Member

Unshure commented Jan 14, 2026

non-blocker - We should try out the newest gemini model as well for this agent, @mr-lee mentioned its very good for code reivews.

Unshure
Unshure previously approved these changes Jan 14, 2026
Copy link
Member

@Unshure Unshure left a comment

Choose a reason for hiding this comment

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

Lets run the pr review sop to review this pr :D

pgrayy
pgrayy previously approved these changes Jan 15, 2026
@mehtarac mehtarac enabled auto-merge January 15, 2026 21:03
@Unshure Unshure dismissed stale reviews from pgrayy and themself via 171eea1 January 20, 2026 15:17
@Unshure Unshure closed this Jan 20, 2026
auto-merge was automatically disabled January 20, 2026 15:35

Pull request was closed

@Unshure Unshure mentioned this pull request Jan 20, 2026
7 tasks
@Unshure
Copy link
Member

Unshure commented Jan 20, 2026

Attempted to rebase the branch here, but ended up not having the correct permissions to fix the original main branch. Moving this review to this pr: #409

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.

4 participants