groove-dev 0.27.180 → 0.27.181

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

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

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

@@ -363,6 +363,14 @@ export class ProcessManager {
     this._stalledAgents = new Set(); // agentIds already flagged as stalled (avoids duplicate broadcasts)
     this._exitHandled = new Set();
     this._resultReceived = new Set();
+    // In-flight user messages, kept until a turn actually completes so a
+    // dropped turn can be re-issued instead of losing the user's message.
+    // agentId -> { message, attempts }
+    this._pendingUserMessage = new Map();
+    // Retry count carried across the agent-id change that resume() performs.
+    // Kept separate so teardown of the old agent can't reset it to 0 and
+    // turn a bounded retry into an infinite respawn loop. oldAgentId -> attempts
+    this._retryAttemptCarry = new Map();
     this._truncationFlagged = new Set(); // agentIds that have had any truncation in their session
     this._lastAssistantBlocks = new Map(); // agentId -> last assistant content blocks (for abandoned tool_use detection)
     this._previousCacheReadTokens = new Map(); // agentId -> previous turn's cacheReadTokens
@@ -1559,6 +1567,18 @@ For normal file edits within your scope, proceed without review.
 
     // Session result data (cost, duration, turns)
     if (output.type === 'result') {
+      // Phantom-interrupt abort: on --resume with an orphaned background shell
+      // task, the CLI injects a synthetic "[Request interrupted by user]" that
+      // aborts the turn before the model is ever called. Fingerprint is an
+      // error result that never reached the API (apiDurationMs === 0, so zero
+      // cost). Retrying is free and succeeds ~94% of the time — treating it as
+      // a normal result is what made user messages vanish silently.
+      if (output.isError && output.apiDurationMs === 0) {
+        if (this._retryDroppedTurn(agentId, agent, output)) return;
+      }
+      // A turn completed for real — the user's message is safely delivered.
+      this._pendingUserMessage.delete(agentId);
+
       tokens.recordResult(agentId, {
         costUsd: output.cost, durationMs: output.duration, turns: output.turns,
       });
@@ -2510,6 +2530,59 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
    * spawns fresh with the full conversation thread instead of --resume.
    * This avoids degraded context from internal compaction.
    */
+  /**
+   * Re-issue a user message whose turn was aborted before reaching the model.
+   *
+   * Returns true if a retry was scheduled (caller should skip normal result
+   * handling), false if the turn should be treated as a real result.
+   */
+  _retryDroppedTurn(agentId, agent, output) {
+    const pending = this._pendingUserMessage.get(agentId);
+    if (!pending) return false; // nothing to re-issue — let it fall through
+
+    const MAX_RETRIES = 2;
+    if (pending.attempts >= MAX_RETRIES) {
+      this._pendingUserMessage.delete(agentId);
+      this.daemon.broadcast({
+        type: 'agent:output',
+        agentId,
+        data: {
+          type: 'activity',
+          subtype: 'error',
+          data: `Message could not be delivered after ${MAX_RETRIES + 1} attempts `
+            + `(${output.terminalReason || output.subtype || 'turn aborted'}). Please re-send.`,
+        },
+      });
+      return false;
+    }
+
+    pending.attempts += 1;
+    this._retryAttemptCarry.set(agentId, pending.attempts);
+    this.daemon.broadcast({
+      type: 'agent:output',
+      agentId,
+      data: {
+        type: 'activity',
+        subtype: 'info',
+        data: `Turn aborted before reaching the model — retrying (${pending.attempts}/${MAX_RETRIES})…`,
+      },
+    });
+
+    // Let the aborted process finish tearing down before re-spawning; resume()
+    // kills the current handle and re-registers under a new agent id.
+    setTimeout(() => {
+      this.resume(agentId, pending.message).catch((err) => {
+        this.daemon.broadcast({
+          type: 'agent:output',
+          agentId,
+          data: { type: 'activity', subtype: 'error', data: `Retry failed: ${err.message}` },
+        });
+      });
+    }, 500);
+
+    return true;
+  }
+
   async resume(agentId, message) {
     const { registry, locks } = this.daemon;
     const agent = registry.get(agentId);
@@ -2577,8 +2650,19 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
       workingDir: config.workingDir || this.daemon.config?.defaultWorkingDir || undefined,
       name: config.name,
       teamId: config.teamId,
+      authMode: config.authMode,
     });
 
+    // Hold the message until a turn actually completes, so a turn aborted
+    // before reaching the model can be re-issued rather than lost. Carries the
+    // attempt count forward from the agent id we're replacing.
+    if (message) {
+      const carried = this._retryAttemptCarry.get(agentId);
+      this._retryAttemptCarry.delete(agentId);
+      this._pendingUserMessage.delete(agentId);
+      this._pendingUserMessage.set(newAgent.id, { message, attempts: carried || 0 });
+    }
+
     // Carry cumulative tokens
     if (config.tokensUsed > 0) {
       registry.update(newAgent.id, { tokensUsed: config.tokensUsed });
@@ -2959,6 +3043,10 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
     if (throttle?.timer) clearTimeout(throttle.timer);
     this._streamThrottle.delete(agentId);
     this.pendingMessages.delete(agentId);
+    // Safe during a retry: resume() re-registers the message under the new
+    // agent id from its own `message` argument, and the attempt count lives in
+    // _retryAttemptCarry (which kill() must not touch).
+    this._pendingUserMessage.delete(agentId);
 
     // Unregister ambassador if this agent was one
     const agent = this.daemon.registry.get(agentId);

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

@@ -257,6 +257,13 @@ export class ClaudeCodeProvider extends Provider {
             cost: data.total_cost_usd,
             duration: data.duration_ms,
             turns: data.num_turns,
+            // Error/abort signals. Without these an `error_during_execution`
+            // result looks identical to a successful one and the user's
+            // message is silently dropped.
+            subtype: data.subtype,
+            isError: data.is_error === true || data.subtype === 'error_during_execution',
+            apiDurationMs: data.duration_api_ms,
+            terminalReason: data.terminal_reason,
           });
         }
       } catch {

package/node_modules/@groove-dev/daemon/test/claude-code-provider.test.js

@@ -0,0 +1,76 @@
+// GROOVE — Claude Code Provider Tests
+// FSL-1.1-Apache-2.0 — see LICENSE
+
+import { describe, it } from 'node:test';
+import assert from 'node:assert/strict';
+import { ClaudeCodeProvider } from '../src/providers/claude-code.js';
+
+const provider = new ClaudeCodeProvider();
+
+describe('ClaudeCodeProvider result parsing', () => {
+  // Regression: the "phantom interrupt" bug. On --resume with an orphaned
+  // background shell task, the CLI aborts the turn before calling the model.
+  // These fields were previously dropped, so the abort was indistinguishable
+  // from a successful turn and the user's message vanished with no UI signal.
+  it('surfaces error signals from an aborted turn', () => {
+    const out = provider.parseOutput(JSON.stringify({
+      type: 'result',
+      subtype: 'error_during_execution',
+      is_error: true,
+      duration_ms: 5126,
+      duration_api_ms: 0,
+      num_turns: 2,
+      total_cost_usd: 0,
+      terminal_reason: 'aborted_streaming',
+    }));
+
+    assert.equal(out.type, 'result');
+    assert.equal(out.isError, true);
+    assert.equal(out.apiDurationMs, 0, 'api duration 0 proves the model was never reached');
+    assert.equal(out.terminalReason, 'aborted_streaming');
+  });
+
+  it('treats an error_during_execution subtype as an error even without is_error', () => {
+    const out = provider.parseOutput(JSON.stringify({
+      type: 'result', subtype: 'error_during_execution', duration_api_ms: 0,
+    }));
+    assert.equal(out.isError, true);
+  });
+
+  it('does not flag a successful turn as an error', () => {
+    const out = provider.parseOutput(JSON.stringify({
+      type: 'result',
+      subtype: 'success',
+      is_error: false,
+      duration_ms: 17850,
+      duration_api_ms: 17672,
+      num_turns: 3,
+      total_cost_usd: 0.42,
+      terminal_reason: 'completed',
+    }));
+
+    assert.equal(out.isError, false);
+    assert.equal(out.apiDurationMs, 17672);
+    assert.equal(out.cost, 0.42);
+  });
+
+  // The retry gate is (isError && apiDurationMs === 0). A turn that failed
+  // *after* reaching the model has real cost and must not be auto-retried.
+  it('distinguishes a pre-model abort from a post-model failure', () => {
+    const preModel = provider.parseOutput(JSON.stringify({
+      type: 'result', subtype: 'error_during_execution', is_error: true, duration_api_ms: 0,
+    }));
+    const postModel = provider.parseOutput(JSON.stringify({
+      type: 'result', subtype: 'error_during_execution', is_error: true, duration_api_ms: 8400,
+    }));
+
+    assert.equal(preModel.isError && preModel.apiDurationMs === 0, true, 'free to retry');
+    assert.equal(postModel.isError && postModel.apiDurationMs === 0, false, 'already billed — do not retry');
+  });
+});
+
+describe('ClaudeCodeProvider models', () => {
+  it('offers Fable 5', () => {
+    assert.ok(ClaudeCodeProvider.models.some((m) => m.id === 'claude-fable-5'));
+  });
+});

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

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

package/packages/daemon/src/process.js

@@ -363,6 +363,14 @@ export class ProcessManager {
     this._stalledAgents = new Set(); // agentIds already flagged as stalled (avoids duplicate broadcasts)
     this._exitHandled = new Set();
     this._resultReceived = new Set();
+    // In-flight user messages, kept until a turn actually completes so a
+    // dropped turn can be re-issued instead of losing the user's message.
+    // agentId -> { message, attempts }
+    this._pendingUserMessage = new Map();
+    // Retry count carried across the agent-id change that resume() performs.
+    // Kept separate so teardown of the old agent can't reset it to 0 and
+    // turn a bounded retry into an infinite respawn loop. oldAgentId -> attempts
+    this._retryAttemptCarry = new Map();
     this._truncationFlagged = new Set(); // agentIds that have had any truncation in their session
     this._lastAssistantBlocks = new Map(); // agentId -> last assistant content blocks (for abandoned tool_use detection)
     this._previousCacheReadTokens = new Map(); // agentId -> previous turn's cacheReadTokens
@@ -1559,6 +1567,18 @@ For normal file edits within your scope, proceed without review.
 
     // Session result data (cost, duration, turns)
     if (output.type === 'result') {
+      // Phantom-interrupt abort: on --resume with an orphaned background shell
+      // task, the CLI injects a synthetic "[Request interrupted by user]" that
+      // aborts the turn before the model is ever called. Fingerprint is an
+      // error result that never reached the API (apiDurationMs === 0, so zero
+      // cost). Retrying is free and succeeds ~94% of the time — treating it as
+      // a normal result is what made user messages vanish silently.
+      if (output.isError && output.apiDurationMs === 0) {
+        if (this._retryDroppedTurn(agentId, agent, output)) return;
+      }
+      // A turn completed for real — the user's message is safely delivered.
+      this._pendingUserMessage.delete(agentId);
+
       tokens.recordResult(agentId, {
         costUsd: output.cost, durationMs: output.duration, turns: output.turns,
       });
@@ -2510,6 +2530,59 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
    * spawns fresh with the full conversation thread instead of --resume.
    * This avoids degraded context from internal compaction.
    */
+  /**
+   * Re-issue a user message whose turn was aborted before reaching the model.
+   *
+   * Returns true if a retry was scheduled (caller should skip normal result
+   * handling), false if the turn should be treated as a real result.
+   */
+  _retryDroppedTurn(agentId, agent, output) {
+    const pending = this._pendingUserMessage.get(agentId);
+    if (!pending) return false; // nothing to re-issue — let it fall through
+
+    const MAX_RETRIES = 2;
+    if (pending.attempts >= MAX_RETRIES) {
+      this._pendingUserMessage.delete(agentId);
+      this.daemon.broadcast({
+        type: 'agent:output',
+        agentId,
+        data: {
+          type: 'activity',
+          subtype: 'error',
+          data: `Message could not be delivered after ${MAX_RETRIES + 1} attempts `
+            + `(${output.terminalReason || output.subtype || 'turn aborted'}). Please re-send.`,
+        },
+      });
+      return false;
+    }
+
+    pending.attempts += 1;
+    this._retryAttemptCarry.set(agentId, pending.attempts);
+    this.daemon.broadcast({
+      type: 'agent:output',
+      agentId,
+      data: {
+        type: 'activity',
+        subtype: 'info',
+        data: `Turn aborted before reaching the model — retrying (${pending.attempts}/${MAX_RETRIES})…`,
+      },
+    });
+
+    // Let the aborted process finish tearing down before re-spawning; resume()
+    // kills the current handle and re-registers under a new agent id.
+    setTimeout(() => {
+      this.resume(agentId, pending.message).catch((err) => {
+        this.daemon.broadcast({
+          type: 'agent:output',
+          agentId,
+          data: { type: 'activity', subtype: 'error', data: `Retry failed: ${err.message}` },
+        });
+      });
+    }, 500);
+
+    return true;
+  }
+
   async resume(agentId, message) {
     const { registry, locks } = this.daemon;
     const agent = registry.get(agentId);
@@ -2577,8 +2650,19 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
       workingDir: config.workingDir || this.daemon.config?.defaultWorkingDir || undefined,
       name: config.name,
       teamId: config.teamId,
+      authMode: config.authMode,
     });
 
+    // Hold the message until a turn actually completes, so a turn aborted
+    // before reaching the model can be re-issued rather than lost. Carries the
+    // attempt count forward from the agent id we're replacing.
+    if (message) {
+      const carried = this._retryAttemptCarry.get(agentId);
+      this._retryAttemptCarry.delete(agentId);
+      this._pendingUserMessage.delete(agentId);
+      this._pendingUserMessage.set(newAgent.id, { message, attempts: carried || 0 });
+    }
+
     // Carry cumulative tokens
     if (config.tokensUsed > 0) {
       registry.update(newAgent.id, { tokensUsed: config.tokensUsed });
@@ -2959,6 +3043,10 @@ After fixing all issues, run tests (npm test) and build (npm run build) to verif
     if (throttle?.timer) clearTimeout(throttle.timer);
     this._streamThrottle.delete(agentId);
     this.pendingMessages.delete(agentId);
+    // Safe during a retry: resume() re-registers the message under the new
+    // agent id from its own `message` argument, and the attempt count lives in
+    // _retryAttemptCarry (which kill() must not touch).
+    this._pendingUserMessage.delete(agentId);
 
     // Unregister ambassador if this agent was one
     const agent = this.daemon.registry.get(agentId);

package/packages/daemon/src/providers/claude-code.js

@@ -257,6 +257,13 @@ export class ClaudeCodeProvider extends Provider {
             cost: data.total_cost_usd,
             duration: data.duration_ms,
             turns: data.num_turns,
+            // Error/abort signals. Without these an `error_during_execution`
+            // result looks identical to a successful one and the user's
+            // message is silently dropped.
+            subtype: data.subtype,
+            isError: data.is_error === true || data.subtype === 'error_during_execution',
+            apiDurationMs: data.duration_api_ms,
+            terminalReason: data.terminal_reason,
           });
         }
       } catch {

package/packages/gui/package.json

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