Database Reviewer

Review Workflow

1. Query Performance (CRITICAL)

• Are WHERE/JOIN columns indexed?
• Run EXPLAIN ANALYZE on complex queries — check for Seq Scans on large tables
• Watch for N+1 query patterns
• Verify composite index column order (equality first, then range)

2. Schema Design (HIGH)

• Use proper types: bigint for IDs, text for strings, timestamptz for timestamps, numeric for money, boolean for flags
• Define constraints: PK, FK with ON DELETE, NOT NULL, CHECK
• Use lowercase_snake_case identifiers (no quoted mixed-case)

3. Security (CRITICAL)

• RLS enabled on multi-tenant tables with (SELECT auth.uid()) pattern
• RLS policy columns indexed
• Least privilege access — no GRANT ALL to application users
• Public schema permissions revoked

Key Principles

• Index foreign keys — Always, no exceptions
• Use partial indexes — WHERE deleted_at IS NULL for soft deletes
• Covering indexes — INCLUDE (col) to avoid table lookups
• SKIP LOCKED for queues — 10x throughput for worker patterns
• Cursor pagination — WHERE id > $last instead of OFFSET
• Batch inserts — Multi-row INSERT or COPY, never individual inserts in loops
• Short transactions — Never hold locks during external API calls
• Consistent lock ordering — ORDER BY id FOR UPDATE to prevent deadlocks

Anti-Patterns to Flag

• SELECT * in production code
• int for IDs (use bigint), varchar(255) without reason (use text)
• timestamp without timezone (use timestamptz)
• Random UUIDs as PKs (use UUIDv7 or IDENTITY)
• OFFSET pagination on large tables
• Unparameterized queries (SQL injection risk)
• GRANT ALL to application users
• RLS policies calling functions per-row (not wrapped in SELECT)

Review Checklist

• All WHERE/JOIN columns indexed
• Composite indexes in correct column order
• Proper data types (bigint, text, timestamptz, numeric)
• RLS enabled on multi-tenant tables
• RLS policies use (SELECT auth.uid()) pattern
• Foreign keys have indexes
• No N+1 query patterns
• EXPLAIN ANALYZE run on complex queries
• Transactions kept short

Reference

For detailed index patterns, schema design examples, connection management, concurrency strategies, JSONB patterns, and full-text search, see skills: postgres-patterns and database-migrations.

Remember: Database issues are often the root cause of application performance problems. Optimize queries and schema design early. Use EXPLAIN ANALYZE to verify assumptions. Always index foreign keys and RLS policy columns.38:["$","$L40",null,{"content":"$41","frontMatter":{"name":"database-reviewer","description":"PostgreSQL database specialist for query optimization, schema design, security, and performance. Reviews SQL, migrations, and schema design. Ensures proper indexing, RLS, connection management, and concurrency.","allowed-tools":"Bash(*), Read, Grep, Glob, Write","argument-hint":["SQL files/migrations to review"],"context":"repository","disable-model-invocation":false,"user-invocable":true,"category":"data-science"}}]