polotno-node 3.10.0 → 3.11.0

package/lib/instance.d.ts

@@ -13,16 +13,7 @@ declare global {
         __polotnoVideoHeartbeat?: () => Promise<void>;
     }
 }
-export declare const URL_FIELDS: string[];
-/**
- * Visit every URL-holding field in a design: `visit(node, key, value)` for each
- * string under one of `URL_FIELDS`, `custom` skipped whole.
- *
- * Exported because `@polotno/fidelity` inlines a fixture's local assets over
- * the same fields — which subtrees a rewrite may enter is one rule, and two
- * copies of it drift. Changing either one needs that package's `test:unit`.
- */
-export declare const forEveryUrlField: (node: any, visit: any) => void;
+export { URL_FIELDS, forEveryUrlField } from '@polotno/schema/asset-urls';
 /**
  * Rewrite every `file:` source in a design to a URL the client can load with
  * any API. Designs without one — the common case — come back untouched, so this

package/lib/instance.js

@@ -7,6 +7,7 @@ import { pathToFileURL, fileURLToPath } from 'url';
 import { Readable } from 'stream';
 import { pipeline } from 'stream/promises';
 import { assertValidDesign } from '@polotno/schema';
+import { forEveryUrlField } from '@polotno/schema/asset-urls';
 import * as fontCache from './font-cache.js';
 import * as localMedia from './local-media.js';
 import { polotnoError, rehydrateError } from './errors.js';
@@ -53,48 +54,8 @@ const collectMediaSourcePointers = (json) => {
     }
     return pointers;
 };
-// Design fields that carry a URL the renderer loads. A `file:` value in any of
-// them is served over the local-media channel instead — see `local-media.ts`
-// for why the browser must never see that scheme. Every other value is passed
-// through exactly as the caller wrote it.
-//
-// A list, rather than "rewrite every `file:` string anywhere", because a design
-// carries text: a text element whose content reads `file:///notes.txt` is
-// content, and rewriting it corrupts the render. `custom` is arbitrary caller
-// data with the same problem, so the walk does not enter it at all. And
-// `test/unit/local-media.test.mjs` fails when the schema grows a field named
-// like one that is missing here.
-export const URL_FIELDS = ['src', 'clipSrc', 'maskSrc', 'background', 'url'];
-const URL_FIELD_SET = new Set(URL_FIELDS);
-/**
- * Visit every URL-holding field in a design: `visit(node, key, value)` for each
- * string under one of `URL_FIELDS`, `custom` skipped whole.
- *
- * Exported because `@polotno/fidelity` inlines a fixture's local assets over
- * the same fields — which subtrees a rewrite may enter is one rule, and two
- * copies of it drift. Changing either one needs that package's `test:unit`.
- */
-export const forEveryUrlField = (node, visit) => {
-    if (!node || typeof node !== 'object')
-        return;
-    if (Array.isArray(node)) {
-        for (const item of node)
-            forEveryUrlField(item, visit);
-        return;
-    }
-    for (const key of Object.keys(node)) {
-        if (key === 'custom')
-            continue;
-        const value = node[key];
-        if (typeof value === 'string') {
-            if (URL_FIELD_SET.has(key))
-                visit(node, key, value);
-        }
-        else if (value && typeof value === 'object') {
-            forEveryUrlField(value, visit);
-        }
-    }
-};
+// Keep the published deep-import API while schema owns the asset vocabulary.
+export { URL_FIELDS, forEveryUrlField } from '@polotno/schema/asset-urls';
 // Not `toLowerCase().startsWith`: a `src` is routinely a data URL of several
 // MB, and that would copy every one of them on every render.
 const isFileUrl = (value) => /^file:/i.test(value);
@@ -1160,11 +1121,12 @@ export const createInstance = async ({ key, url, useParallelPages = true, browse
     });
     const jsonToVideoDataURL = (json, attrs = {}) => renderVideoString(json, attrs, true);
     const jsonToVideoBase64 = (json, attrs = {}) => renderVideoString(json, attrs, false);
+    let closing;
     const instance = {
-        close: async () => {
+        close: () => (closing ?? (closing = (async () => {
             createdInstances.splice(createdInstances.indexOf(instance), 1);
             await browser.close();
-        },
+        })())),
         firstPage,
         browser,
         run,
@@ -1185,7 +1147,7 @@ export const createInstance = async ({ key, url, useParallelPages = true, browse
 };
 // Function to close all created instances in parallel
 const closeAllInstances = async () => {
-    await Promise.all(createdInstances.map((instance) => instance.close()));
+    await Promise.all([...createdInstances].map((instance) => instance.close()));
 };
 // Also handle other termination signals
 ['SIGINT', 'SIGTERM', 'SIGQUIT', 'exit'].forEach((signal) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "polotno-node",
-  "version": "3.10.0",
+  "version": "3.11.0",
   "description": "Polotno workflow from NodeJS",
   "license": "SEE LICENSE IN LICENSE.md",
   "type": "module",
@@ -40,11 +40,11 @@
     "@types/node": "^26.6.1",
     "dotenv": "^17.4.2",
     "jspdf": "^4.2.1",
-    "konva": "^10.5.0",
+    "konva": "^10.7.0",
     "mediabunny": "^1.56.3",
     "pixelmatch": "^7.2.0",
     "pngjs": "^7.0.0",
-    "polotno": "4.13.0",
+    "polotno": "4.14.0",
     "react": "^19.3.0",
     "react-dom": "^19.3.0",
     "tmp": "^0.2.7",
@@ -61,7 +61,7 @@
   },
   "author": "Anton Lavrenov",
   "dependencies": {
-    "@polotno/schema": "0.6.2",
+    "@polotno/schema": "0.7.0",
     "@sparticuz/chromium": "^153.0.0",
     "puppeteer": "25.11.0",
     "puppeteer-core": "25.11.0"
@@ -76,7 +76,7 @@
     "typecheck": "tsc --noEmit -p tsconfig.json",
     "test": "if [ -z \"$TURBO_HASH\" ]; then pnpm exec turbo run build --filter=polotno-node --output-logs=new-only --log-order=stream; fi && vitest run test/unit test/static",
     "test:update-goldens": "rm -f test/static/goldens/*.png test/static/goldens/matrix/*.png && pnpm run test",
-    "test:video": "pnpm exec turbo run build --filter=polotno-node && VITEST_MAX_CONCURRENCY=${VITEST_MAX_CONCURRENCY:-3} vitest run test/video/video.test.mjs",
+    "test:video": "if [ -z \"$TURBO_HASH\" ]; then pnpm exec turbo run build --filter=polotno-node --output-logs=new-only; fi && vitest run test/video/video.test.mjs",
     "test:all": "pnpm exec turbo run build --filter=polotno-node && vitest run test/unit test/static test/video",
     "dev": "vite ./client.html"
   }