polotno-node 3.8.0 → 3.8.1

package/lib/instance.js

@@ -11,7 +11,6 @@ import * as fontCache from './font-cache.js';
 import * as localMedia from './local-media.js';
 import { polotnoError, rehydrateError } from './errors.js';
 import { assertUrlAllowed } from './ssrf-guard.js';
-import { isRetryableStatus, withRetry } from '@polotno/core/retry';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 // Compiled to lib/instance.js; the client bundle is published to ../dist.
 const DEFAULT_CLIENT = `file:${path.join(__dirname, '..', 'dist', 'index.html')}`;
@@ -205,13 +204,12 @@ const guardedFetch = async (startUrl, policy, signal) => {
 // start failing exports that work today. A connect/TTFB that never arrives is
 // unambiguous, so that is what this bounds.
 const HEADERS_TIMEOUT_MS = 30000;
+const MAX_ATTEMPTS = 3;
 const downloadUrlToFile = async (urlString, destinationPath, policy = {}) => {
     if (typeof fetch !== 'function') {
         throw polotnoError('FETCH_FAILED', 'Global fetch API is not available in this Node.js runtime. Please use Node 18+.', { url: urlString });
     }
-    let attempts = 0;
-    const attemptDownload = async (attempt) => {
-        attempts = attempt;
+    const attemptDownload = async () => {
         // Ensure no partial file exists from a previous attempt
         await fs.promises.rm(destinationPath, { force: true });
         const controller = new AbortController();
@@ -252,38 +250,26 @@ const downloadUrlToFile = async (urlString, destinationPath, policy = {}) => {
         await pipeline(Readable.fromWeb(res.body), fs.createWriteStream(destinationPath));
         return res.headers.get('content-type') || '';
     };
-    try {
-        return await withRetry(attemptDownload, {
-            // The shared policy (@polotno/core/retry), with this path's own pacing:
-            // local media is staged before the render starts, so it can afford to be
-            // quicker off the mark than an in-render asset fetch.
-            baseDelayMs: 300,
-            // Stated as what is TERMINAL, not what is transient: a transport
-            // allow-list silently missed this loop's own timeout abort.
-            shouldRetry: (e) => {
-                // Blocked by policy (SSRF) — deterministic, and re-resolving the same
-                // bad host would only burn the budget.
-                if (e?.ssrfBlocked)
-                    return false;
-                const status = e?.details
-                    ?.status;
-                // A 4xx means the media is not there. This loop used to retry it three
-                // times before saying so.
-                if (typeof status === 'number')
-                    return isRetryableStatus(status);
-                return true;
-            },
-        });
-    }
-    catch (lastError) {
-        // A policy rejection is surfaced untouched, as it was before this loop
-        // delegated its retrying: `ssrfBlocked` is what callers branch on, and a
-        // second FETCH_FAILED wrap buries it on `cause`.
-        if (lastError?.ssrfBlocked) {
-            throw lastError;
+    // A local loop rather than @polotno/core/retry: importing that made core (and
+    // konva) a runtime dependency of polotno-node for one call site.
+    for (let attempt = 1;; attempt++) {
+        try {
+            return await attemptDownload();
+        }
+        catch (e) {
+            // Surfaced untouched: callers branch on `ssrfBlocked`, and re-resolving
+            // the same blocked host cannot succeed.
+            if (e?.ssrfBlocked)
+                throw e;
+            const status = e?.details?.status;
+            // A 4xx other than 429 is deterministic; anything else may be transient.
+            const retryable = status === undefined || status === 429 || status >= 500;
+            if (!retryable || attempt >= MAX_ATTEMPTS) {
+                const message = e?.message || String(e);
+                throw polotnoError('FETCH_FAILED', `Failed to download media after ${attempt} attempt${attempt === 1 ? '' : 's'}: ${urlString}\nLast error: ${message}`, { url: urlString, attempts: attempt }, e);
+            }
+            await new Promise((r) => setTimeout(r, 300 * 2 ** (attempt - 1)));
         }
-        const message = lastError?.message || String(lastError);
-        throw polotnoError('FETCH_FAILED', `Failed to download media after ${attempts} attempt${attempts === 1 ? '' : 's'}: ${urlString}\nLast error: ${message}`, { url: urlString, attempts }, lastError);
     }
 };
 // Returns a human-readable reason string if the video file needs transcoding

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polotno-node",
-  "version": "3.8.0",
+  "version": "3.8.1",
   "description": "Polotno workflow from NodeJS",
   "license": "SEE LICENSE IN LICENSE.md",
   "type": "module",
@@ -51,8 +51,8 @@
     "vite": "^8.2.2",
     "vitest": "^4.1.11",
     "web-audio-api": "^1.5.5",
-    "polotno": "4.12.0",
     "@polotno/video-export": "0.2.5",
+    "polotno": "4.12.0",
     "@polotno/design-fixtures": "0.0.0"
   },
   "overrides": {
@@ -66,7 +66,6 @@
     "@sparticuz/chromium": "^149.0.0",
     "puppeteer": "25.9.0",
     "puppeteer-core": "25.9.0",
-    "@polotno/core": "0.10.0",
     "@polotno/schema": "0.6.1"
   },
   "bugs": {