groove-dev 0.27.196 → 0.27.197

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

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

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

@@ -28,6 +28,7 @@ import { registerFileRoutes, resetEditorRoot } from './routes/files.js';
 import { registerNetworkRoutes } from './routes/network.js';
 import { registerScheduleRoutes } from './routes/schedules.js';
 import { registerInnerChatRoutes } from './routes/innerchat.js';
+import { registerChatHistoryRoutes } from './routes/chat-history.js';
 import { registerWatchRoutes } from './routes/watch.js';
 import { registerAutoAgentRoutes } from './routes/auto-agents.js';
 
@@ -179,6 +180,7 @@ export function createApi(app, daemon) {
   registerNetworkRoutes(app, daemon);
   registerScheduleRoutes(app, daemon);
   registerInnerChatRoutes(app, daemon);
+  registerChatHistoryRoutes(app, daemon);
   registerWatchRoutes(app, daemon);
   registerAutoAgentRoutes(app, daemon);
 
@@ -250,8 +252,6 @@ export function createApi(app, daemon) {
     }
   });
 
-  // --- Conversations ---
-
   // --- Approvals ---
 
   app.get('/api/approvals', (req, res) => {
@@ -1348,10 +1348,8 @@ 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);
@@ -1439,7 +1437,7 @@ Keep responses concise. Help them think, don't lecture them about the system the
     const ALLOWED_KEYS = [
       'port', 'journalistInterval', 'rotationThreshold', 'autoRotation',
       'qcThreshold', 'maxAgents', 'defaultProvider', 'defaultWorkingDir',
-      'onboardingDismissed', 'defaultModel', 'defaultChatProvider', 'defaultChatModel',
+      'onboardingDismissed', 'defaultModel',
       'dataSharingDismissed', 'replayCeiling', 'velocityCeiling', 'journalistModelTier',
       'resumeBudgetChars',
     ];

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

@@ -0,0 +1,104 @@
+// FSL-1.1-Apache-2.0 — see LICENSE
+
+import { resolve } from 'path';
+import { existsSync, readFileSync, writeFileSync, renameSync } from 'fs';
+
+// Per-agent history cap and attachment handling mirror the GUI's own limits so
+// the two stay in step.
+const MAX_PER_AGENT = 200;
+const SAVE_DEBOUNCE_MS = 1500;
+
+/**
+ * Server-side chat history.
+ *
+ * The GUI used to keep chat history only in the browser's localStorage, which
+ * is scoped per origin (scheme://host:PORT). For a remote GUI reached over an
+ * SSH tunnel the local port changes across reconnects, so the origin — and thus
+ * the entire chat store — changed out from under the user, stranding history on
+ * old ports. Keeping it on the daemon makes it independent of port, origin, and
+ * even which machine connects: reconnect from anywhere and the chats are there.
+ */
+export class ChatStore {
+  constructor(daemon) {
+    this.daemon = daemon;
+    this.path = resolve(daemon.grooveDir, 'chat-history.json');
+    this.history = this._load();
+    this._saveTimer = null;
+  }
+
+  _load() {
+    try {
+      if (existsSync(this.path)) {
+        const data = JSON.parse(readFileSync(this.path, 'utf8'));
+        if (data && typeof data === 'object') return data;
+      }
+    } catch { /* corrupt or missing — start empty rather than throw */ }
+    return {};
+  }
+
+  _scheduleSave() {
+    if (this._saveTimer) return;
+    this._saveTimer = setTimeout(() => {
+      this._saveTimer = null;
+      this._saveNow();
+    }, SAVE_DEBOUNCE_MS);
+    if (this._saveTimer.unref) this._saveTimer.unref();
+  }
+
+  _saveNow() {
+    try {
+      const tmp = `${this.path}.tmp`;
+      writeFileSync(tmp, JSON.stringify(this.history), { mode: 0o600 });
+      renameSync(tmp, this.path); // atomic — a reader never sees a half-written file
+    } catch { /* best effort */ }
+  }
+
+  // Strip base64 attachment payloads — metadata only. Keeping data URLs would
+  // bloat the file the same way it bloated localStorage.
+  _clean(message) {
+    if (!message || typeof message !== 'object') return null;
+    const out = { ...message };
+    if (Array.isArray(out.attachments) && out.attachments.length) {
+      out.attachments = out.attachments.map(({ dataUrl, ...rest }) => rest);
+    }
+    return out;
+  }
+
+  append(agentId, message) {
+    if (!agentId) return;
+    const clean = this._clean(message);
+    if (!clean) return;
+    const arr = this.history[agentId] || [];
+    arr.push(clean);
+    this.history[agentId] = arr.slice(-MAX_PER_AGENT);
+    this._scheduleSave();
+  }
+
+  // Replace an agent's whole history — used when the GUI syncs a batch (e.g.
+  // messages it recorded while briefly disconnected).
+  replace(agentId, messages) {
+    if (!agentId || !Array.isArray(messages)) return;
+    this.history[agentId] = messages.map((m) => this._clean(m)).filter(Boolean).slice(-MAX_PER_AGENT);
+    this._scheduleSave();
+  }
+
+  getAll() {
+    return this.history;
+  }
+
+  get(agentId) {
+    return this.history[agentId] || [];
+  }
+
+  remove(agentId) {
+    if (this.history[agentId]) {
+      delete this.history[agentId];
+      this._scheduleSave();
+    }
+  }
+
+  stop() {
+    if (this._saveTimer) { clearTimeout(this._saveTimer); this._saveTimer = null; }
+    this._saveNow();
+  }
+}

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

@@ -44,9 +44,9 @@ import { ModelLab } from './model-lab.js';
 import { LlamaServerManager } from './llama-server.js';
 import { MLXServerManager } from './mlx-server.js';
 import { RepoImporter } from './repo-import.js';
-import { ConversationManager } from './conversations.js';
 import { Toys } from './toys.js';
 import { InnerChat } from './innerchat.js';
+import { ChatStore } from './chatstore.js';
 import { Watcher } from './watcher.js';
 import { AutoState } from './autostate.js';
 import { Orchestrator } from './orchestrator.js';
@@ -139,7 +139,6 @@ export class Daemon {
     this.rotator = new Rotator(this);
     this.adaptive = new AdaptiveThresholds(this.grooveDir);
     this.teams = new Teams(this);
-    this.conversations = new ConversationManager(this);
     this.credentials = new CredentialStore(this.grooveDir);
     this.classifier = new TaskClassifier();
     this.router = new ModelRouter(this);
@@ -163,6 +162,7 @@ export class Daemon {
     this.modelLab = new ModelLab(this);
     this.toys = new Toys(this);
     this.innerchat = new InnerChat(this);
+    this.chatStore = new ChatStore(this);
     this.watcher = new Watcher(this);
     this.autoState = new AutoState(this.grooveDir);
     this.orchestrator = new Orchestrator(this);
@@ -874,6 +874,7 @@ export class Daemon {
     this.scheduler.stop();
     this.watcher.stop();
     this.innerchat.stop();
+    this.chatStore.stop();
     this.orchestrator.stop();
     this.timeline.stop();
     if (this._gcInterval) clearInterval(this._gcInterval);

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

@@ -71,7 +71,7 @@ Do NOT write code unless explicitly asked. Use your MCP tools (database queries,
 - Adapting tone and style to the audience (formal, conversational, technical, marketing)
 - Editing and polishing drafts for grammar, flow, and impact
 - Researching topics to produce accurate, substantive writing
-You CAN use code tools to create and edit text files, markdown documents, and structured content. For best results, apply a writing skill from the Marketplace that matches your task.
+You CAN use code tools to create and edit text files, markdown documents, and structured content. For best results, apply a writing skill from the Integrations tab that matches your task.
 
 `,
   chat: `You are a Chat agent — a conversational companion, friend, and assistant. You are warm, curious, and genuinely engaged. You can discuss anything: ideas, philosophy, science, culture, coding, life.

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

@@ -2,9 +2,8 @@
 
 import { resolve } from 'path';
 import { existsSync, readFileSync, readdirSync, statSync, writeFileSync, mkdirSync } from 'fs';
-import { validateAgentConfig, validateReasoningEffort, validateVerbosity } from '../validate.js';
+import { validateAgentConfig } from '../validate.js';
 import { ROLE_INTEGRATIONS, wrapWithRoleReminder } from '../process.js';
-import { getProvider } from '../providers/index.js';
 import { deliverInstruction } from '../deliver.js';
 import { renameAgent } from '../rename.js';
 
@@ -218,179 +217,6 @@ export function registerAgentRoutes(app, daemon) {
     res.json(suggestion);
   });
 
-  // --- Conversations ---
-
-  app.get('/api/conversations', (req, res) => {
-    res.json({ conversations: daemon.conversations.list() });
-  });
-
-  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 must be a string' });
-      }
-      if (mode && mode !== 'api' && mode !== 'agent') {
-        return res.status(400).json({ error: 'mode must be "api" or "agent"' });
-      }
-      const validatedEffort = validateReasoningEffort(reasoning_effort);
-      const validatedVerbosity = validateVerbosity(verbosity);
-      const conversation = await daemon.conversations.create(provider, model, title, mode || 'api', {
-        reasoningEffort: validatedEffort,
-        verbosity: validatedVerbosity,
-      });
-      daemon.audit.log('conversation.create', { id: conversation.id, provider, model, mode: conversation.mode });
-      res.status(201).json(conversation);
-    } catch (err) {
-      res.status(400).json({ error: err.message });
-    }
-  });
-
-  app.get('/api/conversations/:id', (req, res) => {
-    const conversation = daemon.conversations.get(req.params.id);
-    if (!conversation) return res.status(404).json({ error: 'Conversation not found' });
-    res.json(conversation);
-  });
-
-  app.patch('/api/conversations/:id', async (req, res) => {
-    try {
-      const conv = daemon.conversations.get(req.params.id);
-      if (!conv) return res.status(404).json({ error: 'Conversation not found' });
-      if (req.body.title !== undefined) daemon.conversations.rename(req.params.id, req.body.title);
-      if (req.body.pinned !== undefined) daemon.conversations.pin(req.params.id, req.body.pinned);
-      if (req.body.archived !== undefined) daemon.conversations.archive(req.params.id, req.body.archived);
-      if (req.body.model !== undefined || req.body.provider !== undefined) {
-        const newProvider = req.body.provider || conv.provider;
-        const newModel = req.body.model || conv.model;
-        daemon.conversations.updateModel(req.params.id, newProvider, newModel);
-      }
-      if (req.body.mode !== undefined) {
-        if (req.body.mode !== 'api' && req.body.mode !== 'agent') {
-          return res.status(400).json({ error: 'mode must be "api" or "agent"' });
-        }
-        await daemon.conversations.setMode(req.params.id, req.body.mode);
-      }
-      if (req.body.reasoning_effort !== undefined || req.body.verbosity !== undefined) {
-        const validatedEffort = req.body.reasoning_effort !== undefined ? validateReasoningEffort(req.body.reasoning_effort) : undefined;
-        const validatedVerbosity = req.body.verbosity !== undefined ? validateVerbosity(req.body.verbosity) : undefined;
-        daemon.conversations.updateReasoningSettings(req.params.id, validatedEffort, validatedVerbosity);
-      }
-      daemon.audit.log('conversation.update', { id: req.params.id, provider: req.body.provider, model: req.body.model, mode: req.body.mode });
-      res.json(daemon.conversations.get(req.params.id));
-    } catch (err) {
-      res.status(400).json({ error: err.message });
-    }
-  });
-
-  app.delete('/api/conversations/:id', async (req, res) => {
-    try {
-      const conv = daemon.conversations.get(req.params.id);
-      if (!conv) return res.status(404).json({ error: 'Conversation not found' });
-      await daemon.conversations.delete(req.params.id);
-      daemon.audit.log('conversation.delete', { id: req.params.id });
-      res.json({ ok: true });
-    } catch (err) {
-      res.status(400).json({ error: err.message });
-    }
-  });
-
-  app.post('/api/conversations/:id/message', async (req, res) => {
-    try {
-      const { message, history, reasoning_effort, verbosity } = req.body;
-      if (!message || typeof message !== 'string' || !message.trim()) {
-        return res.status(400).json({ error: 'message is required' });
-      }
-      const validatedEffort = validateReasoningEffort(reasoning_effort);
-      const validatedVerbosity = validateVerbosity(verbosity);
-
-      const conv = daemon.conversations.get(req.params.id);
-      if (!conv) return res.status(404).json({ error: 'Conversation not found' });
-
-      daemon.conversations.autoTitle(req.params.id, message.trim());
-      daemon.conversations.touchUpdatedAt(req.params.id);
-
-      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 });
-    }
-  });
-
-  app.post('/api/conversations/:id/stop', (req, res) => {
-    try {
-      const conv = daemon.conversations.get(req.params.id);
-      if (!conv) return res.status(404).json({ error: 'Conversation not found' });
-      daemon.conversations.stopStreaming(req.params.id);
-      res.json({ ok: true });
-    } catch (err) {
-      res.status(400).json({ error: err.message });
-    }
-  });
-
-  // --- Image Generation ---
-
-  app.post('/api/conversations/:id/generate-image', async (req, res) => {
-    try {
-      const { prompt, model, size, quality } = req.body;
-      if (!prompt || typeof prompt !== 'string' || !prompt.trim()) {
-        return res.status(400).json({ error: 'prompt is required' });
-      }
-      const conv = daemon.conversations.get(req.params.id);
-      if (!conv) return res.status(404).json({ error: 'Conversation not found' });
-
-      let providerName = conv.provider;
-      let provider = getProvider(providerName);
-
-      // If a specific image model was requested, find the right provider
-      if (model) {
-        const imageProviders = ['codex', 'grok', 'nano-banana'];
-        for (const pid of imageProviders) {
-          const p = getProvider(pid);
-          if (p?.constructor.models.some((m) => m.id === model)) {
-            provider = p;
-            providerName = pid;
-            break;
-          }
-        }
-      }
-
-      if (!provider?.generateImage) {
-        return res.status(400).json({ error: 'Provider does not support image generation' });
-      }
-
-      const apiKey = daemon.conversations._getApiKey(providerName);
-      if (!apiKey) {
-        return res.status(400).json({ error: `No API key configured for ${providerName}` });
-      }
-
-      daemon.broadcast({
-        type: 'conversation:image-progress',
-        data: { conversationId: req.params.id, status: 'generating', prompt: prompt.trim() },
-      });
-
-      const result = await provider.generateImage(prompt.trim(), { model, size, quality, apiKey });
-
-      daemon.broadcast({
-        type: 'conversation:image',
-        data: { conversationId: req.params.id, ...result, prompt: prompt.trim() },
-      });
-
-      daemon.conversations.touchUpdatedAt(req.params.id);
-      daemon.audit.log('conversation.image', { id: req.params.id, model: result.model, provider: result.provider });
-      res.json(result);
-    } catch (err) {
-      daemon.broadcast({
-        type: 'conversation:image-progress',
-        data: { conversationId: req.params.id, status: 'error', error: err.message },
-      });
-      res.status(500).json({ error: err.message });
-    }
-  });
-
   // Stop an agent's current work without killing the agent
   app.post('/api/agents/:id/stop', async (req, res) => {
     try {

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

@@ -0,0 +1,34 @@
+// FSL-1.1-Apache-2.0 — see LICENSE
+
+export function registerChatHistoryRoutes(app, daemon) {
+  // Full history for all agents — the GUI loads this on connect so chats are
+  // present regardless of which origin/port the tunnel came up on.
+  app.get('/api/chat-history', (req, res) => {
+    res.json({ history: daemon.chatStore.getAll() });
+  });
+
+  // Append a single message for an agent.
+  app.post('/api/chat-history/:agentId', (req, res) => {
+    const { message } = req.body || {};
+    if (!message || typeof message !== 'object') {
+      return res.status(400).json({ error: 'message object required' });
+    }
+    daemon.chatStore.append(req.params.agentId, message);
+    res.json({ ok: true });
+  });
+
+  // Replace an agent's whole history (batch sync).
+  app.put('/api/chat-history/:agentId', (req, res) => {
+    const { messages } = req.body || {};
+    if (!Array.isArray(messages)) {
+      return res.status(400).json({ error: 'messages array required' });
+    }
+    daemon.chatStore.replace(req.params.agentId, messages);
+    res.json({ ok: true });
+  });
+
+  app.delete('/api/chat-history/:agentId', (req, res) => {
+    daemon.chatStore.remove(req.params.agentId);
+    res.json({ ok: true });
+  });
+}

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

@@ -0,0 +1,86 @@
+// GROOVE — ChatStore tests
+// FSL-1.1-Apache-2.0 — see LICENSE
+
+import { describe, it, beforeEach, afterEach } from 'node:test';
+import assert from 'node:assert/strict';
+import { mkdtempSync, rmSync, existsSync, readFileSync } from 'fs';
+import { tmpdir } from 'os';
+import { resolve } from 'path';
+import { ChatStore } from '../src/chatstore.js';
+
+describe('ChatStore', () => {
+  let dir, store;
+
+  beforeEach(() => {
+    dir = mkdtempSync(resolve(tmpdir(), 'groove-chat-'));
+    store = new ChatStore({ grooveDir: dir });
+  });
+  afterEach(() => {
+    store.stop();
+    try { rmSync(dir, { recursive: true, force: true }); } catch { /* ignore */ }
+  });
+
+  it('appends and returns per-agent history', () => {
+    store.append('a1', { from: 'user', text: 'hello', timestamp: 1 });
+    store.append('a1', { from: 'agent', text: 'hi', timestamp: 2 });
+    store.append('a2', { from: 'user', text: 'other', timestamp: 3 });
+
+    assert.equal(store.get('a1').length, 2);
+    assert.equal(store.get('a2').length, 1);
+    assert.deepEqual(store.getAll().a1.map((m) => m.text), ['hello', 'hi']);
+  });
+
+  it('strips attachment data URLs but keeps metadata', () => {
+    store.append('a1', {
+      from: 'user', text: 'pic', timestamp: 1,
+      attachments: [{ name: 'x.png', size: 9, dataUrl: 'data:image/png;base64,AAAA' }],
+    });
+    const m = store.get('a1')[0];
+    assert.equal(m.attachments[0].dataUrl, undefined);
+    assert.equal(m.attachments[0].name, 'x.png');
+  });
+
+  it('persists across instances (survives a daemon restart)', () => {
+    store.append('a1', { from: 'user', text: 'durable', timestamp: 1 });
+    store.stop(); // flushes synchronously
+
+    const reloaded = new ChatStore({ grooveDir: dir });
+    assert.equal(reloaded.get('a1')[0].text, 'durable');
+    reloaded.stop();
+  });
+
+  it('writes the file atomically (no stray .tmp left)', () => {
+    store.append('a1', { from: 'user', text: 'x', timestamp: 1 });
+    store.stop();
+    assert.ok(existsSync(resolve(dir, 'chat-history.json')));
+    assert.ok(!existsSync(resolve(dir, 'chat-history.json.tmp')));
+    const onDisk = JSON.parse(readFileSync(resolve(dir, 'chat-history.json'), 'utf8'));
+    assert.equal(onDisk.a1[0].text, 'x');
+  });
+
+  it('caps per-agent history', () => {
+    for (let i = 0; i < 250; i++) store.append('a1', { from: 'user', text: `m${i}`, timestamp: i });
+    const h = store.get('a1');
+    assert.equal(h.length, 200);
+    assert.equal(h[0].text, 'm50');       // oldest 50 dropped
+    assert.equal(h.at(-1).text, 'm249');
+  });
+
+  it('replace() swaps an agent history wholesale', () => {
+    store.append('a1', { from: 'user', text: 'old' });
+    store.replace('a1', [{ from: 'user', text: 'new1' }, { from: 'agent', text: 'new2' }]);
+    assert.deepEqual(store.get('a1').map((m) => m.text), ['new1', 'new2']);
+  });
+
+  it('remove() deletes an agent history', () => {
+    store.append('a1', { from: 'user', text: 'x' });
+    store.remove('a1');
+    assert.equal(store.get('a1').length, 0);
+  });
+
+  it('starts empty when the file is missing or corrupt', () => {
+    const empty = new ChatStore({ grooveDir: mkdtempSync(resolve(tmpdir(), 'groove-chat-empty-')) });
+    assert.deepEqual(empty.getAll(), {});
+    empty.stop();
+  });
+});