@jackle.dev/zalox-plugin 1.0.11 → 1.0.14

package/package.json

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

package/src/listener.ts

@@ -373,10 +373,10 @@ export async function startInProcessListener(
         ? mentions.some((m: any) => String(m.uid || m.id || m) === ownId)
         : false;
       const name = account.name || 'Bot';
-      const textMention = content.includes(`@${name}`);
+      const textMention = content.includes(`@${name}`) || content.includes('@Tiệp Lê');
       
       // DEBUG Group logic
-      // console.log(`[ZaloX] Group msg: mentioned=${isMentioned} textMention=${textMention} (name=${name})`);
+      console.log(`[ZaloX] Group msg: mentioned=${isMentioned} textMention=${textMention} name=${name} content="${content}"`);
 
       if (!isMentioned && !textMention && !content.startsWith('/')) {
         return;
@@ -411,20 +411,8 @@ export async function startInProcessListener(
 
     if (!normalized.content?.trim() && !normalized.mediaUrl) return;
 
-    // React with ❤️
-    try {
-      const reactThreadType = isGroup ? ThreadType.Group : ThreadType.User;
-      api.addReaction(Reactions.HEART, {
-        threadId: normalized.threadId,
-        type: reactThreadType,
-        data: {
-          msgId: String(data.msgId || ''),
-          cliMsgId: String(data.cliMsgId || ''),
-        },
-      }).catch((err: any) => {
-        runtime.error?.(`[${account.accountId}] ZaloX reaction failed: ${String(err)}`);
-      });
-    } catch {}
+    // 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)}`);

package/src/send.ts

@@ -16,6 +16,26 @@ export type SendResult = {
   error?: string;
 };
 
+function cleanText(input: string): string {
+  if (!input) return '';
+  // Remove <ctrl...> artifacts
+  return input.replace(/<ctrl\d+>/gi, '').trim();
+}
+
+async function withRetry<T>(fn: () => Promise<T>, retries = 3, delay = 1000): Promise<T> {
+  let lastErr;
+  for (let i = 0; i < retries; i++) {
+    try {
+      return await fn();
+    } catch (err) {
+      lastErr = err;
+      console.warn(`[ZaloX] Send failed (attempt ${i+1}/${retries}): ${err}`);
+      await new Promise(r => setTimeout(r, delay * (i + 1)));
+    }
+  }
+  throw lastErr;
+}
+
 export async function sendText(
   threadId: string,
   text: string,
@@ -24,8 +44,10 @@ export async function sendText(
   if (!threadId?.trim()) {
     return { ok: false, error: 'No threadId provided' };
   }
-  if (!text?.trim()) {
-    return { ok: false, error: 'No text provided' };
+  
+  const cleaned = cleanText(text);
+  if (!cleaned) {
+    return { ok: false, error: 'No text provided (or empty after cleanup)' };
   }
 
   const cached = getCachedApi(options.profile);
@@ -35,10 +57,19 @@ export async function sendText(
 
   try {
     const type = options.isGroup ? ThreadType.Group : ThreadType.User;
-    const result = await cached.api.sendMessage(text.slice(0, 2000), threadId.trim(), type);
+    
+    console.log(`[ZaloX] Sending text to ${threadId} (type=${type}): "${cleaned.slice(0, 50)}..."`);
+    
+    // Add retry logic
+    const result = await withRetry(() => 
+      cached.api.sendMessage(cleaned.slice(0, 2000), threadId.trim(), type)
+    );
+    
+    console.log(`[ZaloX] Send result:`, JSON.stringify(result));
     const msgId = result?.message?.msgId ? String(result.message.msgId) : undefined;
     return { ok: true, messageId: msgId };
   } catch (err: any) {
+    console.error(`[ZaloX] Final send error: ${err}`);
     return { ok: false, error: err.message || String(err) };
   }
 }
@@ -57,6 +88,8 @@ export async function sendMedia(
   if (!cached) {
     return { ok: false, error: `No active API session for profile "${options.profile}". Listener may not be running.` };
   }
+  
+  const cleaned = cleanText(text);
 
   try {
     const type = options.isGroup ? ThreadType.Group : ThreadType.User;
@@ -71,11 +104,13 @@ export async function sendMedia(
     }
 
     if (filePath) {
-      // Send with local file attachment
-      const result = await cached.api.sendMessage(
-        { msg: text || '', attachments: [filePath] },
-        threadId.trim(),
-        type,
+      // Send with local file attachment (with retry)
+      const result = await withRetry(() => 
+        cached.api.sendMessage(
+          { msg: cleaned || '', attachments: [filePath!] },
+          threadId.trim(),
+          type,
+        )
       );
       const msgId = result?.message?.msgId ? String(result.message.msgId) : undefined;
       return { ok: true, messageId: msgId };
@@ -96,10 +131,13 @@ export async function sendMedia(
         writeFileSync(tmpPath, buffer);
         
         try {
-          const result = await cached.api.sendMessage(
-            { msg: text || '', attachments: [tmpPath] },
-            threadId.trim(),
-            type,
+          // Send attachment with retry
+          const result = await withRetry(() => 
+            cached.api.sendMessage(
+              { msg: cleaned || '', attachments: [tmpPath!] },
+              threadId.trim(),
+              type,
+            )
           );
           
           const msgId = result?.message?.msgId ? String(result.message.msgId) : undefined;
@@ -107,12 +145,12 @@ export async function sendMedia(
         } catch (sendErr) {
           console.error(`[ZaloX] Send attachment failed: ${sendErr}`);
           // Fallback: send text with URL as link
-          const fullText = text ? `${text}\n${mediaUrl}` : mediaUrl;
+          const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
           return sendText(threadId, fullText, options);
         }
       } catch (dlErr: any) {
         // Fallback: send text with URL as link
-        const fullText = text ? `${text}\n${mediaUrl}` : mediaUrl;
+        const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
         return sendText(threadId, fullText, options);
       } finally {
         if (tmpPath) try { unlinkSync(tmpPath); } catch {}
@@ -120,7 +158,7 @@ export async function sendMedia(
     }
 
     // Fallback: send text only
-    const fullText = text ? `${text}\n${mediaUrl}` : mediaUrl;
+    const fullText = cleaned ? `${cleaned}\n${mediaUrl}` : mediaUrl;
     return sendText(threadId, fullText, options);
   } catch (err: any) {
     return { ok: false, error: err.message || String(err) };