@ui5/builder 3.0.0-rc.2 → 3.0.0-rc.4

package/CHANGELOG.md

@@ -2,10 +2,19 @@
 All notable changes to this project will be documented in this file.  
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
-A list of unreleased changes can be found [here](https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.2...HEAD).
+A list of unreleased changes can be found [here](https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.4...HEAD).
+
+<a name="v3.0.0-rc.4"></a>
+## [v3.0.0-rc.4] - 2023-02-03
+### Features
+- **minify task:** Use workerpool for terser ([#875](https://github.com/SAP/ui5-builder/issues/875)) [`ff4c6fb`](https://github.com/SAP/ui5-builder/commit/ff4c6fbd81c912239cd578fdcd452364ac4ae859)
+
+
+<a name="v3.0.0-rc.3"></a>
+## [v3.0.0-rc.3] - 2023-01-24
 
 <a name="v3.0.0-rc.2"></a>
-## [v3.0.0-rc.2] - 2023-01-22
+## [v3.0.0-rc.2] - 2023-01-23
 ### Dependency Updates
 - Bump rimraf from 3.0.2 to 4.1.1 ([#873](https://github.com/SAP/ui5-builder/issues/873)) [`ddb9660`](https://github.com/SAP/ui5-builder/commit/ddb96602fea5b96ba37afdebda0993d847462f52)
 
@@ -843,6 +852,8 @@ to load the custom bundle file instead.
 - Add ability to configure component preloads and custom bundles [`2241e5f`](https://github.com/SAP/ui5-builder/commit/2241e5ff98fd95f1f80cc74959655ae7a9c660e7)
 
 
+[v3.0.0-rc.4]: https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.3...v3.0.0-rc.4
+[v3.0.0-rc.3]: https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.2...v3.0.0-rc.3
 [v3.0.0-rc.2]: https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.1...v3.0.0-rc.2
 [v3.0.0-rc.1]: https://github.com/SAP/ui5-builder/compare/v3.0.0-rc.0...v3.0.0-rc.1
 [v3.0.0-rc.0]: https://github.com/SAP/ui5-builder/compare/v3.0.0-beta.6...v3.0.0-rc.0

package/lib/lbt/analyzer/ComponentAnalyzer.js

@@ -20,8 +20,8 @@
  */
 
 import {fromUI5LegacyName} from "../utils/ModuleName.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:ComponentAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:ComponentAnalyzer");
 
 // ---------------------------------------------------------------------------------------------------------
 

package/lib/lbt/analyzer/FioriElementsAnalyzer.js

@@ -61,8 +61,8 @@ import {fromUI5LegacyName} from "../utils/ModuleName.js";
 import SapUiDefine from "../calls/SapUiDefine.js";
 import {parseJS, Syntax} from "../utils/parseUtils.js";
 import {getValue, isMethodCall, getStringValue} from "../utils/ASTUtils.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:FioriElementAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:FioriElementAnalyzer");
 
 // ---------------------------------------------------------------------------------------------------------
 

package/lib/lbt/analyzer/JSModuleAnalyzer.js

@@ -13,9 +13,9 @@ import {
 	getStringValue,
 } from "../utils/ASTUtils.js";
 
-import logger from "@ui5/logger";
+import {getLogger} from "@ui5/logger";
 
-const log = logger.getLogger("lbt:analyzer:JSModuleAnalyzer");
+const log = getLogger("lbt:analyzer:JSModuleAnalyzer");
 
 // ------------------------------------------------------------------------------------------------------------------
 

package/lib/lbt/analyzer/SmartTemplateAnalyzer.js

@@ -28,8 +28,8 @@ import {fromUI5LegacyName} from "../utils/ModuleName.js";
 import SapUiDefine from "../calls/SapUiDefine.js";
 import {parseJS, Syntax} from "../utils/parseUtils.js";
 import {getValue, isMethodCall, getStringValue} from "../utils/ASTUtils.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:SmartTemplateAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:SmartTemplateAnalyzer");
 
 // ---------------------------------------------------------------------------------------------------------
 

package/lib/lbt/analyzer/XMLCompositeAnalyzer.js

@@ -3,8 +3,8 @@ import {Syntax} from "../utils/parseUtils.js";
 import SapUiDefine from "../calls/SapUiDefine.js";
 import {getValue, isMethodCall, getStringValue} from "../utils/ASTUtils.js";
 import {fromUI5LegacyName} from "../utils/ModuleName.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:XMLCompositeAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:XMLCompositeAnalyzer");
 
 const CALL_DEFINE = ["define"];
 const CALL_SAP_UI_DEFINE = ["sap", "ui", "define"];

package/lib/lbt/analyzer/XMLTemplateAnalyzer.js

@@ -2,8 +2,8 @@
 import xml2js from "xml2js";
 import {fromUI5LegacyName, fromRequireJSName} from "../utils/ModuleName.js";
 import JSTokenizer from "../utils/JSTokenizer.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:XMLTemplateAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:XMLTemplateAnalyzer");
 
 // ---------------------------------------------------------------------------------------------------------
 

