guild-agents 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "guild-agents",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "A multi-agent framework for Claude Code — specialized AI teams for every project",
   "type": "module",
   "files": [

package/src/templates/skills/build-feature/SKILL.md

@@ -85,6 +85,16 @@ When running a single build-feature, a simple `git checkout -b` is sufficient.
 
 **Output:** Codigo implementado + tests + commits realizados
 
+### Gate pre-Review (obligatorio)
+
+Antes de avanzar a Fase 5, ejecutar verificacion automatizada:
+
+1. Ejecuta los comandos de test del proyecto (ej: `npm test`) — si falla, el Developer debe corregir antes de avanzar
+2. Ejecuta los comandos de lint del proyecto (ej: `npm run lint`) — si falla, el Developer debe corregir antes de avanzar
+3. Solo hacer checkpoint commit **despues** de que ambos pasen
+
+Este gate NO se puede saltar, incluso si el usuario solicito skip de fases. Los comandos concretos estan en la seccion "Comandos CLI" de CLAUDE.md.
+
 ### Fase 5 — Review (Code Reviewer)
 
 **Agente:** Lee `.claude/agents/code-reviewer.md` via Task tool
@@ -97,22 +107,16 @@ When running a single build-feature, a simple `git checkout -b` is sufficient.
 **Output:** Reporte de review con hallazgos clasificados
 **Condicion de loop:** Si hay hallazgos de tipo Blocker, vuelve a **Fase 4** para que el Developer los corrija. Maximo 2 iteraciones de review-fix.
 
-### Fase 6 — QA
-
-**Agente:** Lee `.claude/agents/qa.md` via Task tool
-**Input:** Criterios de aceptacion del PO + codigo implementado
-**Proceso:**
+### Fase 6 — QA (delega a /qa-cycle)
 
-1. QA ejecuta los tests y valida criterios de aceptacion
-2. Prueba edge cases y escenarios de error
-3. Si encuentra bugs, reporta con pasos de reproduccion
+Ejecuta el skill `/qa-cycle` pasandole los criterios de aceptacion del PO como contexto. El qa-cycle se encarga de:
 
-**Output:** Reporte QA con resultado por cada criterio de aceptacion
-**Condicion de loop:** Si hay bugs:
+1. Ejecutar tests y lint del proyecto
+2. Validar criterios de aceptacion
+3. Probar edge cases y escenarios de error
+4. Ciclo bugfix si hay problemas (maximo 3 ciclos)
 
-1. Invoca agente Bugfix (lee `.claude/agents/bugfix.md` via Task tool) para corregir
-2. Vuelve a **Fase 5** (Review) para verificar el fix
-3. Maximo 2 ciclos de bugfix-review-QA
+**Condicion de loop adicional:** Si el bugfix del qa-cycle introduce cambios significativos, vuelve a **Fase 5** (Review) para verificar. Maximo 2 ciclos de review-QA.
 
 ## Checkpoint Commits
 
@@ -138,9 +142,19 @@ Also update SESSION.md at each phase transition:
 - [timestamp] | build-feature | Phase N ([phase-name]) complete for [feature]
 ```
 
+## Gate final (obligatorio antes de Finalizacion)
+
+Antes de declarar el pipeline como completado, ejecutar verificacion final:
+
+1. Ejecuta tests del proyecto — si falla, volver a Fase 6 (QA/Bugfix)
+2. Ejecuta lint del proyecto — si falla, volver a Fase 4 (Developer)
+3. Ambos deben pasar con exit code 0
+
+Este gate es la ultima red de seguridad. NO se puede saltar bajo ninguna circunstancia.
+
 ## Finalizacion
 
-Al completar todas las fases exitosamente:
+Al completar todas las fases y el gate final exitosamente:
 
 1. Presenta resumen del pipeline:
    - Feature implementada
@@ -195,6 +209,6 @@ Feature complete. PR ready for merge.
 
 ## Notas
 
-- Si el usuario quiere saltar fases (ej: "ya la evaluo, implementa directo"), permite saltar a Fase 4 pero advierte que se pierde validacion
+- Si el usuario quiere saltar fases (ej: "ya la evaluo, implementa directo"), permite saltar a Fase 4 pero advierte que se pierde validacion. Los gates de verificacion (pre-Review y final) NUNCA se saltan
 - El pipeline es secuencial: cada fase depende del output de la anterior
 - Los loops de review/QA tienen limite para evitar ciclos infinitos

package/src/templates/skills/qa-cycle/SKILL.md

@@ -20,33 +20,43 @@ Ejecuta un ciclo de validacion QA seguido de bugfix hasta que todos los criterio
 
 ## Proceso
 
-### Paso 1 — Validacion QA
+### Paso 1 — Verificacion automatizada (obligatorio)
+
+Antes de invocar al agente QA, ejecutar los comandos de verificacion del proyecto. Los comandos concretos estan en la seccion "Comandos CLI" de CLAUDE.md:
+
+1. Ejecuta tests del proyecto (ej: `npm test`) — registra resultado y output
+2. Ejecuta lint del proyecto (ej: `npm run lint`) — registra resultado y output
+3. Si alguno falla, esto se convierte en input para el reporte QA como bug automatico de tipo Blocker
+
+### Paso 2 — Validacion QA
 
 Invoca el agente QA usando Task tool:
 
 1. Lee `.claude/agents/qa.md` para asumir el rol de QA
 2. Lee CLAUDE.md y SESSION.md para contexto
-3. Revisa los criterios de aceptacion de la tarea en curso (si existen en SESSION.md)
-4. Ejecuta los tests del proyecto
-5. Valida edge cases y escenarios de error
-6. Reporta resultados
+3. Recibe los resultados de tests y lint del Paso 1
+4. Si tests o lint fallaron, incluirlos como bugs Blocker en el reporte
+5. Revisa los criterios de aceptacion de la tarea en curso (si existen en SESSION.md)
+6. Valida edge cases y escenarios de error
+7. Reporta resultados
 
-### Paso 2 — Bugfix (si hay bugs)
+### Paso 3 — Bugfix (si hay bugs)
 
-Si QA reporta bugs, invoca el agente Bugfix usando Task tool:
+Si QA reporta bugs (incluyendo fallos de tests/lint), invoca el agente Bugfix usando Task tool:
 
 1. Lee `.claude/agents/bugfix.md` para asumir el rol de Bugfix
 2. Recibe el reporte de bugs de QA como input
 3. Diagnostica la causa raiz de cada bug
 4. Implementa la correccion minima
 5. Verifica que el fix resuelve el problema
+6. Ejecuta tests y lint para confirmar que no introdujo regresiones
 
-### Paso 3 — Re-validacion
+### Paso 4 — Re-validacion
 
-Vuelve al Paso 1 para re-validar despues del bugfix.
-Maximo 3 ciclos de QA-bugfix para evitar loops infinitos.
+Vuelve al Paso 1 (verificacion automatizada) para re-validar despues del bugfix.
+Maximo 3 ciclos de verificacion-QA-bugfix para evitar loops infinitos.
 
-### Paso 4 — Resultado final
+### Paso 5 — Resultado final
 
 Presenta el resultado:
 

package/src/templates/skills/review/SKILL.md

@@ -20,7 +20,7 @@ Ejecuta un code review independiente sobre los cambios actuales del proyecto. In
 
 ## Proceso
 
-### Paso 1 — Obtener diff
+### Paso 1 — Obtener diff y estado de verificacion
 
 Obtiene los cambios actuales:
 
@@ -28,14 +28,24 @@ Obtiene los cambios actuales:
 2. Si no hay cambios en staging, usa `git diff` (cambios sin stage)
 3. Si no hay ningun cambio, informa que no hay nada que revisar
 
+Ejecuta verificacion automatizada para dar contexto al reviewer:
+
+4. Ejecuta tests del proyecto (ej: `npm test`) — captura resultado
+5. Ejecuta lint del proyecto (ej: `npm run lint`) — captura resultado
+6. Incluye ambos resultados como contexto para el Code Reviewer
+
+Nota: El Code Reviewer no tiene acceso a Bash (solo Read, Glob, Grep), por eso los tests y lint se ejecutan aqui antes de invocar al reviewer.
+
 ### Paso 2 — Invocar Code Reviewer
 
 Invoca al agente Code Reviewer usando Task tool:
 
 1. Lee `.claude/agents/code-reviewer.md` para asumir el rol
 2. Lee CLAUDE.md para entender las convenciones del proyecto
-3. Revisa el diff completo
-4. Clasifica cada hallazgo por severidad:
+3. Recibe el diff completo + resultados de tests y lint del Paso 1
+4. Si tests o lint fallaron, esto es automaticamente un hallazgo Blocker
+5. Revisa el diff completo
+6. Clasifica cada hallazgo por severidad:
    - **Blocker**: Debe corregirse antes de merge
    - **Warning**: Deberia corregirse, introduce deuda tecnica
    - **Suggestion**: Mejora opcional