groove-dev 0.27.1 → 0.27.2

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # 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.

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

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.1",
+  "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.1",
+  "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/rotator.js

@@ -136,69 +136,53 @@ export class Rotator extends EventEmitter {
     return result;
   }
 
-  // Per-role safety multipliers. Exploration-heavy roles burn tokens fast
-  // by design (reading codebases, searching files) — a planner running
-  // normally can hit 2M+ tokens in 5 min just reading. One-size-fits-all
-  // thresholds produce false positives on exactly the roles that need to
-  // read fast. Multipliers scale both the velocity threshold and the
-  // instance ceiling. User can override via config.safety.roleMultipliers.
+  // 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];
-    // Defaults tuned from observed legitimate velocity
     const defaults = {
-      planner: 10,    // heavy exploration — effectively exempt in practice
+      planner: 10,    // heavy exploration by design
       fullstack: 4,   // QC auditors read broadly
-      analyst: 5,     // research/analysis roles
-      security: 4,    // audit roles
-      docs: 1,        // focused edits
+      analyst: 5,
+      security: 4,
+      docs: 1,
     };
     return defaults[role] || 1;
   }
 
-  // Safety triggers — runaway agent detection. Scoped to `spawnedAt` so
-  // a rotation doesn't re-trigger on inherited cumulative tokens.
+  // 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 multiplier = this._getRoleMultiplier(agent.role);
-    const spawnedAtMs = new Date(agent.spawnedAt).getTime();
-
     const baseCeiling = safety.tokenCeilingPerAgent;
-    const ceiling = baseCeiling > 0 ? Math.round(baseCeiling * multiplier) : 0;
-    if (ceiling > 0) {
-      const instanceTokens = this.daemon.tokens.getTokensInWindow(agent.id, spawnedAtMs);
-      if (instanceTokens >= ceiling) {
-        return {
-          reason: 'token_limit_exceeded',
-          instanceTokens,
-          ceiling,
-          multiplier,
-        };
-      }
-    }
+    if (!baseCeiling || baseCeiling <= 0) return null;
 
-    const windowMs = (safety.velocityWindowSeconds || 300) * 1000;
-    const baseVelocityThreshold = safety.velocityTokenThreshold;
-    const velocityThreshold = baseVelocityThreshold > 0
-      ? Math.round(baseVelocityThreshold * multiplier)
-      : 0;
-    if (velocityThreshold > 0) {
-      const velocity = this.daemon.tokens.getVelocity(agent.id, windowMs);
-      if (velocity >= velocityThreshold) {
-        return {
-          reason: 'runaway_velocity',
-          velocity,
-          windowMs,
-          threshold: velocityThreshold,
-          multiplier,
-        };
-      }
+    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;
   }
 
@@ -236,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;
       }
@@ -508,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/rotator.test.js

@@ -207,85 +207,34 @@ 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,
-        },
-      };
-      mockDaemon.tokens.getTokensInWindow = () => 500_000; // under ceiling
-      mockDaemon.tokens.getVelocity = () => 1_500_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);
-    });
-
-    it('ceiling check takes priority over velocity check', () => {
-      mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 1_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_000_000,
-        },
-      };
-      mockDaemon.tokens.getTokensInWindow = () => 2_000_000;
-      mockDaemon.tokens.getVelocity = () => 2_000_000;
-      const trigger = rotator._checkSafetyTriggers(mkAgent());
-      assert.equal(trigger.reason, 'token_limit_exceeded');
-    });
-
-    it('returns null when neither threshold hit', () => {
-      mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 5_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_500_000,
-        },
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
       };
       mockDaemon.tokens.getTokensInWindow = () => 100_000;
-      mockDaemon.tokens.getVelocity = () => 10_000;
       const trigger = rotator._checkSafetyTriggers(mkAgent());
       assert.equal(trigger, null);
     });
 
-    it('planner gets a 10x multiplier so normal exploration does not trigger', () => {
+    it('planner gets a 10x ceiling — normal heavy exploration does not trigger', () => {
       mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 5_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_500_000,
-        },
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
       };
-      // 2M in 5min would have triggered under v0.27.0 defaults (user bug report)
+      // 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;
-      mockDaemon.tokens.getVelocity = () => 2_053_414;
       const trigger = rotator._checkSafetyTriggers(mkAgent({ role: 'planner' }));
-      assert.equal(trigger, null, 'planner should NOT trigger on 2M velocity');
+      assert.equal(trigger, null, 'planner should NOT trigger at 3M when base ceiling is 5M');
     });
 
