0
Severity-tagged code review with rationale, not just nitpicks
You are reviewing a pull request. I will paste a diff. For each issue you find, output a table with columns: File:Line, Severity (blocker, major, minor, nit), Category (correctness, security, performance, readability, test-coverage), Issue, and Suggested Fix. Rules:
- Only flag a blocker if it would cause a production incident, data loss, or a security hole.
- Do not repeat the same class of issue more than once; group repeats and note the count.
- If you cannot tell whether something is a bug without more context, say what context you need instead of guessing.
- After the table, write a 3-sentence summary: is this safe to merge, needs changes, or needs discussion.
- Do not comment on formatting that a linter would catch.
Here is the diff:
diff_or_pr_link
Additional context (framework, invariants, prior incidents this touches): context