fix: retry on EMFILE when writing autofix results (#19926)

TKDev7 · web-flow · commit 07fac6cafa04 · 2025-07-10T06:14:14.000+02:00
* fix: retry on EMFILE when writing autofix results

* chore: update test
diff --git a/lib/eslint/eslint.js b/lib/eslint/eslint.js
@@ -530,6 +530,11 @@ class ESLint {
 			throw new Error("'results' must be an array");
 		}
 
+		const retryCodes = new Set(["ENFILE", "EMFILE"]);
+		const retrier = new Retrier(error => retryCodes.has(error.code), {
+			concurrency: 100,
+		});
+
 		await Promise.all(
 			results
 				.filter(result => {
@@ -541,7 +546,9 @@ class ESLint {
 						path.isAbsolute(result.filePath)
 					);
 				})
-				.map(r => fs.writeFile(r.filePath, r.output)),
+				.map(r =>
+					retrier.retry(() => fs.writeFile(r.filePath, r.output)),
+				),
 		);
 	}
 
diff --git a/tests/fixtures/emfile/eslint.config.js b/tests/fixtures/emfile/eslint.config.js
@@ -1,5 +1,5 @@
 module.exports = {
     rules: {
-        "no-unused-vars": "error"
+        "capitalized-comments": "error"
     }
 };
diff --git a/tools/check-emfile-handling.js b/tools/check-emfile-handling.js
@@ -63,7 +63,7 @@ function generateFiles() {
 
 	for (let i = 0; i < FILE_COUNT; i++) {
 		const fileName = `file_${i}.js`;
-		const fileContent = `// This is file ${i}`;
+		const fileContent = `// this is file ${i}`;
 
 		fs.writeFileSync(`${OUTPUT_DIRECTORY}/${fileName}`, fileContent);
 	}
@@ -97,7 +97,7 @@ generateFiles();
 
 console.log("Running ESLint...");
 execSync(
-	`node bin/eslint.js ${OUTPUT_DIRECTORY} -c ${CONFIG_DIRECTORY}/eslint.config.js`,
+	`node bin/eslint.js ${OUTPUT_DIRECTORY} -c ${CONFIG_DIRECTORY}/eslint.config.js --fix`,
 	{ stdio: "inherit" },
 );
 console.log("✅ No errors encountered running ESLint.");

Original file line number	Diff line number	Diff line change
`@@ -1,5 +1,5 @@`
`1`	`1`	`module.exports = {`
`2`	`2`	`rules: {`
`3`		`- "no-unused-vars": "error"`
	`3`	`+ "capitalized-comments": "error"`
`4`	`4`	`}`
`5`	`5`	`};`