-    it('planner still triggers on genuinely runaway velocity (>15M per 5min)', () => {
+    it('planner still triggers on genuinely runaway burn (>50M instance tokens)', () => {
       mockDaemon.config = {
-        safety: {
-          autoRotate: true,
-          tokenCeilingPerAgent: 5_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 1_500_000,
-        },
+        safety: { autoRotate: true, tokenCeilingPerAgent: 5_000_000 },
       };
-      mockDaemon.tokens.getTokensInWindow = () => 1_000_000;
-      mockDaemon.tokens.getVelocity = () => 20_000_000;
+      mockDaemon.tokens.getTokensInWindow = () => 60_000_000;
       const trigger = rotator._checkSafetyTriggers(mkAgent({ role: 'planner' }));
-      assert.equal(trigger.reason, 'runaway_velocity');
-      assert.equal(trigger.threshold, 15_000_000, 'planner threshold = 1.5M × 10');
+      assert.equal(trigger.reason, 'token_limit_exceeded');
+      assert.equal(trigger.ceiling, 50_000_000, 'planner ceiling = 5M × 10');
     });
 
     it('role multipliers are config-overridable', () => {
@@ -293,15 +242,23 @@ describe('Rotator', () => {
         safety: {
           autoRotate: true,
           tokenCeilingPerAgent: 1_000_000,
-          velocityWindowSeconds: 300,
-          velocityTokenThreshold: 500_000,
-          roleMultipliers: { backend: 2, frontend: 2 },
+          roleMultipliers: { backend: 2 },
         },
       };
-      mockDaemon.tokens.getTokensInWindow = () => 1_500_000; // above base ceiling
-      mockDaemon.tokens.getVelocity = () => 0;
-      const backendTrigger = rotator._checkSafetyTriggers(mkAgent({ role: 'backend' }));
-      assert.equal(backendTrigger, null, 'backend with 2x multiplier should allow 2M ceiling');
+      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, 'velocity alone should never trigger a rotation');
     });
 
     it('stats track safety-triggered rotations separately', async () => {

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

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/gui",
-  "version": "0.27.1",
+  "version": "0.27.2",
   "description": "GROOVE GUI — visual agent control plane",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "groove-dev",
-  "version": "0.27.1",
+  "version": "0.27.2",
   "description": "Open-source agent orchestration layer — the AI company OS. Local model agent engine (GGUF/Ollama/llama-server), HuggingFace model browser, MCP integrations (Slack, Gmail, Stripe, 15+), agent scheduling (cron), business roles (CMO, CFO, EA). GUI dashboard, multi-agent coordination, zero cold-start, infinite sessions. Works with Claude Code, Codex, Gemini CLI, Ollama, any local model.",
   "license": "FSL-1.1-Apache-2.0",
   "author": "Groove Dev <hello@groovedev.ai> (https://groovedev.ai)",

package/packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/cli",
-  "version": "0.27.1",
+  "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/packages/daemon/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/daemon",
-  "version": "0.27.1",
+  "version": "0.27.2",
   "description": "GROOVE daemon — agent orchestration engine",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",

package/packages/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/packages/daemon/src/rotator.js

@@ -136,69 +136,53 @@ export class Rotator extends EventEmitter {
     return result;
   }
 
-  // Per-role safety multipliers. Exploration-heavy roles burn tokens fast
-  // by design (reading codebases, searching files) — a planner running
-  // normally can hit 2M+ tokens in 5 min just reading. One-size-fits-all
-  // thresholds produce false positives on exactly the roles that need to
-  // read fast. Multipliers scale both the velocity threshold and the
-  // instance ceiling. User can override via config.safety.roleMultipliers.
+  // 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];
-    // Defaults tuned from observed legitimate velocity
     const defaults = {
-      planner: 10,    // heavy exploration — effectively exempt in practice
+      planner: 10,    // heavy exploration by design
       fullstack: 4,   // QC auditors read broadly
-      analyst: 5,     // research/analysis roles
-      security: 4,    // audit roles
-      docs: 1,        // focused edits
+      analyst: 5,
+      security: 4,
+      docs: 1,
     };
     return defaults[role] || 1;
   }
 
-  // Safety triggers — runaway agent detection. Scoped to `spawnedAt` so
-  // a rotation doesn't re-trigger on inherited cumulative tokens.
+  // 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 multiplier = this._getRoleMultiplier(agent.role);
-    const spawnedAtMs = new Date(agent.spawnedAt).getTime();
-
     const baseCeiling = safety.tokenCeilingPerAgent;
-    const ceiling = baseCeiling > 0 ? Math.round(baseCeiling * multiplier) : 0;
-    if (ceiling > 0) {
-      const instanceTokens = this.daemon.tokens.getTokensInWindow(agent.id, spawnedAtMs);
-      if (instanceTokens >= ceiling) {
-        return {
-          reason: 'token_limit_exceeded',
-          instanceTokens,
-          ceiling,
-          multiplier,
-        };
-      }
-    }
+    if (!baseCeiling || baseCeiling <= 0) return null;
 
-    const windowMs = (safety.velocityWindowSeconds || 300) * 1000;
-    const baseVelocityThreshold = safety.velocityTokenThreshold;
-    const velocityThreshold = baseVelocityThreshold > 0
-      ? Math.round(baseVelocityThreshold * multiplier)
-      : 0;
-    if (velocityThreshold > 0) {
-      const velocity = this.daemon.tokens.getVelocity(agent.id, windowMs);
-      if (velocity >= velocityThreshold) {
-        return {
-          reason: 'runaway_velocity',
-          velocity,
-          windowMs,
-          threshold: velocityThreshold,
-          multiplier,
-        };
-      }
+    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;
   }
 
@@ -236,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;
       }
@@ -508,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/packages/gui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@groove-dev/gui",
-  "version": "0.27.1",
+  "version": "0.27.2",
   "description": "GROOVE GUI — visual agent control plane",
   "license": "FSL-1.1-Apache-2.0",
   "type": "module",