@jsenv/core 24.6.3 → 25.0.0-alpha.1

package/src/execute.js

@@ -46,6 +46,9 @@ export const execute = async ({
   port,
   babelPluginMap,
   customCompilers,
+  workers,
+  serviceWorkers,
+  importMapInWebWorkers,
   compileServerCanReadFromFilesystem,
   compileServerCanWriteOnFilesystem,
 
@@ -105,10 +108,13 @@ export const execute = async ({
       ip,
       port,
       babelPluginMap,
-      customCompilers,
       runtimeSupport: normalizeRuntimeSupport({
         [runtime.name]: runtime.version,
       }),
+      customCompilers,
+      workers,
+      serviceWorkers,
+      importMapInWebWorkers,
       compileServerCanReadFromFilesystem,
       compileServerCanWriteOnFilesystem,
     })

package/src/executeTestPlan.js

@@ -79,6 +79,9 @@ export const executeTestPlan = async ({
   compileServerCanWriteOnFilesystem,
   babelPluginMap,
   babelConfigFileUrl,
+  workers,
+  serviceWorkers,
+  importMapInWebWorkers,
   customCompilers,
   jsenvDirectoryClean,
 }) => {
@@ -179,6 +182,9 @@ export const executeTestPlan = async ({
     compileServerCanWriteOnFilesystem,
     babelPluginMap,
     babelConfigFileUrl,
+    workers,
+    serviceWorkers,
+    importMapInWebWorkers,
     customCompilers,
   })
 

package/src/internal/browser_feature_detection/browser_feature_detection.js

@@ -36,17 +36,14 @@ export const scanBrowserRuntimeFeatures = async ({
     failFastOnFeatureDetection,
     inlineImportMapIntoHTML,
   })
-  const pluginRequiredNameArray = await pluginRequiredNamesFromGroupInfo(
-    groupInfo,
-    {
-      featuresReport,
-      coverageHandledFromOutside,
-    },
-  )
+  const missingFeatureNames = await adjustMissingFeatureNames(groupInfo, {
+    featuresReport,
+    coverageHandledFromOutside,
+  })
 
   const canAvoidCompilation =
     customCompilerPatterns.length === 0 &&
-    pluginRequiredNameArray.length === 0 &&
+    missingFeatureNames.length === 0 &&
     featuresReport.importmap &&
     featuresReport.dynamicImport &&
     featuresReport.topLevelAwait
@@ -55,7 +52,7 @@ export const scanBrowserRuntimeFeatures = async ({
     canAvoidCompilation,
     featuresReport,
     customCompilerPatterns,
-    pluginRequiredNameArray,
+    missingFeatureNames,
     inlineImportMapIntoHTML,
     outDirectoryRelativeUrl,
     compileId,
@@ -63,27 +60,25 @@ export const scanBrowserRuntimeFeatures = async ({
   }
 }
 
-const pluginRequiredNamesFromGroupInfo = async (
+const adjustMissingFeatureNames = async (
   groupInfo,
   { featuresReport, coverageHandledFromOutside },
 ) => {
-  const { pluginRequiredNameArray } = groupInfo
-  const requiredPluginNames = pluginRequiredNameArray.slice()
-  const markPluginAsSupported = (name) => {
-    const index = requiredPluginNames.indexOf(name)
+  const { missingFeatureNames } = groupInfo
+  const missingFeatureNamesCopy = missingFeatureNames.slice()
+  const markAsSupported = (name) => {
+    const index = missingFeatureNamesCopy.indexOf(name)
     if (index > -1) {
-      requiredPluginNames.splice(index, 1)
+      missingFeatureNamesCopy.splice(index, 1)
     }
   }
-
   // When instrumentation CAN be handed by playwright
   // https://playwright.dev/docs/api/class-chromiumcoverage#chromiumcoveragestartjscoverageoptions
   // coverageHandledFromOutside is true and "transform-instrument" becomes non mandatory
   if (coverageHandledFromOutside) {
-    markPluginAsSupported("transform-instrument")
+    markAsSupported("transform-instrument")
   }
-
-  if (pluginRequiredNameArray.includes("transform-import-assertions")) {
+  if (missingFeatureNames.includes("transform-import-assertions")) {
     const jsonImportAssertions = await supportsJsonImportAssertions()
     featuresReport.jsonImportAssertions = jsonImportAssertions
 
@@ -91,17 +86,15 @@ const pluginRequiredNamesFromGroupInfo = async (
     featuresReport.cssImportAssertions = cssImportAssertions
 
     if (jsonImportAssertions && cssImportAssertions) {
-      markPluginAsSupported("transform-import-assertions")
+      markAsSupported("transform-import-assertions")
     }
   }
-
-  if (pluginRequiredNameArray.includes("new-stylesheet-as-jsenv-import")) {
+  if (missingFeatureNames.includes("new-stylesheet-as-jsenv-import")) {
     const newStylesheet = supportsNewStylesheet()
     featuresReport.newStylesheet = newStylesheet
-    markPluginAsSupported("new-stylesheet-as-jsenv-import")
+    markAsSupported("new-stylesheet-as-jsenv-import")
   }
-
-  return requiredPluginNames
+  return missingFeatureNamesCopy
 }
 
 const detectSupportedFeatures = async ({

package/src/internal/browser_runtime/browser_runtime.js

@@ -127,12 +127,12 @@ const onExecutionError = (
   },
 ) => {
   const error = executionResult.error
-  if (error.code === "NETWORK_FAILURE") {
+  if (error && error.code === "NETWORK_FAILURE") {
     if (currentScript) {
       const errorEvent = new Event("error")
       currentScript.dispatchEvent(errorEvent)
     }
-  } else {
+  } else if (typeof error === "object") {
     const { parsingError } = error
     const globalErrorEvent = new Event("error")
     if (parsingError) {

package/src/internal/browser_runtime/createBrowserRuntime.js

@@ -40,7 +40,7 @@ export const createBrowserRuntime = async ({
   // if there is an importmap in the document we use it instead of fetching.
   // systemjs style with systemjs-importmap
   const importmapScript = document.querySelector(
-    `script[type="jsenv-importmap"]`,
+    `script[type="systemjs-importmap"]`,
   )
   let importMap
   let importMapUrl

package/src/internal/browser_runtime/displayErrorInDocument.js

@@ -102,6 +102,8 @@ const errorToHTML = (error) => {
     }
   } else if (typeof error === "string") {
     html = error
+  } else if (error === undefined) {
+    html = "undefined"
   } else {
     html = JSON.stringify(error)
   }

package/src/internal/browser_runtime/displayErrorNotification.js

@@ -7,7 +7,7 @@ const displayErrorNotificationImplementation = (error, { icon } = {}) => {
   if (Notification.permission === "granted") {
     const notification = new Notification("An error occured", {
       lang: "en",
-      body: error.stack,
+      body: error ? error.stack : "undefined",
       icon,
     })
     notification.onclick = () => {

package/src/internal/building/buildUsingRollup.js

@@ -13,7 +13,6 @@ import {
   isNodePartOfSupportedRuntimes,
   isBrowserPartOfSupportedRuntimes,
 } from "@jsenv/core/src/internal/generateGroupMap/runtime_support.js"
-import { featuresCompatMap } from "@jsenv/core/src/internal/generateGroupMap/featuresCompatMap.js"
 import { createRuntimeCompat } from "@jsenv/core/src/internal/generateGroupMap/runtime_compat.js"
 import { createRollupPlugins } from "./rollup_plugin_jsenv.js"
 
@@ -22,7 +21,7 @@ export const buildUsingRollup = async ({
   logger,
 
   projectDirectoryUrl,
-  entryPointMap,
+  entryPoints,
   compileServerOrigin,
   compileDirectoryRelativeUrl,
   buildDirectoryUrl,
@@ -38,7 +37,6 @@ export const buildUsingRollup = async ({
   globals,
   babelPluginMap,
   runtimeSupport,
-  transformTopLevelAwait,
 
   urlMappings,
   importResolutionMethod,
@@ -50,9 +48,10 @@ export const buildUsingRollup = async ({
   workers,
   serviceWorkers,
   serviceWorkerFinalizer,
+  classicWorkers,
+  classicServiceWorkers,
 
   urlVersioning,
-  urlVersionningForEntryPoints,
   lineBreakNormalization,
   jsConcatenation,
   cssConcatenation,
@@ -70,21 +69,17 @@ export const buildUsingRollup = async ({
 
   const runtimeCompatMap = createRuntimeCompat({
     runtimeSupport,
-    pluginMap: {
-      import_assertion_type_json: true,
-      import_assertion_type_css: true,
-    },
-    pluginCompatMap: featuresCompatMap,
+    featureNames: ["import_assertion_type_json", "import_assertion_type_css"],
   })
   const importAssertionsSupport = {
     json:
       format === "esmodule" &&
-      !runtimeCompatMap.pluginRequiredNameArray.includes(
+      !runtimeCompatMap.missingFeatureNames.includes(
         "import_assertion_type_json",
       ),
     css:
       format === "esmodule" &&
-      !runtimeCompatMap.pluginRequiredNameArray.includes(
+      !runtimeCompatMap.missingFeatureNames.includes(
         "import_assertion_type_json",
       ),
   }
@@ -100,7 +95,7 @@ export const buildUsingRollup = async ({
     logger,
 
     projectDirectoryUrl,
-    entryPointMap,
+    entryPoints,
     compileServerOrigin,
     compileDirectoryRelativeUrl,
     buildDirectoryUrl,
@@ -108,7 +103,6 @@ export const buildUsingRollup = async ({
     format,
     systemJsUrl,
     babelPluginMap,
-    transformTopLevelAwait,
     node,
     browser,
     importAssertionsSupport,
@@ -123,9 +117,10 @@ export const buildUsingRollup = async ({
     workers,
     serviceWorkers,
     serviceWorkerFinalizer,
+    classicWorkers,
+    classicServiceWorkers,
 
     urlVersioning,
-    urlVersionningForEntryPoints,
     lineBreakNormalization,
     jsConcatenation,
     cssConcatenation,
@@ -206,7 +201,7 @@ export const buildUsingRollup = async ({
     buildFileContents,
     buildInlineFileContents,
     buildStats,
-  } = getResult()
+  } = await getResult()
 
   if (writeOnFileSystem) {
     if (buildDirectoryClean) {
@@ -266,7 +261,7 @@ const useRollup = async ({
 
   const rollupInputOptions = {
     // about cache here, we should/could reuse previous rollup call
-    // to get the cache from the entryPointMap
+    // to get the cache from the entryPoints
     // as shown here: https://rollupjs.org/guide/en#cache
     // it could be passed in arguments to this function
     // however parallelism and having different rollup options per
@@ -309,7 +304,7 @@ const useRollup = async ({
       logger.warn(String(warning))
     },
     // on passe input: [] car c'est le plugin jsenv qui se chargera d'emit des chunks
-    // en fonction de entryPointMap
+    // en fonction de entryPoints
     // on fait cela car sinon rollup est pénible si on passe un input de type html
     input: [],
     preserveEntrySignatures,

package/src/internal/building/build_logs.js

@@ -5,8 +5,8 @@ import { byteAsFileSize } from "../logs/byteAsFileSize.js"
 import { msAsDuration } from "../logs/msAsDuration.js"
 import { stringifyUrlSite } from "./url_trace.js"
 
-export const formatBuildStartLog = ({ entryPointMap }) => {
-  const entryFileRelativeUrls = Object.keys(entryPointMap)
+export const formatBuildStartLog = ({ entryPoints }) => {
+  const entryFileRelativeUrls = Object.keys(entryPoints)
   if (entryFileRelativeUrls.length === 1) {
     return `
 building ${entryFileRelativeUrls[0]}...`

package/src/internal/building/build_stats.js

@@ -59,7 +59,13 @@ const getProjectFileContents = (ressourceBuilder) => {
 
   Object.keys(ressourceMap).forEach((url) => {
     const ressource = ressourceMap[url]
-    const { isInline, isExternal, isPlaceholder, bufferBeforeBuild } = ressource
+    const {
+      isInline,
+      isExternal,
+      isPlaceholder,
+      isSourcemap,
+      bufferBeforeBuild,
+    } = ressource
     if (isInline) {
       // inline ressources are not files
       return
@@ -80,6 +86,10 @@ const getProjectFileContents = (ressourceBuilder) => {
       // to generate it in the build
       return
     }
+    if (isSourcemap && !bufferBeforeBuild) {
+      // sourcemap generated by rollup do not have bufferBeforeBuild
+      return
+    }
     projectFileContents[url] = bufferBeforeBuild
   })
   return projectFileContents

package/src/internal/building/html/parseHtmlRessource.js

@@ -16,12 +16,7 @@ Or be sure to also reference this url somewhere in the html file like
 
 */
 
-import {
-  urlToFilename,
-  urlToRelativeUrl,
-  resolveUrl,
-  urlToParentUrl,
-} from "@jsenv/filesystem"
+import { urlToFilename, urlToRelativeUrl, resolveUrl } from "@jsenv/filesystem"
 
 import {
   parseHtmlString,
@@ -255,7 +250,7 @@ const regularScriptTextNodeVisitor = (
   })
   return () => {
     const { bufferAfterBuild } = jsReference.ressource
-    textNode.value = bufferAfterBuild
+    textNode.value = String(bufferAfterBuild)
   }
 }
 
@@ -359,8 +354,6 @@ const moduleScriptTextNodeVisitor = (
     }
     const { bufferAfterBuild } = jsReference.ressource
     const jsText = String(bufferAfterBuild)
-    // ici on voudrait pouvoir ajouter le commentaire de la sourcemap
-    // sauf que cela se produit un poil plus tard je crois?
     textNode.value = jsText
   }
 }
@@ -386,23 +379,6 @@ const importmapScriptSrcVisitor = (
     contentTypeExpected: "application/importmap+json",
     ressourceSpecifier: srcAttribute.value,
     ...referenceLocationFromHtmlNode(script, "src"),
-    // here we want to force the fileName for the importmap
-    // so that we don't have to rewrite its content
-    // the goal is to put the importmap at the same relative path
-    // than in the project
-    fileNamePattern: () => {
-      const importmapReferenceUrl = importmapReference.referenceUrl
-      const importmapRessourceUrl = importmapReference.ressource.url
-      const importmapUrlRelativeToImporter = urlToRelativeUrl(
-        importmapRessourceUrl,
-        importmapReferenceUrl,
-      )
-      const importmapParentRelativeUrl = urlToRelativeUrl(
-        urlToParentUrl(resolveUrl(importmapUrlRelativeToImporter, "file://")),
-        "file://",
-      )
-      return `${importmapParentRelativeUrl}[name]-[hash][extname]`
-    },
   })
   return ({ getUrlRelativeToImporter }) => {
     const { ressource } = importmapReference

package/src/internal/building/js/parseJsRessource.js

@@ -103,9 +103,10 @@ export const parseJsRessource = async (
     map.file = urlToFilename(jsBuildUrl)
     if (map.sources) {
       map.sources = map.sources.map((source) => {
-        const sourceUrl = resolveUrl(source, jsOriginalUrl)
+        const sourceUrl = resolveUrl(source, jsUrl)
+        const sourceOriginalUrl = asOriginalUrl(sourceUrl)
         const sourceUrlRelativeToSourceMap = urlToRelativeUrl(
-          sourceUrl,
+          sourceOriginalUrl,
           sourcemapPrecomputedBuildUrl,
         )
         return sourceUrlRelativeToSourceMap

package/src/internal/building/json_module.js

@@ -0,0 +1,11 @@
+export const convertJsonTextToJavascriptModule = ({ code }) => {
+  // here we could do the following
+  // return export default jsonText
+  // This would return valid js, that would be minified later
+  // however we will prefer using JSON.parse because it's faster
+  // for js engine to parse JSON than JS
+
+  return {
+    code: `export default JSON.parse(${JSON.stringify(code.trim())})`,
+  }
+}

package/src/internal/building/parseRessource.js

@@ -22,7 +22,7 @@ import { parseJsRessource } from "./js/parseJsRessource.js"
 import { parseJsonRessource } from "./json/parseJsonRessource.js"
 import { parseWebmanifestRessource } from "./webmanifest/parseWebmanifestRessource.js"
 
-export const parseRessource = (
+export const parseRessource = async (
   ressource,
   notifiers,
   {