agent-mp 0.5.27 → 0.5.29

package/dist/core/engine.js

@@ -415,11 +415,15 @@ MODO SPEC — Feature cargada: "${activeFeatureId}"
 Los archivos raw YA ESTÁN en el CONTEXTO DEL PROYECTO arriba.
 
 REGLAS CRÍTICAS:
-1. ANTES DE ESCRIBIR CUALQUIER COSA: leé completo cada archivo raw del contexto.
-2. NO preguntes sobre información que ya está en los archivos raw. Si el lenguaje, stack, objetivo, contexto o requisitos están en los archivos, USÁ ESA INFORMACIÓN directamente.
-3. Solo hacé preguntas sobre información que GENUINAMENTE falta y que no podés inferir de los archivos.
-4. Si los archivos ya tienen suficiente información para entender el objetivo, NO hagas preguntas — generá el spec.md directamente.
-5. Cuando tengas todo claro (o si los archivos ya tienen suficiente info), generá el spec.md con este formato EXACTO:
+1. ANTES DE ESCRIBIR CUALQUIER COSA: leé completo el CONTEXTO DEL PROYECTO, incluyendo la sección "TAREA EXISTENTE" si está presente.
+2. SI EXISTE "TAREA EXISTENTE" en el contexto:
+   - Si hay SPEC.MD EXISTENTE: NO lo regeneres. Usalo como referencia.
+   - Si hay RESULTADO PREVIO: FAIL: informá al usuario qué falló y preguntá si quiere corregir eso específico.
+   - Si ya hay plan.json: NO generes otro spec — la tarea ya está definida.
+3. NO preguntes sobre información que ya está en los archivos raw o en TAREA EXISTENTE.
+4. Solo hacé preguntas sobre información que GENUINAMENTE falta y que no podés inferir del contexto.
+5. Si los archivos ya tienen suficiente información para entender el objetivo, NO hagas preguntas — usá el spec existente.
+6. Cuando tengas todo claro (o si ya hay spec), generá el spec.md SOLO si NO existe uno previo:
 
 === SPEC.MD ===
 # Feature: ${activeFeatureId}
@@ -440,7 +444,7 @@ REGLAS CRÍTICAS:
 [consideraciones de implementación]
 === END SPEC.MD ===
 
-6. Cuando el spec esté aprobado, escribí EXACTAMENTE al final: ${READY_SENTINEL}` : `
+7. Cuando el spec esté aprobado (o si ya existe), escribí EXACTAMENTE al final: ${READY_SENTINEL}` : `
 MODO TAREA LIBRE — No se especificó feature todavía.
 ${featureListBlock}
 REGLAS:
@@ -463,8 +467,10 @@ ${conversationHistory}
 
 INSTRUCCIONES GENERALES:
 - Hablá en forma NATURAL, como un compañero de equipo experimentado.
-- ANTES DE RESPONDER: leé todo el CONTEXTO DEL PROYECTO y los archivos raw de la feature. Si algo ya está documentado ahí, NO lo preguntes.
-- NO preguntés cosas que ya están respondidas en los archivos raw o en la conversación.
+- ANTES DE RESPONDER: leé todo el CONTEXTO DEL PROYECTO, incluyendo "TAREA EXISTENTE" si está presente.
+- Si hay una TAREA EXISTENTE con RESULTADO PREVIO: FAIL, informá al usuario qué falló y preguntá si quiere corregir eso específico.
+- Si ya hay SPEC.MD o plan.json para la tarea, NO los regeneres — usá los existentes.
+- NO preguntés cosas que ya están respondidas en los archivos raw, TAREA EXISTENTE o la conversación.
 - Hacé como máximo UNA pregunta por turno.
 - NO uses JSON, hablá normalmente.
 - Sé directo y eficiente — respetá el tiempo del programador.${specInstructions}`;
@@ -981,6 +987,41 @@ REGLA: Al terminar, reporta todo lo que encontraste de forma clara y estructurad
                     ctx += `--- END FEATURE ---\n`;
                 }
             }
+            // VERIFICAR SI EXISTE TAREA PARA ESTA FEATURE — incluir estado y resultado previo
+            const taskDir = path.join(this.projectDir, '.agent', 'tasks', featureId);
+            if (await fileExists(taskDir)) {
+                ctx += `\n\n--- TAREA EXISTENTE: ${featureId} ---\n`;
+                // Leer plan.json si existe
+                const planPath = path.join(taskDir, 'plan.json');
+                if (await fileExists(planPath)) {
+                    const plan = await readJson(planPath);
+                    ctx += `\nPLAN EXISTENTE:\n- Descripción: ${plan.description || 'N/A'}\n- Steps: ${plan.steps?.length || 0}\n`;
+                }
+                // Leer progress.json si existe
+                const progressPath = path.join(taskDir, 'progress.json');
+                if (await fileExists(progressPath)) {
+                    const progress = await readJson(progressPath);
+                    ctx += `- Estado: ${progress.status || 'unknown'}\n`;
+                }
+                // Leer result.md si existe — ESTO ES CRÍTICO PARA RE-INTENTOS
+                const resultPath = path.join(taskDir, 'result.md');
+                if (await fileExists(resultPath)) {
+                    const result = await readFile(resultPath);
+                    const verdict = result.includes('PASS') || result.includes('✅') ? 'PASS' : 'FAIL';
+                    ctx += `- RESULTADO PREVIO: ${verdict}\n`;
+                    if (verdict === 'FAIL') {
+                        ctx += `\n⚠️ ESTA TAREA YA FALLÓ ANTES. Razones del FAIL:\n${result.slice(0, 1500)}\n`;
+                        ctx += `\nIMPORTANTE: Si el usuario pide re-implementar, debés corregir ESPECÍFICAMENTE los puntos marcados como FAIL.\n`;
+                    }
+                }
+                // Leer spec.md si existe
+                const specPath = path.join(this.projectDir, '.agent', 'docs', featureId, 'spec.md');
+                if (await fileExists(specPath)) {
+                    const spec = await readFile(specPath);
+                    ctx += `\n--- SPEC.MD EXISTENTE (usar como referencia, NO regenerar) ---\n${spec.slice(0, 2000)}\n`;
+                }
+                ctx += `--- END TAREA ---\n`;
+            }
         }
         return ctx;
     }
@@ -1085,6 +1126,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 +1175,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 +1195,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 +1208,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.29",
   "description": "Deterministic multi-agent CLI orchestrator — plan, code, review",
   "type": "module",
   "main": "./dist/index.js",