lib: move ToNamespacedPath call to c++

PR-URL: nodejs#53654 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Hawthorne001 · Jul 4, 2024 · 14e9119 · 14e9119
1 parent 20fbfaf
commit 14e9119
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/lib/internal/modules/esm/formats.js b/lib/internal/modules/esm/formats.js
@@ -6,7 +6,6 @@ const {
 
 const { getOptionValue } = require('internal/options');
 const { getValidatedPath } = require('internal/fs/utils');
-const pathModule = require('path');
 const fsBindings = internalBinding('fs');
 const { fs: fsConstants } = internalBinding('constants');
 
@@ -48,9 +47,7 @@ function mimeToFormat(mime) {
  */
 function getFormatOfExtensionlessFile(url) {
   if (!experimentalWasmModules) { return 'module'; }
-
-  const path = pathModule.toNamespacedPath(getValidatedPath(url));
-
+  const path = getValidatedPath(url);
   switch (fsBindings.getFormatOfExtensionlessFile(path)) {
     case fsConstants.EXTENSIONLESS_FORMAT_WASM:
       return 'wasm';

diff --git a/src/node_file.cc b/src/node_file.cc
@@ -2966,8 +2966,8 @@ static void GetFormatOfExtensionlessFile(
   CHECK(args[0]->IsString());
 
   Environment* env = Environment::GetCurrent(args);
-  node::Utf8Value input(args.GetIsolate(), args[0]);
-
+  BufferValue input(args.GetIsolate(), args[0]);
+  ToNamespacedPath(env, &input);
   THROW_IF_INSUFFICIENT_PERMISSIONS(
       env, permission::PermissionScope::kFileSystemRead, input.ToStringView());