Revisión de código con IA

En la lección anterior aprendimos a escribir una descripción de PR que resume varios commits. Ahora, una vez que el PR está abierto y el CI ha corrido, llega el paso que es más fácil de saltarse: leer el diff en sí, exactamente como leerías el PR de un desarrollador humano. Un CI en verde significa

Revisar un diff que escribió un agente es como inspeccionar un paquete que te entregan en la puerta antes de firmar el recibo — no basta con que la caja esté sellada y se vea bien por fuera, hay que abrirla y confirmar que lo que hay adentro es realmente lo que pediste, no algo extra.

diff de un Pull Request: La vista completa de cada línea agregada y eliminada en un PR, en formato unified diff; esto es lo que se lee línea por línea para revisar si un cambio hace exactamente lo que se pidió.
request changes (pedir cambios): Un veredicto de revisión que bloquea la fusión hasta que el autor corrija puntos específicos; distinto de un comentario libre — en la práctica impide la fusión hasta que el punto se resuelva.
CI en verde (y lo que no significa): Confirmación de que el código pasó el lint, la verificación de tipos, y las pruebas existentes; no confirma que el diff esté acotado a lo que pidió el ticket, ni revisa valores límite que ninguna prueba cubre.
cambio no relacionado incluido en un diff: Una edición que aparece en el mismo PR pero no está relacionada con la corrección solicitada; incluso si pasa el CI en silencio, debería extraerse o revisarse por separado.