groove-dev 0.27.71 → 0.27.72

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

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

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

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

package/node_modules/@groove-dev/daemon/src/api.js

@@ -2,7 +2,7 @@
 // FSL-1.1-Apache-2.0 — see LICENSE
 
 import express from 'express';
-import { resolve, dirname, join, sep, relative } from 'path';
+import { resolve, dirname, join, sep, relative, isAbsolute } from 'path';
 import { fileURLToPath } from 'url';
 import { existsSync, readFileSync, readdirSync, statSync, writeFileSync, mkdirSync, unlinkSync, renameSync, rmSync, createReadStream, copyFileSync, realpathSync } from 'fs';
 import { spawn, execFile, execFileSync } from 'child_process';
@@ -611,6 +611,7 @@ export function createApi(app, daemon) {
 
     const proc = spawn('npm', ['install', '-g', pkg], {
       stdio: ['ignore', 'pipe', 'pipe'],
+      shell: true,
       env: { ...process.env, NODE_ENV: undefined },
     });
 
@@ -719,6 +720,7 @@ export function createApi(app, daemon) {
 
           const proc = spawn('codex', ['login'], {
             stdio: ['ignore', 'pipe', 'pipe'],
+            shell: true,
           });
           let stdout = '';
           let stderr = '';
@@ -763,7 +765,7 @@ export function createApi(app, daemon) {
     if (customPath.length > 500) {
       return res.status(400).json({ error: 'Path too long' });
     }
-    if (!customPath.startsWith('/')) {
+    if (!isAbsolute(customPath)) {
       return res.status(400).json({ error: 'Path must be absolute' });
     }
 
@@ -830,6 +832,7 @@ export function createApi(app, daemon) {
           encoding: 'utf8',
           timeout: 5000,
           stdio: ['pipe', 'pipe', 'pipe'],
+          shell: true,
         }).trim();
       } catch (err) {
         version = null;
@@ -1033,7 +1036,7 @@ export function createApi(app, daemon) {
     }
     try {
       daemon.setProjectDir(dirPath);
-      editorRootDir = daemon.projectDir;
+      editorRootOverride = null;
       res.json({ projectDir: daemon.projectDir, recentProjects: daemon.config.recentProjects || [] });
     } catch (err) {
       res.status(400).json({ error: err.message });
@@ -2660,12 +2663,13 @@ Keep responses concise. Help them think, don't lecture them about the system the
     return LANG_MAP[ext] || 'text';
   }
 
-  const IGNORED_NAMES = new Set(['.git', 'node_modules', '.DS_Store', '.groove', '__pycache__', '.next', '.cache', 'dist', 'coverage']);
+  const IGNORED_NAMES = new Set(['.DS_Store', '__pycache__']);
 
-  // Editor root directory — defaults to projectDir but can be changed at runtime
-  let editorRootDir = daemon.projectDir;
+  // Editor root directory — always tracks daemon.projectDir unless explicitly
+  // overridden via POST /api/files/root. Reset on project-dir change.
+  let editorRootOverride = null;
 
-  function getEditorRoot() { return editorRootDir; }
+  function getEditorRoot() { return editorRootOverride || daemon.projectDir; }
 
   function validateFilePath(relPath, projectDir) {
     if (!relPath || typeof relPath !== 'string') return { error: 'path is required' };
@@ -2689,13 +2693,12 @@ Keep responses concise. Help them think, don't lecture them about the system the
 
   // Get/set the editor working directory
   app.get('/api/files/root', (req, res) => {
-    res.json({ root: editorRootDir });
+    res.json({ root: getEditorRoot() });
   });
 
   app.post('/api/files/root', (req, res) => {
     const { root } = req.body || {};
     if (!root || typeof root !== 'string') return res.status(400).json({ error: 'root path is required' });
-    // Must be absolute and exist
     if (!root.startsWith('/')) return res.status(400).json({ error: 'root must be an absolute path' });
     if (root.includes('\0') || root.includes('..')) return res.status(400).json({ error: 'Invalid path' });
     if (!existsSync(root)) return res.status(404).json({ error: 'Directory not found' });
@@ -2703,9 +2706,9 @@ Keep responses concise. Help them think, don't lecture them about the system the
       const stat = statSync(root);
       if (!stat.isDirectory()) return res.status(400).json({ error: 'Path is not a directory' });
     } catch { return res.status(400).json({ error: 'Cannot access directory' }); }
-    editorRootDir = root;
+    editorRootOverride = root;
     daemon.audit.log('editor.root.set', { root });
-    res.json({ ok: true, root: editorRootDir });
+    res.json({ ok: true, root: getEditorRoot() });
   });
 
   // File tree — returns dirs + files for a given path
@@ -2730,18 +2733,22 @@ Keep responses concise. Help them think, don't lecture them about the system the
       const raw = readdirSync(fullPath, { withFileTypes: true });
       const entries = [];
 
-      // Dirs first (sorted), then files (sorted)
-      const isDir = (e) => {
+      const dirs = raw.filter((e) => {
+        if (e.name === '.DS_Store') return false;
         if (e.isDirectory()) return true;
-        if (e.isSymbolicLink()) { try { return statSync(resolve(fullPath, e.name)).isDirectory(); } catch { return false; } }
+        if (e.isSymbolicLink()) {
+          try { return statSync(resolve(fullPath, e.name)).isDirectory(); }
+          catch { return true; }
+        }
         return false;
-      };
-      const dirs = raw.filter((e) => isDir(e) && !IGNORED_NAMES.has(e.name) && !e.name.startsWith('.'))
-        .sort((a, b) => a.name.localeCompare(b.name));
+      }).sort((a, b) => a.name.localeCompare(b.name));
       const files = raw.filter((e) => {
-        if (e.name.startsWith('.')) return false;
+        if (e.name === '.DS_Store') return false;
         if (e.isFile()) return true;
-        if (e.isSymbolicLink()) { try { return statSync(resolve(fullPath, e.name)).isFile(); } catch { return false; } }
+        if (e.isSymbolicLink()) {
+          try { return statSync(resolve(fullPath, e.name)).isFile(); }
+          catch { return false; }
+        }
         return false;
       }).sort((a, b) => a.name.localeCompare(b.name));
 
@@ -2751,7 +2758,7 @@ Keep responses concise. Help them think, don't lecture them about the system the
         let hasChildren = false;
         try {
           const children = readdirSync(childFull, { withFileTypes: true });
-          hasChildren = children.some((c) => !c.name.startsWith('.') && !IGNORED_NAMES.has(c.name));
+          hasChildren = children.some((c) => c.name !== '.DS_Store');
         } catch { /* unreadable */ }
         entries.push({ name: d.name, type: 'dir', path: childPath, hasChildren });
       }
@@ -4077,6 +4084,7 @@ Keep responses concise. Help them think, don't lecture them about the system the
 
     const proc = spawn('npm', ['install', '-g', pkg], {
       stdio: ['ignore', 'pipe', 'pipe'],
+      shell: true,
       env: { ...process.env, NODE_ENV: undefined },
     });
 

package/node_modules/@groove-dev/daemon/src/providers/claude-code.js

@@ -336,6 +336,7 @@ export class ClaudeCodeProvider extends Provider {
     const child = cpSpawn('claude', ['auth', 'login', '--claudeai'], {
       detached: true,
       stdio: 'ignore',
+      shell: true,
     });
     child.unref();
     return { pid: child.pid };
@@ -359,6 +360,7 @@ export class ClaudeCodeProvider extends Provider {
     return new Promise((resolve) => {
       const child = cpSpawn('claude', ['auth', 'login', '--claudeai'], {
         stdio: ['ignore', 'pipe', 'pipe'],
+        shell: true,
       });
       let stdout = '';
       let stderr = '';

package/node_modules/@groove-dev/daemon/src/providers/codex.js

@@ -72,6 +72,7 @@ export class CodexProvider extends Provider {
     return new Promise((res) => {
       const proc = spawn('codex', ['login', '--with-api-key'], {
         stdio: ['pipe', 'pipe', 'pipe'],
+        shell: true,
         timeout: 15000,
       });
       let stderr = '';

package/node_modules/@groove-dev/daemon/src/providers/index.js

@@ -2,7 +2,7 @@
 // FSL-1.1-Apache-2.0 — see LICENSE
 
 import { execSync } from 'child_process';
-import { dirname as pathDirname } from 'path';
+import { dirname as pathDirname, delimiter as pathDelimiter } from 'path';
 import { ClaudeCodeProvider } from './claude-code.js';
 import { CodexProvider } from './codex.js';
 import { GeminiProvider } from './gemini.js';
@@ -26,10 +26,10 @@ function _augmentPathWithCustomPaths() {
   for (const p of Object.values(_providerPaths)) {
     if (p && typeof p === 'string') {
       const dir = pathDirname(p);
-      if (dir && !cur.split(':').includes(dir)) dirs.push(dir);
+      if (dir && !cur.split(pathDelimiter).includes(dir)) dirs.push(dir);
     }
   }
-  if (dirs.length) process.env.PATH = [...dirs, cur].join(':');
+  if (dirs.length) process.env.PATH = [...dirs, cur].join(pathDelimiter);
 }
 
 export function getProviderPath(id) {