Code review between senior engineers

Nível: C1

Objetivo

Negociar estilos, princípios e trade-offs em code review entre seniors em C1, discordando sem desmotivar, citando dado, e fechando a review com decisão. O foco é review entre pares experientes onde a hierarquia não ajuda — só o argumento convence.

Explicação

Code review entre seniors em C1 é diferente de review com juniors: a hierarquia não existe, e só o argumento convence. Quatro movimentos:

1. Discordar com dado, não com opinião

Senior para senior, 'eu acho' não sobrevive. C1 cita: 'I disagree because the data shows that the new design adds 50 ms to the critical path'. Discordar com dado é o que separa o argumento do gosto.

2. Negociar estilos com princípio, não com preferência

'Estilo de indentação' é preferência; 'SRP sugere' é princípio. C1 cita: 'I prefer a different style, but SRP is the principle here; let's follow the principle and discuss the style separately'.

3. Reconhecer quando o outro está certo

C1 não defende posição por orgulho. `I see your point; the data is on your side; let's go with your proposal` é maturidade. Defender posição errada custa credibilidade.

4. Fechar a review com decisão

Review entre seniors sem fechamento vira debate infinito. C1 fecha com: `I will go with your proposal; the data supports it. Let's move forward.` Decisão + razão + movimento.

Exemplos

Erros comuns

• Defender posição por orgulho: 'I was right' quando os dados mostram que não, custa credibilidade.
• Argumentar com opinião: 'eu acho' não convence senior; só dado convence.
• Misturar estilo com princípio: estilo é preferência; princípio é regra; sem separar, o debate trava.
• Não fechar: review entre seniors sem decisão vira debate infinito; C1 fecha com movimento.
• Tomar como pessoal: code review é sobre código, não sobre quem escreveu; C1 mantém o foco no código.

Prática guiada

Reescreva cada movimento abaixo no padrão C1 de review entre seniors.

1. Discordar: cite dado, não opinião.
2. Negociar estilo: separe princípio de preferência.
3. Reconhecer: 'os dados estão do seu lado' é maturidade.
4. Fechar: decisão + razão + movimento.

Resumo

Code review entre seniors em C1 usa dado em vez de opinião, princípio em vez de preferência, reconhecimento quando o outro está certo, e fechamento com decisão. Sem dado, é opinião; sem princípio, é estilo; sem reconhecimento, é orgulho; sem fechamento, é debate infinito.

História: Code review between senior engineers

Nível: C1

História em inglês

Closing a contentious code review with another senior

On Friday, I had a code review with another senior engineer on the caching layer. We disagreed on the cache invalidation strategy: I preferred TTL, she preferred explicit invalidation. I opened with data: the prototype shows that explicit invalidation adds 30 ms to every write, but TTL can serve stale data for up to 5 minutes; for inventory, staleness is acceptable. She acknowledged the data: I see your point; the data is on your side; let's go with TTL. I closed the review: I will go with TTL for this sprint; we can revisit if the data changes. Let's move forward.

Tradução linha por linha

Notas de vocabulário

• the prototype shows that explicit invalidation adds 30 ms to every write = dado, não opinião.
• I see your point; the data is on your side = reconhecimento com maturidade.
• I will go with TTL for this sprint; we can revisit if the data changes = fechamento: decisão + cláusula de revisão + movimento.

Perguntas de compreensão

1. O que diferencia review entre seniors de review com juniors?
2. Como discordar com senior em C1?
3. Por que separar princípio de preferência?
4. Como fechar review entre seniors?

Prática com a história

1. Reescreva 'eu discordo' como discordar com senior em C1.
2. Reescreva 'talvez você esteja certo' como reconhecimento C1.
3. Escreva um fechamento C1 para review entre seniors.