groove-dev 0.27.153 → 0.27.154

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

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

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

@@ -644,20 +644,14 @@ export class Journalist {
         proc.stdin.write(stdinData);
         proc.stdin.end();
         proc.stdout.on('data', (d) => { stdout += d.toString(); });
-        const timer = setTimeout(() => { proc.kill(); reject(new Error('Headless timeout')); }, 60_000);
+        const timer = setTimeout(() => { proc.kill(); reject(new Error('Headless timeout')); }, 120_000);
         proc.on('exit', (code) => {
           clearTimeout(timer);
           if (code !== 0) return reject(new Error(`Headless exited with code ${code}`));
           this._recordHeadlessUsage(stdout, trackAs, modelId);
-          const lines = stdout.split('\n');
-          for (const line of lines) {
-            try {
-              const json = JSON.parse(line);
-              if (json.result) return resolve(json.result);
-              if (json.content?.[0]?.text) return resolve(json.content[0].text);
-            } catch { /* not json */ }
-          }
-          resolve(stdout.trim());
+          const extracted = this._parseHeadlessOutput(stdout);
+          if (extracted) return resolve(extracted);
+          reject(new Error('Headless produced no usable output'));
         });
         return;
       }
@@ -666,24 +660,61 @@ export class Journalist {
         env: { ...process.env, ...env },
         cwd: this.daemon.projectDir,
         maxBuffer: 1024 * 1024 * 5,
-        timeout: 60_000,
+        timeout: 120_000,
       }, (err, stdout, stderr) => {
         if (err) return reject(err);
         this._recordHeadlessUsage(stdout, trackAs, modelId);
-        const lines = stdout.split('\n');
-        for (const line of lines) {
-          try {
-            const data = JSON.parse(line);
-            if (data.type === 'result' && data.result) {
-              return resolve(data.result);
-            }
-          } catch { /* skip */ }
-        }
-        resolve(stdout);
+        const extracted = this._parseHeadlessOutput(stdout);
+        if (extracted) return resolve(extracted);
+        reject(new Error('Headless produced no usable output'));
       });
     });
   }
 