package/lib/lbt/analyzer/analyzeLibraryJS.js

@@ -1,8 +1,8 @@
 
 import {parseJS, Syntax, VisitorKeys} from "../utils/parseUtils.js";
 import {getPropertyKey, isMethodCall, isIdentifier, getStringArray} from "../utils/ASTUtils.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:analyzer:LibraryJS");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:analyzer:LibraryJS");
 
 const CALL__SAP_UI_GETCORE = ["sap", "ui", "getCore"];
 

package/lib/lbt/bundle/AutoSplitter.js

@@ -3,8 +3,8 @@ import {pd} from "pretty-data";
 import {toRequireJSName} from "../utils/ModuleName.js";
 import {SectionType} from "./BundleDefinition.js";
 import escapePropertiesFile from "../utils/escapePropertiesFile.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:bundle:AutoSplitter");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:bundle:AutoSplitter");
 
 const xmlHtmlPrePattern = /<(?:\w+:)?pre\b/;
 

package/lib/lbt/bundle/Builder.js

@@ -15,8 +15,8 @@ import BundleResolver from "./Resolver.js";
 import BundleSplitter from "./AutoSplitter.js";
 import {SectionType} from "./BundleDefinition.js";
 import BundleWriter from "./BundleWriter.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:bundle:Builder");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:bundle:Builder");
 
 const sourceMappingUrlPattern = /\/\/# sourceMappingURL=(.+)\s*$/;
 const httpPattern = /^https?:\/\//i;

package/lib/lbt/bundle/Resolver.js

@@ -7,8 +7,8 @@ import {getRendererName} from "../UI5ClientConstants.js";
 import ResourceFilterList from "../resources/ResourceFilterList.js";
 import {SectionType} from "./BundleDefinition.js";
 import ResolvedBundleDefinition from "./ResolvedBundleDefinition.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:bundle:Resolver");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:bundle:Resolver");
 
 let dependencyTracker;
 

