groove-dev 0.27.39 → 0.27.40

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

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

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

@@ -2868,23 +2868,28 @@ Keep responses concise. Help them think, don't lecture them about the system the
     res.json(result);
   });
 
-  // Clean up stale artifacts (old plans, recommended teams, etc.)
+  // Clean up stale artifacts. Scope to a single team when teamId is provided —
+  // wiping every agent's working dir on a global cleanup would delete other
+  // in-flight teams' unlaunched plans. When called with no teamId, only the
+  // daemon-root plan file is touched (safe baseline).
   app.post('/api/cleanup', (req, res) => {
+    const teamId = req.body?.teamId || req.query?.teamId || null;
     let cleaned = 0;
-    // Clean recommended-team.json from all known locations
     const locations = [resolve(daemon.grooveDir, 'recommended-team.json')];
-    for (const agent of daemon.registry.getAll()) {
-      if (agent.workingDir) {
-        locations.push(resolve(agent.workingDir, '.groove', 'recommended-team.json'));
+
+    if (teamId) {
+      // Only agents in this team get their workspace scanned
+      for (const agent of daemon.registry.getAll()) {
+        if (agent.teamId === teamId && agent.workingDir) {
+          locations.push(resolve(agent.workingDir, '.groove', 'recommended-team.json'));
+        }
       }
     }
-    const defaultDir = daemon.config?.defaultWorkingDir;
-    if (defaultDir) locations.push(resolve(defaultDir, '.groove', 'recommended-team.json'));
 
     for (const p of locations) {
       if (existsSync(p)) { try { unlinkSync(p); cleaned++; } catch { /* */ } }
     }
-    daemon.audit.log('cleanup', { cleaned });
+    daemon.audit.log('cleanup', { cleaned, teamId });
     res.json({ ok: true, cleaned });
   });
 

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

@@ -19,6 +19,11 @@ import { lookup as mimeLookup } from './mimetypes.js';
 
 const READY_TIMEOUT_MS = 60_000;  // give dev servers a minute to boot
 const MAX_STDOUT_BYTES = 256 * 1024;
+// Strip CSI/OSC/other ANSI escape sequences — Vite prints URLs with inline
+// bold/color codes (e.g. "http://localhost:\x1b[1m5175\x1b[22m/") which would
+// otherwise break port-number regexes.
+const ANSI_REGEX = /[\u001B\u009B][[\]()#;?]*(?:(?:(?:(?:;[-a-zA-Z\d/#&.:=?%@~_]+)*|[a-zA-Z\d]+(?:;[-a-zA-Z\d/#&.:=?%@~_]*)*)?\u0007)|(?:(?:\d{1,4}(?:;\d{0,4})*)?[\dA-PR-TZcf-nq-uy=><~]))/g;
+function stripAnsi(s) { return s.replace(ANSI_REGEX, ''); }
 
 export class PreviewService {
   constructor(daemon) {
@@ -168,11 +173,12 @@ export class PreviewService {
       };
 
       const timer = setTimeout(() => {
-        finish({ launched: false, reason: `timeout waiting for url in stdout; last stderr: ${stderrBuf.slice(-400)}` });
+        const tail = stripAnsi(stderrBuf).slice(-400) || stripAnsi(stdoutBuf).slice(-400) || '(no output)';
+        finish({ launched: false, reason: `timeout waiting for url in stdout; last output: ${tail}` });
       }, READY_TIMEOUT_MS);
 
       const tryMatch = () => {
-        const combined = stdoutBuf + '\n' + stderrBuf;
+        const combined = stripAnsi(stdoutBuf + '\n' + stderrBuf);
         if (readyText && !combined.includes(readyText)) return;
         const m = combined.match(urlPattern);
         if (!m) return;

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

@@ -10,8 +10,10 @@ const DEFAULT_THRESHOLD = 0.65;      // For non-self-managing providers (was 0.7
 const HARD_CEILING = 0.80;           // Force rotate (was 0.85) — only for non-self-managing
 const CHECK_INTERVAL = 15_000;
 const QUALITY_THRESHOLD = 40;   // Score below this triggers quality rotation
-const MIN_EVENTS = 10;          // Minimum classifier events before scoring
-const MIN_AGE_SEC = 120;        // Minimum agent age before quality rotation
+const MIN_EVENTS = 20;          // Minimum classifier events before scoring
+const MIN_AGE_SEC = 300;        // Minimum agent age before quality rotation (5 min)
+const QUALITY_MIN_TOKENS = 20_000; // Minimum real token work before quality rotation can fire
+const QUALITY_MIN_FILES = 3;    // Or: 3 successful file writes proves the agent is productive
 const SCORE_HISTORY_MAX = 40;   // ~10 min at 15s intervals
 const COOLDOWN_MS = 5 * 60 * 1000;   // 5 minutes between rotations per agent
 const QUALITY_COOLDOWN_MS = 2 * 60 * 1000; // 2 minutes for quality degradation rotations
@@ -128,7 +130,20 @@ export class Rotator extends EventEmitter {
       return { score: 70, signals: {}, hasEnoughData: false, ageSec: Math.round(ageSec), eventCount: events.length };
     }
 
-    const signals = this.daemon.adaptive.extractSignals(events, agent.scope);
+    // Productive-work floor: don't even score an agent that hasn't produced
+    // enough to judge. A frontend agent scaffolding a project naturally emits
+    // noisy signals (npm install warnings, Write retries) in its first few
+    // minutes; killing it mid-scaffold destroys the context it was building.
+    // Only allow the score to gate rotation once EITHER substantial tokens
+    // have flowed OR the agent has already written multiple files successfully.
+    const tokens = agent.tokensUsed || 0;
+    const signalsEarly = this.daemon.adaptive.extractSignals(events, agent.scope);
+    const filesWritten = signalsEarly.filesWritten || 0;
+    if (tokens < QUALITY_MIN_TOKENS && filesWritten < QUALITY_MIN_FILES) {
+      return { score: 70, signals: signalsEarly, hasEnoughData: false, ageSec: Math.round(ageSec), eventCount: events.length, reason: 'below_productive_floor' };
+    }
+
+    const signals = signalsEarly;
     let score = this.daemon.adaptive.scoreSession(signals);
 
     if (ageSec > 1800) score -= 5;