@jackle.dev/zalox-plugin 1.0.18 → 1.0.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jackle.dev/zalox-plugin",
-  "version": "1.0.18",
+  "version": "1.0.20",
   "description": "OpenClaw channel plugin for Zalo via zca-js (in-process, single login)",
   "type": "module",
   "main": "./src/index.ts",

package/src/listener.ts

@@ -80,9 +80,7 @@ async function downloadZaloMedia(url: string, profile: string, filename?: string
     // Determine extension
     let ext = extname(url).split('?')[0];
     if (!ext || ext.length > 5) {
-       if (contentType?.includes('jpeg')) ext = '.jpg';
-       else if (contentType?.includes('png')) ext = '.png';
-       else ext = '.jpg';
+       if (contentType?.includes('jpeg')) ext = '.jpg';\n       else if (contentType?.includes('png')) ext = '.png';\n       else ext = '.jpg';
     }
 
     const name = filename || `zalox-${Date.now()}-${randomUUID().slice(0, 8)}${ext}`;
@@ -151,291 +149,4 @@ async function processMessage(
   const useAccessGroups = config.commands?.useAccessGroups !== false;
   const commandAuthorized = shouldComputeAuth
     ? core.channel.commands.resolveCommandAuthorizedFromAuthorizers({
-        useAccessGroups,
-        authorizers: [
-          { configured: effectiveAllowFrom.length > 0, allowed: senderAllowedForCommands },
-        ],
-      })
-    : undefined;
-
-  if (!isGroup) {
-    if (dmPolicy === 'disabled') {
-      runtime.log?.(`[zalox] Blocked DM from ${senderId} (dmPolicy=disabled)`);
-      return;
-    }
-    if (dmPolicy !== 'open') {
-      if (!senderAllowedForCommands) {
-        if (dmPolicy === 'pairing') {
-          const { code, created } = await core.channel.pairing.upsertPairingRequest({
-            channel: 'zalox',
-            id: senderId,
-            meta: { name: senderName || undefined },
-          });
-          if (created) {
-            runtime.log?.(`[zalox] pairing request sender=${senderId}`);
-            try {
-              await sendText(
-                chatId,
-                core.channel.pairing.buildPairingReply({
-                  channel: 'zalox',
-                  idLine: `Your Zalo user id: ${senderId}`,
-                  code,
-                }),
-                { profile },
-              );
-              statusSink?.({ lastOutboundAt: Date.now() });
-            } catch (err) {
-              runtime.error?.(`[zalox] pairing reply failed: ${String(err)}`);
-            }
-          }
-        }
-        return;
-      }
-    }
-  }
-
-  // Group command authorization
-  if (
-    isGroup &&
-    core.channel.commands.isControlCommandMessage(rawBody, config) &&
-    commandAuthorized !== true
-  ) {
-    return;
-  }
-
-  // Route to agent
-  const peer = isGroup
-    ? { kind: 'group' as const, id: chatId }
-    : { kind: 'dm' as const, id: senderId };
-
-  const route = core.channel.routing.resolveAgentRoute({
-    cfg: config,
-    channel: 'zalox',
-    accountId: account.accountId,
-    peer,
-  });
-
-  const fromLabel = isGroup ? `group:${chatId}` : senderName || `user:${senderId}`;
-  const storePath = core.channel.session.resolveStorePath(config.session?.store, {
-    agentId: route.agentId,
-  });
-  const envelopeOptions = core.channel.reply.resolveEnvelopeFormatOptions(config);
-  const previousTimestamp = core.channel.session.readSessionUpdatedAt({
-    storePath,
-    sessionKey: route.sessionKey,
-  });
-  const body = core.channel.reply.formatAgentEnvelope({
-    channel: 'Zalo (ZaloX)',
-    from: fromLabel,
-    timestamp: timestamp ? timestamp * 1000 : undefined,
-    previousTimestamp,
-    envelope: envelopeOptions,
-    body: rawBody || (mediaPath ? '[Image]' : ''),
-  });
-
-  const ctxPayload = core.channel.reply.finalizeInboundContext({
-    Body: body,
-    RawBody: rawBody,
-    CommandBody: rawBody,
-    From: isGroup ? `zalox:group:${chatId}` : `zalox:${senderId}`,
-    To: `zalox:${chatId}`,
-    SessionKey: route.sessionKey,
-    AccountId: route.accountId,
-    ChatType: isGroup ? 'group' : 'direct',
-    ConversationLabel: fromLabel,
-    SenderName: senderName || undefined,
-    SenderId: senderId,
-    CommandAuthorized: commandAuthorized,
-    Provider: 'zalox',
-    Surface: 'zalox',
-    MessageSid: message.msgId ?? `${timestamp}`,
-    OriginatingChannel: 'zalox',
-    OriginatingTo: `zalox:${chatId}`,
-    MediaPath: mediaPath, // Pass media path here
-  });
-
-  await core.channel.session.recordInboundSession({
-    storePath,
-    sessionKey: ctxPayload.SessionKey ?? route.sessionKey,
-    ctx: ctxPayload,
-    onRecordError: (err: unknown) => {
-      runtime.error?.(`[zalox] session meta error: ${String(err)}`);
-    },
-  });
-
-  await core.channel.reply.dispatchReplyWithBufferedBlockDispatcher({
-    ctx: ctxPayload,
-    cfg: config,
-    dispatcherOptions: {
-      deliver: async (payload: any) => {
-        const text = payload.text ?? '';
-        if (!text) return;
-
-        const tableMode = core.channel.text.resolveMarkdownTableMode({
-          cfg: config,
-          channel: 'zalox',
-          accountId: account.accountId,
-        });
-        const converted = core.channel.text.convertMarkdownTables(text, tableMode ?? 'code');
-        const chunkMode = core.channel.text.resolveChunkMode(config, 'zalox', account.accountId);
-        const chunks = core.channel.text.chunkMarkdownTextWithMode(converted, 2000, chunkMode);
-
-        for (const chunk of chunks) {
-          try {
-            await sendText(chatId, chunk, { profile, isGroup });
-            statusSink?.({ lastOutboundAt: Date.now() });
-          } catch (err) {
-            runtime.error?.(`[zalox] send failed: ${String(err)}`);
-          }
-        }
-      },
-      onError: (err: unknown, info: any) => {
-        runtime.error?.(`[${account.accountId}] ZaloX ${info.kind} reply failed: ${String(err)}`);
-      },
-    },
-  });
-}
-
-export async function startInProcessListener(
-  options: ListenerOptions,
-): Promise<{ stop: () => void }> {
-  const { account, config, runtime, api, ownId, profile, abortSignal, statusSink } = options;
-  let stopped = false;
-
-  const listener = api.listener;
-
-  // Handle incoming messages
-  listener.on('message', (msg: any) => {
-    if (stopped || abortSignal.aborted) return;
-
-    const data = msg.data || msg;
-    const isGroup = msg.type === 1;
-    const senderId = data.uidFrom ? String(data.uidFrom) : '';
-    
-    // Skip own messages
-    if (senderId === ownId) return;
-
-    const threadId = isGroup
-      ? String(data.idTo || data.threadId || '')
-      : senderId;
-
-    const msgType = data.msgType || 0;
-    
-    // DEBUG: Force reply details for every DM to user (DISABLED)
-    // if (!isGroup) { ... }
-
-    let mediaUrl = undefined;
-    // Prioritize high-res URL if available
-    mediaUrl = data.url || data.href;
-    if (!mediaUrl && (msgType === 2 || String(msgType) === 'chat.photo')) {
-       // Try to find URL in params or content
-       if (data.params?.url) mediaUrl = data.params.url;
-       else if (typeof data.content === 'object' && data.content !== null) {
-          // Already parsed object
-          mediaUrl = (data.content as any).href || (data.content as any).url || (data.content as any).thumb || (data.content as any).normalUrl;
-       }
-       else if (typeof data.content === 'string' && (data.content.includes('http') || data.content.startsWith('{'))) {
-          try {
-             const parsed = JSON.parse(data.content);
-             mediaUrl = parsed.href || parsed.url || parsed.thumb || parsed.normalUrl;
-          } catch {
-             // Maybe content IS the url?
-             if (data.content.startsWith('http')) mediaUrl = data.content;
-          }
-       }
-    }
-    // Also check for image extension if URL exists but type != 2
-    if (!mediaUrl && data.url && (data.url.includes('.jpg') || data.url.includes('.png'))) {
-        mediaUrl = data.url;
-    }
-
-    // Check quote/reply for media
-    if (!mediaUrl && data.quote) {
-       try {
-         const q = data.quote;
-         if (q.attach) {
-            const attach = typeof q.attach === 'string' ? JSON.parse(q.attach) : q.attach;
-            mediaUrl = attach.href || attach.url || attach.thumb || attach.normalUrl;
-         }
-         if (!mediaUrl && (q.href || q.url)) {
-            mediaUrl = q.href || q.url;
-         }
-       } catch {}
-    }
-
-    // Group messages: only respond when bot is mentioned (@tagged)
-    // UNLESS it's a media message — we might want to analyze all images? 
-    if (isGroup) {
-      const mentions = data.mentions || data.mentionIds || [];
-      const content = typeof data.content === 'string' ? data.content : (data.msg || '');
-      
-      const isMentioned = Array.isArray(mentions)
-        ? mentions.some((m: any) => String(m.uid || m.id || m) === ownId)
-        : false;
-      const name = account.name || 'Bot';
-      const textMention = content.includes(`@${name}`) || content.includes('@Tiệp Lê');
-      
-      // DEBUG Group logic
-      console.log(`[ZaloX] Group Check: ownId=${ownId} mentioned=${isMentioned} textMention=${textMention} content="${content}"`);
-
-      if (!isMentioned && !textMention && !content.startsWith('/')) {
-        return;
-      }
-    }
-
-    statusSink?.({ lastInboundAt: Date.now() });
-
-    const normalized = {
-      threadId,
-      msgId: String(data.msgId || data.cliMsgId || ''),
-      content: (() => {
-        let c = data.content || data.msg || '';
-        if (typeof c !== 'string') return '';
-        if (isGroup) {
-          const name = account.name || 'Bot';
-          // Simple cleanup without regex if possible, or use safe regex
-          // But escaping special chars is safer
-          const escaped = name.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-          c = c.replace(new RegExp(`@${escaped}\\s*`, 'gi'), '').trim();
-        }
-        return c;
-      })(),
-      msgType,
-      mediaUrl,
-      timestamp: data.ts ? Math.floor(data.ts / 1000) : Math.floor(Date.now() / 1000),
-      isGroup,
-      senderId,
-      senderName: data.dName || data.senderName || undefined,
-      groupName: data.threadName || data.groupName || undefined,
-    };
-
-    if (!normalized.content?.trim() && !normalized.mediaUrl) return;
-
-    // React logic removed.
-    // Agent should decide when to react.
-
-    processMessage(normalized, account, config, runtime, profile, statusSink).catch((err) => {
-      runtime.error?.(`[${account.accountId}] ZaloX process error: ${String(err)}`);
-    });
-  });
-
-  // Handle errors
-  listener.on('error', (err: any) => {
-    runtime.error?.(`[${account.accountId}] ZaloX listener error: ${err.message || err}`);
-  });
-
-  // Start WebSocket listener with retry
-  listener.start({ retryOnClose: true });
-  runtime.log?.(`[${account.accountId}] ZaloX: WebSocket listener started`);
-
-  const stop = () => {
-    stopped = true;
-    try {
-      listener.stop();
-    } catch {}
-  };
-
-  abortSignal.addEventListener('abort', stop, { once: true });
-
-  return { stop };
-}
+        useAccessGroups,\n        authorizers: [\n          { configured: effectiveAllowFrom.length > 0, allowed: senderAllowedForCommands },\n        ],\n      })\n    : undefined;\n\n  if (!isGroup) {\n    if (dmPolicy === 'disabled') {\n      runtime.log?.(`[zalox] Blocked DM from ${senderId} (dmPolicy=disabled)`);\n      return;\n    }\n    if (dmPolicy !== 'open') {\n      if (!senderAllowedForCommands) {\n        if (dmPolicy === 'pairing') {\n          const { code, created } = await core.channel.pairing.upsertPairingRequest({\n            channel: 'zalox',\n            id: senderId,\n            meta: { name: senderName || undefined },\n          });\n          if (created) {\n            runtime.log?.(`[zalox] pairing request sender=${senderId}`);\n            try {\n              await sendText(\n                chatId,\n                core.channel.pairing.buildPairingReply({\n                  channel: 'zalox',\n                  idLine: `Your Zalo user id: ${senderId}`,\n                  code,\n                }),\n                { profile },\n              );\n              statusSink?.({ lastOutboundAt: Date.now() });\n            } catch (err) {\n              runtime.error?.(`[zalox] pairing reply failed: ${String(err)}`);\n            }\n          }\n        }\n        return;\n      }\n    }\n  }\n\n  // Group command authorization\n  if (\n    isGroup &&\n    core.channel.commands.isControlCommandMessage(rawBody, config) &&\n    commandAuthorized !== true\n  ) {\n    return;\n  }\n\n  // Route to agent\n  const peer = isGroup\n    ? { kind: 'group' as const, id: chatId }\n    : { kind: 'dm' as const, id: senderId };\n\n  const route = core.channel.routing.resolveAgentRoute({\n    cfg: config,\n    channel: 'zalox',\n    accountId: account.accountId,\n    peer,\n  });\n\n  const fromLabel = isGroup ? `group:${chatId}` : senderName || `user:${senderId}`;\n  const storePath = core.channel.session.resolveStorePath(config.session?.store, {\n    agentId: route.agentId,\n  });\n  const envelopeOptions = core.channel.reply.resolveEnvelopeFormatOptions(config);\n  const previousTimestamp = core.channel.session.readSessionUpdatedAt({\n    storePath,\n    sessionKey: route.sessionKey,\n  });\n  const body = core.channel.reply.formatAgentEnvelope({\n    channel: 'Zalo (ZaloX)',\n    from: fromLabel,\n    timestamp: timestamp ? timestamp * 1000 : undefined,\n    previousTimestamp,\n    envelope: envelopeOptions,\n    body: rawBody || (mediaPath ? '[Image]' : ''),\n  });\n\n  const ctxPayload = core.channel.reply.finalizeInboundContext({\n    Body: body,\n    RawBody: rawBody,\n    CommandBody: rawBody,\n    From: isGroup ? `zalox:group:${chatId}` : `zalox:${senderId}`,\n    To: `zalox:${chatId}`,\n    SessionKey: route.sessionKey,\n    AccountId: route.accountId,\n    ChatType: isGroup ? 'group' : 'direct',\n    ConversationLabel: fromLabel,\n    SenderName: senderName || undefined,\n    SenderId: senderId,\n    CommandAuthorized: commandAuthorized,\n    Provider: 'zalox',\n    Surface: 'zalox',\n    MessageSid: message.msgId ?? `${timestamp}`,\n    OriginatingChannel: 'zalox',\n    OriginatingTo: `zalox:${chatId}`,\n    MediaPath: mediaPath, // Pass media path here\n  });\n\n  await core.channel.session.recordInboundSession({\n    storePath,\n    sessionKey: ctxPayload.SessionKey ?? route.sessionKey,\n    ctx: ctxPayload,\n    onRecordError: (err: unknown) => {\n      runtime.error?.(`[zalox] session meta error: ${String(err)}`);\n    },\n  });\n\n  await core.channel.reply.dispatchReplyWithBufferedBlockDispatcher({\n    ctx: ctxPayload,\n    cfg: config,\n    dispatcherOptions: {\n      deliver: async (payload: any) => {\n        const text = payload.text ?? '';\n        if (!text) return;\n\n        const tableMode = core.channel.text.resolveMarkdownTableMode({\n          cfg: config,\n          channel: 'zalox',\n          accountId: account.accountId,\n        });\n        const converted = core.channel.text.convertMarkdownTables(text, tableMode ?? 'code');\n        const chunkMode = core.channel.text.resolveChunkMode(config, 'zalox', account.accountId);\n        const chunks = core.channel.text.chunkMarkdownTextWithMode(converted, 2000, chunkMode);\n\n        for (const chunk of chunks) {\n          try {\n            await sendText(chatId, chunk, { profile, isGroup });\n            statusSink?.({ lastOutboundAt: Date.now() });\n          } catch (err) {\n            runtime.error?.(`[zalox] send failed: ${String(err)}`);\n          }\n        }\n      },\n      onError: (err: unknown, info: any) => {\n        runtime.error?.(`[${account.accountId}] ZaloX ${info.kind} reply failed: ${String(err)}`);\n      },\n    },\n  });\n}\n\nexport async function startInProcessListener(\n  options: ListenerOptions,\n): Promise<{ stop: () => void }> {\n  const { account, config, runtime, api, ownId, profile, abortSignal, statusSink } = options;\n  let stopped = false;\n\n  const listener = api.listener;\n\n  // Handle incoming messages\n  listener.on('message', (msg: any) => {\n    if (stopped || abortSignal.aborted) return;\n\n    const data = msg.data || msg;\n    const isGroup = msg.type === 1;\n    const senderId = data.uidFrom ? String(data.uidFrom) : '';\n    \n    // Skip own messages\n    if (senderId === ownId) return;\n\n    const threadId = isGroup\n      ? String(data.idTo || data.threadId || '')\n      : senderId;\n\n    const msgType = data.msgType || 0;\n    \n    let mediaUrl = undefined;\n    // Prioritize high-res URL if available\n    mediaUrl = data.url || data.href;\n    if (!mediaUrl && (msgType === 2 || String(msgType) === 'chat.photo')) {\n       // Try to find URL in params or content\n       if (data.params?.url) mediaUrl = data.params.url;\n       else if (typeof data.content === 'object' && data.content !== null) {\n          // Already parsed object\n          mediaUrl = (data.content as any).href || (data.content as any).url || (data.content as any).thumb || (data.content as any).normalUrl;\n       }\n       else if (typeof data.content === 'string' && (data.content.includes('http') || data.content.startsWith('{'))) {\n          try {\n             const parsed = JSON.parse(data.content);\n             mediaUrl = parsed.href || parsed.url || parsed.thumb || parsed.normalUrl;\n          } catch {\n             // Maybe content IS the url?\n             if (data.content.startsWith('http')) mediaUrl = data.content;\n          }\n       }\n    }\n    // Also check for image extension if URL exists but type != 2\n    if (!mediaUrl && data.url && (data.url.includes('.jpg') || data.url.includes('.png'))) {\n        mediaUrl = data.url;\n    }\n\n    // Check quote/reply for media\n    if (!mediaUrl && data.quote) {\n       try {\n         const q = data.quote;\n         if (q.attach) {\n            const attach = typeof q.attach === 'string' ? JSON.parse(q.attach) : q.attach;\n            mediaUrl = attach.href || attach.url || attach.thumb || attach.normalUrl;\n         }\n         if (!mediaUrl && (q.href || q.url)) {\n            mediaUrl = q.href || q.url;\n         }\n       } catch {}\n    }\n\n    // Group messages: only respond when bot is mentioned (@tagged)\n    if (isGroup) {\n      const mentions = data.mentions || data.mentionIds || [];\n      const content = typeof data.content === 'string' ? data.content : (data.msg || '');\n      \n      const isMentioned = Array.isArray(mentions)\n        ? mentions.some((m: any) => String(m.uid || m.id || m) === ownId)\n        : false;\n      const name = account.name || 'Bot';\n      \n      // V1.0.20 FIX: Removed unsafe name check (@Tiệp Lê)\n      const textMention = \n           content.includes(`@${name}`) || \n           content.includes('@Bot') || \n           content.includes('@Javis');\n      \n      // DEBUG Group logic\n      console.log(`[ZaloX] Group Check: ownId=${ownId} mentions=${JSON.stringify(mentions)} textMention=${textMention} content=\"${content}\"`);\n\n      // Allow /whoami bypass\n      if (content.trim() === '/whoami') {\n          // Pass\n      } else if (!isMentioned && !textMention && !content.startsWith('/')) {\n        return;\n      }\n    }\n\n    statusSink?.({ lastInboundAt: Date.now() });\n\n    const normalized = {\n      threadId,\n      msgId: String(data.msgId || data.cliMsgId || ''),\n      content: (() => {\n        let c = data.content || data.msg || '';\n        if (typeof c !== 'string') return '';\n        if (isGroup) {\n          const name = account.name || 'Bot';\n          const escaped = name.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\$&');\n          c = c.replace(new RegExp(`@${escaped}\\\\s*`, 'gi'), '').trim();\n          c = c.replace(/@Javis\\s*/gi, '').trim();\n          // Removed @Tiệp replacement\n        }\n        return c;\n      })(),\n      msgType,\n      mediaUrl,\n      timestamp: data.ts ? Math.floor(data.ts / 1000) : Math.floor(Date.now() / 1000),\n      isGroup,\n      senderId,\n      senderName: data.dName || data.senderName || undefined,\n      groupName: data.threadName || data.groupName || undefined,\n    };\n\n    if (!normalized.content?.trim() && !normalized.mediaUrl) return;\n\n    // React with ❤️ to confirm reception\n    try {\n      const reactThreadType = isGroup ? ThreadType.Group : ThreadType.User;\n      api.addReaction(Reactions.HEART, {\n        threadId: normalized.threadId,\n        type: reactThreadType,\n        data: {\n          msgId: String(data.msgId || ''),\n          cliMsgId: String(data.cliMsgId || ''),\n        },\n      }).catch(() => {});\n    } catch {}\n\n    processMessage(normalized, account, config, runtime, profile, statusSink).catch((err) => {\n      runtime.error?.(`[${account.accountId}] ZaloX process error: ${String(err)}`);\n    });\n  });\n\n  // Handle errors\n  listener.on('error', (err: any) => {\n    runtime.error?.(`[${account.accountId}] ZaloX listener error: ${err.message || err}`);\n  });\n\n  // Start WebSocket listener with retry\n  listener.start({ retryOnClose: true });\n  runtime.log?.(`[${account.accountId}] ZaloX: WebSocket listener started`);\n\n  const stop = () => {\n    stopped = true;\n    try {\n      listener.stop();\n    } catch {}\n  };\n\n  abortSignal.addEventListener('abort', stop, { once: true });\n\n  return { stop };\n}\n

