@jsenv/core 27.0.0-alpha.95 → 27.0.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jsenv/core",
-  "version": "27.0.0-alpha.95",
+  "version": "27.0.2",
   "description": "Tool to develop, test and build js projects",
   "license": "MIT",
   "author": {
@@ -43,17 +43,17 @@
     "eslint": "npx eslint . --ext=.js,.mjs,.cjs,.html",
     "dev": "node --conditions=development ./scripts/dev/dev.mjs",
     "test": "node --conditions=development ./scripts/test/test.mjs",
-    "test-packages": "npm run test --workspaces --if-present -- --workspace",
+    "test:coverage": "npm run test -- --coverage",
     "build": "node --conditions=development ./scripts/build/build.mjs",
-    "start_file_server": "node ./scripts/dev/start_file_server.mjs",
-    "workspace-versions": "node ./scripts/publish/workspace_versions.mjs",
-    "workspace-publish": "node ./scripts/publish/workspace_publish.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",
     "file-size": "node ./scripts/file_size/file_size.mjs --log",
+    "start_file_server": "node ./scripts/dev/start_file_server.mjs",
     "prettier": "prettier --write .",
     "playwright-install": "npx playwright install-deps && npx playwright install",
     "certificate-install": "node ./scripts/dev/install_certificate_authority.mjs",
-    "test-with-coverage": "npm run test -- --coverage",
     "prepublishOnly": "npm run build"
   },
   "optionalDependencies": {
@@ -61,7 +61,7 @@
   },
   "dependencies": {
     "@babel/plugin-proposal-dynamic-import": "7.16.7",
-    "@babel/plugin-transform-modules-systemjs": "7.18.5",
+    "@babel/plugin-transform-modules-systemjs": "7.18.6",
     "@babel/plugin-transform-modules-umd": "7.18.0",
     "@c88/v8-coverage": "0.1.1",
     "@financial-times/polyfill-useragent-normaliser": "2.0.1",
@@ -71,13 +71,13 @@
     "@jsenv/importmap": "1.2.1",
     "@jsenv/integrity": "0.0.1",
     "@jsenv/node-esm-resolution": "0.1.0",
-    "@jsenv/server": "12.7.3",
+    "@jsenv/server": "12.7.4",
     "@jsenv/uneval": "1.6.0",
     "@jsenv/url-meta": "7.0.0",
     "@jsenv/urls": "1.2.6",
     "@jsenv/utils": "2.0.1",
     "@jsenv/babel-plugins": "1.0.5",
-    "@jsenv/log": "2.1.0",
+    "@jsenv/log": "3.0.2",
     "@jsenv/sourcemap": "1.0.1",
     "acorn-import-assertions": "1.8.0",
     "cuid": "2.1.8",
@@ -99,15 +99,15 @@
     "@babel/plugin-syntax-import-assertions": "7.17.12",
     "@jsenv/assert": "2.6.0",
     "@jsenv/eslint-config": "16.0.9",
-    "@jsenv/file-size-impact": "13.0.0",
+    "@jsenv/file-size-impact": "13.0.1",
     "@jsenv/https-local": "2.1.0",
-    "@jsenv/package-workspace": "0.3.0",
-    "@jsenv/performance-impact": "3.0.0",
-    "eslint": "8.18.0",
+    "@jsenv/package-workspace": "0.4.1",
+    "@jsenv/performance-impact": "3.0.1",
+    "eslint": "8.19.0",
     "eslint-plugin-html": "6.2.0",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-react": "7.30.1",
-    "playwright": "1.22.2",
+    "playwright": "1.23.1",
     "prettier": "2.7.1"
   }
 }

package/src/build/build.js

