@yaoyuanchao/dingtalk 1.3.2 → 1.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yaoyuanchao/dingtalk",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "type": "module",
   "description": "DingTalk channel plugin for Clawdbot with Stream Mode support",
   "license": "MIT",

package/src/api.ts

@@ -93,12 +93,16 @@ export async function getDingTalkAccessToken(clientId: string, clientSecret: str
 export async function sendViaSessionWebhook(
   sessionWebhook: string,
   text: string,
-): Promise<{ ok: boolean }> {
+): Promise<{ ok: boolean; errcode?: number; errmsg?: string }> {
   const res = await jsonPost(sessionWebhook, {
     msgtype: "text",
     text: { content: text },
   });
-  return { ok: res?.errcode === 0 || !res?.errcode };
+  const ok = res?.errcode === 0 || !res?.errcode;
+  if (!ok) {
+    console.warn(`[dingtalk] SessionWebhook text error: errcode=${res?.errcode}, errmsg=${res?.errmsg}`);
+  }
+  return { ok, errcode: res?.errcode, errmsg: res?.errmsg };
 }
 
 /** Send markdown via sessionWebhook */
@@ -106,12 +110,16 @@ export async function sendMarkdownViaSessionWebhook(
   sessionWebhook: string,
   title: string,
   text: string,
-): Promise<{ ok: boolean }> {
+): Promise<{ ok: boolean; errcode?: number; errmsg?: string }> {
   const res = await jsonPost(sessionWebhook, {
     msgtype: "markdown",
     markdown: { title, text },
   });
-  return { ok: res?.errcode === 0 || !res?.errcode };
+  const ok = res?.errcode === 0 || !res?.errcode;
+  if (!ok) {
+    console.warn(`[dingtalk] SessionWebhook markdown error: errcode=${res?.errcode}, errmsg=${res?.errmsg}`);
+  }
+  return { ok, errcode: res?.errcode, errmsg: res?.errmsg };
 }
 
 /** Send image via sessionWebhook using markdown format */

package/src/monitor.ts

@@ -577,10 +577,13 @@ async function processInboundMessage(
         cfg: actualCfg,
         dispatcherOptions: {
           deliver: async (payload: any) => {
+            log?.info?.("[dingtalk] Deliver payload keys: " + Object.keys(payload || {}).join(',') + " text?=" + (typeof payload?.text) + " markdown?=" + (typeof payload?.markdown));
             const textToSend = resolveDeliverText(payload, log);
             if (textToSend) {
               await deliverReply(replyTarget, textToSend, log);
               setStatus?.({ lastOutboundAt: Date.now() });
+            } else {
+              log?.info?.("[dingtalk] Deliver: no text resolved from payload");
             }
           },
           onError: (err: any) => {
@@ -680,8 +683,12 @@ async function dispatchWithFullPipeline(params: {
     responsePrefix: '',
     deliver: async (payload: any) => {
       try {
+        log?.info?.("[dingtalk] Pipeline deliver payload keys: " + Object.keys(payload || {}).join(',') + " text?=" + (typeof payload?.text) + " markdown?=" + (typeof payload?.markdown));
         const textToSend = resolveDeliverText(payload, log);
-        if (!textToSend) return { ok: true };
+        if (!textToSend) {
+          log?.info?.("[dingtalk] Pipeline deliver: no text resolved from payload");
+          return { ok: true };
+        }
         await deliverReply(replyTarget, textToSend, log);
         setStatus?.({ lastOutboundAt: Date.now() });
         return { ok: true };
@@ -709,7 +716,15 @@ async function dispatchWithFullPipeline(params: {
  * We merge them into the text as markdown image syntax so DingTalk can render them.
  */
 function resolveDeliverText(payload: any, log?: any): string | undefined {
-  let text = payload.markdown || payload.text;
+  // payload.markdown may be a boolean flag (not the actual text), so check type
+  let text = (typeof payload.markdown === 'string' && payload.markdown) || payload.text;
+
+  // Guard: ensure text is a string (platform might send unexpected types)
+  if (text != null && typeof text !== 'string') {
+    log?.info?.("[dingtalk] Deliver payload has non-string text type=" + typeof text + ", payload keys=" + Object.keys(payload).join(','));
+    text = String(text);
+  }
+
   const mediaUrl = payload.mediaUrl || payload.media || payload.imageUrl || payload.image;
 
   if (mediaUrl && typeof mediaUrl === 'string' && mediaUrl.startsWith('http')) {
@@ -762,13 +777,17 @@ async function deliverReply(target: any, text: string, log?: any): Promise<void>
       for (let attempt = 1; attempt <= maxRetries; attempt++) {
         try {
           log?.info?.("[dingtalk] Using sessionWebhook (attempt " + attempt + "/" + maxRetries + "), format=" + messageFormat);
-          log?.info?.("[dingtalk] Sending text: " + chunk.substring(0, 200));
+          log?.info?.("[dingtalk] Sending text (" + chunk.length + " chars): " + chunk.substring(0, 200));
+          let sendResult: { ok: boolean; errcode?: number; errmsg?: string };
           if (isMarkdown) {
-            await sendMarkdownViaSessionWebhook(target.sessionWebhook, "Reply", chunk);
+            sendResult = await sendMarkdownViaSessionWebhook(target.sessionWebhook, "Reply", chunk);
           } else {
-            await sendViaSessionWebhook(target.sessionWebhook, chunk);
+            sendResult = await sendViaSessionWebhook(target.sessionWebhook, chunk);
+          }
+          if (!sendResult.ok) {
+            throw new Error(`SessionWebhook rejected: errcode=${sendResult.errcode}, errmsg=${sendResult.errmsg}`);
           }
-          log?.info?.("[dingtalk] SessionWebhook send OK");
+          log?.info?.("[dingtalk] SessionWebhook send OK (errcode=" + (sendResult.errcode ?? 0) + ")");
           webhookSuccess = true;
           break;
         } catch (err) {