groove-dev 0.27.32 → 0.27.34

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

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

package/node_modules/@groove-dev/cli/src/commands/start.js

@@ -14,15 +14,18 @@ export async function start(options) {
   // ── First-run interactive wizard ────────────────────────────
   let setupKeys = {};
   if (isFirstRun) {
-    try {
-      const result = await runSetupWizard();
-      setupKeys = result.keys || {};
-    } catch (err) {
-      // If stdin is not interactive (piped), skip wizard
-      if (err.code === 'ERR_USE_AFTER_CLOSE') {
-        console.log(chalk.dim('  Non-interactive mode — skipping setup wizard.'));
-      } else {
-        throw err;
+    if (!process.stdin.isTTY) {
+      console.log(chalk.dim('  Non-interactive mode — skipping setup wizard.'));
+    } else {
+      try {
+        const result = await runSetupWizard();
+        setupKeys = result.keys || {};
+      } catch (err) {
+        if (err.code === 'ERR_USE_AFTER_CLOSE') {
+          console.log(chalk.dim('  Non-interactive mode — skipping setup wizard.'));
+        } else {
+          throw err;
+        }
       }
     }
   }

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

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

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

@@ -695,6 +695,28 @@ export function createApi(app, daemon) {
     });
   });
 
+  // --- Project Directory ---
+
+  app.get('/api/project-dir', (req, res) => {
+    res.json({
+      projectDir: daemon.projectDir,
+      recentProjects: daemon.config.recentProjects || [],
+    });
+  });
+
+  app.post('/api/project-dir', (req, res) => {
+    const { path: dirPath } = req.body || {};
+    if (!dirPath || typeof dirPath !== 'string') {
+      return res.status(400).json({ error: 'path is required' });
+    }
+    try {
+      daemon.setProjectDir(dirPath);
+      res.json({ projectDir: daemon.projectDir, recentProjects: daemon.config.recentProjects || [] });
+    } catch (err) {
+      res.status(400).json({ error: err.message });
+    }
+  });
+
   // --- Teams (live agent groups) ---
 
   app.get('/api/teams', (req, res) => {
@@ -3330,7 +3352,12 @@ Keep responses concise. Help them think, don't lecture them about the system the
 
   app.post('/api/tunnels/:id/connect', proOnly, async (req, res) => {
     try {
-      const result = await daemon.tunnelManager.connect(req.params.id);
+      const opts = {};
+      if (req.body?.skipTest && req.body?.testResult) {
+        opts.skipTest = true;
+        opts.testResult = req.body.testResult;
+      }
+      const result = await daemon.tunnelManager.connect(req.params.id, opts);
       res.json(result);
     } catch (err) {
       const body = { error: err.message };

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

@@ -25,6 +25,7 @@ const DEFAULT_CONFIG = {
     autoRotate: true,
     tokenCeilingPerAgent: 5_000_000,
   },
+  recentProjects: [],
 };
 
 export function isFirstRun(grooveDir) {

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

@@ -337,6 +337,20 @@ export class Daemon {
     }
   }
 
+  setProjectDir(dirPath) {
+    if (!dirPath || typeof dirPath !== 'string') throw new Error('Invalid path');
+    const resolved = resolve(dirPath);
+    if (!existsSync(resolved) || !statSync(resolved).isDirectory()) {
+      throw new Error('Directory does not exist');
+    }
+    this.projectDir = resolved;
+    const recents = (this.config.recentProjects || []).filter((r) => r.path !== resolved);
+    recents.unshift({ path: resolved, name: resolved.split('/').pop(), openedAt: new Date().toISOString() });
+    this.config.recentProjects = recents.slice(0, 10);
+    saveConfig(this.grooveDir, this.config);
+    this.broadcast({ type: 'project-dir:changed', data: { projectDir: resolved } });
+  }
+
   async _pollSubscription() {
     if (!this.authToken) return;
     const API_BASE = 'https://docs.groovedev.ai/api/v1';

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

@@ -22,6 +22,7 @@ export class Journalist {
     this.history = []; // recent synthesis summaries
     this._debounceTimer = null;
     this._debounceReason = null;
+    this._forceNextCycle = false;
   }
 
   start(intervalMs = DEFAULT_INTERVAL) {
@@ -61,6 +62,9 @@ export class Journalist {
   }
 
   requestSynthesis(reason = 'unknown') {
+    if (reason === 'completion' || reason === 'rotation_complete') {
+      this._forceNextCycle = true;
+    }
     if (this._debounceTimer) {
       this._debounceReason = reason;
       return;
@@ -84,7 +88,10 @@ export class Journalist {
   }
 
   async cycle() {
-    if (this.synthesizing) return; // Don't overlap
+    if (this.synthesizing) {
+      console.log('  Journalist: skipping cycle (already synthesizing)');
+      return;
+    }
 
     const agents = this.daemon.registry.getAll();
     const running = agents.filter((a) => a.status === 'running');
@@ -99,14 +106,19 @@ export class Journalist {
     const activeAgents = [...running, ...recentlyCompleted];
 
     // Skip if no agents to synthesize
-    if (activeAgents.length === 0) return;
+    if (activeAgents.length === 0) {
+      console.log('  Journalist: skipping cycle (no active agents)');
+      return;
+    }
 
-    // Smart scheduling: skip if no new log output since last cycle
-    if (this.lastCycleAt && !this.hasNewActivity(activeAgents)) {
+    // Smart scheduling: skip if no new log output since last cycle (unless forced by completion/rotation)
+    if (this.lastCycleAt && !this._forceNextCycle && !this.hasNewActivity(activeAgents)) {
+      console.log('  Journalist: skipping cycle (no new activity)');
       return;
     }
 
     this.synthesizing = true;
+    this._forceNextCycle = false;
     this.cycleCount++;
     this.lastCycleAt = Date.now();
 

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

@@ -165,12 +165,13 @@ export class MemoryStore {
       const entries = [];
       const blocks = content.split(/\n(?=## Rotation )/);
       for (const block of blocks) {
-        const headerMatch = block.match(/^## Rotation (\d+) —/);
+        const headerMatch = block.match(/^## Rotation (\d+) — [^(]*\(([\w?-]+) →/);
         if (!headerMatch) continue;
         const body = block.replace(/\n---\s*$/, '').trim();
         entries.push({
           rotationN: parseInt(headerMatch[1], 10),
           body,
+          agentId: headerMatch[2] || null,
         });
       }
       return entries;
@@ -182,6 +183,10 @@ export class MemoryStore {
   appendHandoffBrief(role, entry, workingDir, teamId) {
     if (!role || !entry) return false;
     const chain = this.getHandoffChain(role, workingDir, teamId);
+
+    // Dedup: prevent the same agent from having multiple entries in the chain
+    if (entry.agentId && chain.some(c => c.agentId === entry.agentId)) return false;
+
     const nextN = (chain[0]?.rotationN || 0) + 1;
 
     const block = [

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

@@ -296,6 +296,7 @@ export class ProcessManager {
     this.peakContextUsage = new Map(); // agentId -> highest contextUsage seen
     this.pendingMessages = new Map(); // agentId -> { message, timestamp }
     this._streamThrottle = new Map(); // agentId -> { timer, pending }
+    this._rotatingAgents = new Set(); // agentIds currently being rotated (rotator wrote handoff)
 
   }
 
@@ -623,7 +624,11 @@ For normal file edits within your scope, proceed without review.
           const files = this.daemon.journalist?.getAgentFiles(agent) || [];
           if (files.length > 0) this._triggerIdleQC(agent);
           this._processHandoffs(agent);
-          this._writeCompletionHandoff(agent);
+          if (this._rotatingAgents.has(agent.id)) {
+            this._rotatingAgents.delete(agent.id);
+          } else {
+            this._writeCompletionHandoff(agent).catch(err => console.error(`[Groove] Completion handoff failed for ${agent.name}:`, err.message));
+          }
         }
       });
 
@@ -815,7 +820,11 @@ For normal file edits within your scope, proceed without review.
         const files = this.daemon.journalist?.getAgentFiles(agent) || [];
         if (files.length > 0) this._triggerIdleQC(agent);
         this._processHandoffs(agent);
-        this._writeCompletionHandoff(agent);
+        if (this._rotatingAgents.has(agent.id)) {
+          this._rotatingAgents.delete(agent.id);
+        } else {
+          this._writeCompletionHandoff(agent).catch(err => console.error(`[Groove] Completion handoff failed for ${agent.name}:`, err.message));
+        }
       }
 
       // Update Layer 7 specialization profile for this agent's session
@@ -1149,34 +1158,41 @@ For normal file edits within your scope, proceed without review.
     });
   }
 
-  _writeCompletionHandoff(agent) {
+  async _writeCompletionHandoff(agent) {
     if (!this.daemon.memory || !this.daemon.journalist) return;
     try {
       const agentData = this.daemon.registry.get(agent.id);
-      const filteredLogs = this.daemon.journalist.collectFilteredLogs([agent]);
-      const agentLog = filteredLogs[agent.id];
-      const entries = agentLog?.entries || [];
-      const files = this.daemon.journalist.getAgentFiles(agent) || [];
-
-      const toolSummary = entries
-        .filter(e => e.type === 'tool')
-        .map(e => `- ${e.tool}: ${e.input}`)
-        .slice(-15)
-        .join('\n');
-
-      const errorSummary = entries
-        .filter(e => e.type === 'error')
-        .map(e => `- ${e.text}`)
-        .slice(-5)
-        .join('\n');
-
-      const brief = [
-        `Agent ${agent.name} (${agent.role}) completed.`,
-        agent.prompt ? `Task: ${agent.prompt.slice(0, 300)}` : '',
-        files.length > 0 ? `\nFiles modified:\n${files.slice(0, 15).map(f => '- ' + f).join('\n')}` : '',
-        toolSummary ? `\nRecent actions:\n${toolSummary}` : '',
-        errorSummary ? `\nErrors encountered:\n${errorSummary}` : '',
-      ].filter(Boolean).join('\n');
+
+      let brief;
+      try {
+        brief = await this.daemon.journalist.generateHandoffBrief(agent, { reason: 'completed' });
+      } catch {
+        // Fallback to structural brief if AI synthesis fails
+        const filteredLogs = this.daemon.journalist.collectFilteredLogs([agent]);
+        const agentLog = filteredLogs[agent.id];
+        const entries = agentLog?.entries || [];
+        const files = this.daemon.journalist.getAgentFiles(agent) || [];
+
+        const toolSummary = entries
+          .filter(e => e.type === 'tool')
+          .map(e => `- ${e.tool}: ${e.input}`)
+          .slice(-15)
+          .join('\n');
+
+        const errorSummary = entries
+          .filter(e => e.type === 'error')
+          .map(e => `- ${e.text}`)
+          .slice(-5)
+          .join('\n');
+
+        brief = [
+          `Agent ${agent.name} (${agent.role}) completed.`,
+          agent.prompt ? `Task: ${agent.prompt.slice(0, 300)}` : '',
+          files.length > 0 ? `\nFiles modified:\n${files.slice(0, 15).map(f => '- ' + f).join('\n')}` : '',
+          toolSummary ? `\nRecent actions:\n${toolSummary}` : '',
+          errorSummary ? `\nErrors encountered:\n${errorSummary}` : '',
+        ].filter(Boolean).join('\n');
+      }
 
       this.daemon.memory.appendHandoffBrief(agent.role, {
         timestamp: new Date().toISOString(),
@@ -1184,7 +1200,7 @@ For normal file edits within your scope, proceed without review.
         reason: 'completed',
         oldTokens: agentData?.tokensUsed || 0,
         contextUsage: agentData?.contextUsage || 0,
-        brief: brief.slice(0, 4000),
+        brief: (typeof brief === 'string' ? brief : '').slice(0, 4000),
       }, agent.workingDir, agent.teamId);
     } catch { /* best-effort */ }
   }

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

@@ -309,6 +309,7 @@ export class Rotator extends EventEmitter {
         timestamp: new Date().toISOString(),
       };
 
+      processes._rotatingAgents.add(agentId);
       await processes.kill(agentId);
 
       const routingMode = this.daemon.router.getMode(agentId);

package/node_modules/@groove-dev/daemon/src/tunnel-manager.js

@@ -3,10 +3,19 @@
 
 import { execFileSync, spawn } from 'child_process';
 import { existsSync, writeFileSync, readFileSync, statSync } from 'fs';
-import { resolve } from 'path';
+import { resolve, dirname, join } from 'path';
+import { fileURLToPath } from 'url';
 import { createConnection } from 'net';
 import crypto from 'crypto';
 
+const __dirname = dirname(fileURLToPath(import.meta.url));
+function getLocalVersion() {
+  try {
+    const pkg = JSON.parse(readFileSync(join(__dirname, '..', '..', '..', 'package.json'), 'utf8'));
+    return pkg.version || '0.0.0';
+  } catch { return '0.0.0'; }
+}
+
 const REMOTE_PORT = 31415;
 const DEFAULT_LOCAL_PORT = 31416;
 const MAX_PORT_ATTEMPTS = 10;
@@ -185,7 +194,7 @@ export class TunnelManager {
         '-o', 'StrictHostKeyChecking=accept-new',
         '-o', 'BatchMode=yes',
         target,
-        `curl -sf http://localhost:${REMOTE_PORT}/api/health 2>/dev/null || (which groove >/dev/null 2>&1 && echo __GROOVE_STOPPED__ || echo __GROOVE_NOT_INSTALLED__)`,
+        `bash -lc 'curl -sf http://localhost:${REMOTE_PORT}/api/health 2>/dev/null || (which groove >/dev/null 2>&1 && echo __GROOVE_VER__$(groove --version 2>/dev/null || echo unknown)__GROOVE_STOPPED__ || echo __GROOVE_NOT_INSTALLED__)'`,
       ], {
         encoding: 'utf8',
         timeout: 20000,
@@ -196,7 +205,9 @@ export class TunnelManager {
         return { reachable: true, daemonRunning: false, grooveInstalled: false };
       }
       if (result.includes('__GROOVE_STOPPED__')) {
-        return { reachable: true, daemonRunning: false, grooveInstalled: true };
+        const verMatch = result.match(/__GROOVE_VER__(.+?)__GROOVE_STOPPED__/);
+        const remoteVersion = verMatch ? verMatch[1].trim() : null;
+        return { reachable: true, daemonRunning: false, grooveInstalled: true, remoteVersion };
       }
       return { reachable: true, daemonRunning: true, grooveInstalled: true };
     } catch (err) {
@@ -211,7 +222,7 @@ export class TunnelManager {
     }
   }
 
-  async connect(id) {
+  async connect(id, opts = {}) {
     const config = this.saved.get(id);
     if (!config) throw new Error(`Remote ${id} not found`);
 
@@ -220,7 +231,14 @@ export class TunnelManager {
       return { localPort: existing.localPort, pid: existing.pid };
     }
 
-    const testResult = await this.test(id);
+    this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'testing' } });
+
+    let testResult;
+    if (opts.skipTest && opts.testResult) {
+      testResult = opts.testResult;
+    } else {
+      testResult = await this.test(id);
+    }
     if (!testResult.reachable) {
       throw new Error(testResult.error || 'Host unreachable');
     }
@@ -229,10 +247,17 @@ export class TunnelManager {
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'installing' } });
       await this.remoteInstall(id);
     } else if (!testResult.daemonRunning && testResult.grooveInstalled) {
+      const localVer = getLocalVersion();
+      if (testResult.remoteVersion && testResult.remoteVersion !== localVer) {
+        this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'upgrading', from: testResult.remoteVersion, to: localVer } });
+        await this._remoteUpgrade(id, config);
+      }
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'starting' } });
       await this.autoStart(id);
     }
 
+    this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'connecting' } });
+
     const localPort = await this._findAvailablePort();
     const target = `${config.user}@${config.host}`;
     const keyArgs = config.sshKeyPath ? ['-i', config.sshKeyPath] : [];
