@jsenv/core 30.3.9 → 30.4.1

package/src/execute/runtimes/browsers/webkit.js

@@ -2,7 +2,9 @@ import { createRuntimeFromPlaywright } from "./from_playwright.js"
 
 export const webkit = createRuntimeFromPlaywright({
   browserName: "webkit",
-  browserVersion: "16.4", // to update, check https://github.com/microsoft/playwright/releases
+  // browserVersion will be set by "browser._initializer.version"
+  // see also https://github.com/microsoft/playwright/releases
+  browserVersion: "unset",
   ignoreErrorHook: (error) => {
     // we catch error during execution but safari throw unhandled rejection
     // in a non-deterministic way.

package/src/execute/runtimes/readme.md

@@ -0,0 +1,13 @@
+# runtimes/
+
+Code implementing runtimes can be found here.
+
+# Description
+
+A runtime is an object with a "run" method.
+The run method is roughly doing the following:
+
+1. spawn a runtime (browser,Node.js)
+2. set various listeners to monitor file execution
+3. execute the file
+4. return info about file execution (logs and errors for instance)

package/src/kitchen/kitchen.js

@@ -356,7 +356,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         type,
         subtype,
         originalUrl,
-        originalContent,
+        originalContent = content,
         sourcemap,
         filename,
 
@@ -381,8 +381,13 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
       urlInfo.subtype = subtype || reference.expectedSubtype || ""
       // during build urls info are reused and load returns originalUrl/originalContent
       urlInfo.originalUrl = originalUrl || urlInfo.originalUrl
-      urlInfo.originalContent =
-        originalContent === undefined ? content : originalContent
+      if (originalContent !== urlInfo.originalContent) {
+        urlInfo.originalContentEtag = undefined // set by "initTransformations"
+      }
+      if (content !== urlInfo.content) {
+        urlInfo.contentEtag = undefined // set by "applyFinalTransformations"
+      }
+      urlInfo.originalContent = originalContent
       urlInfo.content = content
       urlInfo.sourcemap = sourcemap
       if (data) {

package/src/kitchen/readme.md

@@ -0,0 +1,8 @@
+# kitchen/
+
+Code implementing jsenv kitchen can be found here.
+
+# Description
+
+Jsenv kitchen is an abstraction representing the place where files content is transformed to obtain something.
+Jsenv dev server and build go through the kitchen to transform file content.

package/src/kitchen/url_graph.js

@@ -37,12 +37,30 @@ export const createUrlGraph = () => {
     if (!parentUrlInfo) {
       return null
     }
-    const firstReferenceOnThatUrl = parentUrlInfo.references.find(
-      (reference) => {
+    const seen = []
+    const search = (urlInfo) => {
+      const firstReferenceFound = urlInfo.references.find((reference) => {
         return urlSpecifierEncoding.decode(reference) === specifier
-      },
-    )
-    return firstReferenceOnThatUrl
+      })
+      if (firstReferenceFound) {
+        return firstReferenceFound
+      }
+      for (const dependencyUrl of parentUrlInfo.dependencies) {
+        if (seen.includes(dependencyUrl)) {
+          continue
+        }
+        seen.push(dependencyUrl)
+        const dependencyUrlInfo = getUrlInfo(dependencyUrl)
+        if (dependencyUrlInfo.isInline) {
+          const firstRef = search(dependencyUrlInfo)
+          if (firstRef) {
+            return firstRef
+          }
+        }
+      }
+      return null
+    }
+    return search(parentUrlInfo)
   }
   const findDependent = (urlInfo, visitor) => {
     const seen = [urlInfo.url]

package/src/plugins/cache_control/jsenv_plugin_cache_control.js

@@ -1,15 +1,19 @@
-export const jsenvPluginCacheControl = () => {
+export const jsenvPluginCacheControl = ({
+  versionedUrls = true,
+  maxAge = SECONDS_IN_30_DAYS,
+}) => {
   return {
     name: "jsenv:cache_control",
     appliesDuring: "dev",
     augmentResponse: ({ reference }) => {
       if (
+        versionedUrls &&
         reference.searchParams.has("v") &&
         !reference.searchParams.has("hmr")
       ) {
         return {
           headers: {
-            "cache-control": `private,max-age=${SECONDS_IN_30_DAYS},immutable`,
+            "cache-control": `private,max-age=${maxAge},immutable`,
           },
         }
       }

package/src/plugins/plugins.js

@@ -38,11 +38,15 @@ export const getCorePlugins = ({
   clientFileChangeCallbackList,
   clientFilesPruneCallbackList,
   explorer,
+  cacheControl,
   ribbon = true,
 } = {}) => {
   if (explorer === true) {
     explorer = {}
   }
+  if (cacheControl === true) {
+    cacheControl = {}
+  }
   if (supervisor === true) {
     supervisor = {}
   }
@@ -59,6 +63,7 @@ export const getCorePlugins = ({
   } else {
     clientMainFileUrl = String(clientMainFileUrl)
   }
+
   if (ribbon === true) {
     ribbon = {}
   }
@@ -97,7 +102,7 @@ export const getCorePlugins = ({
           }),
         ]
       : []),
-    jsenvPluginCacheControl(),
+    ...(cacheControl ? [jsenvPluginCacheControl(cacheControl)] : []),
     ...(explorer
       ? [jsenvPluginExplorer({ ...explorer, clientMainFileUrl })]
       : []),

package/src/plugins/readme.md

@@ -0,0 +1,18 @@
+# plugins/
+
+Code implementing jsenv internal plugins can be found here.
+
+# Description
+
+These plugins can be configured using parameters of "startDevServer" and "build" functions.
+
+A few examples of plugins that can be found here:
+
+- inject code to autoreload during dev
+- apply node module resolution on js imports
+
+They are here and not inside a separate NPM package because:
+
+1. they are considered useful enough to be available by default
+2. putting them into a separate package would force people to enable a bunch of plugins
+   to obtain what they want instead of having sensible defaults

package/src/plugins/server_events/client/server_events_client.js

@@ -1,5 +1,3 @@
-/* globals self */
-
 import { createWebSocketConnection } from "./web_socket_connection.js"
 
 const websocketScheme = self.location.protocol === "https:" ? "wss" : "ws"

package/src/plugins/transpilation/as_js_classic/client/s.js

@@ -12,7 +12,6 @@
 
 ;(function () {
   /* eslint-env browser */
-  /* globals self */
 
   const loadRegistry = Object.create(null)
   const registerRegistry = Object.create(null)

package/src/plugins/transpilation/babel/jsenv_plugin_babel.js

@@ -5,18 +5,9 @@ import { babelPluginInstrument } from "@jsenv/core/src/test/coverage/babel_plugi
 import { RUNTIME_COMPAT } from "@jsenv/core/src/kitchen/compat/runtime_compat.js"
 import { getBaseBabelPluginStructure } from "./helpers/babel_plugin_structure.js"
 import { babelPluginBabelHelpersAsJsenvImports } from "./helpers/babel_plugin_babel_helpers_as_jsenv_imports.js"
-import {
-  babelPluginNewStylesheetAsJsenvImport,
-  newStylesheetClientFileUrl,
-} from "./new_stylesheet/babel_plugin_new_stylesheet_as_jsenv_import.js"
-import {
-  babelPluginGlobalThisAsJsenvImport,
-  globalThisClientFileUrl,
-} from "./global_this/babel_plugin_global_this_as_jsenv_import.js"
-import {
-  babelPluginRegeneratorRuntimeAsJsenvImport,
-  regeneratorRuntimeClientFileUrl,
-} from "./regenerator_runtime/babel_plugin_regenerator_runtime_as_jsenv_import.js"
+import { babelPluginNewStylesheetAsJsenvImport } from "./new_stylesheet/babel_plugin_new_stylesheet_as_jsenv_import.js"
+import { babelPluginGlobalThisAsJsenvImport } from "./global_this/babel_plugin_global_this_as_jsenv_import.js"
+import { babelPluginRegeneratorRuntimeAsJsenvImport } from "./regenerator_runtime/babel_plugin_regenerator_runtime_as_jsenv_import.js"
 
 export const jsenvPluginBabel = ({
   getCustomBabelPlugins,
@@ -115,17 +106,6 @@ export const jsenvPluginBabel = ({
   return {
     name: "jsenv:babel",
     appliesDuring: "*",
-    transformUrlContent: (urlInfo) => {
-      if (urlInfo.url === regeneratorRuntimeClientFileUrl) {
-        urlInfo.data.isBabelClientFile = true
-      }
-      if (urlInfo.url === globalThisClientFileUrl) {
-        urlInfo.data.isBabelClientFile = true
-      }
-      if (urlInfo.url === newStylesheetClientFileUrl) {
-        urlInfo.data.isBabelClientFile = true
-      }
-    },
     finalizeUrlContent: {
       js_classic: transformWithBabel,
       js_module: transformWithBabel,

package/src/test/execute_plan.js

@@ -326,6 +326,7 @@ export const executePlan = async (
 
         const afterExecutionInfo = {
           ...beforeExecutionInfo,
+          runtimeVersion: runtime.version,
           endMs: Date.now(),
           executionResult,
         }

package/src/test/readme.md

@@ -0,0 +1,3 @@
+# test/
+
+Code implementing "executeTestPlan" can be found here