groove-dev 0.27.30 → 0.27.33

package/packages/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/packages/daemon/src/providers/local.js

@@ -93,14 +93,16 @@ export class LocalProvider extends Provider {
 
   static _hasOllama() {
     try {
-      execSync('which ollama', { stdio: 'ignore' });
+      const cmd = process.platform === 'win32' ? 'where ollama' : 'which ollama';
+      execSync(cmd, { stdio: 'ignore' });
       return true;
     } catch { return false; }
   }
 
   static _hasLlamaServer() {
     try {
-      execSync('which llama-server', { stdio: 'ignore' });
+      const cmd = process.platform === 'win32' ? 'where llama-server' : 'which llama-server';
+      execSync(cmd, { stdio: 'ignore' });
       return true;
     } catch { return false; }
   }

package/packages/daemon/src/providers/ollama.js

@@ -52,7 +52,8 @@ export class OllamaProvider extends Provider {
 
   static isInstalled() {
     try {
-      execSync('which ollama', { stdio: 'ignore' });
+      const cmd = process.platform === 'win32' ? 'where ollama' : 'which ollama';
+      execSync(cmd, { stdio: 'ignore' });
       return true;
     } catch {
       return false;
@@ -67,6 +68,9 @@ export class OllamaProvider extends Provider {
     if (platform === 'linux') {
       return { command: 'curl -fsSL https://ollama.ai/install.sh | sh', platform: 'Linux' };
     }
+    if (platform === 'win32') {
+      return { command: 'winget install Ollama.Ollama', alt: 'Or download from https://ollama.ai/download', platform: 'Windows' };
+    }
     return { command: 'Download from https://ollama.ai/download', platform: 'other' };
   }
 
@@ -98,6 +102,22 @@ export class OllamaProvider extends Provider {
         }
       }
     }
+    if (platform === 'win32') {
+      try {
+        let cmd = 'ollama';
+        try {
+          execSync('where ollama', { stdio: 'ignore' });
+        } catch {
+          const localAppData = process.env.LOCALAPPDATA || '';
+          const fallback = localAppData + '\\Programs\\Ollama\\ollama.exe';
+          cmd = fallback;
+        }
+        execFile(cmd, ['serve'], { stdio: 'ignore', detached: true, shell: true }).unref();
+        return { started: true, method: 'ollama serve' };
+      } catch {
+        return { started: false, command: 'ollama serve' };
+      }
+    }
     // Linux / other
     try {
       execFile('ollama', ['serve'], { stdio: 'ignore', detached: true }).unref();
@@ -109,6 +129,14 @@ export class OllamaProvider extends Provider {
 
   static stopServer() {
     const platform = process.platform;
+    if (platform === 'win32') {
+      try {
+        execSync('taskkill /IM ollama.exe /F', { stdio: 'ignore', timeout: 5000 });
+        return { stopped: true, method: 'taskkill' };
+      } catch {
+        return { stopped: false };
+      }
+    }
     if (platform === 'darwin') {
       try {
         execSync('brew services stop ollama', { stdio: 'ignore', timeout: 10000 });
@@ -135,7 +163,11 @@ export class OllamaProvider extends Provider {
       minRAM: 4,
       recommendedRAM: 16,
       gpuRecommended: true,
-      note: 'Apple Silicon Macs use unified memory — all RAM is GPU RAM. NVIDIA GPUs recommended on Linux.',
+      note: process.platform === 'win32'
+        ? 'NVIDIA or AMD GPUs recommended. Ensure GPU drivers are up to date.'
+        : process.platform === 'darwin'
+          ? 'Apple Silicon Macs use unified memory — all RAM is GPU RAM.'
+          : 'NVIDIA GPUs recommended on Linux.',
     };
   }
 
@@ -151,7 +183,7 @@ export class OllamaProvider extends Provider {
     let gpu = null;
     if (isAppleSilicon) {
       gpu = { type: 'apple-silicon', name: cpuModel.replace(/Apple /g, ''), vram: totalRamGb, note: 'Unified memory — all RAM available to GPU' };
-    } else if (platform === 'linux') {
+    } else if (platform === 'linux' || platform === 'win32') {
       try {
         const out = execSync('nvidia-smi --query-gpu=name,memory.total --format=csv,noheader,nounits', { encoding: 'utf8', timeout: 5000 });
         const [name, vram] = out.trim().split(', ');

package/packages/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/packages/daemon/src/tunnel-manager.js

@@ -211,7 +211,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 +220,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');
     }
@@ -233,6 +240,8 @@ export class TunnelManager {
       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 +266,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');