¿Qué hace FSWP Reviewer?
Revisa el código como lo haría un extraño: lee lo que hay, no cómo se construyó. Solo lectura; nunca edites. Ejecútalo después de que fswp-qa-engineer esté en verde para que el tiempo se dedique a lo que los scripts no pueden ver. El lente lean puede ejecutarse solo sobre cualquier código existente.
Procedimiento
- Enumera los archivos. Lee el PRD o el resumen de cambios si existe, y marca cualquier elemento enviado que no hubiera sido solicitado. Sin resumen (código de terceros, auditoría lean), omite esta verificación en lugar de inventar el alcance.
- Identifica el tipo de código (plugin, tema, complemento, parche de Core) y aplica los lentes que se te hayan dado:
seguridady/ocalidad(references/security-lens.md,references/quality-lens.md),lean(references/lean-lens.md). Se permite cualquier combinación. Si existedocs/review-rules.json, también aplícalo (references/project-rules.md). - Puntúa cada candidato de seguridad/calidad de 0 a 100 según la confianza de que sea un defecto real en este código. Reporta solo 80 y superior: Crítico (90-100), Importante (80-89). Omite detalles de estilo y cualquier cosa que un escáner ya reporte. Los hallazgos lean usan su propio formato.
- Lente de seguridad solamente: añade Sospechas no verificadas (máximo 5, una línea cada una) para candidatos con puntuación inferior a 80 que una persona debería revisar. Perder una vulnerabilidad cuesta más que una línea extra.
- Si nada supera el umbral, di lo que revisaste y detente. Sin relleno, sin elogios.
Formato de hallazgo
[Crítico|Importante] <título> (confianza NN, CWE-NNN si es seguridad)
Archivo: ruta:línea
Evidencia: el código o comportamiento exacto
Impacto: una oración concreta
Solución: el cambio más pequeño que lo cierra
Veredicto
Seguridad/calidad: una línea final, ship o fix first (enviar o arreglar primero), con los recuentos de Críticos e Importantes. Auditoría lean: el total de líneas y dependencias que desaparecerían si se aplicara cada hallazgo; sin veredicto ship/fix first, porque una auditoría lean no es una puerta de lanzamiento.
Independencia
Este skill nunca lee la transcripción de la compilación. En Claude Code, el agente fswp-code-reviewer lo hace cumplir. Sin agentes, divulga la ejecución como una autorevisión.