agent-mp 0.5.26 → 0.5.28

package/dist/core/engine.js

@@ -411,13 +411,15 @@ export class AgentEngine {
                 : '\nNo hay features en .agent/docs/ todavía.\n';
             const READY_SENTINEL = '[LISTO_PARA_LANZAR]';
             const specInstructions = hasFeature ? `
-MODO SPEC: Se cargó la feature "${activeFeatureId}". Los archivos raw están en el contexto.
-Tu objetivo es construir colaborativamente un spec.md con el programador.
+MODO SPEC — Feature cargada: "${activeFeatureId}"
+Los archivos raw YA ESTÁN en el CONTEXTO DEL PROYECTO arriba.
 
-FLUJO:
-1. Revisá los archivos raw en el contexto.
-2. Hacé UNA pregunta a la vez para aclarar dudas o completar info faltante.
-3. Cuando tengas todo claro, generá el spec.md con este formato EXACTO:
+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:
 
 === SPEC.MD ===
 # Feature: ${activeFeatureId}
@@ -438,17 +440,20 @@ FLUJO:
 [consideraciones de implementación]
 === END SPEC.MD ===
 
-4. Cuando el spec esté aprobado por el programador, escribí EXACTAMENTE al final de tu mensaje: ${READY_SENTINEL}` : `
-MODO TAREA LIBRE: El programador no especificó una feature de .agent/docs/.
+6. Cuando el spec esté aprobado, escribí EXACTAMENTE al final: ${READY_SENTINEL}` : `
+MODO TAREA LIBRE — No se especificó feature todavía.
 ${featureListBlock}
 REGLAS:
-- Primero entendé QUÉ quiere hacer el programador. Hacé preguntas hasta tener claro el objetivo.
-- Si hay features disponibles, preguntá si alguna corresponde a esta tarea (mencioná los nombres).
-- Si el programador da un ID de feature, respondé EXACTAMENTE con: CARGAR_FEATURE: <id>
-- SOLO cuando tengas el objetivo completamente claro y el programador esté de acuerdo, escribí EXACTAMENTE al final de tu mensaje: ${READY_SENTINEL}
-- NUNCA escribas ${READY_SENTINEL} si solo recibiste un saludo o no tenés un objetivo concreto.`;
-            const prompt = `Sos el COORDINADOR de un equipo multi-agente de desarrollo.
-Tu trabajo es ENTENDER lo que el programador necesita haciendo PREGUNTAS si es necesario.
+- Primero entendé QUÉ quiere hacer el programador.
+- Si hay features disponibles, preguntá UNA VEZ si alguna corresponde (mencioná los nombres). No insistas.
+- Si el programador menciona un path o ID de feature, está confirmando esa feature. NO preguntés "¿tiene que ver con X?" — ya lo dijo. Respondé EXACTAMENTE con: CARGAR_FEATURE: <id>
+- Si el programador menciona solo el nombre o fragmento de un ID de feature disponible, igual respondé con: CARGAR_FEATURE: <id>
+- Cuando el objetivo esté claro, escribí EXACTAMENTE al final: ${READY_SENTINEL}
+- NUNCA escribas ${READY_SENTINEL} si solo recibiste un saludo o falta información concreta.`;
+            const prompt = `Sos el COORDINADOR de un equipo de desarrollo de software.
+Tu trabajo es entender qué necesita el programador usando la información disponible y haciendo SOLO las preguntas estrictamente necesarias.
+
+REGLA DE ORO: ANTES de escribir cualquier respuesta, LEÉ COMPLETO el CONTEXTO DEL PROYECTO que está arriba. Toda la información que necesitás para no hacer preguntas obvias está ahí.
 
 CONTEXTO DEL PROYECTO:
 ${context}
@@ -456,11 +461,13 @@ ${context}
 CONVERSACION PREVIA:
 ${conversationHistory}
 
-INSTRUCCIONES:
-- Habla de forma NATURAL, como un compañero de equipo.
-- Si necesitas mas info, hace UNA pregunta clara y espera la respuesta.
-- NO uses JSON, habla normalmente.
-- Sé breve y directo.${specInstructions}`;
+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.
+- Hacé como máximo UNA pregunta por turno.
+- NO uses JSON, hablá normalmente.
+- Sé directo y eficiente — respetá el tiempo del programador.${specInstructions}`;
             log.info('Coordinador analizando...');
             const envOverride = {};
             let res;
@@ -1078,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)) {
@@ -1092,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}
@@ -1104,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:
@@ -1117,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.26",
+  "version": "0.5.28",
   "description": "Deterministic multi-agent CLI orchestrator — plan, code, review",
   "type": "module",
   "main": "./dist/index.js",