Agent skill
review-pr
Review a PR diff for quality, security, and correctness.
Install this agent skill to your Project
npx add-skill https://github.com/majiayu000/claude-skill-registry/tree/main/skills/other/other/review-pr-daemn256-workspace-repo-templ
SKILL.md
Review PR
The Reviewer drives this workflow. Analyze a PR's diff for quality, security, and correctness. Produce structured feedback and a verdict.
Prerequisites: PR diff available (pasted or accessible), context about the PR's purpose.
Phase 1: Understand Scope
Understand what the PR is trying to accomplish.
Steps
- Read PR title and description
- Identify the issue being addressed
- Understand the intended behavior change
- Note the scope (files, areas affected)
Phase 2: Analyze Changes
Review the implementation against requirements and standards.
Steps
- Check structure — Are changes organized logically?
- Check correctness — Does implementation achieve the goal?
- Check conventions — Does it follow established patterns?
- Check security — Any security implications?
- Check tests — Adequate coverage?
Review Principles
| Principle | Application |
|---|---|
| Focus on what matters | Blocking issues > style nits |
| Be specific | File, line, concrete suggestion |
| Explain why | Help author understand, not just comply |
| Acknowledge good work | Reinforcement helps |
| Stay in scope | Review what's in the PR, not wishlist |
Phase 3: Provide Feedback
Present structured, actionable feedback.
Steps
- Determine verdict (Approve, Request Changes, Comment)
- List blocking issues (must fix before merge)
- List suggestions (non-blocking improvements)
- Note positive observations
Feedback Categories
| Category | Meaning |
|---|---|
| Blocking | Must be addressed before merge |
| Important | Should be addressed, but not blocking |
| Suggestion | Nice to have improvement |
| Nitpick | Style preference, optional |
Output
## Context Anchors
- **PR:** #<number> - <title>
- **Author:** <author>
- **Target:** `<branch>` → `<base>`
- **Scope:** <brief description>
## Review Summary
**Verdict:** <Approve | Request Changes | Comment>
<One paragraph summary>
## Feedback
### Blocking Issues
1. **[File:Line] <Issue title>**
- Problem: <what's wrong>
- Suggestion: <how to fix>
### Suggestions
1. **[File:Line] <Suggestion title>**
- Current: <what it does now>
- Suggested: <what would be better>
### Positive Notes
- <positive observation>
## Next Step
<If Request Changes: "Address blocking issues and re-request review">
<If Approve: "Ready to merge">
<If Comment: "Consider suggestions; no changes required">
```
### ⛔ CHECKPOINT
**STOP.** Present review for confirmation before posting.
### Board Integration
When issuing an **APPROVE** verdict and merge is confirmed in the same session:
1. Read `workspace.config.yaml` for `board.project_id`, `board.fields.status.field_id`, and `board.status_options.done.option_id`
2. Set issue status to **Done**
Only perform this update if the full approve → merge cycle completes within this review session. If merge is deferred, the Done transition belongs to Orchestrator.
---
## Mechanical Review Checklist
Before providing any review verdict, verify these mechanically:
- [ ] All changed files reviewed (none skipped)
- [ ] Build passes on the branch
- [ ] Tests pass with counts reported
- [ ] No unresolved merge conflicts
- [ ] Commit messages follow conventional format
- [ ] Branch naming follows convention
- [ ] PR description references the issue
- [ ] No `TODO` or `FIXME` markers in new code (unless tracked by issue)
---
## Error Handling
| Error | Recovery |
| ------------------------ | --------------------------------------------------------------- |
| Missing PR context | Request PR number or diff |
| Unclear feedback | Ask for clarification |
| Conflicting requirements | Escalate to author/maintainer |
```
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
agent-ops-spec
Manage specification documents in .agent/specs/. Use when user provides requirements, acceptance criteria, or feature descriptions that need to be tracked and validated against implementation.
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
agent-ops-testing
Test strategy, execution, and coverage analysis. Use when designing tests, running test suites, or analyzing test results beyond baseline checks.
agent-ops-state
Maintain .agent state files. Use at session start, after meaningful steps, and before concluding: read/update constitution/memory/focus/issues/baseline consistently.
Didn't find tool you were looking for?