@arcadialdev/arcality 2.6.1 → 2.6.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arcadialdev/arcality",
-  "version": "2.6.1",
+  "version": "2.6.3",
   "description": "El primer ingeniero QA Autónomo integrado al CI/CD. Creado por Arcadial.",
   "main": "index.js",
   "scripts": {

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}`));
@@ -740,19 +770,62 @@ async function main() {
 
             const finalProjectId = process.env.ARCALITY_PROJECT_ID || selectedProjectId;
 
-            // ── SIGINT Handler: Cancelación manual (Ctrl+C / ESC) ──
-            // Se registra AQUÍ, justo antes del spawn, para que solo aplique durante la ejecución.
-            // Al cancelar: mata el proceso hijo, guarda estado "canceled", llama endMission y genera reporte.
-            let activeChildProcess = null;  // Referencia al proceso hijo de Playwright
+            // ── Cancelación Manual (Ctrl+C) — Estrategia bulletproof ──
+            // El problema: @clack/prompts registra su propio SIGINT handler cada tick del spinner,
+            // que llama a process.exit(0) directamente. removeAllListeners() no es suficiente porque
+            // Clack lo re-registra en cada frame de animación.
+            //
+            // La solución: interceptar process.exit() antes de que Clack lo llame,
+            // y usar prependListener para que NUESTRO handler siempre se ejecute primero.
+
+            let activeChildProcess = null;
             let userCanceledMission = false;
-            const cancelHandler = async () => {
-                if (userCanceledMission) return; // Evitar doble disparo
+            let sigintCount = 0;
+            let _missionActive = true;  // Mientras true, bloqueamos el process.exit de Clack
+
+            // Guardar el process.exit original y reemplazarlo temporalmente
+            const _origProcessExit = process.exit.bind(process);
+            process.exit = (code) => {
+                if (_missionActive) {
+                    // Clack (u otra lib) está intentando matar el proceso — lo bloqueamos.
+                    // Nuestro cancelHandler ya habrá sido invocado vía prependListener.
+                    // El proceso se cerrará naturalmente cuando el child process termine.
+                    return;
+                }
+                _origProcessExit(code);
+            };
+
+            const cancelHandler = () => {
+                sigintCount++;
+                if (sigintCount > 3) {
+                    console.log(chalk.red('\n🛑 Forzando salida inmediata...'));
+                    _missionActive = false;
+                    _origProcessExit(1);
+                    return;
+                }
+
+                if (userCanceledMission) {
+                    console.log(chalk.yellow('\n⏳ Cancelación en progreso, esperando reporte... (Ctrl+C x4 para forzar cierre)'));
+                    return;
+                }
                 userCanceledMission = true;
 
                 console.log(chalk.yellow('\n\n⚠️  [ARCALITY] Misión cancelada por el usuario.'));
-                if (!process.argv.includes('--debug')) s.stop(chalk.yellow('⚠️  Misión cancelada por el usuario.'));
+                if (!process.argv.includes('--debug')) {
+                    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 */ }
+                }
 
-                // 1. Escribir un agent-log de cancelación para que el reporte lo refleje
                 try {
                     const ctxDir = process.env.CONTEXT_DIR;
                     if (ctxDir && fs.existsSync(ctxDir)) {
@@ -770,28 +843,15 @@ async function main() {
                     }
                 } catch { /* silencioso */ }
 
-                // 2. Registraremos la misión como 'canceled' en el bloque finally
-                console.log(chalk.gray('   >> Registrando misión como "cancelada"...'));
-
-                // 3. Matar el proceso hijo de Playwright — hacerlo AL FINAL para dar tiempo
-                //    a que el agente-log y endMission terminen antes del kill.
-                //    NO llamar process.exit() aquí: el finally del try/catch principal
-                //    se encarga de generar el reporte visual y abrirlo en el browser.
-                if (activeChildProcess) {
-                    try {
-                        if (process.platform === 'win32') {
-                            spawn('taskkill', ['/pid', activeChildProcess.pid, '/f', '/t'], { stdio: 'ignore', windowsHide: true });
-                        } else {
-                            activeChildProcess.kill('SIGTERM');
-                        }
-                    } catch { /* silencioso */ }
-                }
-                // El bloque finally del try/catch de run() tomará el control
-                // a partir de aquí (el proceso hijo ya terminó → run() rechaza → catch → finally).
+                console.log(chalk.gray('   >> Esperando que Playwright cierre y genere el reporte...'));
+                // Playwright también recibe el SIGINT nativamente desde la terminal Windows
+                // y cerrará su proceso limpiamente, lo que resolverá la Promise del run() → finally
             };
 
-            process.once('SIGINT', cancelHandler);
-            process.once('SIGTERM', cancelHandler);
+            // prependListener asegura que nuestro handler se ejecuta ANTES que el de Clack
+            process.prependListener('SIGINT', cancelHandler);
+            process.prependListener('SIGTERM', cancelHandler);
+
 
             // Build env for the runner — merge arcality.config values.
             // ARCALITY_API_URL must be explicit — it comes from the library's internal config,
@@ -831,27 +891,76 @@ 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 relative to PROJECT_ROOT exactly as Node handles module resolution.
-            // This prevents the "did not expect test() to be called here" singleton duplication error.
+            // 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".
+            //
+            // 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 {
-                // Find the core @playwright/test that the local project uses
-                const testRunnerPath = require.resolve('@playwright/test', { paths: [PROJECT_ROOT] });
-                // Target the cli.js file strictly inside its parallel playwright dependency
-                playwrightCli = path.join(testRunnerPath, '..', '..', '..', 'playwright', 'cli.js');
-                if (!fs.existsSync(playwrightCli)) {
-                    throw new Error(`CLI not found exactly at: ${playwrightCli}`);
+                const searchPaths = [ORIGINAL_CWD, PROJECT_ROOT];
+                let testRunnerPath = null;
+                let resolveError = null;
+
+                for (const searchPath of searchPaths) {
+                    try {
+                        testRunnerPath = require.resolve('@playwright/test', { paths: [searchPath] });
+                        break;
+                    } catch (e) {
+                        resolveError = e;
+                    }
+                }
+
+                if (!testRunnerPath) throw resolveError;
+
+                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');
+
+                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')) {
+                    console.log(chalk.gray(`[DEBUG] Using playwright CLI: ${playwrightCli}`));
                 }
             } catch (e) {
-                console.error(chalk.red('\n[FATAL] Playwright core not found! Make sure playwright is properly installed.'));
+                console.error(chalk.red('\n[FATAL] Playwright core not found. Make sure @playwright/test is installed.'));
                 console.error(chalk.red(e.message));
-                process.exit(1);
+                console.error(chalk.yellow('\nTip: Run `npm install @playwright/test` in your project directory.'));
+                _missionActive = false;
+                process.exit = _origProcessExit;
+                _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'
@@ -869,14 +978,19 @@ async function main() {
                     onSpawn: (child) => { activeChildProcess = child; }  // ← Capturar referencia para cancelHandler
                 });
 
-                // Eliminar handlers SIGINT ahora que el proceso terminó normalmente
-                process.off('SIGINT', cancelHandler);
-                process.off('SIGTERM', cancelHandler);
+                process.removeAllListeners('SIGINT');
+                process.removeAllListeners('SIGTERM');
+                process.on('SIGINT', postProcessSigint);
+                process.on('SIGTERM', postProcessSigint);
 
                 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';
@@ -932,15 +1046,10 @@ async function main() {
                     }
                 }
             } catch (e) {
-                // Eliminar handlers SIGINT en el path de error también
-                process.off('SIGINT', cancelHandler);
-                process.off('SIGTERM', cancelHandler);
-
+                // SIGINT ya fue manejado por cancelHandler — solo clasificamos el resultado
                 if (userCanceledMission) {
                     missionResult = 'canceled';
                     finalFailReason = 'user_canceled';
-                    // El cancelHandler ya registró la cancelación en el servidor.
-                    // Solo mostramos un mensaje y dejamos que el finally genere el reporte.
                     console.log(chalk.yellow('\n⚠️  Misión cancelada — generando reporte de evidencia...'));
                 } else {
                     missionResult = 'failed';
@@ -952,14 +1061,29 @@ async function main() {
                     }
                 }
             } finally {
+                // Desactivar el bloqueo de process.exit y restaurar el original
+                _missionActive = false;
+                process.exit = _origProcessExit;
+                process.removeListener('SIGINT', cancelHandler);
+                process.removeListener('SIGTERM', cancelHandler);
+
                 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));
                 }
 
+                process.on('SIGINT', postProcessSigint);
+                process.on('SIGTERM', postProcessSigint);
+
                 const sRep = spinner();
-                sRep.start('📊 Generating report...');
+                if (!process.argv.includes('--debug')) {
+                    sRep.start('📊 Generating report...');
+                } else {
+                    console.log('📊 Generating report...');
+                }
+
                 await run("node", [path.join(PROJECT_ROOT, "scripts/rebrand-report.mjs")]).catch(() => { });
 
                 const reportDir = process.env.REPORTS_DIR || 'tests-report';
@@ -977,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";
                             }
@@ -984,37 +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...');
-                        
-                        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.'));
+                // ── 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.'));
+                        }
+                    } 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) {
@@ -1047,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) {
@@ -1060,6 +1209,10 @@ async function main() {
                 agentMode = false;
                 prompt = "";
                 firstRun = false;
+
+                // Limpiar handlers del post-proceso
+                process.removeListener('SIGINT', postProcessSigint);
+                process.removeListener('SIGTERM', postProcessSigint);
             }
         }
         firstRun = false;

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) {
@@ -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();
       }
@@ -4027,7 +4058,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}`);
   }