@@ -988,7 +988,7 @@ const applyUrlVersioning = async ({
       // - ignored files:
       //   we don't know their content
       // - unused files without reference
-      //   File updated such as style.css -> style.css.js or file.js->file.es5.js
+      //   File updated such as style.css -> style.css.js or file.js->file.nomodule.js
       //   Are used at some point just to be discarded later because they need to be converted
       //   There is no need to version them and we could not because the file have been ignored
       //   so their content is unknown

package/src/build/start_build_server.js

@@ -69,7 +69,9 @@ export const startBuildServer = async ({
         `buildIndexPath must be a string, got ${buildIndexPath}`,
       )
     }
-    if (buildIndexPath[0] !== "/") {
+    if (buildIndexPath[0] === "/") {
+      buildIndexPath = buildIndexPath.slice(1)
+    } else {
       const buildIndexUrl = new URL(buildIndexPath, buildDirectoryUrl).href
       if (!buildIndexUrl.startsWith(buildDirectoryUrl)) {
         throw new Error(

package/src/dev/plugins/toolbar/client/jsenv_logo.svg

@@ -4,53 +4,51 @@
   viewBox="0 0 430 430"
   enable-background="new 0 0 430 430"
 >
-  <g>
-    <g>
-      <path
-        id="j-letter"
-        fill="#F7931E"
-        d="M78.879,272.229c3.381-0.441,7.056-1.176,9.408-3.822c2.793-2.939,3.234-8.086,3.234-19.111v-63.064
+  <g id="letters">
+    <path
+      id="j"
+      fill="#F7931E"
+      d="M78.879,272.229c3.381-0.441,7.056-1.176,9.408-3.822c2.793-2.939,3.234-8.086,3.234-19.111v-63.064
           h19.404v66.74c0,16.023-3.381,23.52-9.555,28.371c-5.292,4.264-14.112,6.174-21.021,6.174L78.879,272.229z M111.072,167.121
           c0,6.174-4.41,10.438-10.143,10.438c-5.292,0-9.702-4.263-9.555-10.438c-0.147-6.468,4.41-10.437,9.849-10.437
           S111.072,160.653,111.072,167.121z"
-      />
-      <path
-        id="s-letter"
-        fill="#F7931E"
-        d="M123.714,240.77c2.793,1.764,8.085,3.822,12.936,3.822c4.998,0,7.791-2.205,7.791-6.469
+    />
+    <path
+      id="s"
+      fill="#F7931E"
+      d="M123.714,240.77c2.793,1.764,8.085,3.822,12.936,3.822c4.998,0,7.791-2.205,7.791-6.469
           c0-3.675-2.058-6.027-8.379-9.702c-11.76-6.615-14.553-14.112-14.553-20.727c0-12.937,9.261-22.639,23.667-22.639
           c6.027,0,10.731,1.47,14.553,3.234l-3.381,14.112c-2.793-1.617-6.174-2.94-9.996-2.94c-4.851,0-7.203,2.94-7.203,6.174
           c0,3.234,1.617,4.998,8.379,8.967c10.584,6.027,14.553,13.524,14.553,21.756c0,14.699-10.584,22.639-25.432,22.639
           c-6.027,0-13.083-1.617-16.464-4.117L123.714,240.77z"
-      />
-      <path
-        id="e-letter"
-        fill="#F7931E"
-        d="M204.414,223.57c0.148,21.756,8.674,26.461,16.906,26.461c4.852,0,8.967-1.176,11.76-2.793l1.764,8.379
+    />
+    <path
+      id="e"
+      fill="#F7931E"
+      d="M204.414,223.57c0.148,21.756,8.674,26.461,16.906,26.461c4.852,0,8.967-1.176,11.76-2.793l1.764,8.379
           c-3.969,2.205-9.996,3.381-15.582,3.381c-17.052,0-25.872-13.965-25.872-35.722c0-23.08,9.702-37.486,23.961-37.486
           c14.553,0,20.285,15.289,20.285,31.459c0,2.646,0,4.557-0.146,6.321H204.414z M226.76,215.191
           c0.293-14.259-5.145-20.581-10.584-20.581c-7.352,0-11.174,10.878-11.613,20.581H226.76z"
-      />
-      <path
-        id="n-letter"
-        fill="#F7931E"
-        d="M249.836,205.342c0-8.379-0.441-12.643-0.588-18.375h9.85l0.588,8.526h0.293
+    />
+    <path
+      id="n"
+      fill="#F7931E"
+      d="M249.836,205.342c0-8.379-0.441-12.643-0.588-18.375h9.85l0.588,8.526h0.293
           c3.088-5.733,8.82-9.702,16.318-9.702c9.85,0,17.199,7.35,17.199,24.55v47.775h-11.318V212.25c0-8.379-1.617-16.905-9.85-16.905
           c-4.705,0-9.262,3.969-10.732,11.613c-0.293,1.764-0.439,3.969-0.439,6.321v44.835h-11.32V205.342z"
-      />
-      <path
-        id="v-letter"
-        fill="#F7931E"
-        d="M313.783,186.966l8.967,39.985c1.029,4.851,2.205,11.172,3.088,17.346h0.441
+    />
+    <path
+      id="v"
+      fill="#F7931E"
+      d="M313.783,186.966l8.967,39.985c1.029,4.851,2.205,11.172,3.088,17.346h0.441
           c0.734-5.879,1.91-11.907,2.939-17.346l8.82-39.985h11.172l-18.523,71.149h-10.143l-18.521-71.149H313.783z"
-      />
-    </g>
-    <g>
-      <g>
-        <path
-          id="arc1"
-          fill="#F7931E"
-          d="M32.118,204.979c0,0,0.105,1.801,0.302,5.179c0.124,3.4,0.638,8.29,1.434,14.48
+    />
+  </g>
+  <g id="arcs">
+    <path
+      id="arc1"
+      fill="#F7931E"
+      d="M32.118,204.979c0,0,0.105,1.801,0.302,5.179c0.124,3.4,0.638,8.29,1.434,14.48
             c0.249,1.546,0.511,3.174,0.786,4.882c0.137,0.853,0.278,1.725,0.421,2.617c0.154,0.89,0.376,1.789,0.558,2.712
             c0.384,1.843,0.783,3.761,1.197,5.75c0.444,1.981,1.027,4.008,1.56,6.112c2.274,8.378,5.328,17.708,9.758,27.396
             c4.406,9.688,10.04,19.776,17.062,29.673c7.003,9.901,15.393,19.609,25.102,28.499c9.702,8.888,20.652,17.025,32.581,23.85
@@ -74,13 +72,11 @@
             c-12.467-7.743-23.632-16.999-33.41-26.861c-9.758-9.893-18.135-20.455-24.833-31.222c-6.743-10.739-11.988-21.536-15.97-31.794
             c-4.006-10.258-6.601-20.021-8.441-28.721c-1.917-8.698-2.688-16.429-3.245-22.764c-0.396-6.374-0.584-11.386-0.485-14.752
             c0.026-3.383,0.04-5.188,0.04-5.188c0.003-0.47,0.387-0.848,0.857-0.844C31.725,204.181,32.091,204.536,32.118,204.979z"
-        />
-      </g>
-      <g>
-        <path
-          id="arc2"
-          fill="#F7931E"
-          d="M393.625,225.021c0,0-0.104-1.731-0.297-4.979c-0.049-0.812-0.103-1.718-0.162-2.716
+    />
+    <path
+      id="arc2"
+      fill="#F7931E"
+      d="M393.625,225.021c0,0-0.104-1.731-0.297-4.979c-0.049-0.812-0.103-1.718-0.162-2.716
             c-0.029-0.499-0.061-1.021-0.093-1.565c-0.059-0.538-0.119-1.097-0.182-1.679c-0.269-2.323-0.534-4.99-0.906-7.966
             c-0.241-1.486-0.496-3.051-0.763-4.693c-0.133-0.82-0.269-1.658-0.408-2.516c-0.126-0.859-0.286-1.731-0.487-2.616
             c-0.778-3.54-1.458-7.393-2.614-11.413c-0.266-1.009-0.537-2.034-0.812-3.075c-0.264-1.043-0.553-2.096-0.908-3.147
@@ -119,26 +115,26 @@
             c0.173,3.065,0.257,5.808,0.365,8.183c0.022,0.594,0.045,1.165,0.066,1.714c-0.006,0.545-0.01,1.068-0.015,1.567
             c-0.009,1-0.017,1.908-0.024,2.721c-0.029,3.252-0.045,4.987-0.045,4.987c-0.004,0.47-0.389,0.847-0.857,0.843
             C394.018,225.817,393.652,225.463,393.625,225.021z"
-        />
-      </g>
-      <circle
-        id="circle2"
-        fill="#F7931E"
-        stroke-width="9"
-        stroke-miterlimit="10"
-        cx="70.536"
-        cy="110.694"
-        r="36.011"
-      />
-      <circle
-        id="circle1"
-        fill="#F7931E"
-        stroke-width="9"
-        stroke-miterlimit="10"
-        cx="362.721"
-        cy="309.694"
-        r="36.012"
-      />
-    </g>
+    />
+  </g>
+  <g id="circles">
+    <circle
+      id="circle2"
+      fill="#F7931E"
+      stroke-width="9"
+      stroke-miterlimit="10"
+      cx="70.536"
+      cy="110.694"
+      r="36.011"
+    />
+    <circle
+      id="circle1"
+      fill="#F7931E"
+      stroke-width="9"
+      stroke-miterlimit="10"
+      cx="362.721"
+      cy="309.694"
+      r="36.012"
+    />
   </g>
-</svg>
+</svg>

package/src/dev/start_dev_server.js

@@ -31,6 +31,8 @@ export const startDevServer = async ({
   certificate,
   privateKey,
   keepProcessAlive = true,
+  serverPlugins,
+
   rootDirectoryUrl,
   clientFiles = {
     "./src/": true,
@@ -225,6 +227,7 @@ export const startDevServer = async ({
     urlGraph,
     kitchen,
     scenario: "dev",
+    serverPlugins,
   })
   startDevServerTask.done()
   logger.info(``)

package/src/execute/runtimes/browsers/from_playwright.js

@@ -30,7 +30,7 @@ export const createRuntimeFromPlaywright = ({
   let browserAndContextPromise
   runtime.run = async ({
     signal = new AbortController().signal,
-    // logger,
+    logger,
     rootDirectoryUrl,
     fileRelativeUrl,
     server,
@@ -82,9 +82,9 @@ export const createRuntimeFromPlaywright = ({
             browser.on("disconnected", disconnectedCallback)
           })
         : Promise.resolve()
-      // for some reason without this 100ms timeout
+      // for some reason without this 150ms timeout
       // browser.close() never resolves (playwright does not like something)
-      await new Promise((resolve) => setTimeout(resolve, 100))
+      await new Promise((resolve) => setTimeout(resolve, 150))
       try {
         await browser.close()
       } catch (e) {
@@ -275,6 +275,9 @@ export const createRuntimeFromPlaywright = ({
               stopAfterAllSignal.notify = async () => {
                 await notifyPrevious()
                 browser.removeListener("disconnected", disconnectedCallback)
+                logger.debug(
+                  `stopAfterAllSignal notified -> closing ${browserName}`,
+                )
                 await closeBrowser()
               }
             }

package/src/omega/kitchen.js

@@ -64,6 +64,7 @@ export const createKitchen = ({
       return RUNTIME_COMPAT.isSupported(runtimeCompat, feature)
     },
   }
+  pluginController.callHooks("init", kitchenContext)
   const createReference = ({
     data = {},
     node,

package/src/omega/server/file_service.js

@@ -61,7 +61,13 @@ export const createFileService = ({
     const urlInfo = urlGraph.reuseOrCreateUrlInfo(reference.url)
 
     const ifNoneMatch = request.headers["if-none-match"]
-    if (ifNoneMatch && urlInfo.contentEtag === ifNoneMatch) {
+    if (
+      ifNoneMatch &&
+      urlInfo.contentEtag === ifNoneMatch &&
+      // - isValid is true by default
+      // - isValid can be overriden by plugins such as cjs_to_esm
+      urlInfo.isValid()
+    ) {
       return {
         status: 304,
         headers: {
@@ -83,6 +89,7 @@ export const createFileService = ({
         urlInfo.originalContent = null
         urlInfo.type = null
         urlInfo.subtype = null
+        urlInfo.dependsOnPackageJson = false
         urlInfo.timing = {}
       }
       const { runtimeName, runtimeVersion } = parseUserAgentHeader(

package/src/omega/url_graph.js

@@ -121,33 +121,48 @@ export const createUrlGraph = ({
   }
 
   if (clientFileChangeCallbackList) {
-    const updateModifiedTimestamp = (urlInfo, modifiedTimestamp) => {
-      const seen = []
-      const iterate = (urlInfo) => {
-        if (seen.includes(urlInfo.url)) {
-          return
-        }
-        seen.push(urlInfo.url)
-        urlInfo.modifiedTimestamp = modifiedTimestamp
-        urlInfo.dependents.forEach((dependentUrl) => {
-          const dependentUrlInfo = urlInfos[dependentUrl]
-          const { hotAcceptDependencies = [] } = dependentUrlInfo.data
-          if (!hotAcceptDependencies.includes(urlInfo.url)) {
-            iterate(dependentUrlInfo)
-          }
-        })
-      }
-      iterate(urlInfo)
-    }
     clientFileChangeCallbackList.push(({ url }) => {
       const urlInfo = urlInfos[url]
       if (urlInfo) {
-        updateModifiedTimestamp(urlInfo, Date.now())
-        urlInfo.contentEtag = null
+        considerModified(urlInfo, Date.now())
       }
     })
   }
 
+  const considerModified = (urlInfo, modifiedTimestamp = Date.now()) => {
+    const seen = []
+    const iterate = (urlInfo) => {
+      if (seen.includes(urlInfo.url)) {
+        return
+      }
+      seen.push(urlInfo.url)
+      urlInfo.modifiedTimestamp = modifiedTimestamp
+      urlInfo.contentEtag = undefined
+      urlInfo.dependents.forEach((dependentUrl) => {
+        const dependentUrlInfo = urlInfos[dependentUrl]
+        const { hotAcceptDependencies = [] } = dependentUrlInfo.data
+        if (!hotAcceptDependencies.includes(urlInfo.url)) {
+          iterate(dependentUrlInfo)
+        }
+      })
+    }
+    iterate(urlInfo)
+  }
+
+  const getRelatedUrlInfos = (url) => {
+    const urlInfosUntilNotInline = []
+    const parentUrlInfo = getUrlInfo(url)
+    if (parentUrlInfo) {
+      urlInfosUntilNotInline.push(parentUrlInfo)
+      if (parentUrlInfo.inlineUrlSite) {
+        urlInfosUntilNotInline.push(
+          ...getRelatedUrlInfos(parentUrlInfo.inlineUrlSite.url),
+        )
+      }
+    }
+    return urlInfosUntilNotInline
+  }
+
   return {
     urlInfos,
     reuseOrCreateUrlInfo,
@@ -156,6 +171,8 @@ export const createUrlGraph = ({
     inferReference,
     findDependent,
     updateReferences,
+    considerModified,
+    getRelatedUrlInfos,
 
     toJSON: (rootDirectoryUrl) => {
       const data = {}
@@ -176,6 +193,9 @@ export const createUrlGraph = ({
 const createUrlInfo = (url) => {
   return {
     modifiedTimestamp: 0,
+    contentEtag: null,
+    dependsOnPackageJson: false,
+    isValid,
     data: {}, // plugins can put whatever they want here
     references: [],
     dependencies: new Set(),
@@ -192,10 +212,12 @@ const createUrlInfo = (url) => {
     shouldHandle: undefined,
     originalContent: undefined,
     content: undefined,
-    contentEtag: null,
+
     sourcemap: null,
     sourcemapReference: null,
     timing: {},
     headers: {},
   }
 }
+
+const isValid = () => true

package/src/plugins/bundling/js_module/bundle_js_module.js

@@ -147,7 +147,11 @@ const rollupPluginJsenv = ({
     outputOptions: (outputOptions) => {
       // const sourcemapFile = buildDirectoryUrl
       Object.assign(outputOptions, {
-        format: "esm",
+        format: jsModuleUrlInfos.some((jsModuleUrlInfo) =>
+          jsModuleUrlInfo.filename.endsWith(".cjs"),
+        )
+          ? "cjs"
+          : "esm",
         dir: fileUrlConverter.asFilePath(buildDirectoryUrl),
         sourcemap: sourcemaps === "file" || sourcemaps === "inline",
         // sourcemapFile,

package/src/plugins/node_esm_resolution/jsenv_plugin_node_esm_resolution.js

@@ -18,10 +18,19 @@ import {
 
 export const jsenvPluginNodeEsmResolution = ({
   rootDirectoryUrl,
+  urlGraph,
   runtimeCompat,
   packageConditions,
   filesInvalidatingCache = ["package.json", "package-lock.json"],
 }) => {
+  const nodeRuntimeEnabled = Object.keys(runtimeCompat).includes("node")
+  // https://nodejs.org/api/esm.html#resolver-algorithm-specification
+  packageConditions = packageConditions || [
+    ...readCustomConditionsFromProcessArgs(),
+    nodeRuntimeEnabled ? "node" : "browser",
+    "import",
+  ]
+
   const packageScopesCache = new Map()
   const lookupPackageScope = (url) => {
     const fromCache = packageScopesCache.get(url)
@@ -44,65 +53,63 @@ export const jsenvPluginNodeEsmResolution = ({
   }
 
   const unregisters = []
+  const onFileChange = () => {
+    packageScopesCache.clear()
+    packageJsonsCache.clear()
+    Object.keys(urlGraph.urlInfos).forEach((url) => {
+      const urlInfo = urlGraph.getUrlInfo(url)
+      if (urlInfo.dependsOnPackageJson) {
+        urlGraph.considerModified(urlInfo)
+      }
+    })
+  }
   filesInvalidatingCache.forEach((file) => {
     const unregister = registerFileLifecycle(new URL(file, rootDirectoryUrl), {
       added: () => {
-        packageScopesCache.clear()
-        packageJsonsCache.clear()
+        onFileChange()
       },
       updated: () => {
-        packageScopesCache.clear()
-        packageJsonsCache.clear()
+        onFileChange()
       },
       removed: () => {
-        packageScopesCache.clear()
-        packageJsonsCache.clear()
+        onFileChange()
       },
       keepProcessAlive: false,
     })
     unregisters.push(unregister)
   })
 
-  return [
-    jsenvPluginNodeEsmResolver({
-      runtimeCompat,
-      packageConditions,
-      lookupPackageScope,
-      readPackageJson,
-    }),
-    jsenvPluginNodeModulesVersionInUrls({
-      lookupPackageScope,
-      readPackageJson,
-    }),
-  ]
-}
-
-const jsenvPluginNodeEsmResolver = ({
-  runtimeCompat,
-  packageConditions,
-  lookupPackageScope,
-  readPackageJson,
-}) => {
-  const nodeRuntimeEnabled = Object.keys(runtimeCompat).includes("node")
-  // https://nodejs.org/api/esm.html#resolver-algorithm-specification
-  packageConditions = packageConditions || [
-    ...readCustomConditionsFromProcessArgs(),
-    nodeRuntimeEnabled ? "node" : "browser",
-    "import",
-  ]
   return {
-    name: "jsenv:node_esm_resolve",
+    name: "jsenv:node_esm_resolution",
     appliesDuring: "*",
     resolveUrl: {
       js_import_export: (reference) => {
         const { parentUrl, specifier } = reference
-        const { url } = applyNodeEsmResolution({
+        const { type, url } = applyNodeEsmResolution({
           conditions: packageConditions,
           parentUrl,
           specifier,
           lookupPackageScope,
           readPackageJson,
         })
+
+        // this reference depend on package.json and node_modules
+        // to be resolved. Each file using this specifier
+        // must be invalidated when package.json or package_lock.json
+        // changes
+        const dependsOnPackageJson =
+          type !== "relative_specifier" &&
+          type !== "absolute_specifier" &&
+          type !== "node_builtin_specifier"
+        const relatedUrlInfos = urlGraph.getRelatedUrlInfos(reference.parentUrl)
+        relatedUrlInfos.forEach((relatedUrlInfo) => {
+          if (relatedUrlInfo.dependsOnPackageJson) {
+            // the url may depend due to an other reference
+            // in that case keep dependsOnPackageJson to true
+            return
+          }
+          relatedUrlInfo.dependsOnPackageJson = dependsOnPackageJson
+        })
         return url
       },
     },
@@ -116,20 +123,10 @@ const jsenvPluginNodeEsmResolver = ({
       }
       return null
     },
-  }
-}
-
-const jsenvPluginNodeModulesVersionInUrls = ({
-  lookupPackageScope,
-  readPackageJson,
-}) => {
-  return {
-    name: "jsenv:node_modules_version_in_urls",
-    appliesDuring: {
-      dev: true,
-      test: true,
-    },
     transformUrlSearchParams: (reference, context) => {
+      if (context.scenario === "build") {
+        return null
+      }
       if (!reference.url.startsWith("file:")) {
         return null
       }
@@ -158,5 +155,8 @@ const jsenvPluginNodeModulesVersionInUrls = ({
         v: packageVersion,
       }
     },
+    destroy: () => {
+      unregisters.forEach((unregister) => unregister())
+    },
   }
 }

package/src/plugins/plugin_controller.js

@@ -4,6 +4,7 @@ export const createPluginController = ({
   plugins,
   scenario,
   hooks = [
+    "init",
     "resolveUrl",
     "redirectUrl",
     "fetchUrlContent",

package/src/plugins/plugins.js

@@ -65,6 +65,7 @@ export const getCorePlugins = ({
     // before url resolution to handle "js_import_export" resolution
     jsenvPluginNodeEsmResolution({
       rootDirectoryUrl,
+      urlGraph,
       runtimeCompat,
       ...nodeEsmResolution,
     }),