@jsenv/core 41.5.26 → 41.5.28

package/dist/build/build.js

@@ -11555,6 +11555,11 @@ const createBuildSpecifierManager = ({
               // when versioning is dynamic no need to take into account
               continue;
             }
+            if (referenceVersioningInfo.type === "not_versioned") {
+              // the specifier is emitted bare (entry point, webmanifest, ...):
+              // the referencing file's bytes cannot change with that file's content
+              continue;
+            }
             placeholderInfluencingVersionSet.add(containedPlaceholder);
             const referencedUrlInfo = reference.urlInfo;
             visitContainedPlaceholders(referencedUrlInfo);
@@ -13847,6 +13852,29 @@ const prepareEntryPointBuild = async (
         if (!otherEntryBuildInfo) {
           return null;
         }
+        if (urlInfo.subtypeHint === "service_worker") {
+          // A registered service worker is already an entry of the build
+          // registering it: that build owns its bundle and the resources list
+          // it precaches. A second owner from "entryPoints" would receive
+          // neither, so the configuration is refused.
+          const workerRelativeUrl = urlToRelativeUrl(
+            urlInfo.url,
+            sourceDirectoryUrl,
+          );
+          const registeringRelativeUrl = urlToRelativeUrl(
+            urlInfo.firstReference.ownerUrlInfo.url,
+            sourceDirectoryUrl,
+          );
+          throw new Error(
+            createDetailedMessage(
+              `"${workerRelativeUrl}" is registered as a service worker and declared in "entryPoints"`,
+              {
+                "registered by": registeringRelativeUrl,
+                "suggestion": `remove "./${workerRelativeUrl}" from "entryPoints": a registered service worker is built as its own entry by the build registering it, and its "injections" go under "${sourceRelativeUrl}" keyed by "./${workerRelativeUrl}"`,
+              },
+            ),
+          );
+        }
         urlInfo.otherEntryBuildInfo = otherEntryBuildInfo;
         return {
           type: "entry_build", // this ensure the rest of jsenv do not try to scan or modify the content

package/package.json

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

package/src/build/build.js

@@ -29,6 +29,7 @@ import {
 } from "@jsenv/filesystem";
 import {
   ANSI,
+  createDetailedMessage,
   createDynamicLog,
   createLogger,
   createTaskLog,
@@ -1234,6 +1235,29 @@ const prepareEntryPointBuild = async (
         if (!otherEntryBuildInfo) {
           return null;
         }
+        if (urlInfo.subtypeHint === "service_worker") {
+          // A registered service worker is already an entry of the build
+          // registering it: that build owns its bundle and the resources list
+          // it precaches. A second owner from "entryPoints" would receive
+          // neither, so the configuration is refused.
+          const workerRelativeUrl = urlToRelativeUrl(
+            urlInfo.url,
+            sourceDirectoryUrl,
+          );
+          const registeringRelativeUrl = urlToRelativeUrl(
+            urlInfo.firstReference.ownerUrlInfo.url,
+            sourceDirectoryUrl,
+          );
+          throw new Error(
+            createDetailedMessage(
+              `"${workerRelativeUrl}" is registered as a service worker and declared in "entryPoints"`,
+              {
+                "registered by": registeringRelativeUrl,
+                "suggestion": `remove "./${workerRelativeUrl}" from "entryPoints": a registered service worker is built as its own entry by the build registering it, and its "injections" go under "${sourceRelativeUrl}" keyed by "./${workerRelativeUrl}"`,
+              },
+            ),
+          );
+        }
         urlInfo.otherEntryBuildInfo = otherEntryBuildInfo;
         return {
           type: "entry_build", // this ensure the rest of jsenv do not try to scan or modify the content

package/src/build/build_specifier_manager.js

@@ -626,6 +626,11 @@ export const createBuildSpecifierManager = ({
               // when versioning is dynamic no need to take into account
               continue;
             }
+            if (referenceVersioningInfo.type === "not_versioned") {
+              // the specifier is emitted bare (entry point, webmanifest, ...):
+              // the referencing file's bytes cannot change with that file's content
+              continue;
+            }
             placeholderInfluencingVersionSet.add(containedPlaceholder);
             const referencedUrlInfo = reference.urlInfo;
             visitContainedPlaceholders(referencedUrlInfo);