lib: improve esm resolve performance

nodejs · Feb 14, 2023 · 76fb5d6 · 76fb5d6
1 parent ecd385e
commit 76fb5d6
Showing 1 changed file with 14 additions and 17 deletions.
diff --git a/lib/internal/modules/esm/resolve.js b/lib/internal/modules/esm/resolve.js
@@ -27,8 +27,6 @@ const internalFS = require('internal/fs/utils');
 const { BuiltinModule } = require('internal/bootstrap/loaders');
 const {
   realpathSync,
-  statSync,
-  Stats,
 } = require('fs');
 const { getOptionValue } = require('internal/options');
 // Do not eagerly grab .manifest, it may be in TDZ
@@ -41,7 +39,7 @@ const preserveSymlinksMain = getOptionValue('--preserve-symlinks-main');
 const experimentalNetworkImports =
   getOptionValue('--experimental-network-imports');
 const typeFlag = getOptionValue('--input-type');
-const { URL, pathToFileURL, fileURLToPath } = require('internal/url');
+const { URL, pathToFileURL, fileURLToPath, toPathIfFileURL } = require('internal/url');
 const {
   ERR_INPUT_TYPE_NOT_ALLOWED,
   ERR_INVALID_MODULE_SPECIFIER,
@@ -59,6 +57,7 @@ const {
 const { Module: CJSModule } = require('internal/modules/cjs/loader');
 const { getPackageConfig, getPackageScopeConfig } = require('internal/modules/esm/package_config');
 const { getConditionsSet } = require('internal/modules/esm/utils');
+const { internalModuleStat } = internalBinding('fs');
 
 /**
  * @typedef {import('internal/modules/esm/package_config.js').PackageConfig} PackageConfig
@@ -135,19 +134,12 @@ function emitLegacyIndexDeprecation(url, packageJSONUrl, base, main) {
 
 const realpathCache = new SafeMap();
 
-/**
- * @param {string | URL} path
- * @returns {import('fs').Stats}
- */
-const tryStatSync =
-  (path) => statSync(path, { throwIfNoEntry: false }) ?? new Stats();
-
 /**
  * @param {string | URL} url
  * @returns {boolean}
  */
 function fileExists(url) {
-  return statSync(url, { throwIfNoEntry: false })?.isFile() ?? false;
+  return internalModuleStat(toPathIfFileURL(url)) === 0;
 }
 
 /**
@@ -218,13 +210,16 @@ function finalizeResolution(resolved, base, preserveSymlinks) {
 
   const path = fileURLToPath(resolved);
 
-  const stats = tryStatSync(StringPrototypeEndsWith(path, '/') ?
+  const stats = internalModuleStat(StringPrototypeEndsWith(path, '/') ?
     StringPrototypeSlice(path, -1) : path);
-  if (stats.isDirectory()) {
+
+  // Check for stats.isDirectory()
+  if (stats === 1) {
     const err = new ERR_UNSUPPORTED_DIR_IMPORT(path, fileURLToPath(base));
     err.url = String(resolved);
     throw err;
-  } else if (!stats.isFile()) {
+  } else if (stats !== 0) {
+    // Check for !stats.isFile()
     if (process.env.WATCH_REPORT_DEPENDENCIES && process.send) {
       process.send({ 'watch:require': [path || resolved.pathname] });
     }
@@ -760,9 +755,11 @@ function packageResolve(specifier, base, conditions) {
   let packageJSONPath = fileURLToPath(packageJSONUrl);
   let lastPath;
   do {
-    const stat = tryStatSync(StringPrototypeSlice(packageJSONPath, 0,
-                                                  packageJSONPath.length - 13));
-    if (!stat.isDirectory()) {
+    const stat = internalModuleStat(StringPrototypeSlice(packageJSONPath, 0,
+                                                         packageJSONPath.length - 13));
+
+    // Check for !stat.isDirectory
+    if (stat !== 1) {
       lastPath = packageJSONPath;
       packageJSONUrl = new URL((isScoped ?
         '../../../../node_modules/' : '../../../node_modules/') +