code-review

7 posts

Las Revisiones de Pull Request Detectan el 15-30 % de Defectos. Los Datos lo Dicen Hace 50 Años.

Múltiples estudios en IBM, AT&T, HP y Microsoft confirman que la revisión informal de código detecta aproximadamente una cuarta parte de los defectos. Aquí está lo que los datos realmente dicen, por qué el número es tan bajo y cómo solucionarlo.

La revisión informal de código detecta entre el 15 y el 30 por ciento de los defectos presentes en el código que se revisa. Esa no es una opinión. Es un…

Una checklist genérica no detecta nada. Una estructurada detecta el 60 % de los defectos.

La mayoría de las checklists de revisión son listas copiadas y pegadas de buenas intenciones. Una checklist estructurada al estilo Fagan inspection se construye a partir de datos reales de defectos, está dirigida a tipos de artefactos específicos y se utiliza durante la preparación individual. Así es como se construye una que funciona.

Si tu equipo tiene una checklist para code review, hay una buena probabilidad de que viva en una página wiki que nadie abre. Probablemente dice cosas como…

Un LLM puede pre-inspeccionar tu código. No puede dirigir la reunión.

Las inspecciones Fagan necesitan de cuatro a seis personas y dos horas para revisar 250 líneas. Un LLM puede reducir ese costo gestionando la preparación y el cumplimiento de checklist, pero no puede sustituir los roles humanos que encuentran los defectos más costosos.

Una inspección Fagan completa necesita un moderador, un lector, de dos a cuatro inspectores y el autor. El equipo dedica dos horas a revisar aproximadamente…

Las Fagan Inspections detectaban el 90 % de los defectos antes de las pruebas. Luego dejamos de hacerlas.

El proceso de revisión estructurado de Michael Fagan en IBM atrapaba casi todos los defectos antes de que llegaran al compilador. También consumía el 15-20 % del esfuerzo total del proyecto. Aquí está por qué desapareció el método de revisión más efectivo en la historia del software, y qué es lo que los equipos realmente extrañan.

En 1976, Michael Fagan publicó un artículo en el IBM Systems Journal que describía un proceso de revisión tan eficaz que se convirtió en el estándar de oro…

Tu mejor revisor se pierde la mayoría de los defectos. Fagan lo midió en IBM en 1976.

Incluso los ingenieros senior detectan solo una fracción de los defectos en revisiones no estructuradas. La investigación de Michael Fagan en IBM mostró por qué, y construyó un proceso de inspección estructurado para solucionarlo.

Dos ingenieros senior revisan el mismo pull request. Uno señala un null check faltante. El otro detecta una race condition en el cleanup path. Ninguno…

La mayoría de las code reviews detectan el 20 % de defectos. Las Fagan inspections detectan el 90 %.

La revisión de código informal detecta el 15-30 % de defectos. Las Fagan inspections, un proceso estructurado de 50 años, reportan consistentemente tasas de eliminación del 60-90 %. Aquí se explica cómo funcionan, por qué los equipos las evitan y cómo ejecutar una versión ligera.

La mayoría de las code reviews detectan entre el 15 y el 30 por ciento de los defectos que deberían encontrar. Eso no es una suposición. IBM lo midió en los…

En la era de la AI, la revisión de código se convierte en revisión de especificaciones

Cuando la AI puede generar implementación, tests y contracts a partir de una especificación, el trabajo humano de más alto impacto se mueve aguas arriba. Lo que más escrutinio necesita es la propia especificación.

Si llevas más de unas pocas semanas sacando cambios con AI, probablemente ya conoces esta sensación. Abres un pull request. El código está lo bastante limpio.…