@jsenv/core 41.5.23 → 41.5.25

package/dist/browserslist_index/browserslist_index.js

@@ -2873,6 +2873,14 @@ var require$$1 = [
     security: false,
     v8: "13.6.233.17"
   },
+  {
+    name: "nodejs",
+    version: "24.21.0",
+    date: "2026-09-07",
+    lts: "Krypton",
+    security: false,
+    v8: "13.6.233.17"
+  },
   {
     name: "nodejs",
     version: "25.0.0",
@@ -3410,8 +3418,11 @@ function requireVersions () {
 		"43.3": "150",
 		"43.4": "150",
 		"43.5": "150",
+		"43.6": "150",
 		"44.0": "152",
 		"44.1": "152",
+		"44.2": "152",
+		"44.3": "152",
 		"45.0": "155"
 	};
 	return versions;

package/dist/build/build.js

@@ -2460,7 +2460,7 @@ ${urlInfo.url}`,
 const injectionSymbol = Symbol.for("jsenv_injection");
 const INJECTIONS = {
   /**
-   * Inject `Object.assign(window, { [key]: value })` at the top of the file
+   * Inject `Object.assign(globalThis, { [key]: value })` at the top of the file
    * (into a script for html, into the module itself for js) instead of
    * replacing a placeholder: the value is read at runtime as a global.
    */
@@ -2611,7 +2611,7 @@ const injectGlobals = (content, globals, urlInfo) => {
     return globalInjectorOnHtml(content, globals, urlInfo);
   }
   if (urlInfo.type === "js_classic" || urlInfo.type === "js_module") {
-    return globalsInjectorOnJs(content, globals, urlInfo);
+    return globalsInjectorOnJs(content, globals);
   }
   throw new Error(
     createDetailedMessage(`cannot inject globals into "${urlInfo.type}"`, {
@@ -2631,9 +2631,7 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     url: urlInfo.url,
     storeOriginalPositions: false,
   });
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker: false,
-  });
+  const clientCode = generateClientCodeForGlobals(globals);
   injectJsenvScript(htmlAst, {
     content: clientCode,
     pluginName: "jsenv:inject_globals",
@@ -2642,24 +2640,17 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     content: stringifyHtmlAst(htmlAst),
   };
 };
-const globalsInjectorOnJs = (content, globals, urlInfo) => {
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker:
-      urlInfo.subtype === "worker" ||
-      urlInfo.subtype === "service_worker" ||
-      urlInfo.subtype === "shared_worker",
-  });
+const globalsInjectorOnJs = (content, globals) => {
+  const clientCode = generateClientCodeForGlobals(globals);
   const magicSource = createMagicSource(content);
   magicSource.prepend(clientCode);
   return magicSource.toContentAndSourcemap();
 };
-const generateClientCodeForGlobals = (globals, { isWebWorker = false }) => {
-  const globalName = isWebWorker ? "self" : "window";
-  return `Object.assign(${globalName}, ${JSON.stringify(
-    globals,
-    null,
-    "  ",
-  )});`;
+// "globalThis" is the global object in a window, a worker and a service worker alike;
+// naming one of "window"/"self" would require knowing the file's subtype, which is not
+// known yet when the browser fetches a service worker on its own (update check).
+const generateClientCodeForGlobals = (globals) => {
+  return `Object.assign(globalThis, ${JSON.stringify(globals, null, "  ")});`;
 };
 
 const defineGettersOnPropertiesDerivedFromOriginalContent = (
@@ -10858,28 +10849,33 @@ const logsDefault = {
 // we nevery minify those because they are already very small
 // and would hurt the readability of something that can be critical to debug
 const injectGlobalMappings = async (urlInfo, mappings) => {
-  if (urlInfo.type === "html") {
-    // const minification = Boolean(
-    //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
-    // );
-    const content = generateClientCodeForMappings(mappings, {
-      globalName: "window",
-      minification: false,
-    });
-    await prependContent(urlInfo, { type: "js_classic", content });
+  if (
+    urlInfo.type !== "html" &&
+    urlInfo.type !== "js_classic" &&
+    urlInfo.type !== "js_module"
+  ) {
     return;
   }
-  if (urlInfo.type === "js_classic" || urlInfo.type === "js_module") {
-    // const minification = Boolean(
-    //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
-    // );
-    const content = generateClientCodeForMappings(mappings, {
-      globalName: isWebWorkerUrlInfo(urlInfo) ? "self" : "window",
-      minification: false,
-    });
-    await prependContent(urlInfo, { type: "js_classic", content });
-    return;
+  // const minification = Boolean(
+  //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
+  // );
+  const content = generateClientCodeForMappings(mappings, {
+    globalName: getGlobalName(urlInfo),
+    minification: false,
+  });
+  await prependContent(urlInfo, { type: "js_classic", content });
+};
+
+// "globalThis" names the global object in a window and in a worker alike;
+// the window/self split is only for runtimes predating it.
+const getGlobalName = (urlInfo) => {
+  if (urlInfo.context.isSupportedOnCurrentClients("global_this")) {
+    return "globalThis";
+  }
+  if (isWebWorkerUrlInfo(urlInfo)) {
+    return "self";
   }
+  return "window";
 };
 
 const generateClientCodeForMappings = (
@@ -12801,7 +12797,7 @@ const jsenvPluginMappings = (mappings) => {
  *        `<script>window.backendUrl = __BACKEND_URL__;</script>` gets the JS literal,
  *        which is how a value is shared with every js file of the page.
  *        Use INJECTIONS.optional(value) for a placeholder that may be absent from the file
- *        and INJECTIONS.global(value) to inject `Object.assign(window, { ... })` instead of
+ *        and INJECTIONS.global(value) to inject `Object.assign(globalThis, { ... })` instead of
  *        replacing a placeholder.
  *
  * @return {Promise<Object>} buildReturnValue

package/dist/jsenv_core.js

@@ -4,7 +4,7 @@ import "@jsenv/sourcemap";
 const injectionSymbol = Symbol.for("jsenv_injection");
 const INJECTIONS = {
   /**
-   * Inject `Object.assign(window, { [key]: value })` at the top of the file
+   * Inject `Object.assign(globalThis, { [key]: value })` at the top of the file
    * (into a script for html, into the module itself for js) instead of
    * replacing a placeholder: the value is read at runtime as a global.
    */

package/dist/start_dev_server/start_dev_server.js

@@ -5407,7 +5407,7 @@ const jsenvPluginDirectoryReferenceEffect = (
 const injectionSymbol = Symbol.for("jsenv_injection");
 const INJECTIONS = {
   /**
-   * Inject `Object.assign(window, { [key]: value })` at the top of the file
+   * Inject `Object.assign(globalThis, { [key]: value })` at the top of the file
    * (into a script for html, into the module itself for js) instead of
    * replacing a placeholder: the value is read at runtime as a global.
    */
@@ -5558,7 +5558,7 @@ const injectGlobals = (content, globals, urlInfo) => {
     return globalInjectorOnHtml(content, globals, urlInfo);
   }
   if (urlInfo.type === "js_classic" || urlInfo.type === "js_module") {
-    return globalsInjectorOnJs(content, globals, urlInfo);
+    return globalsInjectorOnJs(content, globals);
   }
   throw new Error(
     createDetailedMessage(`cannot inject globals into "${urlInfo.type}"`, {
@@ -5578,9 +5578,7 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     url: urlInfo.url,
     storeOriginalPositions: false,
   });
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker: false,
-  });
+  const clientCode = generateClientCodeForGlobals(globals);
   injectJsenvScript(htmlAst, {
     content: clientCode,
     pluginName: "jsenv:inject_globals",
@@ -5589,24 +5587,17 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     content: stringifyHtmlAst(htmlAst),
   };
 };
-const globalsInjectorOnJs = (content, globals, urlInfo) => {
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker:
-      urlInfo.subtype === "worker" ||
-      urlInfo.subtype === "service_worker" ||
-      urlInfo.subtype === "shared_worker",
-  });
+const globalsInjectorOnJs = (content, globals) => {
+  const clientCode = generateClientCodeForGlobals(globals);
   const magicSource = createMagicSource(content);
   magicSource.prepend(clientCode);
   return magicSource.toContentAndSourcemap();
 };
-const generateClientCodeForGlobals = (globals, { isWebWorker = false }) => {
-  const globalName = isWebWorker ? "self" : "window";
-  return `Object.assign(${globalName}, ${JSON.stringify(
-    globals,
-    null,
-    "  ",
-  )});`;
+// "globalThis" is the global object in a window, a worker and a service worker alike;
+// naming one of "window"/"self" would require knowing the file's subtype, which is not
+// known yet when the browser fetches a service worker on its own (update check).
+const generateClientCodeForGlobals = (globals) => {
+  return `Object.assign(globalThis, ${JSON.stringify(globals, null, "  ")});`;
 };
 
 const jsenvPluginInjections = (rawAssociations) => {
@@ -12124,8 +12115,20 @@ const devServerPluginServeSourceFiles = ({
                 rootDirectoryUrl: sourceDirectoryUrl,
               })
             : sourceDirectoryUrl;
+          // What the graph knows this resource as: the specifier a reference
+          // decodes to never carries "?hot" (the client adds it to re-import,
+          // see jsenv_plugin_hot_search_param), so the request is compared
+          // without it — or nothing inline ever matches its own re-import,
+          // and a file that a re-cook could create is created twice.
+          const requestResourceWithoutHot = WEB_URL_CONVERTER.asWebUrl(
+            requestedUrl,
+            {
+              origin: request.origin,
+              rootDirectoryUrl: sourceDirectoryUrl,
+            },
+          ).slice(request.origin.length);
           let reference = kitchen.graph.inferReference(
-            request.resource,
+            requestResourceWithoutHot,
             parentUrl,
           );
           if (!reference) {
@@ -12168,7 +12171,7 @@ const devServerPluginServeSourceFiles = ({
                 });
               }
               reference = kitchen.graph.inferReference(
-                request.resource,
+                requestResourceWithoutHot,
                 inlineParentUrl,
               );
               if (!reference) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.5.23",
+  "version": "41.5.25",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {
@@ -73,13 +73,13 @@
     "test:snapshot_clear": "npx @jsenv/filesystem clear **/tests/**/side_effects/"
   },
   "dependencies": {
-    "@jsenv/ast": "6.10.5",
-    "@jsenv/js-module-fallback": "1.6.9",
+    "@jsenv/ast": "6.10.6",
+    "@jsenv/js-module-fallback": "1.6.10",
     "@jsenv/plugin-bundling": "2.10.28",
-    "@jsenv/plugin-minification": "1.7.22",
-    "@jsenv/plugin-supervisor": "1.8.24",
-    "@jsenv/plugin-transpilation": "1.7.9",
-    "@jsenv/server": "17.6.5",
+    "@jsenv/plugin-minification": "1.7.23",
+    "@jsenv/plugin-supervisor": "1.8.25",
+    "@jsenv/plugin-transpilation": "1.7.10",
+    "@jsenv/server": "17.6.6",
     "@jsenv/sourcemap": "1.4.9",
     "react-table": "7.8.0"
   },

package/src/build/build.js

@@ -177,7 +177,7 @@ import { jsenvPluginMappings } from "./jsenv_plugin_mappings.js";
  *        `<script>window.backendUrl = __BACKEND_URL__;</script>` gets the JS literal,
  *        which is how a value is shared with every js file of the page.
  *        Use INJECTIONS.optional(value) for a placeholder that may be absent from the file
- *        and INJECTIONS.global(value) to inject `Object.assign(window, { ... })` instead of
+ *        and INJECTIONS.global(value) to inject `Object.assign(globalThis, { ... })` instead of
  *        replacing a placeholder.
  *
  * @return {Promise<Object>} buildReturnValue

package/src/build/mappings_injection.js

@@ -18,28 +18,33 @@ import { prependContent } from "../kitchen/prepend_content.js";
 // we nevery minify those because they are already very small
 // and would hurt the readability of something that can be critical to debug
 export const injectGlobalMappings = async (urlInfo, mappings) => {
-  if (urlInfo.type === "html") {
-    // const minification = Boolean(
-    //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
-    // );
-    const content = generateClientCodeForMappings(mappings, {
-      globalName: "window",
-      minification: false,
-    });
-    await prependContent(urlInfo, { type: "js_classic", content });
+  if (
+    urlInfo.type !== "html" &&
+    urlInfo.type !== "js_classic" &&
+    urlInfo.type !== "js_module"
+  ) {
     return;
   }
-  if (urlInfo.type === "js_classic" || urlInfo.type === "js_module") {
-    // const minification = Boolean(
-    //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
-    // );
-    const content = generateClientCodeForMappings(mappings, {
-      globalName: isWebWorkerUrlInfo(urlInfo) ? "self" : "window",
-      minification: false,
-    });
-    await prependContent(urlInfo, { type: "js_classic", content });
-    return;
+  // const minification = Boolean(
+  //   urlInfo.context.getPluginMeta("willMinifyJsClassic"),
+  // );
+  const content = generateClientCodeForMappings(mappings, {
+    globalName: getGlobalName(urlInfo),
+    minification: false,
+  });
+  await prependContent(urlInfo, { type: "js_classic", content });
+};
+
+// "globalThis" names the global object in a window and in a worker alike;
+// the window/self split is only for runtimes predating it.
+const getGlobalName = (urlInfo) => {
+  if (urlInfo.context.isSupportedOnCurrentClients("global_this")) {
+    return "globalThis";
+  }
+  if (isWebWorkerUrlInfo(urlInfo)) {
+    return "self";
   }
+  return "window";
 };
 
 const generateClientCodeForMappings = (

package/src/dev/dev_server_plugins/dev_server_plugin_serve_source_files.js

@@ -274,8 +274,20 @@ export const devServerPluginServeSourceFiles = ({
                 rootDirectoryUrl: sourceDirectoryUrl,
               })
             : sourceDirectoryUrl;
+          // What the graph knows this resource as: the specifier a reference
+          // decodes to never carries "?hot" (the client adds it to re-import,
+          // see jsenv_plugin_hot_search_param), so the request is compared
+          // without it — or nothing inline ever matches its own re-import,
+          // and a file that a re-cook could create is created twice.
+          const requestResourceWithoutHot = WEB_URL_CONVERTER.asWebUrl(
+            requestedUrl,
+            {
+              origin: request.origin,
+              rootDirectoryUrl: sourceDirectoryUrl,
+            },
+          ).slice(request.origin.length);
           let reference = kitchen.graph.inferReference(
-            request.resource,
+            requestResourceWithoutHot,
             parentUrl,
           );
           if (!reference) {
@@ -318,7 +330,7 @@ export const devServerPluginServeSourceFiles = ({
                 });
               }
               reference = kitchen.graph.inferReference(
-                request.resource,
+                requestResourceWithoutHot,
                 inlineParentUrl,
               );
               if (!reference) {

package/src/kitchen/url_graph/url_info_injections.js

@@ -5,7 +5,7 @@ import { composeTwoSourcemaps, createMagicSource } from "@jsenv/sourcemap";
 const injectionSymbol = Symbol.for("jsenv_injection");
 export const INJECTIONS = {
   /**
-   * Inject `Object.assign(window, { [key]: value })` at the top of the file
+   * Inject `Object.assign(globalThis, { [key]: value })` at the top of the file
    * (into a script for html, into the module itself for js) instead of
    * replacing a placeholder: the value is read at runtime as a global.
    */
@@ -156,7 +156,7 @@ export const injectGlobals = (content, globals, urlInfo) => {
     return globalInjectorOnHtml(content, globals, urlInfo);
   }
   if (urlInfo.type === "js_classic" || urlInfo.type === "js_module") {
-    return globalsInjectorOnJs(content, globals, urlInfo);
+    return globalsInjectorOnJs(content, globals);
   }
   throw new Error(
     createDetailedMessage(`cannot inject globals into "${urlInfo.type}"`, {
@@ -176,9 +176,7 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     url: urlInfo.url,
     storeOriginalPositions: false,
   });
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker: false,
-  });
+  const clientCode = generateClientCodeForGlobals(globals);
   injectJsenvScript(htmlAst, {
     content: clientCode,
     pluginName: "jsenv:inject_globals",
@@ -187,22 +185,15 @@ const globalInjectorOnHtml = (content, globals, urlInfo) => {
     content: stringifyHtmlAst(htmlAst),
   };
 };
-const globalsInjectorOnJs = (content, globals, urlInfo) => {
-  const clientCode = generateClientCodeForGlobals(globals, {
-    isWebWorker:
-      urlInfo.subtype === "worker" ||
-      urlInfo.subtype === "service_worker" ||
-      urlInfo.subtype === "shared_worker",
-  });
+const globalsInjectorOnJs = (content, globals) => {
+  const clientCode = generateClientCodeForGlobals(globals);
   const magicSource = createMagicSource(content);
   magicSource.prepend(clientCode);
   return magicSource.toContentAndSourcemap();
 };
-const generateClientCodeForGlobals = (globals, { isWebWorker = false }) => {
-  const globalName = isWebWorker ? "self" : "window";
-  return `Object.assign(${globalName}, ${JSON.stringify(
-    globals,
-    null,
-    "  ",
-  )});`;
+// "globalThis" is the global object in a window, a worker and a service worker alike;
+// naming one of "window"/"self" would require knowing the file's subtype, which is not
+// known yet when the browser fetches a service worker on its own (update check).
+const generateClientCodeForGlobals = (globals) => {
+  return `Object.assign(globalThis, ${JSON.stringify(globals, null, "  ")});`;
 };