zalo-agent-cli 1.5.0 → 1.5.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "zalo-agent-cli",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "CLI tool for Zalo automation — multi-account, proxy, bank transfers, QR payments, Official Account API v3.0",
   "type": "module",
   "bin": {

package/src/commands/mcp.js

@@ -14,6 +14,7 @@ import { registerTools } from "../mcp/mcp-tools.js";
 import { createHTTPServer } from "../mcp/mcp-http-transport.js";
 import { ZaloNotifier } from "../mcp/notifier.js";
 import { ThreadNameCache } from "../mcp/thread-name-cache.js";
+import { autoDownloadImage } from "../mcp/image-downloader.js";
 
 /** Zalo close code for duplicate web session — fatal, do not retry */
 const CLOSE_DUPLICATE = 3000;
@@ -120,6 +121,15 @@ export function registerMCPCommands(program) {
                     // Apply noise filter (stickers, system msgs, short emoji)
                     if (!filter.shouldKeep(normalized)) return;
 
+                    // Auto-download images in background (non-blocking)
+                    if (normalized.attachment?.url) {
+                        const threadName = nameCache?.get(normalized.threadId)?.name || null;
+                        autoDownloadImage(normalized, {
+                            downloadDir: config.images?.downloadDir || undefined,
+                            threadName,
+                        });
+                    }
+
                     buffer.push(normalized.threadId, normalized);
                     notifier.onMessage(normalized);
                     console.error(`[mcp] Buffered ${normalized.threadType} msg from ${normalized.threadId}`);

package/src/mcp/image-downloader.js

@@ -75,9 +75,10 @@ function buildFileName(message, ext) {
 
 /**
  * Open a file with the system's default viewer (cross-platform).
+ * Exported for use by MCP tools.
  * @param {string} filePath
  */
-function openWithSystemViewer(filePath) {
+export function openFile(filePath) {
     const cmds = { darwin: "open", win32: "start", linux: "xdg-open" };
     const cmd = cmds[platform()] || "xdg-open";
     // Use double quotes for paths with spaces; detach so CLI doesn't block
@@ -86,6 +87,28 @@ function openWithSystemViewer(filePath) {
     });
 }
 
+/**
+ * Auto-download image in background when a message arrives.
+ * Non-blocking — fires and forgets. Mutates message.attachment.localPath on success.
+ * @param {object} message - Normalized message (will be mutated with localPath)
+ * @param {object} [options]
+ * @param {string} [options.downloadDir] - Base download directory
+ * @param {string} [options.threadName] - Thread display name from cache
+ */
+export function autoDownloadImage(message, options = {}) {
+    if (!message.attachment?.url) return;
+    // Fire-and-forget: download in background, don't block message processing
+    downloadImage(message, { ...options, autoOpen: false }).then(
+        (result) => {
+            message.attachment.localPath = result.path;
+            console.error(`[image-dl] Saved: ${result.path}`);
+        },
+        (err) => {
+            console.error(`[image-dl] Auto-download failed: ${err.message}`);
+        },
+    );
+}
+
 /**
  * Download an image from URL and save to organized local folder.
  * @param {object} message - Normalized message with attachment.url
@@ -118,7 +141,7 @@ export async function downloadImage(message, options = {}) {
 
     // Auto-open with system viewer if configured
     if (options.autoOpen) {
-        openWithSystemViewer(filePath);
+        openFile(filePath);
     }
 
     return { success: true, path: filePath, folder, fileName };

package/src/mcp/mcp-tools.js

@@ -4,7 +4,7 @@
  */
 
 import { z } from "zod";
-import { downloadImage } from "./image-downloader.js";
+import { downloadImage, openFile } from "./image-downloader.js";
 
 /** Thread type constants matching zca-js ThreadType enum */
 const THREAD_USER = 0;
@@ -202,19 +202,19 @@ export function registerTools(server, api, buffer, filter, config, nameCache) {
         {
             title: "View Zalo Image",
             description:
-                "Download a Zalo image to local filesystem and optionally open it with system viewer. " +
-                "Images are organized by thread folder and named with date/sender metadata. " +
-                "Pass the message ID from zalo_get_messages to download its attached image.",
+                "Open a Zalo image with the system viewer. Images are auto-downloaded when received, " +
+                "organized by thread folder with date/sender metadata filenames. " +
+                "If not yet downloaded, downloads first then opens.",
             inputSchema: z.object({
                 messageId: z.string().describe("Message ID from zalo_get_messages that has an image attachment"),
                 threadId: z.string().optional().describe("Thread ID to search in. Omit to search all threads."),
-                autoOpen: z
+                open: z
                     .boolean()
                     .default(imgConfig.autoOpen ?? true)
-                    .describe("Open image with system viewer after download"),
+                    .describe("Open image with system viewer"),
             }),
         },
-        async ({ messageId, threadId, autoOpen }) => {
+        async ({ messageId, threadId, open }) => {
             try {
                 // Find the message in the buffer by ID
                 const allMessages = buffer.read(threadId, 0, 9999).messages;
@@ -222,16 +222,21 @@ export function registerTools(server, api, buffer, filter, config, nameCache) {
                 if (!message) return err(`Message ${messageId} not found in buffer`);
                 if (!message.attachment?.url) return err(`Message ${messageId} has no image attachment`);
 
-                // Resolve thread name for folder organization
-                const threadName = nameCache?.get(message.threadId)?.name || null;
+                // Use local file if already auto-downloaded, otherwise download now
+                let localPath = message.attachment.localPath;
+                if (!localPath) {
+                    const threadName = nameCache?.get(message.threadId)?.name || null;
+                    const result = await downloadImage(message, {
+                        downloadDir: imgConfig.downloadDir || undefined,
+                        autoOpen: false,
+                        threadName,
+                    });
+                    localPath = result.path;
+                }
 
-                const result = await downloadImage(message, {
-                    downloadDir: imgConfig.downloadDir || undefined,
-                    autoOpen,
-                    threadName,
-                });
+                if (open) openFile(localPath);
 
-                return ok(result);
+                return ok({ success: true, path: localPath });
             } catch (e) {
                 console.error("[mcp-tools] zalo_view_image error:", e.message);
                 return err(e.message);