@arcadialdev/arcality 2.6.2 → 2.6.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arcadialdev/arcality",
-  "version": "2.6.2",
+  "version": "2.6.4",
   "description": "El primer ingeniero QA Autónomo integrado al CI/CD. Creado por Arcadial.",
   "main": "index.js",
   "scripts": {
@@ -70,4 +70,4 @@
     "terminal-image": "^1.1.0",
     "tsx": "^4.21.0"
   }
-}
+}

package/scripts/gen-and-run.mjs

@@ -27,6 +27,29 @@ import {
 // Load global config at startup (injects keys into process.env)
 setupProcessEnv();
 
+// ── WINDOWS FIX: Interceptar Ctrl+C antes que cmd.exe ──
+// En Windows, `arcality` se ejecuta vía arcality.cmd (creado por npm para el campo bin).
+// cmd.exe intercepta Ctrl+C y muestra "¿Desea terminar el trabajo por lotes (S/N)?".
+// Al presionar S, mata el árbol de procesos antes de que nuestro finally pueda correr.
+//
+// La solución: setRawMode(true) hace que Node capture los bytes del teclado directamente,
+// ANTES de que cmd.exe los procese. Al detectar Ctrl+C (byte 0x03) lo convertimos en
+// un SIGINT de Node — cmd.exe nunca lo ve y nunca muestra el prompt.
+if (process.stdin.isTTY && process.stdin.setRawMode) {
+    try {
+        process.stdin.setRawMode(true);
+        process.stdin.resume();
+        process.stdin.setEncoding('utf8');
+        process.stdin.on('data', (key) => {
+            if (key === '\u0003') { // Ctrl+C
+                process.emit('SIGINT');
+            }
+        });
+    } catch (e) {
+        // Si falla (ej. en CI sin TTY), continuar sin raw mode
+    }
+}
+
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 const PROJECT_ROOT = process.env.ARCALITY_ROOT || path.join(__dirname, "..");
@@ -317,8 +340,13 @@ function run(cmd, args, options = {}) {
 
         // Use standard cross-platform spawn. When shell is false, Node's child_process 
         // properly wraps arguments with spaces in quotes natively for CreateProcessW on Windows.
+        //
+        // CRITICAL (Windows): stdin MUST be 'ignore' (not 'inherit') to prevent cmd.exe from
+        // showing the "¿Desea terminar el trabajo por lotes (S/N)?" prompt on Ctrl+C.
+        // When stdin is 'inherit', cmd.exe intercepts Ctrl+C at the console level and kills
+        // the entire process tree before our SIGINT handler can run, bypassing the finally block.
         const p = spawn(cmd, args, {
-            stdio: isDebug ? ['inherit', 'inherit', 'inherit'] : ['ignore', 'pipe', 'pipe'],
+            stdio: isDebug ? ['ignore', 'inherit', 'inherit'] : ['ignore', 'pipe', 'pipe'],
             shell: false,
             cwd,
             env,
@@ -347,24 +375,26 @@ function run(cmd, args, options = {}) {
             }
         };
 
-        if (!isDebug && p.stdout) {
-            p.stdout.on('data', handleData);
-        }
-        if (!isDebug && p.stderr) {
+        if (p.stdout) p.stdout.on('data', handleData);
+        if (p.stderr) {
             p.stderr.on('data', (data) => {
-                process.stderr.write(data);
+                if (isDebug) process.stderr.write(data);
+                else process.stderr.write(data);
                 outputBuffer += data.toString();
             });
         }
 
-        p.on('exit', (code) => {
+        p.on('exit', (code, signal) => {
             const lastRunLog = path.join(LOGS_DIR, 'last-run.log');
             try {
                 if (!fs.existsSync(LOGS_DIR)) fs.mkdirSync(LOGS_DIR, { recursive: true });
                 fs.writeFileSync(lastRunLog, outputBuffer);
             } catch (e) { }
 
-            if (code === 0) resolve();
+            // code===0 → clean success
+            // code===null + signal → killed by signal (Ctrl+C) — treat as non-fatal so finally runs
+            // code===1 → test failed (Playwright returns 1 on test failure) — also non-fatal
+            if (code === 0 || signal !== null || code === 1) resolve();
             else {
                 if (isDebug) console.error(chalk.red(`\n[DEBUG] Process exited with code ${code}`));
                 reject(new Error(`Process exited with code ${code}`));
@@ -785,6 +815,17 @@ async function main() {
                     try { s.stop(chalk.yellow('⚠️  Misión cancelada.')); } catch { /* ignore */ }
                 }
 
+                // Terminar el proceso hijo de Playwright directamente (Windows-safe)
+                if (activeChildProcess && !activeChildProcess.killed) {
+                    try {
+                        if (process.platform === 'win32') {
+                            spawn('taskkill', ['/pid', String(activeChildProcess.pid), '/f', '/t'], { stdio: 'ignore', windowsHide: true });
+                        } else {
+                            activeChildProcess.kill('SIGTERM');
+                        }
+                    } catch { /* silencioso */ }
+                }
+
                 try {
                     const ctxDir = process.env.CONTEXT_DIR;
                     if (ctxDir && fs.existsSync(ctxDir)) {
@@ -850,14 +891,24 @@ async function main() {
             const testFile = path.join(PROJECT_ROOT, 'tests', '_helpers', 'agentic-runner.spec.ts');
             const configFile = path.join(PROJECT_ROOT, 'playwright.config.js');
 
-            // Resolve playwright CLI — CRITICAL: must use the same @playwright/test instance
-            // that the agentic-runner.bundle.spec.js was compiled against. If the user's project
-            // has its own @playwright/test, we must use THAT one to avoid the singleton conflict
-            // ("Playwright Test did not expect test() to be called here").
+            // Resolve playwright CLI — CRITICAL singleton alignment:
+            //
+            // @playwright/test uses `playwright` internally. When npm detects a version mismatch
+            // between the top-level `playwright` and what `@playwright/test` needs, it installs
+            // a NESTED copy at: @playwright/test/node_modules/playwright/
+            //
+            // Playwright's test runner tracks the active suite via a module-level singleton
+            // (currentSuite). If the CLI uses a DIFFERENT playwright instance than the one
+            // @playwright/test loaded, their singletons are out of sync → "did not expect test()
+            // to be called here".
             //
-            // Resolution order:
-            //   1. ORIGINAL_CWD (user's project) — preferred, avoids singleton mismatch
-            //   2. PROJECT_ROOT (arcality package) — fallback for clean installs
+            // Fix: ALWAYS prefer the playwright nested INSIDE @playwright/test, because that is
+            // guaranteed to be the same instance that @playwright/test uses for its internals.
+            //
+            // Resolution priority:
+            //   1. <project>/@playwright/test/node_modules/playwright/cli.js  ← nested (safe)
+            //   2. <project>/playwright/cli.js                                 ← sibling top-level
+            //   (repeated for ORIGINAL_CWD and PROJECT_ROOT)
             let playwrightCli;
             try {
                 const searchPaths = [ORIGINAL_CWD, PROJECT_ROOT];
@@ -875,14 +926,21 @@ async function main() {
 
                 if (!testRunnerPath) throw resolveError;
 
-                // Navigate from @playwright/test/index.js → playwright/cli.js (sibling package)
-                playwrightCli = path.join(testRunnerPath, '..', '..', '..', 'playwright', 'cli.js');
+                const testRunnerDir = path.dirname(testRunnerPath); // → .../node_modules/@playwright/test/
+
+                // Priority 1: playwright nested INSIDE @playwright/test — same singleton as the package uses
+                const nestedCli = path.join(testRunnerDir, 'node_modules', 'playwright', 'cli.js');
+                // Priority 2: playwright as a top-level sibling (clean installs, versions aligned)
+                const siblingCli = path.join(testRunnerDir, '..', '..', 'playwright', 'cli.js');
 
-                // Some versions nest it differently — try alternate location
-                if (!fs.existsSync(playwrightCli)) {
-                    const alt = path.join(testRunnerPath, '..', '..', 'playwright', 'cli.js');
-                    if (fs.existsSync(alt)) playwrightCli = alt;
-                    else throw new Error(`Playwright CLI not found. Tried:\n  ${playwrightCli}\n  ${alt}`);
+                if (fs.existsSync(nestedCli)) {
+                    playwrightCli = nestedCli;
+                } else if (fs.existsSync(siblingCli)) {
+                    playwrightCli = siblingCli;
+                } else {
+                    throw new Error(
+                        `Playwright CLI not found. Tried:\n  (nested) ${nestedCli}\n  (sibling) ${siblingCli}`
+                    );
                 }
 
                 if (process.argv.includes('--debug')) {
@@ -894,15 +952,15 @@ async function main() {
                 console.error(chalk.yellow('\nTip: Run `npm install @playwright/test` in your project directory.'));
                 _missionActive = false;
                 process.exit = _origProcessExit;
-                process.exit(1);
+                _origProcessExit(1);
             }
 
-
             const postProcessSigint = () => console.log(chalk.yellow('\n⏳ Finalizando tareas de reporte, por favor espera...'));
             let missionResult = null;
             let finalFailReason = null;
             let finalUsageData = undefined;
             let finalReportUrl = null;
+            let finalFailReasoning = null;
 
             try {
                 // No test file path is passed — playwright.config.js uses testMatch:'agentic-runner.spec.ts'
@@ -928,7 +986,11 @@ async function main() {
                 if (userCanceledMission) {
                     missionResult = 'canceled';
                     finalFailReason = 'user_canceled';
-                    return; // El handler ya tomó el control, finally se ejecutará y luego saldrá del CLI
+                    // Detener el spinner explícitamente antes de salir
+                    if (!process.argv.includes('--debug')) {
+                        try { s.stop(chalk.yellow('⚠️  Misión cancelada por el usuario.')); } catch { }
+                    }
+                    return; // El handler ya tomó el control, finally se ejecutará
                 }
 
                 missionResult = 'success';
@@ -1008,6 +1070,7 @@ async function main() {
                 if (userCanceledMission) {
                     missionResult = 'canceled';
                     finalFailReason = 'user_canceled';
+                    finalFailReasoning = 'La misión fue detenida manualmente por el usuario (Ctrl+C) antes de completarse. No se trata de un fallo técnico del sistema ni del portal.';
                     await new Promise(r => setTimeout(r, 800));
                 }
 
@@ -1038,6 +1101,7 @@ async function main() {
                             const lastLog = path.join(ctxDir, logFiles[logFiles.length - 1]);
                             const logData = JSON.parse(fs.readFileSync(lastLog, 'utf8'));
                             finalUsageData = logData.usage || finalUsageData;
+                            finalFailReasoning = logData.fail_reasoning || null;
                             if (missionResult === 'failed' && !finalFailReason) {
                                 finalFailReason = logData.fail_reason || "timeout_or_crash";
                             }
@@ -1045,43 +1109,51 @@ async function main() {
                     }
                 } catch(e) {}
 
-                // ── Upload Evidence ──
-                try {
-                    const { endMission, getEvidenceSasToken } = await import('../src/arcalityClient.mjs');
-                    const { uploadEvidence } = await import('../src/EvidenceUploader.mjs');
-
-                    const sasData = await getEvidenceSasToken(mission.mission_id, finalProjectId);
-                    if (process.argv.includes('--debug')) console.log(`[DEBUG] sasData received:`, sasData);
-                    
-                    // Soporte para camelCase y snake_case debido al serializador JSON del backend
-                    const actualSasUrl = sasData?.sasTokenUrl || sasData?.sasUrl || sasData?.sas_url;
-                    const actualBasePath = sasData?.basePath || sasData?.base_path;
-
-                    if (sasData && actualSasUrl && actualBasePath) {
-                        if (!process.argv.includes('--debug')) {
-                            sRep.start('☁️  Uploading evidence to Azure...');
+                // ── Upload Evidence (solo si NO fue cancelado por usuario) ──
+                if (missionResult !== 'canceled') {
+                    try {
+                        const { getEvidenceSasToken } = await import('../src/arcalityClient.mjs');
+                        const { uploadEvidence } = await import('../src/EvidenceUploader.mjs');
+
+                        const sasData = await getEvidenceSasToken(mission.mission_id, finalProjectId);
+                        if (process.argv.includes('--debug')) console.log(`[DEBUG] sasData received:`, sasData);
+
+                        const actualSasUrl = sasData?.sasTokenUrl || sasData?.sasUrl || sasData?.sas_url;
+                        const actualBasePath = sasData?.basePath || sasData?.base_path;
+
+                        if (sasData && actualSasUrl && actualBasePath) {
+                            if (!process.argv.includes('--debug')) {
+                                sRep.start('☁️  Uploading evidence to Azure...');
+                            }
+                            process.removeAllListeners('SIGINT');
+                            process.removeAllListeners('SIGTERM');
+                            process.on('SIGINT', postProcessSigint);
+                            process.on('SIGTERM', postProcessSigint);
+
+                            await uploadEvidence(reportDir, actualSasUrl, actualBasePath);
+
+                            try {
+                                const parsedSas = new URL(actualSasUrl);
+                                finalReportUrl = `${parsedSas.origin}${parsedSas.pathname}/${actualBasePath}/index.html`;
+                            } catch(e) {}
+
+                            if (!process.argv.includes('--debug')) sRep.stop(chalk.green('✅ Evidence uploaded successfully.'));
                         }
-                        process.removeAllListeners('SIGINT');
-                        process.removeAllListeners('SIGTERM');
-                        process.on('SIGINT', postProcessSigint);
-                        process.on('SIGTERM', postProcessSigint);
-                        
-                        await uploadEvidence(reportDir, actualSasUrl, actualBasePath);
-                        
-                        // Construct the public URL
-                        try {
-                            const parsedSas = new URL(actualSasUrl);
-                            finalReportUrl = `${parsedSas.origin}${parsedSas.pathname}/${actualBasePath}/index.html`;
-                        } catch(e) {}
-                        
-                        if (!process.argv.includes('--debug')) sRep.stop(chalk.green('✅ Evidence uploaded successfully.'));
+                    } catch (e) {
+                        if (process.argv.includes('--debug')) console.log(chalk.yellow(`⚠️  Could not upload evidence: ${e.message}`));
+                        if (!process.argv.includes('--debug')) sRep.stop(chalk.yellow('⚠️  Evidence upload failed.'));
                     }
+                } else {
+                    console.log(chalk.gray('   >> Skipping evidence upload (mission was canceled).'));
+                }
 
-                    // ── End Mission ──
-                    await endMission(mission.mission_id, missionResult || 'failed', finalUsageData, finalFailReason, finalReportUrl);
+                // ── End Mission — SIEMPRE se llama, independientemente de si la evidencia subió ──
+                try {
+                    const { endMission } = await import('../src/arcalityClient.mjs');
+                    await endMission(mission.mission_id, missionResult || 'failed', finalUsageData, finalFailReason, finalReportUrl, finalFailReasoning);
+                    if (process.argv.includes('--debug')) console.log(chalk.gray(`[DEBUG] endMission called → result: ${missionResult}, failReason: ${finalFailReason}`));
                 } catch (e) {
-                    if (process.argv.includes('--debug')) console.log(chalk.yellow(`⚠️  Could not upload evidence or end mission: ${e.message}`));
-                    if (!process.argv.includes('--debug')) sRep.stop(chalk.yellow('⚠️  Evidence upload or end mission failed.'));
+                    if (process.argv.includes('--debug')) console.log(chalk.yellow(`⚠️  Could not end mission in backend: ${e.message}`));
                 }
 
                 if (globalReportProcess) {
@@ -1114,9 +1186,19 @@ async function main() {
                     process.env.ARCALITY_API_KEY
                 );
 
-                await text({
-                    message: 'Press Enter to return to menu...',
-                    placeholder: 'Enter'
+                // Esperar Enter para regresar al menú
+                // Usamos escucha directa de stdin (raw mode) porque @clack/text
+                // no procesa Enter correctamente cuando setRawMode(true) está activo.
+                console.log(chalk.gray('\n   Press Enter to return to menu...'));
+                await new Promise(resolve => {
+                    const onKey = (key) => {
+                        if (key === '\r' || key === '\n' || key === '\u0003') {
+                            process.stdin.off('data', onKey);
+                            if (key === '\u0003') process.emit('SIGINT'); // Ctrl+C = salir
+                            else resolve();
+                        }
+                    };
+                    process.stdin.on('data', onKey);
                 });
 
                 if (skipMenu) {

package/src/arcalityClient.mjs

@@ -250,7 +250,7 @@ export async function startMission(prompt, targetUrl) {
  * @param {'success'|'failed'|'cancelled'} result
  * @param {object} usage - Optional usage statistics directly from the AI
  */
-export async function endMission(missionId, result, usage = null, failReason = null, reportUrl = null) {
+export async function endMission(missionId, result, usage = null, failReason = null, reportUrl = null, failReasoning = null) {
     const apiBase = getEffectiveApiBase();
     if (!apiBase || !missionId) return { ok: true };
 
@@ -262,7 +262,7 @@ export async function endMission(missionId, result, usage = null, failReason = n
                 'Content-Type': 'application/json',
                 'x-api-key': key
             },
-            body: JSON.stringify({ result, usage, failReason, reportUrl })
+            body: JSON.stringify({ result, usage, failReason, failReasoning, reportUrl })
         });
     } catch { }
     return { ok: true };

package/tests/_helpers/agentic-runner.bundle.spec.js

@@ -2810,6 +2810,7 @@ function captureValidationRule(errorMessage, context) {
   let aiMarkedSuccess = false;
   let hasCriticalError = false;
   let failReason = "";
+  let failReasoning = "";
   let resultsSaved = false;
   let isFinished = false;
   let stepCount = 0;
@@ -2902,6 +2903,7 @@ function captureValidationRule(errorMessage, context) {
         success: finalSuccess,
         error: hasCriticalError,
         fail_reason: failReason,
+        fail_reasoning: failReasoning,
         timestamp: Date.now()
       };
       memories.push(missionData);
@@ -2912,7 +2914,7 @@ function captureValidationRule(errorMessage, context) {
     }
     try {
       const logPath = path2.join(contextDir, `agent-log-${Date.now()}.json`);
-      fs2.writeFileSync(logPath, JSON.stringify({ prompt, history, steps: stepCount, success: finalSuccess, error: hasCriticalError, fail_reason: failReason, usage: totalMissionUsage, timestamp: (/* @__PURE__ */ new Date()).toISOString() }, null, 2));
+      fs2.writeFileSync(logPath, JSON.stringify({ prompt, history, steps: stepCount, success: finalSuccess, error: hasCriticalError, fail_reason: failReason, fail_reasoning: failReasoning, usage: totalMissionUsage, timestamp: (/* @__PURE__ */ new Date()).toISOString() }, null, 2));
       try {
         fs2.appendFileSync(path2.join(contextDir, "arcality-history.log"), JSON.stringify({ ts: (/* @__PURE__ */ new Date()).toISOString(), mission: prompt.substring(0, 150), success: finalSuccess, steps: stepCount, usage: totalMissionUsage }) + "\n");
       } catch (e) {
@@ -3066,8 +3068,8 @@ function captureValidationRule(errorMessage, context) {
       const promptKeywords = normalize(prompt);
       const patternKeywords = normalize(bestMatch.prompt || bestMatch.original_prompt || bestMatch.originalPrompt || bestMatch.name || "");
       const matches = promptKeywords.filter((kw) => patternKeywords.some((pkw) => pkw.includes(kw) || kw.includes(pkw))).length;
-      const matchRatio = matches / Math.max(promptKeywords.length, 1);
-      if (matchRatio >= 0.85) {
+      const isVectorMatch = true;
+      if (isVectorMatch) {
         console.log(`>>ARCALITY_STATUS>> \u{1F9E0} Memoria Colectiva detectada (id: ${bestMatch.id}). Descargando a memoria local...`);
         reusedPatternId = bestMatch.id || "unknown";
         let patternJsonObj = {};
@@ -3117,7 +3119,7 @@ function captureValidationRule(errorMessage, context) {
         } catch (err) {
           console.warn(`Error al sincronizar memoria local: ${err}`);
         }
-      } else if (matchRatio >= 0.5) {
+      } else if (!isVectorMatch) {
         let patternJsonObj = {};
         try {
           patternJsonObj = typeof bestMatch.pattern_json === "string" ? JSON.parse(bestMatch.pattern_json) : bestMatch.pattern_json || bestMatch.patternJson || {};
@@ -3129,7 +3131,7 @@ function captureValidationRule(errorMessage, context) {
         patternContext = `Existe un patr\xF3n similar previamente aprobado que logr\xF3 el \xE9xito. \xDAsalo como referencia para tu estrategia:
 ${readableHistory}`;
       } else {
-        console.log(`>>ARCALITY_STATUS>> \u2139\uFE0F Patr\xF3n de BD descartado por baja similitud (${matchRatio.toFixed(2)}).`);
+        console.log(`>>ARCALITY_STATUS>> \u2139\uFE0F Patr\xF3n de BD descartado por baja similitud.`);
       }
     } else {
       console.log(`>>ARCALITY_STATUS>> \u2139\uFE0F No se encontraron patrones previos (no_match: true).`);
@@ -3248,6 +3250,8 @@ ${readableHistory}`;
       console.log(`>>ARCALITY_STATUS>> \u{1F6D1} Error interno del Portal/Red detectado: "${networkError}"`);
       hasCriticalError = true;
       failReason = "portal_network_error";
+      failReasoning = `El portal web fall\xF3 al intentar comunicarse con su backend (ej. una petici\xF3n XHR o Fetch fall\xF3 o dio timeout). El agente QA no tiene control sobre las ca\xEDdas del servidor.
+Detalles t\xE9cnicos capturados en la red: ${networkError}`;
       isFinished = true;
       history.push(`Turno ${stepCount}: \u{1F6D1} ERROR CR\xCDTICO DE SISTEMA/RED: ${networkError}. El portal fall\xF3 internamente al ejecutar una petici\xF3n XHR/Fetch.`);
       saveMissionResults();
@@ -3258,6 +3262,8 @@ ${readableHistory}`;
       console.log(`>>ARCALITY_STATUS>> \u{1F6D1} Error Fatal de JavaScript detectado: "${jsError}"`);
       hasCriticalError = true;
       failReason = "portal_js_crash";
+      failReasoning = `La aplicaci\xF3n web sufri\xF3 una excepci\xF3n no manejada de JavaScript en el navegador, provocando que la p\xE1gina dejara de responder o funcionara de manera err\xE1tica.
+Detalles de la excepci\xF3n en consola: ${jsError}`;
       isFinished = true;
       history.push(`Turno ${stepCount}: \u{1F6D1} ERROR CR\xCDTICO DE APLICACI\xD3N: ${jsError}. El portal sufri\xF3 una excepci\xF3n no manejada y dej\xF3 de responder.`);
       saveMissionResults();
@@ -3275,6 +3281,7 @@ ${readableHistory}`;
             console.error(`   >> Tipo: portal_internal_error | El agente NO puede corregir errores del servidor.`);
             hasCriticalError = true;
             failReason = "portal_internal_error";
+            failReasoning = `El servidor devolvi\xF3 un error sist\xE9mico irrecuperable que fue mostrado en la interfaz visual: "${sysErrTxt.trim()}". El agente QA no puede reparar bugs l\xF3gicos o de infraestructura del servidor, por lo que la misi\xF3n fue detenida preventivamente.`;
             isFinished = true;
             history.push(`Turno ${stepCount}: \u{1F6D1} ERROR SIST\xC9MICO DEL PORTAL: "${sysErrTxt.trim().substring(0, 200)}". Este es un fallo del servidor, no un error de datos del agente. El portal devolvi\xF3 HTTP 200 pero report\xF3 un error interno en la UI.`);
             saveMissionResults();
@@ -3359,7 +3366,24 @@ ${readableHistory}`;
       const effectivePrompt = patternContext ? `${prompt}
 
 ${patternContext}` : prompt;
-      response = await agent.askIA(effectivePrompt, history, stepCount);
+      try {
+        response = await agent.askIA(effectivePrompt, history, stepCount);
+      } catch (err) {
+        if (err.message?.includes("APPLICATION BUG:")) {
+          const bugDesc = err.message.replace("APPLICATION BUG:", "").trim();
+          console.log(`>>ARCALITY_STATUS>> \u{1F6D1} Application Bug reportado: ${bugDesc}`);
+          aiMarkedSuccess = false;
+          hasCriticalError = true;
+          failReason = "application_bug";
+          failReasoning = bugDesc;
+          isFinished = true;
+          history.push(`Turno ${stepCount}: \u{1F6D1} [APPLICATION BUG] ${bugDesc}`);
+          saveMissionResults();
+          break;
+        } else {
+          throw err;
+        }
+      }
       if (response.usage) {
         const inputs = response.usage.input_tokens || 0;
         const cacheCreates = response.usage.cache_creation_input_tokens || 0;
@@ -3381,13 +3405,17 @@ ${patternContext}` : prompt;
           aiMarkedSuccess = false;
           hasCriticalError = true;
           failReason = "cost_limit_reached";
+          failReasoning = response.thought;
         }
       }
     }
+    if (response && response.thought && response.thought.includes("DISCULPA:")) {
+      failReasoning = response.thought.replace("DISCULPA:", "").trim();
+    }
     console.log(`
 ======================================================`);
     console.log(`\u{1F916} [RAZONAMIENTO DEL AGENTE]`);
-    console.log(`${response.thought}`);
+    console.log(`${response ? response.thought : "N/A"}`);
     console.log(`======================================================
 `);
     if (!response.finish && history.length >= 6) {
@@ -3417,6 +3445,8 @@ ${patternContext}` : prompt;
         aiMarkedSuccess = false;
         hasCriticalError = true;
         failReason = "loop_detected";
+        failReasoning = `El agente QA entr\xF3 en un bucle l\xF3gico intentando ejecutar la misma acci\xF3n repetidamente sin observar cambios significativos en el DOM.
+Acci\xF3n bloqueada: Bucle de tipo ${loopType} sobre los elementos ${Array.from(uniqueTargets).join(", ")}.`;
       } else if ((isClickLoop || isFillLoop) && agentRecovered) {
         console.log(`>>ARCALITY_STATUS>> \u{1F504} Loop candidato detectado pero el agente se recuper\xF3 en este turno. Continuando misi\xF3n.`);
       }
@@ -3441,6 +3471,7 @@ ${patternContext}` : prompt;
         aiMarkedSuccess = false;
         hasCriticalError = true;
         failReason = "portal_rendering_failure";
+        failReasoning = response.thought;
         history.push(`Turno ${stepCount}: \u{1F6D1} [PORTAL_RENDER_FAILURE] La p\xE1gina ${blankUrl} carg\xF3 en blanco (${consecutiveWaits} esperas sin progreso). Los widgets no se renderizaron. Bug del portal detectado por Arcality QA. Evidencia: ${blankEvidencePath}`);
         saveMissionResults();
       }
@@ -3918,13 +3949,14 @@ ${patternContext}` : prompt;
   if (aiMarkedSuccess) {
     hasCriticalError = false;
     await persistCollectiveMemory();
-    if (!usedDirectReuse && stepsData.length > 0) {
+    const finalUsedDirectReuse = reusedPatternId !== null && guideIdx !== 999999;
+    if (!finalUsedDirectReuse && stepsData.length > 0) {
       const normalizedPrompt = prompt.toLowerCase().trim();
       const patternKey = crypto2.createHash("sha256").update(normalizedPrompt).digest("hex");
       if (!hasCriticalError) {
-        const isDuplicate = patternContext !== "" || usedDirectReuse;
+        const isDuplicate = finalUsedDirectReuse;
         if (isDuplicate) {
-          console.log(`>>ARCALITY_STATUS>> \u2139\uFE0F No se guardar\xE1 el patr\xF3n. duplicate_skipped: true (ya existe uno similar o se us\xF3 directamente)`);
+          console.log(`>>ARCALITY_STATUS>> \u2139\uFE0F No se guardar\xE1 el patr\xF3n. duplicate_skipped: true (se us\xF3 directamente con \xE9xito)`);
         }
         const stepsToSave = stepsData.length > 0 ? stepsData : stepsDataBackup;
         const hasCompleteSteps = stepsToSave.length >= stepsDataBackup.length;
@@ -4027,7 +4059,21 @@ Evidence: ${JSON.stringify(v.evidence).substring(0, 400)}`,
     console.log(`>>ARCALITY_STATUS>> \u{1F4CA} Uso total de tokens esta misi\xF3n: IN=${totalMissionUsage.input_tokens} | OUT=${totalMissionUsage.output_tokens} | Cache writes=${totalMissionUsage.cache_creation_input_tokens} | Cache reads=${totalMissionUsage.cache_read_input_tokens}`);
     await persistCollectiveMemory(true).catch(() => {
     });
-    throw new Error(`Misi\xF3n no completada o finalizada con errores. Raz\xF3n: ${failReason}`);
+    let finalErrorMsg = `Misi\xF3n no completada o finalizada con errores.
+
+`;
+    finalErrorMsg += `======================================================
+`;
+    finalErrorMsg += `\u{1F3AF} TIPO DE FALLO: ${failReason}
+`;
+    if (failReasoning) {
+      finalErrorMsg += `\u{1F9E0} AN\xC1LISIS DEL AGENTE DE QA:
+${failReasoning}
+`;
+    }
+    finalErrorMsg += `======================================================
+`;
+    throw new Error(finalErrorMsg);
   } else {
     console.log(`>>ARCALITY_STATUS>> \u{1F4CA} Uso total de tokens esta misi\xF3n: IN=${totalMissionUsage.input_tokens} | OUT=${totalMissionUsage.output_tokens} | Cache writes=${totalMissionUsage.cache_creation_input_tokens} | Cache reads=${totalMissionUsage.cache_read_input_tokens}`);
   }