elsabro 7.3.2 → 7.5.0

package/README.md

@@ -1,18 +1,18 @@
-# ELSABRO v7.3.1
+# ELSABRO v7.5.0
 
-**Tu asistente AI para crear apps increibles** — Orquestacion de agentes con flow engine, checkpointing y ejecucion paralela.
+**Tu asistente AI para crear apps increibles** — Orquestacion de agentes con flow engine, checkpointing, ejecucion paralela y skill auto-discovery.
 
-ELSABRO es un sistema de desarrollo AI-powered para Claude Code que te guia paso a paso en la creacion de aplicaciones. Incluye un flow engine con 44 nodos, 12 agentes especializados, un CLI para control de flujo, y un sistema de checkpoints para recuperacion automatica.
+ELSABRO es un sistema de desarrollo AI-powered para Claude Code que te guia paso a paso en la creacion de aplicaciones. Incluye un flow engine con 47 nodos, 12 agentes especializados, un CLI para control de flujo, un sistema de checkpoints para recuperacion automatica, y un pipeline completo de skill discovery e invocacion.
 
-## Que Hay de Nuevo en v7.3
+## Que Hay de Nuevo en v7.4
 
-- **Execute.md reescrito** — Reducido 89% (1624 → 182 lineas) como thin wrapper del flow engine
-- **Observabilidad** — 4 senales de telemetria JSONL: node_skip_counter, gate_bypass_detector, execution_time_anomaly, diff_size_vs_review_ratio
-- **Limpieza de codigo fantasma** — Eliminados phantom classes, worktrees inexistentes, 5 secciones duplicadas de Agent Teams
-- **Flow Engine Runtime** — Motor de ejecucion basado en grafos con 9 tipos de nodos, checkpointing automatico
-- **PartyEngine** — Discusiones multi-agente con seleccion automatica de participantes
-- **279 tests** — Zero dependencias externas, Node 18+
-- **Design-UI Routing** — Integracion con Stitch AI para generar pantallas desde texto
+- **Skill Invocation Protocol** — Los agentes reciben `availableSkills` en sus inputs y siguen prioridad: skills instalados > recomendados > implementacion manual
+- **Skill Auto-Install Flow** — 2 nodos nuevos descubren top 5 skills relevantes, preguntan al usuario, y batch-instalan en paralelo
+- **Review Gate Enforcement** — Hooks reales en Claude Code bloquean git commit/push si hay codigo sin code review
+- **Auto-Sync Validation** — Hook que valida 6 archivos criticos despues de cada implementacion con modo `--fix`
+- **Agent Teams Cleanup** — Verificacion con 30s timeout antes de TeamDelete, telemetria de anomalias
+- **Context7 Integration** — Consulta silenciosa de documentacion antes de tocar codigo con librerias externas
+- **399 tests** — Zero dependencias externas, Node 18+
 
 ## Para Quien es ELSABRO?
 
@@ -111,13 +111,13 @@ ELSABRO incluye un motor de ejecucion de flujos basado en grafos. El engine es u
 ### Arquitectura
 
 ```
-JSON Flow Definition (44 nodos)
+JSON Flow Definition (47 nodos)
         |
         v
   graph.js — Construye Map + valida referencias
         |
         v
-  runner.js — While-loop: nodo → executor → checkpoint → siguiente
+  runner.js — While-loop: nodo -> executor -> checkpoint -> siguiente
         |
         v
   executors.js — 9 tipos de nodos (entry, exit, condition, router,
@@ -128,9 +128,9 @@ JSON Flow Definition (44 nodos)
               (inyectados en runtime, el engine no sabe de Claude Code)
 ```
 
-- **279 tests**, 75 suites, 0 dependencias externas
+- **399 tests**, 92 suites, 0 dependencias externas
 - **Checkpoint despues de cada nodo** — recuperacion automatica desde cualquier punto
-- **Template resolver** — 7 tipos de expresiones `{{...}}` sin `eval()`
+- **Template resolver** — 7 tipos de expresiones sin code execution
 
 Para mas detalles ver [ARCHITECTURE.md](ARCHITECTURE.md).
 
