lmiranda
11a05799d3
- CLAUDE.md: Update version 3.0.1 → 3.1.2, projman 3.0.0 → 3.1.0, command count 12 → 13 - README.md: Add debug-report/debug-review to projman commands, add DEBUGGING-CHECKLIST.md to docs table - CANONICAL-PATHS.md: Update version, remove non-existent docs/workflows/, add COMMANDS-CHEATSHEET.md - projman/README.md: Fix "Three-Agent" → "Four-Agent", update architecture to show symlink - pr-review/README.md: Add missing setup commands (initial-setup, project-init, project-sync) - cmdb-assistant/README.md: Add initial-setup.md to architecture - project-hygiene/README.md: Fix invalid hook event name (task-completed → PostToolUse) - doc-guardian/plugin.json: Add missing commands field - code-sentinel/plugin.json: Add missing commands field Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
130 lines
3.1 KiB
Markdown
130 lines
3.1 KiB
Markdown
# pr-review
|
|
|
|
Multi-agent pull request review with confidence scoring and actionable feedback.
|
|
|
|
## Overview
|
|
|
|
pr-review conducts comprehensive code reviews using specialized agents for security, performance, maintainability, and test coverage. Each finding includes a confidence score to reduce noise and focus on real issues.
|
|
|
|
## Commands
|
|
|
|
| Command | Description |
|
|
|---------|-------------|
|
|
| `/pr-review <pr#>` | Full multi-agent review |
|
|
| `/pr-summary <pr#>` | Quick summary without full review |
|
|
| `/pr-findings <pr#>` | Filter findings by category/confidence |
|
|
| `/initial-setup` | Full interactive setup wizard |
|
|
| `/project-init` | Quick project setup (system already configured) |
|
|
| `/project-sync` | Sync configuration with current git remote |
|
|
|
|
## Review Agents
|
|
|
|
| Agent | Focus |
|
|
|-------|-------|
|
|
| **Security Reviewer** | Injections, auth, data exposure, crypto |
|
|
| **Performance Analyst** | N+1 queries, complexity, memory, caching |
|
|
| **Maintainability Auditor** | Complexity, duplication, naming, coupling |
|
|
| **Test Validator** | Coverage, test quality, flaky tests |
|
|
|
|
## Confidence Scoring
|
|
|
|
Findings are scored 0.0 - 1.0:
|
|
|
|
| Range | Label | Action |
|
|
|-------|-------|--------|
|
|
| 0.9 - 1.0 | HIGH | Must address |
|
|
| 0.7 - 0.89 | MEDIUM | Should address |
|
|
| 0.5 - 0.69 | LOW | Consider addressing |
|
|
| < 0.5 | (suppressed) | Not reported |
|
|
|
|
## Installation
|
|
|
|
Add to your project's `.claude/settings.json`:
|
|
|
|
```json
|
|
{
|
|
"plugins": ["pr-review"]
|
|
}
|
|
```
|
|
|
|
Requires Gitea MCP server configuration.
|
|
|
|
## Configuration
|
|
|
|
```bash
|
|
# Minimum confidence to report (default: 0.5)
|
|
PR_REVIEW_CONFIDENCE_THRESHOLD=0.5
|
|
|
|
# Auto-submit review to Gitea (default: false)
|
|
PR_REVIEW_AUTO_SUBMIT=false
|
|
```
|
|
|
|
## Usage Examples
|
|
|
|
### Full Review
|
|
|
|
```
|
|
/pr-review 123
|
|
|
|
═══════════════════════════════════════════════════
|
|
PR Review Report: #123
|
|
═══════════════════════════════════════════════════
|
|
|
|
Summary:
|
|
Files changed: 12
|
|
Lines: +234 / -45
|
|
|
|
Findings: 8 total
|
|
🔴 Critical: 1
|
|
🟠 Major: 2
|
|
🟡 Minor: 3
|
|
💡 Suggestions: 2
|
|
|
|
[Detailed findings...]
|
|
|
|
VERDICT: REQUEST_CHANGES
|
|
═══════════════════════════════════════════════════
|
|
```
|
|
|
|
### Filter Findings
|
|
|
|
```
|
|
/pr-findings 123 --category security
|
|
|
|
# Shows only security-related findings
|
|
```
|
|
|
|
### Quick Summary
|
|
|
|
```
|
|
/pr-summary 123
|
|
|
|
# Shows change overview without full analysis
|
|
```
|
|
|
|
## Output
|
|
|
|
Review reports include:
|
|
- Summary statistics
|
|
- Findings grouped by severity
|
|
- Code snippets with context
|
|
- Suggested fixes
|
|
- Overall verdict (APPROVE/COMMENT/REQUEST_CHANGES)
|
|
|
|
## Verdict Logic
|
|
|
|
| Condition | Verdict |
|
|
|-----------|---------|
|
|
| Any critical finding | REQUEST_CHANGES |
|
|
| 2+ major findings | REQUEST_CHANGES |
|
|
| Only minor/suggestions | COMMENT |
|
|
| No significant findings | APPROVE |
|
|
|
|
## Integration
|
|
|
|
For CLAUDE.md integration instructions, see `claude-md-integration.md`.
|
|
|
|
## License
|
|
|
|
MIT
|