fix(project): Prevent post-destroy builds and EBUSY on Windows in integration tests

- Add #destroyed flag to BuildServer to prevent new builds from being
  triggered after destroy() is called
- Clear pending build queue and timeout before awaiting active build
  to avoid race with setTimeout-based queue processing
- Add maxRetries to rmrf in integration tests for Windows file handle
  release delays after SQLite close

Author: RandomByte

packages/project/lib/build/BuildServer.js

@@ -44,6 +44,7 @@ class BuildServer extends EventEmitter {
 	#pendingBuildRequest = new Set();
 	#activeBuild = null;
 	#processBuildRequestsTimeout;
+	#destroyed = false;
 	#allReader;
 	#rootReader;
 	#dependenciesReader;
@@ -124,6 +125,10 @@ class BuildServer extends EventEmitter {
 	}
 
 	async destroy() {
+		this.#destroyed = true;
+		// Prevent any queued builds from starting after shutdown
+		clearTimeout(this.#processBuildRequestsTimeout);
+		this.#pendingBuildRequest.clear();
 		await this.#watchHandler.destroy();
 		if (this.#activeBuild) {
 			// Await active build to finish
@@ -313,7 +318,7 @@ class BuildServer extends EventEmitter {
 	}
 
 	#triggerRequestQueue() {
-		if (this.#activeBuild) {
+		if (this.#destroyed || this.#activeBuild) {
 			return;
 		}
 		// If no build is active, trigger queue processing debounced

packages/project/test/lib/build/BuildServer.integration.js

@@ -364,7 +364,7 @@ function getTmpPath(folderName) {
 }
 
 async function rmrf(dirPath) {
-	return fs.rm(dirPath, {recursive: true, force: true});
+	return fs.rm(dirPath, {recursive: true, force: true, maxRetries: 3, retryDelay: 200});
 }
 
 class FixtureTester {

packages/project/test/lib/build/ProjectBuilder.integration.js

@@ -2460,7 +2460,7 @@ function getTmpPath(folderName) {
 }
 
 async function rmrf(dirPath) {
-	return fs.rm(dirPath, {recursive: true, force: true});
+	return fs.rm(dirPath, {recursive: true, force: true, maxRetries: 3, retryDelay: 200});
 }
 
 class FixtureTester {