package/lib/lbt/graph/dependencyGraph.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:graph:dependencyGraph");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:graph:dependencyGraph");
 
 class Node {
 	constructor(name) {

package/lib/lbt/graph/dominatorTree.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:graph:dominatorTree");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:graph:dominatorTree");
 
 /**
  * Creates a dependency graph starting from the given set of root modules.

package/lib/lbt/graph/topologicalSort.js

@@ -1,6 +1,6 @@
 
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:graph:topologicalSort");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:graph:topologicalSort");
 
 /**
  * Represents a module and its dependencies in a dependency graph.

package/lib/lbt/resources/LibraryFileAnalyzer.js

@@ -3,8 +3,8 @@
  * It does not (yet) analyze dependencies of the library and therefore isn't part of the analyzer package (yet).
  */
 import xml2js from "xml2js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:resources:LibraryFileAnalyzer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:resources:LibraryFileAnalyzer");
 
 const parser = new xml2js.Parser({
 	// explicitChildren: true,

package/lib/lbt/resources/ResourceCollector.js

@@ -1,8 +1,8 @@
 import ResourceInfoList from "./ResourceInfoList.js";
 import ResourceFilterList from "./ResourceFilterList.js";
 import ResourceInfo from "./ResourceInfo.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:resources:ResourceCollector");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:resources:ResourceCollector");
 
 const LOCALE = /^((?:[^/]+\/)*[^/]+?)_([A-Z]{2}(?:_[A-Z]{2}(?:_[A-Z0-9_]+)?)?)(\.properties|\.hdbtextbundle)$/i;
 const THEME = /^((?:[^/]+\/)*)themes\/([^/]+)\//;

package/lib/lbt/resources/ResourceFilterList.js

@@ -1,6 +1,6 @@
 
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:resources:ResourceFilterList");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:resources:ResourceFilterList");
 
 const FILTER_PREFIXES = /^[-!+]/;
 

package/lib/lbt/resources/ResourcePool.js

@@ -8,8 +8,8 @@ import XMLTemplateAnalyzer from "../analyzer/XMLTemplateAnalyzer.js";
 import {getDependencyInfos} from "./LibraryFileAnalyzer.js";
 import ModuleInfo from "./ModuleInfo.js";
 import ResourceFilterList from "./ResourceFilterList.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("lbt:resources:ResourcePool");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("lbt:resources:ResourcePool");
 /*
 import Resource from "./Resource";
  */

package/lib/processors/bootstrapHtmlTransformer.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:bootstrapHtmlTransformer");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:bootstrapHtmlTransformer");
 import cheerio from "cheerio";
 
 /**

package/lib/processors/bundlers/flexChangesBundler.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:bundlers:flexChangesBundler");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:bundlers:flexChangesBundler");
 import {createResource} from "@ui5/fs/resourceFactory";
 
 /**

package/lib/processors/bundlers/moduleBundler.js

@@ -1,8 +1,8 @@
 import BundleBuilder from "../../lbt/bundle/Builder.js";
 import LocatorResourcePool from "../../lbt/resources/LocatorResourcePool.js";
 import EvoResource from "@ui5/fs/Resource";
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:bundlers:moduleBundler");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:bundlers:moduleBundler");
 
 /**
  * @public

package/lib/processors/jsdoc/lib/createIndexFiles.cjs

@@ -1,7 +1,7 @@
 /*
  * Node script to create cross-library API index files for use in the UI5 SDKs.
  *
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 

package/lib/processors/jsdoc/lib/transformApiJson.cjs

@@ -1,7 +1,7 @@
 /*
  * Node script to preprocess api.json files for use in the UI5 SDKs.
  *
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 

package/lib/processors/jsdoc/lib/ui5/plugin.cjs

@@ -1,7 +1,7 @@
 /*
  * JSDoc3 plugin for UI5 documentation generation.
  *
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -21,12 +21,18 @@
  *   experimental
  *
  *   final
+ * 
+ *   hideconstructor
  *
  *   interface
  *
  *   implements
  *
+ *   ui5-omissible-param
  *
+ *   ui5-restricted
+ *
+ * 
  *
  * It furthermore listens to the following JSDoc3 events to implement additional functionality
  *
@@ -2594,6 +2600,13 @@ exports.defineTags = function(dictionary) {
 			}
 		}
 	});
+	dictionary.defineTag('ui5-omissible-params', {
+		onTagged: function(doclet, tag) {
+			if ( tag.value ) {
+				ui5data(doclet).omissibleParams = tag.value.trim().split(/\s*,\s*/);
+			}
+		}
+	});
 
 	/**
 	 * Mark a doclet as synthetic.
@@ -3151,13 +3164,19 @@ exports.astNodeVisitor = {
 					typeString += `(${paramTypes.join(", ")})`;
 				}
 				if (parsedType.result) {
-					typeString += `:${toTypeString(parsedType.result)}`;
+					let resultType = toTypeString(parsedType.result);
+					// ensure the function result remains belonging together even when a union of the entire function with other types is created
+					// Example: ensure parentheses around function result in:  function(int):({rows: int, columns: int}|null)|undefined
+					if (parsedType.result.type === "TypeUnion") {
+						resultType = `(${resultType})`;
+					}
+					typeString += `:${resultType}`;
 				}
 				types.push(typeString);
 				// #### END: MODIFIED BY SAP
 				break;
 			 case TYPES.NameExpression:
-				 types.push(parsedType.name);
+				types.push(parsedType.name);
 				break;
 			case TYPES.NullLiteral:
 				types.push('null');

package/lib/processors/jsdoc/lib/ui5/template/publish.cjs

@@ -1,7 +1,7 @@
 /*
  * JSDoc3 template for UI5 documentation generation.
  *
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 
@@ -2856,6 +2856,7 @@ function createAPIJSON4Symbol(symbol, omitDefaults) {
 			}
 		}
 
+		const omissibleParams = new Set(member.__ui5 && member.__ui5.omissibleParams);
 		if ( member.params && member.params.length > 0 ) {
 			collection("parameters");
 			for ( let j = 0; j < member.params.length; j++) {
@@ -2867,6 +2868,13 @@ function createAPIJSON4Symbol(symbol, omitDefaults) {
 				attrib("name", param.name);
 				attrib("type", listTypes(param.type));
 				attrib("optional", !!param.optional, false, /* raw = */true);
