@jsenv/core 40.0.7 → 40.0.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "40.0.7",
+  "version": "40.0.8",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -78,22 +78,22 @@
   "dependencies": {
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.3.1",
-    "@jsenv/ast": "6.6.5",
-    "@jsenv/filesystem": "4.14.3",
+    "@jsenv/ast": "6.6.6",
+    "@jsenv/filesystem": "4.14.4",
     "@jsenv/humanize": "1.3.1",
     "@jsenv/importmap": "1.2.2",
     "@jsenv/integrity": "0.0.2",
-    "@jsenv/js-module-fallback": "1.4.3",
+    "@jsenv/js-module-fallback": "1.4.4",
     "@jsenv/node-esm-resolution": "1.1.0",
-    "@jsenv/plugin-bundling": "2.8.4",
+    "@jsenv/plugin-bundling": "2.8.5",
     "@jsenv/plugin-minification": "1.6.2",
-    "@jsenv/plugin-supervisor": "1.6.10",
-    "@jsenv/plugin-transpilation": "1.5.6",
+    "@jsenv/plugin-supervisor": "1.6.11",
+    "@jsenv/plugin-transpilation": "1.5.7",
     "@jsenv/runtime-compat": "1.3.4",
     "@jsenv/server": "16.0.4",
-    "@jsenv/sourcemap": "1.3.2",
+    "@jsenv/sourcemap": "1.3.3",
     "@jsenv/url-meta": "8.5.5",
-    "@jsenv/urls": "2.6.3",
+    "@jsenv/urls": "2.7.0",
     "@jsenv/utils": "2.2.1",
     "string-width": "7.2.0"
   },

package/src/build/build.js

@@ -179,7 +179,7 @@ export const build = async ({
           outDirectoryUrl = `${packageDirectoryUrl}.jsenv/`;
         }
       }
-    } else if (outDirectoryUrl !== null && outDirectoryUrl !== false) {
+    } else if (outDirectoryUrl) {
       outDirectoryUrl = assertAndNormalizeDirectoryUrl(
         outDirectoryUrl,
         "outDirectoryUrl",
@@ -307,6 +307,7 @@ build ${entryPointKeys.length} entry points`);
           minification,
           versioning,
           versioningMethod,
+          outDirectoryUrl,
         },
         onCustomBuildDirectory: (subBuildRelativeUrl) => {
           buildDirectoryCleanPatterns = {
@@ -528,30 +529,27 @@ build ${entryPointKeys.length} entry points`);
               }
               const referencedUrlInfo = referenceToOther.urlInfo;
               if (referencedUrlInfo.isInline) {
-                if (referencedUrlInfo.type === "js_module") {
-                  // bundle inline script type module deps
-                  referencedUrlInfo.referenceToOthersSet.forEach(
-                    (jsModuleReferenceToOther) => {
-                      if (jsModuleReferenceToOther.type === "js_import") {
-                        const inlineUrlInfo = jsModuleReferenceToOther.urlInfo;
-                        addToBundlerIfAny(inlineUrlInfo);
-                      }
-                    },
-                  );
+                if (referencedUrlInfo.type !== "js_module") {
+                  continue;
                 }
-                // inline content cannot be bundled
+                addToBundlerIfAny(referencedUrlInfo);
                 continue;
               }
               addToBundlerIfAny(referencedUrlInfo);
             }
             return;
           }
