groove-dev 0.27.88 → 0.27.91

package/CLAUDE.md

@@ -263,14 +263,3 @@ Audit-driven release. Multi-agent orchestration system with 7 coordination layer
 - Dashboard: routing donut, cache panel, context health gauges
 - Monitor/QC agent mode (stay active, loop)
 - Distribution: demo video, HN launch, Twitter content
-
-<!-- GROOVE:START -->
-## GROOVE Orchestration (auto-injected)
-Active agents: 2
-| Name | Role | Scope |
-|------|------|-------|
-| fullstack-23 | fullstack | - |
-| fullstack-12 | fullstack | - |
-See AGENTS_REGISTRY.md for full agent state.
-**Memory policy:** GROOVE manages project memory automatically. Do not read or write MEMORY.md or .groove/memory/ files directly.
-<!-- GROOVE:END -->

package/moe-training/client/parsers/claude-code.js

@@ -30,7 +30,6 @@ export class ClaudeCodeParser {
           results.push({
             type: 'thought',
             content: block.text || '',
-            token_count: jsonEvent.message?.usage?.output_tokens || 0,
           });
         } else if (block.type === 'tool_use') {
           this._pendingToolUse.set(block.id, block);
@@ -67,7 +66,6 @@ export class ClaudeCodeParser {
       return {
         type: 'resolution',
         content: typeof jsonEvent.result === 'string' ? jsonEvent.result : JSON.stringify(jsonEvent.result),
-        token_count: jsonEvent.total_tokens_used || 0,
       };
     }
 

package/moe-training/client/session-attestation.js

@@ -2,6 +2,7 @@
 
 import { platform, arch, cpus, totalmem, hostname, release, endianness } from 'node:os';
 import { createHash } from 'node:crypto';
+import { fileURLToPath } from 'node:url';
 import { generateECDHKeypair, deriveSharedSecret, signEnvelope, computeAppHash } from '../shared/crypto.js';
 
 export class SessionAttestation {
@@ -14,7 +15,7 @@ export class SessionAttestation {
     const keypair = generateECDHKeypair();
     let appVersionHash = '';
     try {
-      appVersionHash = computeAppHash(new URL(import.meta.url).pathname);
+      appVersionHash = computeAppHash(fileURLToPath(import.meta.url));
     } catch {
       appVersionHash = 'unknown';
     }

package/moe-training/client/trajectory-capture.js

@@ -172,6 +172,12 @@ export class TrajectoryCapture {
       ev.arguments = this._scrubObject(ev.arguments);
     }
 
+    if (!ev.token_count || ev.token_count < 2) {
+      const text = ev.content || '';
+      const argsLen = ev.arguments ? JSON.stringify(ev.arguments).length : 0;
+      ev.token_count = Math.max(1, Math.ceil((text.length + argsLen) / 4));
+    }
+
     const step = {
       step: ++ctx.stepCount,
       type: ev.type,

package/moe-training/test/client/parsers/claude-code.test.js

@@ -17,7 +17,7 @@ describe('ClaudeCodeParser', () => {
     const result = parser.parseEvent(event);
     assert.equal(result.type, 'thought');
     assert.equal(result.content, 'I need to fix the bug');
-    assert.equal(result.token_count, 50);
+    assert.equal(result.token_count, undefined);
   });
 
   it('parses tool_use block as action', () => {
@@ -79,7 +79,7 @@ describe('ClaudeCodeParser', () => {
     const result = parser.parseEvent(event);
     assert.equal(result.type, 'resolution');
     assert.equal(result.content, 'Task completed successfully');
-    assert.equal(result.token_count, 500);
+    assert.equal(result.token_count, undefined);
   });
 
   it('extracts tokens from assistant event', () => {

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

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.88",
+  "version": "0.27.91",
   "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.88",
+  "version": "0.27.91",
   "description": "GROOVE daemon — agent orchestration engine",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

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

@@ -146,7 +146,10 @@ export function createApi(app, daemon) {
         const team = daemon.teams.get(config.teamId);
         if (team?.workingDir) config.workingDir = team.workingDir;
       }
-      // Inherit configured default model if the request didn't pick one
+      // Inherit configured defaults if the request didn't pick them
+      if (!config.provider && daemon.config?.defaultProvider) {
+        config.provider = daemon.config.defaultProvider;
+      }
       if (!config.model && daemon.config?.defaultModel) {
         config.model = daemon.config.defaultModel;
       }
@@ -1112,8 +1115,8 @@ export function createApi(app, daemon) {
   app.post('/api/conversations', async (req, res) => {
     try {
       const { provider, model, title, mode, reasoning_effort, verbosity } = req.body;
-      if (!provider || typeof provider !== 'string') {
-        return res.status(400).json({ error: 'provider is required' });
+      if (provider && typeof provider !== 'string') {
+        return res.status(400).json({ error: 'provider must be a string' });
       }
       if (mode && mode !== 'api' && mode !== 'agent') {
         return res.status(400).json({ error: 'mode must be "api" or "agent"' });
@@ -4501,8 +4504,10 @@ Keep responses concise. Help them think, don't lecture them about the system the
     }
 
     daemon.config.defaultProvider = provider;
+    daemon.config.defaultChatProvider = provider;
     if (model && typeof model === 'string' && model.length <= 100) {
       daemon.config.defaultModel = model.trim();
+      daemon.config.defaultChatModel = model.trim();
     }
     const { saveConfig } = await import('./firstrun.js');
     saveConfig(daemon.grooveDir, daemon.config);
@@ -4534,15 +4539,20 @@ Keep responses concise. Help them think, don't lecture them about the system the
     saveConfig(daemon.grooveDir, daemon.config);
 
     if (enabled) {
-      const userId = ConsentManager.getOrCreateUserId();
-      const consent = new ConsentManager();
       try {
-        consent.recordConsent(userId, true, '1.0');
-      } finally {
-        consent.close();
+        const userId = ConsentManager.getOrCreateUserId();
+        const consent = new ConsentManager();
+        try {
+          consent.recordConsent(userId, true, '1.0');
+        } finally {
+          consent.close();
+        }
+        await daemon._initTrajectoryCapture();
+        daemon.state.set('training_enrolled_at', new Date().toISOString());
+      } catch (e) {
+        daemon.config.training_opt_in = false;
+        return res.status(500).json({ error: 'Failed to enable data sharing', detail: e.message });
       }
-      await daemon._initTrajectoryCapture();
-      daemon.state.set('training_enrolled_at', new Date().toISOString());
     } else {
       if (daemon.trajectoryCapture) {
         try { await daemon.trajectoryCapture.shutdown(); } catch (e) { /* */ }

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

@@ -55,6 +55,22 @@ export class ConversationManager {
     return null;
   }
 
+  _resolveAutoProviderModel(preferredProvider) {
+    const priority = ['claude-code', 'codex', 'gemini', 'grok', 'ollama'];
+    const candidates = preferredProvider ? [preferredProvider] : priority;
+
+    for (const pid of candidates) {
+      if (!isProviderInstalled(pid)) continue;
+      const p = getProvider(pid);
+      if (!p) continue;
+      const models = p.constructor.models || [];
+      const chatModel = models.find((m) => m.type !== 'image') || models[0];
+      if (chatModel) return { provider: pid, model: chatModel.id };
+    }
+
+    return { provider: 'claude-code', model: 'claude-sonnet-4-6' };
+  }
+
   async create(provider, model, title, mode = 'api', options = {}) {
     if (!provider && this.daemon.config?.defaultChatProvider) {
       provider = this.daemon.config.defaultChatProvider;
@@ -63,6 +79,12 @@ export class ConversationManager {
       model = this.daemon.config.defaultChatModel;
     }
 
+    if (!provider || !model) {
+      const resolved = this._resolveAutoProviderModel(provider);
+      if (!provider) provider = resolved.provider;
+      if (!model) model = resolved.model;
+    }
+
     const id = randomUUID().slice(0, 12);
     const now = new Date().toISOString();
 
@@ -328,12 +350,16 @@ export class ConversationManager {
     let providerName = conv.provider;
 
     if (!provider || !isProviderInstalled(conv.provider)) {
-      const priority = ['claude-code', 'gemini', 'codex', 'ollama'];
-      const fallbackId = priority.find((p) => isProviderInstalled(p));
-      if (!fallbackId) throw new Error('No provider available for chat');
-      provider = getProvider(fallbackId);
-      providerName = fallbackId;
-      modelId = null;
+      const resolved = this._resolveAutoProviderModel(null);
+      provider = getProvider(resolved.provider);
+      if (!provider) throw new Error('No provider available for chat');
+      providerName = resolved.provider;
+      modelId = resolved.model;
+    }
+
+    if (!modelId) {
+      const resolved = this._resolveAutoProviderModel(providerName);
+      modelId = resolved.model;
     }
 
     // Build messages array for direct API call

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

@@ -191,7 +191,7 @@ export class PreviewService {
     // expansion, and shell builtins work as the planner wrote them.
     const proc = cpSpawn('bash', ['-lc', command], {
       cwd: baseDir,
-      env: { ...process.env, FORCE_COLOR: '0', CI: '' },
+      env: { ...process.env, FORCE_COLOR: '0', CI: '', BROWSER: 'none' },
       stdio: ['ignore', 'pipe', 'pipe'],
       detached: false,
     });

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

@@ -432,10 +432,17 @@ export class ProcessManager {
       if (installed.length === 0) {
         throw new Error('No AI providers installed. Install Claude Code, Gemini CLI, Codex, or Ollama first.');
       }
-      // Priority: claude-code > gemini > codex > local (local replaces ollama in UI)
-      const priority = ['claude-code', 'gemini', 'codex', 'local', 'ollama'];
-      const best = priority.find((p) => installed.some((i) => i.id === p)) || installed[0].id;
-      providerName = best;
+      // If a model is specified, find the provider that supports it
+      if (config.model && config.model !== 'auto') {
+        const match = installed.find((p) => p.models?.some((m) => m.id === config.model));
+        if (match) providerName = match.id;
+      }
+      // Fallback: priority-based selection
+      if (!providerName) {
+        const priority = ['claude-code', 'gemini', 'codex', 'local', 'ollama'];
+        const best = priority.find((p) => installed.some((i) => i.id === p)) || installed[0].id;
+        providerName = best;
+      }
     }
 
     const provider = getProvider(providerName);
@@ -1282,6 +1289,10 @@ For normal file edits within your scope, proceed without review.
       }
     }
 
+    // Phase 2 spawns are async — _checkPhase2 may have consumed the pending
+    // entry but the spawn() calls haven't resolved yet. Wait for them.
+    if (this._phase2Spawning?.has(teamId)) return;
+
     const teamAgents = this.daemon.registry.getAll().filter((a) => a.teamId === teamId && a.role !== 'planner');
     if (teamAgents.length === 0) return;
     const terminal = new Set(['completed', 'crashed', 'stopped', 'killed']);
@@ -1367,6 +1378,12 @@ For normal file edits within your scope, proceed without review.
           return files.length === 0;
         });
 
+        // Track that phase 2 spawns are in-flight for this team so
+        // _checkPreviewReady doesn't race ahead of the async spawn calls.
+        const groupTeamId = group.agents[0]?.teamId || this.daemon.teams.getDefault()?.id || null;
+        if (!this._phase2Spawning) this._phase2Spawning = new Map();
+        const spawnPromises = [];
+
         // Auto-spawn phase 2 agents — if phase 1 was idle, clear the prompt
         // so QC also waits for instructions instead of auditing nothing
         for (const config of group.agents) {
@@ -1406,7 +1423,7 @@ For normal file edits within your scope, proceed without review.
           try {
             const validated = validateAgentConfig(config);
             if (!validated.teamId) validated.teamId = this.daemon.teams.getDefault()?.id || null;
-            this.spawn(validated).then((agent) => {
+            const p = this.spawn(validated).then((agent) => {
               this.daemon.broadcast({
                 type: 'phase2:spawned',
                 agentId: agent.id,
@@ -1422,6 +1439,7 @@ For normal file edits within your scope, proceed without review.
                 error: err.message,
               });
             });
+            spawnPromises.push(p);
           } catch (err) {
             console.error(`[Groove] Phase 2 config invalid for ${config.role}: ${err.message}`);
             this.daemon.broadcast({
@@ -1431,6 +1449,17 @@ For normal file edits within your scope, proceed without review.
             });
           }
         }
+
+        // Mark this team as having phase 2 spawns in-flight. Cleared once
+        // all spawn promises settle so _checkPreviewReady won't race ahead.
+        if (spawnPromises.length > 0 && groupTeamId) {
+          this._phase2Spawning.set(groupTeamId, (this._phase2Spawning.get(groupTeamId) || 0) + spawnPromises.length);
+          Promise.allSettled(spawnPromises).then(() => {
+            const remaining = (this._phase2Spawning.get(groupTeamId) || 0) - spawnPromises.length;
+            if (remaining <= 0) this._phase2Spawning.delete(groupTeamId);
+            else this._phase2Spawning.set(groupTeamId, remaining);
+          });
+        }
       }
     }
   }

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

@@ -282,7 +282,7 @@ export class ClaudeCodeProvider extends Provider {
     let finished = false;
     const finish = () => { if (!finished) { finished = true; onDone(); } };
     const body = JSON.stringify({
-      model: model || 'claude-sonnet-4-6',
+      model,
       messages,
       max_tokens: 8192,
       stream: true,

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

@@ -163,7 +163,7 @@ export class CodexProvider extends Provider {
     const effort = reasoningEffort || 'medium';
     const verb = verbosity || 'medium';
     const body = {
-      model: model || 'gpt-5.5',
+      model,
       input: previousResponseId ? [messages[messages.length - 1]] : messages,
       stream: true,
       reasoning: { effort },

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

@@ -94,7 +94,7 @@ export class GeminiProvider extends Provider {
     const controller = new AbortController();
     let finished = false;
     const finish = () => { if (!finished) { finished = true; onDone(); } };
-    const m = model || 'gemini-2.5-flash';
+    const m = model;
     const contents = messages.map((msg) => ({
       role: msg.role === 'assistant' ? 'model' : 'user',
       parts: [{ text: msg.content }],

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

@@ -58,7 +58,7 @@ export class GrokProvider extends Provider {
   getLoopConfig(agent) {
     return {
       apiBase: 'https://api.x.ai/v1',
-      model: agent.model || 'grok-4-1-fast',
+      model: agent.model,
       contextWindow: 131072,
       temperature: 0.1,
       maxResponseTokens: 16384,
@@ -97,7 +97,7 @@ export class GrokProvider extends Provider {
         'Content-Type': 'application/json',
       },
       body: JSON.stringify({
-        model: model || 'grok-4-1-fast',
+        model,
         messages,
         stream: true,
       }),