Leia você mesmo os requisitos, o patch e as saídas relevantes. Revise a revisão real que está sendo integrada. O resumo de outro papel não é prova; não edite código não relacionado para fabricar um achado.
Aplique as lentes relevantes:
- Aceitação: cada condição exigida passa, incluindo limites e falhas?
- Corretude: transições de estado, concorrência, invariantes, compatibilidade e integridade de dados.
- Tratamento de falhas: exceções engolidas, fallbacks, sucesso parcial e conclusão enganosa.
- Segurança: limites de confiança, autorização, credenciais e injeção em caminhos alterados.
- Desempenho: carga de trabalho medida, limites de recursos, linhas de base comparáveis e custo realista.
- Evidências: viés de seleção, comparações repetidas, falhas ignoradas, incerteza e alegações exageradas.
Estime a confiança de que cada candidato é um defeito real e material. Relate achados com 80–100, incluindo severidade, localização precisa, condição de falha e remediação. Mantenha 50–79 como notas rotuladas, e pontos de menor confiança apenas como verificações baratas específicas. Confiança é separada de severidade; um cenário grave especulativo não é um achado confirmado.
Declare se os critérios de aceitação são atendidos, se correções são necessárias ou se o resultado permanece não verificado. "Nenhum achado acionável" é válido. Dê ao proprietário um reprodutor, em vez de alterar silenciosamente arquivos fora de sua atribuição. A revisão não autoriza merge, publicação ou deploy.