nothumanallowed 13.5.135 → 13.5.137

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.135",
+  "version": "13.5.137",
   "description": "NotHumanAllowed — 38 AI agents, 80 tools, Studio (visual agentic workflows). Email, calendar, browser automation, screen capture, canvas, cron/heartbeat, Alexandria E2E messaging, GitHub, Notion, Slack, voice chat, free AI (Liara), 28 languages. Zero-dependency CLI.",
   "type": "module",
   "bin": {

package/src/commands/ui.mjs

@@ -5643,9 +5643,20 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
           // Also include: any file whose name appears in the user message, plus error-related files.
           const msgLower = (message || '').toLowerCase();
           const alwaysInclude = ['package.json', 'server/index.js', '.env.example'];
+
+          // For autofix: extract file paths directly from stack trace (e.g. "at Object.<anonymous> (/path/server/routes/auth.js:11:8)")
+          const stackFileMatches = autofix ? [...(message.matchAll(/\(([^)]+\.(?:js|mjs|cjs|ts)):\d+:\d+\)/g))] : [];
+          const stackFiles = new Set(stackFileMatches.map(m => {
+            const abs = m[1];
+            // Make relative to sandboxDir
+            return abs.startsWith(sandboxDir) ? abs.slice(sandboxDir.length + 1) : abs.split('/').slice(-3).join('/');
+          }));
+
           const relevantFiles = allFiles.filter(f => {
             const nameLower = f.name.toLowerCase();
             if (alwaysInclude.some(a => nameLower.endsWith(a))) return true;
+            // Always include files mentioned in stack trace (autofix)
+            if (stackFiles.size > 0 && [...stackFiles].some(sf => f.name.includes(sf) || sf.includes(f.name))) return true;
             // Include if file name or path fragment mentioned in message
             const parts = f.name.split('/');
             if (parts.some(p => msgLower.includes(p.replace(/\.[^.]+$/, '').toLowerCase()))) return true;
@@ -5691,7 +5702,7 @@ Per leggere un file (se hai bisogno di piu contesto):
 
 REGOLE CRITICHE:
 - Spiega SEMPRE in linguaggio naturale cosa stai facendo PRIMA dei blocchi tool
-- Usa "edit" (old/new) quando possibile, "write" solo per file nuovi o riscritture complete
+- ${autofix ? 'MODALITA AUTO-FIX: usa "write" per riscrivere i file problematici nella loro interezza — è più affidabile di "edit" quando il file ha già subito modifiche. Includi il contenuto COMPLETO del file nel campo "content".' : 'Usa "edit" (old/new) quando possibile, "write" solo per file nuovi o riscritture complete'}
 - old_string deve essere ESATTO come appare nel file (copy-paste)
 - Non inventare moduli npm: usa solo quelli in package.json o standard Node.js
 - Dopo ogni fix spiega brevemente cosa hai cambiato e perche
@@ -5845,7 +5856,24 @@ REGOLE CRITICHE:
                   oldSnippet: (toolCall.old || '').slice(0, 300),
                   newSnippet: (toolCall.new || '').slice(0, 300) });
               } else {
-                sendEv({ type: 'tool', op: 'edit', path: toolCall.path, result: 'old_string non trovato — patch fallita' });
+                // edit fallback: old_string not found → do a second LLM call with the full file
+                // and ask it to rewrite the file correctly. No client loop needed.
+                sendEv({ type: 'tool', op: 'edit', path: toolCall.path, result: 'patch chirurgica fallita — riscrittura automatica in corso...' });
+                try {
+                  const fallbackSys = `Sei un esperto di Node.js. Riscrivi il seguente file correggendo questo problema: ${message.slice(0, 500)}
+Rispondi SOLO con il contenuto completo del file corretto, senza markdown fence, senza spiegazioni.`;
+                  const fallbackUser = `FILE: ${toolCall.path}\n\`\`\`\n${content}\n\`\`\``;
+                  const newContent = await callLLM(config, fallbackSys, fallbackUser, { max_tokens: 4096 });
+                  const cleaned = newContent.replace(/^```[\w]*\n?/m, '').replace(/\n?```\s*$/m, '').trim();
+                  if (cleaned && cleaned.length > 20) {
+                    fs.writeFileSync(fp, cleaned, 'utf8');
+                    toolResults.push({ op: 'write', path: toolCall.path, ok: true });
+                    sendEv({ type: 'tool', op: 'write', path: toolCall.path, result: 'ok (fallback rewrite)',
+                      oldSnippet: content.slice(0, 300), newSnippet: cleaned.slice(0, 300) });
+                  }
+                } catch(fallbackErr) {
+                  sendEv({ type: 'tool', op: 'edit', path: toolCall.path, result: 'fallback rewrite fallito: ' + fallbackErr.message.slice(0, 80) });
+                }
               }
             } else if (toolCall.op === 'write') {
               const fp = path.join(sandboxDir, toolCall.path.replace(/^\/+/, ''));

package/src/constants.mjs

@@ -5,7 +5,7 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export const VERSION = '13.5.135';
+export const VERSION = '13.5.137';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/services/web-ui.mjs

@@ -8371,7 +8371,8 @@ async function wcTriggerCrashFix(errorMsg) {
     });
     if (wcChatRunning) return;
   }
