Revisão
Use esta habilidade para avaliar trabalho que já existe. A revisão nunca cria a mudança que está julgando.
Este é o fluxo de trabalho de revisão canônico. merge-readiness direciona para cá, e
ai-slop-cleaner é uma pista opcional dentro dele.
Objetivo
Encontrar o que está realmente errado, classificado por gravidade, com detalhes suficientes para agir.
Fluxo de Trabalho
- Estabelecer o que mudou e qual era a intenção.
- Ler a mudança contra essa intenção.
- Verificar correção primeiro, depois risco, depois simplificação.
- Verificar cada achado candidato antes de relatá-lo.
- Relatar achados do mais grave ao menos grave.
O que verificar
- Correção — defeitos lógicos, arestas, caminhos de erro, concorrência
- Risco — limites de segurança, operações destrutivas, integridade de dados
- Reuso — utilitários ou padrões existentes que a mudança deveria ter usado
- Simplificação — código que poderia ser excluído ou colapsado
- Cobertura — comportamento que é enviado sem teste
Regras
- Canais separados: o revisor não pode ser o mesmo contexto ativo do autor.
- Verificar antes de relatar. Um achado que parece plausível mas não se reproduz é ruído.
- Declarar gravidade honestamente; não inflar a lista para parecer abrangente.
- "Nenhum achado" é um resultado válido quando o trabalho é sólido.
- Consultivo por padrão — a revisão informa, não bloqueia. Portões rígidos (lançamento, segurança, operações destrutivas) permanecem separados e falham fechados.
Saída
- Achados, do mais grave ao menos grave, cada um com arquivo, linha e cenário de falha concreto
- O que foi verificado e considerado limpo
- Qualquer coisa que não pôde ser avaliada