groove-dev 0.27.112 → 0.27.113

package/node_modules/@groove-dev/gui/dist/index.html

@@ -6,7 +6,7 @@
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <link rel="icon" type="image/png" href="/favicon.png" />
     <title>Groove GUI</title>
-    <script type="module" crossorigin src="/assets/index-CHu5w3i3.js"></script>
+    <script type="module" crossorigin src="/assets/index-BYh6iHqL.js"></script>
     <link rel="modulepreload" crossorigin href="/assets/vendor-26L3JoZv.js">
     <link rel="modulepreload" crossorigin href="/assets/reactflow-DoBZjiHE.js">
     <link rel="modulepreload" crossorigin href="/assets/codemirror-CFF1Lrnz.js">

package/node_modules/@groove-dev/gui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/gui",
-  "version": "0.27.112",
+  "version": "0.27.113",
   "description": "GROOVE GUI — visual agent control plane",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/node_modules/@groove-dev/gui/src/components/preview/preview-workspace.jsx

@@ -237,7 +237,9 @@ export function PreviewWorkspace({ embedded = false }) {
     return <EmptyPreview />;
   }
 
-  const iframeSrc = previewState.url;
+  const iframeSrc = previewState.teamId
+    ? `/api/preview/${previewState.teamId}/proxy/`
+    : previewState.url;
 
   const deviceWidth = DEVICE_WIDTHS[previewState.deviceSize] || '100%';
   const isFullWidth = previewState.deviceSize === 'desktop';

package/node_modules/@groove-dev/gui/src/stores/groove.js

