genesis-reviewerlisted

## Tarefa Revisar o código em busca de bugs, anti-patterns e drift arquitetural. Produza um relatório de findings com severidade e sugestão de correção para cada item. Execute os passos abaixo **na ordem**. ## O que revisar (e o que não revisar) **Revisar:** 1. **Bugs reais** — não estilo, não opinião — bugs que vão quebrar em produção 2. **Anti-patterns** — código que vai criar dívida técnica significativa 3. **Drift arquitetural** — código que viola as decisões documentadas nos ADRs 4. **Oportunidades de simplificação** — complexidade desnecessária 5. **Segurança** — vulnerabilidades óbvias **Não revisar:** - Estilo subjetivo ("eu preferiria escrever assim") - Otimizações prematuras sem evidência de problema - Preferências pessoais de nomenclatura quando o código é consistente --- ## Processo de revisão ### 1. Ler contexto ``` .genesis/architecture/patterns.md → convenções adotadas .genesis/architecture/adrs/ → decisões arquiteturais .genesis/contracts/openapi.yaml → contratos de API ``` ### 2. Revisar cada arquivo modificado Para cada arquivo, analise: #### Bugs críticos (severidade P0) - Exceptions não tratadas que crasham o servidor - Race conditions - Queries N+1 sem índice em tabelas grandes - Credentials ou dados sensíveis em logs - SQL injection, XSS, CSRF, SSRF - Integer overflow em cálculos financeiros - Mutex não liberado, deadlock #### Anti-patterns (severidade P1) - Lógica de negócio no controller/router (deve estar no service) - Q