-  var fixMsg = 'CRASH FIX RICHIESTO.' + String.fromCharCode(10) + 'ERRORE:' + String.fromCharCode(10) + errorMsg + String.fromCharCode(10) + String.fromCharCode(10) + 'ISTRUZIONI OBBLIGATORIE:' + String.fromCharCode(10) + '1. Leggi i file coinvolti nello stack trace' + String.fromCharCode(10) + '2. Individua la riga esatta del problema' + String.fromCharCode(10) + '3. Usa OBBLIGATORIAMENTE il tool edit_file o write_file per correggere il codice' + String.fromCharCode(10) + '4. NON limitarti a spiegare il problema - DEVI modificare i file' + String.fromCharCode(10) + '5. Dopo aver modificato, il sandbox verrà riavviato automaticamente';
+  var attemptNum = _wcAutoFixAttempts;
+  var fixMsg = 'CRASH FIX RICHIESTO (tentativo ' + attemptNum + '/3).' + String.fromCharCode(10) + 'ERRORE:' + String.fromCharCode(10) + errorMsg + String.fromCharCode(10) + String.fromCharCode(10) + 'REGOLE OBBLIGATORIE:' + String.fromCharCode(10) + '1. Leggi i file coinvolti nello stack trace con view_file' + String.fromCharCode(10) + '2. Individua la riga esatta del problema' + String.fromCharCode(10) + '3. Usa edit_file per patch chirurgiche. SE edit_file fallisce con "old_string non trovato", usa SUBITO write_file per riscrivere il file intero correttamente' + String.fromCharCode(10) + '4. NON spiegare - MODIFICA i file. Se non usi edit_file o write_file il problema non viene risolto' + String.fromCharCode(10) + (attemptNum > 1 ? '5. ATTENZIONE: tentativi precedenti sono falliti. Usa write_file per riscrivere completamente i file problematici invece di patch parziali.' : '5. Dopo la modifica il sandbox si riavvia automaticamente');
   wcChat.push({ role: 'user', text: '\uD83E\uDD16 Auto-fix crash: ' + errorMsg });
   wcChatRunning = true;
   renderWebCraft(document.getElementById('content'));
@@ -9310,7 +9311,9 @@ async function wcStartSandbox() {
             wcState.sandbox.running = false;
             wcState.sandbox.port = evt.port;
             wcState.sandbox.dir = evt.dir;
-            _wcAutoFixAttempts = 0;
+            // Reset counter only after stable uptime (5s), not immediately on first ready
+            // This prevents infinite loop: crash → fix → restart → crash → reset → crash → ...
+            setTimeout(function() { _wcAutoFixAttempts = 0; }, 5000);
             wcStartAutoFixPoller();
             // Reload skills so newly written log file appears in the panel
             _wcSkillsLoaded = false;