Revisar el diff
Revisa lo que has cambiado antes de subirlo y separa lo que rompe de lo que solo es gusto personal.
Cuándo se usa
Antes de cada commit importante o de abrir una pull request.
SKILL.md
# Revisar el diff
Revisa los cambios de la rama o del rango indicado. Sin argumento, revisa lo que
haya sin commitear. Lee el archivo entero de lo que cambia, no solo el trozo del
diff: la mitad de los fallos reales están en cómo encaja con lo que ya había.
## Buscar, en este orden
1. **Corrección.** Casos límite sin cubrir: lista vacía, valor nulo, error de
red, doble clic, respuesta que llega tarde.
2. **Estado y efectos.** Efectos sin limpiar, listeners sin quitar, dependencias
mal declaradas, escrituras a un componente ya desmontado.
3. **Datos del usuario.** Todo lo que venga de fuera se valida antes de usarse.
Ni claves ni tokens en el código.
4. **Contrato roto.** Firmas cambiadas sin actualizar quien las llama, campos
renombrados sin migración.
5. **Repetición con peso.** Un patrón copiado tres veces ya es una función. Dos
veces, todavía no.
## Cómo responder
Un hallazgo por línea, ordenados de más a menos grave, con archivo, línea y el
escenario concreto que falla ("si el usuario pulsa dos veces seguidas, la
petición se lanza dos veces"). Si no hay nada que decir, dilo en una línea.
Separa siempre lo que rompe de lo que es preferencia, y marca esto último como
opcional. No reescribas el estilo del autor.