fs: load rimraf lazily in fs/promises

Avoid the potential circular dependency and make fs/promises load faster
when rimraf is not used.

PR-URL: #51617
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Vinícius Lourenço Claro Cardoso <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Author: joyeecheung

lib/internal/fs/promises.js

@@ -43,7 +43,7 @@ const {
   aggregateTwoErrors,
 } = require('internal/errors');
 const { isArrayBufferView } = require('internal/util/types');
-const { rimrafPromises } = require('internal/fs/rimraf');
+
 const {
   constants: {
     kIoMaxLength,
@@ -92,6 +92,7 @@ const {
   kEmptyObject,
   lazyDOMException,
   promisify,
+  getLazy,
 } = require('internal/util');
 const { EventEmitterMixin } = require('internal/event_target');
 const { StringDecoder } = require('string_decoder');
@@ -135,6 +136,8 @@ function lazyFsStreams() {
   return fsStreams ??= require('internal/fs/streams');
 }
 
+const lazyRimRaf = getLazy(() => require('internal/fs/rimraf').rimrafPromises);
+
 class FileHandle extends EventEmitterMixin(JSTransferable) {
   /**
    * @param {InternalFSBinding.FileHandle | undefined} filehandle
@@ -749,7 +752,7 @@ async function ftruncate(handle, len = 0) {
 async function rm(path, options) {
   path = pathModule.toNamespacedPath(getValidatedPath(path));
   options = await validateRmOptionsPromise(path, options, false);
-  return rimrafPromises(path, options);
+  return lazyRimRaf()(path, options);
 }
 
 async function rmdir(path, options) {
@@ -760,7 +763,7 @@ async function rmdir(path, options) {
     emitRecursiveRmdirWarning();
     const stats = await stat(path);
     if (stats.isDirectory()) {
-      return rimrafPromises(path, options);
+      return lazyRimRaf()(path, options);
     }
   }