@jsenv/node-esm-resolution 1.2.16 → 1.3.0

package/package.json

@@ -1,25 +1,24 @@
 {
   "name": "@jsenv/node-esm-resolution",
-  "version": "1.2.16",
+  "version": "1.3.0",
+  "type": "module",
   "repository": {
     "type": "git",
     "url": "https://github.com/jsenv/core",
     "directory": "packages/tooling/node-esm-resolution"
   },
   "license": "MIT",
-  "sideEffects": false,
-  "type": "module",
+  "main": "./main.js",
   "exports": {
     ".": {
       "import": "./main.js"
     },
     "./*": "./*"
   },
-  "main": "./main.js",
   "files": [
     "/dist/",
-    "/src/",
-    "/main.js"
+    "/main.js",
+    "/src/"
   ],
   "dependencies": {
     "@jsenv/filesystem": "4.15.15"
@@ -31,5 +30,6 @@
   },
   "publishConfig": {
     "access": "public"
-  }
+  },
+  "sideEffects": false
 }

package/src/node_builtin_specifiers.js

@@ -1,30 +1,27 @@
 export const isSpecifierForNodeBuiltin = (specifier) => {
   return (
     specifier.startsWith("node:") ||
-    NODE_BUILTIN_MODULE_SPECIFIERS.includes(specifier)
+    NODE_BUILTIN_MODULE_SPECIFIER_SET.has(specifier)
   );
 };
 
-const NODE_BUILTIN_MODULE_SPECIFIERS = [
+const NODE_BUILTIN_MODULE_SPECIFIER_SET = new Set([
   "assert",
   "assert/strict",
   "async_hooks",
-  "buffer_ieee754",
   "buffer",
   "child_process",
   "cluster",
   "console",
   "constants",
   "crypto",
-  "_debugger",
   "diagnostics_channel",
   "dgram",
   "dns",
+  "dns/promises",
   "domain",
   "events",
-  "freelist",
   "fs",
-  "fsevents",
   "fs/promises",
   "_http_agent",
   "_http_client",
@@ -36,21 +33,21 @@ const NODE_BUILTIN_MODULE_SPECIFIERS = [
   "http2",
   "https",
   "inspector",
-  "_linklist",
+  "inspector/promises",
   "module",
   "net",
-  "node-inspect/lib/_inspect",
-  "node-inspect/lib/internal/inspect_client",
-  "node-inspect/lib/internal/inspect_repl",
   "os",
   "path",
+  "path/posix",
+  "path/win32",
   "perf_hooks",
   "process",
   "punycode",
   "querystring",
   "readline",
+  "readline/promises",
   "repl",
-  "smalloc",
+  "sea",
   "sqlite",
   "_stream_duplex",
   "_stream_transform",
@@ -59,10 +56,15 @@ const NODE_BUILTIN_MODULE_SPECIFIERS = [
   "_stream_readable",
   "_stream_writable",
   "stream",
+  "stream/consumers",
   "stream/promises",
+  "stream/web",
   "string_decoder",
   "sys",
+  "test",
+  "test/reporters",
   "timers",
+  "timers/promises",
   "_tls_common",
   "_tls_legacy",
   "_tls_wrap",
@@ -71,17 +73,12 @@ const NODE_BUILTIN_MODULE_SPECIFIERS = [
   "tty",
   "url",
   "util",
-  "v8/tools/arguments",
-  "v8/tools/codemap",
-  "v8/tools/consarray",
-  "v8/tools/csvparser",
-  "v8/tools/logreader",
-  "v8/tools/profile_view",
-  "v8/tools/splaytree",
+  "util/types",
   "v8",
   "vm",
+  "wasi",
   "worker_threads",
   "zlib",
   // global is special
   "global",
-];
+]);

package/src/node_esm_resolution.js

@@ -647,20 +647,22 @@ const applyLegacySubpathResolution = (packageSubpath, resolutionContext) => {
 
 const applyLegacyMainResolution = (packageSubpath, resolutionContext) => {
   const { conditions, packageDirectoryUrl, packageJson } = resolutionContext;
-  for (const condition of conditions) {
-    const conditionResolver = mainLegacyResolvers[condition];
-    if (!conditionResolver) {
-      continue;
-    }
-    const resolved = conditionResolver(resolutionContext);
-    if (resolved) {
-      return createResolutionResult({
-        type: resolved.type,
-        isMain: resolved.isMain,
-        packageDirectoryUrl,
-        packageJson,
-        url: new URL(resolved.path, packageDirectoryUrl).href,
-      });
+  if (packageJson) {
+    for (const condition of conditions) {
+      const conditionResolver = mainLegacyResolvers[condition];
+      if (!conditionResolver) {
+        continue;
+      }
+      const resolved = conditionResolver(resolutionContext);
+      if (resolved) {
+        return createResolutionResult({
+          type: resolved.type,
+          isMain: resolved.isMain,
+          packageDirectoryUrl,
+          packageJson,
+          url: new URL(resolved.path, packageDirectoryUrl).href,
+        });
+      }
     }
   }
   return createResolutionResult({