@xfxstudio/claworld 0.2.23-beta.0 → 0.2.23-beta.1

package/openclaw.plugin.json

@@ -8,7 +8,7 @@
   ],
   "name": "Claworld Persona Relay",
   "description": "Claworld relay world channel plugin for OpenClaw.",
-  "version": "0.2.23-beta.0",
+  "version": "0.2.23-beta.1",
   "configSchema": {
     "type": "object",
     "additionalProperties": false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xfxstudio/claworld",
-  "version": "0.2.23-beta.0",
+  "version": "0.2.23-beta.1",
   "description": "Claworld channel plugin for OpenClaw",
   "type": "module",
   "main": "index.js",

package/src/openclaw/plugin/claworld-channel-plugin.js

@@ -174,6 +174,20 @@ const CLAWORLD_RELAY_OPERATIONAL_NOTICE_PATTERNS = [
   /^⚠️\s*Agent failed before reply:/i,
 ];
 
+// Older/runtime-variant OpenClaw hosts may surface provider/runtime failures as
+// plain final text without setting `isError`. Keep this fallback at the bridge
+// boundary so business logic never has to guess.
+const CLAWORLD_RELAY_RUNTIME_ERROR_PATTERNS = [
+  /^⚠️\s*Agent failed before reply:/i,
+  /^LLM request failed:/i,
+  /^LLM request timed out\./i,
+  /^LLM request unauthorized\./i,
+  /^The AI service is temporarily overloaded\./i,
+  /^The AI service returned an error\./i,
+  /^⚠️\s*API rate limit reached\./i,
+  /^⚠️\s*.+\s+returned a billing error\b/i,
+];
+
 const CLAWORLD_RELAY_OPERATIONAL_SUFFIX_PATTERNS = [
   /^Usage:\s+.+\s+in\s+\/\s+.+\s+out(?:\s+·\s+est\s+.+)?$/i,
 ];
@@ -201,18 +215,21 @@ function classifyRelayContinuationText(text) {
   if (!normalized) {
     return {
       text: '',
-      operational: Boolean(String(text || '').trim()),
+      operationalNotice: Boolean(String(text || '').trim()),
+      runtimeError: false,
     };
   }
   if (CLAWORLD_RELAY_OPERATIONAL_NOTICE_PATTERNS.some((pattern) => pattern.test(normalized))) {
     return {
       text: '',
-      operational: true,
+      operationalNotice: true,
+      runtimeError: false,
     };
   }
   return {
     text: normalized,
-    operational: false,
+    operationalNotice: false,
+    runtimeError: false,
   };
 }
 
@@ -220,6 +237,45 @@ function sanitizeRelayContinuationText(text) {
   return classifyRelayContinuationText(text).text;
 }
 
+function classifyRelayContinuationPayload(payload = {}) {
+  const rawText = String(payload?.text ?? payload?.body ?? '').trim();
+  const normalized = stripRelayOperationalSuffix(rawText);
+  const textClassification = classifyRelayContinuationText(rawText);
+  const runtimeError = payload?.isError === true
+    || CLAWORLD_RELAY_RUNTIME_ERROR_PATTERNS.some((pattern) => pattern.test(normalized));
+  if (runtimeError) {
+    return {
+      text: '',
+      previewText: normalized,
+      operationalNotice: false,
+      runtimeError: true,
+      nonRenderable: true,
+    };
+  }
+  return {
+    text: textClassification.text,
+    previewText: normalized,
+    operationalNotice: textClassification.operationalNotice,
+    runtimeError: false,
+    nonRenderable: textClassification.operationalNotice,
+  };
+}
+
+function resolveRelaySilentReason(runtimeOutputSummary = {}, continuation = {}) {
+  const counts = runtimeOutputSummary?.counts || {};
+  if (Number(counts.runtimeErrorFinal || 0) > 0) {
+    return 'runtime_failed_before_reply';
+  }
+  if (Number(counts.operationalNotice || 0) > 0 && Number(counts.nonRenderableFinal || 0) === Number(counts.final || 0)) {
+    return 'operational_notice_only';
+  }
+  const normalizedSource = normalizePluginOptionalText(continuation?.source);
+  if (normalizedSource && normalizedSource !== 'none') {
+    return normalizedSource;
+  }
+  return 'no_renderable_reply';
+}
+
 function previewRuntimeOutputText(text, maxLength = 120) {
   const normalized = String(text || '').replace(/\s+/g, ' ').trim();
   if (!normalized) return '';
@@ -1269,7 +1325,9 @@ function createDeliveryReplyDispatcher({
       reasoningEnd: 0,
       compactionStart: 0,
       compactionEnd: 0,
+      nonRenderableFinal: 0,
       operationalNotice: 0,
+      runtimeErrorFinal: 0,
     },
     previews: {
       final: [],
@@ -1278,6 +1336,7 @@ function createDeliveryReplyDispatcher({
       partial: [],
       reasoning: [],
       operationalNotice: [],
+      runtimeErrorFinal: [],
     },
     relayContinuationSource: 'none',
     relayContinuationPreview: null,
@@ -1288,14 +1347,21 @@ function createDeliveryReplyDispatcher({
     runtimeOutputSummary.counts[kind] += 1;
     const text = String(payload?.text ?? payload?.body ?? '').trim();
     if (kind === 'final') {
-      if (text) {
-        finalTexts.push(text);
-        appendRuntimeOutputPreview(runtimeOutputSummary.previews.final, text);
-        const classified = classifyRelayContinuationText(text);
-        if (classified.operational) {
-          runtimeOutputSummary.counts.operationalNotice += 1;
-          appendRuntimeOutputPreview(runtimeOutputSummary.previews.operationalNotice, text);
-        }
+      const classified = classifyRelayContinuationPayload(payload);
+      if (classified.text) {
+        finalTexts.push(classified.text);
+        appendRuntimeOutputPreview(runtimeOutputSummary.previews.final, classified.text);
+      }
+      if (classified.nonRenderable) {
+        runtimeOutputSummary.counts.nonRenderableFinal += 1;
+      }
+      if (classified.operationalNotice) {
+        runtimeOutputSummary.counts.operationalNotice += 1;
+        appendRuntimeOutputPreview(runtimeOutputSummary.previews.operationalNotice, classified.previewText || text);
+      }
+      if (classified.runtimeError) {
+        runtimeOutputSummary.counts.runtimeErrorFinal += 1;
+        appendRuntimeOutputPreview(runtimeOutputSummary.previews.runtimeErrorFinal, classified.previewText || text);
       }
       return;
     }
@@ -1411,21 +1477,35 @@ function createDeliveryReplyDispatcher({
   const markDispatchIdle = async () => {
     await dispatchApi.dispatcher.waitForIdle?.();
     if (!replied && !suppressed) {
-      const continuation = buildRelayContinuationText({
+      const allowPartialFallback = (
+        runtimeOutputSummary.counts.final > 0
+        && finalTexts.length === 0
+        && blockTexts.length === 0
+        && runtimeOutputSummary.counts.nonRenderableFinal === 0
+      );
+      const safeContinuation = buildRelayContinuationText({
         finalTexts,
         blockTexts,
         partialText: partialContinuationText,
-        allowPartialFallback:
-          runtimeOutputSummary.counts.final > 0 && finalTexts.length === 0 && blockTexts.length === 0,
+        allowPartialFallback,
       });
-      runtimeOutputSummary.relayContinuationSource = continuation.source;
-      runtimeOutputSummary.relayContinuationPreview = continuation.text
-        ? previewRuntimeOutputText(continuation.text)
+      runtimeOutputSummary.relayContinuationSource = safeContinuation.source;
+      runtimeOutputSummary.relayContinuationPreview = safeContinuation.text
+        ? previewRuntimeOutputText(safeContinuation.text)
         : null;
-      if (continuation.text) {
-        await flushReply(continuation.text);
+      if (safeContinuation.text) {
+        await flushReply(safeContinuation.text);
       } else {
-        await flushKeptSilent(continuation.source);
+        const silentReason = resolveRelaySilentReason(runtimeOutputSummary, safeContinuation);
+        if (runtimeOutputSummary.counts.runtimeErrorFinal > 0) {
+          logger.warn?.(`[claworld:${runtimeAccountId}] runtime produced non-renderable error finals; returning kept_silent`, {
+            deliveryId,
+            sessionKey,
+            localAgentId,
+            runtimeOutputSummary,
+          });
+        }
+        await flushKeptSilent(silentReason);
       }
     }
     await dispatchApi.markDispatchIdle?.();
@@ -1470,13 +1550,14 @@ function createDeliveryReplyDispatcher({
         final: [...runtimeOutputSummary.previews.final],
         block: [...runtimeOutputSummary.previews.block],
         tool: [...runtimeOutputSummary.previews.tool],
-        partial: [...runtimeOutputSummary.previews.partial],
-        reasoning: [...runtimeOutputSummary.previews.reasoning],
-        operationalNotice: [...runtimeOutputSummary.previews.operationalNotice],
-      },
-      relayContinuationSource: runtimeOutputSummary.relayContinuationSource,
-      relayContinuationPreview: runtimeOutputSummary.relayContinuationPreview,
-      replyTransport,
+      partial: [...runtimeOutputSummary.previews.partial],
+      reasoning: [...runtimeOutputSummary.previews.reasoning],
+      operationalNotice: [...runtimeOutputSummary.previews.operationalNotice],
+      runtimeErrorFinal: [...runtimeOutputSummary.previews.runtimeErrorFinal],
+    },
+    relayContinuationSource: runtimeOutputSummary.relayContinuationSource,
+    relayContinuationPreview: runtimeOutputSummary.relayContinuationPreview,
+    replyTransport,
       replyFallbackUsed,
       keptSilentTransport,
       keptSilentFallbackUsed,
@@ -1749,8 +1830,8 @@ async function maybeBridgeRuntimeDelivery({
     && metadata.allowReply !== false
     && replied !== true
     && runtimeOutputSummary.counts.final > 0
-    && runtimeOutputSummary.counts.operationalNotice > 0
-    && runtimeOutputSummary.counts.final === runtimeOutputSummary.counts.operationalNotice
+    && runtimeOutputSummary.counts.nonRenderableFinal > 0
+    && runtimeOutputSummary.counts.final === runtimeOutputSummary.counts.nonRenderableFinal
     && runtimeOutputSummary.counts.block === 0
     && runtimeOutputSummary.counts.tool === 0
     && runtimeOutputSummary.counts.partial === 0