Code review com IA

Na aula anterior aprendemos a escrever uma descrição de PR que resume vários commits. Agora, depois que o PR está aberto e o CI rodou, chega a etapa mais fácil de pular: ler o próprio diff, exatamente como você leria o PR de um desenvolvedor humano. CI verde significa que o código não quebrou um tes

Verificar um diff que um agente escreveu é como inspecionar um pacote entregue na sua porta antes de assinar o recibo — não basta que a caixa esteja lacrada e pareça bem por fora, você precisa abri-la e confirmar que o que está dentro é realmente o que você pediu, não algo a mais.

diff de um Pull Request: A visão completa de cada linha adicionada e removida em um PR, no formato unified diff; é isso que você lê linha por linha para verificar se uma mudança faz exatamente o que foi pedido.
request changes (pedir mudanças): Um veredito de revisão que bloqueia o merge até que o autor corrija pontos específicos; diferente de um comentário livre — ele realmente impede o merge até que o ponto seja resolvido.
CI verde (e o que ele não significa): Confirmação de que o código passou no lint, na verificação de tipos e nos testes existentes; não confirma que o diff está limitado ao que o ticket pediu, nem verifica valores-limite que nenhum teste cobre.
mudança não relacionada incluída em um diff: Uma edição que aparece no mesmo PR mas não está relacionada à correção solicitada; mesmo que passe pelo CI em silêncio, ela deve ser retirada ou revisada separadamente.