Reviewing Code

Priority Levels

Checklist

Blockers (Security & Correctness)

• No secrets/credentials in code
• User input validated/sanitized
• Auth checks present where needed
• Edge cases handled (null, empty, errors)
• No race conditions in async code

High Priority (Performance & Logic)

• No N+1 queries
• Large lists paginated
• Expensive ops not in render loops
• Error handling appropriate

Medium Priority (Maintainability)

• Functions have single responsibility
• No copy-paste duplication
• Naming is clear

Output Format

## Summary
[Approve / Request Changes / Discuss]

## 🚨 Blockers
- `file:line` - Issue and fix

## ⚠️ Suggestions
- `file:line` - Suggestion and why

## 💭 Nits
- `file:line` - Minor improvement

## ✨ Good Stuff
- What was done well

Common Issues

any