"Não gostei desse nome de variável." Tem comentário de code review que parece distúrbio de personalidade vestindo avatar do GitHub.
O review deveria checar a arquitetura: essa mudança se encaixa no sistema ou briga com ele? Precisa olhar se o código faz o que promete e trata os edge cases. E revisar segurança, incluindo vulnerabilidade, vazamento de dado e confiança em input do usuário.
O resto é preferência de estilo e deveria estar no linter. Se o time não configurou linter e formatter pra aplicar o estilo, falta resolver o tooling. Segurar o código dos outros porque tu prefere camelCase e a pessoa usou snake_case não resolve. Configura o Prettier e segue tua vida, mano.
Os piores são os do "nit". Deixam 47 comentários como se o prefixo absolvesse tudo: "nit: linha em branco extra", "nit: eu colocaria esse import acima do outro", "nit: podemos renomear de 'data' pra 'userData'?" Faz três dias que tô esperando teu review e tu passou esse tempo catalogando espaço em branco. Não vamos renomear agora, cara.
"Essa lógica de retry não tem backoff exponencial. Se o serviço cair, a gente vai martelar ele. Aqui vai uma sugestão." Isso pega um problema real e oferece solução. Já "não gostei que tu usou reduce em vez de for loop" é preferência se passando por expertise.
Quero shipar código e resolver problema real. Permissão de merge não é convite pra fazer cosplay de arquiteto.