@jsenv/core 39.3.5 → 39.3.7

package/README.md

@@ -14,7 +14,7 @@ Jsenv is a suite of tools that can be used in projects involving JavaScript.
 It favors standards and simplicity.  
 As a result it can be enjoyed by people without much experience in tooling or seeking for simple tools without hidden complexities.
 
-If you want to try jsenv on your machine, use [@jsenv/cli](./packages/related/cli/readme.md).
+If you want to try jsenv on your machine, use [@jsenv/cli](./packages/related/cli/#jsenvcli).
 
 Link to [documentation](./md/users/users.md)
 

package/dist/jsenv_core.js

@@ -11562,6 +11562,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(resolveError, {
+      isJsenvCookingError: true,
       name: "RESOLVE_URL_ERROR",
       code,
       reason,
@@ -11610,6 +11611,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(fetchError, {
+      isJsenvCookingError: true,
       name: "FETCH_URL_CONTENT_ERROR",
       code,
       reason,
@@ -11670,6 +11672,9 @@ const createTransformUrlContentError = ({
     return error;
   }
   if (error.code === "PARSE_ERROR") {
+    if (error.isJsenvCookingError) {
+      return error;
+    }
     const reference = urlInfo.firstReference;
     let trace = reference.trace;
     let line = error.line;
@@ -11717,13 +11722,16 @@ const createTransformUrlContentError = ({
 ${trace.message}
 ${error.message}`,
         {
-          "first reference": `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`,
+          "first reference": reference.trace.url
+            ? `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`
+            : reference.trace.message,
           ...detailsFromFirstReference(reference),
           ...detailsFromPluginController(pluginController),
         },
       ),
     );
     defineNonEnumerableProperties(transformError, {
+      isJsenvCookingError: true,
       name: "TRANSFORM_URL_CONTENT_ERROR",
       code: "PARSE_ERROR",
       reason: error.message,
@@ -11753,6 +11761,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(transformError, {
+      isJsenvCookingError: true,
       cause: error,
       name: "TRANSFORM_URL_CONTENT_ERROR",
       code,
@@ -11788,6 +11797,7 @@ ${reference.trace.message}`,
     ),
   );
   defineNonEnumerableProperties(finalizeError, {
+    isJsenvCookingError: true,
     ...(error && error instanceof Error ? { cause: error } : {}),
     name: "FINALIZE_URL_CONTENT_ERROR",
     reason: `"finalizeUrlContent" error on "${urlInfo.type}"`,
@@ -11849,9 +11859,9 @@ const detailsFromValueThrown = (valueThrownByPlugin) => {
   };
 };
 
-const defineNonEnumerableProperties = (assertionError, properties) => {
+const defineNonEnumerableProperties = (object, properties) => {
   for (const key of Object.keys(properties)) {
-    Object.defineProperty(assertionError, key, {
+    Object.defineProperty(object, key, {
       configurable: true,
       writable: true,
       value: properties[key],
@@ -14486,7 +14496,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         urlInfo,
         error,
       });
-      urlInfo.error = transformError;
       throw transformError;
     }
   };
@@ -14518,9 +14527,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
 
     // urlInfo objects are reused, they must be "reset" before cooking them again
     if (urlInfo.error || urlInfo.content !== undefined) {
-      if (urlInfo.isInline) {
-        return;
-      }
       urlInfo.error = null;
       urlInfo.type = null;
       urlInfo.subtype = null;
@@ -14542,30 +14548,46 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         });
       } catch (e) {
         urlInfo.error = e;
-        if (
-          urlInfo.isInline &&
-          e.code !== "DIRECTORY_REFERENCE_NOT_ALLOWED" &&
-          errorOnInlineContentCanSkipThrow(urlInfo)
-        ) {
-          // When something like <style> or <script> contains syntax error
-          // the HTML in itself it still valid
-          // keep the syntax error and continue with the HTML
-          if (e.code === "PARSE_ERROR") {
-            logger.error(`parse error on "${urlInfo.type}"
+        if (urlInfo.isInline) {
+          const parentUrlInfo = urlInfo.findParentIfInline();
+          parentUrlInfo.error = e;
+        }
+        let errorWrapperMessage;
+        if (e.code === "PARSE_ERROR") {
+          errorWrapperMessage =
+            e.name === "TRANSFORM_URL_CONTENT_ERROR"
+              ? e.message
+              : `parse error on "${urlInfo.type}"
 ${e.trace?.message}
 ${e.reason}
 --- declared in ---
-${urlInfo.firstReference.trace.message}`);
-            return;
-          }
-          logger.error(
-            `Error while cooking ${urlInfo.type}
-${urlInfo.firstReference.trace.message}
-${e.stack}`,
-          );
+${urlInfo.firstReference.trace.message}`;
+        } else if (e.isJsenvCookingError) {
+          errorWrapperMessage = e.message;
+        } else {
+          errorWrapperMessage = `Error while cooking ${urlInfo.type}
+${urlInfo.firstReference.trace.message}`;
+        }
+        // if we are cooking inline content during dev it's better not to throw
+        // because the main url info (html) is still valid and can be returned to the browser
+        if (
+          urlInfo.isInline &&
+          urlInfo.context.dev &&
+          // but if we are explicitely requesting inline content file then we throw
+          // to properly send 500 to the browser
+          urlInfo.context.reference !== urlInfo.url
+        ) {
+          logger.error(errorWrapperMessage);
           return;
         }
-        throw e;
+        if (e.isJsenvCookingError) {
+          throw e;
+        }
+        const error = new Error(errorWrapperMessage, { cause: e });
+        defineNonEnumerableProperties(error, {
+          __INTERNAL_ERROR__: true,
+        });
+        throw error;
       }
     }
 
@@ -14655,23 +14677,6 @@ ${e.stack}`,
   return kitchen;
 };
 
-// if we are cooking the inline content internally it's better not to throw
-// because the main url info (html) is still valid
-// but if we are explicitely requesting inline content during dev
-// then we should throw
-const errorOnInlineContentCanSkipThrow = (urlInfo) => {
-  if (urlInfo.context.build) {
-    return true;
-  }
-  if (
-    urlInfo.context.reference &&
-    urlInfo.context.reference.url === urlInfo.url
-  ) {
-    return false;
-  }
-  return true;
-};
-
 const debounceCook = (cook) => {
   const pendingDishes = new Map();
   return async (urlInfo, context) => {
@@ -15047,7 +15052,10 @@ const jsenvPluginDirectoryReferenceEffect = (
       }
       if (actionForDirectory === "error") {
         const error = new Error("Reference leads to a directory");
-        error.code = "DIRECTORY_REFERENCE_NOT_ALLOWED";
+        defineNonEnumerableProperties(error, {
+          isJsenvCookingError: true,
+          code: "DIRECTORY_REFERENCE_NOT_ALLOWED",
+        });
         throw error;
       }
       if (actionForDirectory === "preserve") {
@@ -16450,13 +16458,8 @@ const jsenvPluginHtmlReferenceAnalysis = ({
             });
 
             actions.push(async () => {
-              try {
-                await inlineReference.urlInfo.cook();
-              } catch (e) {
-                if (!e || e.code !== "PARSE_ERROR") {
-                  throw e;
-                }
-              }
+              const inlineUrlInfo = inlineReference.urlInfo;
+              await inlineUrlInfo.cook();
               mutations.push(() => {
                 if (hotAccept) {
                   removeHtmlNodeText(node);
@@ -16464,7 +16467,7 @@ const jsenvPluginHtmlReferenceAnalysis = ({
                     "jsenv-cooked-by": "jsenv:html_inline_content_analysis",
                   });
                 } else {
-                  setHtmlNodeText(node, inlineReference.urlInfo.content, {
+                  setHtmlNodeText(node, inlineUrlInfo.content, {
                     indentation: false, // indentation would decrease stack trace precision
                   });
                   setHtmlNodeAttributes(node, {
@@ -16737,7 +16740,8 @@ const jsenvPluginHtmlReferenceAnalysis = ({
           }
           mutations.forEach((mutation) => mutation());
           mutations.length = 0;
-          return stringifyHtmlAst(htmlAst);
+          const html = stringifyHtmlAst(htmlAst);
+          return html;
         } catch (e) {
           importmapLoaded();
           throw e;
@@ -17104,8 +17108,12 @@ const jsenvPluginInlineContentFetcher = () => {
       //   when updating the file, first reference is the previous version
       // - we cannot use urlInfo.lastReference because it can be the reference created by "http_request"
       let lastInlineReference;
+      let originalContent = urlInfo.originalContent;
       for (const reference of urlInfo.referenceFromOthersSet) {
         if (reference.isInline) {
+          if (originalContent === undefined) {
+            originalContent = reference.content;
+          }
           lastInlineReference = reference;
         }
       }
@@ -17120,8 +17128,17 @@ const jsenvPluginInlineContentFetcher = () => {
         }
       }
       return {
-        originalContent: urlInfo.originalContent,
-        content: lastInlineReference.content,
+        originalContent,
+        content:
+          // we must favor original content to re-apply the same plugin logic
+          // so that the same references are generated
+          // without this we would try to resolve references like
+          // "/node_modules/package/file.js" instead of "package/file.js"
+          // meaning we would not create the implicit dependency to package.json
+          // resulting in a reload of the browser (as implicit reference to package.json is gone)
+          originalContent === undefined
+            ? lastInlineReference.content
+            : originalContent,
         contentType: lastInlineReference.contentType,
       };
     },
@@ -18274,47 +18291,46 @@ const createNodeEsmResolver = ({
       specifier: reference.specifier,
       preservesSymlink,
     });
-    if (ownerUrlInfo.context.dev) {
-      const dependsOnPackageJson =
-        type !== "relative_specifier" &&
-        type !== "absolute_specifier" &&
-        type !== "node_builtin_specifier";
-      if (dependsOnPackageJson) {
-        // this reference depends on package.json and node_modules
-        // to be resolved. Each file using this specifier
-        // must be invalidated when corresponding package.json changes
-        addRelationshipWithPackageJson({
-          reference,
-          packageJsonUrl: `${packageDirectoryUrl}package.json`,
-          field: type.startsWith("field:")
-            ? `#${type.slice("field:".length)}`
-            : "",
-        });
-      }
+    if (ownerUrlInfo.context.build) {
+      return url;
     }
-    if (ownerUrlInfo.context.dev) {
-      // without this check a file inside a project without package.json
-      // could be considered as a node module if there is a ancestor package.json
-      // but we want to version only node modules
-      if (url.includes("/node_modules/")) {
-        const packageDirectoryUrl = defaultLookupPackageScope(url);
-        if (
-          packageDirectoryUrl &&
-          packageDirectoryUrl !== ownerUrlInfo.context.rootDirectoryUrl
-        ) {
-          const packageVersion =
-            defaultReadPackageJson(packageDirectoryUrl).version;
-          // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
-          if (packageVersion) {
-            addRelationshipWithPackageJson({
-              reference,
-              packageJsonUrl: `${packageDirectoryUrl}package.json`,
-              field: "version",
-              hasVersioningEffect: true,
-            });
-          }
-          reference.version = packageVersion;
+    const dependsOnPackageJson =
+      type !== "relative_specifier" &&
+      type !== "absolute_specifier" &&
+      type !== "node_builtin_specifier";
+    if (dependsOnPackageJson) {
+      // this reference depends on package.json and node_modules
+      // to be resolved. Each file using this specifier
+      // must be invalidated when corresponding package.json changes
+      addRelationshipWithPackageJson({
+        reference,
+        packageJsonUrl: `${packageDirectoryUrl}package.json`,
+        field: type.startsWith("field:")
+          ? `#${type.slice("field:".length)}`
+          : "",
+      });
+    }
+    // without this check a file inside a project without package.json
+    // could be considered as a node module if there is a ancestor package.json
+    // but we want to version only node modules
+    if (url.includes("/node_modules/")) {
+      const packageDirectoryUrl = defaultLookupPackageScope(url);
+      if (
+        packageDirectoryUrl &&
+        packageDirectoryUrl !== ownerUrlInfo.context.rootDirectoryUrl
+      ) {
+        const packageVersion =
+          defaultReadPackageJson(packageDirectoryUrl).version;
+        // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
+        if (packageVersion) {
+          addRelationshipWithPackageJson({
+            reference,
+            packageJsonUrl: `${packageDirectoryUrl}package.json`,
+            field: "version",
+            hasVersioningEffect: true,
+          });
         }
+        reference.version = packageVersion;
       }
     }
     return url;
@@ -23241,7 +23257,6 @@ const startDevServer = async ({
               };
             }
           }
-
           await urlInfo.cook({ request, reference });
           let { response } = urlInfo;
           if (response) {
@@ -23284,9 +23299,8 @@ const startDevServer = async ({
             },
           );
           return response;
-        } catch (e) {
-          urlInfo.error = e;
-          const originalError = e ? e.cause || e : e;
+        } catch (error) {
+          const originalError = error ? error.cause || error : error;
           if (originalError.asResponse) {
             return originalError.asResponse();
           }
@@ -23298,13 +23312,13 @@ const startDevServer = async ({
             if (urlInfo.content !== undefined) {
               kitchen.context.logger.error(`Error while handling ${request.url}:
 ${originalError.reasonCode || originalError.code}
-${e.trace?.message}`);
+${error.trace?.message}`);
               return {
                 url: reference.url,
                 status: 200,
                 // reason becomes the http response statusText, it must not contain invalid chars
                 // https://github.com/nodejs/node/blob/0c27ca4bc9782d658afeaebcec85ec7b28f1cc35/lib/_http_common.js#L221
-                statusText: e.reason,
+                statusText: error.reason,
                 statusMessage: originalError.message,
                 headers: {
                   "content-type": urlInfo.contentType,
@@ -23317,7 +23331,7 @@ ${e.trace?.message}`);
             return {
               url: reference.url,
               status: 500,
-              statusText: e.reason,
+              statusText: error.reason,
               statusMessage: originalError.message,
               headers: {
                 "cache-control": "no-store",
@@ -23352,8 +23366,11 @@ ${e.trace?.message}`);
           return {
             url: reference.url,
             status: 500,
-            statusText: e.reason,
-            statusMessage: e.stack,
+            statusText: error.reason,
+            statusMessage: error.stack,
+            headers: {
+              "cache-control": "no-store",
+            },
           };
         }
       },

package/package.json

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

package/src/dev/start_dev_server.js

@@ -448,7 +448,6 @@ export const startDevServer = async ({
               };
             }
           }
-
           await urlInfo.cook({ request, reference });
           let { response } = urlInfo;
           if (response) {
@@ -491,9 +490,8 @@ export const startDevServer = async ({
             },
           );
           return response;
-        } catch (e) {
-          urlInfo.error = e;
-          const originalError = e ? e.cause || e : e;
+        } catch (error) {
+          const originalError = error ? error.cause || error : error;
           if (originalError.asResponse) {
             return originalError.asResponse();
           }
@@ -505,13 +503,13 @@ export const startDevServer = async ({
             if (urlInfo.content !== undefined) {
               kitchen.context.logger.error(`Error while handling ${request.url}:
 ${originalError.reasonCode || originalError.code}
-${e.trace?.message}`);
+${error.trace?.message}`);
               return {
                 url: reference.url,
                 status: 200,
                 // reason becomes the http response statusText, it must not contain invalid chars
                 // https://github.com/nodejs/node/blob/0c27ca4bc9782d658afeaebcec85ec7b28f1cc35/lib/_http_common.js#L221
-                statusText: e.reason,
+                statusText: error.reason,
                 statusMessage: originalError.message,
                 headers: {
                   "content-type": urlInfo.contentType,
@@ -524,7 +522,7 @@ ${e.trace?.message}`);
             return {
               url: reference.url,
               status: 500,
-              statusText: e.reason,
+              statusText: error.reason,
               statusMessage: originalError.message,
               headers: {
                 "cache-control": "no-store",
@@ -559,8 +557,11 @@ ${e.trace?.message}`);
           return {
             url: reference.url,
             status: 500,
-            statusText: e.reason,
-            statusMessage: e.stack,
+            statusText: error.reason,
+            statusMessage: error.stack,
+            headers: {
+              "cache-control": "no-store",
+            },
           };
         }
       },

package/src/kitchen/errors.js

@@ -25,6 +25,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(resolveError, {
+      isJsenvCookingError: true,
       name: "RESOLVE_URL_ERROR",
       code,
       reason,
@@ -73,6 +74,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(fetchError, {
+      isJsenvCookingError: true,
       name: "FETCH_URL_CONTENT_ERROR",
       code,
       reason,
@@ -133,6 +135,9 @@ export const createTransformUrlContentError = ({
     return error;
   }
   if (error.code === "PARSE_ERROR") {
+    if (error.isJsenvCookingError) {
+      return error;
+    }
     const reference = urlInfo.firstReference;
     let trace = reference.trace;
     let line = error.line;
@@ -180,13 +185,16 @@ export const createTransformUrlContentError = ({
 ${trace.message}
 ${error.message}`,
         {
-          "first reference": `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`,
+          "first reference": reference.trace.url
+            ? `${reference.trace.url}:${reference.trace.line}:${reference.trace.column}`
+            : reference.trace.message,
           ...detailsFromFirstReference(reference),
           ...detailsFromPluginController(pluginController),
         },
       ),
     );
     defineNonEnumerableProperties(transformError, {
+      isJsenvCookingError: true,
       name: "TRANSFORM_URL_CONTENT_ERROR",
       code: "PARSE_ERROR",
       reason: error.message,
@@ -216,6 +224,7 @@ ${reason}`,
       ),
     );
     defineNonEnumerableProperties(transformError, {
+      isJsenvCookingError: true,
       cause: error,
       name: "TRANSFORM_URL_CONTENT_ERROR",
       code,
@@ -251,6 +260,7 @@ ${reference.trace.message}`,
     ),
   );
   defineNonEnumerableProperties(finalizeError, {
+    isJsenvCookingError: true,
     ...(error && error instanceof Error ? { cause: error } : {}),
     name: "FINALIZE_URL_CONTENT_ERROR",
     reason: `"finalizeUrlContent" error on "${urlInfo.type}"`,
@@ -312,9 +322,9 @@ const detailsFromValueThrown = (valueThrownByPlugin) => {
   };
 };
 
-const defineNonEnumerableProperties = (assertionError, properties) => {
+export const defineNonEnumerableProperties = (object, properties) => {
   for (const key of Object.keys(properties)) {
-    Object.defineProperty(assertionError, key, {
+    Object.defineProperty(object, key, {
       configurable: true,
       writable: true,
       value: properties[key],

package/src/kitchen/kitchen.js

@@ -17,6 +17,7 @@ import {
   createFinalizeUrlContentError,
   createResolveUrlError,
   createTransformUrlContentError,
+  defineNonEnumerableProperties,
 } from "./errors.js";
 import { assertFetchedContentCompliance } from "./fetched_content_compliance.js";
 import { createUrlGraph } from "./url_graph/url_graph.js";
@@ -426,7 +427,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         urlInfo,
         error,
       });
-      urlInfo.error = transformError;
       throw transformError;
     }
   };
@@ -458,9 +458,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
 
     // urlInfo objects are reused, they must be "reset" before cooking them again
     if (urlInfo.error || urlInfo.content !== undefined) {
-      if (urlInfo.isInline) {
-        return;
-      }
       urlInfo.error = null;
       urlInfo.type = null;
       urlInfo.subtype = null;
@@ -482,30 +479,46 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         });
       } catch (e) {
         urlInfo.error = e;
-        if (
-          urlInfo.isInline &&
-          e.code !== "DIRECTORY_REFERENCE_NOT_ALLOWED" &&
-          errorOnInlineContentCanSkipThrow(urlInfo)
-        ) {
-          // When something like <style> or <script> contains syntax error
-          // the HTML in itself it still valid
-          // keep the syntax error and continue with the HTML
-          if (e.code === "PARSE_ERROR") {
-            logger.error(`parse error on "${urlInfo.type}"
+        if (urlInfo.isInline) {
+          const parentUrlInfo = urlInfo.findParentIfInline();
+          parentUrlInfo.error = e;
+        }
+        let errorWrapperMessage;
+        if (e.code === "PARSE_ERROR") {
+          errorWrapperMessage =
+            e.name === "TRANSFORM_URL_CONTENT_ERROR"
+              ? e.message
+              : `parse error on "${urlInfo.type}"
 ${e.trace?.message}
 ${e.reason}
 --- declared in ---
-${urlInfo.firstReference.trace.message}`);
-            return;
-          }
-          logger.error(
-            `Error while cooking ${urlInfo.type}
-${urlInfo.firstReference.trace.message}
-${e.stack}`,
-          );
+${urlInfo.firstReference.trace.message}`;
+        } else if (e.isJsenvCookingError) {
+          errorWrapperMessage = e.message;
+        } else {
+          errorWrapperMessage = `Error while cooking ${urlInfo.type}
+${urlInfo.firstReference.trace.message}`;
+        }
+        // if we are cooking inline content during dev it's better not to throw
+        // because the main url info (html) is still valid and can be returned to the browser
+        if (
+          urlInfo.isInline &&
+          urlInfo.context.dev &&
+          // but if we are explicitely requesting inline content file then we throw
+          // to properly send 500 to the browser
+          urlInfo.context.reference !== urlInfo.url
+        ) {
+          logger.error(errorWrapperMessage);
           return;
         }
-        throw e;
+        if (e.isJsenvCookingError) {
+          throw e;
+        }
+        const error = new Error(errorWrapperMessage, { cause: e });
+        defineNonEnumerableProperties(error, {
+          __INTERNAL_ERROR__: true,
+        });
+        throw error;
       }
     }
 
@@ -595,23 +608,6 @@ ${e.stack}`,
   return kitchen;
 };
 
-// if we are cooking the inline content internally it's better not to throw
-// because the main url info (html) is still valid
-// but if we are explicitely requesting inline content during dev
-// then we should throw
-const errorOnInlineContentCanSkipThrow = (urlInfo) => {
-  if (urlInfo.context.build) {
-    return true;
-  }
-  if (
-    urlInfo.context.reference &&
-    urlInfo.context.reference.url === urlInfo.url
-  ) {
-    return false;
-  }
-  return true;
-};
-
 const debounceCook = (cook) => {
   const pendingDishes = new Map();
   return async (urlInfo, context) => {

package/src/plugins/directory_reference_effect/jsenv_plugin_directory_reference_effect.js

@@ -1,4 +1,5 @@
 import { urlToFilename } from "@jsenv/urls";
+import { defineNonEnumerableProperties } from "../../kitchen/errors.js";
 
 export const jsenvPluginDirectoryReferenceEffect = (
   directoryReferenceEffect = "error",
@@ -53,7 +54,10 @@ export const jsenvPluginDirectoryReferenceEffect = (
       }
       if (actionForDirectory === "error") {
         const error = new Error("Reference leads to a directory");
-        error.code = "DIRECTORY_REFERENCE_NOT_ALLOWED";
+        defineNonEnumerableProperties(error, {
+          isJsenvCookingError: true,
+          code: "DIRECTORY_REFERENCE_NOT_ALLOWED",
+        });
         throw error;
       }
       if (actionForDirectory === "preserve") {

package/src/plugins/reference_analysis/html/jsenv_plugin_html_reference_analysis.js

@@ -274,13 +274,8 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
             });
 
             actions.push(async () => {
-              try {
-                await inlineReference.urlInfo.cook();
-              } catch (e) {
-                if (!e || e.code !== "PARSE_ERROR") {
-                  throw e;
-                }
-              }
+              const inlineUrlInfo = inlineReference.urlInfo;
+              await inlineUrlInfo.cook();
               mutations.push(() => {
                 if (hotAccept) {
                   removeHtmlNodeText(node);
@@ -288,7 +283,7 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
                     "jsenv-cooked-by": "jsenv:html_inline_content_analysis",
                   });
                 } else {
-                  setHtmlNodeText(node, inlineReference.urlInfo.content, {
+                  setHtmlNodeText(node, inlineUrlInfo.content, {
                     indentation: false, // indentation would decrease stack trace precision
                   });
                   setHtmlNodeAttributes(node, {
@@ -561,7 +556,8 @@ export const jsenvPluginHtmlReferenceAnalysis = ({
           }
           mutations.forEach((mutation) => mutation());
           mutations.length = 0;
-          return stringifyHtmlAst(htmlAst);
+          const html = stringifyHtmlAst(htmlAst);
+          return html;
         } catch (e) {
           importmapLoaded();
           throw e;

package/src/plugins/reference_analysis/jsenv_plugin_reference_analysis.js

@@ -45,8 +45,12 @@ const jsenvPluginInlineContentFetcher = () => {
       //   when updating the file, first reference is the previous version
       // - we cannot use urlInfo.lastReference because it can be the reference created by "http_request"
       let lastInlineReference;
+      let originalContent = urlInfo.originalContent;
       for (const reference of urlInfo.referenceFromOthersSet) {
         if (reference.isInline) {
+          if (originalContent === undefined) {
+            originalContent = reference.content;
+          }
           lastInlineReference = reference;
         }
       }
@@ -61,8 +65,17 @@ const jsenvPluginInlineContentFetcher = () => {
         }
       }
       return {
-        originalContent: urlInfo.originalContent,
-        content: lastInlineReference.content,
+        originalContent,
+        content:
+          // we must favor original content to re-apply the same plugin logic
+          // so that the same references are generated
+          // without this we would try to resolve references like
+          // "/node_modules/package/file.js" instead of "package/file.js"
+          // meaning we would not create the implicit dependency to package.json
+          // resulting in a reload of the browser (as implicit reference to package.json is gone)
+          originalContent === undefined
+            ? lastInlineReference.content
+            : originalContent,
         contentType: lastInlineReference.contentType,
       };
     },

package/src/plugins/resolution_node_esm/node_esm_resolver.js

@@ -53,47 +53,46 @@ export const createNodeEsmResolver = ({
       specifier: reference.specifier,
       preservesSymlink,
     });
-    if (ownerUrlInfo.context.dev) {
-      const dependsOnPackageJson =
-        type !== "relative_specifier" &&
-        type !== "absolute_specifier" &&
-        type !== "node_builtin_specifier";
-      if (dependsOnPackageJson) {
-        // this reference depends on package.json and node_modules
-        // to be resolved. Each file using this specifier
-        // must be invalidated when corresponding package.json changes
-        addRelationshipWithPackageJson({
-          reference,
-          packageJsonUrl: `${packageDirectoryUrl}package.json`,
-          field: type.startsWith("field:")
-            ? `#${type.slice("field:".length)}`
-            : "",
-        });
-      }
+    if (ownerUrlInfo.context.build) {
+      return url;
+    }
+    const dependsOnPackageJson =
+      type !== "relative_specifier" &&
+      type !== "absolute_specifier" &&
+      type !== "node_builtin_specifier";
+    if (dependsOnPackageJson) {
+      // this reference depends on package.json and node_modules
+      // to be resolved. Each file using this specifier
+      // must be invalidated when corresponding package.json changes
+      addRelationshipWithPackageJson({
+        reference,
+        packageJsonUrl: `${packageDirectoryUrl}package.json`,
+        field: type.startsWith("field:")
+          ? `#${type.slice("field:".length)}`
+          : "",
+      });
     }
-    if (ownerUrlInfo.context.dev) {
-      // without this check a file inside a project without package.json
-      // could be considered as a node module if there is a ancestor package.json
-      // but we want to version only node modules
-      if (url.includes("/node_modules/")) {
-        const packageDirectoryUrl = defaultLookupPackageScope(url);
-        if (
-          packageDirectoryUrl &&
-          packageDirectoryUrl !== ownerUrlInfo.context.rootDirectoryUrl
-        ) {
-          const packageVersion =
-            defaultReadPackageJson(packageDirectoryUrl).version;
-          // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
-          if (packageVersion) {
-            addRelationshipWithPackageJson({
-              reference,
-              packageJsonUrl: `${packageDirectoryUrl}package.json`,
-              field: "version",
-              hasVersioningEffect: true,
-            });
-          }
-          reference.version = packageVersion;
+    // without this check a file inside a project without package.json
+    // could be considered as a node module if there is a ancestor package.json
+    // but we want to version only node modules
+    if (url.includes("/node_modules/")) {
+      const packageDirectoryUrl = defaultLookupPackageScope(url);
+      if (
+        packageDirectoryUrl &&
+        packageDirectoryUrl !== ownerUrlInfo.context.rootDirectoryUrl
+      ) {
+        const packageVersion =
+          defaultReadPackageJson(packageDirectoryUrl).version;
+        // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
+        if (packageVersion) {
+          addRelationshipWithPackageJson({
+            reference,
+            packageJsonUrl: `${packageDirectoryUrl}package.json`,
+            field: "version",
+            hasVersioningEffect: true,
+          });
         }
+        reference.version = packageVersion;
       }
     }
     return url;