+  _parseHeadlessOutput(stdout) {
+    const lines = stdout.split('\n');
+    let resultText = '';
+    let assistantText = '';
+    let codexText = '';
+    let networkText = '';
+
+    for (const line of lines) {
+      try {
+        const json = JSON.parse(line);
+
+        // Claude/Gemini stream-json: {"type":"result","result":"..."}
+        if (typeof json.result === 'string' && json.result.trim()) {
+          resultText = json.result;
+        }
+
+        // Claude/Gemini assistant message: {"type":"assistant","message":{"content":[{"type":"text","text":"..."}]}}
+        const msgContent = json.message?.content?.[0]?.text || json.content?.[0]?.text;
+        if (msgContent && msgContent.trim()) {
+          assistantText = msgContent;
+        }
+
+        // Codex --json: {"type":"item.completed","item":{"type":"agent_message","text":"..."}}
+        if (json.type === 'item.completed' && json.item?.type === 'agent_message' && json.item.text?.trim()) {
+          codexText = json.item.text;
+        }
+
+        // Groove Network: {"type":"done|complete|result","text":"..."}
+        if ((json.type === 'done' || json.type === 'complete') && typeof json.text === 'string' && json.text.trim()) {
+          networkText = json.text;
+        }
+      } catch { /* not json */ }
+    }
+
+    if (resultText) return resultText;
+    if (codexText) return codexText;
+    if (networkText) return networkText;
+    if (assistantText) return assistantText;
+    // Ollama / plain-text providers: raw text output (no JSON)
+    const plain = stdout.trim();
+    if (plain && !plain.startsWith('{') && plain.length > 20) return plain;
+    return null;
+  }
+
   parseSynthesisResult(text, agents) {
     // Parse the structured output from AI
     const sections = {

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

@@ -58,7 +58,7 @@ export class Keeper {
 
   save(tag, content) {
     if (!tag || typeof tag !== 'string') throw new Error('Tag is required');
-    if (content === undefined || content === null) throw new Error('Content is required');
+    if (content === undefined || content === null || !String(content).trim()) throw new Error('Content is required');
     const normalized = this._normalize(tag);
     if (!normalized) throw new Error('Tag is required');
     const filePath = this._tagToPath(normalized);
@@ -129,7 +129,7 @@ export class Keeper {
   update(tag, content) {
     const normalized = this._normalize(tag);
     if (!normalized) throw new Error('Tag is required');
-    if (content === undefined || content === null) throw new Error('Content is required');
+    if (content === undefined || content === null || !String(content).trim()) throw new Error('Content is required');
     const filePath = this._tagToPath(normalized);
     if (!existsSync(filePath)) throw new Error(`Memory #${normalized} does not exist`);
     writeFileSync(filePath, String(content));
@@ -153,6 +153,41 @@ export class Keeper {
     return true;
   }
 
+  move(oldTag, newTag) {
+    const oldNorm = this._normalize(oldTag);
+    const newNorm = this._normalize(newTag);
+    if (!oldNorm || !newNorm) throw new Error('Both old and new tags are required');
+    if (oldNorm === newNorm) return this._index[oldNorm];
+    const oldPath = this._tagToPath(oldNorm);
+    if (!existsSync(oldPath)) throw new Error(`Memory #${oldNorm} does not exist`);
+    if (this._index[newNorm]) throw new Error(`Memory #${newNorm} already exists`);
+    const content = readFileSync(oldPath, 'utf8');
+    const newPath = this._tagToPath(newNorm);
+    this._ensureParentDir(newPath);
+    writeFileSync(newPath, content);
+    unlinkSync(oldPath);
+    this._index[newNorm] = { ...this._index[oldNorm], tag: newNorm, updatedAt: new Date().toISOString() };
+    delete this._index[oldNorm];
+    // Move children too (e.g. moving "a" to "b/a" also moves "a/child" to "b/a/child")
+    const prefix = oldNorm + '/';
+    for (const tag of Object.keys(this._index)) {
+      if (tag.startsWith(prefix)) {
+        const childSuffix = tag.slice(prefix.length);
+        const childNewTag = newNorm + '/' + childSuffix;
+        const childOldPath = this._tagToPath(tag);
+        const childNewPath = this._tagToPath(childNewTag);
+        const childContent = readFileSync(childOldPath, 'utf8');
+        this._ensureParentDir(childNewPath);
+        writeFileSync(childNewPath, childContent);
+        unlinkSync(childOldPath);
+        this._index[childNewTag] = { ...this._index[tag], tag: childNewTag, updatedAt: new Date().toISOString() };
+        delete this._index[tag];
+      }
+    }
+    this._saveIndex();
+    return { tag: newNorm, ...this._index[newNorm] };
+  }
+
   // ── Doc (AI-generated) ───────────────────────────────────
 
   saveDoc(tag, content) {

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

@@ -249,6 +249,19 @@ export function registerCoordinationRoutes(app, daemon) {
     }
   });
 
+  app.post('/api/keeper/move', (req, res) => {
+    try {
+      const { oldTag, newTag } = req.body || {};
+      if (!oldTag || !newTag) return res.status(400).json({ error: 'oldTag and newTag are required' });
+      const item = daemon.keeper.move(oldTag, newTag);
+      daemon.audit.log('keeper.move', { oldTag, newTag: item.tag });
+      daemon.broadcast({ type: 'keeper:moved', oldTag, item });
+      res.json(item);
+    } catch (err) {
+      res.status(err.message.includes('does not exist') ? 404 : 400).json({ error: err.message });
+    }
+  });
+
   app.delete('/api/keeper/link/:tag(*)', (req, res) => {
     try {
       const { docPath } = req.body || {};
@@ -289,6 +302,9 @@ export function registerCoordinationRoutes(app, daemon) {
       } else {
         doc = `# ${tag}\n\n*Auto-generated document from conversation*\n\n${transcript.slice(0, 5000)}`;
       }
+      if (!doc || !doc.trim()) {
+        return res.status(502).json({ error: 'AI synthesis returned empty content — try again' });
+      }
       const item = daemon.keeper.saveDoc(tag, doc);
       daemon.audit.log('keeper.doc', { tag: item.tag, agentId });
       daemon.broadcast({ type: 'keeper:saved', item });

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

@@ -1,5 +1,5 @@
 // FSL-1.1-Apache-2.0 — see LICENSE
-import { resolve, sep, isAbsolute } from 'path';
+import { resolve, sep, isAbsolute, basename } from 'path';
 import { existsSync, readFileSync, readdirSync, statSync, writeFileSync, mkdirSync, unlinkSync, renameSync, rmSync, createReadStream, realpathSync } from 'fs';
 import { execFile, execFileSync } from 'child_process';
 import { homedir } from 'os';
@@ -331,6 +331,53 @@ export function registerFileRoutes(app, daemon) {
     }
   });
 
+  // Download a file (serves raw with Content-Disposition)
+  app.get('/api/files/download', (req, res) => {
+    const relPath = req.query.path;
+    const result = validateFilePath(relPath, getEditorRoot(daemon));
+    if (result.error) return res.status(400).json({ error: result.error });
+    if (!existsSync(result.fullPath)) return res.status(404).json({ error: 'File not found' });
+
+    const stat = statSync(result.fullPath);
+    if (stat.isDirectory()) return res.status(400).json({ error: 'Cannot download a directory' });
+
+    const name = basename(result.fullPath);
+    const mime = mimeLookup(name) || 'application/octet-stream';
+    res.setHeader('Content-Disposition', `attachment; filename="${encodeURIComponent(name)}"`);
+    res.setHeader('Content-Type', mime);
+    res.setHeader('Content-Length', stat.size);
+    createReadStream(result.fullPath).pipe(res);
+  });
+
+  // Upload files (base64-encoded) to a target directory
+  app.post('/api/files/upload', (req, res) => {
+    const { dir = '', files } = req.body;
+    const rootDir = getEditorRoot(daemon);
+    if (!rootDir) return res.status(400).json({ error: 'Editor root not set' });
+    if (!Array.isArray(files) || files.length === 0) return res.status(400).json({ error: 'files[] required' });
+    if (files.length > 50) return res.status(400).json({ error: 'Max 50 files per upload' });
+
+    const uploaded = [];
+    for (const file of files) {
+      if (!file.name || !file.content) continue;
+      const safeName = String(file.name).replace(/\.\./g, '').replace(/\//g, '_');
+      if (!safeName) continue;
+      const relPath = dir ? `${dir}/${safeName}` : safeName;
+      const result = validateFilePath(relPath, rootDir);
+      if (result.error) continue;
+
+      try {
+        const parentDir = resolve(result.fullPath, '..');
+        mkdirSync(parentDir, { recursive: true });
+        const buf = Buffer.from(file.content, 'base64');
+        writeFileSync(result.fullPath, buf);
+        daemon.audit.log('file.upload', { path: relPath, size: buf.length });
+        uploaded.push({ path: relPath, size: buf.length });
+      } catch { /* skip failed files */ }
+    }
+    res.json({ uploaded, total: uploaded.length });
+  });
+
   // Create a new file
   app.post('/api/files/create', (req, res) => {
     const { path: relPath, content = '' } = req.body;
@@ -595,9 +642,31 @@ export function registerFileRoutes(app, daemon) {
     if (!agent) return res.status(404).json({ error: 'Agent not found' });
     const rawFiles = daemon.registry.getFilesTouched(req.params.id);
     const rootDir = agent.workingDir || daemon.projectDir;
+
+    // Build git diff numstat for line-level +/- counts
+    let numstatMap = {};
+    const writtenPaths = rawFiles.filter(f => f.writes > 0).map(f => f.path);
+    if (writtenPaths.length > 0) {
+      try {
+        const out = execFileSync('git', ['diff', '--numstat', '--', ...writtenPaths], {
+          cwd: rootDir, timeout: 10000, maxBuffer: 2 * 1024 * 1024,
+        }).toString();
+        for (const line of out.split('\n')) {
+          const m = line.match(/^(\d+|-)\t(\d+|-)\t(.+)$/);
+          if (m) {
+            numstatMap[m[3]] = {
+              additions: m[1] === '-' ? 0 : Number(m[1]),
+              deletions: m[2] === '-' ? 0 : Number(m[2]),
+            };
+          }
+        }
+      } catch { /* git not available or not a repo */ }
+    }
+
     const files = rawFiles.map(f => {
       const fullPath = isAbsolute(f.path) ? f.path : resolve(rootDir, f.path);
-      return { ...f, exists: existsSync(fullPath) };
+      const stats = numstatMap[f.path] || null;
+      return { ...f, exists: existsSync(fullPath), additions: stats?.additions ?? null, deletions: stats?.deletions ?? null };
     });
     res.json({ files, total: files.length });
   });