Revisión de cambios de código

Cuándo usarla

Usar cuando el contexto requiera este procedimiento: Revisa un cambio por corrección, riesgo y mantenibilidad con hallazgos accionables.

Procedimiento

1. Leer el objetivo y las reglas aplicables antes de evaluar estilo o arquitectura.
2. Reconstruir el flujo afectado y enumerar contratos, entradas, salidas y efectos secundarios.
3. Inspeccionar la diferencia en contexto suficiente, incluidos llamadores y pruebas relacionadas.
4. Buscar errores demostrables: condiciones límite, manejo de errores, concurrencia, seguridad, compatibilidad y pérdida de datos.
5. Ejecutar las validaciones pertinentes cuando sea seguro; registrar comandos y resultados reales.
6. Redactar solo hallazgos que el autor pueda actuar, con ubicación, escenario de falla, impacto y corrección sugerida.
7. Separar bloqueantes de mejoras opcionales y evitar observaciones puramente estilísticas ya cubiertas por herramientas.
8. Cerrar con cobertura revisada, pruebas no ejecutadas y veredicto condicionado a la evidencia.

Entradas

• Diferencia o conjunto de archivos modificados.
• Objetivo del cambio.
• Instrucciones del repositorio y pruebas disponibles.

Salidas

• Resumen del cambio.
• Hallazgos priorizados y localizables.
• Resultado de validaciones y riesgos residuales.

Validaciones

• Cada hallazgo señala una conducta defectuosa, no una preferencia.
• La prioridad corresponde al impacto probable.
• Las ubicaciones son precisas y el escenario es reproducible.
• No se afirma que una prueba pasó si no se ejecutó.
• El resumen representa el cambio completo.

Herramientas

• Lector de archivos y diferencias.
• Ejecutores de pruebas o analizadores estáticos son opcionales.

Ejemplos

Al revisar un cambio de paginación, detecta que el cursor se calcula antes del filtro y muestra cómo eso omite resultados en la página siguiente.

Acerca del recurso