groove-dev 0.27.168 → 0.27.169

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

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

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

@@ -717,9 +717,22 @@ export class Daemon {
     this._gc();
     this._gcInterval = setInterval(() => this._gc(), 24 * 60 * 60 * 1000);
 
-    // Periodic state save — crash protection (every 30s)
+    // Periodic state save — crash protection (every 5s)
+    this._lastSavedAgentCount = this.registry.getAll().length;
     this._stateSaveInterval = setInterval(async () => {
-      try { this.state.set('agents', this.registry.getAll()); await this.state.save(); } catch {}
+      try {
+        const agents = this.registry.getAll();
+        // Safety: never overwrite a populated state with an empty one.
+        // An empty registry while agents existed before signals a bug, not
+        // legitimate cleanup — skip the save so .bak stays valid.
+        if (agents.length === 0 && this._lastSavedAgentCount > 0) {
+          console.warn(`[Groove:State] Skipping save — registry unexpectedly empty (was ${this._lastSavedAgentCount} agents). This prevents silent data loss.`);
+          return;
+        }
+        this._lastSavedAgentCount = agents.length;
+        this.state.set('agents', agents);
+        await this.state.save();
+      } catch {}
     }, 5000);
   }
 

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

@@ -2526,7 +2526,7 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
     locks.release(agentId);
 
     // Build resume command
-    const { command: rawCommand, args, env } = provider.buildResumeCommand(sessionId, message, config.model);
+    const { command: rawCommand, args, env } = provider.buildResumeCommand(sessionId, message, config.model, { fast: config.fast });
     const command = resolveProviderCommand(config.provider || 'claude-code') || rawCommand;
 
     // Set up log capture

package/node_modules/@groove-dev/daemon/src/providers/claude-code.js

@@ -39,6 +39,7 @@ export class ClaudeCodeProvider extends Provider {
   static authType = 'subscription';
   static managesOwnContext = true; // Claude Code compacts context internally (~25-37% → 2-8%)
   static models = [
+    { id: 'claude-opus-4-8', name: 'Claude Opus 4.8', tier: 'heavy', contextWindow: 1_000_000 },
     { id: 'claude-opus-4-6', name: 'Claude Opus 4.6', tier: 'heavy', contextWindow: 1_000_000 },
     { id: 'claude-sonnet-4-6', name: 'Claude Sonnet 4.6', tier: 'medium', contextWindow: 200_000 },
     { id: 'claude-haiku-4-5-20251001', name: 'Claude Haiku 4.5', tier: 'light', contextWindow: 200_000 },
@@ -104,6 +105,10 @@ export class ClaudeCodeProvider extends Provider {
       args.push('--effort', agent.effort);
     }
 
+    if (agent.fast) {
+      args.push('--fast');
+    }
+
     // Pass the initial prompt as positional arg (includes GROOVE context)
     const fullPrompt = this.buildFullPrompt(agent);
     if (fullPrompt) {
@@ -117,13 +122,14 @@ export class ClaudeCodeProvider extends Provider {
     };
   }
 
-  buildResumeCommand(sessionId, prompt, model) {
+  buildResumeCommand(sessionId, prompt, model, options = {}) {
     // Resume a previous session — preserves full conversation history
     // No cold start, no handoff brief needed
     const args = ['--resume', sessionId, '--output-format', 'stream-json', '--verbose', '--dangerously-skip-permissions'];
     const knockSettings = ClaudeCodeProvider._buildKnockSettings();
     if (knockSettings) args.push('--settings', knockSettings);
     if (model) args.push('--model', model);
+    if (options.fast) args.push('--fast');
     if (prompt) args.push(prompt);
     return { command: 'claude', args, env: {} };
   }

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

@@ -97,6 +97,9 @@ export class Registry extends EventEmitter {
     const agent = this.agents.get(id);
     if (!agent) return false;
 
+    const caller = new Error().stack.split('\n').slice(1, 4).map(l => l.trim()).join(' <- ');
+    console.log(`[Groove:Registry] REMOVE agent ${agent.name} (${id}) role=${agent.role} status=${agent.status} team=${agent.teamId} silent=${silent} | ${caller}`);
+
     this.agents.delete(id);
     if (silent) {
       this._pendingRemovals.push(id);

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

@@ -3,7 +3,8 @@
 
 import { readFileSync, writeFileSync, existsSync, readdirSync, unlinkSync, renameSync, copyFileSync } from 'fs';
 import { writeFile } from 'node:fs/promises';
-import { resolve } from 'path';
+import { resolve, dirname } from 'path';
+import { randomUUID } from 'crypto';
 
 export class StateManager {
   constructor(grooveDir) {
@@ -39,7 +40,11 @@ export class StateManager {
     if (existsSync(this.path)) {
       try { copyFileSync(this.path, this.backupPath); } catch { /* non-fatal */ }
     }
-    await writeFile(this.path, JSON.stringify(this.data, null, 2));
+    // Atomic write: write to temp file, then rename (rename is atomic on POSIX).
+    // Prevents SIGKILL during write from leaving state.json truncated/empty.
+    const tmpPath = resolve(dirname(this.path), `state.${randomUUID().slice(0, 8)}.tmp`);
+    await writeFile(tmpPath, JSON.stringify(this.data, null, 2));
+    renameSync(tmpPath, this.path);
   }
 
   get(key) {

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

@@ -139,6 +139,7 @@ export function validateAgentConfig(config) {
     verbosity,
     effort,
     routingMode,
+    fast: config.fast === true ? true : undefined,
     labPresetId: (typeof config.labPresetId === 'string' && config.labPresetId.length <= 64) ? config.labPresetId : undefined,
     keeperTags: Array.isArray(config.keeperTags) ? config.keeperTags.filter(t => typeof t === 'string').slice(0, 20) : undefined,
   };