sandeshp1995/claude-code-team-kit

Revise alterações de código quanto à correção, legibilidade, testes e problemas de segurança óbvios. Use quando o usuário solicitar uma revisão de suas alterações, um diff ou uma branch, ou antes de abrir um merge request.

O que é claude-code-team-kit?

Este skill para agentes AI (como Claude Code, Cursor, Codex) automatiza a revisão de código com foco em qualidade e segurança. Ele orienta o agente a identificar a base de comparação correta (usando git merge-base contra main, master ou develop, ou diff de alterações não commitadas) e ler o diff completo, abrindo o código circundante quando necessário para avaliar mudanças complexas. A revisão segue uma checklist estruturada: correção (bugs lógicos, tratamento de bordas vazias/None/loops concorrentes), cobertura de testes (comportamento, não implementação), legibilidade (nomes claros, ausência de código morto) e segurança básica (sem secrets, validação de entrada, consultas parametrizadas). Os achados são reportados por severidade: 'Must fix' para bugs e riscos de segurança, 'Should fix' para problemas de manutenibilidade e 'Nit' para polimento opcional, sempre com arquivo:linha, problema e sugestão concreta. O veredito final é uma linha (pronto para merge, pronto após must-fixes ou precisa de nova passagem). O skill não inventa problemas e deixa categorias sem achados de fora. Útil para equipes que querem revisões consistentes e rápidas diretamente no fluxo de trabalho do agente AI.

Funciona com✓Claude Code~Codex CLI~Cursor

npx skills add https://github.com/sandeshp1995/claude-code-team-kit/tree/HEAD/.claude/skills/code-review-basics

Documentação

Noções básicas de revisão de código

Revise as alterações que o usuário apontar. Se ele não especificar, revise a branch atual contra sua base:

Encontre a base: tente git merge-base HEAD origin/main, depois origin/master, depois origin/develop. Se nenhuma existir, revise alterações não commitadas (git diff HEAD).
Leia o diff completo e, em seguida, abra o código circundante para qualquer alteração que você não consiga julgar apenas pelo diff.

Checklist

Percorra cada item. Apenas relate itens com uma descoberta real.

Correção

O código faz o que a alteração pretende? Procure por erros de off-by-one, condições erradas, None/null não tratado e exceções engolidas.
Casos extremos são tratados: entrada vazia, entrada muito grande, chamadas concorrentes, retentativas?

Testes

Cada mudança de comportamento é coberta por um teste que falharia sem ela?
Os testes verificam comportamento, não detalhes de implementação?

Legibilidade

Os nomes são precisos? Um novo membro da equipe entenderia isso em uma leitura?
Há código morto, saída de depuração ou código comentado deixado para trás?

Segurança (básica)

Nenhum segredo, token ou credencial no código, configuração ou logs.
A entrada do usuário é validada e as consultas são parametrizadas, nunca construídas por concatenação de strings.

Consistência

Segue as convenções em CLAUDE.md e no código circundante?

Saída

Agrupe descobertas por gravidade:

Must fix: bugs, problemas de segurança, testes ausentes para lógica arriscada.
Should fix: problemas de manutenibilidade que custarão tempo mais tarde.
Nit: polimento opcional. Mantenha-os poucos.

Para cada descoberta, forneça arquivo:linha, o problema e uma sugestão concreta de correção. Termine com um veredito de uma linha: pronto para merge, pronto após must-fixes ou precisa de nova passagem. Se você não encontrar nada em uma categoria, deixe a categoria de fora. Não invente problemas para preencher espaço.

A edição Pro adiciona checklists completos de segurança, desempenho, acessibilidade e design de API, além de um comando /review que seleciona os itens certos para o diff.