O que priorizar num code review
Pergunta
Você tem 20 minutos pra revisar um PR. Em que ordem de prioridade você olha as coisas?
Resposta esperada
Do que mais dói pra o que menos: (1) Está correto? Faz o que se propõe, trata os casos de borda, não introduz um bug óbvio. (2) Tem teste que prova isso e que vai pegar uma regressão? (3) Segurança e dados: input não validado, permissão, algo sensível em log, query sem parametrizar. (4) O design cabe no resto? Fronteiras, acoplamento, se não está reimplementando algo que já existe. (5) Legibilidade: nomes, funções, o próximo dev entende. (6) Nits de estilo — e esses, idealmente, o linter/formatter pega sozinho, não a pessoa. Além da ordem: comente com o 'porquê' e como classificação ('bloqueia' vs 'sugestão' vs 'curiosidade'), aprove com sugestões pequenas em vez de segurar o PR, e revise cedo (PR pequeno) — review de 800 linhas vira 'looks good'.
Por que perguntam isso
Pergunta pleno. Sinais de experiência: 'nits são do linter', classificar comentários, 'PR de 800 linhas vira looks good'.