nodejs
diff --git a/Collapse file
‎lib/internal/fs/promises.js‎
Copy file name to clipboardExpand all lines: lib/internal/fs/promises.js
+4Lines changed: 4 additions & 0 deletions b/Collapse file
‎lib/internal/fs/promises.js‎
Copy file name to clipboardExpand all lines: lib/internal/fs/promises.js
+4Lines changed: 4 additions & 0 deletions
diff --git a/Collapse file
‎test/parallel/test-fs-readdir-types.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-readdir-types.js
+8Lines changed: 8 additions & 0 deletions b/Collapse file
‎test/parallel/test-fs-readdir-types.js‎
Copy file name to clipboardExpand all lines: test/parallel/test-fs-readdir-types.js
+8Lines changed: 8 additions & 0 deletions
@@ -944,6 +944,10 @@ async function readdirRecursive(originalPath, options) {
 
 async function readdir(path, options) {
   options = getOptions(options);
+
+  // Make shallow copy to prevent mutating options from affecting results
+  options = copyObject(options);
+
   path = getValidatedPath(path);
   if (options.recursive) {
     return readdirRecursive(path, options);
 
@@ -78,6 +78,14 @@ fs.readdir(readdirDir, {
   assertDirents(dirents);
 })().then(common.mustCall());
 
+// Check that mutating options doesn't affect results
+(async () => {
+  const options = { withFileTypes: true };
+  const direntsPromise = fs.promises.readdir(readdirDir, options);
+  options.withFileTypes = false;
+  assertDirents(await direntsPromise);
+})().then(common.mustCall());
+
 // Check for correct types when the binding returns unknowns
 const UNKNOWN = constants.UV_DIRENT_UNKNOWN;
 const oldReaddir = binding.readdir;