@@ -155,9 +155,52 @@ node flow-engine/src/cli.js status --flow flows/development-flow.json
 ### Flow Definitions
 
 ELSABRO incluye 2 flow definitions:
-- `flows/development-flow.json` — Flujo completo de desarrollo (44 nodos, ~1080 lineas)
+- `flows/development-flow.json` — Flujo completo de desarrollo (47 nodos, ~1200 lineas)
 - `flows/quick-flow.json` — Flujo rapido para tareas simples (8 nodos)
 
+## Hooks System
+
+ELSABRO incluye hooks automaticos que se integran con Claude Code:
+
+| Hook | Evento | Funcion |
+|------|--------|---------|
+| `review-gate.sh` | PostToolUse (Write/Edit) | Bloquea git commit si hay codigo sin code review |
+| `auto-sync-check.sh` | PostCommand | Valida sincronizacion de 6 archivos criticos |
+| `check-review-skills.sh` | Pre-review | Verifica que skills de review estan instalados |
+| `skill-discovery.sh` | PreFlow | Descubre y recomienda skills relevantes |
+
+### Review Gate
+
+Cada vez que se modifica codigo, el review gate se activa automaticamente:
+```
+Write/Edit -> flag set -> code review -> fix issues -> repeat until clean -> flag clear -> commit allowed
+```
+
+### Auto-Sync Validation
+
+Valida que archivos criticos esten sincronizados:
+```bash
+# Verificar sincronizacion
+bash hooks/auto-sync-check.sh
+
+# Auto-corregir desyncs simples
+bash hooks/auto-sync-check.sh --fix
+```
+
+## Skill Discovery
+
+ELSABRO descubre automaticamente skills relevantes antes de cada ejecucion:
+
+1. **Descubrimiento** — Escanea skills instalados (`~/.claude/skills/`), skills ELSABRO (`./skills/`), y el registry externo
+2. **Ranking** — Ordena por relevancia usando keyword matching con pesos
+3. **Presentacion** — Muestra top 5 al usuario con opcion de instalar
+4. **Inyeccion** — Skills disponibles se pasan como `availableSkills` a todos los agentes
+
+Los agentes siguen el **Skill Invocation Protocol** (seccion 3.1 de execute.md):
+- Prioridad: skills instalados > recomendados > implementacion manual
+- Context7 se consulta silenciosamente para documentacion de librerias externas
+- Skills que fallan no bloquean la ejecucion
+
 ## Agentes Especializados
 
 12 agentes con personalidades distintas, basados en BMAD-METHOD:
@@ -177,6 +220,8 @@ ELSABRO incluye 2 flow definitions:
 | `elsabro-quick-dev` | Barry | Desarrollo rapido con minima ceremonia |
 | `elsabro-yolo-dev` | Flash | Desarrollo ultra-rapido, maximo riesgo |
 
+Cada agente tiene una seccion `<skills_usage>` personalizada que define que tipos de skills debe buscar y como usarlos segun su rol.
+
 ## PartyEngine
 
 Sistema de discusion multi-agente. Selecciona 2-3 agentes relevantes para un tema y ejecuta rondas de debate con sintesis automatica.
@@ -194,6 +239,9 @@ El engine usa un mapa de relevancia (18 categorias de keywords) para elegir los
 | `default` | Balance entre velocidad y seguridad |
 | `yolo` | Maximo velocidad, minima ceremonia |
 | `careful` | Maximo seguridad, verificacion exhaustiva |
+| `teams` | Agent Teams con coordinacion peer-to-peer |
+| `persistent-teams` | Persistent mode + Agent Teams + blocking review |
+| `bmad` | Spec-Driven Development con PRD y Solution Design |
 
 Cambiar perfil:
 ```
@@ -203,10 +251,10 @@ Cambiar perfil:
 ## Como Funciona
 
 ```
