- Scope: [PR #N / branch name / module path / full codebase]
- Review date: [YYYY-MM-DD]
- Review mode: [Change review / Codebase review]
[1-3 sentences: what was reviewed, overall impression, key risk areas.]
[Issues that must be resolved. If none, write "None."]
| # | Severity | Area | Location | Finding | Suggested fix |
|---|---|---|---|---|---|
| 1 | Blocker | [Design/Functionality/Security/...] | file#Lx-Ly |
[Description] | [Direction] |
[Issues strongly recommended to fix. If none, write "None."]
| # | Severity | Area | Location | Finding | Suggested fix |
|---|---|---|---|---|---|
| 1 | High | [Area] | file#Lx-Ly |
[Description] | [Direction] |
[Issues worth addressing. If none, write "None."]
| # | Severity | Area | Location | Finding | Suggested fix |
|---|---|---|---|---|---|
| 1 | Medium | [Area] | file#Lx-Ly |
[Description] | [Direction] |
[Minor suggestions. If none, write "None."]
| # | Severity | Area | Location | Finding | Suggested fix |
|---|---|---|---|---|---|
| 1 | Nit | [Area] | file#Lx-Ly |
[Description] | [Direction] |
[Note well-written code, good patterns, thorough tests, or other strengths.]
| Area | Status | Notes |
|---|---|---|
| Design | [Pass / Needs work / N/A] | |
| Functionality | [Pass / Needs work / N/A] | |
| Complexity | [Pass / Needs work / N/A] | |
| Security | [Pass / Needs work / N/A] | |
| Performance | [Pass / Needs work / N/A] | |
| Code Quality | [Pass / Needs work / N/A] | |
| Tests | [Pass / Needs work / N/A] | |
| Naming | [Pass / Needs work / N/A] | |
| Comments | [Pass / Needs work / N/A] | |
| Style | [Pass / Needs work / N/A] | |
| Documentation | [Pass / Needs work / N/A] |
- No hardcoded secrets
- Input validation present
- Error handling complete
- Types/interfaces defined
- Tests added for new code
- No obvious performance issues
- Code is readable and documented
- Breaking changes documented
- Decision: [approve / request changes / comment only]
- Justification: [1-2 sentences]
- Confidence: [0.0 - 1.0]