5caeff2f1c
Reorder folders based on learning dependencies, complexity, and frequency of use: - 01-slash-commands (unchanged) - Quick wins for beginners - 02-memory (was 03) - Essential foundation - 03-skills (was 05) - Auto-invoked capabilities - 04-subagents (was 02) - Task delegation - 05-mcp (was 04) - External integration - 06-hooks (was 07) - Event automation - 07-plugins (was 06) - Bundled solutions - 08-checkpoints (unchanged) - Safe experimentation - 09-advanced-features (unchanged) - Power user tools Documentation improvements: - Add LEARNING-ROADMAP.md with detailed milestones and exercises - Simplify README.md for better scannability - Consolidate Quick Start and Getting Started sections - Combine Feature Comparison and Use Case Matrix tables - Reorder README sections: Learning Path → Quick Reference → Getting Started - Update all cross-references across module READMEs 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
89 lines
1.6 KiB
Markdown
89 lines
1.6 KiB
Markdown
|
|
|
|
# PR Review Plugin
|
|
|
|
Complete PR review workflow with security, testing, and documentation checks.
|
|
|
|
## Features
|
|
|
|
✅ Security analysis
|
|
✅ Test coverage checking
|
|
✅ Documentation verification
|
|
✅ Code quality assessment
|
|
✅ Performance impact analysis
|
|
|
|
## Installation
|
|
|
|
```bash
|
|
/plugin install pr-review
|
|
```
|
|
|
|
## What's Included
|
|
|
|
### Slash Commands
|
|
- `/review-pr` - Comprehensive PR review
|
|
- `/check-security` - Security-focused review
|
|
- `/check-tests` - Test coverage analysis
|
|
|
|
### Subagents
|
|
- `security-reviewer` - Security vulnerability detection
|
|
- `test-checker` - Test coverage analysis
|
|
- `performance-analyzer` - Performance impact evaluation
|
|
|
|
### MCP Servers
|
|
- GitHub integration for PR data
|
|
|
|
### Hooks
|
|
- `pre-review.js` - Pre-review validation
|
|
|
|
## Usage
|
|
|
|
### Basic PR Review
|
|
```
|
|
/review-pr
|
|
```
|
|
|
|
### Security Check Only
|
|
```
|
|
/check-security
|
|
```
|
|
|
|
### Test Coverage Check
|
|
```
|
|
/check-tests
|
|
```
|
|
|
|
## Requirements
|
|
|
|
- Claude Code 1.0+
|
|
- GitHub access
|
|
- Git repository
|
|
|
|
## Configuration
|
|
|
|
Set up your GitHub token:
|
|
```bash
|
|
export GITHUB_TOKEN="your_github_token"
|
|
```
|
|
|
|
## Example Workflow
|
|
|
|
```
|
|
User: /review-pr
|
|
|
|
Claude:
|
|
1. Runs pre-review hook (validates git repo)
|
|
2. Fetches PR data via GitHub MCP
|
|
3. Delegates security review to security-reviewer subagent
|
|
4. Delegates testing to test-checker subagent
|
|
5. Delegates performance to performance-analyzer subagent
|
|
6. Synthesizes all findings
|
|
7. Provides comprehensive review report
|
|
|
|
Result:
|
|
✅ Security: No critical issues found
|
|
⚠️ Testing: Coverage is 65%, recommend 80%+
|
|
✅ Performance: No significant impact
|
|
📝 Recommendations: Add tests for edge cases
|
|
```
|