@@ -257,13 +282,16 @@ export class TunnelManager {
     let stderrBuf = '';
     tunnel.stderr.on('data', (chunk) => { stderrBuf += chunk.toString(); });
 
-    await new Promise((r) => setTimeout(r, 2000));
-
-    if (tunnel.exitCode !== null) {
-      throw new Error(`Tunnel failed to start: ${stderrBuf.trim() || 'unknown error'}`);
+    let tunnelUp = false;
+    for (let elapsed = 0; elapsed < 8000; elapsed += 500) {
+      await new Promise((r) => setTimeout(r, 500));
+      if (tunnel.exitCode !== null) {
+        throw new Error(`Tunnel failed to start: ${stderrBuf.trim() || 'unknown error'}`);
+      }
+      tunnelUp = await this._isPortInUse(localPort);
+      if (tunnelUp) break;
     }
 
-    const tunnelUp = await this._isPortInUse(localPort);
     if (!tunnelUp) {
       try { process.kill(tunnel.pid); } catch { /* ignore */ }
       throw new Error('Tunnel started but port forward not active');
@@ -320,6 +348,24 @@ export class TunnelManager {
     }
   }
 
+  async _remoteUpgrade(id, config) {
+    const target = `${config.user}@${config.host}`;
+    const keyArgs = config.sshKeyPath ? ['-i', config.sshKeyPath] : [];
+    const sshBase = [...keyArgs, '-p', String(config.port || 22), '-o', 'ConnectTimeout=10', '-o', 'BatchMode=yes', target];
+    const installCmd = config.user === 'root' ? 'npm i -g groove-dev' : 'sudo npm i -g groove-dev';
+
+    try {
+      execFileSync('ssh', [...sshBase, `bash -lc '${installCmd}'`], {
+        encoding: 'utf8',
+        timeout: 120000,
+        stdio: ['pipe', 'pipe', 'pipe'],
+      });
+    } catch (err) {
+      const output = err.stdout?.toString() || err.stderr?.toString() || err.message;
+      throw new Error(`Remote upgrade failed: ${output.slice(-400)}`);
+    }
+  }
+
   async autoStart(id) {
     const config = this.saved.get(id);
     if (!config) throw new Error(`Remote ${id} not found`);
@@ -334,7 +380,7 @@ export class TunnelManager {
         '-o', 'ConnectTimeout=10',
         '-o', 'BatchMode=yes',
         target,
-        `bash -lc 'nohup groove start > /tmp/groove-daemon.log 2>&1 < /dev/null & disown; sleep 4; curl -sf http://localhost:${REMOTE_PORT}/api/health > /dev/null && echo __DAEMON_OK__ || echo __DAEMON_FAIL__'`,
+        `bash -lc 'nohup groove start > /tmp/groove-daemon.log 2>&1 < /dev/null & disown; sleep 5; curl -sf http://localhost:${REMOTE_PORT}/api/health > /dev/null && echo __DAEMON_OK__ || (echo __DAEMON_FAIL__; tail -20 /tmp/groove-daemon.log 2>/dev/null)'`,
       ], {
         encoding: 'utf8',
         timeout: 45000,
@@ -342,10 +388,12 @@ export class TunnelManager {
       });
 
       if (result.includes('__DAEMON_FAIL__')) {
-        throw new Error('Daemon process started but health check failed — check /tmp/groove-daemon.log on remote');
+        const logLines = result.split('__DAEMON_FAIL__')[1]?.trim() || '';
+        const detail = logLines ? `: ${logLines.slice(-300)}` : '';
+        throw new Error(`Remote daemon failed to start${detail}`);
       }
     } catch (err) {
-      if (err.message.includes('Daemon process started')) throw err;
+      if (err.message.includes('Remote daemon failed')) throw err;
       const output = err.stdout?.toString() || err.stderr?.toString() || err.message;
       throw new Error(`Failed to start remote daemon: ${output.slice(-300)}`);
     }
@@ -411,7 +459,7 @@ export class TunnelManager {
     try {
       const result = execFileSync('ssh', [
         ...sshBase,
-        remoteCmd(`nohup groove start > /tmp/groove-daemon.log 2>&1 < /dev/null & disown; sleep 4; curl -sf http://localhost:${REMOTE_PORT}/api/health > /dev/null && echo __DAEMON_OK__ || echo __DAEMON_FAIL__`),
+        remoteCmd(`nohup groove start > /tmp/groove-daemon.log 2>&1 < /dev/null & disown; sleep 5; curl -sf http://localhost:${REMOTE_PORT}/api/health > /dev/null && echo __DAEMON_OK__ || (echo __DAEMON_FAIL__; tail -20 /tmp/groove-daemon.log 2>/dev/null)`),
       ], {
         encoding: 'utf8',
         timeout: 45000,
@@ -419,7 +467,9 @@ export class TunnelManager {
       });
 
       if (result.includes('__DAEMON_FAIL__')) {
-        throw new Error('Groove installed but daemon failed to start — check /tmp/groove-daemon.log on remote');
+        const logLines = result.split('__DAEMON_FAIL__')[1]?.trim() || '';
+        const detail = logLines ? `: ${logLines.slice(-300)}` : '';
+        throw new Error(`Groove installed but daemon failed to start${detail}`);
       }
     } catch (err) {
       if (err.message.includes('Groove installed')) throw err;

package/node_modules/@groove-dev/daemon/test/rotator.test.js

@@ -23,6 +23,7 @@ describe('Rotator', () => {
     mockDaemon = {
       registry: mockRegistry,
       processes: {
+        _rotatingAgents: new Set(),
         async kill(id) { /* mock */ },
         async spawn(config) { return { id: 'new-' + config.role, name: config.name, ...config }; },
       },