Revisão de Código para SQL

## Checklist de Revisão de PR SQL (copiar no comentário da revisão)
 
### Migrações
- [ ] Versão única; nome do arquivo tem ID do ticket
- [ ] Risco de lock: baixo / médio / alto (Skill de Segurança de Migração)
- [ ] CONCURRENTLY para índice de produção em tabelas grandes
- [ ] SQL de Rollback em db/rollback/ ou corpo do PR
- [ ] Ordem de deploy: migração antes da aplicação
 
### Queries
- [ ] EXPLAIN (ANALYZE, BUFFERS) de staging anexado
- [ ] Sem regressão de seq scan em tabelas grandes
- [ ] Queries parametrizadas ($1 / binds nomeados)
- [ ] LIMIT em endpoints de lista
 
### Segurança
- [ ] Privilégio mínimo da role (não superusuário)
- [ ] search_path definido ou nomes com schema qualificado
- [ ] Políticas RLS se tabela de tenant

EXPLAIN (ANALYZE, BUFFERS)
SELECT id, total, created_at
FROM orders
WHERE tenant_id = $1 AND status = 'open'
ORDER BY created_at DESC
LIMIT 100;

Index Scan using idx_orders_tenant_status on orders
  Buffers: shared hit=120 read=4
  Execution Time: 1.2 ms

SELECT * FROM orders WHERE tenant_id = $1 OR tenant_id = $2;
-- Sem EXPLAIN, sem LIMIT, OR impede índice em alguns planejadores

Solicitar alterações:
1. Anexar EXPLAIN (ANALYZE, BUFFERS) de staging
2. Substituir OR por padrão IN (...) ou UNION ALL
3. Listar colunas explicitamente; remover SELECT *
4. Ver Skill de Revisão EXPLAIN para rubrica de aprovação/falha

# RUIM: N+1
for order in orders:
    items = session.query(Item).filter_by(order_id=order.id).all()
 
# BOM: evidência de joinedload no PR
orders = session.query(Order).options(joinedload(Order.items)).filter(...)

-- rollback/V20260709_128__orders_priority_down.sql
DROP INDEX CONCURRENTLY IF EXISTS idx_orders_priority;
ALTER TABLE orders DROP COLUMN IF EXISTS priority;