fs: improve ExistsSync performance

nodejs · Jun 21, 2024 · 21d2b35 · 21d2b35
1 parent d335487
commit 21d2b35
Showing 1 changed file with 2 additions and 18 deletions.
diff --git a/src/node_file.cc b/src/node_file.cc
@@ -1003,24 +1003,8 @@ static void ExistsSync(const FunctionCallbackInfo<Value>& args) {
   THROW_IF_INSUFFICIENT_PERMISSIONS(
       env, permission::PermissionScope::kFileSystemRead, path.ToStringView());
 
-  uv_fs_t req;
-  auto make = OnScopeLeave([&req]() { uv_fs_req_cleanup(&req); });
-  FS_SYNC_TRACE_BEGIN(access);
-  int err = uv_fs_access(nullptr, &req, path.out(), 0, nullptr);
-  FS_SYNC_TRACE_END(access);
-
-#ifdef _WIN32
-  // In case of an invalid symlink, `uv_fs_access` on win32
-  // will **not** return an error and is therefore not enough.
-  // Double check with `uv_fs_stat()`.
-  if (err == 0) {
-    FS_SYNC_TRACE_BEGIN(stat);
-    err = uv_fs_stat(nullptr, &req, path.out(), nullptr);
-    FS_SYNC_TRACE_END(stat);
-  }
-#endif  // _WIN32
-
-  args.GetReturnValue().Set(err == 0);
+  auto file_path = std::filesystem::path(path.ToStringView());
+  args.GetReturnValue().Set(std::filesystem::exists(file_path));
 }
 
 // Used to speed up module loading.  Returns 0 if the path refers to