Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
60 changes: 56 additions & 4 deletions .agents/commands/git/pr-review.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
allowed-tools: mcp__github_inline_comment__create_inline_comment,Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*)
allowed-tools: mcp__github_inline_comment__create_inline_comment,Bash(gh issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(gh api repos/*/pulls/*/comments:*),Bash(gh api repos/*/pulls/*/comments/*/replies:*)
description: Review a pull request
---

Expand All @@ -18,12 +18,64 @@ Perform a premortem analysis: identify potential failure scenarios (e.g., edge c

Use the repository's CLAUDE.md for guidance on style and conventions. Be constructive and helpful in your feedback.

## AI Bot Inline Comment Evaluation

Before starting your own review, evaluate existing AI bot inline review comments to reduce the maintainer's cognitive load:

1. **Fetch inline review comments**:
```bash
gh api repos/{owner}/{repo}/pulls/{pr_number}/comments
```

2. **Filter bot inline comments**:
- Only evaluate comments where `user.type === "Bot"` and `path` field is not null (inline comments only)
- **SKIP comments from `claude`** - do not respond to Claude's own comments
- **SKIP if Claude already replied** - for each bot comment, check if any comment exists where `user.login` contains `claude` and `in_reply_to_id` matches the bot comment's `id`
- Target bots: `gemini-code-assist[bot]`, `coderabbitai[bot]`, etc.

3. **Judge priority for each inline comment**:
- **Required**: Security issues, clear bugs, potential crashes, critical logic errors
- **Recommended**: Code quality improvements, best practice violations, maintainability concerns
- **Not needed**: Style suggestions, false positives, already addressed in code, out of scope for this PR

4. **Reply to each bot inline comment** with your judgment (in English):
```bash
gh api repos/{owner}/{repo}/pulls/{pr_number}/comments/{comment_id}/replies -f body="\`Priority: {Required/Recommended/Not needed}\`\n\n{Brief explanation of your judgment}"
```

5. **If clarification is needed**, ask in the reply:
```
`Priority: Recommended`

This suggestion appears valid, but I need clarification: Is this pattern used elsewhere in the codebase?
```

6. **Comment format examples**:
```
`Priority: Required`

This is a valid security concern. The input should be sanitized to prevent injection attacks.
```

```
`Priority: Not needed`

This is a false positive. The suggested change would actually break the existing API contract.
```

```
`Priority: Recommended`

Good refactoring suggestion. However, this is out of scope for the current PR. Consider creating a separate issue.
```

## How to Comment:
1. Before starting your review, read ALL existing comments on this PR using `gh pr view --comments` to see the full conversation
2. If there are any previous comments from you (Claude), identify what feedback you've already provided
3. Only provide NEW feedback that hasn't been mentioned yet, or updates to previous feedback if the code has changed
4. Avoid repeating feedback that has already been given - focus on providing incremental value with each review
5. For highlighting specific code issues, use `mcp__github_inline_comment__create_inline_comment` to leave inline comments
5. **Evaluate AI bot inline comments and reply with priority judgment** (see above section)
6. For highlighting specific code issues, use `mcp__github_inline_comment__create_inline_comment` to leave inline comments
- When possible, provide actionable fix suggestions with code examples
6. Use `gh pr comment` with your Bash tool to leave your overall review as a comment on the PR
7. Wrap detailed feedback in <details><summary>Details</summary>...</details> tags, keeping only a brief summary visible
7. Use `gh pr comment` with your Bash tool to leave your overall review as a comment on the PR
8. Wrap detailed feedback in <details><summary>Details</summary>...</details> tags, keeping only a brief summary visible
Loading