Cuándo detenerte a revisar — autorrevisión antes de push
Hasta ahora revisamos un PR que ya estaba abierto, después de que el CI ya había corrido. Esta lección retrocede un paso más, al momento antes incluso de hacer push: el momento en que el agente terminó de editar código, y todavía estás a solas con el diff, antes de que se convierta en el problema de
Antes de salir de casa con la maleta armada, la abres de nuevo y revisas con tus propios ojos qué hay realmente adentro. No alcanza con que el cierre esté cerrado y se vea prolijo por fuera — necesitas asegurarte de que no entró nada que no pusiste a propósito.
- autorrevisión antes de push
- El hábito de leer el diff completo y correr comprobaciones locales antes de hacer push del código, para detectar un problema antes de que se convierta en un PR público que otra persona tenga que exponer.
- git diff --stat
- Un resumen rápido de cada archivo modificado y cuántas líneas cambiaron en cada uno — el primer paso de la autorrevisión, antes de entrar en los detalles de cualquier archivo.
- efecto secundario silencioso
- Una edición que aparece en el mismo diff pero no tiene relación con la tarea solicitada; pasa el CI en silencio porque ningún test existente la comprueba, y solo la lectura del diff en sí la expone.
- comprobación local antes de push
- Correr lint y tests en tu propia máquina antes de hacer push, para detectar problemas en el momento más barato posible para corregirlos — antes de que se conviertan en feedback público en un PR abierto.