@jsenv/core 41.4.14 → 41.4.16

package/dist/build/build.js

@@ -6089,9 +6089,15 @@ const createNodeEsmResolver = ({
         }
       }
       version_relationship: {
-        const packageVersion = packageDirectory.read(
+        const closestPackageJson = packageDirectory.read(
           closestPackageDirectoryUrl,
-        ).version;
+        );
+        if (!closestPackageJson) {
+          // package.json can be momentarily missing while a package manager
+          // is installing/updating that package (npm install running in watch mode)
+          break version_relationship;
+        }
+        const packageVersion = closestPackageJson.version;
         if (!packageVersion) {
           // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
           break version_relationship;
@@ -9824,6 +9830,17 @@ const jsenvPluginWorkspaceBundle = ({ packageDirectory }) => {
         // main is node-only code (node:url) that cannot be served to a browser.
         return null;
       }
+      const ownerPackageDirectoryUrl = packageDirectory.find(
+        reference.ownerUrlInfo.url,
+      );
+      if (ownerPackageDirectoryUrl === packageDirectoryUrl) {
+        // Import between two files of the same package: the owner was reached
+        // outside the package bundle (an HTML <script src> entry point, see
+        // above). Consolidation only applies to references crossing into a
+        // package from outside; redirecting an intra-package relative import
+        // to the package main would run the wrong module.
+        return null;
+      }
       // we make sure we target the bundle version of the package
       // otherwise we might execute some parts of the package code multiple times.
       // so we need to redirect the potential reference to non entry point to the package main entry point

package/dist/start_dev_server/start_dev_server.js

@@ -2805,9 +2805,15 @@ const createNodeEsmResolver = ({
         }
       }
       version_relationship: {
-        const packageVersion = packageDirectory.read(
+        const closestPackageJson = packageDirectory.read(
           closestPackageDirectoryUrl,
-        ).version;
+        );
+        if (!closestPackageJson) {
+          // package.json can be momentarily missing while a package manager
+          // is installing/updating that package (npm install running in watch mode)
+          break version_relationship;
+        }
+        const packageVersion = closestPackageJson.version;
         if (!packageVersion) {
           // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
           break version_relationship;
@@ -7674,6 +7680,17 @@ const jsenvPluginWorkspaceBundle = ({ packageDirectory }) => {
         // main is node-only code (node:url) that cannot be served to a browser.
         return null;
       }
+      const ownerPackageDirectoryUrl = packageDirectory.find(
+        reference.ownerUrlInfo.url,
+      );
+      if (ownerPackageDirectoryUrl === packageDirectoryUrl) {
+        // Import between two files of the same package: the owner was reached
+        // outside the package bundle (an HTML <script src> entry point, see
+        // above). Consolidation only applies to references crossing into a
+        // package from outside; redirecting an intra-package relative import
+        // to the package main would run the wrong module.
+        return null;
+      }
       // we make sure we target the bundle version of the package
       // otherwise we might execute some parts of the package code multiple times.
       // so we need to redirect the potential reference to non entry point to the package main entry point

package/package.json

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

package/src/plugins/resolution_node_esm/node_esm_resolver.js

@@ -154,9 +154,15 @@ export const createNodeEsmResolver = ({
         }
       }
       version_relationship: {
-        const packageVersion = packageDirectory.read(
+        const closestPackageJson = packageDirectory.read(
           closestPackageDirectoryUrl,
-        ).version;
+        );
+        if (!closestPackageJson) {
+          // package.json can be momentarily missing while a package manager
+          // is installing/updating that package (npm install running in watch mode)
+          break version_relationship;
+        }
+        const packageVersion = closestPackageJson.version;
         if (!packageVersion) {
           // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
           break version_relationship;

package/src/plugins/workspace_bundle/jsenv_plugin_workspace_bundle.js

@@ -51,6 +51,17 @@ export const jsenvPluginWorkspaceBundle = ({ packageDirectory }) => {
         // main is node-only code (node:url) that cannot be served to a browser.
         return null;
       }
+      const ownerPackageDirectoryUrl = packageDirectory.find(
+        reference.ownerUrlInfo.url,
+      );
+      if (ownerPackageDirectoryUrl === packageDirectoryUrl) {
+        // Import between two files of the same package: the owner was reached
+        // outside the package bundle (an HTML <script src> entry point, see
+        // above). Consolidation only applies to references crossing into a
+        // package from outside; redirecting an intra-package relative import
+        // to the package main would run the wrong module.
+        return null;
+      }
       // we make sure we target the bundle version of the package
       // otherwise we might execute some parts of the package code multiple times.
       // so we need to redirect the potential reference to non entry point to the package main entry point