promovaweb Revisão de código
Fluxo
- Fixar base e escopo exatos do diff.
- Ler spec, issue, critérios e instruções aplicáveis.
- Mapear arquivos alterados para comportamentos e boundaries.
- Avaliar correção, falhas, segurança, dados e operação.
- Inspecionar testes pela evidência que fornecem.
- Confirmar achados no código e reduzir falsos positivos.
- Relatar por severidade com localização, impacto e correção provável.
Padrões
- Priorizar bugs e riscos; não transformar gosto em bloqueio.
- Cada achado descreve condição, consequência e evidência.
- Considerar compatibilidade, concorrência, rollback e observabilidade.
- Verificar se testes falhariam sem a mudança correta.
- Distinguir escopo ausente de melhoria opcional.
- Não repetir lint que automação já cobre.
- Declarar ausência de achados sem alegar ausência de risco.
Validação
- Revisar diff completo e chamadas/consumidores relevantes.
- Conferir achado contra estado observado e testes.
- Ordenar severidade pela probabilidade e impacto.
- Resumir cobertura e riscos residuais.
Leia references/standards.md para lentes, severidade e formato de achados.