@searchfe/openclaw-baiduapp 0.1.0-beta.1 → 0.1.0-beta.3

package/dist/index.js

@@ -4231,7 +4231,16 @@ async function getAccessToken(account) {
   }
   const url = `${TOKEN_URL}?appKey=${encodeURIComponent(account.appKey)}&appSecret=${encodeURIComponent(account.appSecret)}`;
   const resp = await fetch(url);
-  const data = await resp.json();
+  const text = await resp.text();
+  if (!text) {
+    throw new Error(`gettoken failed: empty response (status=${resp.status})`);
+  }
+  let data;
+  try {
+    data = JSON.parse(text);
+  } catch {
+    throw new Error(`gettoken failed: invalid JSON response (status=${resp.status}, body=${text.slice(0, 200)})`);
+  }
   if (data.errcode !== void 0 && data.errcode !== 0) {
     throw new Error(`gettoken failed: ${data.errmsg ?? "unknown error"} (errcode=${data.errcode})`);
   }
@@ -4270,7 +4279,24 @@ async function sendBaiduAppMessage(account, target, message) {
     body: JSON.stringify(payload),
     headers: { "Content-Type": "application/json" }
   });
-  const data = await resp.json();
+  const text = await resp.text();
+  if (!text) {
+    return {
+      ok: false,
+      errcode: resp.status,
+      errmsg: `empty response from server (status=${resp.status})`
+    };
+  }
+  let data;
+  try {
+    data = JSON.parse(text);
+  } catch {
+    return {
+      ok: false,
+      errcode: resp.status,
+      errmsg: `invalid JSON response (status=${resp.status}, body=${text.slice(0, 200)})`
+    };
+  }
   return {
     ok: data.errcode === 0,
     errcode: data.errcode,
@@ -4308,8 +4334,9 @@ async function dispatchBaiduAppMessage(params) {
   const accountConfig = account?.config ?? {};
   const dmPolicy = resolveDmPolicy(accountConfig);
   const allowFrom = resolveAllowFrom(accountConfig);
+  logger.debug(`dispatch: sender=${senderId} dmPolicy=${dmPolicy} allowFrom=[${allowFrom.join(",")}]`);
   if (dmPolicy === "disabled") {
-    logger.debug("policy rejected: dm disabled");
+    logger.info(`dispatch rejected: dm disabled for account ${account.accountId}`);
     return;
   }
   const policyResult = checkDmPolicy({
@@ -4318,12 +4345,12 @@ async function dispatchBaiduAppMessage(params) {
     allowFrom
   });
   if (!policyResult.allowed) {
-    logger.debug(`policy rejected: ${policyResult.reason}`);
+    logger.info(`dispatch rejected: policy=${dmPolicy} reason=${policyResult.reason} sender=${senderId}`);
     return;
   }
   const channel = core.channel;
   if (!channel?.routing?.resolveAgentRoute || !channel.reply?.dispatchReplyWithBufferedBlockDispatcher) {
-    logger.debug("core routing or buffered dispatcher missing, skipping dispatch");
+    logger.warn("core routing or buffered dispatcher missing, skipping dispatch");
     return;
   }
   const route = channel.routing.resolveAgentRoute({
@@ -4332,7 +4359,13 @@ async function dispatchBaiduAppMessage(params) {
     accountId: account.accountId,
     peer: { kind: "dm", id: chatId }
   });
+  logger.info(
+    `route resolved: sessionKey=${route.sessionKey} agentId=${route.agentId ?? "default"} accountId=${route.accountId}`
+  );
   const rawBody = extractBaiduAppContent(msg);
+  logger.debug(
+    `message content extracted: len=${rawBody.length} preview="${rawBody.slice(0, 80)}${rawBody.length > 80 ? "..." : ""}"`
+  );
   const fromLabel = `user:${senderId}`;
   const storePath = channel.session?.resolveStorePath?.(safeCfg.session?.store, {
     agentId: route.agentId
@@ -4398,6 +4431,7 @@ async function dispatchBaiduAppMessage(params) {
     });
   }
   const tableMode = channel.text?.resolveMarkdownTableMode ? channel.text.resolveMarkdownTableMode({ cfg: safeCfg, channel: "baidu-app", accountId: account.accountId }) : void 0;
+  logger.info(`dispatching to agent: sessionKey=${route.sessionKey} sender=${senderId}`);
   await channel.reply.dispatchReplyWithBufferedBlockDispatcher({
     ctx: ctxPayload,
     cfg: safeCfg,
@@ -4405,9 +4439,11 @@ async function dispatchBaiduAppMessage(params) {
       deliver: async (payload) => {
         const rawText = payload.text ?? "";
         if (!rawText.trim()) {
+          logger.debug("deliver callback: empty text, skipping");
           return;
         }
         const converted = channel.text?.convertMarkdownTables && tableMode ? channel.text.convertMarkdownTables(rawText, tableMode) : rawText;
+        logger.debug(`deliver callback: textLen=${converted.length}`);
         hooks.onChunk(converted);
       },
       onError: (err, info) => {
@@ -4416,6 +4452,7 @@ async function dispatchBaiduAppMessage(params) {
       }
     }
   });
+  logger.info(`agent reply dispatch complete: sessionKey=${route.sessionKey} sender=${senderId}`);
 }
 function decodeEncodingAESKey(encodingAESKey) {
   const trimmed = encodingAESKey.trim();
@@ -4863,14 +4900,14 @@ async function handleBaiduAppWebhookRequest(req, res) {
     msgSignature = xmlData.MsgSignature ?? signature;
     msgTimestamp = xmlData.TimeStamp ?? timestamp;
     msgNonce = xmlData.Nonce ?? nonce;
-    logger.info(
-      `[baidu-app] inbound xml parsed: hasEncrypt=${Boolean(encrypt)}, msg_signature=${msgSignature ? "yes" : "no"}`
-    );
+    logger.info(`inbound xml parsed: hasEncrypt=${Boolean(encrypt)}, msg_signature=${msgSignature ? "yes" : "no"}`);
   } else {
     try {
       const record = JSON.parse(rawBody);
       encrypt = String(record.encrypt ?? record.Encrypt ?? "");
+      logger.info(`inbound json parsed: hasEncrypt=${Boolean(encrypt)}`);
     } catch {
+      logger.warn(`inbound payload parse failed: not valid xml or json`);
       res.statusCode = 400;
       res.end("invalid payload format");
       return true;
@@ -4894,12 +4931,15 @@ async function handleBaiduAppWebhookRequest(req, res) {
     });
   });
   if (signatureMatched.length === 0) {
+    logger.warn(`signature verification failed: checked ${targets.length} account(s), none matched`);
     res.statusCode = 401;
     res.end("unauthorized");
     return true;
   }
+  logger.debug(`signature verified: ${signatureMatched.length} account(s) matched`);
   const decryptable = signatureMatched.filter((candidate) => Boolean(candidate.account.encodingAESKey));
   if (decryptable.length === 0) {
+    logger.warn(`no account has encodingAESKey configured`);
     res.statusCode = 500;
     res.end("baidu-app not configured");
     return true;
@@ -4909,6 +4949,7 @@ async function handleBaiduAppWebhookRequest(req, res) {
     encrypt
   });
   if (decryptedCandidates.length === 0) {
+    logger.warn(`decrypt failed for all ${decryptable.length} candidate account(s)`);
     res.statusCode = 400;
     res.end("decrypt failed");
     return true;
@@ -4916,6 +4957,7 @@ async function handleBaiduAppWebhookRequest(req, res) {
   const selected = selectDecryptedTarget({ candidates: decryptedCandidates, logger });
   const target = selected.target;
   if (!target.account.configured || !target.account.token || !target.account.encodingAESKey) {
+    logger.warn(`selected account ${target.account.accountId} not fully configured`);
     res.statusCode = 500;
     res.end("baidu-app not configured");
     return true;
@@ -4924,9 +4966,16 @@ async function handleBaiduAppWebhookRequest(req, res) {
   target.statusSink?.({ lastInboundAt: Date.now() });
   const msgtype = String(msg.msgtype ?? msg.MsgType ?? "").toLowerCase();
   const msgid = msg.msgid ?? msg.MsgId ? String(msg.msgid ?? msg.MsgId) : void 0;
+  const inboundSenderId = msg.from?.userid?.trim() ?? msg.FromUserName?.trim();
+  logger.info(
+    `inbound: type=${msgtype || "unknown"} msgid=${msgid ?? "none"} sender=${inboundSenderId ?? "unknown"} account=${target.account.accountId}`
+  );
   if (msgtype === "stream") {
     const streamId2 = String(msg.stream?.id ?? "").trim();
     const state = streamId2 ? streams.get(streamId2) : void 0;
+    logger.info(
+      `[REPLY-MODE:STREAM-POLL] stream poll request: streamId=${streamId2 || "none"} found=${Boolean(state)} finished=${state?.finished ?? "n/a"} contentLen=${state?.content.length ?? 0}`
+    );
     const reply = state ? buildStreamReplyFromState(state) : buildStreamReplyFromState({
       streamId: streamId2 || "unknown",
       finished: true,
@@ -4945,6 +4994,7 @@ async function handleBaiduAppWebhookRequest(req, res) {
   }
   if (msgid && msgidToStreamId.has(msgid)) {
     const streamId2 = msgidToStreamId.get(msgid) ?? "";
+    logger.debug(`duplicate msgid detected: msgid=${msgid} streamId=${streamId2}, returning placeholder`);
     const reply = buildStreamPlaceholderReply(streamId2);
     jsonOk(
       res,
@@ -4961,11 +5011,13 @@ async function handleBaiduAppWebhookRequest(req, res) {
     const eventtype = String(
       msg.event?.eventtype ?? msg.Event ?? ""
     ).toLowerCase();
+    logger.info(`event received: type=${eventtype || "unknown"}`);
     if (eventtype === "enter_chat" || eventtype === "subscribe") {
       const welcome = target.account.config.welcomeText?.trim();
       if (welcome && target.account.canSendActive) {
         const senderId2 = msg.from?.userid?.trim() ?? msg.FromUserName?.trim();
         if (senderId2) {
+          logger.info(`sending welcome message to ${senderId2}`);
           sendBaiduAppMessage(target.account, { userId: senderId2 }, welcome).catch((err) => {
             logger.error(`failed to send welcome message: ${String(err)}`);
           });
@@ -5006,6 +5058,7 @@ async function handleBaiduAppWebhookRequest(req, res) {
     finished: false,
     content: ""
   });
+  logger.info(`stream created: streamId=${streamId} msgid=${msgid ?? "none"}`);
   const core = tryGetBaiduAppRuntime();
   const senderId = msg.from?.userid?.trim() ?? msg.FromUserName?.trim();
   if (core) {
@@ -5013,6 +5066,9 @@ async function handleBaiduAppWebhookRequest(req, res) {
     if (state) {
       state.started = true;
     }
+    logger.info(
+      `agent dispatch started: streamId=${streamId} sender=${senderId ?? "unknown"} canSendActive=${target.account.canSendActive}`
+    );
     const hooks = {
       onChunk: (text) => {
         const current = streams.get(streamId);
@@ -5020,6 +5076,9 @@ async function handleBaiduAppWebhookRequest(req, res) {
           return;
         }
         appendStreamContent(current, text);
+        logger.debug(
+          `chunk received: streamId=${streamId} chunkLen=${text.length} totalLen=${current.content.length}`
+        );
         target.statusSink?.({ lastOutboundAt: Date.now() });
       },
       onError: (err) => {
@@ -5046,15 +5105,34 @@ async function handleBaiduAppWebhookRequest(req, res) {
       if (current) {
         current.finished = true;
         current.updatedAt = Date.now();
+        const contentLen = current.content.trim().length;
+        logger.info(
+          `agent dispatch done: streamId=${streamId} contentLen=${contentLen} canSendActive=${target.account.canSendActive} sender=${senderId ?? "none"}`
+        );
+        if (!target.account.canSendActive) {
+          logger.warn(
+            `active send skipped: appKey/appSecret not configured for account ${target.account.accountId}`
+          );
+        } else if (!senderId) {
+          logger.warn(`active send skipped: senderId is empty`);
+        } else if (!contentLen) {
+          logger.warn(`active send skipped: agent produced no content`);
+        }
         if (target.account.canSendActive && senderId && current.content.trim()) {
           try {
             const chunks = splitMessageByBytes(current.content, MAX_MESSAGE_BYTES);
-            for (const chunk of chunks) {
-              await sendBaiduAppMessage(target.account, { userId: senderId }, chunk);
+            logger.info(
+              `[REPLY-MODE:ACTIVE-SEND] active send starting: streamId=${streamId} to=${senderId} chunks=${chunks.length} contentLen=${contentLen}`
+            );
+            for (let i = 0; i < chunks.length; i++) {
+              await sendBaiduAppMessage(target.account, { userId: senderId }, chunks[i]);
+              logger.debug(`active send chunk ${i + 1}/${chunks.length} sent: streamId=${streamId}`);
             }
-            logger.info(`active send complete: streamId=${streamId}, ${chunks.length} chunks`);
+            logger.info(
+              `[REPLY-MODE:ACTIVE-SEND] active send complete: streamId=${streamId} chunks=${chunks.length}`
+            );
           } catch (err) {
-            logger.error(`active send failed: ${String(err)}`);
+            logger.error(`active send failed: streamId=${streamId} error=${String(err)}`);
           }
         }
       }
@@ -5066,9 +5144,10 @@ async function handleBaiduAppWebhookRequest(req, res) {
         current.finished = true;
         current.updatedAt = Date.now();
       }
-      logger.error(`baidu-app agent failed: ${String(err)}`);
+      logger.error(`agent dispatch failed: streamId=${streamId} error=${String(err)}`);
     });
   } else {
+    logger.warn(`runtime not available: streamId=${streamId} \u2014 agent dispatch skipped, no reply will be generated`);
     const state = streams.get(streamId);
     if (state) {
       state.finished = true;
@@ -5076,6 +5155,7 @@ async function handleBaiduAppWebhookRequest(req, res) {
     }
   }
   const placeholderReply = buildStreamPlaceholderReply(streamId);
+  logger.debug(`stream placeholder reply sent: streamId=${streamId}`);
   jsonOk(
     res,
     buildEncryptedJsonReply({