@jsenv/core 41.4.2 → 41.4.3

package/dist/start_dev_server/start_dev_server.js

@@ -11239,9 +11239,20 @@ const devServerPluginServeSourceFiles = ({
               // dependency: the file re-requested did not change, one below
               // it did), so isValid() alone cannot catch this.
               !request.searchParams.has("hot") &&
+              // ...and the mirror guard: content cooked UNDER a "?hot" request
+              // carries "?hot" in its rewritten references, and is correct for
+              // that request only. Served from memory to a normal request (a
+              // fresh tab), the browser would then load "file.js" from one
+              // importer and "file.js?hot=..." from another — the same module
+              // evaluated twice, which breaks anything module-level (e.g. a
+              // signal registry throwing on duplicate ids). Re-cooking under
+              // the normal request rewrites the references clean.
+              !urlInfo.contentCookedForHotRequest &&
               urlInfo.isValid();
             if (!servableFromMemory) {
               await urlInfo.cook({ request, reference });
+              urlInfo.contentCookedForHotRequest =
+                request.searchParams.has("hot");
             }
             let { response } = urlInfo;
             if (response) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.4.2",
+  "version": "41.4.3",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {
@@ -78,7 +78,7 @@
     "@jsenv/plugin-minification": "1.7.5",
     "@jsenv/plugin-supervisor": "1.8.8",
     "@jsenv/plugin-transpilation": "1.5.78",
-    "@jsenv/server": "17.5.0",
+    "@jsenv/server": "17.6.0",
     "@jsenv/sourcemap": "1.4.2",
     "react-table": "7.8.0"
   },

package/src/dev/dev_server_plugins/dev_server_plugin_serve_source_files.js

@@ -318,9 +318,20 @@ export const devServerPluginServeSourceFiles = ({
               // dependency: the file re-requested did not change, one below
               // it did), so isValid() alone cannot catch this.
               !request.searchParams.has("hot") &&
+              // ...and the mirror guard: content cooked UNDER a "?hot" request
+              // carries "?hot" in its rewritten references, and is correct for
+              // that request only. Served from memory to a normal request (a
+              // fresh tab), the browser would then load "file.js" from one
+              // importer and "file.js?hot=..." from another — the same module
+              // evaluated twice, which breaks anything module-level (e.g. a
+              // signal registry throwing on duplicate ids). Re-cooking under
+              // the normal request rewrites the references clean.
+              !urlInfo.contentCookedForHotRequest &&
               urlInfo.isValid();
             if (!servableFromMemory) {
               await urlInfo.cook({ request, reference });
+              urlInfo.contentCookedForHotRequest =
+                request.searchParams.has("hot");
             }
             let { response } = urlInfo;
             if (response) {