@jsenv/core 28.6.0 → 29.0.0

package/src/plugins/url_resolution/node_esm_resolver.js

@@ -0,0 +1,110 @@
+/*
+ * - should I restore eventual search params lost during node esm resolution
+ * - what about symlinks?
+ *   It feels like I should apply symlink (when we don't want to preserve them)
+ *   once a file:/// url is found, regardless
+ *   if that comes from node resolution or anything else (not even magic resolution)
+ *   it should likely be an other plugin happening after the others
+ */
+
+import { readFileSync } from "node:fs"
+import { bufferToEtag } from "@jsenv/filesystem"
+import {
+  applyNodeEsmResolution,
+  readCustomConditionsFromProcessArgs,
+  defaultLookupPackageScope,
+  defaultReadPackageJson,
+} from "@jsenv/node-esm-resolution"
+
+export const createNodeEsmResolver = ({ runtimeCompat, packageConditions }) => {
+  const nodeRuntimeEnabled = Object.keys(runtimeCompat).includes("node")
+  // https://nodejs.org/api/esm.html#resolver-algorithm-specification
+  packageConditions = packageConditions || [
+    ...readCustomConditionsFromProcessArgs(),
+    nodeRuntimeEnabled ? "node" : "browser",
+    "import",
+  ]
+
+  return (reference, context) => {
+    const { parentUrl, specifier } = reference
+    const { url, type, packageUrl } = applyNodeEsmResolution({
+      conditions: packageConditions,
+      parentUrl,
+      specifier,
+    })
+    if (context.scenarios.dev) {
+      const dependsOnPackageJson =
+        type !== "relative_specifier" &&
+        type !== "absolute_specifier" &&
+        type !== "node_builtin_specifier"
+      if (dependsOnPackageJson) {
+        // this reference depends on package.json and node_modules
+        // to be resolved. Each file using this specifier
+        // must be invalidated when corresponding package.json changes
+        addRelationshipWithPackageJson({
+          reference,
+          context,
+          packageJsonUrl: `${packageUrl}package.json`,
+          field: type.startsWith("field:")
+            ? `#${type.slice("field:".length)}`
+            : "",
+        })
+      }
+    }
+    if (context.scenarios.dev) {
+      // without this check a file inside a project without package.json
+      // could be considered as a node module if there is a ancestor package.json
+      // but we want to version only node modules
+      if (url.includes("/node_modules/")) {
+        const packageDirectoryUrl = defaultLookupPackageScope(url)
+        if (
+          packageDirectoryUrl &&
+          packageDirectoryUrl !== context.rootDirectoryUrl
+        ) {
+          const packageVersion =
+            defaultReadPackageJson(packageDirectoryUrl).version
+          // package version can be null, see https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
+          if (packageVersion) {
+            addRelationshipWithPackageJson({
+              reference,
+              context,
+              packageJsonUrl: `${packageDirectoryUrl}package.json`,
+              field: "version",
+              hasVersioningEffect: true,
+            })
+          }
+          reference.version = packageVersion
+        }
+      }
+    }
+    return url
+  }
+}
+
+const addRelationshipWithPackageJson = ({
+  context,
+  packageJsonUrl,
+  field,
+  hasVersioningEffect = false,
+}) => {
+  const referenceFound = context.referenceUtils.find(
+    (ref) => ref.type === "package_json" && ref.subtype === field,
+  )
+  if (referenceFound) {
+    return
+  }
+  const [, packageJsonUrlInfo] = context.referenceUtils.inject({
+    type: "package_json",
+    subtype: field,
+    specifier: packageJsonUrl,
+    isImplicit: true,
+    hasVersioningEffect,
+  })
+  if (packageJsonUrlInfo.type === undefined) {
+    const packageJsonContentAsBuffer = readFileSync(new URL(packageJsonUrl))
+    packageJsonUrlInfo.type = "json"
+    packageJsonUrlInfo.content = String(packageJsonContentAsBuffer)
+    packageJsonUrlInfo.originalContentEtag = packageJsonUrlInfo.contentEtag =
+      bufferToEtag(packageJsonContentAsBuffer)
+  }
+}

package/src/plugins/url_version/jsenv_plugin_url_version.js

@@ -9,19 +9,24 @@ export const jsenvPluginUrlVersion = () => {
       // this goal is achieved when we reach this part of the code
       // We get rid of this params so that urlGraph and other parts of the code
       // recognize the url (it is not considered as a different url)
-      const urlObject = new URL(reference.url)
-      urlObject.searchParams.delete("v")
-      return urlObject.href
+      const version = reference.searchParams.get("v")
+      if (version) {
+        const urlObject = new URL(reference.url)
+        urlObject.searchParams.delete("v")
+        reference.version = version
+        return urlObject.href
+      }
+      return null
     },
     transformUrlSearchParams: (reference) => {
-      if (!reference.data.version) {
+      if (!reference.version) {
         return null
       }
       if (reference.searchParams.has("v")) {
         return null
       }
       return {
-        v: reference.data.version,
+        v: reference.version,
       }
     },
   }

package/src/plugins/leading_slash/jsenv_plugin_leading_slash.js

@@ -1,13 +0,0 @@
-export const jsenvPluginLeadingSlash = () => {
-  return {
-    name: "jsenv:leading_slash",
-    appliesDuring: "*",
-    resolveUrl: (reference, context) => {
-      if (reference.specifier[0] !== "/") {
-        return null
-      }
-      return new URL(reference.specifier.slice(1), context.rootDirectoryUrl)
-        .href
-    },
-  }
-}

