Revisión
Usa este skill para evaluar trabajo que ya existe. La revisión nunca escribe el cambio que está juzgando.
Este es el flujo de revisión canónico. merge-readiness redirige aquí, y ai-slop-cleaner es un carril optativo dentro de él.
Objetivo
Encontrar lo que realmente está mal, clasificado por gravedad, con suficiente detalle para actuar.
Flujo de trabajo
- Establecer qué cambió y qué se pretendía lograr.
- Leer el cambio en función de esa intención.
- Verificar primero la corrección, luego el riesgo, luego la simplificación.
- Confirmar cada posible hallazgo antes de reportarlo.
- Reportar los hallazgos del más grave al menos grave.
Qué verificar
- Corrección — defectos lógicos, casos límite, rutas de error, concurrencia
- Riesgo — límites de seguridad, operaciones destructivas, integridad de datos
- Reutilización — utilidades o patrones existentes que el cambio debería haber usado
- Simplificación — código que podría eliminarse o colapsarse
- Cobertura — comportamiento que se entrega sin pruebas
Reglas
- Carriles separados: el revisor no debe tener el mismo contexto activo que el autor.
- Verificar antes de reportar. Un hallazgo que suena plausible pero no se reproduce es ruido.
- Indicar la gravedad con honestidad; no inflar la lista para parecer exhaustivo.
- "Sin hallazgos" es un resultado válido cuando el trabajo es sólido.
- Asesoría por defecto — la revisión informa, no bloquea. Las puertas duras (publicación, seguridad, operaciones destructivas) permanecen separadas y fallan de forma cerrada.
Salida
- Hallazgos, del más grave al menos grave, cada uno con archivo, línea y escenario concreto de fallo
- Lo que se verificó y encontró limpio
- Cualquier cosa que no pudo evaluarse