@jsenv/core 30.3.8 → 30.4.0

package/dist/main.js

@@ -7093,10 +7093,30 @@ const createUrlGraph = () => {
     if (!parentUrlInfo) {
       return null;
     }
-    const firstReferenceOnThatUrl = parentUrlInfo.references.find(reference => {
-      return urlSpecifierEncoding.decode(reference) === specifier;
-    });
-    return firstReferenceOnThatUrl;
+    const seen = [];
+    const search = urlInfo => {
+      const firstReferenceFound = urlInfo.references.find(reference => {
+        return urlSpecifierEncoding.decode(reference) === specifier;
+      });
+      if (firstReferenceFound) {
+        return firstReferenceFound;
+      }
+      for (const dependencyUrl of parentUrlInfo.dependencies) {
+        if (seen.includes(dependencyUrl)) {
+          continue;
+        }
+        seen.push(dependencyUrl);
+        const dependencyUrlInfo = getUrlInfo(dependencyUrl);
+        if (dependencyUrlInfo.isInline) {
+          const firstRef = search(dependencyUrlInfo);
+          if (firstRef) {
+            return firstRef;
+          }
+        }
+      }
+      return null;
+    };
+    return search(parentUrlInfo);
   };
   const findDependent = (urlInfo, visitor) => {
     const seen = [urlInfo.url];
@@ -8852,7 +8872,7 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         type,
         subtype,
         originalUrl,
-        originalContent,
+        originalContent = content,
         sourcemap,
         filename,
         status = 200,
@@ -8875,7 +8895,15 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
       urlInfo.subtype = subtype || reference.expectedSubtype || "";
       // during build urls info are reused and load returns originalUrl/originalContent
       urlInfo.originalUrl = originalUrl || urlInfo.originalUrl;
-      urlInfo.originalContent = originalContent === undefined ? content : originalContent;
+      if (originalContent !== urlInfo.originalContent) {
+        urlInfo.originalContentEtag = undefined; // set by "initTransformations"
+      }
+
+      if (content !== urlInfo.content) {
+        urlInfo.contentEtag = undefined; // set by "applyFinalTransformations"
+      }
+
+      urlInfo.originalContent = originalContent;
       urlInfo.content = content;
       urlInfo.sourcemap = sourcemap;
       if (data) {
@@ -20017,17 +20045,20 @@ const jsenvPluginAutoreload = ({
   })];
 };
 
