code-quality-evidence-review
Auditoría transversal de calidad basada en evidencias
Objetivo: obtener conclusiones comprobables, con coste de verificación proporcional al riesgo. Esta skill no presupone Kotlin, Jenkins, arquitectura hexagonal ni una plataforma CI determinada. Los principios de arquitectura y diseño son hipótesis de revisión: solo se convierten en reglas exigibles si el proyecto las adopta o existe una consecuencia técnica demostrable.
Procedimiento (cada paso tiene criterio de cierre)
-
Fija alcance, invariantes y baseline. Determina tipo de revisión (
PR,focal,repositorio,pre-release), revisión exacta (git rev-parse HEAD, árbol sucio, base del diff), lenguajes, módulos, producto, despliegue y límites de acceso. Consulta instrucciones del proyecto (AGENTS.md, ADR, tests, README y políticas CI) sin presumir que estén actualizadas. No ejecutes scripts del repo no confiable ni envíes código privado a servicios sin autorización. Cierre: se sabe qué se audita, frente a qué se compara y qué es una norma del proyecto frente a una preferencia del auditor. Si no hay checkout, delimita la evidencia disponible. -
Prepara el mapa de evidencia. Distingue fuentes: código de producción, build/lockfiles, grafo de dependencias, tests y resultados, CI por SHA, ejecución y documentación. Enumera entradas, salidas, límites de confianza, consumidor/productor y rutas de ejecución relevantes. Usa
references/01-evidence-workflow.mdy, si hay varios lenguajes,references/04-language-recipes.md. Cierre: inventario de componentes, rutas revisadas, trazas de comprobación y lagunas; las búsquedas negativas incluyen patrones, exclusiones y alcance. -
Selecciona dimensiones sin imponer tecnologías. Para revisión global cubre las 12 dimensiones de
references/02-dimensions-playbook.md; para revisión focal cubre las relacionadas con el diff, los contratos consumidores y riesgos transversales, dejando explícito el resto. Para arquitectura y connascence usareferences/03-architecture-connascence.md; para seguridad,references/07-security-supply-chain.md. Cierre: cada dimensión en alcance tiene pregunta, método de obtención, evidencia oNO VERIFICADO;NO APLICAexige justificación. -
Escoge mecanismos y herramientas por hipótesis. Empieza por Git, lectura dirigida y comprobaciones del build/test ya disponibles. Si un dato requiere una herramienta, consulta el anexo
references/05-tool-catalog.md: selecciona la opción mínima adecuada al lenguaje, coste, sensibilidad de datos y capacidad de exportar evidencia. Si falta una herramienta, consultareferences/08-installation-managers.mdpara ofrecer rutas verificables con asdf-vm, mise y/o Homebrew; detecta primero los gestores presentes y diferencia una CLI de una dependencia del proyecto. No instales herramientas ni edites archivos del repositorio sin permiso. Las herramientas son opcionales y no constituyen dependencias de esta skill. Una alerta automática es un candidato, no un defecto probado. Cierre: cada herramienta elegida responde a una pregunta y se registra versión, configuración, comandos y limitaciones, o se describe una alternativa manual. -
Contrasta los candidatos. Recupera
archivo:líneasy símbolo, vincula dos lados de cada dependencia o incompatibilidad, reproduce el fallo con una prueba mínima segura o formula una verificación pendiente. Distingue explícitamenteEJECUTADO,CÓDIGO,DOCUMENTADO,HISTÓRICO,INFERIDO,NO VERIFICADO. Valida falsos positivos (generados, test fixtures, reflexión, dependencias dinámicas, compatibilidad, invariantes intencionales). Cierre: ninguna inferencia, porcentaje, CVE, afirmación de ausencia o estado de CI se presenta como medida sin respaldo. -
Ejecuta verificaciones proporcionadas al riesgo. Consulta
references/06-test-ci-performance.md: diff → tests focales y contratos afectados; integración/release → gates integrales exigidos en el repositorio y CI del nuevo SHA. Separa fallo real, timeout, flakiness, test omitido y no ejecutado; no cambies configuración, instales herramientas, provoques carga ni lances tests destructivos sin autorización. Cierre: resultados, comandos, entorno, SHA y cobertura del alcance constan o figuran pendientes; un badge o recibo antiguo nunca certifica el HEAD nuevo. -
Emite un informe accionable. Usa
assets/report-template.mdy los criterios de severidad dereferences/01-evidence-workflow.md. Cada hallazgo confirmado: Severidad / Ubicación / Evidencia / Impacto / Recomendación, en ese orden, con criterio reproducible de cierre. Separa defectos confirmados, deuda histórica, riesgos plausibles, propuestas y lagunas. Prioriza acciones sin inventar métricas; no crees hallazgos para completar cuotas. Cierre: otro agente puede repetir cada comprobación y distinguir lo observado de lo que falta medir.