agent-mp 0.5.27 → 0.5.28

package/dist/core/engine.js

@@ -1085,6 +1085,41 @@ INSTRUCCIONES:
     async runImplementor(taskId, plan) {
         const taskDir = path.join(this.projectDir, '.agent', 'tasks', taskId);
         log.phase(2, 'Implementacion', this.config.roles.implementor.cli, this.config.roles.implementor.model);
+        // VERIFICAR SI YA EXISTE UN RESULT.MD PREVIO (para detectar re-intentos)
+        const resultPath = path.join(taskDir, 'result.md');
+        if (await fileExists(resultPath)) {
+            const prevResult = await readFile(resultPath);
+            const isFail = prevResult.toUpperCase().includes('FAIL') || prevResult.includes('❌');
+            if (isFail) {
+                log.warn('⚠️  Esta tarea YA fue implementada antes y el REVIEWER marcó FAIL');
+                console.log('');
+                console.log(chalk.yellow('  Resultado previo:'));
+                console.log(chalk.dim('  ' + prevResult.split('\n').slice(0, 15).join('\n  ')));
+                if (prevResult.split('\n').length > 15) {
+                    console.log(chalk.dim('  ... (ver result.md completo para más detalles)'));
+                }
+                console.log('');
+                // Preguntar al usuario qué hacer
+                const action = await ask('  La tarea falló antes. ¿Qué querés hacer?\n    (r=reintentar arreglando el error / v=ver result.md completo / c=cancelar): ', this.rl, this.fi);
+                if (action.toLowerCase() === 'v') {
+                    console.log('');
+                    console.log(chalk.cyan('  === result.md completo ==='));
+                    console.log(chalk.white(prevResult));
+                    console.log(chalk.cyan('  =========================='));
+                    console.log('');
+                    const retry = await ask('  ¿Reintentar igual? (y/n): ', this.rl, this.fi);
+                    if (retry.toLowerCase() !== 'y') {
+                        log.info('Implementación cancelada por el usuario.');
+                        throw new Error('USER_CANCELLED_AFTER_FAIL');
+                    }
+                }
+                else if (action.toLowerCase() === 'c' || action.toLowerCase() === 'n') {
+                    log.info('Implementación cancelada por el usuario.');
+                    throw new Error('USER_CANCELLED_AFTER_FAIL');
+                }
+                // Si es 'r', continúa con la implementación
+            }
+        }
         const structurePath = path.join(this.projectDir, '.agent', 'rules', 'structure.md');
         let structureRules = '';
         if (await fileExists(structurePath)) {
@@ -1099,6 +1134,14 @@ INSTRUCCIONES:
         }
         const allTargetFiles = [...new Set(plan.steps.flatMap((s) => s.files || []))];
         const stepsText = plan.steps.map((s) => `Step ${s.num}: ${s.description}${s.files?.length ? `\n  Archivos: ${s.files.join(', ')}` : ''}`).join('\n');
+        // Si existe result.md previo con FAIL, incluirlo en el prompt para que el implementor sepa qué corregir
+        let failContext = '';
+        if (await fileExists(resultPath)) {
+            const prevResult = await readFile(resultPath);
+            if (prevResult.toUpperCase().includes('FAIL') || prevResult.includes('❌')) {
+                failContext = `\n\n⚠️ INTENTO PREVIO FALLIDO — El reviewer marcó estos errores:\n=== RESULT.MD PREVIO ===\n${prevResult}\n======================\n\nIMPORTANTE: Debés corregir ESPECÍFICAMENTE los puntos marcados como FAIL en el result.md previo. No vuelvas a implementar lo mismo sin corregir los errores señalados.\n`;
+            }
+        }
         // The engine writes files itself — ask the LLM to generate content in parseable blocks.
         // This avoids depending on CLI tool-use capabilities which are unavailable in API mode.
         const prompt = `TAREA: ${plan.description}
@@ -1111,7 +1154,7 @@ ${allTargetFiles.length ? `ARCHIVOS A CREAR/MODIFICAR:\n${allTargetFiles.map(f =
 
 ${context ? `CONTEXTO DEL PROYECTO:\n${context.slice(0, 2000)}\n` : ''}
 ${archContext ? `ARQUITECTURA EXISTENTE:\n${archContext}\n` : ''}
-${structureRules ? `REGLAS DE ESTRUCTURA:\n${structureRules}\n` : ''}
+${structureRules ? `REGLAS DE ESTRUCTURA:\n${structureRules}\n` : ''}${failContext}
 
 INSTRUCCIONES CRITICAS:
 1. Para CADA archivo que debes crear/modificar, usa este formato EXACTO:
@@ -1124,7 +1167,8 @@ contenido completo del archivo aqui
 3. Genera TODOS los archivos necesarios — uno por bloque FILE.
 4. Incluye el contenido completo de cada archivo (no fragmentos).
 5. Si un step modifica un archivo existente, incluye el archivo completo con los cambios.
-6. NO incluyas explicaciones fuera de los bloques FILE.`;
+6. NO incluyas explicaciones fuera de los bloques FILE.
+7. Si hay un INTENTO PREVIO FALLIDO, leé atentamente los errores del reviewer y CORREGÍ ESPECÍFICAMENTE esos puntos.`;
         const res = await this.runWithFallback('implementor', prompt, 'Implementacion');
         const text = extractCliText(res);
         // Parse === FILE: path === ... === END FILE === blocks written by the LLM

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-mp",
-  "version": "0.5.27",
+  "version": "0.5.28",
   "description": "Deterministic multi-agent CLI orchestrator — plan, code, review",
   "type": "module",
   "main": "./dist/index.js",