fix: add retry logic to fs.rmSync to handle ENOTEMPTY errors

Copilot · hotlong · Copilot · commit 7c81d3759f78 · 2026-02-05T09:56:52.000Z
Add maxRetries and retryDelay options to fs.rmSync() call in build-schemas.ts to handle race conditions that can occur during directory cleanup. This fixes the ENOTEMPTY error that was causing CI builds to fail. Resolves: https://github.com/objectstack-ai/spec/actions/runs/21706243682 Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
diff --git a/packages/spec/scripts/build-schemas.ts b/packages/spec/scripts/build-schemas.ts
@@ -9,7 +9,7 @@ const OUT_DIR = path.resolve(__dirname, '../json-schema');
 // Clean output directory ensures no stale files remain
 if (fs.existsSync(OUT_DIR)) {
   console.log(`Cleaning output directory: ${OUT_DIR}`);
-  fs.rmSync(OUT_DIR, { recursive: true, force: true });
+  fs.rmSync(OUT_DIR, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 });
 }
 
 // Ensure output directory exists

Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ const OUT_DIR = path.resolve(__dirname, '../json-schema');`
`9`	`9`	`// Clean output directory ensures no stale files remain`
`10`	`10`	`if (fs.existsSync(OUT_DIR)) {`
`11`	`11`	console.log(`Cleaning output directory: ${OUT_DIR}`);
`12`		`- fs.rmSync(OUT_DIR, { recursive: true, force: true });`
	`12`	`+ fs.rmSync(OUT_DIR, { recursive: true, force: true, maxRetries: 3, retryDelay: 100 });`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`// Ensure output directory exists`