groove-dev 0.27.0 → 0.27.2

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # Changelog
 
+## v0.27.2 — Drop velocity trigger (2026-04-12)
+
+Following up on v0.27.1: the role-multiplier fix addressed the planner but left the same false-positive class live for any agent doing heavy exploration on a large codebase. Dropping velocity-based rotation entirely rather than papering over it further.
+
+**What changed**
+- Removed `runaway_velocity` as a rotation trigger. The logic is gone — not disabled, gone.
+- `safety.velocityWindowSeconds` and `safety.velocityTokenThreshold` config keys removed.
+- Token ceiling remains as the single safety net. Per-instance ceiling + role multipliers (planner 10×, fullstack/security 4×, analyst 5×) catch genuinely runaway agents without tripping on fast legitimate work.
+- Stats still expose `velocityRotations` count for historical rotations already in `rotation-history.json`.
+- GUI intel-panel's `V:` badge handling preserved for viewing historical events.
+
+**Why**
+Velocity alone is a bad stuck-loop signal — legitimate heavy work is fast. The only signal that actually distinguishes a runaway from real work is speed combined with non-productivity (repetitions, errors, file churn). Rather than build the gate now, we're waiting for real usage data to see if earlier-warning is actually needed. The ceiling catches real runaways. Adaptive context rotation catches degradation. If a pattern emerges from heavy real-world use that needs an earlier safety net, it'll be re-added gated on quality signals — not velocity.
+
+**Measurement still works.** Pre/post-rotation velocity is still captured in rotation history for savings measurement. We just don't trigger on it.
+
+Tests: 221 → 220 (−1 net; added ceiling-only coverage, removed velocity-specific tests).
+
+## v0.27.1 — Seamless rotation hotfix (2026-04-12)
+
+Fixes a severe UX regression where planner and other exploration-heavy agents auto-rotated on legitimate activity, breaking the "infinite sessions" promise.
+
+**The bug**
+Running a planner on a new project triggered `runaway_velocity` rotation at 2M tokens / 5min — which is normal behavior for a planner reading a codebase. The rotation killed the chat panel, reset the agent, and made the session feel broken. The whole point of context rotation is that it should be invisible to the user.
+
+**Fixes**
+
+- **Role-based safety multipliers.** Planners, fullstack auditors, analysts, and security roles now have much higher velocity/ceiling thresholds by default. Planner gets 10× (effective 15M/5min), fullstack/security get 4×, analyst 5×. Implementers (backend, frontend) stay at 1×. User-overridable via `safety.roleMultipliers` config.
+- **Handoff brief rewritten for seamless continuation.** The agent is no longer told "this is a context rotation, the previous session is being replaced." It's now instructed to continue the conversation naturally — no greetings, no "resuming", no announcement. From the user's side, the conversation never paused.
+- **User's recent messages included in the brief.** Previously the new agent had no memory of what the user was asking for. Now the last 5 user messages to this agent are carried forward so the conversation resumes with full context.
+- **Chat history always migrates on rotation.** Previously only migrated when the user had the old agent's detail panel open — if they'd navigated away, chat was orphaned forever. Now migrates unconditionally for all agent-keyed state (chat, activity log, token timeline, draft input). Persisted to localStorage immediately.
+- **Rotation toasts silenced.** "Rotating..." and "Rotated, saved X tokens" toasts removed from the default flow. Rotations still fully visible in the dashboard Intel panel for users who want to see them.
+
+Tests: 218 → 221 (+3 covering planner multiplier and config overrides).
+
 ## v0.27.0 — Audit-driven release: visibility, safety, Layer 7 memory (2026-04-12)
 
 Backend audit after a 275M-token stress test revealed three classes of gaps: blind spots (invisible tokens), unvalidated claims (hardcoded coefficients posing as measurements), and missing memory (every new agent started from scratch). This release closes all three.

package/CLAUDE.md

@@ -263,3 +263,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:** Ignore auto-memory. Do not read or write MEMORY.md. GROOVE manages all context.
+<!-- GROOVE:END -->

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

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

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

