DE//AGENT
Explore
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

  1. Every join states its cardinality (1:1, N:1, N:N) and justifies it.
  2. NOT IN (subquery) is banned where the subquery can return NULL — use NOT EXISTS.
  3. Comparisons across types must be explicit casts, not implicit.
  4. ORDER BY without LIMIT in a view is meaningless — drop it.
  5. 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