-1. Describes tu idea          →  ELSABRO analiza y pregunta
-2. Planifica las fases        →  Crea planes atomicos con TDD
-3. Ejecuta con agentes        →  Parallel execution + checkpointing
-4. Verifica automaticamente   →  3 reviewers + quality gates
+1. Describes tu idea          ->  ELSABRO analiza y pregunta
+2. Planifica las fases        ->  Crea planes atomicos con TDD
+3. Ejecuta con agentes        ->  Parallel execution + checkpointing
+4. Verifica automaticamente   ->  3 reviewers + quality gates
 ```
 
 Persistent mode mantiene ELSABRO activo entre mensajes — escribe naturalmente y ELSABRO rutea al comando correcto.
@@ -227,12 +275,18 @@ Persistent mode mantiene ELSABRO activo entre mensajes — escribe naturalmente
 flow-engine/
   src/                    — 9 modulos del engine (index, graph, template,
                             executors, runner, checkpoint, callbacks, cli, party)
-  tests/                  — 9 archivos de tests (279 tests, 75 suites)
+  tests/                  — 10 archivos de tests (399 tests, 92 suites)
   src/agent-cards.json    — 9 personalidades de agentes
 
 flows/
-  development-flow.json   — Flujo completo (44 nodos)
+  development-flow.json   — Flujo completo (47 nodos)
   quick-flow.json         — Flujo rapido (8 nodos)
+
+hooks/
+  review-gate.sh          — Review gate (set/check/clear/status)
+  auto-sync-check.sh      — Validacion de sincronizacion
+  check-review-skills.sh  — Verificacion de skills de review
+  skill-discovery.sh      — Discovery de skills pre-flow
 ```
 
 ## Actualizacion
@@ -284,7 +338,10 @@ rm -rf .elsabro .planning
 
 | Version | Milestone | Cambio Principal |
 |---------|-----------|-----------------|
-| 7.3.1 | M7 | Documentacion de desinstalacion en README |
+| 7.5.0 | M7+ | Skill gate enforcement, 5-layer enforcement, Agent Teams review, plan verification gate |
+| 7.4.0 | M7+ | Skill invocation protocol, review gate enforcement, auto-sync hook, 47 nodos, 399 tests |
+| 7.3.2 | M7 | Validation errors/warnings separation, case-insensitive hasCriticalIssues |
+| 7.3.1 | M7 | PR #1: 23 critical fixes, skill auto-install hooks, deprecated teams bypass |
 | 7.3.0 | M7 | Execute.md reescrito (-89%), observabilidad JSONL, limpieza codigo fantasma |
 | 7.1.0 | M5 + M6 | AT compliance, design-ui routing, 279 failure mode tests |
 | 7.0.0 | M4 | Feature completeness — 42/42 nodos, CLI, 257 tests |

package/agents/elsabro-executor.md

@@ -94,6 +94,38 @@ CHECKPOINT (preguntar):
 - payments/stripe → skills/payments-setup.md
 </skill_awareness>
 
+<skills_usage>
+## Uso de Skills en Implementacion
+
+### Deteccion Automatica de Skills
+Cuando `availableSkills` esta presente en los inputs del contexto:
+1. **Escanear tech stack** — Identificar frameworks, lenguajes y librerias del plan
+2. **Matchear con skills disponibles** — Buscar skills que cubran el stack detectado
+3. **Priorizar skill especializado > implementacion manual** — Si existe un skill verificado, usarlo
+
+### Skills Prioritarios para Executor
+| Tech Stack | Skill Recomendado | Uso |
+|------------|-------------------|-----|
+| React Native | react-native-best-practices | Patrones de componentes, navegacion, performance |
+| Supabase/Postgres | supabase-postgres-best-practices | Queries, RLS, migrations, edge functions |
+| TypeScript | typescript-pro | Tipos avanzados, generics, utility types |
+| JavaScript | javascript-pro | Patrones modernos, async patterns |
+| APIs REST | api-designer | Endpoints, validacion, error handling |
+
+### Context7 Obligatorio
+**ANTES de tocar codigo de librerias externas**, consultar Context7 silenciosamente:
+- Buscar documentacion actualizada del paquete
+- Verificar API signatures y breaking changes
+- No mostrar la consulta al usuario, solo aplicar el conocimiento
+
+### Reportar Skills Usados
+En cada output de tarea completada, incluir:
+```
+Skills aplicados: [nombre-skill-1], [nombre-skill-2]
+Patrones usados: [patron-del-skill aplicado]
+```
+</skills_usage>
+
 <execution_flow>
 ## Flujo de Ejecución
 

