Skill: Revisión de código (cualitativa)

Revisar el diff de una implementación como lo haría un ingeniero senior revisando el PR de un compañero: razonar sobre la intención del cambio, su diseño y arquitectura, y entregar feedback accionable.

NO te comportes como una herramienta de CI. Aquí no se ejecutan pruebas, linter, tipado ni build. Ese plano —el de «¿el código corre y cumple las reglas?»— pertenece al skill quality-check. Este skill responde la otra mitad: ¿resuelve el problema correcto, está bien diseñado y será mantenible?

Skills independientes. Las tres puertas del cierre —quality-check, code-review y trace-validate— son hermanas: ninguna condiciona el veredicto de otra; cada una emite el suyo y escribe su propio informe. Quien las encadena es el orquestador de cierre (work-integrate, pr-create). Ver Relación con otros skills.

Alcance: analiza, razona y propone. Aplica correcciones solo si el usuario lo autoriza explícitamente —o si .sdd-devkit/settings.json tiene verification.codeReview.confirmFix: "never" (ver Política de corrección)—. No hace commit/push/merge sin instrucción explícita.

Informe idempotente. Si no hubo cambios en los archivos (ni en el código de la rama ni en su base) desde la última vez que se generó docs/audits/code-review.md, no se vuelve a revisar: se devuelven el veredicto y el resumen del informe existente. Ver Reutilización del informe (idempotencia).

Entrada mínima: un diff identificable (rama contra su base, o los archivos que el usuario indique) y la intención del cambio. Sin diff no hay nada que revisar; sin intención inferible, preguntar al usuario. No se exige ningún artefacto del plugin: la intención puede venir de un US-XXX/WI-XXX/FT-XXX, de cualquier otro documento de especificación, o de la rama, los commits y la descripción del PR. Ver Origen de la intención.

Alcance del informe

El informe de este skill cubre todo lo que la rama difiere de su base — commiteado o no: el trabajo que se va a integrar o a abrir como PR, más cualquier cambio aún en el working tree. Nunca el repositorio completo.