-          // File referenced with new URL('./file.js', import.meta.url)
+          // File referenced with
+          // - new URL("./file.js", import.meta.url)
+          // - import.meta.resolve("./file.js")
           // are entry points that should be bundled
           // For instance we will bundle service worker/workers detected like this
           if (rawUrlInfo.type === "js_module") {
             for (const referenceToOther of rawUrlInfo.referenceToOthersSet) {
-              if (referenceToOther.type === "js_url") {
+              if (
+                referenceToOther.type === "js_url" ||
+                referenceToOther.subtype === "import_meta_resolve"
+              ) {
                 const referencedUrlInfo = referenceToOther.urlInfo;
                 let isAlreadyBundled = false;
                 for (const referenceFromOther of referencedUrlInfo.referenceFromOthersSet) {

package/src/build/build_specifier_manager.js

@@ -238,6 +238,7 @@ export const createBuildSpecifierManager = ({
         js_classic: undefined, // TODO: add comment to explain who is using this
         entry_point: undefined,
         dynamic_import: undefined,
+        dynamic_import_id: undefined,
       };
     },
     formatReference: (reference) => {
@@ -276,6 +277,9 @@ export const createBuildSpecifierManager = ({
       const bundleInfo = bundleInfoMap.get(rawUrl);
       if (bundleInfo) {
         finalUrlInfo.remapReference = bundleInfo.remapReference;
+        if (!finalUrlInfo.filenameHint && bundleInfo.data.bundleRelativeUrl) {
+          finalUrlInfo.filenameHint = bundleInfo.data.bundleRelativeUrl;
+        }
         return {
           // url: bundleInfo.url,
           originalUrl: bundleInfo.originalUrl,

package/src/build/build_urls_generator.js

@@ -131,6 +131,10 @@ const determineDirectoryPath = ({
     });
     return parentDirectoryPath;
   }
+  const dynamicImportId = urlInfo.searchParams.get("dynamic_import_id");
+  if (dynamicImportId) {
+    return `${assetsDirectory}${dynamicImportId}/`;
+  }
   if (urlInfo.isEntryPoint && !urlInfo.isDynamicEntryPoint) {
     return "";
   }

package/src/build/jsenv_plugin_subbuilds.js

@@ -18,6 +18,10 @@ export const jsenvPluginSubbuilds = (
       },
       ...defaultChildBuildParams,
       ...subBuildParams,
+      outDirectoryUrl: new URL(
+        `./subbuild_${index}/`,
+        parentBuildParams.outDirectoryUrl,
+      ),
     };
     const subBuildDirectoryUrl = subBuildParams.buildDirectoryUrl;
     if (subBuildDirectoryUrl) {

package/src/kitchen/out_directory_url.js

@@ -40,6 +40,7 @@ export const determineSourcemapFileUrl = (urlInfo) => {
   generatedUrlObject.searchParams.delete("as_json_module");
   generatedUrlObject.searchParams.delete("as_text_module");
   generatedUrlObject.searchParams.delete("dynamic_import");
+  generatedUrlObject.searchParams.delete("dynamic_import_id");
   generatedUrlObject.searchParams.delete("cjs_as_js_module");
   const urlForSourcemap = generatedUrlObject.href;
   return generateSourcemapFileUrl(urlForSourcemap);

package/src/plugins/protocol_http/jsenv_plugin_protocol_http.js

@@ -36,6 +36,12 @@ export const jsenvPluginProtocolHttp = ({ include }) => {
     //   }
     //   return null;
     // },
+    init: (context) => {
+      const outDirectoryUrl = context.outDirectoryUrl;
+      if (!outDirectoryUrl) {
+        throw new Error(`need outDirectoryUrl to write http files`);
+      }
+    },
     redirectReference: (reference) => {
       if (!reference.url.startsWith("http")) {
         return null;

package/src/plugins/reference_analysis/html/jsenv_plugin_html_reference_analysis.js

@@ -315,7 +315,12 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
                   });
                 } else {
                   setHtmlNodeText(node, inlineUrlInfo.content, {
-                    indentation: false, // indentation would decrease stack trace precision
+                    indentation:
+                      inlineUrlInfo.type === "js_classic" ||
+                      inlineUrlInfo.type === "js_module"
+                        ? // indentation would mess with stack trace and sourcemap
+                          false
+                        : "auto",
                   });
                   setHtmlNodeAttributes(node, {
                     "jsenv-cooked-by": "jsenv:html_inline_content_analysis",