package/agents/elsabro-orchestrator.md

@@ -38,6 +38,45 @@ Soy **Quantum**, el orquestador de ejecución paralela. Como un director de orqu
 5. **Reportar en tiempo real** - El usuario sabe qué está pasando
 </principles>
 
+<skills_usage>
+## Uso de Skills en Orquestacion
+
+### Deteccion y Distribucion de Skills
+Cuando `availableSkills` esta presente en los inputs del contexto:
+1. **Inventariar skills disponibles** — Listar todos los skills relevantes para la tarea
+2. **Matchear skills con teammates** — Asignar skills especializados a cada agente
+3. **Distribuir en el prompt de spawn** — Incluir skills relevantes al lanzar cada teammate
+
+### Skills Prioritarios para Orchestrator
+| Area | Skill Recomendado | Uso |
+|------|-------------------|-----|
+| Debugging paralelo | parallel-debugging | Coordinar multiples debuggers simultaneos |
+| Task management | task-coordination-strategies | Dependencias, prioridades, conflict resolution |
+| Team comms | team-communication-protocols | Patrones de comunicacion entre teammates |
+| Architecture | architecture-designer | Decisiones de splitting y paralelizacion |
+
+### Distribucion de Skills a Teammates
+Al crear un team o lanzar agentes, recomendar skills por rol:
+- **executor teammates** → react-native-best-practices, supabase-postgres, typescript-pro
+- **qa teammates** → playwright-expert, test-master, debugging-wizard
+- **reviewer teammates** → code-reviewer, security-reviewer, secure-code-guardian
+- **analyst teammates** → architecture-designer, database-optimizer
+
+### Tracking de Skills por Teammate
+Durante la ejecucion paralela, registrar:
+- Que skills uso cada teammate y en que tareas
+- Si algun skill causo problemas o errores
+- Efectividad del skill vs implementacion manual
+
+### Post-Mortem de Skills
+Al generar el reporte final de ejecucion paralela, incluir:
+```
+Skills distribuidos: [total]
+Skills efectivos: [lista con resultado positivo]
+Skills problematicos: [lista con issues encontrados]
+```
+</skills_usage>
+
 <workflows>
 ## Workflows Disponibles
 

package/agents/elsabro-qa.md

@@ -34,6 +34,43 @@ Soy **Murat**, Master Test Architect. Me especializo en API testing, backend ser
 7. **API tests are first-class citizens** - No solo UI
 </principles>
 
+<skills_usage>
+## Uso de Skills en Testing y QA
+
+### Deteccion Automatica de Skills
+Cuando `availableSkills` esta presente en los inputs del contexto:
+1. **Identificar framework de testing** — Jest, Vitest, Pytest, Playwright, etc.
+2. **Buscar skills de testing** — Matchear con skills disponibles para ese framework
+3. **Priorizar patrones del skill > patrones genericos** — Skills contienen best practices verificadas
+
+### Skills Prioritarios para QA
+| Area | Skill Recomendado | Uso |
+|------|-------------------|-----|
+| E2E Browser | playwright-expert | Selectores, page objects, fixtures, paralelismo |
+| Test Design | test-master | Estrategias de coverage, test doubles, TDD patterns |
+| Debugging Tests | debugging-wizard | Diagnostico de flaky tests, race conditions |
+| Security Testing | secure-code-guardian | OWASP testing, injection tests, auth bypass checks |
+
+### Consultar Skills para Patrones Especificos
+Antes de escribir tests para un framework o libreria:
+- Verificar si el skill tiene patrones de test recomendados
+- Usar fixtures y helpers sugeridos por el skill
+- Aplicar la estructura de archivos de test del skill
+
+### Integracion con CI/CD Skills
+Cuando se configura pipeline de CI:
+- Consultar devops-engineer para configuracion de stages
+- Aplicar quality gates recomendados por el skill
+- Configurar test reporters y coverage thresholds segun best practices
+
+### Reportar Skills Consultados
+En cada test plan o suite generada, documentar:
+```
+Skills consultados: [skill-1], [skill-2]
+Patrones de testing aplicados: [patron-especifico]
+```
+</skills_usage>
+
 <workflows>
 ## Workflows Disponibles
 

