@jsenv/core 39.3.6 → 39.3.8

package/dist/jsenv_core.js

@@ -17108,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 (urlInfo.originalContent === undefined) {
+            originalContent = reference.content;
+          }
           lastInlineReference = reference;
         }
       }
@@ -17119,13 +17123,23 @@ const jsenvPluginInlineContentFetcher = () => {
         if (lastInlineReference.content === undefined) {
           const originalUrlInfo = prev.urlInfo;
           await originalUrlInfo.cook();
+          originalContent = originalUrlInfo.originalContent;
           lastInlineReference.content = originalUrlInfo.content;
           lastInlineReference.contentType = originalUrlInfo.contentType;
         }
       }
       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,
       };
     },
@@ -19984,9 +19998,6 @@ const jsenvPluginAutoreloadServer = ({
                 // are lost and sourcemap is considered as pruned
                 continue;
               }
-              if (lastReferenceFromOther.type === "package_json") {
-                continue;
-              }
               const { ownerUrlInfo } = lastReferenceFromOther;
               if (!ownerUrlInfo.isUsed()) {
                 continue;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "39.3.6",
+  "version": "39.3.8",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -69,7 +69,7 @@
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.3.0",
     "@jsenv/ast": "6.2.16",
-    "@jsenv/filesystem": "4.10.0",
+    "@jsenv/filesystem": "4.10.1",
     "@jsenv/humanize": "1.2.8",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.2",

package/src/plugins/autoreload/jsenv_plugin_autoreload_server.js

@@ -238,9 +238,6 @@ export const jsenvPluginAutoreloadServer = ({
                 // are lost and sourcemap is considered as pruned
                 continue;
               }
-              if (lastReferenceFromOther.type === "package_json") {
-                continue;
-              }
               const { ownerUrlInfo } = lastReferenceFromOther;
               if (!ownerUrlInfo.isUsed()) {
                 continue;

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 (urlInfo.originalContent === undefined) {
+            originalContent = reference.content;
+          }
           lastInlineReference = reference;
         }
       }
@@ -56,13 +60,23 @@ const jsenvPluginInlineContentFetcher = () => {
         if (lastInlineReference.content === undefined) {
           const originalUrlInfo = prev.urlInfo;
           await originalUrlInfo.cook();
+          originalContent = originalUrlInfo.originalContent;
           lastInlineReference.content = originalUrlInfo.content;
           lastInlineReference.contentType = originalUrlInfo.contentType;
         }
       }
       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,
       };
     },