Code Review
A revisão de código é parte essencial da nossa metodologia. Ela segue dois eixos independentes, para que nenhum contamine o outro:
Dois eixos
- Padrões — a mudança segue os padrões de código do repositório e uma baseline de boas práticas (ex: evitar code smells).
- Spec — a mudança é fiel à issue/requisito que a originou.
A revisão em dois eixos roda como sub-agentes paralelos, para que a análise de padrões não "polua" a análise de fidelidade ao spec.
O que revisar
- Minimal diff — a mudança faz só o que precisa, nada a mais.
- Sem over-engineering — resolve o problema atual, não hipotéticos futuros.
- Testes — a mudança está coberta por testes significativos?
- Commits limpos — uma mudança lógica por commit.
Regra de ouro
Ler antes de escrever. Entenda o código existente antes de propor mudanças.