2026-02-18//RANT
Seu Code Review E So Uma Viagem de Ego
"Nao gostei desse nome de variavel." Isso nao e comentario de code review. Isso e disturbio de personalidade vestindo avatar do GitHub.
Code review DE VERDADE checa tres coisas. Arquitetura: essa mudanca se encaixa no design do sistema ou ta brigando com ele? Logica: o codigo realmente faz o que diz que faz, e trata os edge cases? Seguranca: a gente ta introduzindo vulnerabilidade, vazando dado ou confiando em input de usuario?
TODO o resto e preferencia de estilo e deveria ser resolvido por linter. Se seu time nao configurou linter e formatter pra auto-aplicar estilo, isso e problema de tooling, nao comentario de PR. Para de segurar o codigo dos outros como refem porque voce prefere camelCase e a pessoa usou snake_case. Configura o Prettier e segue tua vida, mano.
Os piores sao o povo do "nit." Deixam 47 comentarios, todos com prefixo "nit:" como se esse prefixo absolvesse eles de tudo. "Nit: linha em branco extra." "Nit: eu colocaria esse import acima do outro." "Nit: podemos renomear de 'data' pra 'userData'?" Nao. Nao podemos. Porque faz TRES DIAS que to esperando teu review e voce gastou esse tempo catalogando espaco em branco. Tres dias, cara. Tres.
Quer saber como e um BOM code review? "Essa logica de retry nao tem backoff exponencial, entao se o servico cair a gente vai martelar ele. Aqui vai uma sugestao." ISSO e review. Pega problema real e oferece solucao. "Nao gostei que voce usou reduce em vez de for loop" nao e review. E preferencia se passando por expertise.
Shipa o codigo. Resolve problema real. Para de fazer cosplay de arquiteto so porque voce tem permissao de merge.