Try to fix build on Windows

Author: cte

packages/build/src/esbuild.ts

@@ -3,13 +3,56 @@ import * as path from "path"
 
 import { ViewsContainer, Views, Menus, Configuration, contributesSchema } from "./types.js"
 
+function copyDir(srcDir: string, dstDir: string, count: number): number {
+	const entries = fs.readdirSync(srcDir, { withFileTypes: true })
+
+	for (const entry of entries) {
+		const srcPath = path.join(srcDir, entry.name)
+		const dstPath = path.join(dstDir, entry.name)
+
+		if (entry.isDirectory()) {
+			fs.mkdirSync(dstPath, { recursive: true })
+			count = copyDir(srcPath, dstPath, count)
+		} else {
+			count = count + 1
+			fs.copyFileSync(srcPath, dstPath)
+		}
+	}
+
+	return count
+}
+
+function rmDir(dirPath: string, maxRetries: number = 3): void {
+	for (let attempt = 1; attempt <= maxRetries; attempt++) {
+		try {
+			fs.rmSync(dirPath, { recursive: true, force: true })
+			return
+		} catch (error) {
+			const isLastAttempt = attempt === maxRetries
+			const isEnotemptyError = error instanceof Error && "code" in error && (error.code === 'ENOTEMPTY' || error.code === 'EBUSY')
+
+			if (isLastAttempt || !isEnotemptyError) {
+				throw error // Re-throw if it's the last attempt or not a locking error.
+			}
+
+			// Wait with exponential backoff before retrying.
+			const delay = Math.min(100 * Math.pow(2, attempt - 1), 1000) // Cap at 1s.
+			console.warn(`[rmDir] Attempt ${attempt} failed for ${dirPath}, retrying in ${delay}ms...`)
+
+			// Synchronous sleep for simplicity in build scripts.
+			const start = Date.now()
+			while (Date.now() - start < delay) { /* Busy wait */ }
+		}
+	}
+}
+
 export function copyPaths(copyPaths: [string, string][], srcDir: string, dstDir: string) {
 	copyPaths.forEach(([srcRelPath, dstRelPath]) => {
 		const stats = fs.lstatSync(path.join(srcDir, srcRelPath))
 
 		if (stats.isDirectory()) {
 			if (fs.existsSync(path.join(dstDir, dstRelPath))) {
-				fs.rmSync(path.join(dstDir, dstRelPath), { recursive: true, force: true })
+				rmDir(path.join(dstDir, dstRelPath))
 			}
 
 			fs.mkdirSync(path.join(dstDir, dstRelPath), { recursive: true })
@@ -23,25 +66,6 @@ export function copyPaths(copyPaths: [string, string][], srcDir: string, dstDir:
 	})
 }
 
-export function copyDir(srcDir: string, dstDir: string, count: number): number {
-	const entries = fs.readdirSync(srcDir, { withFileTypes: true })
-
-	for (const entry of entries) {
-		const srcPath = path.join(srcDir, entry.name)
-		const dstPath = path.join(dstDir, entry.name)
-
-		if (entry.isDirectory()) {
-			fs.mkdirSync(dstPath, { recursive: true })
-			count = copyDir(srcPath, dstPath, count)
-		} else {
-			count = count + 1
-			fs.copyFileSync(srcPath, dstPath)
-		}
-	}
-
-	return count
-}
-
 export function copyWasms(srcDir: string, distDir: string): void {
 	const nodeModulesDir = path.join(srcDir, "node_modules")
 

packages/build/src/index.ts

@@ -1,2 +1,2 @@
 export { getGitSha } from "./git.js"
-export { copyPaths, copyDir, copyWasms, copyLocales, setupLocaleWatcher, generatePackageJson } from "./esbuild.js"
+export { copyPaths, copyWasms, copyLocales, setupLocaleWatcher, generatePackageJson } from "./esbuild.js"

src/esbuild.mjs

@@ -44,7 +44,7 @@ async function main() {
 	 */
 	const plugins = [
 		{
-			name: "copy-files",
+			name: "copyFiles",
 			setup(build) {
 				build.onEnd(() => {
 					copyPaths(
@@ -62,13 +62,13 @@ async function main() {
 			},
 		},
 		{
-			name: "copy-wasms",
+			name: "copyWasms",
 			setup(build) {
 				build.onEnd(() => copyWasms(srcDir, distDir))
 			},
 		},
 		{
-			name: "copy-locales",
+			name: "copyLocales",
 			setup(build) {
 				build.onEnd(() => copyLocales(srcDir, distDir))
 			},