@newbase-clawchat/openclaw-clawchat 2026.4.15 → 2026.4.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@newbase-clawchat/openclaw-clawchat",
-  "version": "2026.4.15",
+  "version": "2026.4.20",
   "description": "OpenClaw ClawChat channel plugin",
   "files": [
     "index.ts",

package/src/reply-dispatcher.ts

@@ -58,17 +58,20 @@ type StreamingReplyHooks = {
  * Reply dispatcher for openclaw-clawchat.
  *
  * Streaming mode (`account.replyMode === "stream"`, no replyCtx):
- *   1. `onReplyStart` opens a buffered streaming session — `message.created`
- *      fires immediately.
- *   2. As the agent runs, `onPartialReply` / `onReasoningStream` snapshots
- *      and `deliver(block|tool)` deltas feed into the session; each flush
- *      emits `message.add` with the new text delta (chunked by
+ *   1. `onReplyStart` just resets accumulators. It does NOT open the session
+ *      yet — that would leave a ghost `message.created + done` bubble if the
+ *      run produces nothing (e.g., agent not configured, send-policy denied).
+ *   2. On first real content (deliver block/tool or partial/reasoning
+ *      snapshot), `queueStreamSnapshot` lazily opens the session, which emits
+ *      `message.created` plus the first `message.add`.
+ *   3. Subsequent snapshots/deltas emit `message.add` frames (chunked by
  *      `stream.flushIntervalMs` + `stream.minChunkChars`).
- *   3. On run end (`onIdle`), the session flushes pending buffer, emits
+ *   4. On run end (`onIdle`), the session flushes pending buffer, emits
  *      `message.done`, then the merged full text plus any accumulated
  *      media is emitted as a separate `message.send` / `message.reply` —
  *      mirroring the clawling-channel v1 pattern where streaming `agent`
- *      events are followed by a consolidated `chat` final.
+ *      events are followed by a consolidated `chat` final. Empty runs emit
+ *      nothing at all (no created/done/reply) — they only log a skip line.
  *
  * Static mode or replyCtx: bypass streaming and emit one `message.send` /
  * `message.reply` per deliver with text + media.
@@ -133,10 +136,14 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
   const openSessionIfNeeded = () => {
     if (!streamingEnabled || streamingSession || streamCreatedEmitted) return;
     streamCreatedEmitted = true;
-    // Use the inbound message_id when available so the streaming frames
-    // correlate with the user message that triggered this reply; fall back
-    // to a generated id only when not provided (e.g. tests).
-    streamingMessageId = inboundMessageId ?? `${account.userId}-${Date.now()}`;
+    // Mint a fresh agent-side message_id at `message.created` time. All
+    // subsequent `message.add` / `message.done` / `message.reply` frames for
+    // this stream reuse it. Once the stream finalizes (done or reply), this
+    // id is retired — the next inbound turn spawns a new dispatcher instance
+    // which mints its own id. The inbound user message_id lives on
+    // `replyTo.msgId`; keeping the two distinct avoids the agent's reply
+    // frames shadowing the user turn they answer.
+    streamingMessageId = `${account.userId}-stream-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
     streamingSession = openBufferedStreamingSession({
       client,
       routing,
@@ -269,6 +276,12 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
       // If `onReplyStart` fires again during the same dispatcher instance
       // (e.g. typing controller re-entry), we must NOT tear down the stream
       // session — that would cause a second `message.created`.
+      //
+      // We deliberately do NOT open the streaming session here. Opening it
+      // eagerly would emit `message.created` even for runs that ultimately
+      // produce nothing (unknown agent, send-policy denied, etc.), leaving a
+      // ghost empty bubble. The session opens lazily via queueStreamSnapshot
+      // on the first real content.
       if (!streamCreatedEmitted) {
         streamText = "";
         reasoningText = "";
@@ -277,7 +290,6 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
         streamingClosed = false;
         runDone = false;
       }
-      if (streamingEnabled) openSessionIfNeeded();
     },
     deliver: async (payload: ReplyPayload, info?: { kind: "tool" | "block" | "final" }) => {
       const text = payload.text ?? "";