@@ -224,6 +224,7 @@ export const useGrooveStore = create((set, get) => ({
       get().fetchBetaStatus();
       get().fetchNetworkInstallStatus();
       get().fetchTrainingStatus();
+      get().fetchActivePreviews();
       if (!get().onboardingComplete) get().fetchOnboardingStatus();
       if (window.groove?.auth?.onSubscriptionStatus) {
         window.groove.auth.onSubscriptionStatus((data) => {
@@ -1261,6 +1262,20 @@ export const useGrooveStore = create((set, get) => ({
 
   // ── Preview ──────────────────────────────────────────────
 
+  async fetchActivePreviews() {
+    try {
+      const data = await api.get('/preview');
+      const previews = data.previews || [];
+      if (previews.length > 0) {
+        const p = previews.sort((a, b) => (b.startedAt || 0) - (a.startedAt || 0))[0];
+        set({
+          previewState: { url: `/api/preview/${p.teamId}/proxy/`, teamId: p.teamId, kind: p.kind, deviceSize: 'desktop', screenshotMode: false },
+          showPreviewInAgents: true,
+        });
+      }
+    } catch {}
+  },
+
   openPreview(url, teamId, kind) {
     set({ previewState: { url, teamId, kind, deviceSize: 'desktop', screenshotMode: false }, previewChat: [], showPreviewInAgents: true });
   },

package/node_modules/moe-training/client/parsers/codex.js

@@ -57,15 +57,15 @@ export class CodexParser {
         if (item.type === 'command_execution') {
           const rawOutput = item.aggregated_output || '';
           if (item.exit_code !== 0) {
-            return { type: 'error', content: rawOutput.slice(0, 2000) || `Exit code: ${item.exit_code}` };
+            return { type: 'error', is_error: true, content: rawOutput.slice(0, 2000) || `Exit code: ${item.exit_code}` };
           }
           const obs = truncateObservation(rawOutput);
-          return { type: 'observation', content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
+          return { type: 'observation', is_error: false, content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
         }
         if (item.type === 'file_edit' || item.type === 'file_write' || item.type === 'file_read') {
           const rawOutput = item.output || item.content || '';
           const obs = truncateObservation(rawOutput);
-          return { type: 'observation', content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
+          return { type: 'observation', is_error: false, content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
         }
         return null;
       }

package/node_modules/moe-training/client/parsers/gemini.js

@@ -64,11 +64,11 @@ export class GeminiParser {
         const contentParts = Array.isArray(rawContent) ? rawContent : (typeof rawContent === 'string' ? [{ text: rawContent }] : rawContent ? [rawContent] : []);
         const rawText = contentParts.map((p) => p.text || '').join('');
         const obs = truncateObservation(rawText);
-        return { type: 'observation', content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
+        return { type: 'observation', is_error: false, content: obs.content, truncated: obs.truncated, original_token_count: obs.original_token_count };
       }
 
       case 'error': {
-        return { type: 'error', content: jsonEvent.message || 'Unknown error' };
+        return { type: 'error', is_error: true, content: jsonEvent.message || 'Unknown error' };
       }
 
       case 'agent_end': {

package/node_modules/moe-training/client/step-classifier.js

@@ -1,6 +1,6 @@
 // FSL-1.1-Apache-2.0 — see LICENSE
 
-const ERROR_SIGNAL_RE = /\b(?:error|Error|ERROR|exception|Exception|EXCEPTION|failed|FAILED|exit code [1-9]|ENOENT|EACCES|EPERM|TypeError|ReferenceError|SyntaxError|Cannot find|Module not found|Command failed|non-zero exit)\b/;
+const ERROR_SIGNAL_RE = /(?:^|\n)\s*(?:error[ :\[]\S|Error[ :\[]\S|ERROR[ :\[]\S)|(?:exit code [1-9]|non-zero exit|ENOENT|EACCES|EPERM|Command failed)|(?:(?:TypeError|ReferenceError|SyntaxError|RangeError|URIError):\s)|(?:Cannot find module|Module not found|ModuleNotFoundError|ImportError:)|(?:FATAL|PANIC|Traceback \(most recent)|(?:error TS\d{4}:)/;
 const FIX_SIGNAL_RE = /\b(?:fix|correcting|I see the issue|let me fix|the (?:issue|problem|bug) (?:is|was)|instead I should|my mistake)\b/i;
 
 const CORRECTION_RE = /\b(?:no[,. ](?:that|not|don't|wrong)|that'?s (?:not|wrong|incorrect)|don'?t do that|stop (?:doing|that)|instead (?:of|do)|undo|revert|go back|try (?:again|differently)|you (?:broke|missed|forgot))\b/i;
@@ -54,7 +54,7 @@ export class StepClassifier {
 
     const content = step.content || '';
 
-    if ((step.type === 'action' || step.type === 'observation') && step.is_error !== false && ERROR_SIGNAL_RE.test(content)) {
+    if (step.type === 'observation' && step.is_error !== false && ERROR_SIGNAL_RE.test(content)) {
       step.type = 'error';
     }
 

package/node_modules/moe-training/test/client/step-classifier.test.js

@@ -132,11 +132,11 @@ describe('StepClassifier', () => {
     assert.equal(StepClassifier.countUserInterventions(steps), 0);
   });
 
-  it('reclassifies action with error content to error', () => {
+  it('never reclassifies action to error', () => {
     const classifier = new StepClassifier();
     const step = { type: 'action', content: 'Command failed with exit code 1' };
     const result = classifier.onStep(step);
-    assert.equal(result.type, 'error');
+    assert.equal(result.type, 'action');
   });
 
   it('reclassifies observation with error content to error', () => {
@@ -215,19 +215,75 @@ describe('StepClassifier', () => {
     assert.equal(result.type, 'error');
   });
 
-  it('preserves action type when is_error is false', () => {
+  it('does not reclassify observation containing bare word "error" in source code', () => {
     const classifier = new StepClassifier();
-    const step = { type: 'action', content: 'Command failed with exit code 1', is_error: false };
+    const step = { type: 'observation', content: 'function handleError(err) { console.error(err); }' };
     const result = classifier.onStep(step);
-    assert.equal(result.type, 'action');
+    assert.equal(result.type, 'observation');
   });
 
-  it('still reclassifies action to error when is_error is not set', () => {
+  it('does not reclassify observation with "0 errors" or "found 0 vulnerabilities"', () => {
     const classifier = new StepClassifier();
-    const step = { type: 'action', content: 'Command failed with exit code 1' };
+    const step = { type: 'observation', content: 'Build succeeded\n0 errors, 0 warnings\nfound 0 vulnerabilities' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'observation');
+  });
+
+  it('does not reclassify observation reading a file that mentions exceptions', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: '{"scripts": {"build": "tsc && vite build"}, "name": "my-app"}' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'observation');
+  });
+
+  it('reclassifies observation with real TypeScript build error', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: 'src/main.ts(1,8): error TS2882: Cannot find module' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'error');
+  });
+
+  it('reclassifies observation with Python traceback', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: 'Traceback (most recent call last):\n  File "main.py", line 5' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'error');
+  });
+
+  it('reclassifies observation with actual TypeError message', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: 'TypeError: Cannot read properties of undefined (reading "map")' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'error');
+  });
+
+  it('reclassifies observation with exit code failure', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: 'Process exited with exit code 1' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'error');
+  });
+
+  it('reclassifies observation with ModuleNotFoundError', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'observation', content: 'ModuleNotFoundError: No module named requests' };
     const result = classifier.onStep(step);
     assert.equal(result.type, 'error');
   });
+
+  it('preserves action type even with error keywords', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'action', content: 'Command failed with exit code 1' };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'action');
+  });
+
+  it('preserves action type regardless of is_error flag', () => {
+    const classifier = new StepClassifier();
+    const step = { type: 'action', content: 'ENOENT: no such file', is_error: true };
+    const result = classifier.onStep(step);
+    assert.equal(result.type, 'action');
+  });
 });
 
 describe('StepClassifier.classifyIntent', () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "groove-dev",
-  "version": "0.27.112",
+  "version": "0.27.113",
   "description": "Open-source agent orchestration layer — the AI company OS. Local model agent engine (GGUF/Ollama/llama-server), HuggingFace model browser, MCP integrations (Slack, Gmail, Stripe, 15+), agent scheduling (cron), business roles (CMO, CFO, EA). GUI dashboard, multi-agent coordination, zero cold-start, infinite sessions. Works with Claude Code, Codex, Gemini CLI, Ollama, any local model.",
   "license": "FSL-1.1-Apache-2.0",
   "author": "Groove Dev <hello@groovedev.ai> (https://groovedev.ai)",

package/packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.112",
+  "version": "0.27.113",
   "description": "GROOVE CLI — manage AI coding agents from your terminal",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/daemon/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/daemon",
-  "version": "0.27.112",
+  "version": "0.27.113",
   "description": "GROOVE daemon — agent orchestration engine",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/daemon/src/api.js

@@ -3855,6 +3855,48 @@ Keep responses concise. Help them think, don't lecture them about the system the
   // --- Preview Proxy (same-origin iframe support) ---
   // Forwards HTTP requests to the dev server so the GUI can iframe the preview
   // without cross-origin issues. WebSocket upgrade for HMR is handled in index.js.
+
+  function rewriteAbsoluteUrls(body, proxyBase) {
+    let out = body;
+    // HTML attributes: src, href, action, poster
+    out = out.replace(/((?:src|href|action|poster)\s*=\s*(["']))\/(?!\/|api\/preview\/)/g, `$1${proxyBase}/`);
+    // JS imports: from '/' and import('/')
+    out = out.replace(/(from\s+(["']))\/(?!\/|api\/preview\/)/g, `$1${proxyBase}/`);
+    out = out.replace(/(import\s*\(\s*(["']))\/(?!\/|api\/preview\/)/g, `$1${proxyBase}/`);
+    // CSS url()
+    out = out.replace(/(url\s*\(\s*(["']?))\/(?!\/|api\/preview\/)/g, `$1${proxyBase}/`);
+    return out;
+  }
+
+  const REWRITABLE_TYPES = ['text/html', 'application/javascript', 'text/javascript', 'text/css'];
+
+  function handleProxyResponse(proxyRes, res, proxyBase) {
+    const fwdHeaders = { ...proxyRes.headers };
+    delete fwdHeaders['content-security-policy'];
+    delete fwdHeaders['x-frame-options'];
+
+    const ct = (fwdHeaders['content-type'] || '').toLowerCase();
+    const shouldRewrite = REWRITABLE_TYPES.some((t) => ct.includes(t));
+
+    if (!shouldRewrite) {
+      res.writeHead(proxyRes.statusCode, fwdHeaders);
+      proxyRes.pipe(res);
+      return;
+    }
+
+    const chunks = [];
+    proxyRes.on('data', (c) => chunks.push(c));
+    proxyRes.on('end', () => {
+      let body = Buffer.concat(chunks).toString('utf8');
+      body = rewriteAbsoluteUrls(body, proxyBase);
+      const buf = Buffer.from(body, 'utf8');
+      fwdHeaders['content-length'] = buf.length;
+      delete fwdHeaders['transfer-encoding'];
+      res.writeHead(proxyRes.statusCode, fwdHeaders);
+      res.end(buf);
+    });
+  }
+
   app.all('/api/preview/:teamId/proxy/*', (req, res) => {
     const entry = daemon.preview?.get(req.params.teamId);
     if (!entry || !entry.url) return res.status(404).json({ error: 'No active preview for this team' });
@@ -3865,9 +3907,11 @@ Keep responses concise. Help them think, don't lecture them about the system the
     const proxyPath = req.params[0] || '';
     const search = req.url.includes('?') ? '?' + req.url.split('?').slice(1).join('?') : '';
     const fullPath = '/' + proxyPath + search;
+    const proxyBase = `/api/preview/${req.params.teamId}/proxy`;
 
     const headers = { ...req.headers };
     delete headers.host;
+    delete headers['accept-encoding'];
     headers.host = targetUrl.host;
 
     const proxyReq = httpRequest({
@@ -3876,13 +3920,7 @@ Keep responses concise. Help them think, don't lecture them about the system the
       path: fullPath,
       method: req.method,
       headers,
-    }, (proxyRes) => {
-      const fwdHeaders = { ...proxyRes.headers };
-      delete fwdHeaders['content-security-policy'];
-      delete fwdHeaders['x-frame-options'];
-      res.writeHead(proxyRes.statusCode, fwdHeaders);
-      proxyRes.pipe(res);
-    });
+    }, (proxyRes) => handleProxyResponse(proxyRes, res, proxyBase));
 
     proxyReq.on('error', (err) => {
       if (!res.headersSent) res.status(502).json({ error: `Proxy error: ${err.message}` });
@@ -3899,9 +3937,11 @@ Keep responses concise. Help them think, don't lecture them about the system the
     try { targetUrl = new URL(entry.devUrl || entry.url); } catch { return res.status(500).json({ error: 'Invalid preview URL' }); }
 
     const search = req.url.includes('?') ? '?' + req.url.split('?').slice(1).join('?') : '';
+    const proxyBase = `/api/preview/${req.params.teamId}/proxy`;
 
     const headers = { ...req.headers };
     delete headers.host;
+    delete headers['accept-encoding'];
     headers.host = targetUrl.host;
 
     const proxyReq = httpRequest({
@@ -3910,13 +3950,7 @@ Keep responses concise. Help them think, don't lecture them about the system the
       path: '/' + search,
       method: req.method,
       headers,
-    }, (proxyRes) => {
-      const fwdHeaders = { ...proxyRes.headers };
-      delete fwdHeaders['content-security-policy'];
-      delete fwdHeaders['x-frame-options'];
-      res.writeHead(proxyRes.statusCode, fwdHeaders);
-      proxyRes.pipe(res);
-    });
+    }, (proxyRes) => handleProxyResponse(proxyRes, res, proxyBase));
 
     proxyReq.on('error', (err) => {
       if (!res.headersSent) res.status(502).json({ error: `Proxy error: ${err.message}` });
@@ -6183,13 +6217,15 @@ Keep responses concise. Help them think, don't lecture them about the system the
         stdio: ['ignore', 'pipe', 'pipe'],
         env: tagEnv,
       });
+      daemon._networkCheckProc = proc;
       let stdout = '';
       let stderr = '';
       proc.stdout.on('data', (c) => { stdout += c.toString(); });
       proc.stderr.on('data', (c) => { stderr += c.toString(); });
       const timeout = setTimeout(() => { safeKill(proc, 'SIGTERM'); }, 10_000);
-      proc.on('error', () => { clearTimeout(timeout); resolvePromise(null); });
+      proc.on('error', () => { clearTimeout(timeout); daemon._networkCheckProc = null; resolvePromise(null); });
       proc.on('close', (code) => {
+        daemon._networkCheckProc = null;
         clearTimeout(timeout);
         if (code !== 0) return resolvePromise(null);
         const tags = [];

package/packages/daemon/src/index.js

@@ -290,11 +290,11 @@ export class Daemon {
     });
 
     // Debounced file I/O for registry changes (at most once per 2s)
-    let _registryIoTimer = null;
+    this._registryIoTimer = null;
     const _debouncedRegistryIo = () => {
-      if (_registryIoTimer) return;
-      _registryIoTimer = setTimeout(() => {
-        _registryIoTimer = null;
+      if (this._registryIoTimer) return;
+      this._registryIoTimer = setTimeout(() => {
+        this._registryIoTimer = null;
         this.introducer.writeRegistryFile(this.projectDir);
         this.introducer.injectGrooveSection(this.projectDir);
       }, 2000);
@@ -779,6 +779,11 @@ export class Daemon {
     if (this._stateSaveInterval) clearInterval(this._stateSaveInterval);
     if (this._classifierInterval) clearInterval(this._classifierInterval);
     if (this._subscriptionPollInterval) clearInterval(this._subscriptionPollInterval);
+    if (this._registryIoTimer) clearTimeout(this._registryIoTimer);
+    if (this._networkCheckProc) {
+      try { this._networkCheckProc.kill(); } catch { /* already exited */ }
+      this._networkCheckProc = null;
+    }
 
     // Clean up file watchers and terminal sessions
     this.fileWatcher.unwatchAll();
@@ -823,10 +828,19 @@ export class Daemon {
 
     // Close server
     return new Promise((resolvePromise) => {
-      this.wss.close(() => {
-        this.server.close(() => {
-          console.log('GROOVE daemon stopped.');
-          resolvePromise();
+      this.federationWss.close(() => {
+        this.wss.close(() => {
+          this.server.close(() => {
+            // Unref lingering handles (idle fetch/undici TLS pool connections,
+            // closed servers) so they don't prevent process exit in tests.
+            for (const h of process._getActiveHandles()) {
+              if (typeof h.unref === 'function' && h !== process.stdout && h !== process.stderr) {
+                h.unref();
+              }
+            }
+            console.log('GROOVE daemon stopped.');
+            resolvePromise();
+          });
         });
       });
     });