specsfy-specialist-code-review

Compare original and translation side by side

🇺🇸

Original

English

🇨🇳

Chinese

Acionar quando houver um diff, branch ou PR concreto para avaliar antes de mergear ou publicar.
Acionar também quando o pedido for "segunda opinião" sobre uma mudança já escrita, mesmo sem PR aberto.
Não acionar para desenhar uma decisão estrutural nova do zero — use
```
$specsfy-specialist-software-architecture
```
; a revisão avalia o que já foi decidido e escrito, não substitui a decisão.
Combinar com
```
$specsfy-specialist-application-security
```
quando o diff tocar autenticação, autorização, dados sensíveis ou entrada externa, e com
```
$specsfy-specialist-domain-modeling
```
quando o achado for sobre nomes, invariantes ou fronteiras de domínio confusas.

当存在具体的diff、分支或PR需要在合并或发布前评估时触发。
当用户请求对已编写的变更给出“第二意见”时也可触发，即使未开启PR。
不要用于从零开始制定新的结构性决策——请使用
```
$specsfy-specialist-software-architecture
```
；审查仅针对已确定和编写的内容进行评估，不能替代决策过程。
当diff涉及认证、授权、敏感数据或外部输入时，可结合
```
$specsfy-specialist-application-security
```
使用；当发现领域名称、不变量或边界存在混淆时，可结合
```
$specsfy-specialist-domain-modeling
```
使用。

Fixar a base de comparação e o escopo exato do diff; revisar um diff sem base clara produz achados sobre código que a mudança não tocou.
Ler spec, issue, critérios de aceite e instruções do repositório aplicáveis antes de julgar; sem isso, "correto" vira opinião pessoal.
Mapear cada arquivo alterado para o comportamento e o boundary que ele afeta (dado, permissão, contrato de API, config, dependência).
Avaliar corretude, casos de borda, segurança, concorrência e efeito operacional antes de estilo — estilo só bloqueia quando automação não o cobre.
Inspecionar os testes pela evidência que fornecem: eles falhariam sem a correção, ou só cobrem a linha sem provar o comportamento?
Confirmar cada achado suspeito lendo o código real e os chamadores/ consumidores antes de reportar — reduz falso positivo.
Relatar por severidade, com localização exata (
```
arquivo:linha
```
), condição que dispara a falha, impacto e correção provável.

Priorizar bugs e risco concreto; não transformar preferência de estilo em bloqueador.
Cada achado descreve condição de entrada, consequência observável e a evidência que comprova (linha, teste, log).
Considerar compatibilidade com clientes existentes, concorrência, plano de rollback e observabilidade da mudança, não só o caminho feliz.
Verificar se o teste adicionado falharia sem a correção real — teste que passa antes e depois da mudança não prova nada.
Distinguir escopo ausente do PR (bloqueador de merge) de melhoria futura opcional (comentário, não bloqueio).
Não repetir achado que lint, formatter ou type checker automatizado já cobre; aponte só o que a automação não vê.
Declarar explicitamente "nenhum achado nesta lente" quando for o caso, sem linguagem que implique ausência de risco além do observado.

Bloquear por gosto pessoal de nome de variável ou formatação quando o projeto já tem linter configurado para isso — desperdiça o orçamento de atenção da revisão nos achados que importam.
Aprovar porque "os testes passam", sem checar se o teste novo de fato cobre o comportamento da mudança (teste tautológico ou sem asserção real).
Revisar arquivo por arquivo sem montar o fluxo entre eles — perde efeitos cruzados, como uma função que muda de assinatura sem todos os chamadores ajustados.
Reportar "parece inseguro" sem apontar o vetor concreto — achado de segurança sem trust boundary e entrada específica não é acionável.

Revisar o diff completo, incluindo arquivos de configuração, migrations e chamadores/consumidores fora do diff que o comportamento afeta.
Conferir cada achado relatado contra o estado real do código e da suíte de testes antes de publicar — achado não confirmado não entra no relatório.
Ordenar a lista final por severidade (probabilidade × impacto), não pela ordem em que os arquivos aparecem no diff.
Resumir cobertura da revisão (o que foi olhado) e risco residual (o que não foi possível confirmar) ao final.
Não declarar um diff "seguro" ou "correto" sem a evidência acima — linguagem absoluta sem prova é proibida.