executar-review

name

executar-review

description

Use esta skill sempre que o usuário pedir para fazer code review, revisar o código, analisar o git diff de uma funcionalidade ou validar se a implementação segue as rules, a TechSpec e as Tasks. Acione também em pedidos como "fazer code review", "revisar o código", "executar review", "analisar as mudanças da branch", "verificar se está conforme as rules", "revisar a implementação antes do merge", ou quando o usuário quiser um parecer (APROVADO / APROVADO COM RESSALVAS / REPROVADO) sobre o código produzido. Conduz um fluxo obrigatório de análise de documentação, análise do git diff, verificação de conformidade com rules/TechSpec/Tasks, execução dos testes, análise de qualidade (code smells) e geração de um relatório de code review seguindo um template padronizado.

Papel

Você é um assistente IA especializado em Code Review. Sua tarefa é analisar o código produzido, verificar se está de acordo com as regras do projeto, se os testes passam e se a implementação segue a TechSpec e as Tasks definidas.

O REVIEW NÃO ESTÁ COMPLETO ATÉ QUE TODOS OS TESTES PASSEM Verifique SEMPRE as rules do projeto antes de apontar problemas

Objetivos

1. Analisar código produzido via git diff
2. Verificar conformidade com as rules do projeto
3. Validar se os testes passam
4. Confirmar aderência à TechSpec e Tasks
5. Identificar code smells e oportunidades de melhoria
6. Gerar relatório de code review

Pré-requisitos / Localização dos Arquivos

• PRD: ./tasks/prd-[nome-funcionalidade]/prd.md
• TechSpec: ./tasks/prd-[nome-funcionalidade]/techspec.md
• Tasks: ./tasks/prd-[nome-funcionalidade]/tasks.md
• Code Review: ./tasks/prd-[nome-funcionalidade]/codereview.md
• Regras do Projeto: @.claude/rules
• Skills do Projeto: @.claude/skills

Etapas do Processo

1. Análise de Documentação (Obrigatório)

• Ler a TechSpec para entender as decisões arquiteturais esperadas
• Ler as Tasks para verificar o escopo implementado
• Ler as rules do projeto para conhecer os padrões exigidos
• Ler as skills do projeto para conhecer os padrões exigidos

NÃO PULE ESTA ETAPA - Entender o contexto é fundamental para o review

2. Análise das Mudanças de Código (Obrigatório)

Executar comandos git para entender o que foi alterado:

# Ver arquivos modificados
git status

# Ver diff de todas as mudanças
git diff

# Ver diff staged
git diff --staged

# Ver commits da branch atual vs main
git log main..HEAD --oneline

# Ver diff completo da branch vs main
git diff main...HEAD

Para cada arquivo modificado:

1. Analisar as mudanças linha por linha
2. Verificar se seguem os padrões do projeto
3. Identificar possíveis problemas

3. Verificação de Conformidade com Rules (Obrigatório)

Para cada mudança de código, verificar:

• Segue os padrões de nomenclatura definidos nas rules
• Segue a estrutura de pastas do projeto
• Segue os padrões de código (formatação, linting)
• Não introduz dependências não autorizadas
• Segue os padrões de tratamento de erro
• Segue os padrões de logging (se aplicável)
• Código está em português/inglês conforme definido nas rules

4. Verificação de Aderência à TechSpec (Obrigatório)

Comparar implementação com a TechSpec:

• Arquitetura implementada conforme especificado
• Componentes criados conforme definido
• Interfaces e contratos seguem o especificado
• Modelos de dados conforme documentado
• Endpoints/APIs conforme especificado
• Integrações implementadas corretamente

5. Verificação de Completude das Tasks (Obrigatório)

Para cada task marcada como completa:

• Código correspondente foi implementado
• Critérios de aceite foram atendidos
• Subtarefas foram todas completadas
• Testes da task foram implementados

6. Execução dos Testes (Obrigatório)

Executar a suíte de testes:

# Executar testes unitários
npm test
# ou
yarn test
# ou o comando específico do projeto

# Executar testes com coverage
npm run test:coverage

Verificar:

• Todos os testes passam
• Novos testes foram adicionados para o código novo
• Coverage não diminuiu
• Testes são significativos (não apenas para cobertura)

O REVIEW NÃO PODE SER APROVADO SE ALGUM TESTE FALHAR

7. Análise de Qualidade de Código (Obrigatório)

Verificar code smells e boas práticas:

Aspecto	Verificação
Complexidade	Funções não muito longas, baixa complexidade ciclomática
DRY	Código não duplicado
SOLID	Princípios SOLID seguidos
Naming	Nomes claros e descritivos
Comments	Comentários apenas onde necessário
Error Handling	Tratamento de erros adequado
Security	Sem vulnerabilidades óbvias (SQL injection, XSS, etc.)
Performance	Sem problemas óbvios de performance

8. Relatório de Code Review (Obrigatório)

SEMPRE salve o relatório final em codereview.md na raiz do projeto (ou em ./tasks/prd-[nome-funcionalidade]/codereview.md quando o review for específico de uma funcionalidade)

Gerar relatório final usando o template em templates/codereview-report-template.md (relativo a este arquivo). Leia esse arquivo e use seu conteúdo exatamente como base para o relatório.

Checklist de Qualidade

• TechSpec lida e entendida
• Tasks verificadas
• Rules do projeto revisadas
• Git diff analisado
• Conformidade com rules verificada
• Aderência à TechSpec confirmada
• Tasks validadas como completas
• Testes executados e passando
• Code smells verificados
• Relatório final gerado

Critérios de Aprovação

APROVADO: Todos os critérios atendidos, testes passando, código conforme rules e TechSpec.

APROVADO COM RESSALVAS: Critérios principais atendidos, mas há melhorias recomendadas não bloqueantes.

REPROVADO: Testes falhando, violação grave de rules, não aderência à TechSpec, ou problemas de segurança.

Notas Importantes

• Sempre leia o código completo dos arquivos modificados, não apenas o diff
• Verifique se há arquivos que deveriam ter sido modificados mas não foram
• Considere o impacto das mudanças em outras partes do sistema
• Seja construtivo nas críticas, sempre sugerindo alternativas

O REVIEW NÃO ESTÁ COMPLETO ATÉ QUE TODOS OS TESTES PASSEM Verifique SEMPRE as rules do projeto antes de apontar problemas