package/src/send.ts

@@ -22,6 +22,12 @@ function cleanText(input: string): string {
   return input.replace(/<ctrl\d+>/gi, '').trim();
 }
 
+function cleanThreadId(id: string): string {
+  if (!id) return '';
+  // Remove zalox: or zx: prefix commonly added by OpenClaw routing
+  return id.replace(/^(zalox|zx):/i, '').trim();
+}
+
 async function withRetry<T>(fn: () => Promise<T>, retries = 3, delay = 1000): Promise<T> {
   let lastErr;
   for (let i = 0; i < retries; i++) {
@@ -41,7 +47,8 @@ export async function sendText(
   text: string,
   options: { profile: string; isGroup?: boolean } = { profile: 'default' },
 ): Promise<SendResult> {
-  if (!threadId?.trim()) {
+  const cleanId = cleanThreadId(threadId);
+  if (!cleanId) {
     return { ok: false, error: 'No threadId provided' };
   }
   
@@ -58,11 +65,11 @@ export async function sendText(
   try {
     const type = options.isGroup ? ThreadType.Group : ThreadType.User;
     
-    console.log(`[ZaloX] Sending text to ${threadId} (type=${type}): "${cleaned.slice(0, 50)}..."`);
+    console.log(`[ZaloX] Sending text to ${cleanId} (type=${type}): "${cleaned.slice(0, 50)}..."`);
     
     // Add retry logic
     const result = await withRetry(() => 
-      cached.api.sendMessage(cleaned.slice(0, 2000), threadId.trim(), type)
+      cached.api.sendMessage(cleaned.slice(0, 2000), cleanId, type)
     );
     
     console.log(`[ZaloX] Send result:`, JSON.stringify(result));
@@ -80,7 +87,8 @@ export async function sendMedia(
   mediaUrl: string,
   options: { profile: string; isGroup?: boolean } = { profile: 'default' },
 ): Promise<SendResult> {
-  if (!threadId?.trim()) {
+  const cleanId = cleanThreadId(threadId);
+  if (!cleanId) {
     return { ok: false, error: 'No threadId provided' };
   }
 
@@ -108,7 +116,7 @@ export async function sendMedia(
       const result = await withRetry(() => 
         cached.api.sendMessage(
           { msg: cleaned || '', attachments: [filePath!] },
-          threadId.trim(),
+          cleanId,
           type,
         )
       );
@@ -135,7 +143,7 @@ export async function sendMedia(
           const result = await withRetry(() => 
             cached.api.sendMessage(
               { msg: cleaned || '', attachments: [tmpPath!] },
-              threadId.trim(),
+              cleanId,
               type,
             )
           );
@@ -146,12 +154,12 @@ export async function sendMedia(
           console.error(`[ZaloX] Send attachment failed: ${sendErr}`);
           // Fallback: send text with URL as link
           const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
-          return sendText(threadId, fullText, options);
+          return sendText(cleanId, fullText, options);
         }
       } catch (dlErr: any) {
         // Fallback: send text with URL as link
         const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
-        return sendText(threadId, fullText, options);
+        return sendText(cleanId, fullText, options);
       } finally {
         if (tmpPath) try { unlinkSync(tmpPath); } catch {}
       }
@@ -159,7 +167,7 @@ export async function sendMedia(
 
     // Fallback: send text only
     const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
-    return sendText(threadId, fullText, options);
+    return sendText(cleanId, fullText, options);
   } catch (err: any) {
     return { ok: false, error: err.message || String(err) };
   }