---
description: Review my current changes for bugs, security issues and missing tests
agent: ask
---

Review the changes in this workspace (staged and unstaged diff). Do not edit any files.

Check, in priority order:
1. **Correctness:** logic errors, unhandled errors and null cases, race conditions, wrong async usage, off-by-one.
2. **Security:** missing auth or ownership checks, injection, secrets in code/logs, unvalidated input.
3. **Data and compatibility:** migrations, breaking API/schema changes, missing transactions.
4. **Tests:** new behavior without tests; tests that don't assert anything meaningful.
5. **Maintainability:** confusing names, duplication, overly complex functions.

Skip formatting and anything a linter enforces.

## Output format
- First line: `Verdict: approve | approve with nits | changes requested`.
- Then sections **Blockers**, **Should fix**, **Nits** (omit empty ones). Each item: `file:line` — the problem, why it matters, and a concrete fix (a code snippet if short).
- End with up to three questions for the author if intent is unclear.