+				if ( omissibleParams.has(param.name) ) {
+					if ( !param.optional ) {
+						throw new Error(`@param ${param.name} is specified in '@ui5-omissible-params' for '${member.name}', but not marked as optional. Only optional params can be omissible.`);
+					}
+					attrib("omissible", true, false, /* raw = */true);
+					omissibleParams.delete(param.name);
+				}
 				if ( param.variable ) {
 					attrib("repeatable", !!param.variable, false, /* raw = */true);
 				}
@@ -2882,6 +2890,9 @@ function createAPIJSON4Symbol(symbol, omitDefaults) {
 			}
 			endCollection("parameters");
 		}
+		if ( omissibleParams.size > 0 ) {
+			throw new Error(`parameter(s) '${[...omissibleParams].join("' and '")}' specified as '@ui5-omissible-params' for '${member.name}' missing among the actual @params`);
+		} 
 
 		exceptions(member);
 

package/lib/processors/libraryLessGenerator.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:libraryLessGenerator");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:libraryLessGenerator");
 
 import {promisify} from "node:util";
 import posixPath from "node:path/posix";

package/lib/processors/manifestCreator.js

@@ -2,8 +2,8 @@
 import posixPath from "node:path/posix";
 import semver from "semver";
 const {SemVer: Version} = semver;
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:manifestCreator");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:manifestCreator");
 import Resource from "@ui5/fs/Resource";
 import xml2js from "xml2js";
 import analyzeLibraryJS from "../lbt/analyzer/analyzeLibraryJS.js";

package/lib/processors/minifier.js

@@ -1,21 +1,43 @@
+import {fileURLToPath} from "node:url";
 import posixPath from "node:path/posix";
-import {minify} from "terser";
+import os from "node:os";
+import workerpool from "workerpool";
 import Resource from "@ui5/fs/Resource";
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:minifier");
 