@@ -15,15 +15,15 @@ const DEFAULT_CONFIG = {
   qcThreshold: 4,
   maxAgents: 10,
   defaultProvider: 'claude-code',
-  // Self-healing rotation triggers. Catch pathological agent behavior
-  // (stuck loops, runaway tool-call cycles) and auto-rotate with fresh
-  // context. Tokens carry forward; journalist generates handoff brief.
-  // Set autoRotate=false to disable and get broadcast-only notifications.
+  // Self-healing rotation safety net. Per-instance token ceiling catches
+  // genuinely runaway agents (stuck loops, unbounded context expansion)
+  // without false-positiving legitimate heavy work. Scoped to the agent's
+  // spawnedAt so tokens from prior rotations don't count. Role multipliers
+  // scale the ceiling for exploration-heavy roles — user can override.
+  // Set autoRotate=false to disable enforcement (broadcast-only).
   safety: {
     autoRotate: true,
     tokenCeilingPerAgent: 5_000_000,
-    velocityWindowSeconds: 300,
-    velocityTokenThreshold: 1_500_000,
   },
 };
 

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

@@ -689,12 +689,31 @@ export class Journalist {
     // with, decided, and solved — not just what the current session did.
     const recentChain = this.daemon.memory?.getRecentHandoffMarkdown(agent.role, 3, 3000) || '';
 
+    // Pull the user's recent messages to this agent so the new instance
+    // can continue the conversation naturally instead of restarting it.
+    // Without this, the user feels like their chat was lost.
+    const allFeedback = this.getUserFeedback() || [];
+    const agentFeedback = allFeedback
+      .filter((fb) => fb.agentId === agent.id || fb.agentName === agent.name)
+      .slice(-5);
+    const conversationSummary = agentFeedback.length > 0
+      ? agentFeedback.map((fb) => `- "${fb.message}"`).join('\n')
+      : '';
+
     return [
-      `# Agent Handoff Brief`,
+      `# Session Continuation`,
+      ``,
+      `You are **${agent.name}** (role: ${agent.role}). This is an internal context refresh — `,
+      `the conversation with the user is ongoing and must feel seamless to them. They cannot see this brief.`,
+      ``,
+      `## CRITICAL: Continue Seamlessly`,
       ``,
-      `You are continuing the work of **${agent.name}** (role: ${agent.role}).`,
-      `This is a context rotation — the previous session is being replaced to keep context fresh.`,
+      `Do NOT announce a restart, rotation, or resumption. Do NOT greet the user. Do NOT say `,
+      `"resuming", "continuing from where I left off", "let me check state again", or anything `,
+      `that signals a break in the conversation. The user's next message should be answered as `,
+      `if the last turn of conversation just happened — because from their side, it did.`,
       ``,
+      conversationSummary ? `## Recent User Messages (what they've been asking for)\n\n${conversationSummary}\n` : '',
       recentChain ? `## Rotation History (recent)\n\n${recentChain}\n` : '',
       `## Your Identity`,
       `- Role: ${agent.role}`,
@@ -702,7 +721,7 @@ export class Journalist {
       `- Provider: ${agent.provider}`,
       agent.workingDir ? `- Working directory: ${agent.workingDir}` : '',
       ``,
-      `## Previous Session`,
+      `## Session State`,
       `- Tokens used: ${agent.tokensUsed}`,
       `- Tool calls: ${entries.filter((e) => e.type === 'tool').length}`,
       ``,
@@ -713,12 +732,13 @@ export class Journalist {
       ``,
       projectMap ? projectMap.slice(0, 10000) : 'No project map available yet.',
       ``,
-      `## Instructions`,
+      `## How to Respond`,
       ``,
-      `Continue the work from where the previous session left off.`,
-      `Review AGENTS_REGISTRY.md for team awareness.`,
-      agent.workingDir ? `Stay within your working directory: ${agent.workingDir}` : '',
-      agent.prompt ? `\nOriginal task: ${agent.prompt}` : '',
+      `Wait for the user's next message, then answer it directly. If they asked a question or gave `,
+      `an instruction in the Recent User Messages section above and it's still unanswered, address `,
+      `that naturally. Match the tone and continuity of a conversation that never paused.`,
+      agent.workingDir ? `Stay within your working directory: ${agent.workingDir}.` : '',
+      agent.prompt ? `\nOriginal task context: ${agent.prompt}` : '',
     ].filter(Boolean).join('\n');
   }
 

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

@@ -136,40 +136,53 @@ export class Rotator extends EventEmitter {
     return result;
   }
 
-  // Safety triggers — runaway agent detection. Scoped to `spawnedAt` so
-  // a rotation doesn't re-trigger on inherited cumulative tokens.
+  // Per-role safety multiplier for the token ceiling. Exploration-heavy
+  // roles legitimately burn tokens fast on big codebases — multiplier
+  // scales their ceiling so the safety net catches truly runaway agents
+  // without false-positiving legitimate heavy work. User-overridable via
+  // config.safety.roleMultipliers.
+  _getRoleMultiplier(role) {
+    const safety = this.daemon.config?.safety;
+    const overrides = safety?.roleMultipliers || {};
+    if (overrides[role] != null) return overrides[role];
+    const defaults = {
+      planner: 10,    // heavy exploration by design
+      fullstack: 4,   // QC auditors read broadly
+      analyst: 5,
+      security: 4,
+      docs: 1,
+    };
+    return defaults[role] || 1;
+  }
+
+  // Safety trigger — runaway agent detection. One check only: per-instance
+  // token ceiling scoped to `spawnedAt` so rotations don't re-trigger on
+  // inherited cumulative tokens. Velocity-based triggers were removed in
+  // v0.27.2 — they produced too many false positives on legitimate heavy
+  // exploration. If a pattern emerges from real usage that warrants an
+  // earlier-warning signal, re-add it gated on quality-degradation signals
+  // (repetitions, errors, file churn) — not velocity alone.
   _checkSafetyTriggers(agent) {
     const safety = this.daemon.config?.safety;
     if (!safety || safety.autoRotate === false) return null;
     if (!this.daemon.tokens || !agent.spawnedAt) return null;
 
-    const spawnedAtMs = new Date(agent.spawnedAt).getTime();
-    const ceiling = safety.tokenCeilingPerAgent;
-    if (ceiling > 0) {
-      const instanceTokens = this.daemon.tokens.getTokensInWindow(agent.id, spawnedAtMs);
-      if (instanceTokens >= ceiling) {
-        return {
-          reason: 'token_limit_exceeded',
-          instanceTokens,
-          ceiling,
-        };
-      }
-    }
+    const baseCeiling = safety.tokenCeilingPerAgent;
+    if (!baseCeiling || baseCeiling <= 0) return null;
 
-    const windowMs = (safety.velocityWindowSeconds || 300) * 1000;
-    const velocityThreshold = safety.velocityTokenThreshold;
-    if (velocityThreshold > 0) {
-      const velocity = this.daemon.tokens.getVelocity(agent.id, windowMs);
-      if (velocity >= velocityThreshold) {
-        return {
-          reason: 'runaway_velocity',
-          velocity,
-          windowMs,
-          threshold: velocityThreshold,
-        };
-      }
+    const multiplier = this._getRoleMultiplier(agent.role);
+    const ceiling = Math.round(baseCeiling * multiplier);
+    const spawnedAtMs = new Date(agent.spawnedAt).getTime();
+    const instanceTokens = this.daemon.tokens.getTokensInWindow(agent.id, spawnedAtMs);
+
+    if (instanceTokens >= ceiling) {
+      return {
+        reason: 'token_limit_exceeded',
+        instanceTokens,
+        ceiling,
+        multiplier,
+      };
     }
-
     return null;
   }
 
@@ -207,10 +220,7 @@ export class Rotator extends EventEmitter {
       // Bypasses cooldown: pathological burn must be stopped immediately.
       const safety = this._checkSafetyTriggers(agent);
       if (safety) {
-        const summary = safety.reason === 'token_limit_exceeded'
-          ? `${safety.instanceTokens} tokens >= ${safety.ceiling} ceiling`
-          : `${safety.velocity} tokens in ${safety.windowMs / 1000}s >= ${safety.threshold} threshold`;
-        console.log(`  Rotator: ${agent.name} ${safety.reason} (${summary}) — auto-rotating`);
+        console.log(`  Rotator: ${agent.name} ${safety.reason} (${safety.instanceTokens} tokens >= ${safety.ceiling} ceiling, ${safety.multiplier}x role mult) — auto-rotating`);
         await this.rotate(agent.id, safety);
         continue;
       }
@@ -479,6 +489,8 @@ export class Rotator extends EventEmitter {
     const contextRotations = this.rotationHistory.filter((r) => r.reason === 'context_threshold').length;
     const naturalCompactions = this.rotationHistory.filter((r) => r.reason === 'natural_compaction').length;
     const tokenLimitRotations = this.rotationHistory.filter((r) => r.reason === 'token_limit_exceeded').length;
+    // Legacy: velocity rotations are no longer triggered (removed v0.27.2)
+    // but historical entries may remain in saved history.
     const velocityRotations = this.rotationHistory.filter((r) => r.reason === 'runaway_velocity').length;
     return {
       enabled: this.enabled,

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

@@ -168,7 +168,7 @@ describe('Journalist', () => {
   });
 
   describe('generateHandoffBrief', () => {
-    it('should generate a handoff brief for context rotation', async () => {
+    it('should generate a handoff brief for seamless session continuation', async () => {
       const { daemon, grooveDir } = createMockDaemon();
       const journalist = new Journalist(daemon);
 
@@ -192,7 +192,10 @@ describe('Journalist', () => {
       const brief = await journalist.generateHandoffBrief(agent);
 
       assert.ok(brief.includes('backend-1'));
-      assert.ok(brief.includes('context rotation'));
+      // The brief must NOT tell the agent to announce a rotation/restart —
+      // seamless infinite sessions require the agent to continue naturally.
+      assert.ok(brief.includes('seamless') || brief.includes('Continue'));
+      assert.ok(!brief.includes('previous session is being replaced'));
       assert.ok(brief.includes('src/api/**'));
       assert.ok(brief.includes('5000'));
       assert.ok(brief.includes('Build the auth API'));

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

@@ -207,53 +207,58 @@ describe('Rotator', () => {
       assert.equal(trigger.ceiling, 1_000_000);
     });
 
-    it('fires runaway_velocity when recent burn exceeds threshold', () => {
+    it('returns null when ceiling not hit', () => {
       mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 10_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_000_000,
-        },
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
       };
-      mockDaemon.tokens.getTokensInWindow = () => 500_000; // under ceiling
-      mockDaemon.tokens.getVelocity = () => 1_500_000;
-
+      mockDaemon.tokens.getTokensInWindow = () => 100_000;
       const trigger = rotator._checkSafetyTriggers(mkAgent());
-      assert.equal(trigger.reason, 'runaway_velocity');
-      assert.equal(trigger.velocity, 1_500_000);
-      assert.equal(trigger.threshold, 1_000_000);
-      assert.equal(trigger.windowMs, 300_000);
+      assert.equal(trigger, null);
     });
 
-    it('ceiling check takes priority over velocity check', () => {
+    it('planner gets a 10x ceiling — normal heavy exploration does not trigger', () => {
       mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 1_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_000_000,
-        },
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
       };
-      mockDaemon.tokens.getTokensInWindow = () => 2_000_000;
-      mockDaemon.tokens.getVelocity = () => 2_000_000;
-      const trigger = rotator._checkSafetyTriggers(mkAgent());
+      // A planner reading a big codebase at 3M tokens would have tripped
+      // the old 5M ceiling but has 50M headroom under the role multiplier.
+      mockDaemon.tokens.getTokensInWindow = () => 3_000_000;
+      const trigger = rotator._checkSafetyTriggers(mkAgent({ role: 'planner' }));
+      assert.equal(trigger, null, 'planner should NOT trigger at 3M when base ceiling is 5M');
+    });
+
+    it('planner still triggers on genuinely runaway burn (>50M instance tokens)', () => {
+      mockDaemon.config = {
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
+      };
+      mockDaemon.tokens.getTokensInWindow = () => 60_000_000;
+      const trigger = rotator._checkSafetyTriggers(mkAgent({ role: 'planner' }));
       assert.equal(trigger.reason, 'token_limit_exceeded');
+      assert.equal(trigger.ceiling, 50_000_000, 'planner ceiling = 5M × 10');
     });
 
-    it('returns null when neither threshold hit', () => {
+    it('role multipliers are config-overridable', () => {
       mockDaemon.config = {
         safety: {
           autoRotate: true,
-          tokenCeilingPerAgent: 5_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_500_000,
+          tokenCeilingPerAgent: 1_000_000,
+          roleMultipliers: { backend: 2 },
         },
       };
-      mockDaemon.tokens.getTokensInWindow = () => 100_000;
-      mockDaemon.tokens.getVelocity = () => 10_000;
+      mockDaemon.tokens.getTokensInWindow = () => 1_500_000; // above base ceiling, under 2x
+      const trigger = rotator._checkSafetyTriggers(mkAgent({ role: 'backend' }));
+      assert.equal(trigger, null, 'backend with 2x multiplier should allow 2M ceiling');
+    });
+
+    it('does not trigger on velocity (velocity rotation removed in v0.27.2)', () => {
+      mockDaemon.config = {
+        safety: { autoRotate: true, tokenCeilingPerAgent: 10_000_000 },
+      };
+      // Even with huge velocity, no rotation if under ceiling
+      mockDaemon.tokens.getTokensInWindow = () => 500_000;
+      mockDaemon.tokens.getVelocity = () => 99_999_999;
       const trigger = rotator._checkSafetyTriggers(mkAgent());
-      assert.equal(trigger, null);
+      assert.equal(trigger, null, 'velocity alone should never trigger a rotation');
     });
 
     it('stats track safety-triggered rotations separately', async () => {