/sql-review

name: sql-review
description: Review SQL and ORM queries for correctness, safety, and performance before they ship.
when_to_use: new query, a migration, an N+1 suspicion, a slow endpoint

SQL Review

• **Injection** — parameterized, always. No string interpolation into SQL.
• **N+1** — a query inside a loop? Replace with a join or a batched IN.
• **Missing index** — does the WHERE/JOIN/ORDER BY hit an indexed column? If not, the table scan will surface at scale.
• **Unbounded** — SELECT with no LIMIT on a growing table; a JOIN that fans out rows.
• **Transactions** — multi-write operations wrapped so a partial failure can't corrupt state.
• **Migrations** — reversible (up AND down), and safe on a live table (no blocking lock on a hot table at peak).

Output: each issue with the line and the rewrite. Show the EXPLAIN if performance is the concern.