nothumanallowed 16.0.47 → 16.0.48

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "16.0.47",
+  "version": "16.0.48",
   "description": "Local AI assistant: 80 tools (Gmail, Calendar, Drive, GitHub, Slack, browser, code, files), 38 agents, visual workflows (Studio, AWF, WebCraft). Install with `npm i -g nothumanallowed`, run with `nha ui`. Free tier built-in (Liara), no API key required. Your data stays on your PC — OAuth tokens local, no cloud. Open-source MIT.",
   "type": "module",
   "bin": {

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 = '16.0.47';
+export const VERSION = '16.0.48';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/server/routes/webcraft.mjs

@@ -3146,48 +3146,59 @@ export function register(router) {
   }
 
   router.post('/api/studio/webcraft/lint', async (req, res) => {
+    // ROBUST lint endpoint — NEVER returns 500. If any linter crashes,
+    // returns 200 with empty diagnostics + the error logged. A failed lint
+    // must not break the IDE streaming flow.
     try {
       const { projectName, path: relPath } = await parseBody(req);
       if (!projectName || !relPath) return sendError(res, 400, 'projectName and path required');
-      const content = ProjectStore.readFile(projectName, relPath);
-      if (content === null) return sendJSON(res, 200, { diagnostics: [] });
+      let content;
+      try { content = ProjectStore.readFile(projectName, relPath); }
+      catch { content = null; }
+      if (content === null || content === undefined) return sendJSON(res, 200, { diagnostics: [] });
 
       const ext = (relPath.split('.').pop() || '').toLowerCase();
       let diagnostics = [];
 
-      // JavaScript / JSX — try TypeScript checkJs first, fallback to acorn
+      // Each linter wrapped individually — a crash in one doesn't kill the others
       if (['js', 'mjs', 'jsx', 'cjs'].includes(ext)) {
-        const projectDir = ProjectStore.dir(projectName);
-        const tsDiags = await lintJSWithTypeScript(projectDir, relPath);
-        diagnostics = tsDiags || lintJS(content, relPath, projectName);
-      }
-
-      // JSON — parse errors with precise location
-      if (ext === 'json') {
-        try { JSON.parse(content); } catch (e) {
-          const posMatch = e.message.match(/position (\d+)/i);
-          const pos = posMatch ? parseInt(posMatch[1]) : 0;
-          const before = content.slice(0, pos).split('\n');
-          diagnostics.push({
-            from: { line: before.length, col: (before[before.length - 1] || '').length },
-            severity: 'error',
-            message: e.message,
-          });
+        try {
+          const projectDir = ProjectStore.dir(projectName);
+          const tsDiags = await lintJSWithTypeScript(projectDir, relPath).catch(() => null);
+          diagnostics = tsDiags || lintJS(content, relPath, projectName) || [];
+        } catch (e) {
+          console.error('[lint] JS linter crashed for', relPath, ':', e.message);
+          diagnostics = [];
         }
-      }
-
-      // CSS — brace balance + property validation
-      if (ext === 'css') {
-        diagnostics = lintCSS(content);
-      }
-
-      // HTML/HTM — tag balance + reference validation
-      if (ext === 'html' || ext === 'htm') {
-        diagnostics = lintHTML(content, relPath, projectName);
+      } else if (ext === 'json') {
+        try {
+          JSON.parse(content);
+        } catch (e) {
+          try {
+            const posMatch = e.message.match(/position (\d+)/i);
+            const pos = posMatch ? parseInt(posMatch[1]) : 0;
+            const before = content.slice(0, pos).split('\n');
+            diagnostics.push({
+              from: { line: before.length, col: (before[before.length - 1] || '').length },
+              severity: 'error',
+              message: e.message,
+            });
+          } catch {}
+        }
+      } else if (ext === 'css') {
+        try { diagnostics = lintCSS(content) || []; }
+        catch (e) { console.error('[lint] CSS linter crashed:', e.message); diagnostics = []; }
+      } else if (ext === 'html' || ext === 'htm') {
+        try { diagnostics = lintHTML(content, relPath, projectName) || []; }
+        catch (e) { console.error('[lint] HTML linter crashed:', e.message); diagnostics = []; }
       }
 
       sendJSON(res, 200, { diagnostics });
-    } catch (e) { sendError(res, 500, e.message); }
+    } catch (e) {
+      // Even the outer catch returns 200 — lint failures must not break the IDE
+      console.error('[lint] outer error:', e.message);
+      sendJSON(res, 200, { diagnostics: [], error: e.message });
+    }
   });
 
   // ── File write (from IDE editor) ──────────────────────────────────────────