groove-dev 0.27.144 → 0.27.145

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

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

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

@@ -89,27 +89,10 @@ export class ConversationManager {
     const id = randomUUID().slice(0, 12);
     const now = new Date().toISOString();
 
-    let agentId = null;
-
-    if (mode === 'agent') {
-      const defaultTeam = this.daemon.teams.getDefault();
-      const workingDir = defaultTeam?.workingDir || this.daemon.projectDir;
-
-      const agent = await this.daemon.processes.spawn({
-        role: 'chat',
-        provider,
-        model: model || null,
-        workingDir,
-        teamId: defaultTeam?.id || null,
-        permission: 'full',
-      });
-      agentId = agent.id;
-    }
-
     const conversation = {
       id,
       title: title || 'New Chat',
-      agentId,
+      agentId: null,
       provider,
       model: model || null,
       mode: mode === 'agent' ? 'agent' : 'api',
@@ -265,35 +248,7 @@ export class ConversationManager {
     this._modeChanging.add(id);
 
     try {
-      if (mode === 'agent') {
-        const existingAgent = conv.agentId ? this.daemon.registry.get(conv.agentId) : null;
-        const alive = existingAgent && (existingAgent.status === 'running' || existingAgent.status === 'starting');
-
-        if (!alive) {
-          const defaultTeam = this.daemon.teams.getDefault();
-          const workingDir = defaultTeam?.workingDir || this.daemon.projectDir;
-          const agent = await this.daemon.processes.spawn({
-            role: 'chat',
-            provider: conv.provider,
-            model: conv.model || null,
-            workingDir,
-            teamId: defaultTeam?.id || null,
-            permission: 'full',
-          });
-          conv.agentId = agent.id;
-        }
-      } else {
-        // Switching to API mode — kill the agent if running
-        this._killStreamingProcess(id);
-        if (conv.agentId) {
-          const agent = this.daemon.registry.get(conv.agentId);
-          if (agent && (agent.status === 'running' || agent.status === 'starting')) {
-            try { await this.daemon.processes.kill(conv.agentId); } catch { /* ignore */ }
-          }
-          if (agent) this.daemon.registry.remove(conv.agentId);
-          conv.agentId = null;
-        }
-      }
+      this._killStreamingProcess(id);
 
       conv.mode = mode;
       conv.updatedAt = new Date().toISOString();
@@ -354,7 +309,6 @@ export class ConversationManager {
   async sendMessage(id, message, history, { reasoningEffort, verbosity } = {}) {
     const conv = this.conversations.get(id);
     if (!conv) throw new Error('Conversation not found');
-    if (conv.mode !== 'api') throw new Error('sendMessage only works in API mode');
 
     this._killStreamingProcess(id);
 
@@ -395,6 +349,17 @@ export class ConversationManager {
       try { tcAgentId = tc.onChatTurnStart(id, providerName, modelId, message); } catch { /* never block chat */ }
     }
 
+    // Agent mode prefers headless CLI (has tools + web search), falls back to API
+    if (conv.mode === 'agent') {
+      const headlessCmd = provider.buildHeadlessCommand(
+        this._buildHistoryPrompt(history, message), modelId,
+      );
+      if (headlessCmd) {
+        return this._sendViaHeadlessCLI(id, conv, provider, providerName, modelId, message, history, tcAgentId, tc);
+      }
+      // Provider has no CLI — fall through to API streaming
+    }
+
     // Try direct API streaming first (sub-second latency)
     const controller = provider.streamChat(
       messages, modelId, apiKey,
@@ -448,6 +413,11 @@ export class ConversationManager {
     }
 
     // Fallback: headless CLI spawn (for providers without streamChat or missing API key)
+    return this._sendViaHeadlessCLI(id, conv, provider, providerName, modelId, message, history, tcAgentId, tc);
+  }
+
+  _sendViaHeadlessCLI(id, conv, provider, providerName, modelId, message, history, tcAgentId, tc) {
+    let tcResponseText = '';
     const prompt = this._buildHistoryPrompt(history, message);
     const headlessCmd = provider.buildHeadlessCommand(prompt, modelId);
     if (!headlessCmd) {

package/node_modules/@groove-dev/daemon/src/routes/agents.js

@@ -285,89 +285,12 @@ export function registerAgentRoutes(app, daemon) {
       daemon.conversations.autoTitle(req.params.id, message.trim());
       daemon.conversations.touchUpdatedAt(req.params.id);
 
-      // API mode — lightweight headless streaming, no agent spawned
-      if (conv.mode === 'api' || !conv.agentId) {
-        await daemon.conversations.sendMessage(req.params.id, message.trim(), history || [], {
-          reasoningEffort: validatedEffort,
-          verbosity: validatedVerbosity,
-        });
-        daemon.audit.log('conversation.message', { id: req.params.id, mode: 'api' });
-        return res.json({ status: 'streaming', mode: 'api' });
-      }
-
-      // Agent mode — existing behavior
-      const agent = daemon.registry.get(conv.agentId);
-      if (!agent) return res.status(400).json({ error: 'Agent no longer exists' });
-
-      // Record user feedback for journalist context
-      if (daemon.journalist) daemon.journalist.recordUserFeedback(agent, message.trim());
-
-      // Agent loop path — send message directly to the running loop
-      if (daemon.processes.hasAgentLoop(conv.agentId)) {
-        const sent = await daemon.processes.sendMessage(conv.agentId, message.trim());
-        if (sent) {
-          daemon.audit.log('conversation.message', { id: req.params.id, agentId: conv.agentId });
-          return res.json({ id: conv.agentId, status: 'message_sent' });
-        }
-      }
-
-      // One-shot providers: kill and respawn with the message as prompt
-      const provider = getProvider(agent.provider);
-      if (provider?.constructor?.isOneShot) {
-        const oldConfig = { ...agent };
-        if (daemon.processes.isRunning(conv.agentId)) {
-          await daemon.processes.kill(conv.agentId);
-        }
-        daemon.registry.remove(conv.agentId);
-        daemon.locks.release(conv.agentId);
-
-        const newAgent = await daemon.processes.spawn({
-          role: 'chat',
-          scope: oldConfig.scope,
-          provider: oldConfig.provider,
-          model: oldConfig.model,
-          prompt: message.trim(),
-          permission: oldConfig.permission || 'full',
-          workingDir: oldConfig.workingDir,
-          name: oldConfig.name,
-          teamId: oldConfig.teamId,
-        });
-
-        // Update conversation to point to new agent
-        const convObj = daemon.conversations.conversations.get(req.params.id);
-        if (convObj) {
-          convObj.agentId = newAgent.id;
-          daemon.conversations._save();
-        }
-        daemon.audit.log('conversation.message', { id: req.params.id, agentId: newAgent.id, oneShot: true });
-        return res.json({ id: newAgent.id, status: 'respawned' });
-      }
-
-      // Running CLI agent — queue the message
-      if (daemon.processes.isRunning(conv.agentId)) {
-        daemon.processes.queueMessage(conv.agentId, message.trim());
-        daemon.audit.log('conversation.message', { id: req.params.id, agentId: conv.agentId, queued: true });
-        return res.json({ id: conv.agentId, status: 'message_queued' });
-      }
-
-      // CLI agent — session resume or rotation
-      const SESSION_RESUME_CEILING = 5_000_000;
-      const resumed = !!agent.sessionId && (agent.tokensUsed || 0) < SESSION_RESUME_CEILING;
-      const newAgent = resumed
-        ? await daemon.processes.resume(conv.agentId, message.trim())
-        : await daemon.rotator.rotate(conv.agentId, { additionalPrompt: message.trim() });
-
-      // Update conversation to point to new agent if rotated
-      if (newAgent.id !== conv.agentId) {
-        const convObj = daemon.conversations.conversations.get(req.params.id);
-        if (convObj) {
-          convObj.agentId = newAgent.id;
-          daemon.conversations._save();
-        }
-      }
-
-      daemon.audit.log('conversation.message', { id: req.params.id, agentId: newAgent.id, resumed });
-      res.json({ id: newAgent.id, status: resumed ? 'resumed' : 'rotated' });
+      await daemon.conversations.sendMessage(req.params.id, message.trim(), history || [], {
+        reasoningEffort: validatedEffort,
+        verbosity: validatedVerbosity,
+      });
+      daemon.audit.log('conversation.message', { id: req.params.id, mode: conv.mode || 'api' });
+      res.json({ status: 'streaming', mode: conv.mode || 'api' });
     } catch (err) {
       res.status(400).json({ error: err.message });
     }