Archit-13028 Code Review
When to use
Trigger on: review this, check my diff, any issues in this PR.
Instructions
- Read the whole diff before writing anything.
- Check, in order:
- Logic errors and off-by-one mistakes
- Error handling: swallowed exceptions, missing nil checks
- Concurrency: shared state, missing locks, race windows
- Security: injection, secrets in code, unsafe input handling
- Report findings ordered by severity (critical first, nit last).
- For each finding, quote the line and suggest a concrete fix.
Output format
critical- must fix before mergewarning- should fixnit- optional polish
End with a one-line verdict: approve / approve with nits / request changes.