@jsenv/core 28.4.1 → 28.5.0

package/dist/js/supervisor.js

@@ -277,6 +277,7 @@ window.__supervisor__ = (() => {
 
     const getErrorStackWithoutErrorMessage = error => {
       let stack = error.stack;
+      if (!stack) return "";
       const messageInStack = `${error.name}: ${error.message}`;
 
       if (stack.startsWith(messageInStack)) {

package/dist/main.js

@@ -10301,15 +10301,24 @@ const jsenvPluginAsJsClassicHtml = ({
           }
 
           if (needsSystemJs) {
-            mutations.push(() => {
-              const [systemJsReference] = context.referenceUtils.inject({
+            mutations.push(async () => {
+              const systemJsFileContent = readFileSync$1(new URL(systemJsClientFileUrl), {
+                encoding: "utf8"
+              });
+              const [systemJsReference, systemJsUrlInfo] = context.referenceUtils.inject({
                 type: "script_src",
                 expectedType: "js_classic",
-                specifier: systemJsClientFileUrl
+                isInline: true,
+                contentType: "text/javascript",
+                content: systemJsFileContent,
+                specifier: "s.js"
+              });
+              await context.cook(systemJsUrlInfo, {
+                reference: systemJsReference
               });
               injectScriptNodeAsEarlyAsPossible(htmlAst, createHtmlNode({
                 tagName: "script",
-                src: systemJsReference.generatedSpecifier
+                textContent: systemJsUrlInfo.content
               }), "jsenv:as_js_classic_html");
             });
           }
@@ -10319,7 +10328,7 @@ const jsenvPluginAsJsClassicHtml = ({
           return null;
         }
 
-        mutations.forEach(mutation => mutation());
+        await Promise.all(mutations.map(mutation => mutation()));
         return stringifyHtmlAst(htmlAst);
       }
     }
@@ -10556,7 +10565,8 @@ const bundleJsModules = async ({
   const {
     babelHelpersChunk = true,
     include,
-    preserveDynamicImport = false
+    preserveDynamicImport = false,
+    strictExports = false
   } = options;
   const {
     jsModuleBundleUrlInfos
@@ -10571,7 +10581,8 @@ const bundleJsModules = async ({
     sourcemaps,
     include,
     babelHelpersChunk,
-    preserveDynamicImport
+    preserveDynamicImport,
+    strictExports
   });
   return jsModuleBundleUrlInfos;
 };
@@ -10586,6 +10597,7 @@ const rollupPluginJsenv = ({
   include,
   babelHelpersChunk,
   preserveDynamicImport,
+  strictExports,
   resultRef
 }) => {
   let _rollupEmitFile = () => {
@@ -10636,8 +10648,8 @@ const rollupPluginJsenv = ({
 
         emitChunk({
           id,
-          implicitlyLoadedAfterOneOf: previousNonEntryPointModuleId ? [previousNonEntryPointModuleId] : null // preserveSignature: "allow-extension",
-
+          implicitlyLoadedAfterOneOf: previousNonEntryPointModuleId ? [previousNonEntryPointModuleId] : null,
+          preserveSignature: strictExports ? "strict" : jsModuleUrlInfo.dependents.size < 2 ? "allow-extension" : "strict"
         });
         previousNonEntryPointModuleId = id;
       });
@@ -20213,13 +20225,8 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
           // - happens for "as_js_classic" injecting "s.js"
 
           if (reference.injected) {
-            const [ref, rawUrlInfo] = rawGraphKitchen.injectReference({
-              type: reference.type,
-              expectedType: reference.expectedType,
-              expectedSubtype: reference.expectedSubtype,
-              parentUrl: buildToRawUrls[reference.parentUrl],
-              specifier: reference.specifier,
-              injected: true
+            const [ref, rawUrlInfo] = rawGraphKitchen.injectReference({ ...reference,
+              parentUrl: buildToRawUrls[reference.parentUrl]
             });
             await rawGraphKitchen.cook(rawUrlInfo, {
               reference: ref

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "28.4.1",
+  "version": "28.5.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {

package/src/build/build.js

@@ -528,12 +528,8 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
             // - happens for "as_js_classic" injecting "s.js"
             if (reference.injected) {
               const [ref, rawUrlInfo] = rawGraphKitchen.injectReference({
-                type: reference.type,
-                expectedType: reference.expectedType,
-                expectedSubtype: reference.expectedSubtype,
+                ...reference,
                 parentUrl: buildToRawUrls[reference.parentUrl],
-                specifier: reference.specifier,
-                injected: true,
               })
               await rawGraphKitchen.cook(rawUrlInfo, { reference: ref })
               return rawUrlInfo

package/src/plugins/bundling/js_module/bundle_js_modules.js

@@ -39,6 +39,7 @@ export const bundleJsModules = async ({
     babelHelpersChunk = true,
     include,
     preserveDynamicImport = false,
+    strictExports = false,
   } = options
   const { jsModuleBundleUrlInfos } = await buildWithRollup({
     signal,
@@ -54,6 +55,7 @@ export const bundleJsModules = async ({
     include,
     babelHelpersChunk,
     preserveDynamicImport,
+    strictExports,
   })
   return jsModuleBundleUrlInfos
 }
@@ -69,6 +71,7 @@ const rollupPluginJsenv = ({
   include,
   babelHelpersChunk,
   preserveDynamicImport,
+  strictExports,
 
   resultRef,
 }) => {
@@ -116,7 +119,11 @@ const rollupPluginJsenv = ({
           implicitlyLoadedAfterOneOf: previousNonEntryPointModuleId
             ? [previousNonEntryPointModuleId]
             : null,
-          // preserveSignature: "allow-extension",
+          preserveSignature: strictExports
+            ? "strict"
+            : jsModuleUrlInfo.dependents.size < 2
+            ? "allow-extension"
+            : "strict",
         })
         previousNonEntryPointModuleId = id
       })

package/src/plugins/supervisor/client/supervisor.js

@@ -237,6 +237,7 @@ window.__supervisor__ = (() => {
 
     const getErrorStackWithoutErrorMessage = (error) => {
       let stack = error.stack
+      if (!stack) return ""
       const messageInStack = `${error.name}: ${error.message}`
       if (stack.startsWith(messageInStack)) {
         stack = stack.slice(messageInStack.length)

package/src/plugins/transpilation/as_js_classic/jsenv_plugin_as_js_classic_html.js

@@ -5,6 +5,7 @@
  * - <link rel="modulepreload"> are converted to <link rel="preload">
  */
 
+import { readFileSync } from "node:fs"
 import {
   parseHtmlString,
   visitHtmlNodes,
@@ -160,17 +161,28 @@ export const jsenvPluginAsJsClassicHtml = ({
             }
           }
           if (needsSystemJs) {
-            mutations.push(() => {
-              const [systemJsReference] = context.referenceUtils.inject({
-                type: "script_src",
-                expectedType: "js_classic",
-                specifier: systemJsClientFileUrl,
+            mutations.push(async () => {
+              const systemJsFileContent = readFileSync(
+                new URL(systemJsClientFileUrl),
+                { encoding: "utf8" },
+              )
+              const [systemJsReference, systemJsUrlInfo] =
+                context.referenceUtils.inject({
+                  type: "script_src",
+                  expectedType: "js_classic",
+                  isInline: true,
+                  contentType: "text/javascript",
+                  content: systemJsFileContent,
+                  specifier: "s.js",
+                })
+              await context.cook(systemJsUrlInfo, {
+                reference: systemJsReference,
               })
               injectScriptNodeAsEarlyAsPossible(
                 htmlAst,
                 createHtmlNode({
                   tagName: "script",
-                  src: systemJsReference.generatedSpecifier,
+                  textContent: systemJsUrlInfo.content,
                 }),
                 "jsenv:as_js_classic_html",
               )
@@ -180,7 +192,7 @@ export const jsenvPluginAsJsClassicHtml = ({
         if (mutations.length === 0) {
           return null
         }
-        mutations.forEach((mutation) => mutation())
+        await Promise.all(mutations.map((mutation) => mutation()))
         return stringifyHtmlAst(htmlAst)
       },
     },