@jsenv/core 39.3.12 → 39.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "39.3.12",
+  "version": "39.4.1",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -38,6 +38,7 @@
   },
   "workspaces": [
     "./packages/independent/*",
+    "./packages/independent/workflow/*",
     "./packages/internal/*",
     "./packages/related/*",
     "./packages/related/cli/*"
@@ -52,7 +53,7 @@
     "monorepo:publish": "node ./scripts/monorepo/publish_packages.mjs",
     "monorepo:upgrade_versions": "node ./scripts/monorepo/upgrade_external_versions.mjs",
     "monorepo:node_modules_clear": "npx @jsenv/filesystem clear **/node_modules/",
-    "md:build": "node ./md/build.js",
+    "md:build": "node ./docs/build.js",
     "performances": "node --expose-gc ./scripts/performance/generate_performance_report.mjs --log --once",
     "build:file_size": "node ./scripts/build/build_file_size.mjs --log",
     "prettier": "prettier --write .",
@@ -62,52 +63,52 @@
     "workspace:test:ci": "CI=1 npm run workspace:test",
     "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "playwright:install": "npx playwright install-deps && npx playwright install",
-    "certificate:install": "node ./scripts/dev/install_certificate_authority.mjs",
+    "https:setup": "npx @jsenv/https-local setup",
     "prepublishOnly": "npm run build"
   },
   "dependencies": {
     "@financial-times/polyfill-useragent-normaliser": "1.10.2",
     "@jsenv/abort": "4.3.0",
-    "@jsenv/ast": "6.2.16",
+    "@jsenv/ast": "6.2.17",
     "@jsenv/filesystem": "4.10.2",
     "@jsenv/humanize": "1.2.8",
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.2",
-    "@jsenv/js-module-fallback": "1.3.37",
+    "@jsenv/js-module-fallback": "1.3.38",
     "@jsenv/node-esm-resolution": "1.0.6",
     "@jsenv/plugin-bundling": "2.7.7",
     "@jsenv/plugin-minification": "1.5.5",
-    "@jsenv/plugin-supervisor": "1.5.18",
-    "@jsenv/plugin-transpilation": "1.4.21",
+    "@jsenv/plugin-supervisor": "1.5.19",
+    "@jsenv/plugin-transpilation": "1.4.22",
     "@jsenv/runtime-compat": "1.3.1",
     "@jsenv/server": "15.3.0",
     "@jsenv/sourcemap": "1.2.23",
     "@jsenv/url-meta": "8.5.1",
     "@jsenv/urls": "2.5.2",
     "@jsenv/utils": "2.1.2",
+    "string-width": "7.2.0",
     "anchor-markdown-header": "0.7.0"
   },
   "devDependencies": {
-    "@babel/eslint-parser": "7.25.1",
     "@babel/plugin-syntax-import-attributes": "7.24.7",
     "@babel/plugin-syntax-optional-chaining-assign": "7.24.7",
     "@eslint/compat": "1.1.1",
-    "@jsenv/assert": "./packages/independent/assert/",
-    "@jsenv/cli": "./packages/related/cli/",
+    "@jsenv/assert": "workspace:*",
+    "@jsenv/cli": "workspace:*",
     "@jsenv/core": "./",
-    "@jsenv/eslint-config": "./packages/independent/eslint-config/",
-    "@jsenv/file-size-impact": "14.2.0",
-    "@jsenv/https-local": "3.0.7",
-    "@jsenv/monorepo": "0.0.6",
-    "@jsenv/performance-impact": "4.3.0",
-    "@jsenv/plugin-as-js-classic": "./packages/related/plugin-as-js-classic/",
-    "@jsenv/snapshot": "./packages/independent/snapshot/",
-    "@jsenv/test": "./packages/related/test/",
+    "@jsenv/eslint-config-relax": "workspace:*",
+    "@jsenv/file-size-impact": "workspace:*",
+    "@jsenv/https-local": "workspace:*",
+    "@jsenv/monorepo": "workspace:*",
+    "@jsenv/performance-impact": "workspace:*",
+    "@jsenv/plugin-as-js-classic": "workspace:*",
+    "@jsenv/snapshot": "workspace:*",
+    "@jsenv/test": "workspace:*",
     "@playwright/browser-chromium": "1.46.0",
     "@playwright/browser-firefox": "1.46.0",
     "@playwright/browser-webkit": "1.46.0",
     "babel-plugin-transform-async-to-promises": "0.8.18",
-    "eslint": "9.8.0",
+    "eslint": "9.9.0",
     "marked": "14.0.0",
     "open": "10.1.0",
     "playwright": "1.46.0",

package/src/build/build.js

@@ -93,6 +93,10 @@ export const build = async ({
   signal = new AbortController().signal,
   handleSIGINT = true,
   logLevel = "info",
+  logs = {
+    disabled: false,
+    animation: true,
+  },
   sourceDirectoryUrl,
   buildDirectoryUrl,
   entryPoints = {},
@@ -121,6 +125,7 @@ export const build = async ({
   sourceFilesConfig = {},
   cooldownBetweenFileEvents,
   watch = false,
+  http = false,
 
   directoryToClean,
   sourcemaps = "none",
@@ -231,8 +236,9 @@ export const build = async ({
     const logger = createLogger({ logLevel });
     const createBuildTask = (label) => {
       return createTaskLog(label, {
-        disabled: !logger.levels.debug && !logger.levels.info,
-        animated: !logger.levels.debug,
+        disabled:
+          logs.disabled || (!logger.levels.debug && !logger.levels.info),
+        animated: logs.animation && !logger.levels.debug,
       });
     };
 
@@ -307,6 +313,7 @@ build ${entryPointKeys.length} entry points`);
             jsModuleFallback: false,
           },
           inlining: false,
+          http,
           scenarioPlaceholders,
         }),
       ],

package/src/build/build_specifier_manager.js

@@ -187,6 +187,42 @@ export const createBuildSpecifierManager = ({
       const url = new URL(reference.specifier, parentUrl).href;
       return url;
     },
+    redirectReference: (reference) => {
+      let referenceBeforeInlining = reference;
+      if (
+        referenceBeforeInlining.isInline &&
+        referenceBeforeInlining.prev &&
+        !referenceBeforeInlining.prev.isInline
+      ) {
+        referenceBeforeInlining = referenceBeforeInlining.prev;
+      }
+      const rawUrl = referenceBeforeInlining.url;
+      const rawUrlInfo = rawKitchen.graph.getUrlInfo(rawUrl);
+      if (rawUrlInfo) {
+        reference.filenameHint = rawUrlInfo.filenameHint;
+        return null;
+      }
+      if (referenceBeforeInlining.injected) {
+        return null;
+      }
+      if (
+        referenceBeforeInlining.isInline &&
+        referenceBeforeInlining.ownerUrlInfo.url ===
+          referenceBeforeInlining.ownerUrlInfo.originalUrl
+      ) {
+        const rawUrlInfo = findRawUrlInfoWhenInline(
+          referenceBeforeInlining,
+          rawKitchen,
+        );
+        if (rawUrlInfo) {
+          reference.rawUrl = rawUrlInfo.url;
+          reference.filenameHint = rawUrlInfo.filenameHint;
+          return null;
+        }
+      }
+      reference.filenameHint = referenceBeforeInlining.filenameHint;
+      return null;
+    },
     transformReferenceSearchParams: () => {
       // those search params are reflected into the build file name
       // moreover it create cleaner output
@@ -231,13 +267,10 @@ export const createBuildSpecifierManager = ({
       ) {
         firstReference = firstReference.prev;
       }
-      const rawUrl = firstReference.url;
+      const rawUrl = firstReference.rawUrl || firstReference.url;
       const rawUrlInfo = rawKitchen.graph.getUrlInfo(rawUrl);
       const bundleInfo = bundleInfoMap.get(rawUrl);
       if (bundleInfo) {
-        if (rawUrlInfo && !finalUrlInfo.filenameHint) {
-          finalUrlInfo.filenameHint = rawUrlInfo.filenameHint;
-        }
         finalUrlInfo.remapReference = bundleInfo.remapReference;
         return {
           // url: bundleInfo.url,
@@ -250,9 +283,6 @@ export const createBuildSpecifierManager = ({
         };
       }
       if (rawUrlInfo) {
-        if (rawUrlInfo && !finalUrlInfo.filenameHint) {
-          finalUrlInfo.filenameHint = rawUrlInfo.filenameHint;
-        }
         return rawUrlInfo;
       }
       // reference injected during "shape":
@@ -274,9 +304,6 @@ export const createBuildSpecifierManager = ({
           content: reference.content,
           contentType: reference.contentType,
         });
-        if (!finalUrlInfo.filenameHint) {
-          finalUrlInfo.filenameHint = reference.filenameHint;
-        }
         const rawUrlInfo = rawReference.urlInfo;
         await rawUrlInfo.cook();
         return {
@@ -293,48 +320,17 @@ export const createBuildSpecifierManager = ({
           firstReference.ownerUrlInfo.url ===
           firstReference.ownerUrlInfo.originalUrl
         ) {
-          const rawUrlInfo = GRAPH_VISITOR.find(
-            rawKitchen.graph,
-            (rawUrlInfoCandidate) => {
-              const { inlineUrlSite } = rawUrlInfoCandidate;
-              if (!inlineUrlSite) {
-                return false;
-              }
-              if (
-                inlineUrlSite.url === firstReference.ownerUrlInfo.url &&
-                inlineUrlSite.line === firstReference.specifierLine &&
-                inlineUrlSite.column === firstReference.specifierColumn
-              ) {
-                return true;
-              }
-              if (rawUrlInfoCandidate.content === firstReference.content) {
-                return true;
-              }
-              if (
-                rawUrlInfoCandidate.originalContent === firstReference.content
-              ) {
-                return true;
-              }
-              return false;
-            },
-          );
           if (rawUrlInfo) {
-            if (!finalUrlInfo.filenameHint) {
-              finalUrlInfo.filenameHint = rawUrlInfo.filenameHint;
-            }
             return rawUrlInfo;
           }
         }
-        if (!finalUrlInfo.filenameHint) {
-          finalUrlInfo.filenameHint = firstReference.filenameHint;
-        }
         return {
           originalContent: finalUrlInfo.originalContent,
           content: firstReference.content,
           contentType: firstReference.contentType,
         };
       }
-      throw new Error(createDetailedMessage(`Cannot fetch ${rawUrl}`));
+      throw new Error(createDetailedMessage(`${rawUrl} not found in graph`));
     },
   };
 
@@ -764,10 +760,15 @@ export const createBuildSpecifierManager = ({
           if (urlInfo.isEntryPoint) {
             generateReplacement(urlInfo.firstReference);
           }
-          if (urlInfo.isInline) {
-            generateReplacement(urlInfo.firstReference);
-          }
           if (urlInfo.type === "sourcemap") {
+            const { referenceFromOthersSet } = urlInfo;
+            let lastRef;
+            for (const ref of referenceFromOthersSet) {
+              lastRef = ref;
+            }
+            generateReplacement(lastRef);
+          }
+          if (urlInfo.isInline) {
             generateReplacement(urlInfo.firstReference);
           }
           if (urlInfo.firstReference.type === "side_effect_file") {
@@ -1004,10 +1005,10 @@ export const createBuildSpecifierManager = ({
       GRAPH_VISITOR.forEachUrlInfoStronglyReferenced(
         finalKitchen.graph.rootUrlInfo,
         (urlInfo) => {
-          if (!urlInfo.url.startsWith("file:")) {
+          const buildUrl = urlInfoToBuildUrlMap.get(urlInfo);
+          if (!buildUrl) {
             return;
           }
-          const buildUrl = urlInfoToBuildUrlMap.get(urlInfo);
           const buildSpecifier = buildUrlToBuildSpecifierMap.get(buildUrl);
           const buildSpecifierVersioned = versioning
             ? buildSpecifierToBuildSpecifierVersionedMap.get(buildSpecifier)
@@ -1059,6 +1060,33 @@ export const createBuildSpecifierManager = ({
   };
 };
 
+const findRawUrlInfoWhenInline = (reference, rawKitchen) => {
+  const rawUrlInfo = GRAPH_VISITOR.find(
+    rawKitchen.graph,
+    (rawUrlInfoCandidate) => {
+      const { inlineUrlSite } = rawUrlInfoCandidate;
+      if (!inlineUrlSite) {
+        return false;
+      }
+      if (
+        inlineUrlSite.url === reference.ownerUrlInfo.url &&
+        inlineUrlSite.line === reference.specifierLine &&
+        inlineUrlSite.column === reference.specifierColumn
+      ) {
+        return true;
+      }
+      if (rawUrlInfoCandidate.content === reference.content) {
+        return true;
+      }
+      if (rawUrlInfoCandidate.originalContent === reference.content) {
+        return true;
+      }
+      return false;
+    },
+  );
+  return rawUrlInfo;
+};
+
 // see https://github.com/rollup/rollup/blob/ce453507ab8457dd1ea3909d8dd7b117b2d14fab/src/utils/hashPlaceholders.ts#L1
 // see also "New hashing algorithm that "fixes (nearly) everything"
 // at https://github.com/rollup/rollup/pull/4543

package/src/kitchen/kitchen.js

@@ -1,13 +1,7 @@
-import { ensureWindowsDriveLetter } from "@jsenv/filesystem";
 import { ANSI, createDetailedMessage, createLogger } from "@jsenv/humanize";
 import { RUNTIME_COMPAT } from "@jsenv/runtime-compat";
 import { URL_META } from "@jsenv/url-meta";
-import {
-  moveUrl,
-  normalizeUrl,
-  setUrlFilename,
-  urlIsInsideOf,
-} from "@jsenv/urls";
+import { normalizeUrl } from "@jsenv/urls";
 import { CONTENT_TYPE } from "@jsenv/utils/src/content_type/content_type.js";
 import { jsenvPluginHtmlSyntaxErrorFallback } from "../plugins/html_syntax_error_fallback/jsenv_plugin_html_syntax_error_fallback.js";
 import { createPluginController } from "../plugins/plugin_controller.js";
@@ -19,6 +13,10 @@ import {
   defineNonEnumerableProperties,
 } from "./errors.js";
 import { assertFetchedContentCompliance } from "./fetched_content_compliance.js";
+import {
+  determineFileUrlForOutDirectory,
+  determineSourcemapFileUrl,
+} from "./out_directory_url.js";
 import { createUrlGraph } from "./url_graph/url_graph.js";
 import { createUrlInfoTransformer } from "./url_graph/url_info_transformations.js";
 import { urlSpecifierEncoding } from "./url_graph/url_specifier_encoding.js";
@@ -153,7 +151,11 @@ export const createKitchen = ({
         reference.type !== "js_import"
       ) {
         referenceUrl = `ignore:${referenceUrl}`;
-      } else if (isIgnored(referenceUrl)) {
+      } else if (
+        reference.url && reference.original
+          ? isIgnored(reference.original.url)
+          : isIgnored(referenceUrl)
+      ) {
         referenceUrl = `ignore:${referenceUrl}`;
       }
 
@@ -245,13 +247,16 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
   kitchenContext.resolveReference = resolveReference;
 
   const finalizeReference = (reference) => {
+    const urlInfo = reference.urlInfo;
+    urlInfo.generatedUrl = determineFileUrlForOutDirectory(urlInfo);
+    urlInfo.sourcemapGeneratedUrl = determineSourcemapFileUrl(urlInfo);
+
     if (reference.isImplicit && reference.isWeak) {
       // not needed for implicit references that are not rendered anywhere
       // this condition excludes:
       // - side_effect_file references injected in entry points or at the top of files
       return;
     }
-
     transform_search_params: {
       // This hook must touch reference.generatedUrl, NOT reference.url
       // And this is because this hook inject query params used to:
@@ -358,7 +363,9 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         urlInfo.subtypeHint ||
         "";
       // during build urls info are reused and load returns originalUrl/originalContent
-      urlInfo.originalUrl = originalUrl || urlInfo.originalUrl;
+      urlInfo.originalUrl = originalUrl
+        ? String(originalUrl)
+        : urlInfo.originalUrl;
       if (data) {
         Object.assign(urlInfo.data, data);
       }
@@ -369,7 +376,6 @@ ${ANSI.color(normalizedReturnValue, ANSI.YELLOW)}
         urlInfo,
         content,
       });
-      urlInfo.generatedUrl = determineFileUrlForOutDirectory(urlInfo);
 
       // we wait here to read .contentAst and .originalContentAst
       // so that we don't trigger lazy getters
@@ -705,27 +711,3 @@ const inferUrlInfoType = (urlInfo) => {
   }
   return "other";
 };
-
-const determineFileUrlForOutDirectory = (urlInfo) => {
-  if (!urlInfo.context.outDirectoryUrl) {
-    return urlInfo.url;
-  }
-  if (!urlInfo.url.startsWith("file:")) {
-    return urlInfo.url;
-  }
-  let url = urlInfo.url;
-  if (!urlIsInsideOf(urlInfo.url, urlInfo.context.rootDirectoryUrl)) {
-    const fsRootUrl = ensureWindowsDriveLetter("file:///", urlInfo.url);
-    url = `${urlInfo.context.rootDirectoryUrl}@fs/${url.slice(
-      fsRootUrl.length,
-    )}`;
-  }
-  if (urlInfo.filenameHint) {
-    url = setUrlFilename(url, urlInfo.filenameHint);
-  }
-  return moveUrl({
-    url,
-    from: urlInfo.context.rootDirectoryUrl,
-    to: urlInfo.context.outDirectoryUrl,
-  });
-};

package/src/kitchen/out_directory_url.js

@@ -0,0 +1,45 @@
+import { ensureWindowsDriveLetter } from "@jsenv/filesystem";
+import { generateSourcemapFileUrl } from "@jsenv/sourcemap";
+import { moveUrl, setUrlFilename, urlIsInsideOf } from "@jsenv/urls";
+
+export const determineFileUrlForOutDirectory = (urlInfo) => {
+  let { url, filenameHint } = urlInfo;
+  const { rootDirectoryUrl, outDirectoryUrl } = urlInfo.context;
+  if (!outDirectoryUrl) {
+    return url;
+  }
+  if (!url.startsWith("file:")) {
+    return url;
+  }
+  if (!urlIsInsideOf(url, rootDirectoryUrl)) {
+    const fsRootUrl = ensureWindowsDriveLetter("file:///", url);
+    url = `${rootDirectoryUrl}@fs/${url.slice(fsRootUrl.length)}`;
+  }
+  if (filenameHint) {
+    url = setUrlFilename(url, filenameHint);
+  }
+  return moveUrl({
+    url,
+    from: rootDirectoryUrl,
+    to: outDirectoryUrl,
+  });
+};
+
+export const determineSourcemapFileUrl = (urlInfo) => {
+  // sourcemap is a special kind of reference:
+  // It's a reference to a content generated dynamically the content itself.
+  // when jsenv is done cooking the file
+  //   during build it's urlInfo.url to be inside the build
+  //   but otherwise it's generatedUrl to be inside .jsenv/ directory
+  const generatedUrlObject = new URL(urlInfo.generatedUrl);
+  generatedUrlObject.searchParams.delete("js_module_fallback");
+  generatedUrlObject.searchParams.delete("as_js_module");
+  generatedUrlObject.searchParams.delete("as_js_classic");
+  generatedUrlObject.searchParams.delete("as_css_module");
+  generatedUrlObject.searchParams.delete("as_json_module");
+  generatedUrlObject.searchParams.delete("as_text_module");
+  generatedUrlObject.searchParams.delete("dynamic_import");
+  generatedUrlObject.searchParams.delete("cjs_as_js_module");
+  const urlForSourcemap = generatedUrlObject.href;
+  return generateSourcemapFileUrl(urlForSourcemap);
+};

package/src/kitchen/url_graph/references.js

@@ -310,7 +310,9 @@ const createReference = ({
     if (specifier instanceof URL) {
       specifier = specifier.href;
     } else {
-      throw new TypeError(`"specifier" must be a string, got ${specifier}`);
+      throw new TypeError(
+        `"specifier" must be a string, got ${specifier} in ${ownerUrlInfo.url}`,
+      );
     }
   }
   const reference = {
@@ -715,7 +717,7 @@ const applyReferenceEffectsOnUrlInfo = (reference) => {
   }
   referencedUrlInfo.firstReference = reference;
   referencedUrlInfo.originalUrl =
-    referencedUrlInfo.originalUrl || reference.url;
+    referencedUrlInfo.originalUrl || (reference.original || reference).url;
 
   if (reference.isEntryPoint || isWebWorkerEntryPointReference(reference)) {
     referencedUrlInfo.isEntryPoint = true;

package/src/kitchen/url_graph/url_info_transformations.js

@@ -2,7 +2,6 @@ import { writeFileSync } from "@jsenv/filesystem";
 import {
   composeTwoSourcemaps,
   generateSourcemapDataUrl,
-  generateSourcemapFileUrl,
   SOURCEMAP,
 } from "@jsenv/sourcemap";
 import { isFileSystemPath, urlToRelativeUrl } from "@jsenv/urls";
@@ -136,22 +135,6 @@ export const createUrlInfoTransformer = ({
     if (!may || !shouldHandle) {
       return;
     }
-    // sourcemap is a special kind of reference:
-    // It's a reference to a content generated dynamically the content itself.
-    // when jsenv is done cooking the file
-    //   during build it's urlInfo.url to be inside the build
-    //   but otherwise it's generatedUrl to be inside .jsenv/ directory
-    const generatedUrlObject = new URL(urlInfo.generatedUrl);
-    generatedUrlObject.searchParams.delete("js_module_fallback");
-    generatedUrlObject.searchParams.delete("as_js_module");
-    generatedUrlObject.searchParams.delete("as_js_classic");
-    generatedUrlObject.searchParams.delete("as_css_module");
-    generatedUrlObject.searchParams.delete("as_json_module");
-    generatedUrlObject.searchParams.delete("as_text_module");
-    generatedUrlObject.searchParams.delete("dynamic_import");
-    generatedUrlObject.searchParams.delete("cjs_as_js_module");
-    const urlForSourcemap = generatedUrlObject.href;
-    urlInfo.sourcemapGeneratedUrl = generateSourcemapFileUrl(urlForSourcemap);
 
     // case #1: already loaded during "load" hook
     // - happens during build

package/src/plugins/plugins.js

@@ -37,6 +37,7 @@ export const getCorePlugins = ({
   injections,
   transpilation = true,
   inlining = true,
+  http = false,
 
   clientAutoreload,
   cacheControl,
@@ -52,6 +53,12 @@ export const getCorePlugins = ({
   if (ribbon === true) {
     ribbon = {};
   }
+  if (http === true) {
+    http = { include: true };
+  }
+  if (http === false) {
+    http = { include: false };
+  }
 
   return [
     jsenvPluginReferenceAnalysis(referenceAnalysis),
@@ -66,11 +73,12 @@ export const getCorePlugins = ({
        - reference inside a js module -> resolved by node esm
        - All the rest uses web standard url resolution
      */
+    jsenvPluginProtocolHttp(http),
     jsenvPluginProtocolFile({
       magicExtensions,
       magicDirectoryIndex,
     }),
-    jsenvPluginProtocolHttp(),
+
     ...(nodeEsmResolution
       ? [jsenvPluginNodeEsmResolution(nodeEsmResolution)]
       : []),

package/src/plugins/protocol_file/jsenv_plugin_fs_redirection.js

@@ -74,10 +74,18 @@ export const jsenvPluginFsRedirection = ({
       if (!stat) {
         return null;
       }
-      const urlRaw = preserveSymlinks
-        ? urlObject.href
-        : resolveSymlink(urlObject.href);
-      const resolvedUrl = `${urlRaw}${search}${hash}`;
+      const urlBeforeSymlinkResolution = urlObject.href;
+      if (preserveSymlinks) {
+        return `${urlBeforeSymlinkResolution}${search}${hash}`;
+      }
+      const urlAfterSymlinkResolution = resolveSymlink(
+        urlBeforeSymlinkResolution,
+      );
+      if (urlAfterSymlinkResolution !== urlBeforeSymlinkResolution) {
+        reference.leadsToASymlink = true;
+        // reference.baseUrl = urlBeforeSymlinkResolution;
+      }
+      const resolvedUrl = `${urlAfterSymlinkResolution}${search}${hash}`;
       return resolvedUrl;
     },
   };

package/src/plugins/protocol_file/jsenv_plugin_protocol_file.js

@@ -61,17 +61,17 @@ export const jsenvPluginProtocolFile = ({
         return null;
       },
       formatReference: (reference) => {
-        if (!reference.generatedUrl.startsWith("file:")) {
+        const { generatedUrl } = reference;
+        if (!generatedUrl.startsWith("file:")) {
           return null;
         }
         const { rootDirectoryUrl } = reference.ownerUrlInfo.context;
-        if (urlIsInsideOf(reference.generatedUrl, rootDirectoryUrl)) {
-          return `/${urlToRelativeUrl(
-            reference.generatedUrl,
-            rootDirectoryUrl,
-          )}`;
+        if (urlIsInsideOf(generatedUrl, rootDirectoryUrl)) {
+          const result = `/${urlToRelativeUrl(generatedUrl, rootDirectoryUrl)}`;
+          return result;
         }
-        return `/@fs/${reference.generatedUrl.slice("file:///".length)}`;
+        const result = `/@fs/${generatedUrl.slice("file:///".length)}`;
+        return result;
       },
     },
     {