refactor(project): ProjectBuilder cleanup, add UI5_BUILD_NO_WRITE param

Author: RandomByte

packages/project/lib/build/ProjectBuilder.js

@@ -14,7 +14,6 @@ import OutputStyleEnum from "./helpers/ProjectBuilderOutputStyle.js";
 class ProjectBuilder {
 	#log;
 	#buildIsRunning = false;
-	// #resourceChanges = new Map();
 
 	/**
 	 * Build Configuration
@@ -122,26 +121,9 @@ class ProjectBuilder {
 	}
 
 	resourcesChanged(changes) {
-		// if (!this.#resourceChanges.size) {
-		// 	this.#resourceChanges = changes;
-		// 	return;
-		// }
-		// for (const [project, resourcePaths] of changes.entries()) {
-		// 	if (!this.#resourceChanges.has(project.getName())) {
-		// 		this.#resourceChanges.set(project.getName(), []);
-		// 	}
-		// 	const projectChanges = this.#resourceChanges.get(project.getName());
-		// 	projectChanges.push(...resourcePaths);
-		// }
-
 		return this._buildContext.propagateResourceChanges(changes);
 	}
 
-	// _flushResourceChanges() {
-	// 	this._buildContext.propagateResurceChanges(this.#resourceChanges);
-	// 	this.#resourceChanges = new Map();
-	// }
-
 	async build({
 		includedDependencies = [], excludedDependencies = [],
 	}) {
@@ -188,7 +170,7 @@ class ProjectBuilder {
 		const requestedProjects = this._determineRequestedProjects(
 			includedDependencies, excludedDependencies, dependencyIncludes);
 
-		if (destPath && cleanDest) {
+		if (cleanDest) {
 			this.#log.info(`Cleaning target directory...`);
 			await rmrf(destPath);
 		}
@@ -293,7 +275,7 @@ class ProjectBuilder {
 					pWrites.push(projectBuildContext.getBuildCache().writeCache());
 				}
 
-				if (fsTarget && requestedProjects.includes(projectName)) {
+				if (fsTarget && requestedProjects.includes(projectName) && !process.env.UI5_BUILD_NO_WRITE) {
 					// Only write requested projects to target
 					// (excluding dependencies that were required to be built, but not requested)
 					this.#log.verbose(`Writing out files for project ${projectName}...`);

packages/project/lib/build/cache/ResourceRequestManager.js

@@ -87,7 +87,7 @@ class ResourceRequestManager {
 				const registry = registries.get(node.getParentId());
 				if (!registry) {
 					throw new Error(`Missing tree registry for parent of node ID ${nodeId} of task ` +
-						`'${this.#taskName}' of project '${this.#projectName}'`);
+					`'${taskName}' of project '${projectName}'`);
 				}
 				const resourceIndex = parentResourceIndex.deriveTreeWithIndex(addedResourceIndex);