-const jsenvPluginCacheControl = () => {
+const jsenvPluginCacheControl = ({
+  versionedUrls = true,
+  maxAge = SECONDS_IN_30_DAYS$1
+}) => {
   return {
     name: "jsenv:cache_control",
     appliesDuring: "dev",
     augmentResponse: ({
       reference
     }) => {
-      if (reference.searchParams.has("v") && !reference.searchParams.has("hmr")) {
+      if (versionedUrls && reference.searchParams.has("v") && !reference.searchParams.has("hmr")) {
         return {
           headers: {
-            "cache-control": `private,max-age=${SECONDS_IN_30_DAYS$1},immutable`
+            "cache-control": `private,max-age=${maxAge},immutable`
           }
         };
       }
@@ -20169,11 +20200,15 @@ const getCorePlugins = ({
   clientFileChangeCallbackList,
   clientFilesPruneCallbackList,
   explorer,
+  cacheControl,
   ribbon = true
 } = {}) => {
   if (explorer === true) {
     explorer = {};
   }
+  if (cacheControl === true) {
+    cacheControl = {};
+  }
   if (supervisor === true) {
     supervisor = {};
   }
@@ -20214,7 +20249,7 @@ const getCorePlugins = ({
     ...clientAutoreload,
     clientFileChangeCallbackList,
     clientFilesPruneCallbackList
-  })] : []), jsenvPluginCacheControl(), ...(explorer ? [jsenvPluginExplorer({
+  })] : []), ...(cacheControl ? [jsenvPluginCacheControl(cacheControl)] : []), ...(explorer ? [jsenvPluginExplorer({
     ...explorer,
     clientMainFileUrl
   })] : []), ...(ribbon ? [jsenvPluginRibbon({
@@ -20223,6 +20258,44 @@ const getCorePlugins = ({
   })] : [])];
 };
 
+const ensureUnixLineBreaks = stringOrBuffer => {
+  if (typeof stringOrBuffer === "string") {
+    const stringWithLinuxBreaks = stringOrBuffer.replace(/\r\n/g, "\n");
+    return stringWithLinuxBreaks;
+  }
+  return ensureUnixLineBreaksOnBuffer(stringOrBuffer);
+};
+
+// https://github.com/nodejs/help/issues/1738#issuecomment-458460503
+const ensureUnixLineBreaksOnBuffer = buffer => {
+  const int32Array = new Int32Array(buffer, 0, buffer.length);
+  const int32ArrayWithLineBreaksNormalized = int32Array.filter((element, index, typedArray) => {
+    if (element === 0x0d) {
+      if (typedArray[index + 1] === 0x0a) {
+        // Windows -> Unix
+        return false;
+      }
+      // Mac OS -> Unix
+      typedArray[index] = 0x0a;
+    }
+    return true;
+  });
+  return Buffer.from(int32ArrayWithLineBreaksNormalized);
+};
+
+const jsenvPluginLineBreakNormalization = () => {
+  return {
+    name: "jsenv:line_break_normalizer",
+    appliesDuring: "build",
+    transformUrlContent: urlInfo => {
+      if (CONTENT_TYPE.isTextual(urlInfo.contentType)) {
+        return ensureUnixLineBreaks(urlInfo.content);
+      }
+      return null;
+    }
+  };
+};
+
 const GRAPH = {
   map: (graph, callback) => {
     const array = [];
@@ -20485,42 +20558,13 @@ const injectVersionMappingsAsImportmap = async ({
   });
 };
 
-const ensureUnixLineBreaks = stringOrBuffer => {
-  if (typeof stringOrBuffer === "string") {
-    const stringWithLinuxBreaks = stringOrBuffer.replace(/\r\n/g, "\n");
-    return stringWithLinuxBreaks;
-  }
-  return ensureUnixLineBreaksOnBuffer(stringOrBuffer);
-};
-
-// https://github.com/nodejs/help/issues/1738#issuecomment-458460503
-const ensureUnixLineBreaksOnBuffer = buffer => {
-  const int32Array = new Int32Array(buffer, 0, buffer.length);
-  const int32ArrayWithLineBreaksNormalized = int32Array.filter((element, index, typedArray) => {
-    if (element === 0x0d) {
-      if (typedArray[index + 1] === 0x0a) {
-        // Windows -> Unix
-        return false;
-      }
-      // Mac OS -> Unix
-      typedArray[index] = 0x0a;
-    }
-    return true;
-  });
-  return Buffer.from(int32ArrayWithLineBreaksNormalized);
-};
-
 // https://github.com/rollup/rollup/blob/19e50af3099c2f627451a45a84e2fa90d20246d5/src/utils/FileEmitter.ts#L47
 // https://github.com/rollup/rollup/blob/5a5391971d695c808eed0c5d7d2c6ccb594fc689/src/Chunk.ts#L870
 const createVersionGenerator = () => {
   const hash = createHash("sha256");
   return {
-    augmentWithContent: ({
-      content,
-      contentType = "application/octet-stream",
-      lineBreakNormalization = false
-    }) => {
-      hash.update(lineBreakNormalization && CONTENT_TYPE.isTextual(contentType) ? ensureUnixLineBreaks(content) : content);
+    augmentWithContent: content => {
+      hash.update(content);
     },
     augment: value => {
       hash.update(value);
@@ -20772,7 +20816,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
       build: true,
       runtimeCompat,
       ...contextSharedDuringBuild,
-      plugins: [urlAnalysisPlugin, jsenvPluginAsJsClassic({
+      plugins: [urlAnalysisPlugin, ...(lineBreakNormalization ? [jsenvPluginLineBreakNormalization()] : []), jsenvPluginAsJsClassic({
         jsClassicLibrary: false,
         jsClassicFallback: true,
         systemJsInjection: true
@@ -21364,11 +21408,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
               cleanupJsenvAttributes: true
             }) : urlInfo.content;
             const contentVersionGenerator = createVersionGenerator();
-            contentVersionGenerator.augmentWithContent({
-              content: urlContent,
-              contentType: urlInfo.contentType,
-              lineBreakNormalization
-            });
+            contentVersionGenerator.augmentWithContent(urlContent);
             const contentVersion = contentVersionGenerator.generate();
             contentVersionMap.set(urlInfo.url, contentVersion);
             const versionMutations = [];
@@ -21387,7 +21427,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   }
                   if (preferWithoutVersioning(reference)) {
                     // when versioning is dynamic no need to take into account
-                    // happend for:
+                    // happens for:
                     // - specifier mapped by window.__v__()
                     // - specifier mapped by importmap
                     return null;
@@ -21747,7 +21787,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
           return finalUrlInfo.subtype === "service_worker" && finalUrlInfo.isEntryPoint;
         });
         if (serviceWorkerEntryUrlInfos.length > 0) {
-          const serviceWorkerUrls = {};
+          const serviceWorkerResources = {};
           GRAPH.forEach(finalGraph, urlInfo => {
             if (urlInfo.isInline || !urlInfo.shouldHandle) {
               return;
@@ -21760,26 +21800,27 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
               // so that service worker source still changes and navigator
               // detect there is a change
               const specifier = findKey(buildUrls, urlInfo.url);
-              serviceWorkerUrls[specifier] = {
-                versioned: false,
+              serviceWorkerResources[specifier] = {
                 version: versionMap.get(urlInfo.url)
               };
               return;
             }
+            const specifier = findKey(buildUrls, urlInfo.url);
             const versionedUrl = versionedUrlMap.get(urlInfo.url);
             const versionedSpecifier = findKey(buildUrls, versionedUrl);
-            serviceWorkerUrls[versionedSpecifier] = {
-              versioned: true
+            serviceWorkerResources[specifier] = {
+              version: versionMap.get(urlInfo.url),
+              versionedUrl: versionedSpecifier
             };
           });
           serviceWorkerEntryUrlInfos.forEach(serviceWorkerEntryUrlInfo => {
             const magicSource = createMagicSource(serviceWorkerEntryUrlInfo.content);
-            const urlsWithoutSelf = {
-              ...serviceWorkerUrls
+            const serviceWorkerResourcesWithoutSwScriptItSelf = {
+              ...serviceWorkerResources
             };
             const serviceWorkerSpecifier = findKey(buildUrls, serviceWorkerEntryUrlInfo.url);
-            delete urlsWithoutSelf[serviceWorkerSpecifier];
-            magicSource.prepend(`\nself.serviceWorkerUrls = ${JSON.stringify(urlsWithoutSelf, null, "  ")};\n`);
+            delete serviceWorkerResourcesWithoutSwScriptItSelf[serviceWorkerSpecifier];
+            magicSource.prepend(`\nself.resourcesFromJsenvBuild = ${JSON.stringify(serviceWorkerResourcesWithoutSwScriptItSelf, null, "  ")};\n`);
             const {
               content,
               sourcemap
@@ -22152,6 +22193,7 @@ const createFileService = ({
   clientMainFileUrl,
   cooldownBetweenFileEvents,
   explorer,
+  cacheControl,
   ribbon,
   sourcemaps,
   sourcemapsSourcesProtocol,
@@ -22259,6 +22301,7 @@ const createFileService = ({
         clientFileChangeCallbackList,
         clientFilesPruneCallbackList,
         explorer,
+        cacheControl,
         ribbon
       })],
       minification: false,
@@ -22288,18 +22331,25 @@ const createFileService = ({
           // when file is modified
           return false;
         }
-        if (!watch) {
+        if (!watch && urlInfo.contentEtag) {
+          // file is not watched, check the filesystem
           let fileContentAsBuffer;
           try {
             fileContentAsBuffer = readFileSync$1(new URL(urlInfo.url));
           } catch (e) {
             if (e.code === "ENOENT") {
+              // we should consider calling urlGraph.deleteUrlInfo(urlInfo)
+              urlInfo.originalContentEtag = undefined;
+              urlInfo.contentEtag = undefined;
               return false;
             }
             return false;
           }
           const fileContentEtag = bufferToEtag$1(fileContentAsBuffer);
           if (fileContentEtag !== urlInfo.originalContentEtag) {
+            // we should consider calling urlGraph.considerModified(urlInfo)
+            urlInfo.originalContentEtag = undefined;
+            urlInfo.contentEtag = undefined;
             return false;
           }
         }
@@ -22617,6 +22667,7 @@ const startDevServer = async ({
   transpilation,
   explorer = true,
   // see jsenv_plugin_explorer.js
+  cacheControl = true,
   ribbon = true,
   // toolbar = false,
 
@@ -22755,6 +22806,7 @@ const startDevServer = async ({
         clientMainFileUrl,
         cooldownBetweenFileEvents,
         explorer,
+        cacheControl,
         ribbon,
         sourcemaps,
         sourcemapsSourcesProtocol,
@@ -23389,6 +23441,17 @@ const getCoverageFromReport = async ({
 };
 const isV8Coverage = coverage => Boolean(coverage.result);
 
+/*
+ * Export a function capable to run a file on a runtime.
+ *
+ * - Used internally by "executeTestPlan" part of the documented API
+ * - Used internally by "execute" an advanced API not documented
+ * - logs generated during file execution can be collected
+ * - logs generated during file execution can be mirrored (re-logged to the console)
+ * - File is given allocatedMs to complete
+ * - Errors are collected
+ * - File execution result is returned, it contains status/errors/namespace/consoleCalls
+ */
 const run = async ({
   signal = new AbortController().signal,
   logger,
@@ -24239,6 +24302,7 @@ const executePlan = async (plan, {
         }
         const afterExecutionInfo = {
           ...beforeExecutionInfo,
+          runtimeVersion: runtime.version,
           endMs: Date.now(),
           executionResult
         };
@@ -24631,8 +24695,8 @@ const createRuntimeFromPlaywright = ({
     stopSignal,
     keepRunning,
     onConsole,
-    executablePath,
     headful = keepRunning,
+    playwrightLaunchOptions = {},
     ignoreHTTPSErrors = true
   }) => {
     const cleanupCallbackList = createCallbackListNotifiedOnce();
@@ -24648,11 +24712,14 @@ const createRuntimeFromPlaywright = ({
           signal,
           browserName,
           stopOnExit: true,
-          playwrightOptions: {
-            headless: !headful,
-            executablePath
+          playwrightLaunchOptions: {
+            ...playwrightLaunchOptions,
+            headless: !headful
           }
         });
+        if (browser._initializer.version) {
+          runtime.version = browser._initializer.version;
+        }
         const browserContext = await browser.newContext({
           ignoreHTTPSErrors
         });
@@ -24969,7 +25036,7 @@ const launchBrowserUsingPlaywright = async ({
   signal,
   browserName,
   stopOnExit,
-  playwrightOptions
+  playwrightLaunchOptions
 }) => {
   const launchBrowserOperation = Abort.startOperation();
   launchBrowserOperation.addAbortSignal(signal);
@@ -24990,7 +25057,7 @@ const launchBrowserUsingPlaywright = async ({
   const browserClass = playwright[browserName];
   try {
     const browser = await browserClass.launch({
-      ...playwrightOptions,
+      ...playwrightLaunchOptions,
       // let's handle them to close properly browser + remove listener
       // instead of relying on playwright to do so
       handleSIGINT: false,
@@ -25081,23 +25148,26 @@ const registerEvent = ({
 
 const chromium = createRuntimeFromPlaywright({
   browserName: "chromium",
-  browserVersion: "110.0.5481.38",
-  // to update, check https://github.com/microsoft/playwright/releases
+  // browserVersion will be set by "browser._initializer.version"
+  // see also https://github.com/microsoft/playwright/releases
+  browserVersion: "unset",
   coveragePlaywrightAPIAvailable: true
 });
 const chromiumIsolatedTab = chromium.isolatedTab;
 
 const firefox = createRuntimeFromPlaywright({
   browserName: "firefox",
-  browserVersion: "108.0.2" // to update, check https://github.com/microsoft/playwright/releases
+  // browserVersion will be set by "browser._initializer.version"
+  // see also https://github.com/microsoft/playwright/releases
+  browserVersion: "unset"
 });
-
 const firefoxIsolatedTab = firefox.isolatedTab;
 
 const webkit = createRuntimeFromPlaywright({
   browserName: "webkit",
-  browserVersion: "16.4",
-  // to update, check https://github.com/microsoft/playwright/releases
+  // browserVersion will be set by "browser._initializer.version"
+  // see also https://github.com/microsoft/playwright/releases
+  browserVersion: "unset",
   ignoreErrorHook: error => {
     // we catch error during execution but safari throw unhandled rejection
     // in a non-deterministic way.
@@ -26171,6 +26241,17 @@ const createBuildFilesService = ({
 };
 const SECONDS_IN_30_DAYS = 60 * 60 * 24 * 30;
 
+/*
+ * Export a function capable to execute a file on a runtime (browser or node) and return how it goes.
+ *
+ * - can be useful to execute a file in a browser/node.js programmatically
+ * - not documented
+ * - the most importants parts:
+ *   - fileRelativeUrl: the file to execute inside rootDirectoryUrl
+ *   - runtime: an object with a "run" method.
+ *   The run method will start a browser/node process and execute file in it
+ * - Most of the logic lives in "./run.js" used by executeTestPlan to run tests
+ */
 const execute = async ({
   signal = new AbortController().signal,
   handleSIGINT = true,
@@ -26211,11 +26292,11 @@ const execute = async ({
   };
   if (runtime.type === "browser") {
     if (!devServerOrigin) {
-      throw new TypeError(`devServerOrigin is required when running tests on browser(s)`);
+      throw new TypeError(`devServerOrigin is required to execute file on a browser`);
     }
     const devServerStarted = await pingServer(devServerOrigin);
     if (!devServerStarted) {
-      throw new Error(`dev server not started at ${devServerOrigin}. It is required to run tests`);
+      throw new Error(`no server listening at ${devServerOrigin}. It is required to execute file`);
     }
   }
   let result = await run({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "30.3.8",
+  "version": "30.4.0",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -44,8 +44,8 @@
     "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "test": "node --conditions=development ./scripts/test/test.mjs",
     "test:resource_hints": "npm run test -- --only-resource-hints",
-    "test:workspace": "npm run test --workspaces --if-present -- --workspace",
     "build": "node --conditions=development ./scripts/build/build.mjs",
+    "workspace:test": "npm run test --workspaces --if-present -- --workspace",
     "workspace:versions": "node ./scripts/publish/workspace_versions.mjs",
     "workspace:publish": "node ./scripts/publish/workspace_publish.mjs",
     "performances": "node --expose-gc ./scripts/performance/generate_performance_report.mjs --log --once",
@@ -107,11 +107,11 @@
     "@jsenv/plugin-globals": "./packages/jsenv-plugin-globals/",
     "@jsenv/plugin-placeholders": "./packages/jsenv-plugin-placeholders/",
     "@jsenv/performance-impact": "4.1.0",
-    "eslint": "8.32.0",
+    "eslint": "8.35.0",
     "eslint-plugin-html": "7.1.0",
     "eslint-plugin-import": "2.27.5",
-    "eslint-plugin-react": "7.32.1",
-    "playwright": "1.30.0",
-    "prettier": "2.8.3"
+    "eslint-plugin-react": "7.32.2",
+    "playwright": "1.31.2",
+    "prettier": "2.8.4"
   }
 }

package/src/build/build.js

@@ -67,6 +67,7 @@ import { jsenvPluginUrlAnalysis } from "../plugins/url_analysis/jsenv_plugin_url
 import { jsenvPluginInline } from "../plugins/inline/jsenv_plugin_inline.js"
 import { jsenvPluginAsJsClassic } from "../plugins/transpilation/as_js_classic/jsenv_plugin_as_js_classic.js"
 import { getCorePlugins } from "../plugins/plugins.js"
+import { jsenvPluginLineBreakNormalization } from "./jsenv_plugin_line_break_normalization.js"
 
 import { GRAPH } from "./graph_utils.js"
 import { createBuildUrlsGenerator } from "./build_urls_generator.js"
@@ -333,6 +334,9 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
       ...contextSharedDuringBuild,
       plugins: [
         urlAnalysisPlugin,
+        ...(lineBreakNormalization
+          ? [jsenvPluginLineBreakNormalization()]
+          : []),
         jsenvPluginAsJsClassic({
           jsClassicLibrary: false,
           jsClassicFallback: true,
@@ -1017,11 +1021,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   )
                 : urlInfo.content
             const contentVersionGenerator = createVersionGenerator()
-            contentVersionGenerator.augmentWithContent({
-              content: urlContent,
-              contentType: urlInfo.contentType,
-              lineBreakNormalization,
-            })
+            contentVersionGenerator.augmentWithContent(urlContent)
             const contentVersion = contentVersionGenerator.generate()
             contentVersionMap.set(urlInfo.url, contentVersion)
             const versionMutations = []
@@ -1042,7 +1042,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
                   }
                   if (preferWithoutVersioning(reference)) {
                     // when versioning is dynamic no need to take into account
-                    // happend for:
+                    // happens for:
                     // - specifier mapped by window.__v__()
                     // - specifier mapped by importmap
                     return null
@@ -1477,7 +1477,7 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
           },
         )
         if (serviceWorkerEntryUrlInfos.length > 0) {
-          const serviceWorkerUrls = {}
+          const serviceWorkerResources = {}
           GRAPH.forEach(finalGraph, (urlInfo) => {
             if (urlInfo.isInline || !urlInfo.shouldHandle) {
               return
@@ -1490,31 +1490,36 @@ ${ANSI.color(buildUrl, ANSI.MAGENTA)}
               // so that service worker source still changes and navigator
               // detect there is a change
               const specifier = findKey(buildUrls, urlInfo.url)
-              serviceWorkerUrls[specifier] = {
-                versioned: false,
+              serviceWorkerResources[specifier] = {
                 version: versionMap.get(urlInfo.url),
               }
               return
             }
+            const specifier = findKey(buildUrls, urlInfo.url)
             const versionedUrl = versionedUrlMap.get(urlInfo.url)
             const versionedSpecifier = findKey(buildUrls, versionedUrl)
-            serviceWorkerUrls[versionedSpecifier] = { versioned: true }
+            serviceWorkerResources[specifier] = {
+              version: versionMap.get(urlInfo.url),
+              versionedUrl: versionedSpecifier,
+            }
           })
           serviceWorkerEntryUrlInfos.forEach((serviceWorkerEntryUrlInfo) => {
             const magicSource = createMagicSource(
               serviceWorkerEntryUrlInfo.content,
             )
-            const urlsWithoutSelf = {
-              ...serviceWorkerUrls,
+            const serviceWorkerResourcesWithoutSwScriptItSelf = {
+              ...serviceWorkerResources,
             }
             const serviceWorkerSpecifier = findKey(
               buildUrls,
               serviceWorkerEntryUrlInfo.url,
             )
-            delete urlsWithoutSelf[serviceWorkerSpecifier]
+            delete serviceWorkerResourcesWithoutSwScriptItSelf[
+              serviceWorkerSpecifier
+            ]
             magicSource.prepend(
-              `\nself.serviceWorkerUrls = ${JSON.stringify(
-                urlsWithoutSelf,
+              `\nself.resourcesFromJsenvBuild = ${JSON.stringify(
+                serviceWorkerResourcesWithoutSwScriptItSelf,
                 null,
                 "  ",
               )};\n`,

package/src/build/jsenv_plugin_line_break_normalization.js

@@ -0,0 +1,16 @@
+import { CONTENT_TYPE } from "@jsenv/utils/src/content_type/content_type.js"
+
+import { ensureUnixLineBreaks } from "./line_break_unix.js"
+
+export const jsenvPluginLineBreakNormalization = () => {
+  return {
+    name: "jsenv:line_break_normalizer",
+    appliesDuring: "build",
+    transformUrlContent: (urlInfo) => {
+      if (CONTENT_TYPE.isTextual(urlInfo.contentType)) {
+        return ensureUnixLineBreaks(urlInfo.content)
+      }
+      return null
+    },
+  }
+}

package/src/build/readme.md

@@ -0,0 +1,3 @@
+# build/
+
+Code implementing jsenv build can be found here.

package/src/build/version_generator.js

@@ -1,24 +1,13 @@
 import { createHash } from "node:crypto"
 
-import { CONTENT_TYPE } from "@jsenv/utils/src/content_type/content_type.js"
-import { ensureUnixLineBreaks } from "./line_break_unix.js"
-
 // https://github.com/rollup/rollup/blob/19e50af3099c2f627451a45a84e2fa90d20246d5/src/utils/FileEmitter.ts#L47
 // https://github.com/rollup/rollup/blob/5a5391971d695c808eed0c5d7d2c6ccb594fc689/src/Chunk.ts#L870
 export const createVersionGenerator = () => {
   const hash = createHash("sha256")
 
   return {
-    augmentWithContent: ({
-      content,
-      contentType = "application/octet-stream",
-      lineBreakNormalization = false,
-    }) => {
-      hash.update(
-        lineBreakNormalization && CONTENT_TYPE.isTextual(contentType)
-          ? ensureUnixLineBreaks(content)
-          : content,
-      )
+    augmentWithContent: (content) => {
+      hash.update(content)
     },
     augment: (value) => {
       hash.update(value)