Padrões de Revisão de Código

## Checklist de Revisão de PR do Node
 
### Bloqueadores (devem ser corrigidos)
- [ ] Nenhum promise flutuante (`@typescript-eslint/no-floating-promises`)
- [ ] Consultas de banco de dados parametrizadas (sem concatenação de strings SQL)
- [ ] Nenhum segredo, token ou PII em logs
- [ ] `try/finally` ou `using` para conexões/streams adquiridos
- [ ] Migrações retrocompatíveis com deploy N-1
- [ ] Verificação de AuthZ em cada rota mutável
 
### Devem ser corrigidos
- [ ] Timeouts em requisições HTTP de saída (`AbortSignal.timeout`)
- [ ] Limite de taxa em endpoints públicos
- [ ] Testes para caminhos de erro, não apenas o caminho feliz
 
### Nits (opcional)
- [ ] Clareza de nomes, comentários

// BLOQUEADOR: promise flutuante - o handler retorna antes que o trabalho seja concluído
app.get("/bad", (req, res) => {
  fetchData().then((d) => res.json(d)); // sem .catch
});
 
// OK: handler assíncrono com propagação explícita de erro (Express 5 passa para o middleware de erro)
app.get("/good", async (req, res) => {
  const data = await fetchData();
  res.json(data);
});

// BLOQUEADOR: injeção de SQL
const id = req.params.id;
await pool.query(`SELECT * FROM users WHERE id = ${id}`);
 
// OK: parametrizado
await pool.query("SELECT * FROM users WHERE id = $1", [id]);

// BLOQUEADOR: PII em logs
req.log.info({ email: user.email, ssn: user.ssn });
 
// OK: identificadores redigidos
req.log.info({ userId: user.id, action: "login" });

// BLOQUEADOR: vazamento de conexão no caminho de erro
const client = await pool.connect();
const result = await client.query("SELECT 1");
client.release();
 
// OK: sempre libera
const client = await pool.connect();
try {
  return await client.query("SELECT 1");
} finally {
  client.release();
}

const REDACT = ["password", "token", "email", "ssn", "authorization"];
// Usar caminhos de redação do pino: ['req.headers.authorization', 'body.password']

[ ] Expandir passo apenas neste PR?
[ ] Migração de descida existe OU correção para frente documentada?
[ ] Indexar CONCURRENTLY para tabelas grandes?
[ ] Carga útil do job do worker ainda compatível?