@unityclaw/sdk 1.0.4 → 1.0.5

package/dist/index.cjs

@@ -103,30 +103,45 @@ var TaskFolderManager = class {
   /**
    * Download an attachment from URL to local folder
    * Important: Downloads attachments to avoid tmp_url expiration
+   * Includes retry logic for transient failures
    */
-  async downloadAttachment(ctx, attachment) {
+  async downloadAttachment(ctx, attachment, maxRetries = 3, retryDelayMs = 1e3) {
     if (!this.options.downloadAttachments || !attachment.tmp_url) {
       return attachment.tmp_url;
     }
-    try {
-      const url = attachment.tmp_url;
-      const urlPath = url.split("?")[0];
-      const baseName = urlPath ? (0, import_path.basename)(urlPath) : "";
-      const filename = attachment.name || baseName || `file_${Date.now()}`;
-      const localPath = (0, import_path.join)(ctx.attachmentsPath, filename);
-      const response = await import_axios.default.get(url, {
-        responseType: "arraybuffer",
-        timeout: 6e4
-      });
-      await (0, import_promises.writeFile)(localPath, Buffer.from(response.data));
-      ctx.downloadedFiles.push(localPath);
-      return localPath;
-    } catch (error) {
-      await this.log(ctx, "warn", `Failed to download attachment: ${attachment.name}`, {
-        error: error instanceof Error ? error.message : error
-      });
-      return attachment.tmp_url;
+    const url = attachment.tmp_url;
+    const urlPath = url.split("?")[0];
+    const baseName = urlPath ? (0, import_path.basename)(urlPath) : "";
+    const filename = attachment.name || baseName || `file_${Date.now()}`;
+    const localPath = (0, import_path.join)(ctx.attachmentsPath, filename);
+    let lastError;
+    for (let attempt = 1; attempt <= maxRetries; attempt++) {
+      try {
+        const response = await import_axios.default.get(url, {
+          responseType: "arraybuffer",
+          timeout: 6e4
+        });
+        await (0, import_promises.writeFile)(localPath, Buffer.from(response.data));
+        ctx.downloadedFiles.push(localPath);
+        if (attempt > 1) {
+          await this.log(ctx, "info", `Download succeeded on attempt ${attempt}: ${filename}`);
+        }
+        return localPath;
+      } catch (error) {
+        lastError = error instanceof Error ? error : new Error(String(error));
+        if (attempt < maxRetries) {
+          await this.log(ctx, "warn", `Download attempt ${attempt}/${maxRetries} failed for ${filename}, retrying...`, {
+            error: lastError.message
+          });
+          await new Promise((resolve) => setTimeout(resolve, retryDelayMs * attempt));
+        }
+      }
     }
+    await this.log(ctx, "warn", `Failed to download attachment after ${maxRetries} attempts: ${filename}`, {
+      error: lastError?.message,
+      url: url.substring(0, 100) + "..."
+    });
+    return attachment.tmp_url;
   }
   /**
    * Download multiple attachments

package/dist/index.d.cts

@@ -272,8 +272,9 @@ declare class TaskFolderManager {
     /**
      * Download an attachment from URL to local folder
      * Important: Downloads attachments to avoid tmp_url expiration
+     * Includes retry logic for transient failures
      */
-    downloadAttachment(ctx: TaskFolderContext, attachment: AttachmentFieldItem): Promise<string | undefined>;
+    downloadAttachment(ctx: TaskFolderContext, attachment: AttachmentFieldItem, maxRetries?: number, retryDelayMs?: number): Promise<string | undefined>;
     /**
      * Download multiple attachments
      */

package/dist/index.d.ts

@@ -272,8 +272,9 @@ declare class TaskFolderManager {
     /**
      * Download an attachment from URL to local folder
      * Important: Downloads attachments to avoid tmp_url expiration
+     * Includes retry logic for transient failures
      */
-    downloadAttachment(ctx: TaskFolderContext, attachment: AttachmentFieldItem): Promise<string | undefined>;
+    downloadAttachment(ctx: TaskFolderContext, attachment: AttachmentFieldItem, maxRetries?: number, retryDelayMs?: number): Promise<string | undefined>;
     /**
      * Download multiple attachments
      */

package/dist/index.js

@@ -65,30 +65,45 @@ var TaskFolderManager = class {
   /**
    * Download an attachment from URL to local folder
    * Important: Downloads attachments to avoid tmp_url expiration
+   * Includes retry logic for transient failures
    */
-  async downloadAttachment(ctx, attachment) {
+  async downloadAttachment(ctx, attachment, maxRetries = 3, retryDelayMs = 1e3) {
     if (!this.options.downloadAttachments || !attachment.tmp_url) {
       return attachment.tmp_url;
     }
-    try {
-      const url = attachment.tmp_url;
-      const urlPath = url.split("?")[0];
-      const baseName = urlPath ? basename(urlPath) : "";
-      const filename = attachment.name || baseName || `file_${Date.now()}`;
-      const localPath = join(ctx.attachmentsPath, filename);
-      const response = await axios.get(url, {
-        responseType: "arraybuffer",
-        timeout: 6e4
-      });
-      await writeFile(localPath, Buffer.from(response.data));
-      ctx.downloadedFiles.push(localPath);
-      return localPath;
-    } catch (error) {
-      await this.log(ctx, "warn", `Failed to download attachment: ${attachment.name}`, {
-        error: error instanceof Error ? error.message : error
-      });
-      return attachment.tmp_url;
+    const url = attachment.tmp_url;
+    const urlPath = url.split("?")[0];
+    const baseName = urlPath ? basename(urlPath) : "";
+    const filename = attachment.name || baseName || `file_${Date.now()}`;
+    const localPath = join(ctx.attachmentsPath, filename);
+    let lastError;
+    for (let attempt = 1; attempt <= maxRetries; attempt++) {
+      try {
+        const response = await axios.get(url, {
+          responseType: "arraybuffer",
+          timeout: 6e4
+        });
+        await writeFile(localPath, Buffer.from(response.data));
+        ctx.downloadedFiles.push(localPath);
+        if (attempt > 1) {
+          await this.log(ctx, "info", `Download succeeded on attempt ${attempt}: ${filename}`);
+        }
+        return localPath;
+      } catch (error) {
+        lastError = error instanceof Error ? error : new Error(String(error));
+        if (attempt < maxRetries) {
+          await this.log(ctx, "warn", `Download attempt ${attempt}/${maxRetries} failed for ${filename}, retrying...`, {
+            error: lastError.message
+          });
+          await new Promise((resolve) => setTimeout(resolve, retryDelayMs * attempt));
+        }
+      }
     }
+    await this.log(ctx, "warn", `Failed to download attachment after ${maxRetries} attempts: ${filename}`, {
+      error: lastError?.message,
+      url: url.substring(0, 100) + "..."
+    });
+    return attachment.tmp_url;
   }
   /**
    * Download multiple attachments

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unityclaw/sdk",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Node.js SDK for UnityClaw API - AI-powered image/video generation, media analysis, and more",
   "type": "module",
   "bin": {