Aura Frog Code Reviewer
Priority: HIGH - Use before merging code
When to Use
USE: After implementation, before merge, when requested
Review Process
1. Get Files to Review
git diff --name-only main...HEAD
2. Multi-Agent Review
| Agent | Checks | |-------|--------| | Security | OWASP Top 10, secrets, SQL injection, XSS | | Dev | KISS principle, structure, error handling | | QA | Coverage, test quality, edge cases | | Performance | Algorithms, memory, queries, bundle size |
3. Generate Report
## Code Review Report
### ✅ Passed Checks
- [List]
### ⚠️ Warnings
- [Non-blocking issues]
### ❌ Issues Found
- **[CRITICAL]** [Issue] at [file:line]
- Fix: [recommendation]
### 📊 Metrics
- Coverage: X% (target: Y%)
- Files: N changed
4. Decision
- ✅ APPROVED - No critical issues
- ⚠️ APPROVED WITH COMMENTS - Minor issues
- ❌ CHANGES REQUESTED - Critical issues
Quick Checklist
Security:
- [ ] No hardcoded secrets
- [ ] Input validation
- [ ] Auth checks in place
Quality:
- [ ] KISS principle
- [ ] No duplication
- [ ] Error handling
- [ ] Follows conventions
Testing:
- [ ] Coverage ≥ target
- [ ] Critical paths tested
- [ ] Edge cases covered
Performance:
- [ ] No N+1 queries
- [ ] Efficient algorithms
- [ ] No memory leaks
Critical (Block Merge)
- Hardcoded secrets
- SQL injection / XSS
- Coverage < target
- Breaking changes without migration
Remember: Review improves code quality. Be constructive.