-/**
- * Preserve comments which contain:
- * <ul>
- *  <li>copyright notice</li>
- *  <li>license terms</li>
- *  <li>"@ui5-bundle"</li>
- *  <li>"@ui5-bundle-raw-include"</li>
- * </ul>
- *
- * @type {RegExp}
- */
-const copyrightCommentsAndBundleCommentPattern = /copyright|\(c\)(?:[0-9]+|\s+[0-9A-Za-z])|released under|license|\u00a9|^@ui5-bundle-raw-include |^@ui5-bundle /i;
 const debugFileRegex = /((?:\.view|\.fragment|\.controller|\.designtime|\.support)?\.js)$/;
 
+const MIN_WORKERS = 2;
+const MAX_WORKERS = 4;
+const osCpus = os.cpus().length || 1;
+const maxWorkers = Math.max(Math.min(osCpus - 1, MAX_WORKERS), MIN_WORKERS);
+
+// Shared workerpool across all executions until the taskUtil cleanup is triggered
+let pool;
+
+function getPool(taskUtil) {
+	if (!pool) {
+		log.verbose(`Creating workerpool with up to ${maxWorkers} workers (available CPU cores: ${osCpus})`);
+		const workerPath = fileURLToPath(new URL("./minifierWorker.js", import.meta.url));
+		pool = workerpool.pool(workerPath, {
+			workerType: "auto",
+			maxWorkers
+		});
+		taskUtil.registerCleanupTask(() => {
+			log.verbose(`Terminating workerpool`);
+			const poolToBeTerminated = pool;
+			pool = null;
+			poolToBeTerminated.terminate();
+		});
+	}
+	return pool;
+}
+
+async function minifyInWorker(options, taskUtil) {
+	return getPool(taskUtil).exec("execMinification", [options]);
+}
+
 /**
  * @public
  * @module @ui5/builder/processors/minifier
@@ -40,13 +62,29 @@ const debugFileRegex = /((?:\.view|\.fragment|\.controller|\.designtime|\.suppor
  *
  * @param {object} parameters Parameters
  * @param {@ui5/fs/Resource[]} parameters.resources List of resources to be processed
+ * @param {@ui5/builder/tasks/TaskUtil|object} [parameters.taskUtil] TaskUtil instance.
+ *    Required when using the <code>useWorkers</code> option
  * @param {object} [parameters.options] Options
  * @param {boolean} [parameters.options.addSourceMappingUrl=true]
- * 				Whether to add a sourceMappingURL reference to the end of the minified resource
+ *   Whether to add a sourceMappingURL reference to the end of the minified resource
+ * @param {boolean} [parameters.options.useWorkers=false]
+ *  Whether to offload the minification task onto separate CPU threads. This often speeds up the build process
  * @returns {Promise<module:@ui5/builder/processors/minifier~MinifierResult[]>}
- * 				Promise resolving with object of resource, dbgResource and sourceMap
+ *   Promise resolving with object of resource, dbgResource and sourceMap
  */
