@jsenv/core 41.5.14 → 41.5.15

package/dist/build/build.js

@@ -2131,6 +2131,10 @@ const createUrlInfo = (url, context) => {
     modifiedTimestamp: 0,
     descendantModifiedTimestamp: 0,
     dereferencedTimestamp: 0,
+    // when a client last fetched this url outside a hot request; in other
+    // words the last time this url entered a fresh page (see
+    // jsenv_plugin_hot_search_param)
+    servedWithoutHotTimestamp: 0,
     originalContentEtag: null,
     contentEtag: null,
     isValid: () => false,
@@ -9445,6 +9449,7 @@ const jsenvPluginHotSearchParam = () => {
         modifiedTimestamp,
         descendantModifiedTimestamp,
         dereferencedTimestamp,
+        servedWithoutHotTimestamp,
       } = referencedUrlInfo;
       if (
         !modifiedTimestamp &&
@@ -9468,6 +9473,18 @@ const jsenvPluginHotSearchParam = () => {
         descendantModifiedTimestamp,
         dereferencedTimestamp,
       );
+      // These timestamps say "this url changed at some point", not "the client
+      // is running an outdated version of it": they are never cleared, so a
+      // file modified once keeps them for the rest of the dev server's life.
+      // A client that fetched this url after that modification (a page load:
+      // see rememberServedWithoutHot in the dev server) already runs the
+      // latest content. Sending it "?hot" then makes the browser evaluate a
+      // second, identical copy of a module it already has — a second module
+      // scope for a file that never changed, which breaks everything a module
+      // holds once (registries, contexts, singletons).
+      if (latestTimestamp <= servedWithoutHotTimestamp) {
+        return null;
+      }
       return {
         hot: latestTimestamp,
       };

package/dist/start_dev_server/start_dev_server.js

@@ -7170,6 +7170,7 @@ const jsenvPluginHotSearchParam = () => {
         modifiedTimestamp,
         descendantModifiedTimestamp,
         dereferencedTimestamp,
+        servedWithoutHotTimestamp,
       } = referencedUrlInfo;
       if (
         !modifiedTimestamp &&
@@ -7193,6 +7194,18 @@ const jsenvPluginHotSearchParam = () => {
         descendantModifiedTimestamp,
         dereferencedTimestamp,
       );
+      // These timestamps say "this url changed at some point", not "the client
+      // is running an outdated version of it": they are never cleared, so a
+      // file modified once keeps them for the rest of the dev server's life.
+      // A client that fetched this url after that modification (a page load:
+      // see rememberServedWithoutHot in the dev server) already runs the
+      // latest content. Sending it "?hot" then makes the browser evaluate a
+      // second, identical copy of a module it already has — a second module
+      // scope for a file that never changed, which breaks everything a module
+      // holds once (registries, contexts, singletons).
+      if (latestTimestamp <= servedWithoutHotTimestamp) {
+        return null;
+      }
       return {
         hot: latestTimestamp,
       };
@@ -9675,6 +9688,10 @@ const createUrlInfo = (url, context) => {
     modifiedTimestamp: 0,
     descendantModifiedTimestamp: 0,
     dereferencedTimestamp: 0,
+    // when a client last fetched this url outside a hot request; in other
+    // words the last time this url entered a fresh page (see
+    // jsenv_plugin_hot_search_param)
+    servedWithoutHotTimestamp: 0,
     originalContentEtag: null,
     contentEtag: null,
     isValid: () => false,
@@ -11922,7 +11939,20 @@ const devServerPluginServeSourceFiles = ({
             );
             return response;
           };
+          // What the client holds for this url is what we last sent it: a
+          // request without "?hot" is a page loading this url into an empty
+          // module registry, so from here on the client runs this exact
+          // content. Remembering when that happened is what allows
+          // jsenv_plugin_hot_search_param to tell a modification the client
+          // has already received from one it must re-execute to see.
+          const rememberServedWithoutHot = () => {
+            if (request.searchParams.has("hot")) {
+              return;
+            }
+            urlInfo.servedWithoutHotTimestamp = Date.now();
+          };
           const respondWithNotModified = () => {
+            rememberServedWithoutHot();
             const headers = {
               "cache-control": `private,max-age=0,must-revalidate`,
             };
@@ -12007,6 +12037,9 @@ const devServerPluginServeSourceFiles = ({
             ) {
               return respondWithNotModified();
             }
+            if (urlInfo.status === 200) {
+              rememberServedWithoutHot();
+            }
             response = {
               url: reference.url,
               // a plugin can cook a complete response body for an url that is

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.5.14",
+  "version": "41.5.15",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {

package/src/dev/dev_server_plugins/dev_server_plugin_serve_source_files.js

@@ -387,7 +387,20 @@ export const devServerPluginServeSourceFiles = ({
             );
             return response;
           };
+          // What the client holds for this url is what we last sent it: a
+          // request without "?hot" is a page loading this url into an empty
+          // module registry, so from here on the client runs this exact
+          // content. Remembering when that happened is what allows
+          // jsenv_plugin_hot_search_param to tell a modification the client
+          // has already received from one it must re-execute to see.
+          const rememberServedWithoutHot = () => {
+            if (request.searchParams.has("hot")) {
+              return;
+            }
+            urlInfo.servedWithoutHotTimestamp = Date.now();
+          };
           const respondWithNotModified = () => {
+            rememberServedWithoutHot();
             const headers = {
               "cache-control": `private,max-age=0,must-revalidate`,
             };
@@ -472,6 +485,9 @@ export const devServerPluginServeSourceFiles = ({
             ) {
               return respondWithNotModified();
             }
+            if (urlInfo.status === 200) {
+              rememberServedWithoutHot();
+            }
             response = {
               url: reference.url,
               // a plugin can cook a complete response body for an url that is

package/src/kitchen/url_graph/url_graph.js

@@ -183,6 +183,10 @@ const createUrlInfo = (url, context) => {
     modifiedTimestamp: 0,
     descendantModifiedTimestamp: 0,
     dereferencedTimestamp: 0,
+    // when a client last fetched this url outside a hot request; in other
+    // words the last time this url entered a fresh page (see
+    // jsenv_plugin_hot_search_param)
+    servedWithoutHotTimestamp: 0,
     originalContentEtag: null,
     contentEtag: null,
     isValid: () => false,

package/src/plugins/autoreload/jsenv_plugin_hot_search_param.js

@@ -48,6 +48,7 @@ export const jsenvPluginHotSearchParam = () => {
         modifiedTimestamp,
         descendantModifiedTimestamp,
         dereferencedTimestamp,
+        servedWithoutHotTimestamp,
       } = referencedUrlInfo;
       if (
         !modifiedTimestamp &&
@@ -71,6 +72,18 @@ export const jsenvPluginHotSearchParam = () => {
         descendantModifiedTimestamp,
         dereferencedTimestamp,
       );
+      // These timestamps say "this url changed at some point", not "the client
+      // is running an outdated version of it": they are never cleared, so a
+      // file modified once keeps them for the rest of the dev server's life.
+      // A client that fetched this url after that modification (a page load:
+      // see rememberServedWithoutHot in the dev server) already runs the
+      // latest content. Sending it "?hot" then makes the browser evaluate a
+      // second, identical copy of a module it already has — a second module
+      // scope for a file that never changed, which breaks everything a module
+      // holds once (registries, contexts, singletons).
+      if (latestTimestamp <= servedWithoutHotTimestamp) {
+        return null;
+      }
       return {
         hot: latestTimestamp,
       };