package/agents/elsabro-verifier.md

@@ -29,6 +29,43 @@ El executor puede decir "completado", pero debes verificar:
 4. ¿Hace lo que se pidió?
 </critical_rule>
 
+<skills_usage>
+## Uso de Skills en Verificacion y Review
+
+### Deteccion Automatica de Skills
+Cuando `availableSkills` esta presente en los inputs del contexto:
+1. **Identificar areas de review** — Calidad, seguridad, performance, tipos
+2. **Matchear con skills de review** — Buscar skills especializados para cada area
+3. **Priorizar skill de review especializado > checklist manual** — Skills profundizan mas
+
+### Skills Prioritarios para Verifier
+| Area de Review | Skill Recomendado | Uso |
+|----------------|-------------------|-----|
+| Code Quality | code-reviewer | Naming, DRY, SOLID, complejidad ciclomatica |
+| Security | security-reviewer | OWASP top 10, secrets exposure, auth flaws |
+| Security Deep | secure-code-guardian | Injection, XSS, CSRF, dependency vulnerabilities |
+| Architecture | architecture-designer | Patrones, separation of concerns, coupling |
+
+### Cross-Reference con Security Skills
+Para verificacion de seguridad OWASP:
+- Usar secure-code-guardian para analisis de vulnerabilidades
+- Verificar contra checklist OWASP del skill
+- Reportar findings con severidad clasificada por el skill
+
+### Verificacion Profunda con Skills
+En Etapa 2 (Code Quality), si hay skills disponibles:
+- Ejecutar analisis del skill ADEMAS del checklist standard
+- El skill puede detectar anti-patterns especificos del framework
+- Combinar findings del skill con findings manuales en reporte unificado
+
+### Reportar Skills Aplicados
+En cada VERIFICATION.md generado, incluir:
+```
+Skills de review usados: [skill-1], [skill-2]
+Findings adicionales por skills: [cantidad]
+```
+</skills_usage>
+
 <two_stage_review>
 ## Revisión de 2 Etapas
 

package/bin/install.js

@@ -422,6 +422,77 @@ async function install() {
     }
   }
 