-export default async function({resources, options: {addSourceMappingUrl = true} = {}}) {
+export default async function({resources, taskUtil, options: {addSourceMappingUrl = true, useWorkers = false} = {}}) {
+	let minify;
+	if (useWorkers) {
+		if (!taskUtil) {
+			// TaskUtil is required for worker support
+			throw new Error(`Minifier: Option 'useWorkers' requires a taskUtil instance to be provided`);
+		}
+		minify = minifyInWorker;
+	} else {
+		// Do not use workerpool
+		minify = (await import("./minifierWorker.js")).default;
+	}
+
 	return Promise.all(resources.map(async (resource) => {
 		const dbgPath = resource.getPath().replace(debugFileRegex, "-dbg$1");
 		const dbgResource = await resource.clone();
@@ -54,43 +92,26 @@ export default async function({resources, options: {addSourceMappingUrl = true}
 
 		const filename = posixPath.basename(resource.getPath());
 		const code = await resource.getString();
-		try {
-			const sourceMapOptions = {
-				filename
-			};
-			if (addSourceMappingUrl) {
-				sourceMapOptions.url = filename + ".map";
-			}
-			const dbgFilename = posixPath.basename(dbgPath);
-			const result = await minify({
-				// Use debug-name since this will be referenced in the source map "sources"
-				[dbgFilename]: code
-			}, {
-				output: {
-					comments: copyrightCommentsAndBundleCommentPattern,
-					wrap_func_args: false
-				},
-				compress: false,
-				mangle: {
-					reserved: [
-						"jQuery",
-						"jquery",
-						"sap",
-					]
-				},
-				sourceMap: sourceMapOptions
-			});
-			resource.setString(result.code);
-			const sourceMapResource = new Resource({
-				path: resource.getPath() + ".map",
-				string: result.map
-			});
-			return {resource, dbgResource, sourceMapResource};
-		} catch (err) {
-			// Note: err.filename contains the debug-name
-			throw new Error(
-				`Minification failed with error: ${err.message} in file ${filename} ` +
-				`(line ${err.line}, col ${err.col}, pos ${err.pos})`);
+
+		const sourceMapOptions = {
+			filename
+		};
+		if (addSourceMappingUrl) {
+			sourceMapOptions.url = filename + ".map";
 		}
+		const dbgFilename = posixPath.basename(dbgPath);
+
+		const result = await minify({
+			filename,
+			dbgFilename,
+			code,
+			sourceMapOptions
+		}, taskUtil);
+		resource.setString(result.code);
+		const sourceMapResource = new Resource({
+			path: resource.getPath() + ".map",
+			string: result.map
+		});
+		return {resource, dbgResource, sourceMapResource};
 	}));
 }

package/lib/processors/minifierWorker.js

@@ -0,0 +1,77 @@
+import workerpool from "workerpool";
+import {minify} from "terser";
+
+/**
+ * @private
+ * @module @ui5/builder/tasks/minifyWorker
+ */
+
+/**
+ * Preserve comments which contain:
+ * <ul>
+ *  <li>copyright notice</li>
+ *  <li>license terms</li>
+ *  <li>"@ui5-bundle"</li>
+ *  <li>"@ui5-bundle-raw-include"</li>
+ * </ul>
+ *
+ * @type {RegExp}
+ */
+const copyrightCommentsAndBundleCommentPattern = /copyright|\(c\)(?:[0-9]+|\s+[0-9A-Za-z])|released under|license|\u00a9|^@ui5-bundle-raw-include |^@ui5-bundle /i;
+
+/**
+ * Task to minify resources.
+ *
+ * @private
+ * @function default
+ * @static
+ *
+ * @param {object} parameters Parameters
+ * @param {string} parameters.filename
+ * @param {string} parameters.dbgFilename
+ * @param {string} parameters.code
+ * @param {object} parameters.sourceMapOptions
+ * @returns {Promise<undefined>} Promise resolving once minification of the resource has finished
+ */
+export default async function execMinification({
+	filename,
+	dbgFilename,
+	code,
+	sourceMapOptions
+}) {
+	try {
+		return await minify({
+			// Use debug-name since this will be referenced in the source map "sources"
+			[dbgFilename]: code
+		}, {
+			output: {
+				comments: copyrightCommentsAndBundleCommentPattern,
+				wrap_func_args: false
+			},
+			compress: false,
+			mangle: {
+				reserved: [
+					"jQuery",
+					"jquery",
+					"sap",
+				]
+			},
+			sourceMap: sourceMapOptions
+		});
+	} catch (err) {
+		// Note: err.filename contains the debug-name
+		throw new Error(
+			`Minification failed with error: ${err.message} in file ${filename} ` +
+			`(line ${err.line}, col ${err.col}, pos ${err.pos})`);
+	}
+}
+
+// Test execution via ava is never done on the main thread
+/* istanbul ignore else */
+if (!workerpool.isMainThread) {
+	// Script got loaded through workerpool
+	// => Create a worker and register public functions
+	workerpool.worker({
+		execMinification
+	});
+}

package/lib/processors/resourceListCreator.js

@@ -1,6 +1,6 @@
 
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:resourceListCreator");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:resourceListCreator");
 import ResourceCollector from "../lbt/resources/ResourceCollector.js";
 import LocatorResourcePool from "../lbt/resources/LocatorResourcePool.js";
 import ResourceInfo from "../lbt/resources/ResourceInfo.js";

package/lib/processors/themeBuilder.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:themeBuilder");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:themeBuilder");
 import posixPath from "node:path/posix";
 import less from "less-openui5";
 import Resource from "@ui5/fs/Resource";

package/lib/processors/versionInfoGenerator.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:processors:versionInfoGenerator");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:processors:versionInfoGenerator");
 import {createResource} from "@ui5/fs/resourceFactory";
 import posixPath from "node:path/posix";
 

package/lib/tasks/buildThemes.js

@@ -2,8 +2,8 @@ import path from "node:path";
 import themeBuilder from "../processors/themeBuilder.js";
 import fsInterface from "@ui5/fs/fsInterface";
 import ReaderCollectionPrioritized from "@ui5/fs/ReaderCollectionPrioritized";
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:buildThemes");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:buildThemes");
 
 /**
  * @public

package/lib/tasks/bundlers/generateComponentPreload.js

@@ -1,7 +1,7 @@
 import path from "node:path";
 import moduleBundler from "../../processors/bundlers/moduleBundler.js";
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:bundlers:generateComponentPreload");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:bundlers:generateComponentPreload");
 import {negateFilters} from "../../lbt/resources/ResourceFilterList.js";
 
 /**

package/lib/tasks/bundlers/generateFlexChangesBundle.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:bundlers:generateFlexChangesBundle");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:bundlers:generateFlexChangesBundle");
 import flexChangesBundler from "../../processors/bundlers/flexChangesBundler.js";
 import semver from "semver";
 

package/lib/tasks/bundlers/generateLibraryPreload.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:bundlers:generateLibraryPreload");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:bundlers:generateLibraryPreload");
 import moduleBundler from "../../processors/bundlers/moduleBundler.js";
 import {negateFilters} from "../../lbt/resources/ResourceFilterList.js";
 import createModuleNameMapping from "./utils/createModuleNameMapping.js";

package/lib/tasks/bundlers/generateStandaloneAppBundle.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:bundlers:generateStandaloneAppBundle");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:bundlers:generateStandaloneAppBundle");
 import ReaderCollectionPrioritized from "@ui5/fs/ReaderCollectionPrioritized";
 import moduleBundler from "../../processors/bundlers/moduleBundler.js";
 import createModuleNameMapping from "./utils/createModuleNameMapping.js";

package/lib/tasks/generateCachebusterInfo.js

@@ -1,7 +1,7 @@
 import crypto from "node:crypto";
 import {createResource} from "@ui5/fs/resourceFactory";
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:generateCachebusterInfo");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:generateCachebusterInfo");
 
 /**
  * @public

package/lib/tasks/generateLibraryManifest.js

@@ -1,6 +1,6 @@
 
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:generateLibraryManifest");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:generateLibraryManifest");
 import manifestCreator from "../processors/manifestCreator.js";
 
 /**

package/lib/tasks/generateThemeDesignerResources.js

@@ -1,6 +1,6 @@
 import posixPath from "node:path/posix";
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:generateThemeDesignerResources");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:generateThemeDesignerResources");
 import libraryLessGenerator from "../processors/libraryLessGenerator.js";
 import ReaderCollectionPrioritized from "@ui5/fs/ReaderCollectionPrioritized";
 import Resource from "@ui5/fs/Resource";

package/lib/tasks/jsdoc/executeJsdocSdkTransformation.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:jsdoc:executeJsdocSdkTransformation");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:jsdoc:executeJsdocSdkTransformation");
 import ReaderCollectionPrioritized from "@ui5/fs/ReaderCollectionPrioritized";
 import fsInterface from "@ui5/fs/fsInterface";
 import sdkTransformer from "../../processors/jsdoc/sdkTransformer.js";

package/lib/tasks/jsdoc/generateJsdoc.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:jsdoc:generateJsdoc");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:jsdoc:generateJsdoc");
 import path from "node:path";
 import os from "node:os";
 import fs from "graceful-fs";

package/lib/tasks/minify.js

@@ -25,8 +25,10 @@ export default async function({workspace, taskUtil, options: {pattern, omitSourc
 	const resources = await workspace.byGlob(pattern);
 	const processedResources = await minifier({
 		resources,
+		taskUtil,
 		options: {
-			addSourceMappingUrl: !omitSourceMapResources
+			addSourceMappingUrl: !omitSourceMapResources,
+			useWorkers: !!taskUtil,
 		}
 	});
 

package/lib/tasks/transformBootstrapHtml.js

@@ -1,5 +1,5 @@
-import logger from "@ui5/logger";
-const log = logger.getLogger("builder:tasks:transformBootstrapHtml");
+import {getLogger} from "@ui5/logger";
+const log = getLogger("builder:tasks:transformBootstrapHtml");
 import bootstrapHtmlTransformer from "../processors/bootstrapHtmlTransformer.js";
 
 /* eslint "jsdoc/check-param-names": ["error", {"disableExtraPropertyReporting":true}] */

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ui5/builder",
-	"version": "3.0.0-rc.2",
+	"version": "3.0.0-rc.4",
 	"description": "UI5 Tooling - Builder",
 	"author": {
 		"name": "SAP SE",
@@ -28,7 +28,7 @@
 		"./internal/jsdoc/template/publish": "./lib/processors/jsdoc/lib/ui5/template/publish.cjs"
 	},
 	"engines": {
-		"node": "^16.18.0 || >=18.0.0",
+		"node": "^16.18.0 || >=18.12.0",
 		"npm": ">= 8"
 	},
 	"scripts": {
@@ -118,8 +118,8 @@
 	},
 	"dependencies": {
 		"@jridgewell/sourcemap-codec": "^1.4.14",
-		"@ui5/fs": "^3.0.0-rc.3",
-		"@ui5/logger": "^3.0.1-rc.1",
+		"@ui5/fs": "^3.0.0-rc.5",
+		"@ui5/logger": "^3.0.1-rc.3",
 		"cheerio": "1.0.0-rc.12",
 		"escape-unicode": "^0.2.0",
 		"escope": "^4.0.0",
@@ -128,14 +128,15 @@
 		"jsdoc": "^3.6.11",
 		"less-openui5": "^0.11.6",
 		"pretty-data": "^0.40.0",
-		"rimraf": "^4.1.1",
+		"rimraf": "^4.1.2",
 		"semver": "^7.3.8",
 		"terser": "^5.16.1",
+		"workerpool": "^6.3.1",
 		"xml2js": "^0.4.23"
 	},
 	"devDependencies": {
 		"@istanbuljs/esm-loader-hook": "^0.2.0",
-		"@ui5/project": "^3.0.0-rc.3",
+		"@ui5/project": "^3.0.0-rc.7",
 		"ava": "^5.1.1",
 		"chai": "^4.3.7",
 		"chai-fs": "^2.0.0",
@@ -143,10 +144,10 @@
 		"cross-env": "^7.0.3",
 		"depcheck": "^1.4.3",
 		"docdash": "^2.0.1",
-		"eslint": "^8.32.0",
+		"eslint": "^8.33.0",
 		"eslint-config-google": "^0.14.0",
 		"eslint-plugin-ava": "^14.0.0",
-		"eslint-plugin-jsdoc": "^39.6.7",
+		"eslint-plugin-jsdoc": "^39.7.4",
 		"esmock": "^2.1.0",
 		"nyc": "^15.1.0",
 		"open-cli": "^7.1.0",