---
name: code-review
description: Rubrica de revisão de código em cinco dimensões (correção, legibilidade, arquitetura, segurança, performance) com classificação por severidade e veredito. Use ao revisar mudanças de código, arquivos ou PRs antes do merge.
---

# Revisão de código — rubrica de Staff Engineer

Avalie toda mudança nas cinco dimensões abaixo. Comece pelos testes (eles revelam intenção e cobertura) e leia a tarefa/critérios de aceite antes do código.

## 1. Correção
O código faz o que a spec/tarefa pede? Casos de borda tratados (nulo, vazio, limites, caminhos de erro)? Os testes verificam o comportamento certo? Há condições de corrida, erros de "off-by-one" ou estados inconsistentes?

## 2. Legibilidade
Outro engenheiro entende sem explicação? Nomes descritivos e consistentes com o projeto? Fluxo de controle simples (sem aninhamento profundo)? Código bem organizado?

## 3. Arquitetura
Segue padrões existentes ou introduz um novo (justificado e documentado)? Fronteiras de módulo preservadas, sem dependências circulares? Nível de abstração adequado (nem acoplado demais, nem over-engineered)? Dependências fluem na direção certa?

## 4. Segurança
Entrada do usuário validada/sanitizada nas fronteiras? Segredos fora do código, logs e versionamento? AuthN/AuthZ checadas onde necessário? Queries parametrizadas e saída codificada? Novas dependências com vulnerabilidades conhecidas?

## 5. Performance
Padrões N+1? Loops ou busca de dados sem limite? Operações síncronas que deveriam ser assíncronas? Re-renders desnecessários (UI)? Paginação faltando em endpoints de lista?

## Classificação por severidade

- **Crítico** — impede funcionamento ou causa risco grave (vulnerabilidade, perda de dados). Não aprove com itens Críticos.
- **Alto** — bug importante ou falha arquitetural; corrija antes do merge.
- **Médio** — problema relevante, mas contornável.
- **Baixo** — legibilidade, estilo, otimização opcional.

## Modelo de saída

```md
## Resumo da revisão
**Veredito:** Aprovado | Aprovado com ressalvas | Reprovado
**Visão geral:** [1-2 frases sobre a mudança e a avaliação]

### Críticos
- [arquivo:linha] [descrição + correção recomendada]
### Altos
- [arquivo:linha] [descrição + correção recomendada]
### Médios / Baixos
- [arquivo:linha] [descrição]
### O que está bem feito
- [sempre cite ao menos um ponto positivo específico]
### Verificação
- Testes revisados: [sim/não, observações]
- Build/suite verificada: [sim/não]
- Segurança checada: [sim/não, observações]
```

## Regras
Todo achado Crítico/Alto vem com uma correção específica. Relacione tudo aos critérios de aceite. Se estiver incerto, diga e sugira investigação em vez de adivinhar.
