Skill community beginner
SQL Review
A review pass that catches correctness bugs before the query reaches production.
Reviews a SQL statement for fan-out joins, NULL-unsafe predicates, implicit casts, non-deterministic ordering, and functions that silently change grain.
npx de-agent-platform add sql-review At a glance
| What | A review pass that catches correctness bugs before the query reaches production. |
|---|---|
| Where | SQL · tech-sql · tech-duckdb |
Our assessment
Needs improvement
- The 11/12 evaluation with 0 false positives is self-reported on the author's own fixture; independent verification pending.
Learning objectives
- Prove a join cannot fan out before writing it
- Spot NULL-unsafe `NOT IN` and `=` predicates
- State the grain of a result set in one sentence
Security scan
Last sweep 2026-10-24: 0 finding(s), 0 marked as examples.
Details
SQL Review
The first question
What is the grain of this result? If the author can't answer, stop.
Rules
- Every join states its cardinality (1:1, N:1, N:N) and justifies it.
NOT IN (subquery)is banned where the subquery can return NULL — useNOT EXISTS.- Comparisons across types must be explicit casts, not implicit.
ORDER BYwithoutLIMITin a view is meaningless — drop it.SELECT *in a persisted model freezes an unstable schema.
Deliverable
Findings with the exact predicate, the failure case, and a rewritten clause.
Registry stats
| Verification | community · Sep 30, 2026 |
|---|---|
| Completeness | 96 |
| Trust multiplier | ×1 |
| Composite | 39.9 · adequate |
| Freshness | 24 days since last source check |
| Difficulty | beginner |
| Version | 0.3.0 |
| License | Apache-2.0 |
| Security flags | 1 |
Did this solve your problem?
SQL Review