+  // Install review-gate hooks in settings.json
+  // These hooks enforce code review before git commit/push
+  if (!settings.hooks) {
+    settings.hooks = {};
+  }
+
+  // PostToolUse: Set review-pending flag when code is modified
+  const reviewGateSetHook = {
+    type: 'command',
+    command: 'input=$(cat); file=$(echo "$input" | jq -r \'.tool_input.file_path // "unknown"\' 2>/dev/null); if [ -f "$CLAUDE_PROJECT_DIR/hooks/review-gate.sh" ]; then bash "$CLAUDE_PROJECT_DIR/hooks/review-gate.sh" set "$file" 2>/dev/null; fi; exit 0'
+  };
+
+  // PreToolUse: Block git commit/push if review is pending
+  const reviewGateCheckHook = {
+    type: 'command',
+    command: 'input=$(cat); cmd=$(echo "$input" | jq -r \'.tool_input.command // ""\' 2>/dev/null); if echo "$cmd" | grep -qE \'git commit|git push\'; then if [ -f "$CLAUDE_PROJECT_DIR/hooks/review-gate.sh" ]; then bash "$CLAUDE_PROJECT_DIR/hooks/review-gate.sh" check; exit $?; fi; fi; exit 0'
+  };
+
+  // Merge PostToolUse hooks: remove any stale review-gate entries, then add correct one
+  if (!settings.hooks.PostToolUse || !Array.isArray(settings.hooks.PostToolUse)) {
+    settings.hooks.PostToolUse = [];
+  }
+  // Remove existing review-gate entries (any matcher) to avoid duplicates
+  const prevPostCount = settings.hooks.PostToolUse.length;
+  settings.hooks.PostToolUse = settings.hooks.PostToolUse.filter(entry =>
+    !(entry.hooks && entry.hooks.some(h => h.command && h.command.includes('review-gate.sh')))
+  );
+  // Add canonical review-gate entry with correct matcher
+  settings.hooks.PostToolUse.push({
+    matcher: 'Write|Edit',
+    hooks: [reviewGateSetHook]
+  });
+  const verb1 = prevPostCount > settings.hooks.PostToolUse.length ? 'actualizado' : 'instalado';
+  console.log(`  ${green}✓${reset} Review gate (PostToolUse) ${verb1}`);
+
+  // Merge PreToolUse hooks: remove any stale review-gate entries, then add correct one
+  if (!settings.hooks.PreToolUse || !Array.isArray(settings.hooks.PreToolUse)) {
+    settings.hooks.PreToolUse = [];
+  }
+  // Remove existing review-gate entries (any matcher) to avoid duplicates
+  const prevPreCount = settings.hooks.PreToolUse.length;
+  settings.hooks.PreToolUse = settings.hooks.PreToolUse.filter(entry =>
+    !(entry.hooks && entry.hooks.some(h => h.command && h.command.includes('review-gate.sh')))
+  );
+  // Add canonical review-gate entry with correct matcher and exit $? fix
+  settings.hooks.PreToolUse.push({
+    matcher: 'Bash',
+    hooks: [reviewGateCheckHook]
+  });
+  const verb2 = prevPreCount > settings.hooks.PreToolUse.length ? 'actualizado' : 'instalado';
+  console.log(`  ${green}✓${reset} Review gate (PreToolUse) ${verb2}`);
+
+  // Install skill-gate hooks in settings.json
+  // These hooks enforce skill discovery before code changes (only when ELSABRO mode is active)
+  const skillGateCheckHook = {
+    type: 'command',
+    command: 'if [ -f "$CLAUDE_PROJECT_DIR/hooks/skill-gate.sh" ]; then bash "$CLAUDE_PROJECT_DIR/hooks/skill-gate.sh" check; exit $?; fi; exit 0'
+  };
+
+  // Merge PreToolUse hooks for skill-gate: remove stale entries, then add correct one
+  const prevSkillGateCount = settings.hooks.PreToolUse.length;
+  settings.hooks.PreToolUse = settings.hooks.PreToolUse.filter(entry =>
+    !(entry.hooks && entry.hooks.some(h => h.command && h.command.includes('skill-gate.sh')))
+  );
+  settings.hooks.PreToolUse.push({
+    matcher: 'Write|Edit',
+    hooks: [skillGateCheckHook]
+  });
+  const verb3 = prevSkillGateCount > settings.hooks.PreToolUse.length ? 'actualizado' : 'instalado';
+  console.log(`  ${green}✓${reset} Skill gate (PreToolUse) ${verb3}`);
+
   // Add version tracking
   settings.elsabro = {
     version: pkg.version,

package/commands/elsabro/debug.md

@@ -10,6 +10,9 @@ allowed-tools:
   - Grep
   - WebSearch
   - Task
+  - TeamCreate
+  - TeamDelete
+  - SendMessage
   - AskUserQuestion
   - mcp__plugin_context7_context7__*
 argument-hint: "[descripción del problema]"
@@ -133,25 +136,51 @@ El debugger actualizará el archivo de sesión con:
 - Fix implementado
 - Verificación realizada
 
-## Paso 5: Code Review (OBLIGATORIO)
+## Paso 5: Code Review (OBLIGATORIO - Agent Teams LITE)
 
-**REGLA:** Después de implementar cualquier fix, SIEMPRE ejecutar code review antes de reportar al usuario.
-
-```
-Task(pr-review-toolkit:code-reviewer):
-  Revisa los cambios del fix aplicado.
-  Busca bugs, code smells, seguridad.
-  Archivos modificados: [lista de archivos cambiados]
-```
-
-Si el review encuentra issues, corregir ANTES de reportar al usuario.
+**REGLA:** Después de implementar cualquier fix, SIEMPRE ejecutar code review con Agent Teams antes de reportar al usuario.
 
 ```javascript
+// Crear team de review LITE (3 agentes esenciales)
+TeamCreate({ team_name: "elsabro-review-lite", description: "Quick review - 3 agentes especializados" })
+
+// Lanzar 3 teammates en paralelo (UN SOLO MENSAJE)
+Task({
+  subagent_type: "pr-review-toolkit:code-reviewer",
+  team_name: "elsabro-review-lite",
+  name: "quality-reviewer",
+  model: "sonnet",
+  prompt: "Review code quality of the fix: bugs, code smells, naming, patterns. Focus on modified files. DO NOT ask questions."
+})
+Task({
+  subagent_type: "pr-review-toolkit:silent-failure-hunter",
+  team_name: "elsabro-review-lite",
+  name: "failure-hunter",
+  model: "sonnet",
+  prompt: "Review fix for silent failures, error handling gaps, dangerous fallbacks. Focus on modified files. DO NOT ask questions."
+})
+Task({
+  subagent_type: "pr-review-toolkit:pr-test-analyzer",
+  team_name: "elsabro-review-lite",
+  name: "test-analyzer",
+  model: "sonnet",
+  prompt: "Analyze test coverage for the fix: edge cases, regression tests needed. Focus on modified files. DO NOT ask questions."
+})
+
+// Consolidar hallazgos. Si issues criticos > 0: fix y re-review
+// Shutdown team
+for (const name of ["quality-reviewer", "failure-hunter", "test-analyzer"]) {
+  SendMessage({ type: "shutdown_request", recipient: name, content: "Review complete" })
+}
+TeamDelete()
+
 // MARCAR: code review pasó (para code_review_gate en siguiente_paso)
 state.current_flow.code_review_passed = true;
 Write(".elsabro/state.json", JSON.stringify(state, null, 2));
 ```
 
+Si alguno de los 3 reviewers encuentra issues criticos, corregir y re-review ANTES de reportar al usuario.
+
 ## Paso 6: Verificar con Usuario
 
 Una vez el debugger reporta fix:
@@ -177,18 +206,27 @@ Para verificar:
 ```
 ¿Se implementó un fix (modo find_and_fix)?
 │
-├─ SÍ → ¿Se ejecutó code review (Paso 5)?
+├─ SÍ → ¿Se ejecutó code review con Agent Teams LITE (Paso 5)?
 │    │
 │    ├─ NO → EJECUTAR AHORA:
-│    │   Task(pr-review-toolkit:code-reviewer)
-│    │   Si issues > 0: fix y re-review
-│    │   Solo cuando issues == 0: continuar
+│    │   TeamCreate("elsabro-review-lite")
+│    │   + 3 teammates: quality-reviewer, failure-hunter, test-analyzer
+│    │   Consolidar hallazgos
+│    │   Si issues criticos > 0: fix y re-review
+│    │   Solo cuando issues == 0: shutdown + TeamDelete + continuar
 │    │
-│    └─ SÍ, issues == 0 → Continuar
+│    └─ SÍ, issues == 0, team deleted → Continuar
 │
 └─ NO (solo find_root_cause_only) → Continuar (no aplica)
 ```
 
+**Agent Teams LITE — 3 Reviewers:**
+| Teammate | Plugin | Foco |
+|----------|--------|------|
+| quality-reviewer | pr-review-toolkit:code-reviewer | bugs, patterns, naming |
+| failure-hunter | pr-review-toolkit:silent-failure-hunter | error handling, fallbacks |
+| test-analyzer | pr-review-toolkit:pr-test-analyzer | coverage, edge cases |
+
 **VIOLACIÓN CRÍTICA**: Reportar fix sin code review = ABORTAR OPERACIÓN
 </code_review_gate>