sandeshp1995/claude-code-team-kit

Revisa cambios de código por corrección, legibilidad, pruebas y problemas de seguridad evidentes. Úsalo cuando el usuario solicite una revisión de sus cambios, un diff, o una rama, o antes de abrir una solicitud de fusión.

¿Qué es claude-code-team-kit?

Esta habilidad, llamada "sandeshp1995/claude-code-team-kit", está diseñada para asistentes de IA como Claude Code, Cursor o Codex, y proporciona un flujo de trabajo estructurado para la revisión de código en equipo. Su propósito es analizar cambios de código en busca de errores de corrección, problemas de legibilidad, cobertura de pruebas y vulnerabilidades de seguridad básicas, todo ello siguiendo un checklist detallado. El asistente primero detecta automáticamente la rama base del proyecto (probando orígenes como `origin/main`, `origin/master`, etc.) y lee el diff completo, abriendo el código circundante cuando sea necesario para un análisis más profundo. Luego recorre categorías como corrección, pruebas, legibilidad, seguridad y consistencia, y agrupa los hallazgos por gravedad ("Must fix", "Should fix", "Nit"), proporcionando para cada uno la línea exacta, el problema y una sugerencia concreta. Finalmente, emite un veredicto de una línea ("ready to merge", "ready after must-fixes" o "needs another pass"), evitando inventar problemas si no existen. Esta habilidad es especialmente útil para revisiones rápidas y consistentes antes de abrir un merge request, y su versión Pro añade checklists adicionales de seguridad, rendimiento, accesibilidad y diseño de API, más un comando `/review` que selecciona el checklist adecuado según el diff.

Compatible con✓Claude Code~Codex CLI~Cursor

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

Documentación

Conceptos básicos de revisión de código

Revisa los cambios que el usuario indique. Si no los especifica, revisa la rama actual contra su base:

Encuentra la base: intenta git merge-base HEAD origin/main, luego origin/master, luego origin/develop. Si ninguna existe, revisa los cambios sin confirmar (git diff HEAD).
Lee el diff completo, luego abre el código circundante para cualquier cambio que no puedas juzgar solo con el diff.

Lista de verificación

Recorre cada elemento. Solo informa aquellos que tengan un hallazgo real.

Corrección

¿El código hace lo que pretende el cambio? Busca errores off-by-one, condiciones incorrectas, None/null no manejados y excepciones ignoradas.
¿Se manejan casos extremos: entrada vacía, entrada muy grande, llamadas concurrentes, reintentos?

Pruebas

¿Cada cambio de comportamiento está cubierto por una prueba que fallaría sin él?
¿Las pruebas verifican el comportamiento, no los detalles de implementación?

Legibilidad

¿Los nombres son precisos? ¿Un nuevo miembro del equipo lo entendería en una sola lectura?
¿Queda código muerto, salidas de depuración o código comentado?

Seguridad (básica)

No hay secretos, tokens ni credenciales en código, configuración ni registros.
La entrada del usuario está validada y las consultas están parametrizadas, nunca construidas por concatenación de cadenas.

Consistencia

¿Sigue las convenciones del archivo CLAUDE.md y del código circundante?

Resultado

Agrupa los hallazgos por gravedad:

Debe corregirse: errores, problemas de seguridad, pruebas faltantes para lógica arriesgada.
Debería corregirse: problemas de mantenibilidad que costarán tiempo más adelante.
Detalle: mejoras opcionales. Mantenlas pocas.

Para cada hallazgo proporciona archivo:línea, el problema y una sugerencia concreta de corrección. Termina con un veredicto de una línea: listo para fusionar, listo después de correcciones obligatorias o necesita otra revisión. Si no encuentras nada en una categoría, omítela. No inventes problemas para llenar espacio.

La edición Pro añade listas de verificación completas de seguridad, rendimiento, accesibilidad y diseño de API, y un comando /review que selecciona las adecuadas para el diff.