nothumanallowed 16.0.46 → 16.0.48

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "16.0.46",
+  "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.46';
+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

@@ -229,28 +229,42 @@ class SandboxManager {
       }
     } catch {}
 
-    // Pre-flight: auto-extend CSS if coverage is below threshold.
-    // Detects "incomplete styling" (LLM truncated CSS generation) and calls
-    // the LLM automatically to extend the CSS file with rules for the missing
-    // HTML selectors. NO user intervention required.
-    // Iterates up to 2 times to handle large projects where a single extend
-    // might not cover all selectors (e.g. 200+ missing selectors).
+    // Pre-flight: auto-extend CSS until 100% coverage (or LLM stops making
+    // progress). Target = 100%; max 5 passes as safety; early-exit if two
+    // consecutive passes don't reduce missing count (LLM stuck) or LLM fails.
+    // Goal: ZERO uncovered selectors when the LLM is capable of producing them.
     try {
       const projectName = path.basename(projectDir);
       const cfg = loadConfig();
-      const maxPasses = 2;
+      const maxPasses = 5;
       let totalCovered = 0;
+      let prevMissing = Infinity;
+      let stuckPasses = 0;
       for (let pass = 1; pass <= maxPasses; pass++) {
-        const styleResult = await _autoExtendStylesIfNeeded(projectName, cfg, emit, { minCoverage: 0.85 });
+        const styleResult = await _autoExtendStylesIfNeeded(projectName, cfg, emit, { minCoverage: 1.0 });
         if (!styleResult.extended) {
-          if (styleResult.reason === 'coverage acceptable') break;
-          // LLM failed or skipped — don't loop on a broken provider
+          if (styleResult.reason === 'coverage acceptable') break;  // already 100%
+          // LLM failed or skipped — stop loop
           break;
         }
         const newlyCovered = (styleResult.missingBefore || 0) - (styleResult.missingAfter || 0);
         totalCovered += newlyCovered;
-        emit({ type: 'status', msg: `Pass ${pass}/${maxPasses}: ${styleResult.file} extended — ${newlyCovered} new selectors covered (total ${totalCovered}).` });
-        if ((styleResult.coverageAfter || 0) >= 0.85) break;
+        const missingNow = styleResult.missingAfter || 0;
+        emit({ type: 'status', msg: `Pass ${pass}/${maxPasses}: ${styleResult.file} extended — ${newlyCovered} new selectors covered (${missingNow} still missing, ${((styleResult.coverageAfter || 0) * 100).toFixed(0)}% coverage).` });
+        if (missingNow === 0) {
+          emit({ type: 'status', msg: `100% CSS coverage reached after ${pass} pass${pass === 1 ? '' : 'es'}. Total ${totalCovered} selectors covered.` });
+          break;
+        }
+        if (missingNow >= prevMissing) {
+          stuckPasses++;
+          if (stuckPasses >= 2) {
+            emit({ type: 'warn', msg: `LLM stopped making progress on extension (${missingNow} selectors still missing — likely pseudo-classes or JS-state classes the model can't infer). Stopping at ${((styleResult.coverageAfter || 0) * 100).toFixed(0)}%.` });
+            break;
+          }
+        } else {
+          stuckPasses = 0;
+        }
+        prevMissing = missingNow;
       }
     } catch (e) {
       emit({ type: 'warn', msg: `Auto-extend styles failed: ${(e.message || e).slice(0, 200)}` });
@@ -3132,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) ──────────────────────────────────────────