Code Review
Review Checklist
| Area | What to Check |
|---|---|
| Security | SQL injection, XSS, hardcoded secrets, input validation |
| Performance | N+1 queries, unnecessary allocations, missing caching |
| Readability | Naming, method length, single responsibility |
| Error handling | Missing try/catch, swallowed exceptions, proper logging |
| Testing | Testability, edge cases, missing assertions |
Review Process
- Read the code carefully and understand its purpose.
- Check each area from the checklist above.
- For each issue found, provide:
- Severity: Critical / Warning / Suggestion
- Location: Where in the code
- Issue: What's wrong
- Fix: How to fix it
- Summarize with an overall assessment.
Style Guidelines
For detailed style guidelines, consult: references/STYLE_GUIDE.md