code-review

Code Review

Systematically review code for quality, bugs, and security.

Checklist

Process

1. Context - Read requirement/design
2. High-level - Architecture check
3. Line-by-line - Detailed inspection
4. Tests - Check coverage
5. Document - Summarize findings

Output Format

## Code Review: [Name]

### Summary
[Approved / Changes Requested]

### Issues
- 🔴 [file:line] Problem → Suggestion
- 🟡 [file:line] Problem → Suggestion

### Good Patterns 👍
- [What was done well]

Common Issues

# 🔴 Security
password = "hardcoded"          # Bad
password = os.environ["PASS"]   # Good

# 🔴 SQL Injection
f"SELECT * WHERE id={id}"       # Bad
"SELECT * WHERE id=%s", (id,)   # Good

Tips

• Be constructive
• Explain the "why"
• Acknowledge good patterns