elsabro 3.8.2 → 4.0.0

package/agents/elsabro-orchestrator.md

@@ -3,6 +3,12 @@ name: elsabro-orchestrator
 description: Orquestador de ejecución paralela. Usa este agente para coordinar múltiples agentes trabajando simultáneamente.
 tools:
   - Task
+  - TaskCreate
+  - TaskUpdate
+  - TaskList
+  - TaskGet
+  - Teammate
+  - SendMessage
   - Read
   - Glob
   - Grep
@@ -243,7 +249,7 @@ const result = await retryPolicy.executeWithRetry(
 ║  Agent: elsabro-verifier                           ║
 ║  Error: Timeout after 30 minutes                   ║
 ║                                                    ║
-║  [r] Retry now  [s] Skip  [a] Abort               ║
+║  [r] Retry now  [d] Debug  [a] Abort               ║
 ╚════════════════════════════════════════════════════╝
 ```
 
@@ -338,6 +344,123 @@ Archivos modificados:
 - `elsabro-qa` - Para testing paralelo
 </integration_with_elsabro>
 
+<agent_teams>
+## Agent Teams Integration (v4.0.0)
+
+### Cuándo Usar Agent Teams
+
+Como orquestador, decido automáticamente cuándo usar Agent Teams vs Task subagents:
+
+```
+┌─────────────────────────────────────────────────────────────┐
+│  DECISIÓN: ¿Agent Teams o Subagents?                        │
+├─────────────────────────────────────────────────────────────┤
+│                                                             │
+│  Si profile == "teams":                                     │
+│    → AGENT TEAMS (siempre)                                  │
+│                                                             │
+│  Si complexity == "high" Y implementation:                  │
+│    → AGENT TEAMS (peer-to-peer coordination)                │
+│                                                             │
+│  Si exploration o verification:                              │
+│    → SUBAGENTS (read-only, no necesita coordinación)        │
+│                                                             │
+│  Si complexity == "low" o profile == "yolo":                │
+│    → SUBAGENTS (velocidad > coordinación)                   │
+│                                                             │
+│  Default:                                                   │
+│    → SUBAGENTS (menor overhead)                             │
+│                                                             │
+└─────────────────────────────────────────────────────────────┘
+```
+
+### Spawn Team Pattern
+
+```javascript
+// 1. Crear equipo
+Teammate({
+  operation: "spawnTeam",
+  team_name: "elsabro-wave-N",
+  description: "Wave N implementation"
+})
+
+// 2. Crear tareas compartidas
+TaskCreate({ subject: "Implement feature A", ... })
+TaskCreate({ subject: "Write tests for A", ... })
+
+// 3. Spawn teammates en paralelo
+Task({
+  subagent_type: "elsabro-executor",
+  team_name: "elsabro-wave-N",
+  name: "executor",
+  prompt: "Implement feature A..."
+}) |
+Task({
+  subagent_type: "elsabro-qa",
+  team_name: "elsabro-wave-N",
+  name: "qa",
+  prompt: "Write tests for feature A..."
+})
+
+// 4. Teammates se comunican entre sí con SendMessage
+// executor → qa: "Feature A implementada, estos son los archivos..."
+// qa → executor: "Test falla en edge case X, necesito fix..."
+
+// 5. Shutdown y cleanup
+SendMessage({ type: "shutdown_request", recipient: "executor" })
+SendMessage({ type: "shutdown_request", recipient: "qa" })
+Teammate({ operation: "cleanup" })
+```
+
+### Blocking Review con Agent Teams
+
+Cuando la implementación usa Agent Teams, el review sigue usando subagents pero con protocolo bloqueante:
+
+```
+Agent Team (implementation)
+    ↓ cleanup team
+Quality Gate (tests, types, lint)
+    ↓
+Subagent Review (3 en paralelo) ←──┐
+    ↓                               │
+¿Issues == 0?                       │
+    ↓                               │
+  NO → fix_review_issues ───────────┘
+  YES → verify_final
+```
+
+**REGLA:** El review loop se repite hasta 5 veces. Si después de 5 iteraciones aún hay issues, se bloquea completamente (interrupt_blocked). NO hay opción de "continuar con errores".
+</agent_teams>
+
+<blocking_review>
+## Protocolo de Review Bloqueante
+
+### Reglas Inquebrantables
+
+1. **NUNCA** avanzar a post-mortem con issues de review pendientes
+2. **NUNCA** ofrecer opción "continuar sin fix" o "aceptar con errores"
+3. **SIEMPRE** re-ejecutar los 3 reviewers después de cada fix
+4. **MÁXIMO** 5 iteraciones de fix → re-review
+5. Si se agotan las 5 iteraciones → `interrupt_blocked` (solo retry, manual, o abort)
+
+### Display de Iteraciones
+
+```
+╔══════════════════════════════════════════════════╗
+║  CODE REVIEW - Iteración 3/5                     ║
+╠══════════════════════════════════════════════════╣
+║  Issues encontrados: 2                           ║
+║  Issues corregidos: 5 (de iteraciones previas)   ║
+║                                                  ║
+║  Pendientes:                                     ║
+║  1. Missing null check in auth.ts:42             ║
+║  2. Unused import in utils.ts:7                  ║
+║                                                  ║
+║  → Corrigiendo y re-ejecutando review...         ║
+╚══════════════════════════════════════════════════╝
+```
+</blocking_review>
+
 <retry_protocol>
 ## Protocolo de Retry con Escape
 
@@ -431,7 +554,6 @@ Intento 3:
 ║  [e] Extender timeout (2x actual)                ║
 ║  [d] Debug: investigar con /elsabro:debug       ║
 ║  [m] Manual: ejecutar pasos uno a uno            ║
-║  [s] Skip: continuar sin esta tarea             ║
 ║  [a] Abort: parar ejecución completa            ║
 ╚══════════════════════════════════════════════════╝
 ```

package/bin/install.js

@@ -247,21 +247,20 @@ if (hasUpdate) {
     }
 
     try {
-      // Use execFileSync with the package manager directly
-      // Use getExecutable for Windows compatibility
-      // IMPORTANT: Use --yes and clear npm cache to force fresh download from registry
+      // Use specific version from registry to avoid cache issues
+      // Don't pass --update to avoid infinite recursion
+      const versionToInstall = latestVersion || 'latest';
+      const packageSpec = `elsabro@${versionToInstall}`;
+
       if (packageManager === 'pnpm') {
-        // pnpm dlx always fetches fresh
-        execFileSync(getExecutable('pnpm'), ['dlx', 'elsabro@latest', '--global'], { stdio: 'inherit' });
+        execFileSync(getExecutable('pnpm'), ['dlx', packageSpec, '--global'], { stdio: 'inherit' });
       } else if (packageManager === 'bun') {
-        // bunx fetches fresh by default
-        execFileSync(getExecutable('bunx'), ['elsabro@latest', '--global'], { stdio: 'inherit' });
+        execFileSync(getExecutable('bunx'), [packageSpec, '--global'], { stdio: 'inherit' });
       } else {
-        // npm exec: more reliable than npx for executing remote packages
-        // Use --yes to skip prompts and ensure fresh download from registry
-        execFileSync(getExecutable('npm'), ['exec', '--yes', '--', 'elsabro@latest', '--global'], { stdio: 'inherit' });
+        // npm exec with specific version bypasses cache
+        execFileSync(getExecutable('npm'), ['exec', '--yes', '--', packageSpec, '--global'], { stdio: 'inherit' });
       }
-      console.log(`\n  ${green}✓${reset} ELSABRO actualizado correctamente\n`);
+      console.log(`\n  ${green}✓${reset} ELSABRO actualizado a v${versionToInstall}\n`);
     } catch (error) {
       console.error(`\n  ${red}✗${reset} Error al actualizar: ${error.message}\n`);
       process.exit(1);

package/commands/elsabro/execute.md

@@ -13,6 +13,8 @@ allowed-tools:
   - TaskUpdate
   - TaskList
   - TaskGet
+  - Teammate
+  - SendMessage
   - AskUserQuestion
   - mcp__plugin_context7_context7__*
 argument-hint: "[número de fase]"
@@ -27,16 +29,24 @@ dispatcher:
     model: haiku
     parallel: true
     min_agents: 3
+    method: "subagent"
   implementation:
     agents: [elsabro-executor, feature-dev:code-architect]
     model: opus
     parallel: true
     min_agents: 2
+    method: "auto"
+    agent_team_config:
+      team_members: [elsabro-executor, elsabro-qa, elsabro-planner]
+      when: "complexity == 'high' || profile == 'teams'"
   verification:
     agents: [pr-review-toolkit:code-reviewer, pr-review-toolkit:silent-failure-hunter, pr-review-toolkit:pr-test-analyzer]
     model: opus
     parallel: true
     min_agents: 3
+    method: "subagent"
+    blocking: true
+    max_review_iterations: 5
 ---
 
 # ELSABRO: Execute
@@ -171,6 +181,156 @@ Ejecutar planes de una fase con:
 - **OPUS para verificación:** Análisis profundo de bugs y edge cases
 </dispatcher_integration>
 
+<agent_teams_integration>
+## Agent Teams Integration (v4.0.0)
+
+### Cuándo Usar Agent Teams vs Subagents
+
+```
+┌─────────────────────────────────────────────────────────────────────────┐
+│                    AUTO-ROUTING DECISION                                 │
+├─────────────────────────────────────────────────────────────────────────┤
+│  CONDICIÓN                          │  MÉTODO                           │
+├─────────────────────────────────────┼───────────────────────────────────┤
+│  profile == "teams"                 │  AGENT TEAMS (siempre)            │
+│  complexity == "high"               │  AGENT TEAMS (auto)               │
+│  complexity == "medium" + >3 waves  │  AGENT TEAMS (auto)               │
+│  complexity == "low"                │  SUBAGENTS (siempre)              │
+│  profile == "yolo"                  │  SUBAGENTS (siempre)              │
+│  Exploración                        │  SUBAGENTS (siempre - read-only)  │
+│  Verificación/Review                │  SUBAGENTS (siempre - read-only)  │
+│  Implementación multi-wave          │  AGENT TEAMS (auto)               │
+└─────────────────────────────────────┴───────────────────────────────────┘
+```
+
+### Flujo con Agent Teams
+
+```javascript
+// 1. Spawn team
+Teammate({
+  operation: "spawnTeam",
+  team_name: "elsabro-impl",
+  description: "Implementation team for: " + task
+})
+
+// 2. Create tasks para el team
+TaskCreate({ subject: "Analyze requirements", ... })
+TaskCreate({ subject: "Implement core logic", ... })
+TaskCreate({ subject: "Write tests", ... })
+
+// 3. Spawn teammates
+Task({
+  subagent_type: "elsabro-executor",
+  team_name: "elsabro-impl",
+  name: "executor-1",
+  prompt: "Implement core logic following TDD..."
+})
+Task({
+  subagent_type: "elsabro-qa",
+  team_name: "elsabro-impl",
+  name: "qa-1",
+  prompt: "Write comprehensive tests..."
+})
+
+// 4. Teammates coordinate via SendMessage
+// 5. When done, cleanup
+Teammate({ operation: "cleanup" })
+```
+
+### Agentes que Migran a Agent Teams
+
+| Agente | Rol en Team | Cuándo |
+|--------|-------------|--------|
+| elsabro-orchestrator | Team Lead | Siempre (coordina) |
+| elsabro-executor | Teammate | Implementación |
+| elsabro-planner | Teammate | Planning |
+| elsabro-analyst | Teammate | Análisis |
+| elsabro-qa | Teammate | Testing |
+| elsabro-verifier | Teammate | Verificación |
+| elsabro-debugger | Teammate | Debug (iter >= 2) |
+
+### Agentes que Permanecen como Subagents
+
+| Agente | Razón |
+|--------|-------|
+| elsabro-quick-dev | Tarea simple, no necesita coordinación |
+| elsabro-yolo-dev | Velocidad > coordinación |
+| elsabro-scrum-master | Interacción con usuario |
+| elsabro-tech-writer | Documentación independiente |
+| elsabro-ux-designer | Diseño independiente |
+</agent_teams_integration>
+
+<blocking_review_protocol>
+## Protocolo de Review Bloqueante (v4.0.0)
+
+**REGLA CRÍTICA:** El proceso NO PUEDE avanzar a post-mortem si hay issues de code review sin resolver.
+
+### Flujo
+
+```
+parallel_review (3 agentes)
+       ↓
+review_check: ¿hay issues?
+       ↓
+  ┌────┴────┐
+  │ NO      │ YES
+  │ issues  │ issues
+  ↓         ↓
+verify    fix_review_issues
+final     (max 5 iteraciones)
+              ↓
+          parallel_review (re-check)
+              ↓
+          ¿issues == 0?
+              ↓
+        ┌─────┴─────┐
+        │ SÍ        │ NO (5 iter agotadas)
+        ↓            ↓
+    verify_final  interrupt_blocked
+                  (SIN opción "continuar con errores")
+                      ↓
+                  ┌───┴───┐───────┐
+                  │       │       │
+                retry  manual  abort
+                  ↓       ↓       ↓
+              fix_review  wait   end_cancelled
+              _issues     _manual
+                          _review_fix
+                              ↓
+                          parallel_review (re-check)
+```
+
+### Escape Hatches ELIMINADOS
+
+Los siguientes escape hatches han sido removidos en v4.0.0:
+
+1. ~~`onMaxIterations: "verify_final"`~~ → Ahora va a `interrupt_blocked` (sin opción de continuar)
+2. ~~`"skip": "Continuar sin fix"`~~ → Removido de `interrupt_manual_fix`
+3. ~~`"accept": "Aceptar con errores"`~~ → Removido de `interrupt_verification_failed`
+
+### Iteraciones de Review
+
+```javascript
+// Estado del review loop
+state.reviewIteration = state.reviewIteration || 0;
+state.reviewIteration++;
+
+// Log de cada iteración
+console.log(`[REVIEW] Iteración ${state.reviewIteration}/5`);
+console.log(`[REVIEW] Issues encontrados: ${issueCount}`);
+
+// Solo avanza cuando issues == 0
+if (issueCount === 0) {
+  state.reviewIteration = 0; // Reset
+  // → verify_final
+} else if (state.reviewIteration >= 5) {
+  // → interrupt_blocked (NO verify_final)
+} else {
+  // → fix_review_issues → parallel_review
+}
+```
+</blocking_review_protocol>
+
 <process>
 ## Paso 0: Exploración Pre-Ejecución (HAIKU x3 paralelo)
 
@@ -744,7 +904,7 @@ if (summary.decision === "STOP") {
 │        ↓                                                            │
 │  ┌─────────────────────────────────────────────────────────────┐   │
 │  │ CRITICAL: Notificar inmediatamente, STOP                    │   │
-│  │ HIGH:     Notificar, ofrecer opciones [fix/skip/abort]      │   │
+│  │ HIGH:     Notificar, ofrecer opciones [fix/debug/abort]      │   │
 │  │ MEDIUM:   Agregar a resumen, continuar                      │   │
 │  │ LOW:      Log para debugging, continuar                     │   │
 │  └─────────────────────────────────────────────────────────────┘   │
@@ -815,7 +975,7 @@ await sessionValidator.save(state);
 | Severity | Emoji | Accion |
 |----------|-------|--------|
 | CRITICAL | Parar | No puede continuar de ninguna forma |
-| HIGH | Preguntar | Ofrecer opciones: fix/skip/abort |
+| HIGH | Preguntar | Ofrecer opciones: fix/debug/abort |
 | MEDIUM | Warning | Mostrar y continuar |
 | LOW | Info | Log y continuar |
 
@@ -827,8 +987,8 @@ await sessionValidator.save(state);
 | Falta validacion critica | HIGH | Auto-add |
 | Falta dependencia | HIGH | Auto-install |
 | Cambio arquitectonico | HIGH | CHECKPOINT |
-| Tests fallan | HIGH | Debug o skip |
-| Lint warnings | MEDIUM | Continuar |
+| Tests fallan | HIGH | Auto-fix (5 intentos) |
+| Lint warnings | HIGH | Auto-fix (incluido en quality gate) |
 
 ### Actualizar Estado con Tasks