@jsenv/core 28.3.0 → 28.3.4

package/dist/main.js

@@ -11940,7 +11940,7 @@ const jsenvPluginImportmap = () => {
           // We must precook the importmap to know its content and inline it into the HTML
           // In this situation the ref to the importmap was already discovered
           // when parsing the HTML
-          const importmapReference = context.referenceUtils.findByGeneratedSpecifier(src);
+          const importmapReference = context.referenceUtils.find(ref => ref.generatedSpecifier === src);
           const importmapUrlInfo = context.urlGraph.getUrlInfo(importmapReference.url);
           await context.cook(importmapUrlInfo, {
             reference: importmapReference
@@ -16998,28 +16998,36 @@ const jsenvPluginAutoreloadServer = ({
           url,
           event
         }) => {
-          const urlInfo = urlGraph.getUrlInfo(url); // file not part of dependency graph
+          const onUrlInfo = urlInfo => {
+            const relativeUrl = formatUrlForClient(url);
+            const hotUpdate = propagateUpdate(urlInfo);
 
-          if (!urlInfo) {
-            return;
-          }
+            if (hotUpdate.declined) {
+              notifyDeclined({
+                cause: `${relativeUrl} ${event}`,
+                reason: hotUpdate.reason,
+                declinedBy: hotUpdate.declinedBy
+              });
+            } else {
+              notifyAccepted({
+                cause: `${relativeUrl} ${event}`,
+                reason: hotUpdate.reason,
+                instructions: hotUpdate.instructions
+              });
+            }
+          };
 
-          const relativeUrl = formatUrlForClient(url);
-          const hotUpdate = propagateUpdate(urlInfo);
+          urlGraph.urlInfoMap.forEach(urlInfo => {
+            if (urlInfo.url === url) {
+              onUrlInfo(urlInfo);
+            } else {
+              const urlWithoutSearch = asUrlWithoutSearch(urlInfo.url);
 
-          if (hotUpdate.declined) {
-            notifyDeclined({
-              cause: `${relativeUrl} ${event}`,
-              reason: hotUpdate.reason,
-              declinedBy: hotUpdate.declinedBy
-            });
-          } else {
-            notifyAccepted({
-              cause: `${relativeUrl} ${event}`,
-              reason: hotUpdate.reason,
-              instructions: hotUpdate.instructions
-            });
-          }
+              if (urlWithoutSearch === url) {
+                onUrlInfo(urlInfo);
+              }
+            }
+          });
         });
         clientFilesPruneCallbackList.push((prunedUrlInfos, firstUrlInfo) => {
           const mainHotUpdate = propagateUpdate(firstUrlInfo);
@@ -17800,7 +17808,7 @@ const createUrlInfoTransformer = ({
       // jsenv won't emit a warning and use the following strategy:
       // "no sourcemap is better than wrong sourcemap"
 
-      urlInfo.sourcemapIsWrong = sourcemapIsWrong;
+      urlInfo.sourcemapIsWrong = urlInfo.sourcemapIsWrong || sourcemapIsWrong;
     }
   };
 
@@ -17809,52 +17817,54 @@ const createUrlInfoTransformer = ({
       applyIntermediateTransformations(urlInfo, transformations);
     }
 
-    if (sourcemapsEnabled && urlInfo.sourcemap && !urlInfo.generatedUrl.startsWith("data:")) {
-      // during build this function can be called after the file is cooked
-      // - to update content and sourcemap after "optimize" hook
-      // - to inject versioning into the entry point content
-      // in this scenarion we don't want to call injectSourcemap
-      // just update the content and the
-      const sourcemapReference = urlInfo.sourcemapReference;
-      const sourcemapUrlInfo = urlGraph.getUrlInfo(sourcemapReference.url);
-      sourcemapUrlInfo.contentType = "application/json";
-      const sourcemap = urlInfo.sourcemap;
-
-      if (sourcemapsRelativeSources) {
-        sourcemap.sources = sourcemap.sources.map(source => {
-          const sourceRelative = urlToRelativeUrl(source, urlInfo.url);
-          return sourceRelative || ".";
-        });
-      }
+    if (urlInfo.sourcemapReference) {
+      if (sourcemapsEnabled && urlInfo.sourcemap && !urlInfo.generatedUrl.startsWith("data:")) {
+        // during build this function can be called after the file is cooked
+        // - to update content and sourcemap after "optimize" hook
+        // - to inject versioning into the entry point content
+        // in this scenarion we don't want to call injectSourcemap
+        // just update the content and the
+        const sourcemapReference = urlInfo.sourcemapReference;
+        const sourcemapUrlInfo = urlGraph.getUrlInfo(sourcemapReference.url);
+        sourcemapUrlInfo.contentType = "application/json";
+        const sourcemap = urlInfo.sourcemap;
+
+        if (sourcemapsRelativeSources) {
+          sourcemap.sources = sourcemap.sources.map(source => {
+            const sourceRelative = urlToRelativeUrl(source, urlInfo.url);
+            return sourceRelative || ".";
+          });
+        }
 
-      if (sourcemapsSourcesProtocol !== "file:///") {
-        sourcemap.sources = sourcemap.sources.map(source => {
-          if (source.startsWith("file:///")) {
-            return `${sourcemapsSourcesProtocol}${source.slice("file:///".length)}`;
-          }
+        if (sourcemapsSourcesProtocol !== "file:///") {
+          sourcemap.sources = sourcemap.sources.map(source => {
+            if (source.startsWith("file:///")) {
+              return `${sourcemapsSourcesProtocol}${source.slice("file:///".length)}`;
+            }
 
-          return source;
-        });
-      }
+            return source;
+          });
+        }
 
-      sourcemapUrlInfo.content = JSON.stringify(sourcemap, null, "  ");
+        sourcemapUrlInfo.content = JSON.stringify(sourcemap, null, "  ");
 
-      if (!urlInfo.sourcemapIsWrong) {
-        if (sourcemaps === "inline") {
-          sourcemapReference.generatedSpecifier = generateSourcemapDataUrl(sourcemap);
-        }
+        if (!urlInfo.sourcemapIsWrong) {
+          if (sourcemaps === "inline") {
+            sourcemapReference.generatedSpecifier = generateSourcemapDataUrl(sourcemap);
+          }
 
-        if (sourcemaps === "file" || sourcemaps === "inline") {
-          urlInfo.content = SOURCEMAP.writeComment({
-            contentType: urlInfo.contentType,
-            content: urlInfo.content,
-            specifier: sourcemaps === "file" && sourcemapsRelativeSources ? urlToRelativeUrl(sourcemapReference.url, urlInfo.url) : sourcemapReference.generatedSpecifier
-          });
+          if (sourcemaps === "file" || sourcemaps === "inline") {
+            urlInfo.content = SOURCEMAP.writeComment({
+              contentType: urlInfo.contentType,
+              content: urlInfo.content,
+              specifier: sourcemaps === "file" && sourcemapsRelativeSources ? urlToRelativeUrl(sourcemapReference.url, urlInfo.url) : sourcemapReference.generatedSpecifier
+            });
+          }
         }
+      } else {
+        // in the end we don't use the sourcemap placeholder
+        urlGraph.deleteUrlInfo(urlInfo.sourcemapReference.url);
       }
-    } else if (urlInfo.sourcemapReference) {
-      // in the end we don't use the sourcemap placeholder
-      urlGraph.deleteUrlInfo(urlInfo.sourcemapReference.url);
     }
 
     urlInfo.contentEtag = urlInfo.content === urlInfo.originalContent ? urlInfo.originalContentEtag : bufferToEtag$1(Buffer.from(urlInfo.content));
@@ -18632,6 +18642,8 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
       const references = [];
       context.referenceUtils = {
         _references: references,
+        find: predicate => references.find(predicate),
+        readGeneratedSpecifier,
         add: props => {
           const [reference, referencedUrlInfo] = resolveReference(createReference({
             parentUrl: urlInfo.url,
@@ -18640,8 +18652,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           references.push(reference);
           return [reference, referencedUrlInfo];
         },
-        find: predicate => references.find(predicate),
-        readGeneratedSpecifier,
         found: ({
           trace,
           ...rest
@@ -18703,6 +18713,29 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
 
           return [newReference, newUrlInfo];
         },
+        inject: ({
+          trace,
+          ...rest
+        }) => {
+          if (trace === undefined) {
+            const {
+              url,
+              line,
+              column
+            } = getCallerPosition();
+            trace = traceFromUrlSite({
+              url,
+              line,
+              column
+            });
+          }
+
+          return context.referenceUtils.add({
+            trace,
+            injected: true,
+            ...rest
+          });
+        },
         becomesInline: (reference, {
           isOriginalPosition,
           specifier,
@@ -18729,37 +18762,8 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
             content
           });
         },
-        inject: ({
-          trace,
-          ...rest
-        }) => {
-          if (trace === undefined) {
-            const {
-              url,
-              line,
-              column
-            } = getCallerPosition();
-            trace = traceFromUrlSite({
-              url,
-              line,
-              column
-            });
-          }
-
-          return context.referenceUtils.add({
-            trace,
-            injected: true,
-            ...rest
-          });
-        },
-        findByGeneratedSpecifier: generatedSpecifier => {
-          const reference = references.find(ref => ref.generatedSpecifier === generatedSpecifier);
-
-          if (!reference) {
-            throw new Error(`No reference found using the following generatedSpecifier: "${generatedSpecifier}"`);
-          }
-
-          return reference;
+        becomesExternal: () => {
+          throw new Error("not implemented yet");
         }
       }; // "fetchUrlContent" hook
 
@@ -25305,11 +25309,17 @@ const createFileService = ({
     clientFileChangeCallbackList.push(({
       url
     }) => {
-      const urlInfo = urlGraph.getUrlInfo(url);
+      urlGraph.urlInfoMap.forEach(urlInfo => {
+        if (urlInfo.url === url) {
+          urlGraph.considerModified(urlInfo);
+        } else {
+          const urlWithoutSearch = asUrlWithoutSearch(urlInfo.url);
 
-      if (urlInfo) {
-        urlGraph.considerModified(urlInfo);
-      }
+          if (urlWithoutSearch === url) {
+            urlGraph.considerModified(urlInfo);
+          }
+        }
+      });
     });
     const kitchen = createKitchen({
       signal,
@@ -25486,22 +25496,30 @@ const createFileService = ({
     const urlInfoTargetedByCache = urlGraph.getParentIfInline(urlInfo);
 
     if (ifNoneMatch) {
-      if (urlInfoTargetedByCache.contentEtag === ifNoneMatch && urlInfoTargetedByCache.isValid()) {
+      const [clientOriginalContentEtag, clientContentEtag] = ifNoneMatch.split("_");
+
+      if (urlInfoTargetedByCache.originalContentEtag === clientOriginalContentEtag && urlInfoTargetedByCache.contentEtag === clientContentEtag && urlInfoTargetedByCache.isValid()) {
+        const headers = {
+          "cache-control": `private,max-age=0,must-revalidate`
+        };
+        Object.keys(urlInfo.headers).forEach(key => {
+          if (key !== "content-length") {
+            headers[key] = urlInfo.headers[key];
+          }
+        });
         return {
           status: 304,
-          headers: {
-            "cache-control": `private,max-age=0,must-revalidate`,
-            ...urlInfo.headers
-          }
+          headers
         };
       }
     }
 
     try {
       // urlInfo objects are reused, they must be "reset" before cooking them again
-      if (urlInfo.contentEtag && !urlInfo.isInline && urlInfo.type !== "sourcemap") {
+      if ((urlInfo.error || urlInfo.contentEtag) && !urlInfo.isInline && urlInfo.type !== "sourcemap") {
         urlInfo.error = null;
         urlInfo.sourcemap = null;
+        urlInfo.sourcemapIsWrong = null;
         urlInfo.sourcemapReference = null;
         urlInfo.content = null;
         urlInfo.originalContent = null;
@@ -25526,11 +25544,12 @@ const createFileService = ({
         url: reference.url,
         status: 200,
         headers: {
-          "content-length": Buffer.byteLength(urlInfo.content),
           "cache-control": `private,max-age=0,must-revalidate`,
-          "eTag": urlInfoTargetedByCache.contentEtag,
+          // it's safe to use "_" separator because etag is encoded with base64 (see https://stackoverflow.com/a/13195197)
+          "eTag": `${urlInfoTargetedByCache.originalContentEtag}_${urlInfoTargetedByCache.contentEtag}`,
           ...urlInfo.headers,
-          "content-type": urlInfo.contentType
+          "content-type": urlInfo.contentType,
+          "content-length": Buffer.byteLength(urlInfo.content)
         },
         body: urlInfo.content,
         timing: urlInfo.timing

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "28.3.0",
+  "version": "28.3.4",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {

package/src/omega/kitchen.js

@@ -429,6 +429,8 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
       const references = []
       context.referenceUtils = {
         _references: references,
+        find: (predicate) => references.find(predicate),
+        readGeneratedSpecifier,
         add: (props) => {
           const [reference, referencedUrlInfo] = resolveReference(
             createReference({
@@ -440,8 +442,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           references.push(reference)
           return [reference, referencedUrlInfo]
         },
-        find: (predicate) => references.find(predicate),
-        readGeneratedSpecifier,
         found: ({ trace, ...rest }) => {
           if (trace === undefined) {
             trace = traceFromUrlSite(
@@ -511,6 +511,21 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
           }
           return [newReference, newUrlInfo]
         },
+        inject: ({ trace, ...rest }) => {
+          if (trace === undefined) {
+            const { url, line, column } = getCallerPosition()
+            trace = traceFromUrlSite({
+              url,
+              line,
+              column,
+            })
+          }
+          return context.referenceUtils.add({
+            trace,
+            injected: true,
+            ...rest,
+          })
+        },
         becomesInline: (
           reference,
           {
@@ -544,31 +559,8 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
             content,
           })
         },
-        inject: ({ trace, ...rest }) => {
-          if (trace === undefined) {
-            const { url, line, column } = getCallerPosition()
-            trace = traceFromUrlSite({
-              url,
-              line,
-              column,
-            })
-          }
-          return context.referenceUtils.add({
-            trace,
-            injected: true,
-            ...rest,
-          })
-        },
-        findByGeneratedSpecifier: (generatedSpecifier) => {
-          const reference = references.find(
-            (ref) => ref.generatedSpecifier === generatedSpecifier,
-          )
-          if (!reference) {
-            throw new Error(
-              `No reference found using the following generatedSpecifier: "${generatedSpecifier}"`,
-            )
-          }
-          return reference
+        becomesExternal: () => {
+          throw new Error("not implemented yet")
         },
       }
 

package/src/omega/server/file_service.js

@@ -5,7 +5,7 @@ import {
   composeTwoResponses,
 } from "@jsenv/server"
 import { registerDirectoryLifecycle, bufferToEtag } from "@jsenv/filesystem"
-import { urlIsInsideOf, moveUrl } from "@jsenv/urls"
+import { urlIsInsideOf, moveUrl, asUrlWithoutSearch } from "@jsenv/urls"
 import { URL_META } from "@jsenv/url-meta"
 
 import { getCorePlugins } from "@jsenv/core/src/plugins/plugins.js"
@@ -95,10 +95,16 @@ export const createFileService = ({
     )
     const urlGraph = createUrlGraph()
     clientFileChangeCallbackList.push(({ url }) => {
-      const urlInfo = urlGraph.getUrlInfo(url)
-      if (urlInfo) {
-        urlGraph.considerModified(urlInfo)
-      }
+      urlGraph.urlInfoMap.forEach((urlInfo) => {
+        if (urlInfo.url === url) {
+          urlGraph.considerModified(urlInfo)
+        } else {
+          const urlWithoutSearch = asUrlWithoutSearch(urlInfo.url)
+          if (urlWithoutSearch === url) {
+            urlGraph.considerModified(urlInfo)
+          }
+        }
+      })
     })
     const kitchen = createKitchen({
       signal,
@@ -264,28 +270,38 @@ export const createFileService = ({
     const urlInfoTargetedByCache = urlGraph.getParentIfInline(urlInfo)
 
     if (ifNoneMatch) {
+      const [clientOriginalContentEtag, clientContentEtag] =
+        ifNoneMatch.split("_")
       if (
-        urlInfoTargetedByCache.contentEtag === ifNoneMatch &&
+        urlInfoTargetedByCache.originalContentEtag ===
+          clientOriginalContentEtag &&
+        urlInfoTargetedByCache.contentEtag === clientContentEtag &&
         urlInfoTargetedByCache.isValid()
       ) {
+        const headers = {
+          "cache-control": `private,max-age=0,must-revalidate`,
+        }
+        Object.keys(urlInfo.headers).forEach((key) => {
+          if (key !== "content-length") {
+            headers[key] = urlInfo.headers[key]
+          }
+        })
         return {
           status: 304,
-          headers: {
-            "cache-control": `private,max-age=0,must-revalidate`,
-            ...urlInfo.headers,
-          },
+          headers,
         }
       }
     }
     try {
       // urlInfo objects are reused, they must be "reset" before cooking them again
       if (
-        urlInfo.contentEtag &&
+        (urlInfo.error || urlInfo.contentEtag) &&
         !urlInfo.isInline &&
         urlInfo.type !== "sourcemap"
       ) {
         urlInfo.error = null
         urlInfo.sourcemap = null
+        urlInfo.sourcemapIsWrong = null
         urlInfo.sourcemapReference = null
         urlInfo.content = null
         urlInfo.originalContent = null
@@ -305,11 +321,12 @@ export const createFileService = ({
         url: reference.url,
         status: 200,
         headers: {
-          "content-length": Buffer.byteLength(urlInfo.content),
           "cache-control": `private,max-age=0,must-revalidate`,
-          "eTag": urlInfoTargetedByCache.contentEtag,
+          // it's safe to use "_" separator because etag is encoded with base64 (see https://stackoverflow.com/a/13195197)
+          "eTag": `${urlInfoTargetedByCache.originalContentEtag}_${urlInfoTargetedByCache.contentEtag}`,
           ...urlInfo.headers,
           "content-type": urlInfo.contentType,
+          "content-length": Buffer.byteLength(urlInfo.content),
         },
         body: urlInfo.content,
         timing: urlInfo.timing,

package/src/omega/url_graph/url_info_transformations.js

@@ -164,7 +164,7 @@ export const createUrlInfoTransformer = ({
       // is a nightmare no-one could solve in years so
       // jsenv won't emit a warning and use the following strategy:
       // "no sourcemap is better than wrong sourcemap"
-      urlInfo.sourcemapIsWrong = sourcemapIsWrong
+      urlInfo.sourcemapIsWrong = urlInfo.sourcemapIsWrong || sourcemapIsWrong
     }
   }
 
@@ -172,57 +172,60 @@ export const createUrlInfoTransformer = ({
     if (transformations) {
       applyIntermediateTransformations(urlInfo, transformations)
     }
-    if (
-      sourcemapsEnabled &&
-      urlInfo.sourcemap &&
-      !urlInfo.generatedUrl.startsWith("data:")
-    ) {
-      // during build this function can be called after the file is cooked
-      // - to update content and sourcemap after "optimize" hook
-      // - to inject versioning into the entry point content
-      // in this scenarion we don't want to call injectSourcemap
-      // just update the content and the
-      const sourcemapReference = urlInfo.sourcemapReference
-      const sourcemapUrlInfo = urlGraph.getUrlInfo(sourcemapReference.url)
-      sourcemapUrlInfo.contentType = "application/json"
-      const sourcemap = urlInfo.sourcemap
-      if (sourcemapsRelativeSources) {
-        sourcemap.sources = sourcemap.sources.map((source) => {
-          const sourceRelative = urlToRelativeUrl(source, urlInfo.url)
-          return sourceRelative || "."
-        })
-      }
-      if (sourcemapsSourcesProtocol !== "file:///") {
-        sourcemap.sources = sourcemap.sources.map((source) => {
-          if (source.startsWith("file:///")) {
-            return `${sourcemapsSourcesProtocol}${source.slice(
-              "file:///".length,
-            )}`
-          }
-          return source
-        })
-      }
-      sourcemapUrlInfo.content = JSON.stringify(sourcemap, null, "  ")
-      if (!urlInfo.sourcemapIsWrong) {
-        if (sourcemaps === "inline") {
-          sourcemapReference.generatedSpecifier =
-            generateSourcemapDataUrl(sourcemap)
+    if (urlInfo.sourcemapReference) {
+      if (
+        sourcemapsEnabled &&
+        urlInfo.sourcemap &&
+        !urlInfo.generatedUrl.startsWith("data:")
+      ) {
+        // during build this function can be called after the file is cooked
+        // - to update content and sourcemap after "optimize" hook
+        // - to inject versioning into the entry point content
+        // in this scenarion we don't want to call injectSourcemap
+        // just update the content and the
+        const sourcemapReference = urlInfo.sourcemapReference
+        const sourcemapUrlInfo = urlGraph.getUrlInfo(sourcemapReference.url)
+        sourcemapUrlInfo.contentType = "application/json"
+        const sourcemap = urlInfo.sourcemap
+        if (sourcemapsRelativeSources) {
+          sourcemap.sources = sourcemap.sources.map((source) => {
+            const sourceRelative = urlToRelativeUrl(source, urlInfo.url)
+            return sourceRelative || "."
+          })
         }
-        if (sourcemaps === "file" || sourcemaps === "inline") {
-          urlInfo.content = SOURCEMAP.writeComment({
-            contentType: urlInfo.contentType,
-            content: urlInfo.content,
-            specifier:
-              sourcemaps === "file" && sourcemapsRelativeSources
-                ? urlToRelativeUrl(sourcemapReference.url, urlInfo.url)
-                : sourcemapReference.generatedSpecifier,
+        if (sourcemapsSourcesProtocol !== "file:///") {
+          sourcemap.sources = sourcemap.sources.map((source) => {
+            if (source.startsWith("file:///")) {
+              return `${sourcemapsSourcesProtocol}${source.slice(
+                "file:///".length,
+              )}`
+            }
+            return source
           })
         }
+        sourcemapUrlInfo.content = JSON.stringify(sourcemap, null, "  ")
+        if (!urlInfo.sourcemapIsWrong) {
+          if (sourcemaps === "inline") {
+            sourcemapReference.generatedSpecifier =
+              generateSourcemapDataUrl(sourcemap)
+          }
+          if (sourcemaps === "file" || sourcemaps === "inline") {
+            urlInfo.content = SOURCEMAP.writeComment({
+              contentType: urlInfo.contentType,
+              content: urlInfo.content,
+              specifier:
+                sourcemaps === "file" && sourcemapsRelativeSources
+                  ? urlToRelativeUrl(sourcemapReference.url, urlInfo.url)
+                  : sourcemapReference.generatedSpecifier,
+            })
+          }
+        }
+      } else {
+        // in the end we don't use the sourcemap placeholder
+        urlGraph.deleteUrlInfo(urlInfo.sourcemapReference.url)
       }
-    } else if (urlInfo.sourcemapReference) {
-      // in the end we don't use the sourcemap placeholder
-      urlGraph.deleteUrlInfo(urlInfo.sourcemapReference.url)
     }
+
     urlInfo.contentEtag =
       urlInfo.content === urlInfo.originalContent
         ? urlInfo.originalContentEtag

package/src/plugins/autoreload/jsenv_plugin_autoreload_server.js

@@ -1,4 +1,8 @@
-import { urlIsInsideOf, urlToRelativeUrl } from "@jsenv/urls"
+import {
+  urlIsInsideOf,
+  urlToRelativeUrl,
+  asUrlWithoutSearch,
+} from "@jsenv/urls"
 
 export const jsenvPluginAutoreloadServer = ({
   clientFileChangeCallbackList,
@@ -113,26 +117,33 @@ export const jsenvPluginAutoreloadServer = ({
           return iterate(firstUrlInfo, seen)
         }
         clientFileChangeCallbackList.push(({ url, event }) => {
-          const urlInfo = urlGraph.getUrlInfo(url)
-          // file not part of dependency graph
-          if (!urlInfo) {
-            return
-          }
-          const relativeUrl = formatUrlForClient(url)
-          const hotUpdate = propagateUpdate(urlInfo)
-          if (hotUpdate.declined) {
-            notifyDeclined({
-              cause: `${relativeUrl} ${event}`,
-              reason: hotUpdate.reason,
-              declinedBy: hotUpdate.declinedBy,
-            })
-          } else {
-            notifyAccepted({
-              cause: `${relativeUrl} ${event}`,
-              reason: hotUpdate.reason,
-              instructions: hotUpdate.instructions,
-            })
+          const onUrlInfo = (urlInfo) => {
+            const relativeUrl = formatUrlForClient(url)
+            const hotUpdate = propagateUpdate(urlInfo)
+            if (hotUpdate.declined) {
+              notifyDeclined({
+                cause: `${relativeUrl} ${event}`,
+                reason: hotUpdate.reason,
+                declinedBy: hotUpdate.declinedBy,
+              })
+            } else {
+              notifyAccepted({
+                cause: `${relativeUrl} ${event}`,
+                reason: hotUpdate.reason,
+                instructions: hotUpdate.instructions,
+              })
+            }
           }
+          urlGraph.urlInfoMap.forEach((urlInfo) => {
+            if (urlInfo.url === url) {
+              onUrlInfo(urlInfo)
+            } else {
+              const urlWithoutSearch = asUrlWithoutSearch(urlInfo.url)
+              if (urlWithoutSearch === url) {
+                onUrlInfo(urlInfo)
+              }
+            }
+          })
         })
         clientFilesPruneCallbackList.push((prunedUrlInfos, firstUrlInfo) => {
           const mainHotUpdate = propagateUpdate(firstUrlInfo)

package/src/plugins/importmap/jsenv_plugin_importmap.js

@@ -148,8 +148,9 @@ export const jsenvPluginImportmap = () => {
           // We must precook the importmap to know its content and inline it into the HTML
           // In this situation the ref to the importmap was already discovered
           // when parsing the HTML
-          const importmapReference =
-            context.referenceUtils.findByGeneratedSpecifier(src)
+          const importmapReference = context.referenceUtils.find(
+            (ref) => ref.generatedSpecifier === src,
+          )
           const importmapUrlInfo = context.urlGraph.getUrlInfo(
             importmapReference.url,
           )