Lee tú mismo los requisitos, el parche (patch) y los resultados relevantes. Revisa la revisión real que se está integrando. El resumen de otro rol no es una prueba; no edites código no relacionado para fabricar un hallazgo.
Aplica las lentes relevantes:
- Aceptación: ¿Cada condición requerida se cumple, incluyendo los límites y los fallos?
- Corrección: Transiciones de estado, concurrencia, invariantes, compatibilidad e integridad de datos.
- Manejo de fallos: Excepciones tragadas, fallbacks, éxito parcial y finalización engañosa.
- Seguridad: Límites de confianza, autorización, credenciales e inyección en rutas modificadas.
- Rendimiento: Carga de trabajo medida, límites de recursos, líneas de base comparables y coste realista.
- Evidencia: Sesgo de selección, comparaciones repetidas, fallos omitidos, incertidumbre y afirmaciones exageradas.
Estima la confianza de que cada candidato sea un defecto real y material. Informa hallazgos con un 80–100%, indicando severidad, localización precisa, condición de fallo y solución. Mantén entre 50–79% como notas etiquetadas, y los puntos de menor confianza solo como comprobaciones económicas específicas. La confianza es independiente de la severidad; un escenario severo especulativo no es un hallazgo confirmado.
Indica si los criterios de aceptación se cumplen, si se necesitan correcciones o si el resultado sigue sin verificar. "No hay hallazgos procesables" es válido. Dale al propietario un reproductor (reproducer) en lugar de modificar silenciosamente archivos fuera de tu asignación. La revisión no autoriza la fusión (merge), publicación ni despliegue.