fixed an issue with the currentScript not spotted before

Author: WebReflection

docs/index.js

@@ -1,7 +1,7 @@
 import '@ungap/with-resolvers';
 import { $$ } from 'basic-devtools';
 
-import { JSModules, assign, create, createOverload, createResolved, dedent, defineProperty, nodeInfo, registerJSModules } from './utils.js';
+import { JSModules, isSync, assign, create, createOverload, createResolved, dedent, defineProperty, nodeInfo, registerJSModules } from './utils.js';
 import { getDetails } from './script-handler.js';
 import { registry as defaultRegistry, prefixes, configs } from './interpreters.js';
 import { getRuntimeID } from './loader.js';
@@ -69,7 +69,7 @@ export const handleCustomType = async (node) => {
                         type: runtime,
                         custom: type,
                         config: node.getAttribute('config') || config || {},
-                        async: node.getAttribute('async') !== 'false',
+                        async: !isSync(node),
                         serviceWorker: node.getAttribute('service-worker'),
                     });
                     defineProperty(node, 'xworker', { value: xworker });

docs/index.js.map

@@ -6,7 +6,7 @@ import $xworker from './worker/class.js';
 import workerURL from './worker/url.js';
 import { getRuntime, getRuntimeID } from './loader.js';
 import { registry } from './interpreters.js';
-import { JSModules, all, dispatch, resolve, defineProperty, nodeInfo, registerJSModules } from './utils.js';
+import { JSModules, isSync, all, dispatch, resolve, defineProperty, nodeInfo, registerJSModules } from './utils.js';
 
 const getRoot = (script) => {
     let parent = script;
@@ -55,12 +55,6 @@ const execute = async (currentScript, source, XWorker, isAsync) => {
         source,
     ]);
     try {
-        // temporarily override inherited document.currentScript in a non writable way
-        // but it deletes it right after to preserve native behavior (as it's sync: no trouble)
-        defineProperty(document, 'currentScript', {
-            configurable: true,
-            get: () => currentScript,
-        });
         registerJSModules(type, module, interpreter, JSModules);
         module.registerJSModule(interpreter, 'polyscript', {
             XWorker,
@@ -69,10 +63,16 @@ const execute = async (currentScript, source, XWorker, isAsync) => {
             workers: workersHandler,
         });
         dispatch(currentScript, type, 'ready');
-        const result = module[isAsync ? 'runAsync' : 'run'](interpreter, content);
+        // temporarily override inherited document.currentScript in a non writable way
+        // but it deletes it right after to preserve native behavior (as it's sync: no trouble)
+        defineProperty(document, 'currentScript', {
+            configurable: true,
+            get: () => currentScript,
+        });
         const done = dispatch.bind(null, currentScript, type, 'done');
-        if (isAsync) result.then(done);
-        else done();
+        let result = module[isAsync ? 'runAsync' : 'run'](interpreter, content);
+        if (isAsync) result = await result;
+        done();
         return result;
     } finally {
         delete document.currentScript;
@@ -125,7 +125,6 @@ export const handle = async (script) => {
         // and/or source code with different config or interpreter
         const {
             attributes: {
-                async: asyncAttribute,
                 config,
                 env,
                 name: wn,
@@ -138,7 +137,7 @@ export const handle = async (script) => {
         } = script;
 
         /* c8 ignore start */
-        const isAsync = asyncAttribute?.value !== 'false';
+        const isAsync = !isSync(script);
 
         const versionValue = version?.value;
         const name = getRuntimeID(type, versionValue);

docs/zip-gl8b5xR3.js renamed to docs/zip-C10rja2G.js

@@ -124,6 +124,10 @@ export const importCSS = href => new Promise((onload, onerror) => {
 });
 
 export const isCSS = source => /\.css$/i.test(new URL(source).pathname);
+
+export const isSync = element =>
+    /^(?:false|0|no)$/i.test(element.getAttribute('async'));
+
 /* c8 ignore stop */
 
 export {

docs/zip-C10rja2G.js.map

@@ -42,12 +42,12 @@
         "@playwright/test": "^1.45.3",
         "@rollup/plugin-node-resolve": "^15.2.3",
         "@rollup/plugin-terser": "^0.4.4",
-        "@zip.js/zip.js": "^2.7.47",
+        "@zip.js/zip.js": "^2.7.48",
         "c8": "^10.1.2",
         "chokidar": "^3.6.0",
         "eslint": "^9.8.0",
         "linkedom": "^0.18.4",
-        "rollup": "^4.19.1",
+        "rollup": "^4.19.2",
         "static-handler": "^0.4.3",
         "typescript": "^5.5.4"
     },
@@ -78,14 +78,14 @@
         "@webreflection/idb-map": "^0.3.1",
         "basic-devtools": "^0.1.6",
         "codedent": "^0.1.2",
-        "coincident": "^2.0.1",
-        "gc-hook": "^0.3.1",
+        "coincident": "^2.0.5",
+        "gc-hook": "^0.4.1",
         "html-escaper": "^3.0.3",
         "proxy-target": "^3.0.2",
         "sticky-module": "^0.1.1",
         "to-json-callback": "^0.1.1"
     },
     "worker": {
-        "blob": "sha256-TcNhn9gQ+KzsuVdb4meFvtbYIbGO2LqYd62IMVpt9UM="
+        "blob": "sha256-daao4AY1C1yVgnZJI/0/Oz4VbjaD4MfpJGh+wR8sICk="
     }
 }