@jsenv/core 38.2.8 → 38.2.11

package/README.md

@@ -40,6 +40,7 @@ It will prompt to choose one of the available demo:
 ```console
 ? Select a demo: › - Use arrow-keys. Return to submit.
 ❯   web
+    web-components
     web-react
     web-preact
     node-package

package/dist/jsenv_core.js

@@ -7355,7 +7355,11 @@ const serveDirectory = (
   const directoryContentArray = readdirSync(new URL(url));
   const responseProducers = {
     "application/json": () => {
-      const directoryContentJson = JSON.stringify(directoryContentArray);
+      const directoryContentJson = JSON.stringify(
+        directoryContentArray,
+        null,
+        "  ",
+      );
       return {
         status: 200,
         headers: {
@@ -9343,6 +9347,7 @@ const babelPluginCompatMap = {
     samsung: "9",
     electron: "3",
   },
+  "proposal-decorators": {},
   "transform-parameters": {
     chrome: "49",
     opera: "36",
@@ -9739,6 +9744,14 @@ const getBaseBabelPluginStructure = ({
     babelPluginStructure["proposal-unicode-property-regex"] =
       requireBabelPlugin("@babel/plugin-proposal-unicode-property-regex");
   }
+  // if (isBabelPluginNeeded("proposal-decorators") && content.includes("@")) {
+  //   babelPluginStructure["proposal-decorators"] = [
+  //     requireBabelPlugin("@babel/plugin-proposal-decorators"),
+  //     {
+  //       version: "2023-05",
+  //     },
+  //   ];
+  // }
   if (isBabelPluginNeeded("transform-async-to-promises")) {
     babelPluginStructure["transform-async-to-promises"] = [
       requireBabelPlugin("babel-plugin-transform-async-to-promises"),
@@ -12311,6 +12324,7 @@ const createUrlInfo = (url, context) => {
     context,
     error: null,
     modifiedTimestamp: 0,
+    descendantModifiedTimestamp: 0,
     dereferencedTimestamp: 0,
     originalContentEtag: null,
     contentEtag: null,
@@ -14378,7 +14392,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           // the HTML in itself it still valid
           // keep the syntax error and continue with the HTML
           const errorInfo =
-            e.code === "PARSE_ERROR"
+            e.code === "PARSE_ERROR" && e.cause
               ? `${e.cause.reasonCode}\n${e.traceMessage}`
               : e.stack;
           logger.error(
@@ -14885,6 +14899,9 @@ const jsenvPluginDirectoryReferenceAnalysis = () => {
           urlInfo.url,
           urlInfo.context.rootDirectoryUrl,
         );
+        if (urlInfo.contentType !== "application/json") {
+          return null;
+        }
         const entryNames = JSON.parse(urlInfo.content);
         const newEntryNames = [];
         for (const entryName of entryNames) {
@@ -18050,6 +18067,7 @@ const jsenvPluginProtocolFile = ({
         reference.leadsToADirectory = stat && stat.isDirectory();
         if (reference.leadsToADirectory) {
           const directoryAllowed =
+            reference.type === "http_request" ||
             reference.type === "filesystem" ||
             (typeof directoryReferenceAllowed === "function" &&
               directoryReferenceAllowed(reference)) ||
@@ -18115,7 +18133,6 @@ const jsenvPluginProtocolFile = ({
         }
         const urlObject = new URL(urlInfo.url);
         if (urlInfo.firstReference.leadsToADirectory) {
-          const directoryEntries = readdirSync(urlObject);
           if (!urlInfo.filenameHint) {
             if (urlInfo.firstReference.type === "filesystem") {
               urlInfo.filenameHint = `${
@@ -18125,10 +18142,17 @@ const jsenvPluginProtocolFile = ({
               urlInfo.filenameHint = `${urlToFilename$1(urlInfo.url)}/`;
             }
           }
+          const { headers, body } = serveDirectory(urlObject.href, {
+            headers: urlInfo.context.request
+              ? urlInfo.context.request.headers
+              : {},
+            rootDirectoryUrl: urlInfo.context.rootDirectoryUrl,
+          });
           return {
             type: "directory",
-            contentType: "application/json",
-            content: JSON.stringify(directoryEntries, null, "  "),
+            contentType: headers["content-type"],
+            contentLength: headers["content-length"],
+            content: body,
           };
         }
         if (
@@ -19254,8 +19278,16 @@ const jsenvPluginHotSearchParam = () => {
       // At this stage the parent is using ?hot and we are going to decide if
       // we propagate the search param to child.
       const referencedUrlInfo = reference.urlInfo;
-      const { modifiedTimestamp, dereferencedTimestamp } = referencedUrlInfo;
-      if (!modifiedTimestamp && !dereferencedTimestamp) {
+      const {
+        modifiedTimestamp,
+        descendantModifiedTimestamp,
+        dereferencedTimestamp,
+      } = referencedUrlInfo;
+      if (
+        !modifiedTimestamp &&
+        !descendantModifiedTimestamp &&
+        !dereferencedTimestamp
+      ) {
         return null;
       }
       // The goal is to send an url that will bypass client (the browser) cache
@@ -19268,10 +19300,11 @@ const jsenvPluginHotSearchParam = () => {
       // We use the latest timestamp to ensure it's fresh
       // The dereferencedTimestamp is needed because when a js module is re-referenced
       // browser must re-execute it, even if the code is not modified
-      const latestTimestamp =
-        dereferencedTimestamp && modifiedTimestamp
-          ? Math.max(dereferencedTimestamp, modifiedTimestamp)
-          : dereferencedTimestamp || modifiedTimestamp;
+      const latestTimestamp = Math.max(
+        modifiedTimestamp,
+        descendantModifiedTimestamp,
+        dereferencedTimestamp,
+      );
       return {
         hot: latestTimestamp,
       };
@@ -19333,89 +19366,149 @@ const jsenvPluginAutoreloadServer = ({
           }
           return url;
         };
-        const propagateUpdate = (firstUrlInfo) => {
-          const iterate = (urlInfo, seen) => {
-            if (urlInfo.data.hotAcceptSelf) {
-              return {
-                accepted: true,
-                reason:
-                  urlInfo === firstUrlInfo
-                    ? `file accepts hot reload`
-                    : `a dependent file accepts hot reload`,
-                instructions: [
-                  {
-                    type: urlInfo.type,
-                    boundary: formatUrlForClient(urlInfo.url),
+        const update = (firstUrlInfo) => {
+          const boundaries = new Set();
+          const instructions = [];
+          const propagateUpdate = (firstUrlInfo) => {
+            const iterate = (urlInfo, chain) => {
+              if (urlInfo.data.hotAcceptSelf) {
+                boundaries.add(urlInfo);
+                instructions.push({
+                  type: urlInfo.type,
+                  boundary: formatUrlForClient(urlInfo.url),
+                  acceptedBy: formatUrlForClient(urlInfo.url),
+                });
+                return {
+                  accepted: true,
+                  reason:
+                    urlInfo === firstUrlInfo
+                      ? `file accepts hot reload`
+                      : `a dependent file accepts hot reload`,
+                };
+              }
+              if (urlInfo.data.hotDecline) {
+                return {
+                  declined: true,
+                  reason: `file declines hot reload`,
+                  declinedBy: formatUrlForClient(urlInfo.url),
+                };
+              }
+              let instructionCountBefore = instructions.length;
+              for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
+                if (
+                  referenceFromOther.isImplicit &&
+                  referenceFromOther.isWeak
+                ) {
+                  if (!referenceFromOther.original) {
+                    continue;
+                  }
+                  if (referenceFromOther.original.isWeak) {
+                    continue;
+                  }
+                }
+                const urlInfoReferencingThisOne =
+                  referenceFromOther.ownerUrlInfo;
+                if (urlInfoReferencingThisOne.data.hotDecline) {
+                  return {
+                    declined: true,
+                    reason: `a dependent file declines hot reload`,
+                    declinedBy: formatUrlForClient(
+                      urlInfoReferencingThisOne.url,
+                    ),
+                  };
+                }
+                const { hotAcceptDependencies = [] } =
+                  urlInfoReferencingThisOne.data;
+                if (hotAcceptDependencies.includes(urlInfo.url)) {
+                  boundaries.add(urlInfoReferencingThisOne);
+                  instructions.push({
+                    type: urlInfoReferencingThisOne.type,
+                    boundary: formatUrlForClient(urlInfoReferencingThisOne.url),
                     acceptedBy: formatUrlForClient(urlInfo.url),
-                  },
-                ],
-              };
-            }
-            const instructions = [];
-            for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
-              if (referenceFromOther.isImplicit && referenceFromOther.isWeak) {
-                if (!referenceFromOther.original) {
+                  });
                   continue;
                 }
-                if (referenceFromOther.original.isWeak) {
+                if (chain.includes(urlInfoReferencingThisOne.url)) {
+                  return {
+                    declined: true,
+                    reason: "dead end",
+                    declinedBy: formatUrlForClient(
+                      urlInfoReferencingThisOne.url,
+                    ),
+                  };
+                }
+                const dependentPropagationResult = iterateMemoized(
+                  urlInfoReferencingThisOne,
+                  [...chain, urlInfoReferencingThisOne.url],
+                );
+                if (dependentPropagationResult.accepted) {
                   continue;
                 }
+                if (
+                  // declined explicitely by an other file, it must decline the whole update
+                  dependentPropagationResult.declinedBy
+                ) {
+                  return dependentPropagationResult;
+                }
+                // declined by absence of boundary, we can keep searching
               }
-              const urlInfoReferencingThisOne = referenceFromOther.ownerUrlInfo;
-              if (urlInfoReferencingThisOne.data.hotDecline) {
+              if (instructionCountBefore === instructions.length) {
                 return {
                   declined: true,
-                  reason: `a dependent file declines hot reload`,
-                  declinedBy: urlInfoReferencingThisOne.url,
+                  reason: `there is no file accepting hot reload while propagating update`,
                 };
               }
+              return {
+                accepted: true,
+                reason: `${instructions.length} dependent file(s) accepts hot reload`,
+              };
+            };
+
+            const map = new Map();
+            const iterateMemoized = (urlInfo, chain) => {
+              const resultFromCache = map.get(urlInfo.url);
+              if (resultFromCache) {
+                return resultFromCache;
+              }
+              const result = iterate(urlInfo, chain);
+              map.set(urlInfo.url, result);
+              return result;
+            };
+            map.clear();
+            return iterateMemoized(firstUrlInfo, []);
+          };
+
+          const propagationResult = propagateUpdate(firstUrlInfo);
+          const seen = new Set();
+          const invalidateImporters = (urlInfo) => {
+            // to indicate this urlInfo should be modified
+            for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
+              const urlInfoReferencingThisOne = referenceFromOther.ownerUrlInfo;
               const { hotAcceptDependencies = [] } =
                 urlInfoReferencingThisOne.data;
               if (hotAcceptDependencies.includes(urlInfo.url)) {
-                instructions.push({
-                  type: urlInfoReferencingThisOne.type,
-                  boundary: formatUrlForClient(urlInfoReferencingThisOne.url),
-                  acceptedBy: formatUrlForClient(urlInfo.url),
-                });
                 continue;
               }
-              if (seen.includes(urlInfoReferencingThisOne.url)) {
-                return {
-                  declined: true,
-                  reason: "circular dependency",
-                  declinedBy: formatUrlForClient(urlInfoReferencingThisOne.url),
-                };
-              }
-              const dependentPropagationResult = iterate(
-                urlInfoReferencingThisOne,
-                [...seen, urlInfoReferencingThisOne.url],
-              );
-              if (dependentPropagationResult.accepted) {
-                instructions.push(...dependentPropagationResult.instructions);
+              if (seen.has(urlInfoReferencingThisOne)) {
                 continue;
               }
-              if (
-                // declined explicitely by an other file, it must decline the whole update
-                dependentPropagationResult.declinedBy
-              ) {
-                return dependentPropagationResult;
+              seen.add(urlInfoReferencingThisOne);
+              // see https://github.com/vitejs/vite/blob/ab5bb40942c7023046fa6f6d0b49cabc105b6073/packages/vite/src/node/server/moduleGraph.ts#L205C5-L207C6
+              if (boundaries.has(urlInfoReferencingThisOne)) {
+                return;
               }
-              // declined by absence of boundary, we can keep searching
-            }
-            if (instructions.length === 0) {
-              return {
-                declined: true,
-                reason: `there is no file accepting hot reload while propagating update`,
-              };
+              urlInfoReferencingThisOne.descendantModifiedTimestamp =
+                Date.now();
+              invalidateImporters(urlInfoReferencingThisOne);
             }
-            return {
-              accepted: true,
-              reason: `${instructions.length} dependent file(s) accepts hot reload`,
-              instructions,
-            };
           };
-          const seen = [];
-          return iterate(firstUrlInfo, seen);
+          invalidateImporters(firstUrlInfo);
+          boundaries.clear();
+          seen.clear();
+          return {
+            ...propagationResult,
+            instructions,
+          };
         };
 
         // We are delaying the moment we tell client how to reload because:
@@ -19449,7 +19542,7 @@ const jsenvPluginAutoreloadServer = ({
               if (!changedUrlInfo.isUsed()) {
                 continue;
               }
-              const hotUpdate = propagateUpdate(changedUrlInfo);
+              const hotUpdate = update(changedUrlInfo);
               const relativeUrl = formatUrlForClient(changedUrlInfo.url);
               if (hotUpdate.declined) {
                 reloadMessage = {
@@ -19486,7 +19579,7 @@ const jsenvPluginAutoreloadServer = ({
               if (!ownerUrlInfo.isUsed()) {
                 continue;
               }
-              const ownerHotUpdate = propagateUpdate(ownerUrlInfo);
+              const ownerHotUpdate = update(ownerUrlInfo);
               const cause = `${formatUrlForClient(
                 prunedUrlInfo.url,
               )} is no longer referenced`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "38.2.8",
+  "version": "38.2.11",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -61,20 +61,20 @@
   "dependencies": {
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.2.4",
-    "@jsenv/ast": "5.1.4",
+    "@jsenv/ast": "5.2.0",
     "@jsenv/filesystem": "4.3.2",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
+    "@jsenv/js-module-fallback": "1.3.6",
     "@jsenv/log": "3.4.1",
     "@jsenv/node-esm-resolution": "1.0.1",
-    "@jsenv/js-module-fallback": "1.3.5",
+    "@jsenv/plugin-bundling": "2.5.7",
+    "@jsenv/plugin-minification": "1.5.3",
+    "@jsenv/plugin-supervisor": "1.3.6",
+    "@jsenv/plugin-transpilation": "1.3.5",
     "@jsenv/runtime-compat": "1.2.0",
-    "@jsenv/server": "15.1.3",
+    "@jsenv/server": "15.1.4",
     "@jsenv/sourcemap": "1.2.3",
-    "@jsenv/plugin-bundling": "2.5.6",
-    "@jsenv/plugin-minification": "1.5.3",
-    "@jsenv/plugin-transpilation": "1.3.4",
-    "@jsenv/plugin-supervisor": "1.3.5",
     "@jsenv/url-meta": "8.1.0",
     "@jsenv/urls": "2.2.1",
     "@jsenv/utils": "2.0.1"
@@ -82,22 +82,22 @@
   "devDependencies": {
     "@babel/eslint-parser": "7.22.15",
     "@babel/plugin-syntax-import-assertions": "7.22.5",
-    "babel-plugin-transform-async-to-promises": "0.8.18",
     "@jsenv/assert": "./packages/independent/assert/",
     "@jsenv/core": "./",
     "@jsenv/eslint-config": "./packages/independent/eslint-config/",
-    "@jsenv/file-size-impact": "14.1.2",
+    "@jsenv/file-size-impact": "14.1.3",
     "@jsenv/https-local": "3.0.7",
     "@jsenv/package-workspace": "0.5.3",
-    "@jsenv/performance-impact": "4.1.2",
+    "@jsenv/performance-impact": "4.1.3",
     "@jsenv/plugin-as-js-classic": "./packages/related/plugin-as-js-classic/",
     "@jsenv/test": "./packages/related/test/",
-    "eslint": "8.51.0",
+    "babel-plugin-transform-async-to-promises": "0.8.18",
+    "eslint": "8.52.0",
     "eslint-plugin-html": "7.1.0",
-    "eslint-plugin-import": "2.28.1",
+    "eslint-plugin-import": "2.29.0",
     "eslint-plugin-react": "7.33.2",
     "open": "9.1.0",
-    "playwright": "1.38.1",
+    "playwright": "1.39.0",
     "prettier": "3.0.3"
   }
 }

package/src/kitchen/kitchen.js

@@ -488,7 +488,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           // the HTML in itself it still valid
           // keep the syntax error and continue with the HTML
           const errorInfo =
-            e.code === "PARSE_ERROR"
+            e.code === "PARSE_ERROR" && e.cause
               ? `${e.cause.reasonCode}\n${e.traceMessage}`
               : e.stack;
           logger.error(

package/src/kitchen/url_graph/url_graph.js

@@ -181,6 +181,7 @@ const createUrlInfo = (url, context) => {
     context,
     error: null,
     modifiedTimestamp: 0,
+    descendantModifiedTimestamp: 0,
     dereferencedTimestamp: 0,
     originalContentEtag: null,
     contentEtag: null,

package/src/plugins/autoreload/jsenv_plugin_autoreload_server.js

@@ -18,89 +18,149 @@ export const jsenvPluginAutoreloadServer = ({
           }
           return url;
         };
-        const propagateUpdate = (firstUrlInfo) => {
-          const iterate = (urlInfo, seen) => {
-            if (urlInfo.data.hotAcceptSelf) {
-              return {
-                accepted: true,
-                reason:
-                  urlInfo === firstUrlInfo
-                    ? `file accepts hot reload`
-                    : `a dependent file accepts hot reload`,
-                instructions: [
-                  {
-                    type: urlInfo.type,
-                    boundary: formatUrlForClient(urlInfo.url),
+        const update = (firstUrlInfo) => {
+          const boundaries = new Set();
+          const instructions = [];
+          const propagateUpdate = (firstUrlInfo) => {
+            const iterate = (urlInfo, chain) => {
+              if (urlInfo.data.hotAcceptSelf) {
+                boundaries.add(urlInfo);
+                instructions.push({
+                  type: urlInfo.type,
+                  boundary: formatUrlForClient(urlInfo.url),
+                  acceptedBy: formatUrlForClient(urlInfo.url),
+                });
+                return {
+                  accepted: true,
+                  reason:
+                    urlInfo === firstUrlInfo
+                      ? `file accepts hot reload`
+                      : `a dependent file accepts hot reload`,
+                };
+              }
+              if (urlInfo.data.hotDecline) {
+                return {
+                  declined: true,
+                  reason: `file declines hot reload`,
+                  declinedBy: formatUrlForClient(urlInfo.url),
+                };
+              }
+              let instructionCountBefore = instructions.length;
+              for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
+                if (
+                  referenceFromOther.isImplicit &&
+                  referenceFromOther.isWeak
+                ) {
+                  if (!referenceFromOther.original) {
+                    continue;
+                  }
+                  if (referenceFromOther.original.isWeak) {
+                    continue;
+                  }
+                }
+                const urlInfoReferencingThisOne =
+                  referenceFromOther.ownerUrlInfo;
+                if (urlInfoReferencingThisOne.data.hotDecline) {
+                  return {
+                    declined: true,
+                    reason: `a dependent file declines hot reload`,
+                    declinedBy: formatUrlForClient(
+                      urlInfoReferencingThisOne.url,
+                    ),
+                  };
+                }
+                const { hotAcceptDependencies = [] } =
+                  urlInfoReferencingThisOne.data;
+                if (hotAcceptDependencies.includes(urlInfo.url)) {
+                  boundaries.add(urlInfoReferencingThisOne);
+                  instructions.push({
+                    type: urlInfoReferencingThisOne.type,
+                    boundary: formatUrlForClient(urlInfoReferencingThisOne.url),
                     acceptedBy: formatUrlForClient(urlInfo.url),
-                  },
-                ],
-              };
-            }
-            const instructions = [];
-            for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
-              if (referenceFromOther.isImplicit && referenceFromOther.isWeak) {
-                if (!referenceFromOther.original) {
+                  });
                   continue;
                 }
-                if (referenceFromOther.original.isWeak) {
+                if (chain.includes(urlInfoReferencingThisOne.url)) {
+                  return {
+                    declined: true,
+                    reason: "dead end",
+                    declinedBy: formatUrlForClient(
+                      urlInfoReferencingThisOne.url,
+                    ),
+                  };
+                }
+                const dependentPropagationResult = iterateMemoized(
+                  urlInfoReferencingThisOne,
+                  [...chain, urlInfoReferencingThisOne.url],
+                );
+                if (dependentPropagationResult.accepted) {
                   continue;
                 }
+                if (
+                  // declined explicitely by an other file, it must decline the whole update
+                  dependentPropagationResult.declinedBy
+                ) {
+                  return dependentPropagationResult;
+                }
+                // declined by absence of boundary, we can keep searching
               }
-              const urlInfoReferencingThisOne = referenceFromOther.ownerUrlInfo;
-              if (urlInfoReferencingThisOne.data.hotDecline) {
+              if (instructionCountBefore === instructions.length) {
                 return {
                   declined: true,
-                  reason: `a dependent file declines hot reload`,
-                  declinedBy: urlInfoReferencingThisOne.url,
+                  reason: `there is no file accepting hot reload while propagating update`,
                 };
               }
+              return {
+                accepted: true,
+                reason: `${instructions.length} dependent file(s) accepts hot reload`,
+              };
+            };
+
+            const map = new Map();
+            const iterateMemoized = (urlInfo, chain) => {
+              const resultFromCache = map.get(urlInfo.url);
+              if (resultFromCache) {
+                return resultFromCache;
+              }
+              const result = iterate(urlInfo, chain);
+              map.set(urlInfo.url, result);
+              return result;
+            };
+            map.clear();
+            return iterateMemoized(firstUrlInfo, []);
+          };
+
+          const propagationResult = propagateUpdate(firstUrlInfo);
+          const seen = new Set();
+          const invalidateImporters = (urlInfo) => {
+            // to indicate this urlInfo should be modified
+            for (const referenceFromOther of urlInfo.referenceFromOthersSet) {
+              const urlInfoReferencingThisOne = referenceFromOther.ownerUrlInfo;
               const { hotAcceptDependencies = [] } =
                 urlInfoReferencingThisOne.data;
               if (hotAcceptDependencies.includes(urlInfo.url)) {
-                instructions.push({
-                  type: urlInfoReferencingThisOne.type,
-                  boundary: formatUrlForClient(urlInfoReferencingThisOne.url),
-                  acceptedBy: formatUrlForClient(urlInfo.url),
-                });
                 continue;
               }
-              if (seen.includes(urlInfoReferencingThisOne.url)) {
-                return {
-                  declined: true,
-                  reason: "circular dependency",
-                  declinedBy: formatUrlForClient(urlInfoReferencingThisOne.url),
-                };
-              }
-              const dependentPropagationResult = iterate(
-                urlInfoReferencingThisOne,
-                [...seen, urlInfoReferencingThisOne.url],
-              );
-              if (dependentPropagationResult.accepted) {
-                instructions.push(...dependentPropagationResult.instructions);
+              if (seen.has(urlInfoReferencingThisOne)) {
                 continue;
               }
-              if (
-                // declined explicitely by an other file, it must decline the whole update
-                dependentPropagationResult.declinedBy
-              ) {
-                return dependentPropagationResult;
+              seen.add(urlInfoReferencingThisOne);
+              // see https://github.com/vitejs/vite/blob/ab5bb40942c7023046fa6f6d0b49cabc105b6073/packages/vite/src/node/server/moduleGraph.ts#L205C5-L207C6
+              if (boundaries.has(urlInfoReferencingThisOne)) {
+                return;
               }
-              // declined by absence of boundary, we can keep searching
+              urlInfoReferencingThisOne.descendantModifiedTimestamp =
+                Date.now();
+              invalidateImporters(urlInfoReferencingThisOne);
             }
-            if (instructions.length === 0) {
-              return {
-                declined: true,
-                reason: `there is no file accepting hot reload while propagating update`,
-              };
-            }
-            return {
-              accepted: true,
-              reason: `${instructions.length} dependent file(s) accepts hot reload`,
-              instructions,
-            };
           };
-          const seen = [];
-          return iterate(firstUrlInfo, seen);
+          invalidateImporters(firstUrlInfo);
+          boundaries.clear();
+          seen.clear();
+          return {
+            ...propagationResult,
+            instructions,
+          };
         };
 
         // We are delaying the moment we tell client how to reload because:
@@ -134,7 +194,7 @@ export const jsenvPluginAutoreloadServer = ({
               if (!changedUrlInfo.isUsed()) {
                 continue;
               }
-              const hotUpdate = propagateUpdate(changedUrlInfo);
+              const hotUpdate = update(changedUrlInfo);
               const relativeUrl = formatUrlForClient(changedUrlInfo.url);
               if (hotUpdate.declined) {
                 reloadMessage = {
@@ -171,7 +231,7 @@ export const jsenvPluginAutoreloadServer = ({
               if (!ownerUrlInfo.isUsed()) {
                 continue;
               }
-              const ownerHotUpdate = propagateUpdate(ownerUrlInfo);
+              const ownerHotUpdate = update(ownerUrlInfo);
               const cause = `${formatUrlForClient(
                 prunedUrlInfo.url,
               )} is no longer referenced`;

package/src/plugins/autoreload/jsenv_plugin_hot_search_param.js

@@ -44,8 +44,16 @@ export const jsenvPluginHotSearchParam = () => {
       // At this stage the parent is using ?hot and we are going to decide if
       // we propagate the search param to child.
       const referencedUrlInfo = reference.urlInfo;
-      const { modifiedTimestamp, dereferencedTimestamp } = referencedUrlInfo;
-      if (!modifiedTimestamp && !dereferencedTimestamp) {
+      const {
+        modifiedTimestamp,
+        descendantModifiedTimestamp,
+        dereferencedTimestamp,
+      } = referencedUrlInfo;
+      if (
+        !modifiedTimestamp &&
+        !descendantModifiedTimestamp &&
+        !dereferencedTimestamp
+      ) {
         return null;
       }
       // The goal is to send an url that will bypass client (the browser) cache
@@ -58,10 +66,11 @@ export const jsenvPluginHotSearchParam = () => {
       // We use the latest timestamp to ensure it's fresh
       // The dereferencedTimestamp is needed because when a js module is re-referenced
       // browser must re-execute it, even if the code is not modified
-      const latestTimestamp =
-        dereferencedTimestamp && modifiedTimestamp
-          ? Math.max(dereferencedTimestamp, modifiedTimestamp)
-          : dereferencedTimestamp || modifiedTimestamp;
+      const latestTimestamp = Math.max(
+        modifiedTimestamp,
+        descendantModifiedTimestamp,
+        dereferencedTimestamp,
+      );
       return {
         hot: latestTimestamp,
       };

package/src/plugins/protocol_file/jsenv_plugin_protocol_file.js

@@ -1,4 +1,5 @@
-import { readFileSync, readdirSync, realpathSync, statSync } from "node:fs";
+import { readFileSync, realpathSync, statSync } from "node:fs";
+import { serveDirectory } from "@jsenv/server";
 import { pathToFileURL } from "node:url";
 import {
   urlIsInsideOf,
@@ -98,6 +99,7 @@ export const jsenvPluginProtocolFile = ({
         reference.leadsToADirectory = stat && stat.isDirectory();
         if (reference.leadsToADirectory) {
           const directoryAllowed =
+            reference.type === "http_request" ||
             reference.type === "filesystem" ||
             (typeof directoryReferenceAllowed === "function" &&
               directoryReferenceAllowed(reference)) ||
@@ -163,7 +165,6 @@ export const jsenvPluginProtocolFile = ({
         }
         const urlObject = new URL(urlInfo.url);
         if (urlInfo.firstReference.leadsToADirectory) {
-          const directoryEntries = readdirSync(urlObject);
           if (!urlInfo.filenameHint) {
             if (urlInfo.firstReference.type === "filesystem") {
               urlInfo.filenameHint = `${
@@ -173,10 +174,17 @@ export const jsenvPluginProtocolFile = ({
               urlInfo.filenameHint = `${urlToFilename(urlInfo.url)}/`;
             }
           }
+          const { headers, body } = serveDirectory(urlObject.href, {
+            headers: urlInfo.context.request
+              ? urlInfo.context.request.headers
+              : {},
+            rootDirectoryUrl: urlInfo.context.rootDirectoryUrl,
+          });
           return {
             type: "directory",
-            contentType: "application/json",
-            content: JSON.stringify(directoryEntries, null, "  "),
+            contentType: headers["content-type"],
+            contentLength: headers["content-length"],
+            content: body,
           };
         }
         if (

package/src/plugins/reference_analysis/directory/jsenv_plugin_directory_reference_analysis.js

@@ -11,6 +11,9 @@ export const jsenvPluginDirectoryReferenceAnalysis = () => {
           urlInfo.url,
           urlInfo.context.rootDirectoryUrl,
         );
+        if (urlInfo.contentType !== "application/json") {
+          return null;
+        }
         const entryNames = JSON.parse(urlInfo.content);
         const newEntryNames = [];
         for (const entryName of entryNames) {