package/src/plugins/node_esm_resolution/jsenv_plugin_node_esm_resolution.js

@@ -1,148 +0,0 @@
-/*
- * - should I restore eventual search params lost during node esm resolution
- * - what about symlinks?
- *   It feels like I should apply symlink (when we don't want to preserve them)
- *   once a file:/// url is found, regardless
- *   if that comes from node resolution or anything else (not even magic resolution)
- *   it should likely be an other plugin happening after the others
- */
-
-import { readFileSync } from "node:fs"
-import { bufferToEtag } from "@jsenv/filesystem"
-import {
-  applyNodeEsmResolution,
-  defaultLookupPackageScope,
-  defaultReadPackageJson,
-  readCustomConditionsFromProcessArgs,
-} from "@jsenv/node-esm-resolution"
-
-export const jsenvPluginNodeEsmResolution = ({ packageConditions }) => {
-  const addRelationshipWithPackageJson = ({
-    context,
-    packageJsonUrl,
-    field,
-    hasVersioningEffect = false,
-  }) => {
-    const referenceFound = context.referenceUtils.find(
-      (ref) => ref.type === "package_json" && ref.subtype === field,
-    )
-    if (referenceFound) {
-      return
-    }
-    const [, packageJsonUrlInfo] = context.referenceUtils.inject({
-      type: "package_json",
-      subtype: field,
-      specifier: packageJsonUrl,
-      isImplicit: true,
-      hasVersioningEffect,
-    })
-    if (packageJsonUrlInfo.type === undefined) {
-      const packageJsonContentAsBuffer = readFileSync(new URL(packageJsonUrl))
-      packageJsonUrlInfo.type = "json"
-      packageJsonUrlInfo.content = String(packageJsonContentAsBuffer)
-      packageJsonUrlInfo.originalContentEtag = packageJsonUrlInfo.contentEtag =
-        bufferToEtag(packageJsonContentAsBuffer)
-    }
-  }
-
-  return {
-    name: "jsenv:node_esm_resolution",
-    appliesDuring: "*",
-    init: (context) => {
-      const nodeRuntimeEnabled = Object.keys(context.runtimeCompat).includes(
-        "node",
-      )
-      // https://nodejs.org/api/esm.html#resolver-algorithm-specification
-      packageConditions = packageConditions || [
-        ...readCustomConditionsFromProcessArgs(),
-        nodeRuntimeEnabled ? "node" : "browser",
-        "import",
-      ]
-    },
-    resolveUrl: {
-      js_import_export: (reference, context) => {
-        const { parentUrl, specifier } = reference
-        const { url, type, packageUrl } = applyNodeEsmResolution({
-          conditions: packageConditions,
-          parentUrl,
-          specifier,
-        })
-        if (context.scenarios.dev) {
-          const dependsOnPackageJson =
-            type !== "relative_specifier" &&
-            type !== "absolute_specifier" &&
-            type !== "node_builtin_specifier"
-          if (dependsOnPackageJson) {
-            // this reference depends on package.json and node_modules
-            // to be resolved. Each file using this specifier
-            // must be invalidated when corresponding package.json changes
-            addRelationshipWithPackageJson({
-              reference,
-              context,
-              packageJsonUrl: `${packageUrl}package.json`,
-              field: type.startsWith("field:")
-                ? `#${type.slice("field:".length)}`
-                : "",
-            })
-          }
-        }
-        return url
-      },
-    },
-    transformUrlSearchParams: (reference, context) => {
-      if (reference.type === "package_json") {
-        return null
-      }
-      if (context.scenarios.build) {
-        return null
-      }
-      if (!reference.url.startsWith("file:")) {
-        return null
-      }
-      // without this check a file inside a project without package.json
-      // could be considered as a node module if there is a ancestor package.json
-      // but we want to version only node modules
-      if (!reference.url.includes("/node_modules/")) {
-        return null
-      }
-      if (reference.searchParams.has("v")) {
-        return null
-      }
-      const packageDirectoryUrl = defaultLookupPackageScope(reference.url)
-      if (!packageDirectoryUrl) {
-        return null
-      }
-      if (packageDirectoryUrl === context.rootDirectoryUrl) {
-        return null
-      }
-      // there is a dependency between this file and the package.json version field
-      const packageVersion = defaultReadPackageJson(packageDirectoryUrl).version
-      if (!packageVersion) {
-        // example where it happens: https://github.com/babel/babel/blob/2ce56e832c2dd7a7ed92c89028ba929f874c2f5c/packages/babel-runtime/helpers/esm/package.json#L2
-        return null
-      }
-      if (reference.type === "js_import_export") {
-        addRelationshipWithPackageJson({
-          reference,
-          context,
-          packageJsonUrl: `${packageDirectoryUrl}package.json`,
-          field: "version",
-          hasVersioningEffect: true,
-        })
-      }
-      return {
-        v: packageVersion,
-      }
-    },
-    fetchUrlContent: (urlInfo) => {
-      if (urlInfo.url.startsWith("file:///@ignore/")) {
-        return {
-          content: "export default {}",
-          contentType: "text/javascript",
-          type: "js_module",
-        }
-      }
-      return null
-    },
-  }
-}