@jsenv/core 41.5.15 → 41.5.17

package/dist/build/build.js

@@ -12210,8 +12210,16 @@ const createBuildUrlsGenerator = ({
     // THAT bundle). They are told apart by the entry point they come from —
     // sharing the file would leave one entry importing exports the file on disk
     // does not have.
+    //
+    // A source file is shared between entry points only while they write it to
+    // the same assets directory. Two entry points with their own directory each
+    // get their own copy: the file lands where the entry point that references
+    // it lands, so removing one entry point's directory takes nothing away from
+    // another.
     const insideBuildDirectory = urlIsOrIsInsideOf(url, buildDirectoryUrl);
-    const key = insideBuildDirectory ? `${entryKey} ${url}` : url;
+    const key = insideBuildDirectory
+      ? `${entryKey} ${url}`
+      : `${assetsDirectory} ${url}`;
     const buildUrlFromMap = buildUrlMap.get(key);
     if (buildUrlFromMap) {
       return buildUrlFromMap;
@@ -12243,7 +12251,7 @@ const createBuildUrlsGenerator = ({
     }
     if (urlInfo.type === "entry_build") {
       const buildUrl = new URL(urlInfo.filenameHint, buildDirectoryUrl).href;
-      associateBuildUrl(url, buildUrl);
+      associateBuildUrl(key, buildUrl);
       return buildUrl;
     }
     if (
@@ -12261,7 +12269,7 @@ const createBuildUrlsGenerator = ({
       const urlObject = new URL(url);
       const { search } = urlObject;
       const buildUrl = `${buildDirectoryUrl}${directoryPath}${search}`;
-      associateBuildUrl(url, buildUrl);
+      associateBuildUrl(key, buildUrl);
       return buildUrl;
     }
 
@@ -12276,7 +12284,7 @@ const createBuildUrlsGenerator = ({
     let { search, hash } = urlObject;
     const name = reserveName(directoryPath, getUrlName(url, urlInfo));
     const buildUrl = `${buildDirectoryUrl}${directoryPath}${name}${search}${hash}`;
-    associateBuildUrl(url, buildUrl);
+    associateBuildUrl(key, buildUrl);
     return buildUrl;
   };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "41.5.15",
+  "version": "41.5.17",
   "type": "module",
   "description": "Tool to develop, test and build js projects",
   "repository": {
@@ -44,7 +44,7 @@
   "scripts": {
     "build": "node --conditions=dev:jsenv ./scripts/build/build.mjs",
     "build:file_size": "node ./scripts/build/build_file_size.mjs --log",
-    "build:packages": "npm run build --workspaces --if-present --conditions=developement",
+    "build:packages": "npm run build -- ./packages/",
     "database:install": "npx @jsenv/database install",
     "database:manage": "node --conditions=dev:jsenv ./packages/backend/database/src/cli/manage.js",
     "database:setup": "npm run database:setup --workspaces --if-present --conditions=dev:jsenv",
@@ -73,14 +73,14 @@
     "test:snapshot_clear": "npx @jsenv/filesystem clear **/tests/**/side_effects/"
   },
   "dependencies": {
-    "@jsenv/ast": "6.10.3",
-    "@jsenv/js-module-fallback": "1.6.7",
-    "@jsenv/plugin-bundling": "2.10.27",
-    "@jsenv/plugin-minification": "1.7.20",
-    "@jsenv/plugin-supervisor": "1.8.22",
-    "@jsenv/plugin-transpilation": "1.7.7",
+    "@jsenv/ast": "6.10.4",
+    "@jsenv/js-module-fallback": "1.6.8",
+    "@jsenv/plugin-bundling": "2.10.28",
+    "@jsenv/plugin-minification": "1.7.21",
+    "@jsenv/plugin-supervisor": "1.8.23",
+    "@jsenv/plugin-transpilation": "1.7.8",
     "@jsenv/server": "17.6.4",
-    "@jsenv/sourcemap": "1.4.8",
+    "@jsenv/sourcemap": "1.4.9",
     "react-table": "7.8.0"
   },
   "devDependencies": {

package/src/build/build_urls_generator.js

@@ -59,8 +59,16 @@ export const createBuildUrlsGenerator = ({
     // THAT bundle). They are told apart by the entry point they come from —
     // sharing the file would leave one entry importing exports the file on disk
     // does not have.
+    //
+    // A source file is shared between entry points only while they write it to
+    // the same assets directory. Two entry points with their own directory each
+    // get their own copy: the file lands where the entry point that references
+    // it lands, so removing one entry point's directory takes nothing away from
+    // another.
     const insideBuildDirectory = urlIsOrIsInsideOf(url, buildDirectoryUrl);
-    const key = insideBuildDirectory ? `${entryKey} ${url}` : url;
+    const key = insideBuildDirectory
+      ? `${entryKey} ${url}`
+      : `${assetsDirectory} ${url}`;
     const buildUrlFromMap = buildUrlMap.get(key);
     if (buildUrlFromMap) {
       return buildUrlFromMap;
@@ -92,7 +100,7 @@ export const createBuildUrlsGenerator = ({
     }
     if (urlInfo.type === "entry_build") {
       const buildUrl = new URL(urlInfo.filenameHint, buildDirectoryUrl).href;
-      associateBuildUrl(url, buildUrl);
+      associateBuildUrl(key, buildUrl);
       return buildUrl;
     }
     if (
@@ -110,7 +118,7 @@ export const createBuildUrlsGenerator = ({
       const urlObject = new URL(url);
       const { search } = urlObject;
       const buildUrl = `${buildDirectoryUrl}${directoryPath}${search}`;
-      associateBuildUrl(url, buildUrl);
+      associateBuildUrl(key, buildUrl);
       return buildUrl;
     }
 
@@ -125,7 +133,7 @@ export const createBuildUrlsGenerator = ({
     let { search, hash } = urlObject;
     const name = reserveName(directoryPath, getUrlName(url, urlInfo));
     const buildUrl = `${buildDirectoryUrl}${directoryPath}${name}${search}${hash}`;
-    associateBuildUrl(url, buildUrl);
+    associateBuildUrl(key, buildUrl);
     return buildUrl;
   };