nothumanallowed 13.5.100 → 13.5.101

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.100",
+  "version": "13.5.101",
   "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

@@ -4128,6 +4128,32 @@ ${completedHeadings ? `## SECTIONS ALREADY WRITTEN (headings only):\n${completed
         const sendReady = (port, dir) => res.write(`data: ${JSON.stringify({ type: 'ready', port, dir })}\n\n`);
         const sendError = (msg) => res.write(`data: ${JSON.stringify({ type: 'error', msg })}\n\n`);
 
+        // Helper: write sandbox log to skills/ dir — defined before try so catch can use it
+        const writeSandboxLog = (sandboxDirArg, freePortArg, extraLines, isError) => {
+          try {
+            const _nl = '\n';
+            const _now = new Date();
+            const _pad = n => String(n).padStart(2,'0');
+            const logTs = _now.getFullYear()+'-'+_pad(_now.getMonth()+1)+'-'+_pad(_now.getDate())+' '+_pad(_now.getHours())+':'+_pad(_now.getMinutes())+':'+_pad(_now.getSeconds());
+            const logName = projName + '-latest.log';
+            const logsDir = path.join(sandboxDirArg, 'skills');
+            fs.mkdirSync(logsDir, { recursive: true });
+            const oldLogs = fs.readdirSync(logsDir).filter(f => f.endsWith('.log') && f.startsWith(projName + '-') && f !== logName);
+            oldLogs.forEach(f => { try { fs.unlinkSync(path.join(logsDir, f)); } catch(_) {} });
+            const title = isError ? '# Sandbox Log — ' + projName + ' [ERRORE]' : '# Sandbox Log — ' + projName;
+            const logContent = title + _nl + 'Avviato: ' + logTs + _nl + 'Porta: ' + (freePortArg || '?') + _nl + _nl + _sbLogLines.join(_nl) + (extraLines ? _nl + _nl + extraLines : '');
+            fs.writeFileSync(path.join(logsDir, logName), logContent, 'utf8');
+            const idxPath = path.join(logsDir, '_index.json');
+            let idx = {};
+            try { idx = JSON.parse(fs.readFileSync(idxPath, 'utf8')); } catch(_) {}
+            idx[logName] = 'log';
+            fs.writeFileSync(idxPath, JSON.stringify(idx), 'utf8');
+          } catch(_) {}
+        };
+
+        let _sandboxDir = null;
+        let _freePort = null;
+
         try {
           // Kill previous sandbox if running
           if (global._wcSandboxProc) {
@@ -4136,6 +4162,7 @@ ${completedHeadings ? `## SECTIONS ALREADY WRITTEN (headings only):\n${completed
           }
 
           const sandboxDir = path.join(os.homedir(), '.nha', 'webcraft', projName);
+          _sandboxDir = sandboxDir;
           sendLog(`📁 Percorso sandbox: ${sandboxDir}`);
           fs.mkdirSync(sandboxDir, { recursive: true });
 
@@ -4553,6 +4580,7 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
             const srv = netMod.createServer();
             srv.listen(0, '127.0.0.1', () => { const p = srv.address().port; srv.close(() => resolve(p)); });
           });
+          _freePort = freePort;
 
           // Patch PORT in .env
           fs.writeFileSync(path.join(sandboxDir, '.env'), envContent.replace('PORT=0', `PORT=${freePort}`), 'utf8');
@@ -4603,29 +4631,6 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
             sendLog('  [server] ' + l);
           });
 
-          // Helper: write sandbox log to skills/ dir (called on both success and error)
-          const writeSandboxLog = (extraLines, isError) => {
-            try {
-              const _nl = '\n';
-              const _now = new Date();
-              const _pad = n => String(n).padStart(2,'0');
-              const logTs = _now.getFullYear()+'-'+_pad(_now.getMonth()+1)+'-'+_pad(_now.getDate())+' '+_pad(_now.getHours())+':'+_pad(_now.getMinutes())+':'+_pad(_now.getSeconds());
-              const logName = projName + '-latest.log';
-              const logsDir = path.join(sandboxDir, 'skills');
-              fs.mkdirSync(logsDir, { recursive: true });
-              const oldLogs = fs.readdirSync(logsDir).filter(f => f.endsWith('.log') && f.startsWith(projName + '-') && f !== logName);
-              oldLogs.forEach(f => { try { fs.unlinkSync(path.join(logsDir, f)); } catch(_) {} });
-              const title = isError ? '# Sandbox Log — ' + projName + ' [ERRORE]' : '# Sandbox Log — ' + projName;
-              const logContent = title + _nl + 'Avviato: ' + logTs + _nl + 'Porta: ' + freePort + _nl + _nl + _sbLogLines.join(_nl) + (extraLines ? _nl + _nl + extraLines : '');
-              fs.writeFileSync(path.join(logsDir, logName), logContent, 'utf8');
-              const idxPath = path.join(logsDir, '_index.json');
-              let idx = {};
-              try { idx = JSON.parse(fs.readFileSync(idxPath, 'utf8')); } catch(_) {}
-              idx[logName] = 'log';
-              fs.writeFileSync(idxPath, JSON.stringify(idx), 'utf8');
-            } catch(_) {}
-          };
-
           // Wait for server to be ready (max 10s)
           await new Promise((resolve, reject) => {
             let attempts = 0;
@@ -4642,11 +4647,11 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
           });
 
           sendLog(`✅ Sandbox pronta!`);
-          writeSandboxLog(null, false);
+          writeSandboxLog(sandboxDir, freePort, null, false);
 
           sendReady(freePort, sandboxDir);
         } catch (e) {
-          writeSandboxLog('❌ ERRORE: ' + e.message, true);
+          if (_sandboxDir) writeSandboxLog(_sandboxDir, _freePort, '❌ ERRORE: ' + e.message, true);
           sendError(e.message);
         }
         res.end();