fs: make Dir disposers idempotent

PR-URL: #58692
Refs: #58206
Reviewed-By: Ethan Arrowood <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>

Author: Renegade334

doc/api/fs.md

@@ -6757,8 +6757,8 @@ added: v22.17.0
 
 > Stability: 1 - Experimental
 
-Calls `dir.close()` and returns a promise that fulfills when the
-dir is closed.
+Calls `dir.close()` if the directory handle is open, and returns a promise that
+fulfills when disposal is complete.
 
 #### `dir[Symbol.Dispose]()`
 
@@ -6768,7 +6768,8 @@ added: v22.17.0
 
 > Stability: 1 - Experimental
 
-Calls `dir.closeSync()` and returns `undefined`.
+Calls `dir.closeSync()` if the directory handle is open, and returns
+`undefined`.
 
 ### Class: `fs.Dirent`
 

lib/internal/fs/dir.js

@@ -22,7 +22,6 @@ const {
 
 const { FSReqCallback } = binding;
 const {
-  assignFunctionName,
   promisify,
   SymbolAsyncDispose,
   SymbolDispose,
@@ -296,31 +295,24 @@ class Dir {
       await this.#closePromisified();
     }
   }
+
+  [SymbolDispose]() {
+    if (this.#closed) return;
+    this.closeSync();
+  }
+
+  async [SymbolAsyncDispose]() {
+    if (this.#closed) return;
+    await this.#closePromisified();
+  }
 }
 
-const nonEnumerableDescriptor = {
-  enumerable: false,
-  writable: true,
-  configurable: true,
-};
 ObjectDefineProperties(Dir.prototype, {
-  [SymbolDispose]: {
-    __proto__: null,
-    ...nonEnumerableDescriptor,
-    value: assignFunctionName(SymbolDispose, function() {
-      this.closeSync();
-    }),
-  },
-  [SymbolAsyncDispose]: {
-    __proto__: null,
-    ...nonEnumerableDescriptor,
-    value: assignFunctionName(SymbolAsyncDispose, function() {
-      this.close();
-    }),
-  },
   [SymbolAsyncIterator]: {
     __proto__: null,
-    ...nonEnumerableDescriptor,
+    enumerable: false,
+    writable: true,
+    configurable: true,
     value: Dir.prototype.entries,
   },
 });

test/parallel/test-fs-promises-file-handle-dispose.js

@@ -11,10 +11,14 @@ async function explicitCall() {
 
   const dh = await fs.opendir(__dirname);
   await dh[Symbol.asyncDispose]();
+  // Repeat invocations should not reject
+  await dh[Symbol.asyncDispose]();
   await assert.rejects(dh.read(), { code: 'ERR_DIR_CLOSED' });
 
   const dhSync = opendirSync(__dirname);
   dhSync[Symbol.dispose]();
+  // Repeat invocations should not throw
+  dhSync[Symbol.dispose]();
   assert.throws(() => dhSync.readSync(), { code: 'ERR_DIR_CLOSED' });
 }