groove-dev 0.27.141 → 0.27.143

package/CLAUDE.md

@@ -295,3 +295,10 @@ 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: 0
+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/node_modules/@groove-dev/cli/package.json

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

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

@@ -460,13 +460,14 @@ export function createApi(app, daemon) {
   // Discoveries (error → fix pairs)
   app.get('/api/memory/discoveries', (req, res) => {
     const role = req.query.role;
+    const teamId = req.query.teamId;
     const limit = Math.min(parseInt(req.query.limit) || 100, 500);
-    res.json({ discoveries: daemon.memory.listDiscoveries({ role, limit }) });
+    res.json({ discoveries: daemon.memory.listDiscoveries({ role, teamId, limit }) });
   });
 
   app.post('/api/memory/discoveries', (req, res) => {
-    const { agentId, role, trigger, fix, outcome } = req.body || {};
-    const result = daemon.memory.addDiscovery({ agentId, role, trigger, fix, outcome });
+    const { agentId, role, trigger, fix, outcome, teamId } = req.body || {};
+    const result = daemon.memory.addDiscovery({ agentId, role, trigger, fix, outcome, teamId });
     if (!result.added && result.error) {
       return res.status(400).json(result);
     }
@@ -3171,11 +3172,21 @@ Keep responses concise. Help them think, don't lecture them about the system the
 
   function validateFilePath(relPath, projectDir) {
     if (!relPath || typeof relPath !== 'string') return { error: 'path is required' };
-    if (relPath.startsWith('/') || relPath.includes('..') || relPath.includes('\0')) {
-      return { error: 'Invalid path' };
+    if (relPath.includes('\0')) return { error: 'Invalid path' };
+
+    let fullPath;
+    if (relPath.startsWith('/')) {
+      if (relPath.includes('..')) return { error: 'Invalid path' };
+      if (!relPath.startsWith(projectDir + '/') && relPath !== projectDir) {
+        return { error: 'Path outside project' };
+      }
+      fullPath = relPath;
+    } else {
+      if (relPath.includes('..')) return { error: 'Invalid path' };
+      fullPath = resolve(projectDir, relPath);
+      if (!fullPath.startsWith(projectDir)) return { error: 'Path outside project' };
     }
-    const fullPath = resolve(projectDir, relPath);
-    if (!fullPath.startsWith(projectDir)) return { error: 'Path outside project' };
+
     // Symlink resolution — ensure real path is also within project
     try {
       const realPath = realpathSync(fullPath);

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

@@ -437,7 +437,7 @@ export class Introducer {
         }
 
         if (!isLightPlanner && (hasTask || isRotation)) {
-          const discoveries = this.daemon.memory.getDiscoveriesMarkdown(newAgent.role, 8, 600, newAgent.scope);
+          const discoveries = this.daemon.memory.getDiscoveriesMarkdown(newAgent.role, 8, 600, newAgent.scope, newAgent.teamId);
           if (discoveries) {
             parts.push(`### Known Fixes for ${newAgent.role} Role\n${discoveries}`);
           }

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

@@ -877,7 +877,7 @@ export class Journalist {
     const entries = agentLog?.entries || [];
 
     // Layer 7 memory: discoveries (inline, not pointer — agents lose context with pointers), constraints, specializations
-    const discoveries = this.daemon.memory?.getDiscoveriesMarkdown(agent.role, 10, 1500) || '';
+    const discoveries = this.daemon.memory?.getDiscoveriesMarkdown(agent.role, 10, 1500, agent.scope, agent.teamId) || '';
     const constraints = this.daemon.memory?.getConstraintsMarkdown(2000) || '';
     const specialization = this.daemon.memory?.getSpecialization(agent.id);
     const specLine = specialization?.avgQualityScore != null
@@ -1106,7 +1106,7 @@ export class Journalist {
     if (!thread) return null;
 
     const constraints = this.daemon.memory?.getConstraintsMarkdown(2000) || '';
-    const discoveries = this.daemon.memory?.getDiscoveriesMarkdown(agent.role, 5, 1000) || '';
+    const discoveries = this.daemon.memory?.getDiscoveriesMarkdown(agent.role, 5, 1000, agent.scope, agent.teamId) || '';
 
     let prompt = [
       `# Session Context Resume`,
@@ -1352,6 +1352,7 @@ export class Journalist {
             this.daemon.memory.addDiscovery({
               agentId: agent.id,
               role: agent.role,
+              teamId: agent.teamId || null,
               trigger: trigger.slice(0, 300),
               fix: fix.slice(0, 500),
               outcome: 'success',

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

@@ -254,9 +254,9 @@ export class Keeper {
   // ── Command parser ────────────────────────────────────────
 
   static parseCommand(text) {
-    const cmdMatch = text.match(/\[(save|append|update|delete|view|doc|link|read|instruct)\]|\b(save|append|update|delete|view|doc|link|read)\b(?=\s+#[\w/.-])/i);
+    const cmdMatch = text.match(/\[(save|append|update|delete|view|doc|link|read|instruct)\]/i);
     if (!cmdMatch) return null;
-    const command = (cmdMatch[1] || cmdMatch[2]).toLowerCase();
+    const command = cmdMatch[1].toLowerCase();
     const rest = text.slice(cmdMatch.index + cmdMatch[0].length).trim();
 
     if (command === 'instruct') {

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

@@ -261,7 +261,7 @@ export class MemoryStore {
 
   // --- Discoveries (error → fix pairs) ---
 
-  addDiscovery({ agentId, role, trigger, fix, outcome = 'success' }) {
+  addDiscovery({ agentId, role, trigger, fix, outcome = 'success', teamId }) {
     if (!trigger || !fix) return { added: false, error: 'trigger and fix required' };
     if (outcome !== 'success') return { added: false, reason: 'only successes stored' };
 
@@ -277,6 +277,7 @@ export class MemoryStore {
       ts: new Date().toISOString(),
       agentId: agentId || null,
       role: role || 'unknown',
+      teamId: teamId || null,
       trigger: truncate(String(trigger).trim(), 300),
       fix: truncate(String(fix).trim(), 500),
       outcome,
@@ -298,7 +299,7 @@ export class MemoryStore {
     }
   }
 
-  listDiscoveries({ role, limit = 100 } = {}) {
+  listDiscoveries({ role, teamId, limit = 100 } = {}) {
     if (!existsSync(this.discoveriesPath)) return [];
     try {
       const lines = readFileSync(this.discoveriesPath, 'utf8').split('\n').filter(Boolean);
@@ -306,7 +307,9 @@ export class MemoryStore {
       for (const line of lines) {
         try {
           const e = JSON.parse(line);
-          if (!role || e.role === role) entries.push(e);
+          if (role && e.role !== role) continue;
+          if (teamId && e.teamId && e.teamId !== teamId) continue;
+          entries.push(e);
         } catch { /* skip malformed */ }
       }
       return entries.slice(-limit).reverse(); // newest first
@@ -328,8 +331,8 @@ export class MemoryStore {
     } catch { /* best-effort */ }
   }
 
-  getDiscoveriesMarkdown(role, limit = 20, maxChars = 4000, scope) {
-    let entries = this.listDiscoveries({ role, limit: limit * 3 });
+  getDiscoveriesMarkdown(role, limit = 20, maxChars = 4000, scope, teamId) {
+    let entries = this.listDiscoveries({ role, teamId, limit: limit * 3 });
     if (entries.length === 0) return '';
 
     if (scope && Array.isArray(scope) && scope.length > 0) {

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

@@ -2000,22 +2000,11 @@ For normal file edits within your scope, proceed without review.
       return this.daemon.rotator.rotate(agentId, { additionalPrompt: message });
     }
 
-    // Context-aware idle resume: if the agent has been idle long enough for
-    // internal compaction to degrade context, spawn fresh with the full
-    // conversation thread instead of resuming the compacted session.
-    const IDLE_CONTEXT_THRESHOLD_MS = 5 * 60_000; // 5 minutes (matches prompt cache TTL)
-    const idleMs = agent.lastActivity
-      ? Date.now() - new Date(agent.lastActivity).getTime()
-      : Infinity;
-
-    if (idleMs > IDLE_CONTEXT_THRESHOLD_MS && this.daemon.journalist) {
-      const resumePrompt = this.daemon.journalist.buildConversationResumePrompt(agent, message);
-      if (resumePrompt) {
-        console.log(`[Groove] Agent ${agent.name} idle ${Math.round(idleMs / 60000)}min — using conversation-thread resume`);
-        // Use rotation machinery but with our richer prompt instead of handoff brief
-        return this._conversationResume(agentId, agent, resumePrompt);
-      }
-    }
+    // Conversation-thread resume: available via explicit API call (conversationResume=true)
+    // but NOT automatic. The old 5-min idle threshold was too aggressive — it replaced
+    // every normal --resume with a full re-spawn, losing the Claude Code session.
+    // Normal --resume preserves the actual CC session; conversation-resume is for when
+    // the user explicitly wants a fresh context with the dialogue replayed.
 
     const provider = getProvider(agent.provider || 'claude-code');
     if (!provider?.buildResumeCommand) {

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

@@ -346,14 +346,31 @@ export class Rotator extends EventEmitter {
       delete this.scoreHistory[agentId];
       this.compactionCounts.delete(agentId);
 
-      let brief = await journalist.generateHandoffBrief(agent, {
-        reason: options.reason,
-        qualityScore: options.qualityScore,
-        signals: options.signals,
-      });
-
-      if (options.additionalPrompt) {
-        brief = brief + '\n\n## User Instruction\n\n' + options.additionalPrompt;
+      // Try conversation-thread resume first: replays the actual user↔assistant
+      // dialogue from stream-json logs so the new agent picks up with full context
+      // instead of a lossy journalist summary. Falls back to the brief if logs are
+      // empty or the thread is too short to be useful.
+      let brief;
+      let usedConversationThread = false;
+      if (typeof journalist.buildConversationResumePrompt === 'function') {
+        const conversationPrompt = journalist.buildConversationResumePrompt(
+          agent,
+          options.additionalPrompt || ''
+        );
+        if (conversationPrompt && conversationPrompt.length > 500) {
+          brief = conversationPrompt;
+          usedConversationThread = true;
+        }
+      }
+      if (!usedConversationThread) {
+        brief = await journalist.generateHandoffBrief(agent, {
+          reason: options.reason,
+          qualityScore: options.qualityScore,
+          signals: options.signals,
+        });
+        if (options.additionalPrompt) {
+          brief = brief + '\n\n## User Instruction\n\n' + options.additionalPrompt;
+        }
       }
 
       if (agent.role === 'planner' && !brief.includes('PLANNING ONLY')) {