groove-dev 0.27.183 → 0.27.185

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

@@ -3,94 +3,212 @@
 import { randomUUID } from 'crypto';
 import { deliverInstruction } from './deliver.js';
 
-// How much prior conversation to replay into each relay turn. Enough for the
-// recipient to pick up the thread without re-reading the whole exchange.
+// An unanswered ask holds an HTTP request open and blocks the calling agent,
+// so it has to be bounded.
+const DEFAULT_TIMEOUT_MS = 5 * 60 * 1000;
+const MAX_TIMEOUT_MS = 15 * 60 * 1000;
+
+// Ceiling on autonomous back-and-forth. Two agents that can call each other
+// freely will happily talk until the budget is gone; past this they're told to
+// wrap up and report to the user.
+const MAX_EXCHANGES = 12;
+
 const CONTEXT_TURNS = 4;
 const MAX_TURN_CHARS = 1200;
 
 /**
- * Agent-to-agent relay, user-initiated.
+ * Agent-to-agent consultation.
  *
- * The A→B hop is always driven by a human clicking Relay. The B→A hop is
- * automatic: B's answer is fed straight back into A so A can process it and
- * report to the user, who then decides whether to run another turn.
+ * An agent asks another agent a question and BLOCKS until it answers — the
+ * daemon holds the caller's HTTP request open, delivers the question, waits
+ * for the target's next result, and returns it as the response body. That
+ * makes a normal request/response loop available to the calling agent, so two
+ * agents can iterate to consensus without a human relaying between them.
  *
  * Delivery goes through deliverInstruction — the same pipe as user chat — so
- * a relay reaches its target whether it's mid-task, idle, or stopped.
+ * a question reaches its target whether it's mid-task, idle, or stopped.
  */
 export class InnerChat {
   constructor(daemon) {
     this.daemon = daemon;
     this.threads = new Map();
-    // agentId -> { threadId, sentAt } — which agent we're awaiting a reply from.
+    // agentId being asked -> pending record (resolve/reject/timer/threadId)
     this.awaiting = new Map();
+    // asker id -> id of the agent it is currently blocked on
+    this.blockedOn = new Map();
   }
 
   /**
-   * Relay a message from one agent to another, opening a thread or continuing
-   * an existing one.
+   * Ask another agent a question and wait for its answer.
+   *
+   * Resolves with { reply, threadId, exchanges, remaining }. Rejects when the
+   * target can't be reached, the exchange cap is hit, the call would deadlock,
+   * or the target doesn't answer inside the timeout.
    */
-  async send(fromAgentId, toAgentId, message, threadId = null) {
+  async ask(fromAgentId, toAgentId, message, opts = {}) {
     const fromAgent = this.daemon.registry.get(fromAgentId);
     const toAgent = this.daemon.registry.get(toAgentId);
-    if (!fromAgent) throw new Error(`Sender agent ${fromAgentId} not found`);
+    if (!fromAgent) throw new Error(`Calling agent ${fromAgentId} not found`);
     if (!toAgent) throw new Error(`Target agent ${toAgentId} not found`);
-    if (fromAgentId === toAgentId) throw new Error('Cannot relay an agent to itself');
+    if (fromAgentId === toAgentId) throw new Error('An agent cannot ask itself');
     if (!message || !message.trim()) throw new Error('message is required');
 
-    const thread = threadId ? this.threads.get(threadId) : null;
-    if (threadId && !thread) throw new Error(`Thread ${threadId} not found`);
+    // Deadlock guard: the target is already blocked waiting on us, so it can
+    // never get far enough to read this. Tell the caller to just answer.
+    if (this.blockedOn.get(toAgentId) === fromAgentId) {
+      throw new Error(
+        `${toAgent.name} is currently waiting for YOUR answer — it cannot reply to a new `
+        + 'question until you respond. Answer its question first; your reply is what unblocks it.',
+      );
+    }
+    if (this.awaiting.has(toAgentId)) {
+      throw new Error(`${toAgent.name} is already answering another agent — try again shortly`);
+    }
+
+    const t = this._openThread(fromAgent, toAgent, opts.threadId);
+    this._checkExchangeCap(t, toAgent);
 
-    const t = thread || this._createThread(fromAgent, toAgent);
+    const timeoutMs = Math.min(Number(opts.timeoutMs) || DEFAULT_TIMEOUT_MS, MAX_TIMEOUT_MS);
+
+    const { turn, targetId, skipResults } = await this._deliver(t, fromAgent, toAgent, message.trim(), 'ask');
+    this.daemon.audit.log('innerchat.ask', { thread: t.id, from: fromAgentId, to: targetId });
+
+    const reply = await this._awaitReply(t, fromAgentId, targetId, { skipResults, timeoutMs });
+    const exchanges = t.turns.filter((x) => x.kind === 'ask' || x.kind === 'tell').length;
+
+    return { reply, threadId: t.id, exchanges, remaining: MAX_EXCHANGES - exchanges };
+  }
+
+  /**
+   * Send a message to another agent WITHOUT blocking. Returns as soon as it's
+   * delivered. If the target later replies, the reply is routed back to the
+   * sender asynchronously — resuming the sender if its turn has ended.
+   *
+   * This is the fire-and-forget counterpart to ask(): use it to hand off to an
+   * agent that's heads-down, where waiting out a timeout would waste the turn.
+   */
+  async tell(fromAgentId, toAgentId, message, opts = {}) {
+    const fromAgent = this.daemon.registry.get(fromAgentId);
+    const toAgent = this.daemon.registry.get(toAgentId);
+    if (!fromAgent) throw new Error(`Calling agent ${fromAgentId} not found`);
+    if (!toAgent) throw new Error(`Target agent ${toAgentId} not found`);
+    if (fromAgentId === toAgentId) throw new Error('An agent cannot message itself');
+    if (!message || !message.trim()) throw new Error('message is required');
+    if (this.awaiting.has(toAgentId)) {
+      throw new Error(`${toAgent.name} already has an unanswered message — wait for its reply before sending another`);
+    }
+
+    const t = this._openThread(fromAgent, toAgent, opts.threadId);
+    this._checkExchangeCap(t, toAgent);
+
+    const { turn, targetId, skipResults } = await this._deliver(t, fromAgent, toAgent, message.trim(), 'tell');
+    this.daemon.audit.log('innerchat.tell', { thread: t.id, from: fromAgentId, to: targetId });
+
+    // Register async capture: the reply (if any) is forwarded, not awaited.
+    this.awaiting.set(targetId, {
+      mode: 'async',
+      threadId: t.id,
+      senderId: fromAgentId,
+      skipResults,
+      timer: setTimeout(() => this._clearAwait(targetId), MAX_TIMEOUT_MS),
+    });
+
+    const exchanges = t.turns.filter((x) => x.kind === 'ask' || x.kind === 'tell').length;
+    return { delivered: true, threadId: t.id, exchanges, remaining: MAX_EXCHANGES - exchanges };
+  }
+
+  // Shared send: push a turn, deliver it, and remap the target id if delivery
+  // resumed/rotated the agent. Returns the queued-skip count for reply capture.
+  async _deliver(thread, fromAgent, toAgent, message, kind) {
     const turn = {
       id: randomUUID().slice(0, 12),
       from: peer(fromAgent),
       to: peer(toAgent),
-      text: message.trim(),
-      kind: 'relay',
+      text: message,
+      kind,
       status: 'sending',
       timestamp: Date.now(),
     };
-    t.turns.push(turn);
+    thread.turns.push(turn);
 
-    const wrapped = this._wrap(t, fromAgent, toAgent, message.trim());
+    const wrapped = this._wrap(thread, fromAgent, toAgent, message, kind);
 
     let result;
     try {
-      result = await deliverInstruction(this.daemon, toAgentId, wrapped, { recordFeedback: false });
+      result = await deliverInstruction(this.daemon, toAgent.id, wrapped, { recordFeedback: false });
     } catch (err) {
       turn.status = 'failed';
       turn.error = err.message;
-      t.status = 'failed';
-      this.daemon.broadcast({ type: 'innerchat:turn', data: { thread: t, turn } });
-      throw err;
+      thread.status = 'failed';
+      this._broadcast(thread, turn);
+      throw new Error(`Could not reach ${toAgent.name}: ${err.message}`);
     }
 
-    // A stopped target gets resumed or rotated, which mints a new agent id.
-    // Re-key everything onto it or the reply will never be captured.
-    if (result.agentId !== toAgentId) {
-      this._remapParticipant(t, toAgentId, result.agentId);
-      turn.to.id = result.agentId;
+    // A stopped target gets resumed or rotated, minting a new agent id.
+    const targetId = result.agentId;
+    if (targetId !== toAgent.id) {
+      this._remapParticipant(thread, toAgent.id, targetId);
+      turn.to.id = targetId;
     }
 
     turn.status = result.status;
-    t.status = 'awaiting_reply';
-    t.updatedAt = Date.now();
+    thread.status = 'awaiting_reply';
+    thread.updatedAt = Date.now();
+    this._broadcast(thread, turn);
+
+    // A queued message sits behind the target's current work, so the next
+    // result belongs to that prior task, not to us — skip it.
+    return { turn, targetId, skipResults: result.status === 'message_queued' ? 1 : 0 };
+  }
 
-    // A queued relay sits behind whatever the agent is already doing, so the
-    // next result belongs to that prior task, not to us — skip it.
-    const skipResults = result.status === 'message_queued' ? 1 : 0;
-    this.awaiting.set(turn.to.id, { threadId: t.id, sentAt: Date.now(), skipResults });
+  _openThread(fromAgent, toAgent, threadId) {
+    const thread = threadId ? this.threads.get(threadId) : this._findThread(fromAgent.id, toAgent.id);
+    return thread || this._createThread(fromAgent, toAgent);
+  }
 
-    this.daemon.broadcast({ type: 'innerchat:turn', data: { thread: t, turn } });
-    this.daemon.audit.log('innerchat.send', { thread: t.id, from: fromAgentId, to: turn.to.id });
+  _checkExchangeCap(thread, toAgent) {
+    const used = thread.turns.filter((x) => x.kind === 'ask' || x.kind === 'tell').length;
+    if (used >= MAX_EXCHANGES) {
+      throw new Error(
+        `This conversation has reached its ${MAX_EXCHANGES}-exchange limit. Stop consulting `
+        + `${toAgent.name} and report your conclusion to the user.`,
+      );
+    }
+  }
 
-    return { thread: t, turn };
+  _awaitReply(thread, askerId, targetId, { skipResults, timeoutMs }) {
+    return new Promise((resolve, reject) => {
+      const settle = (fn, value) => {
+        this._clearAwait(targetId);
+        this.blockedOn.delete(askerId);
+        fn(value);
+      };
+
+      const record = {
+        mode: 'block',
+        threadId: thread.id,
+        askerId,
+        skipResults,
+        resolve: (text) => settle(resolve, text),
+        reject: (err) => settle(reject, err),
+        timer: setTimeout(() => {
+          thread.status = 'timeout';
+          thread.updatedAt = Date.now();
+          record.reject(new Error(
+            `No answer within ${Math.round(timeoutMs / 1000)}s. The agent may still be working — `
+            + 'proceed on your own judgement, or use a non-blocking send (tell) next time.',
+          ));
+        }, timeoutMs),
+      };
+
+      this.awaiting.set(targetId, record);
+      this.blockedOn.set(askerId, targetId);
+    });
   }
 
   /**
-   * Watch agent output for the reply to an outstanding relay, then forward it
-   * back to the agent that asked.
+   * Watch agent output for the reply to an outstanding message.
+   * Called from the process manager's output handler.
    */
   onAgentOutput(agentId, output) {
     const pending = this.awaiting.get(agentId);
@@ -98,72 +216,88 @@ export class InnerChat {
     if (output.type !== 'result') return;
 
     const thread = this.threads.get(pending.threadId);
-    if (!thread) { this.awaiting.delete(agentId); return; }
+    if (!thread) { this._settlePending(agentId, pending, null, new Error('Conversation was lost')); return; }
 
-    const responseText = extractText(output.data);
-    if (!responseText) return;
+    const text = extractText(output.data);
+    if (!text) return;
 
-    // Burn off results belonging to work that was already underway when the
-    // relay was queued behind it.
+    // Burn off results belonging to work already underway when we queued.
     if (pending.skipResults > 0) {
       pending.skipResults -= 1;
       return;
     }
 
-    // Claim the reply before the async forward so a second result arriving
-    // mid-flight can't be captured as a duplicate.
-    this.awaiting.delete(agentId);
-
     const responder = this.daemon.registry.get(agentId);
-    const asker = this._otherParticipant(thread, agentId);
-
-    const turn = {
+    const answerTurn = {
       id: randomUUID().slice(0, 12),
       from: responder ? peer(responder) : { id: agentId, name: agentId, role: 'agent' },
-      to: asker,
-      text: responseText,
-      kind: 'reply',
-      status: 'forwarding',
+      to: this._otherParticipant(thread, agentId),
+      text,
+      kind: 'answer',
+      status: 'delivered',
       timestamp: Date.now(),
     };
-    thread.turns.push(turn);
-    thread.status = 'forwarding';
+    thread.turns.push(answerTurn);
+    thread.status = 'idle';
     thread.updatedAt = Date.now();
-
-    this._forwardReply(thread, turn, asker.id, responseText);
+    this._broadcast(thread, answerTurn);
+    this.daemon.audit.log('innerchat.answer', { thread: thread.id, from: agentId });
+
+    if (pending.mode === 'async') {
+      this._clearAwait(agentId);
+      this._forwardAsyncReply(pending, answerTurn, text);
+    } else {
+      pending.resolve(text);
+    }
   }
 
-  async _forwardReply(thread, turn, askerId, responseText) {
-    const from = turn.from;
-    const relay = [
-      `[InnerChat reply from ${from.name} (${from.role})]`,
+  // Deliver an async (tell) reply back to the original sender, resuming it if
+  // its turn has ended. The sender is not blocked, so this just wakes them.
+  async _forwardAsyncReply(pending, answerTurn, text) {
+    const sender = this.daemon.registry.get(pending.senderId)
+      || this.daemon.registry.getAll().find((a) => a.name === answerTurn.to.name);
+    if (!sender) return; // sender is gone — nothing to route back to
+
+    const msg = [
+      `[InnerChat reply from ${answerTurn.from.name} (${answerTurn.from.role})]`,
       '',
-      responseText,
+      text,
       '',
-      `This is the answer to what you relayed to ${from.name}. Process it and report back — `
-        + 'the user will decide whether to send another turn.',
+      'This is a reply to a message you sent earlier (you were not blocking on it). '
+      + 'Fold it into your work, or reply again to continue.',
     ].join('\n');
 
     try {
-      const result = await deliverInstruction(this.daemon, askerId, relay, { recordFeedback: false });
-      if (result.agentId !== askerId) {
-        this._remapParticipant(thread, askerId, result.agentId);
-        turn.to.id = result.agentId;
-      }
-      turn.status = result.status;
-      thread.status = 'idle';
+      await deliverInstruction(this.daemon, sender.id, msg, { recordFeedback: false });
     } catch (err) {
-      turn.status = 'failed';
-      turn.error = err.message;
-      thread.status = 'failed';
+      this.daemon.audit.log('innerchat.forward_failed', { to: sender.id, error: err.message });
     }
+  }
 
-    thread.updatedAt = Date.now();
-    this.daemon.broadcast({ type: 'innerchat:turn', data: { thread, turn } });
-    this.daemon.audit.log('innerchat.reply', { thread: thread.id, from: turn.from.id, to: turn.to.id });
+  /**
+   * An agent that dies mid-question would otherwise leave a blocking asker
+   * stuck until timeout. Async senders aren't waiting, so just clear those.
+   */
+  onAgentGone(agentId, reason = 'stopped') {
+    const pending = this.awaiting.get(agentId);
+    if (!pending) return;
+    if (pending.mode === 'async') { this._clearAwait(agentId); return; }
+    const agent = this.daemon.registry.get(agentId);
+    pending.reject(new Error(`${agent?.name || agentId} ${reason} before answering`));
   }
 
-  // ── Thread helpers ──────────────────────────────────────────
+  _clearAwait(targetId) {
+    const pending = this.awaiting.get(targetId);
+    if (pending?.timer) clearTimeout(pending.timer);
+    this.awaiting.delete(targetId);
+  }
+
+  _settlePending(agentId, pending, value, err) {
+    if (pending.mode === 'async') { this._clearAwait(agentId); return; }
+    if (err) pending.reject(err); else pending.resolve(value);
+  }
+
+  // ── Threads ─────────────────────────────────────────────────
 
   _createThread(fromAgent, toAgent) {
     const thread = {
@@ -178,25 +312,43 @@ export class InnerChat {
     return thread;
   }
 
-  // Rebuild the message the target sees. It reads as a direct message from the
-  // other agent, with enough prior turns that a continuing thread makes sense.
-  _wrap(thread, fromAgent, toAgent, message) {
+  // Consecutive asks between the same pair continue one conversation, so the
+  // exchange cap actually bounds a back-and-forth rather than resetting.
+  _findThread(a, b) {
+    return this.getThreads(a).find((t) => t.participants.some((p) => p.id === b)) || null;
+  }
+
+  // The target sees a direct message from the other agent, with enough prior
+  // turns to follow a continuing conversation. The closing instruction differs
+  // by kind: an ask blocks the sender (answer now), a tell does not (answer
+  // when you reach a good stopping point).
+  _wrap(thread, fromAgent, toAgent, message, kind = 'ask') {
+    const header = kind === 'tell'
+      ? `[InnerChat — ${fromAgent.name} (${fromAgent.role}) sent you a message]`
+      : `[InnerChat — ${fromAgent.name} (${fromAgent.role}) is asking you a question]`;
     const prior = thread.turns.slice(0, -1).slice(-CONTEXT_TURNS);
-    const lines = [`[InnerChat from ${fromAgent.name} (${fromAgent.role})]`, ''];
+    const lines = [header, ''];
 
     if (prior.length) {
       lines.push('Earlier in this conversation:');
-      for (const t of prior) {
-        lines.push(`  ${t.from.name}: ${truncate(t.text, MAX_TURN_CHARS)}`);
-      }
+      for (const t of prior) lines.push(`  ${t.from.name}: ${truncate(t.text, MAX_TURN_CHARS)}`);
       lines.push('');
     }
 
     lines.push(message, '');
-    lines.push(
-      `Reply normally — your response is relayed straight back to ${fromAgent.name}. `
-      + 'Answer them directly; do not address the user.',
-    );
+    if (kind === 'tell') {
+      lines.push(
+        `${fromAgent.name} is NOT blocked on this — finish what you're doing first if you're `
+        + 'mid-task. When you reach a good stopping point, answer directly in a message; your '
+        + `reply is relayed back to ${fromAgent.name}. If no reply is needed, just carry on.`,
+      );
+    } else {
+      lines.push(
+        `${fromAgent.name} is BLOCKED waiting on your answer — it cannot continue until you `
+        + 'respond. Answer directly and concisely in your next message; that message is sent '
+        + 'back to it verbatim. Do not address the user, and do not start unrelated work first.',
+      );
+    }
     return lines.join('\n');
   }
 
@@ -205,10 +357,7 @@ export class InnerChat {
   }
 
   _remapParticipant(thread, oldId, newId) {
-    for (const p of thread.participants) {
-      if (p.id === oldId) p.id = newId;
-    }
-    // Any relay still awaiting a reply from the old id must follow it forward.
+    for (const p of thread.participants) if (p.id === oldId) p.id = newId;
     const pending = this.awaiting.get(oldId);
     if (pending) {
       this.awaiting.delete(oldId);
@@ -216,6 +365,10 @@ export class InnerChat {
     }
   }
 
+  _broadcast(thread, turn) {
+    this.daemon.broadcast({ type: 'innerchat:turn', data: { thread, turn } });
+  }
+
   // ── Queries ─────────────────────────────────────────────────
 
   getThreads(agentId = null) {
@@ -232,6 +385,20 @@ export class InnerChat {
     const pending = this.awaiting.get(agentId);
     return pending ? this.threads.get(pending.threadId) : null;
   }
+
+  // Clear all outstanding timers so a shutdown (or a test) doesn't hang on
+  // pending ask/tell timeouts. Blocking asks are rejected so their HTTP
+  // requests don't hang either.
+  stop() {
+    for (const [targetId, pending] of this.awaiting) {
+      if (pending.timer) clearTimeout(pending.timer);
+      if (pending.mode === 'block') {
+        try { pending.reject(new Error('Daemon shutting down')); } catch { /* already settled */ }
+      }
+      this.awaiting.delete(targetId);
+    }
+    this.blockedOn.clear();
+  }
 }
 
 function peer(agent) {
@@ -249,3 +416,5 @@ function extractText(data) {
   }
   return '';
 }
+
+export { MAX_EXCHANGES, DEFAULT_TIMEOUT_MS };

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

@@ -4,6 +4,7 @@
 import { writeFileSync, readFileSync, existsSync, readdirSync, statSync } from 'fs';
 import { resolve, dirname, basename } from 'path';
 import { escapeMd } from './validate.js';
+import { innerChatInstructions, watchInstructions } from './innerchat-docs.js';
 
 const GROOVE_SECTION_START = '<!-- GROOVE:START -->';
 const GROOVE_SECTION_END = '<!-- GROOVE:END -->';
@@ -556,6 +557,13 @@ export class Introducer {
     return null;
   }
 
+  // Written into every AGENTS_REGISTRY.md so the capability survives context
+  // compaction — the spawn prompt alone can scroll out of a long session.
+  _innerChatSection() {
+    const port = this.daemon.port || 31415;
+    return [...innerChatInstructions(port), '', ...watchInstructions(port)];
+  }
+
   writeRegistryFile(projectDir) {
     const agents = this.daemon.registry.getAll();
 
@@ -596,6 +604,27 @@ export class Introducer {
         lines.push(`| ${escapeMd(a.id)} | ${escapeMd(a.name)} | ${escapeMd(a.role)} | ${escapeMd(a.provider)} | ${agentDir} | ${scope} | ${escapeMd(a.status)} |`);
       }
 
+      // Agents on other teams are reachable via InnerChat, so they have to be
+      // discoverable here — otherwise cross-team consultation means guessing
+      // names. Kept to a separate, minimal table so the team's own roster
+      // stays the primary listing.
+      const others = agents.filter((a) => (a.teamId || '_default') !== teamId);
+      if (others.length > 0) {
+        lines.push('');
+        lines.push(`## Other Teams`);
+        lines.push('');
+        lines.push(`Reachable via InnerChat (see below). Not in your scope — do not edit their files.`);
+        lines.push('');
+        lines.push(`| Name | Role | Team | Status |`);
+        lines.push(`|------|------|------|--------|`);
+        for (const a of others) {
+          const otherTeam = this.daemon.teams?.get(a.teamId)?.name || '-';
+          lines.push(`| ${escapeMd(a.name)} | ${escapeMd(a.role)} | ${escapeMd(otherTeam)} | ${escapeMd(a.status)} |`);
+        }
+      }
+
+      lines.push('');
+      lines.push(...this._innerChatSection());
       lines.push('');
       lines.push(`*Updated: ${new Date().toISOString()}*`);
 
@@ -632,7 +661,8 @@ export class Introducer {
       running.length > 0 ? '|------|------|-------|' : '',
       ...running.map((a) => `| ${a.name} | ${a.role} | ${a.scope?.join(', ') || '-'} |`),
       '',
-      `See AGENTS_REGISTRY.md for full agent state.`,
+      `See AGENTS_REGISTRY.md for full agent state, the names of agents on other teams,`,
+      `and how to consult them directly (InnerChat).`,
       '',
       `**Memory policy:** GROOVE manages project memory automatically. Do not read or write MEMORY.md or .groove/memory/ files directly.`,
       '',

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

@@ -10,6 +10,7 @@ import { LocalProvider } from './providers/local.js';
 import { OllamaProvider } from './providers/ollama.js';
 import { AgentLoop } from './agent-loop.js';
 import { validateAgentConfig } from './validate.js';
+import { innerChatInstructions, watchInstructions } from './innerchat-docs.js';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const SLIDES_ENGINE_SRC = resolve(__dirname, '../templates/groove-slides.cjs');
@@ -506,6 +507,13 @@ export class ProcessManager {
 
     registry.update(agent.id, { status: finalStatus, pid: null });
 
+    // An agent that dies mid-question leaves its asker blocked on an open HTTP
+    // request — fail it now rather than making it wait out the timeout.
+    // 'completed' is normal: the answer arrives as a result before this fires.
+    if (this.daemon.innerchat && finalStatus !== 'completed') {
+      this.daemon.innerchat.onAgentGone(agent.id, finalStatus);
+    }
+
     if (this.daemon.timeline) {
       const agentData = registry.get(agent.id);
       this.daemon.timeline.recordEvent(finalStatus === 'completed' ? 'complete' : finalStatus === 'crashed' ? 'crash' : 'kill', {
@@ -554,12 +562,17 @@ export class ProcessManager {
       const pending = this.consumePendingMessage(agent.id);
       if (pending) {
         const agentData = registry.get(agent.id);
-        if (agentData?.sessionId) {
-          this.resume(agent.id, pending.message).catch((err) => {
-            console.error(`[Groove] Auto-resume with queued message failed for ${agent.name}: ${err.message}`);
-          });
-          return;
-        }
+        // Deliver the queued message: resume if there's a session, otherwise
+        // rotate (fresh session + handoff brief). Previously a missing sessionId
+        // meant the already-consumed message was silently dropped — an
+        // InnerChat message to a busy agent would vanish and never fire it.
+        const deliver = agentData?.sessionId
+          ? this.resume(agent.id, pending.message)
+          : this.daemon.rotator.rotate(agent.id, { additionalPrompt: pending.message });
+        deliver.catch((err) => {
+          console.error(`[Groove] Delivering queued message to ${agent.name} failed: ${err.message}`);
+        });
+        return;
       }
     }
 
@@ -1152,6 +1165,23 @@ For normal file edits within your scope, proceed without review.
       }
     }
 
+    // InnerChat — let the agent consult other agents directly.
+    // Same constraint as PM review: the agent calls back over HTTP, so
+    // sandboxed providers (Codex) can't reach it.
+    if (!sandboxedProviders.includes(providerName) && !isOneShotProvider) {
+      const port = this.daemon.port || 31415;
+      const capabilities = [
+        ...innerChatInstructions(port, agent.name),
+        '',
+        ...watchInstructions(port, agent.name),
+      ].join('\n') + '\n\n';
+      if (spawnConfig.prompt.startsWith('# Handoff Brief')) {
+        spawnConfig.prompt += '\n\n' + capabilities.trim();
+      } else {
+        spawnConfig.prompt = capabilities + spawnConfig.prompt;
+      }
+    }
+
     // Set up log capture (shared between CLI and agent loop paths)
     const logDir = resolve(this.daemon.grooveDir, 'logs');
     mkdirSync(logDir, { recursive: true });