@clawling/clawchat-plugin-openclaw 2026.5.12-39 → 2026.5.13-dev.1

package/src/reply-dispatcher.ts

@@ -19,6 +19,7 @@ import {
 } from "./config.ts";
 import { uploadOutboundMedia, type ClawlingMediaFragment } from "./media-runtime.ts";
 import {
+  mintMessageId,
   sendOpenclawClawlingText,
   type OutboundReplyCtx,
   type SendResult,
@@ -388,8 +389,7 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
   } = options;
   const isGroupTarget = target.chatType === "group";
   const outputVisibility = effectiveOutputVisibility(account, target.chatId, target.chatType);
-  const splitFullOutput = outputVisibility === "full";
-  const splitNormalBlockOutput = outputVisibility === "normal";
+  const splitFullOutput = outputVisibility === "full" && !isGroupTarget;
   const ownerDirectTarget = () => {
     const ownerUserId = account.ownerUserId?.trim();
     return ownerUserId ? { chatId: ownerUserId, chatType: "direct" as const } : null;
@@ -543,8 +543,10 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
     return merged;
   };
 
-  const mintStaticMessageId = () =>
-    `${account.userId}-msg-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+  // §7.6 (binding): client-supplied message_id MUST be `msg-` + a 26-char
+  // Crockford base32 ULID. Use the shared conformant minter — do not invent a
+  // different scheme (planned msghub Phase-4 validation will reject it).
+  const mintStaticMessageId = () => mintMessageId();
 
   const emitTyping = (isTyping: boolean) => {
     if (!isTyping && !typingActive) return;
@@ -671,12 +673,12 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
   };
 
   const emitFullSegment = async (text: string, urls: string[] = []): Promise<void> => {
-    if (outputVisibility !== "full" && !splitNormalBlockOutput) {
+    if (outputVisibility !== "full") {
       appendBufferedText(text);
       appendBufferedUrls(urls);
       return;
     }
-    if (!splitFullOutput && !splitNormalBlockOutput) {
+    if (!splitFullOutput) {
       appendBufferedText(text);
       appendBufferedUrls(urls);
       return;
@@ -751,8 +753,6 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
       if (info?.kind === "block") {
         if (outputVisibility === "full") {
           await emitFullSegment(text, urls);
-        } else if (splitNormalBlockOutput) {
-          await emitFullSegment(text, urls);
         } else if (outputVisibility === "minimal" || outputVisibility === "normal") {
           appendBufferedText(text);
           appendBufferedUrls(urls);
@@ -785,14 +785,6 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
         }
         const finalText = richFragment && account.richInteractions ? mergeFinalText("") : mergeFinalText(text);
         const finalUrls = mergeFinalUrls(urls);
-        if (
-          isClawChatNoopResponseText(finalText) &&
-          !richFragment &&
-          finalUrls.length === 0
-        ) {
-          log?.info?.(`[${account.accountId}] clawchat-plugin-openclaw final suppressed: no-reply token`);
-          return;
-        }
         const mediaFragments = await uploadMediaUrls(finalUrls);
         const result = await sendStatic(
           finalText,
@@ -811,7 +803,13 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
       log?.error?.(
         `[${account.accountId}] clawchat-plugin-openclaw ${info.kind} reply failed: ${errorText}`,
       );
-      if (outputVisibility === "full") void emitFullRuntimeText("error", errorText);
+      if (!isGroupTarget && outputVisibility === "full") void emitFullRuntimeText("error", errorText);
+      if (isGroupTarget) {
+        log?.error?.(
+          `[${account.accountId}] clawchat-plugin-openclaw group runtime failure suppressed from ClawChat clients group=${target.chatId}`,
+        );
+        return;
+      }
     },
     onIdle: async () => {
       emitTyping(false);
@@ -835,10 +833,10 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
     replyOptions: {
       ...base.replyOptions,
       sourceReplyDeliveryMode: "automatic",
-      disableBlockStreaming: !splitNormalBlockOutput,
+      disableBlockStreaming: true,
       suppressDefaultToolProgressMessages: true,
-      allowProgressCallbacksWhenSourceDeliverySuppressed: splitFullOutput ? true : undefined,
-      onReasoningStream: splitFullOutput
+      allowProgressCallbacksWhenSourceDeliverySuppressed: outputVisibility === "full" ? true : undefined,
+      onReasoningStream: outputVisibility === "full"
         ? async (payload: ReplyPayload) => {
             if (consumeTerminalSend("reasoning")) return;
             const text = resolvePayloadText(payload);
@@ -847,13 +845,13 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
             if (trimmed) reasoningText = reasoningText ? `${reasoningText}\n${trimmed}` : trimmed;
           }
         : undefined,
-      onToolStart: splitFullOutput
+      onToolStart: outputVisibility === "full"
         ? async (payload) => {
             if (consumeTerminalSend("tool-start")) return;
             await emitFullSegment(formatToolStartSummary(payload));
           }
         : undefined,
-      onToolResult: splitFullOutput
+      onToolResult: outputVisibility === "full"
         ? async (payload: ReplyPayload) => {
             if (consumeTerminalSend("tool-result")) return;
             const text = resolvePayloadText(payload);
@@ -861,38 +859,38 @@ export function createOpenclawClawlingReplyDispatcher(options: ReplyDispatcherOp
             await emitFullRuntimeText("tool result", text, resolveOutboundMediaUrls(payload).filter(Boolean));
           }
         : undefined,
-      onItemEvent: splitFullOutput
+      onItemEvent: outputVisibility === "full"
         ? async (payload: Record<string, unknown>) => {
             if (consumeTerminalSend("item-event")) return;
             if (isToolProgressItem(payload)) return;
             await emitFullRuntimeText("progress", summarizeProgressPayload(payload));
           }
         : undefined,
-      onPlanUpdate: splitFullOutput
+      onPlanUpdate: outputVisibility === "full"
         ? async (payload: Record<string, unknown>) => {
             if (consumeTerminalSend("plan-update")) return;
             await emitFullRuntimeText("plan", summarizeProgressPayload(payload));
           }
         : undefined,
-      onCommandOutput: splitFullOutput
+      onCommandOutput: outputVisibility === "full"
         ? async (payload: Record<string, unknown>) => {
             if (consumeTerminalSend("command-output")) return;
             await emitFullSegment(formatCommandOutputSummary(payload));
           }
         : undefined,
-      onPatchSummary: splitFullOutput
+      onPatchSummary: outputVisibility === "full"
         ? async (payload: Record<string, unknown>) => {
             if (consumeTerminalSend("patch-summary")) return;
             await emitFullSegment(formatPatchSummary(payload));
           }
         : undefined,
-      onCompactionStart: splitFullOutput
+      onCompactionStart: outputVisibility === "full"
         ? async () => {
             if (consumeTerminalSend("compaction-start")) return;
             await emitFullSegment("[compaction] started");
           }
         : undefined,
-      onCompactionEnd: splitFullOutput
+      onCompactionEnd: outputVisibility === "full"
         ? async () => {
             if (consumeTerminalSend("compaction-end")) return;
             await emitFullSegment("[compaction] finished");