@jackle.dev/zalox-plugin 1.0.10 → 1.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jackle.dev/zalox-plugin",
-  "version": "1.0.10",
+  "version": "1.0.13",
   "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,12 @@ export type SendResult = {
   error?: string;
 };
 
+function cleanText(input: string): string {
+  if (!input) return '';
+  // Remove <ctrl...> artifacts
+  return input.replace(/<ctrl\d+>/gi, '').trim();
+}
+
 export async function sendText(
   threadId: string,
   text: string,
@@ -24,8 +30,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,7 +43,13 @@ 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);
+    
+    // Group support handled by zca-js automatically via threadId format usually,
+    // but ThreadType is explicit here.
+    console.log(`[ZaloX] Sending text to ${threadId} (type=${type}): "${cleaned.slice(0, 50)}..."`);
+    const result = await 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) {
@@ -57,6 +71,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;
@@ -73,7 +89,7 @@ export async function sendMedia(
     if (filePath) {
       // Send with local file attachment
       const result = await cached.api.sendMessage(
-        { msg: text || '', attachments: [filePath] },
+        { msg: cleaned || '', attachments: [filePath] },
         threadId.trim(),
         type,
       );
@@ -97,7 +113,7 @@ export async function sendMedia(
         
         try {
           const result = await cached.api.sendMessage(
-            { msg: text || '', attachments: [tmpPath] },
+            { msg: cleaned || '', attachments: [tmpPath] },
             threadId.trim(),
             type,
           );
@@ -107,12 +123,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 +136,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) };

package/src/tools-scenario.ts

@@ -46,10 +46,11 @@ export const zaloxScenarioTool = {
         // Analyze Requirements (Playbook Logic)
         const reqs = data.requirements || {};
         const flow = data.flow || [];
+        const setup = data.setup || {};
         const warnings = [];
 
         if (reqs.packages && reqs.packages.length > 0) {
-            warnings.push(`📦 Packages required: ${reqs.packages.join(', ')} (Please install via npm)`);
+            warnings.push(`📦 Packages required: ${reqs.packages.join(', ')}`);
         }
         if (reqs.env && reqs.env.length > 0) {
             warnings.push(`🔑 Environment variables needed: ${reqs.env.join(', ')}`);
@@ -60,8 +61,14 @@ export const zaloxScenarioTool = {
 
         if (warnings.length > 0 || flow.length > 0) {
             report += `\n\n🚨 PLAYBOOK REQUIREMENTS:\n${warnings.join('\n')}`;
+            
+            if (setup.commands && setup.commands.length > 0) {
+                report += `\n\n🛠️ SETUP GUIDE (Ask user to run/confirm):\n`;
+                if (setup.guide) report += `ℹ️ ${setup.guide}\n`;
+                report += `Commands:\n${setup.commands.map((c: string) => `> ${c}`).join('\n')}`;
+            }
+
             report += `\n\n📋 EXECUTION FLOW (SOP):\n${flow.join('\n')}`;
-            report += `\n\n👉 Agent: Please read the flow above and ask user to fulfill requirements if missing.`;
         }
 
         return report;