groove-dev 0.27.116 → 0.27.117

package/moe-training/client/domain-tagger.js

@@ -169,6 +169,26 @@ const DOMAIN_TAXONOMY = {
     keywords: ['scientific', 'simulation', 'matlab', 'scipy', 'julia', 'fortran', 'numerical', 'differential equation', 'finite element', 'linear algebra'],
     description: 'Scientific computing, numerical methods, MATLAB/SciPy/Julia, simulations, optimization, statistics',
   },
+  planning_strategy: {
+    keywords: ['plan', 'strategy', 'architect', 'design doc', 'breakdown', 'scope', 'roadmap', 'milestone', 'prioritize', 'tradeoff', 'approach', 'recommend', 'team', 'coordinate', 'delegate', 'assign'],
+    description: 'Project planning, task breakdown, architecture decisions, team coordination, strategy, roadmaps, scoping, prioritization',
+  },
+  conversational_reasoning: {
+    keywords: ['explain', 'why', 'how does', 'what is', 'clarify', 'understand', 'reason', 'think through', 'analyze', 'compare', 'evaluate', 'brainstorm', 'discuss', 'opinion', 'advice'],
+    description: 'Conversational reasoning, explanation, analysis, brainstorming, Q&A, decision-making, advice, evaluation',
+  },
+  documentation_writing: {
+    keywords: ['readme', 'documentation', 'docs', 'markdown', 'api docs', 'changelog', 'tutorial', 'guide', 'specification', 'wiki', 'jsdoc', 'docstring', 'technical writing'],
+    description: 'Documentation writing, READMEs, API docs, changelogs, tutorials, guides, technical writing, specifications',
+  },
+  product_design: {
+    keywords: ['product', 'feature', 'user story', 'requirements', 'ux', 'wireframe', 'prototype', 'feedback', 'iteration', 'mvp', 'spec', 'acceptance criteria', 'stakeholder'],
+    description: 'Product design, feature planning, user stories, requirements gathering, UX, prototyping, MVPs, stakeholder communication',
+  },
+  devops_general: {
+    keywords: ['deploy', 'deployment', 'release', 'rollback', 'staging', 'production', 'environment', 'migration', 'upgrade', 'maintenance', 'incident', 'postmortem', 'runbook'],
+    description: 'DevOps operations, deployments, releases, rollbacks, environment management, incident response, runbooks',
+  },
 };
 
 export class DomainTagger {

package/moe-training/client/trajectory-capture.js

@@ -33,9 +33,11 @@ export class TrajectoryCapture {
     this._transmissionQueue = null;
     this._offlineRetryTimer = null;
     this._contexts = new Map();
+    this._shutdown = false;
   }
 
   async init() {
+    if (this._shutdown) return;
     if (!ConsentManager.isCaptureEnabled()) {
       this._enabled = false;
       return;
@@ -47,12 +49,16 @@ export class TrajectoryCapture {
     this._transmissionQueue.start();
     this._domainTagger = new DomainTagger();
     await this._domainTagger.init();
+    if (this._shutdown) return;
     this._offlineRetryTimer = setInterval(() => {
       this._retryOfflineQueue();
     }, OFFLINE_RETRY_INTERVAL_MS);
+    if (typeof this._offlineRetryTimer.unref === 'function') {
+      this._offlineRetryTimer.unref();
+    }
   }
 
-  async onAgentSpawn(agentId, provider, model, role, teamSize) {
+  async onAgentSpawn(agentId, provider, model, role, teamSize, prompt) {
     if (!this._enabled) return;
 
     const parser = getParser(provider);
@@ -98,9 +104,20 @@ export class TrajectoryCapture {
     ctx.chunkTimer = setInterval(() => {
       this._flushContext(agentId);
     }, CHUNK_TIMEOUT_MS);
+    if (typeof ctx.chunkTimer.unref === 'function') {
+      ctx.chunkTimer.unref();
+    }
 
     this._contexts.set(agentId, ctx);
 
+    if (prompt && typeof prompt === 'string' && prompt.trim()) {
+      this._processStep(agentId, ctx, {
+        type: 'instruction',
+        content: prompt.slice(0, USER_MESSAGE_MAX_CHARS),
+        source: 'user',
+      });
+    }
+
     await this._attestation.openSession(sessionId, metadata);
   }
 
@@ -205,6 +222,7 @@ export class TrajectoryCapture {
   }
 
   async shutdown() {
+    this._shutdown = true;
     if (this._offlineRetryTimer) clearInterval(this._offlineRetryTimer);
     for (const agentId of this._contexts.keys()) {
       await this._closeAgent(agentId, 'SHUTDOWN');
@@ -368,6 +386,23 @@ export class TrajectoryCapture {
   }
 
   _computeTrainingEligibility(ctx, durationSeconds) {
+    const role = ctx.metadata.agent_role || '';
+    const isConversational = role === 'planner' || role === 'chat' || role === 'advisor';
+
+    if (ctx.totalTokens < TRAINING_MIN_TOKENS) {
+      return { eligible: false, exclusionReason: 'insufficient_tokens' };
+    }
+    if (durationSeconds < TRAINING_MIN_DURATION) {
+      return { eligible: false, exclusionReason: 'too_short' };
+    }
+
+    if (isConversational) {
+      if (ctx.stepCount < 2) {
+        return { eligible: false, exclusionReason: 'too_few_steps' };
+      }
+      return { eligible: true, exclusionReason: null };
+    }
+
     if (ctx.stepCount < TRAINING_MIN_STEPS) {
       return { eligible: false, exclusionReason: 'too_few_steps' };
     }
@@ -379,12 +414,6 @@ export class TrajectoryCapture {
     if (!hasObservation) {
       return { eligible: false, exclusionReason: 'no_observations' };
     }
-    if (ctx.totalTokens < TRAINING_MIN_TOKENS) {
-      return { eligible: false, exclusionReason: 'insufficient_tokens' };
-    }
-    if (durationSeconds < TRAINING_MIN_DURATION) {
-      return { eligible: false, exclusionReason: 'too_short' };
-    }
     return { eligible: true, exclusionReason: null };
   }
 

package/moe-training/test/client/trajectory-capture.test.js

@@ -208,7 +208,7 @@ describe('TrajectoryCapture — training eligibility', () => {
     assert.equal(result.exclusionReason, null);
   });
 
-  it('exclusion reasons follow priority order', () => {
+  it('exclusion reasons follow priority order: tokens before duration before steps', () => {
     const tc = makeTc();
     const ctx = makeCtx({
       stepCount: 3,
@@ -220,8 +220,189 @@ describe('TrajectoryCapture — training eligibility', () => {
       ],
     });
     const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.exclusionReason, 'insufficient_tokens');
+  });
+
+  it('duration checked before steps when tokens pass', () => {
+    const tc = makeTc();
+    const ctx = makeCtx({
+      stepCount: 3,
+      totalTokens: 5000,
+      allSteps: [
+        { step: 1, type: 'thought', content: 'thinking' },
+        { step: 2, type: 'thought', content: 'more' },
+        { step: 3, type: 'thought', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.exclusionReason, 'too_short');
+  });
+
+  it('steps checked after tokens and duration pass', () => {
+    const tc = makeTc();
+    const ctx = makeCtx({
+      stepCount: 3,
+      totalTokens: 5000,
+      allSteps: [
+        { step: 1, type: 'thought', content: 'thinking' },
+        { step: 2, type: 'thought', content: 'more' },
+        { step: 3, type: 'thought', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.exclusionReason, 'too_few_steps');
+  });
+});
+
+describe('TrajectoryCapture — planner/conversational eligibility', () => {
+  function makeConversationalCtx(role, overrides = {}) {
+    const ctx = makeCtx(overrides);
+    ctx.metadata.agent_role = role;
+    return ctx;
+  }
+
+  it('planner eligible with only thoughts (no actions/observations)', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'planning' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, true);
+    assert.equal(result.exclusionReason, null);
+  });
+
+  it('chat role eligible with only thoughts', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('chat', {
+      stepCount: 5,
+      totalTokens: 1000,
+      allSteps: [
+        { step: 1, type: 'instruction', content: 'explain React hooks' },
+        { step: 2, type: 'thought', content: 'explaining' },
+        { step: 3, type: 'thought', content: 'more detail' },
+        { step: 4, type: 'thought', content: 'examples' },
+        { step: 5, type: 'resolution', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 30);
+    assert.equal(result.eligible, true);
+  });
+
+  it('advisor role eligible with only thoughts', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('advisor', {
+      stepCount: 3,
+      totalTokens: 800,
+      allSteps: [
+        { step: 1, type: 'instruction', content: 'review approach' },
+        { step: 2, type: 'thought', content: 'analysis' },
+        { step: 3, type: 'resolution', content: 'recommendation' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 20);
+    assert.equal(result.eligible, true);
+  });
+
+  it('planner still requires minimum tokens', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 100,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'plan' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'insufficient_tokens');
+  });
+
+  it('planner still requires minimum duration', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'plan' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'too_short');
+  });
+
+  it('planner requires at least 2 steps', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 1,
+      totalTokens: 2000,
+      allSteps: [{ step: 1, type: 'thought', content: 'plan' }],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
     assert.equal(result.exclusionReason, 'too_few_steps');
   });
+
+  it('coding role (fullstack) still requires actions and observations', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('fullstack', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'thinking' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'no_actions');
+  });
+});
+
+describe('TrajectoryCapture — initial prompt capture', () => {
+  function makeSpawnTc() {
+    const tc = makeTc();
+    tc._enabled = true;
+    tc._scrubber = { scrub: (s) => s };
+    tc._attestation = { openSession: async () => {}, signEnvelope: (sid, e) => e };
+    tc._transmissionQueue = { enqueue: () => {} };
+    tc._domainTagger = null;
+    return tc;
+  }
+
+  it('onAgentSpawn records prompt as instruction step', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p1', 'claude-code', 'opus', 'planner', 1, 'Build a React app');
+
+    const ctx = tc._contexts.get('agent-p1');
+    assert.ok(ctx);
+    assert.equal(ctx.stepCount, 1);
+    assert.equal(ctx.allSteps[0].type, 'instruction');
+    assert.ok(ctx.allSteps[0].content.includes('Build a React app'));
+    assert.equal(ctx.allSteps[0].source, 'user');
+  });
+
+  it('onAgentSpawn with no prompt creates no instruction step', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p2', 'claude-code', 'opus', 'fullstack', 1);
+
+    const ctx = tc._contexts.get('agent-p2');
+    assert.ok(ctx);
+    assert.equal(ctx.stepCount, 0);
+    assert.equal(ctx.allSteps.length, 0);
+  });
+
+  it('onAgentSpawn truncates long prompts', async () => {
+    const tc = makeSpawnTc();
+    const longPrompt = 'x'.repeat(50000);
+    await tc.onAgentSpawn('agent-p3', 'claude-code', 'opus', 'planner', 1, longPrompt);
+
+    const ctx = tc._contexts.get('agent-p3');
+    assert.ok(ctx.allSteps[0].content.length <= 10001);
+  });
+
+  it('onAgentSpawn ignores empty/whitespace prompts', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p4', 'claude-code', 'opus', 'planner', 1, '   ');
+
+    const ctx = tc._contexts.get('agent-p4');
+    assert.equal(ctx.stepCount, 0);
+  });
 });
 
 describe('TrajectoryCapture — user feedback emission', () => {

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

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

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

@@ -330,6 +330,7 @@ export class ProcessManager {
     this._rotatingAgents = new Set(); // agentIds currently being rotated (rotator wrote handoff)
     this._stalledAgents = new Set(); // agentIds already flagged as stalled (avoids duplicate broadcasts)
     this._exitHandled = new Set();
+    this._resultReceived = new Set();
 
     this._stallWatchdog = setInterval(() => this._checkStalls(), STALL_CHECK_INTERVAL_MS);
     if (this._stallWatchdog.unref) this._stallWatchdog.unref();
@@ -389,6 +390,7 @@ export class ProcessManager {
         this._exitHandled.add(agentId);
         setTimeout(() => this._exitHandled.delete(agentId), 30_000);
         this._stalledAgents.delete(agentId);
+        this._resultReceived.delete(agentId);
         const throttle = this._streamThrottle.get(agentId);
         if (throttle?.timer) clearTimeout(throttle.timer);
         this._streamThrottle.delete(agentId);
@@ -425,11 +427,16 @@ export class ProcessManager {
 
     if (this.daemon.locks) this.daemon.locks.release(agent.id);
 
-    const finalStatus = signal === 'SIGTERM' || signal === 'SIGKILL'
-      ? 'killed'
-      : code === 0
-        ? 'completed'
-        : 'crashed';
+    const hadResult = this._resultReceived.has(agent.id);
+    this._resultReceived.delete(agent.id);
+
+    const finalStatus = hadResult
+      ? 'completed'
+      : signal === 'SIGTERM' || signal === 'SIGKILL'
+        ? 'killed'
+        : code === 0
+          ? 'completed'
+          : 'crashed';
 
     const crashError = finalStatus === 'crashed' ? stderrBuf.join('').trim().slice(-500) : null;
 
@@ -552,7 +559,10 @@ export class ProcessManager {
 
     if (this.daemon.locks) this.daemon.locks.release(agent.id);
 
-    const finalStatus = signal === 'SIGTERM' || signal === 'SIGKILL' ? 'killed' : code === 0 ? 'completed' : 'crashed';
+    const hadResult = this._resultReceived.has(agent.id);
+    this._resultReceived.delete(agent.id);
+
+    const finalStatus = hadResult ? 'completed' : signal === 'SIGTERM' || signal === 'SIGKILL' ? 'killed' : code === 0 ? 'completed' : 'crashed';
     registry.update(agent.id, { status: finalStatus, pid: null });
 
     if (this.daemon.trajectoryCapture) {
@@ -727,7 +737,7 @@ export class ProcessManager {
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          agent.id, providerName, config.model || null, config.role, teamSize
+          agent.id, providerName, config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }
@@ -970,6 +980,7 @@ For normal file edits within your scope, proceed without review.
         logStream.end();
         this.handles.delete(agent.id);
         this._stalledAgents.delete(agent.id);
+        this._resultReceived.delete(agent.id);
 
         // Clean up stream throttle so pending timers don't fire for dead agents
         const throttle = this._streamThrottle.get(agent.id);
@@ -1262,6 +1273,21 @@ For normal file edits within your scope, proceed without review.
       if (output.cost) updates.costUsd = (agent.costUsd || 0) + output.cost;
       if (output.duration) updates.durationMs = output.duration;
       if (output.turns) updates.turns = output.turns;
+
+      // Claude Code sometimes hangs after emitting the result event — the
+      // process stays alive instead of exiting.  Record that the result
+      // arrived so exit handlers know this was a successful completion even
+      // if we have to SIGTERM the process.  After a 5s grace period, force-
+      // kill any process that hasn't exited on its own.
+      this._resultReceived.add(agentId);
+      const handle = this.handles.get(agentId);
+      if (handle?.proc && typeof handle.proc.kill === 'function') {
+        setTimeout(() => {
+          if (this.handles.has(agentId) && this._resultReceived.has(agentId)) {
+            try { handle.proc.kill('SIGTERM'); } catch {}
+          }
+        }, 5_000);
+      }
     }
 
     // Context window usage (0-1 scale) — drives rotation threshold
@@ -1845,7 +1871,7 @@ For normal file edits within your scope, proceed without review.
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          newAgent.id, config.provider, config.model || null, config.role, teamSize
+          newAgent.id, config.provider, config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }
@@ -1987,7 +2013,7 @@ For normal file edits within your scope, proceed without review.
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          newAgent.id, config.provider, loopConfig.model || config.model || null, config.role, teamSize
+          newAgent.id, config.provider, loopConfig.model || config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }

package/node_modules/@groove-dev/daemon/src/tunnel-manager.js

@@ -274,9 +274,11 @@ export class TunnelManager {
       throw new Error(testResult.error || 'Host unreachable');
     }
 
+    let preConnectHandled = false;
     if (!testResult.daemonRunning && !testResult.grooveInstalled) {
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'installing' } });
       await this.remoteInstall(id);
+      preConnectHandled = true;
     } else if (!testResult.daemonRunning && testResult.grooveInstalled) {
       const localVer = getLocalVersion();
       if (testResult.remoteVersion && testResult.remoteVersion !== localVer) {
@@ -285,6 +287,7 @@ export class TunnelManager {
       }
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'starting' } });
       await this.autoStart(id);
+      preConnectHandled = true;
     }
 
     this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'connecting' } });
@@ -342,7 +345,9 @@ export class TunnelManager {
       failCount: 0,
     });
 
-    await this._checkAndUpgradeRunning(id, config, localPort);
+    if (!preConnectHandled) {
+      await this._checkAndUpgradeRunning(id, config, localPort);
+    }
 
     try {
       const statusResp = await fetch(`http://localhost:${localPort}/api/status`, { signal: AbortSignal.timeout(5000) });
@@ -473,6 +478,18 @@ export class TunnelManager {
 
       this.daemon.audit.log('tunnel.upgrade', { id, from: oldVersion, to: daemonVer || installedVer });
     } catch (err) {
+      try {
+        const verify = await fetch(`http://localhost:${localPort}/api/status`, { signal: AbortSignal.timeout(5000) });
+        if (verify.ok) {
+          const verifyData = await verify.json();
+          if (verifyData.version === localVer) {
+            this.daemon.broadcast({ type: 'tunnel.version-info', data: { id, localVersion: localVer, remoteVersion: verifyData.version, match: true } });
+            return;
+          }
+          this.daemon.broadcast({ type: 'tunnel.version-mismatch', data: { id, localVersion: localVer, remoteVersion: verifyData.version, message: 'Upgrade timed out but remote is reachable' } });
+          return;
+        }
+      } catch { /* tunnel verification failed */ }
       this.daemon.broadcast({ type: 'tunnel.upgrade-failed', data: { id, error: err.message } });
     }
   }

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

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

package/node_modules/moe-training/client/domain-tagger.js

@@ -169,6 +169,26 @@ const DOMAIN_TAXONOMY = {
     keywords: ['scientific', 'simulation', 'matlab', 'scipy', 'julia', 'fortran', 'numerical', 'differential equation', 'finite element', 'linear algebra'],
     description: 'Scientific computing, numerical methods, MATLAB/SciPy/Julia, simulations, optimization, statistics',
   },
+  planning_strategy: {
+    keywords: ['plan', 'strategy', 'architect', 'design doc', 'breakdown', 'scope', 'roadmap', 'milestone', 'prioritize', 'tradeoff', 'approach', 'recommend', 'team', 'coordinate', 'delegate', 'assign'],
+    description: 'Project planning, task breakdown, architecture decisions, team coordination, strategy, roadmaps, scoping, prioritization',
+  },
+  conversational_reasoning: {
+    keywords: ['explain', 'why', 'how does', 'what is', 'clarify', 'understand', 'reason', 'think through', 'analyze', 'compare', 'evaluate', 'brainstorm', 'discuss', 'opinion', 'advice'],
+    description: 'Conversational reasoning, explanation, analysis, brainstorming, Q&A, decision-making, advice, evaluation',
+  },
+  documentation_writing: {
+    keywords: ['readme', 'documentation', 'docs', 'markdown', 'api docs', 'changelog', 'tutorial', 'guide', 'specification', 'wiki', 'jsdoc', 'docstring', 'technical writing'],
+    description: 'Documentation writing, READMEs, API docs, changelogs, tutorials, guides, technical writing, specifications',
+  },
+  product_design: {
+    keywords: ['product', 'feature', 'user story', 'requirements', 'ux', 'wireframe', 'prototype', 'feedback', 'iteration', 'mvp', 'spec', 'acceptance criteria', 'stakeholder'],
+    description: 'Product design, feature planning, user stories, requirements gathering, UX, prototyping, MVPs, stakeholder communication',
+  },
+  devops_general: {
+    keywords: ['deploy', 'deployment', 'release', 'rollback', 'staging', 'production', 'environment', 'migration', 'upgrade', 'maintenance', 'incident', 'postmortem', 'runbook'],
+    description: 'DevOps operations, deployments, releases, rollbacks, environment management, incident response, runbooks',
+  },
 };
 
 export class DomainTagger {

package/node_modules/moe-training/client/trajectory-capture.js

@@ -33,9 +33,11 @@ export class TrajectoryCapture {
     this._transmissionQueue = null;
     this._offlineRetryTimer = null;
     this._contexts = new Map();
+    this._shutdown = false;
   }
 
   async init() {
+    if (this._shutdown) return;
     if (!ConsentManager.isCaptureEnabled()) {
       this._enabled = false;
       return;
@@ -47,12 +49,16 @@ export class TrajectoryCapture {
     this._transmissionQueue.start();
     this._domainTagger = new DomainTagger();
     await this._domainTagger.init();
+    if (this._shutdown) return;
     this._offlineRetryTimer = setInterval(() => {
       this._retryOfflineQueue();
     }, OFFLINE_RETRY_INTERVAL_MS);
+    if (typeof this._offlineRetryTimer.unref === 'function') {
+      this._offlineRetryTimer.unref();
+    }
   }
 
-  async onAgentSpawn(agentId, provider, model, role, teamSize) {
+  async onAgentSpawn(agentId, provider, model, role, teamSize, prompt) {
     if (!this._enabled) return;
 
     const parser = getParser(provider);
@@ -98,9 +104,20 @@ export class TrajectoryCapture {
     ctx.chunkTimer = setInterval(() => {
       this._flushContext(agentId);
     }, CHUNK_TIMEOUT_MS);
+    if (typeof ctx.chunkTimer.unref === 'function') {
+      ctx.chunkTimer.unref();
+    }
 
     this._contexts.set(agentId, ctx);
 
+    if (prompt && typeof prompt === 'string' && prompt.trim()) {
+      this._processStep(agentId, ctx, {
+        type: 'instruction',
+        content: prompt.slice(0, USER_MESSAGE_MAX_CHARS),
+        source: 'user',
+      });
+    }
+
     await this._attestation.openSession(sessionId, metadata);
   }
 
@@ -205,6 +222,7 @@ export class TrajectoryCapture {
   }
 
   async shutdown() {
+    this._shutdown = true;
     if (this._offlineRetryTimer) clearInterval(this._offlineRetryTimer);
     for (const agentId of this._contexts.keys()) {
       await this._closeAgent(agentId, 'SHUTDOWN');
@@ -368,6 +386,23 @@ export class TrajectoryCapture {
   }
 
   _computeTrainingEligibility(ctx, durationSeconds) {
+    const role = ctx.metadata.agent_role || '';
+    const isConversational = role === 'planner' || role === 'chat' || role === 'advisor';
+
+    if (ctx.totalTokens < TRAINING_MIN_TOKENS) {
+      return { eligible: false, exclusionReason: 'insufficient_tokens' };
+    }
+    if (durationSeconds < TRAINING_MIN_DURATION) {
+      return { eligible: false, exclusionReason: 'too_short' };
+    }
+
+    if (isConversational) {
+      if (ctx.stepCount < 2) {
+        return { eligible: false, exclusionReason: 'too_few_steps' };
+      }
+      return { eligible: true, exclusionReason: null };
+    }
+
     if (ctx.stepCount < TRAINING_MIN_STEPS) {
       return { eligible: false, exclusionReason: 'too_few_steps' };
     }
@@ -379,12 +414,6 @@ export class TrajectoryCapture {
     if (!hasObservation) {
       return { eligible: false, exclusionReason: 'no_observations' };
     }
-    if (ctx.totalTokens < TRAINING_MIN_TOKENS) {
-      return { eligible: false, exclusionReason: 'insufficient_tokens' };
-    }
-    if (durationSeconds < TRAINING_MIN_DURATION) {
-      return { eligible: false, exclusionReason: 'too_short' };
-    }
     return { eligible: true, exclusionReason: null };
   }
 

package/node_modules/moe-training/test/client/trajectory-capture.test.js

@@ -208,7 +208,7 @@ describe('TrajectoryCapture — training eligibility', () => {
     assert.equal(result.exclusionReason, null);
   });
 
-  it('exclusion reasons follow priority order', () => {
+  it('exclusion reasons follow priority order: tokens before duration before steps', () => {
     const tc = makeTc();
     const ctx = makeCtx({
       stepCount: 3,
@@ -220,8 +220,189 @@ describe('TrajectoryCapture — training eligibility', () => {
       ],
     });
     const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.exclusionReason, 'insufficient_tokens');
+  });
+
+  it('duration checked before steps when tokens pass', () => {
+    const tc = makeTc();
+    const ctx = makeCtx({
+      stepCount: 3,
+      totalTokens: 5000,
+      allSteps: [
+        { step: 1, type: 'thought', content: 'thinking' },
+        { step: 2, type: 'thought', content: 'more' },
+        { step: 3, type: 'thought', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.exclusionReason, 'too_short');
+  });
+
+  it('steps checked after tokens and duration pass', () => {
+    const tc = makeTc();
+    const ctx = makeCtx({
+      stepCount: 3,
+      totalTokens: 5000,
+      allSteps: [
+        { step: 1, type: 'thought', content: 'thinking' },
+        { step: 2, type: 'thought', content: 'more' },
+        { step: 3, type: 'thought', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.exclusionReason, 'too_few_steps');
+  });
+});
+
+describe('TrajectoryCapture — planner/conversational eligibility', () => {
+  function makeConversationalCtx(role, overrides = {}) {
+    const ctx = makeCtx(overrides);
+    ctx.metadata.agent_role = role;
+    return ctx;
+  }
+
+  it('planner eligible with only thoughts (no actions/observations)', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'planning' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, true);
+    assert.equal(result.exclusionReason, null);
+  });
+
+  it('chat role eligible with only thoughts', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('chat', {
+      stepCount: 5,
+      totalTokens: 1000,
+      allSteps: [
+        { step: 1, type: 'instruction', content: 'explain React hooks' },
+        { step: 2, type: 'thought', content: 'explaining' },
+        { step: 3, type: 'thought', content: 'more detail' },
+        { step: 4, type: 'thought', content: 'examples' },
+        { step: 5, type: 'resolution', content: 'done' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 30);
+    assert.equal(result.eligible, true);
+  });
+
+  it('advisor role eligible with only thoughts', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('advisor', {
+      stepCount: 3,
+      totalTokens: 800,
+      allSteps: [
+        { step: 1, type: 'instruction', content: 'review approach' },
+        { step: 2, type: 'thought', content: 'analysis' },
+        { step: 3, type: 'resolution', content: 'recommendation' },
+      ],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 20);
+    assert.equal(result.eligible, true);
+  });
+
+  it('planner still requires minimum tokens', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 100,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'plan' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'insufficient_tokens');
+  });
+
+  it('planner still requires minimum duration', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'plan' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 5);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'too_short');
+  });
+
+  it('planner requires at least 2 steps', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('planner', {
+      stepCount: 1,
+      totalTokens: 2000,
+      allSteps: [{ step: 1, type: 'thought', content: 'plan' }],
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
     assert.equal(result.exclusionReason, 'too_few_steps');
   });
+
+  it('coding role (fullstack) still requires actions and observations', () => {
+    const tc = makeTc();
+    const ctx = makeConversationalCtx('fullstack', {
+      stepCount: 10,
+      totalTokens: 2000,
+      allSteps: Array.from({ length: 10 }, (_, i) => ({ step: i + 1, type: 'thought', content: 'thinking' })),
+    });
+    const result = tc._computeTrainingEligibility(ctx, 60);
+    assert.equal(result.eligible, false);
+    assert.equal(result.exclusionReason, 'no_actions');
+  });
+});
+
+describe('TrajectoryCapture — initial prompt capture', () => {
+  function makeSpawnTc() {
+    const tc = makeTc();
+    tc._enabled = true;
+    tc._scrubber = { scrub: (s) => s };
+    tc._attestation = { openSession: async () => {}, signEnvelope: (sid, e) => e };
+    tc._transmissionQueue = { enqueue: () => {} };
+    tc._domainTagger = null;
+    return tc;
+  }
+
+  it('onAgentSpawn records prompt as instruction step', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p1', 'claude-code', 'opus', 'planner', 1, 'Build a React app');
+
+    const ctx = tc._contexts.get('agent-p1');
+    assert.ok(ctx);
+    assert.equal(ctx.stepCount, 1);
+    assert.equal(ctx.allSteps[0].type, 'instruction');
+    assert.ok(ctx.allSteps[0].content.includes('Build a React app'));
+    assert.equal(ctx.allSteps[0].source, 'user');
+  });
+
+  it('onAgentSpawn with no prompt creates no instruction step', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p2', 'claude-code', 'opus', 'fullstack', 1);
+
+    const ctx = tc._contexts.get('agent-p2');
+    assert.ok(ctx);
+    assert.equal(ctx.stepCount, 0);
+    assert.equal(ctx.allSteps.length, 0);
+  });
+
+  it('onAgentSpawn truncates long prompts', async () => {
+    const tc = makeSpawnTc();
+    const longPrompt = 'x'.repeat(50000);
+    await tc.onAgentSpawn('agent-p3', 'claude-code', 'opus', 'planner', 1, longPrompt);
+
+    const ctx = tc._contexts.get('agent-p3');
+    assert.ok(ctx.allSteps[0].content.length <= 10001);
+  });
+
+  it('onAgentSpawn ignores empty/whitespace prompts', async () => {
+    const tc = makeSpawnTc();
+    await tc.onAgentSpawn('agent-p4', 'claude-code', 'opus', 'planner', 1, '   ');
+
+    const ctx = tc._contexts.get('agent-p4');
+    assert.equal(ctx.stepCount, 0);
+  });
 });
 
 describe('TrajectoryCapture — user feedback emission', () => {

package/package.json

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

package/packages/daemon/src/process.js

@@ -330,6 +330,7 @@ export class ProcessManager {
     this._rotatingAgents = new Set(); // agentIds currently being rotated (rotator wrote handoff)
     this._stalledAgents = new Set(); // agentIds already flagged as stalled (avoids duplicate broadcasts)
     this._exitHandled = new Set();
+    this._resultReceived = new Set();
 
     this._stallWatchdog = setInterval(() => this._checkStalls(), STALL_CHECK_INTERVAL_MS);
     if (this._stallWatchdog.unref) this._stallWatchdog.unref();
@@ -389,6 +390,7 @@ export class ProcessManager {
         this._exitHandled.add(agentId);
         setTimeout(() => this._exitHandled.delete(agentId), 30_000);
         this._stalledAgents.delete(agentId);
+        this._resultReceived.delete(agentId);
         const throttle = this._streamThrottle.get(agentId);
         if (throttle?.timer) clearTimeout(throttle.timer);
         this._streamThrottle.delete(agentId);
@@ -425,11 +427,16 @@ export class ProcessManager {
 
     if (this.daemon.locks) this.daemon.locks.release(agent.id);
 
-    const finalStatus = signal === 'SIGTERM' || signal === 'SIGKILL'
-      ? 'killed'
-      : code === 0
-        ? 'completed'
-        : 'crashed';
+    const hadResult = this._resultReceived.has(agent.id);
+    this._resultReceived.delete(agent.id);
+
+    const finalStatus = hadResult
+      ? 'completed'
+      : signal === 'SIGTERM' || signal === 'SIGKILL'
+        ? 'killed'
+        : code === 0
+          ? 'completed'
+          : 'crashed';
 
     const crashError = finalStatus === 'crashed' ? stderrBuf.join('').trim().slice(-500) : null;
 
@@ -552,7 +559,10 @@ export class ProcessManager {
 
     if (this.daemon.locks) this.daemon.locks.release(agent.id);
 
-    const finalStatus = signal === 'SIGTERM' || signal === 'SIGKILL' ? 'killed' : code === 0 ? 'completed' : 'crashed';
+    const hadResult = this._resultReceived.has(agent.id);
+    this._resultReceived.delete(agent.id);
+
+    const finalStatus = hadResult ? 'completed' : signal === 'SIGTERM' || signal === 'SIGKILL' ? 'killed' : code === 0 ? 'completed' : 'crashed';
     registry.update(agent.id, { status: finalStatus, pid: null });
 
     if (this.daemon.trajectoryCapture) {
@@ -727,7 +737,7 @@ export class ProcessManager {
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          agent.id, providerName, config.model || null, config.role, teamSize
+          agent.id, providerName, config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }
@@ -970,6 +980,7 @@ For normal file edits within your scope, proceed without review.
         logStream.end();
         this.handles.delete(agent.id);
         this._stalledAgents.delete(agent.id);
+        this._resultReceived.delete(agent.id);
 
         // Clean up stream throttle so pending timers don't fire for dead agents
         const throttle = this._streamThrottle.get(agent.id);
@@ -1262,6 +1273,21 @@ For normal file edits within your scope, proceed without review.
       if (output.cost) updates.costUsd = (agent.costUsd || 0) + output.cost;
       if (output.duration) updates.durationMs = output.duration;
       if (output.turns) updates.turns = output.turns;
+
+      // Claude Code sometimes hangs after emitting the result event — the
+      // process stays alive instead of exiting.  Record that the result
+      // arrived so exit handlers know this was a successful completion even
+      // if we have to SIGTERM the process.  After a 5s grace period, force-
+      // kill any process that hasn't exited on its own.
+      this._resultReceived.add(agentId);
+      const handle = this.handles.get(agentId);
+      if (handle?.proc && typeof handle.proc.kill === 'function') {
+        setTimeout(() => {
+          if (this.handles.has(agentId) && this._resultReceived.has(agentId)) {
+            try { handle.proc.kill('SIGTERM'); } catch {}
+          }
+        }, 5_000);
+      }
     }
 
     // Context window usage (0-1 scale) — drives rotation threshold
@@ -1845,7 +1871,7 @@ For normal file edits within your scope, proceed without review.
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          newAgent.id, config.provider, config.model || null, config.role, teamSize
+          newAgent.id, config.provider, config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }
@@ -1987,7 +2013,7 @@ For normal file edits within your scope, proceed without review.
       try {
         const teamSize = registry.getAll().filter(a => a.status === 'active' || a.status === 'running' || a.status === 'starting').length;
         this.daemon.trajectoryCapture.onAgentSpawn(
-          newAgent.id, config.provider, loopConfig.model || config.model || null, config.role, teamSize
+          newAgent.id, config.provider, loopConfig.model || config.model || null, config.role, teamSize, config.prompt
         ).catch(() => {});
       } catch (e) { /* fail silent */ }
     }

package/packages/daemon/src/tunnel-manager.js

@@ -274,9 +274,11 @@ export class TunnelManager {
       throw new Error(testResult.error || 'Host unreachable');
     }
 
+    let preConnectHandled = false;
     if (!testResult.daemonRunning && !testResult.grooveInstalled) {
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'installing' } });
       await this.remoteInstall(id);
+      preConnectHandled = true;
     } else if (!testResult.daemonRunning && testResult.grooveInstalled) {
       const localVer = getLocalVersion();
       if (testResult.remoteVersion && testResult.remoteVersion !== localVer) {
@@ -285,6 +287,7 @@ export class TunnelManager {
       }
       this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'starting' } });
       await this.autoStart(id);
+      preConnectHandled = true;
     }
 
     this.daemon.broadcast({ type: 'tunnel.status', data: { id, step: 'connecting' } });
@@ -342,7 +345,9 @@ export class TunnelManager {
       failCount: 0,
     });
 
-    await this._checkAndUpgradeRunning(id, config, localPort);
+    if (!preConnectHandled) {
+      await this._checkAndUpgradeRunning(id, config, localPort);
+    }
 
     try {
       const statusResp = await fetch(`http://localhost:${localPort}/api/status`, { signal: AbortSignal.timeout(5000) });
@@ -473,6 +478,18 @@ export class TunnelManager {
 
       this.daemon.audit.log('tunnel.upgrade', { id, from: oldVersion, to: daemonVer || installedVer });
     } catch (err) {
+      try {
+        const verify = await fetch(`http://localhost:${localPort}/api/status`, { signal: AbortSignal.timeout(5000) });
+        if (verify.ok) {
+          const verifyData = await verify.json();
+          if (verifyData.version === localVer) {
+            this.daemon.broadcast({ type: 'tunnel.version-info', data: { id, localVersion: localVer, remoteVersion: verifyData.version, match: true } });
+            return;
+          }
+          this.daemon.broadcast({ type: 'tunnel.version-mismatch', data: { id, localVersion: localVer, remoteVersion: verifyData.version, message: 'Upgrade timed out but remote is reachable' } });
+          return;
+        }
+      } catch { /* tunnel verification failed */ }
       this.daemon.broadcast({ type: 'tunnel.upgrade-failed', data: { id, error: err.message } });
     }
   }

package/packages/gui/package.json

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