@ui5/builder 3.0.0-alpha.5 → 3.0.0-alpha.8

package/lib/tasks/generateLibraryManifest.js

@@ -2,7 +2,6 @@
 
 const log = require("@ui5/logger").getLogger("builder:tasks:generateLibraryManifest");
 const manifestCreator = require("../processors/manifestCreator");
-const ReaderCollectionPrioritized = require("@ui5/fs").ReaderCollectionPrioritized;
 
 
 /**
@@ -12,23 +11,19 @@ const ReaderCollectionPrioritized = require("@ui5/fs").ReaderCollectionPrioritiz
  * @alias module:@ui5/builder.tasks.generateLibraryManifest
  * @param {object} parameters Parameters
  * @param {module:@ui5/fs.DuplexCollection} parameters.workspace DuplexCollection to read and write files
- * @param {module:@ui5/fs.AbstractReader} parameters.dependencies Reader or Collection to read dependency files
+ * @param {module:@ui5/builder.tasks.TaskUtil|object} [parameters.taskUtil] TaskUtil
  * @param {object} parameters.options Options
  * @param {string} parameters.options.projectName Project name
  * @returns {Promise<undefined>} Promise resolving with <code>undefined</code> once data has been written
  */
-module.exports = function({workspace, dependencies, options: {projectName}}) {
-	const combo = new ReaderCollectionPrioritized({
-		name: `libraryManifestGenerator - prioritize workspace over dependencies: ${projectName}`,
-		readers: [workspace, dependencies]
-	});
+module.exports = function({workspace, taskUtil, options: {projectName}}) {
 	// Note:
 	// *.library files are needed to identify libraries
 	// *.json files are needed to avoid overwriting them
 	// *.js files are needed to identify nested components
 	// *.less, *.css, *.theming and *.theme files are needed to identify supported themes
 	// *.properties to identify existence of i18n bundles (e.g. messagebundle.properties)
-	return combo.byGlob("/**/*.{js,json,library,less,css,theming,theme,properties}").then((resources) => {
+	return workspace.byGlob("/resources/**/*.{js,json,library,less,css,theming,theme,properties}").then((resources) => {
 		// Find all libraries and create a manifest.json file
 		return workspace.byGlob("/resources/**/.library").then((libraryIndicatorResources) => {
 			if (libraryIndicatorResources.length < 1) {
@@ -51,6 +46,9 @@ module.exports = function({workspace, dependencies, options: {projectName}}) {
 						libraryResource: libraryIndicatorResource,
 						namespace: libraryNamespace,
 						resources,
+						getProjectVersion: (projectName) => {
+							return taskUtil?.getProject(projectName)?.getVersion();
+						},
 						options: {
 						}
 					}).then((manifest) => {

package/lib/tasks/generateResourcesJson.js

@@ -43,7 +43,7 @@ function getCreatorOptions(projectName) {
  *
  * <p>
  * The detailed structure can be found in the documentation:
- * {@link https://openui5.hana.ondemand.com/#topic/adcbcf8b50924556ab3f321fcd9353ea}
+ * {@link https://sdk.openui5.org/topic/adcbcf8b50924556ab3f321fcd9353ea}
  * </p>
  *
  * <p>

package/lib/tasks/generateThemeDesignerResources.js

@@ -187,6 +187,7 @@ async function generateCssVariablesLess({workspace, combo, namespace}) {
 	}
 }
 
+/* eslint "jsdoc/check-param-names": ["error", {"disableExtraPropertyReporting":true}] */
 /**
  * Generates resources required for integration with the SAP Theme Designer.
  *
@@ -198,11 +199,16 @@ async function generateCssVariablesLess({workspace, combo, namespace}) {
  * @param {object} parameters.options Options
  * @param {string} parameters.options.projectName Project name
  * @param {string} parameters.options.version Project version
- * @param {string} [parameters.options.namespace] If the project is of type <code>library</code>, provide its namespace.
+ * @param {string} [parameters.options.projectNamespace] If the project is of type <code>library</code>,
+ * 														 provide its namespace.
  * Omit for type <code>theme-library</code>
  * @returns {Promise<undefined>} Promise resolving with <code>undefined</code> once data has been written
  */
-module.exports = async function({workspace, dependencies, options: {projectName, version, namespace}}) {
+module.exports = async function({workspace, dependencies, options}) {
+	const {projectName, version} = options;
+	// Backward compatibility: "namespace" option got renamed to "projectNamespace"
+	const namespace = options.projectNamespace || options.namespace;
+
 	// Skip sap.ui.documentation since it is not intended to be available in SAP Theme Designer to create custom themes
 	if (namespace === "sap/ui/documentation") {
 		return;

package/lib/tasks/generateVersionInfo.js

@@ -19,7 +19,7 @@ module.exports = async ({workspace, dependencies, options: {rootProject, pattern
 	const resources = await dependencies.byGlob(pattern);
 
 	const libraryInfosPromises = resources.map((dotLibResource) => {
-		const namespace = dotLibResource._project.metadata.namespace;
+		const namespace = dotLibResource.getProject().getNamespace();
 		// pass all required resources to the processor
 		// the processor will then filter
 		return dependencies.byGlob(`/resources/${namespace}/**/${MANIFEST_JSON}`).then((manifestResources) => {
@@ -31,8 +31,8 @@ module.exports = async ({workspace, dependencies, options: {rootProject, pattern
 			return {
 				libraryManifest,
 				embeddedManifests,
-				name: dotLibResource._project.metadata.name,
-				version: dotLibResource._project.version
+				name: dotLibResource.getProject().getName(),
+				version: dotLibResource.getProject().getVersion()
 			};
 		});
 	});
@@ -40,8 +40,8 @@ module.exports = async ({workspace, dependencies, options: {rootProject, pattern
 
 	const [versionInfoResource] = await versionInfoGenerator({
 		options: {
-			rootProjectName: rootProject.metadata.name,
-			rootProjectVersion: rootProject.version,
+			rootProjectName: rootProject.getName(),
+			rootProjectVersion: rootProject.getVersion(),
 			libraryInfos
 		}
 	});

package/lib/tasks/taskRepository.js

@@ -36,30 +36,18 @@ function getTask(taskName) {
 	try {
 		const task = require(taskInfo.path);
 		return {
-			task,
-			specVersion: taskInfo.specVersion
+			task
 		};
 	} catch (err) {
 		throw new Error(`taskRepository: Failed to require task module for ${taskName}: ${err.message}`);
 	}
 }
 
-function addTask({name, specVersion, taskPath}) {
-	if (taskInfos[name]) {
-		throw new Error(`taskRepository: A task with the name ${name} has already been registered`);
-	}
-	taskInfos[name] = {
-		path: taskPath,
-		specVersion
-	};
-}
-
 function getAllTaskNames() {
 	return Object.keys(taskInfos);
 }
 
 module.exports = {
 	getTask,
-	addTask,
 	getAllTaskNames
 };

package/lib/tasks/transformBootstrapHtml.js

@@ -1,6 +1,7 @@
 const log = require("@ui5/logger").getLogger("builder:tasks:transformBootstrapHtml");
 const bootstrapHtmlTransformer = require("../processors/bootstrapHtmlTransformer");
 
+/* eslint "jsdoc/check-param-names": ["error", {"disableExtraPropertyReporting":true}] */
 /**
  * Task for transforming the application bootstrap HTML file.
  *
@@ -12,7 +13,11 @@ const bootstrapHtmlTransformer = require("../processors/bootstrapHtmlTransformer
  * @param {string} [parameters.options.namespace] Project namespace
  * @returns {Promise<undefined>} Promise resolving with <code>undefined</code> once data has been written
  */
-module.exports = async function({workspace, options: {projectName, namespace}}) {
+module.exports = async function({workspace, options}) {
+	const {projectName} = options;
+	// Backward compatibility: "namespace" option got renamed to "projectNamespace"
+	const namespace = options.projectNamespace || options.namespace;
+
 	let indexPath;
 	if (namespace) {
 		indexPath = `/resources/${namespace}/index.html`;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ui5/builder",
-	"version": "3.0.0-alpha.5",
+	"version": "3.0.0-alpha.8",
 	"description": "UI5 Tooling - Builder",
 	"author": {
 		"name": "SAP SE",
@@ -40,7 +40,7 @@
 		"version": "git-chglog --sort semver --next-tag v$npm_package_version -o CHANGELOG.md && git add CHANGELOG.md",
 		"postversion": "git push --follow-tags",
 		"release-note": "git-chglog --sort semver -c .chglog/release-config.yml v$npm_package_version",
-		"depcheck": "depcheck --ignores docdash"
+		"depcheck": "depcheck --ignores docdash --ignore-patterns lib/processors/jsdoc/lib/ui5"
 	},
 	"files": [
 		"index.js",
@@ -105,28 +105,27 @@
 		"url": "git@github.com:SAP/ui5-builder.git"
 	},
 	"dependencies": {
-		"@ui5/fs": "^3.0.0-alpha.2",
+		"@jridgewell/sourcemap-codec": "^1.4.14",
+		"@ui5/fs": "^3.0.0-alpha.5",
 		"@ui5/logger": "^3.0.1-alpha.1",
 		"cheerio": "1.0.0-rc.9",
 		"escape-unicode": "^0.2.0",
 		"escope": "^3.6.0",
 		"espree": "^9.3.1",
-		"globby": "^11.1.0",
 		"graceful-fs": "^4.2.9",
 		"jsdoc": "^3.6.7",
 		"less-openui5": "^0.11.2",
 		"make-dir": "^3.1.0",
 		"pretty-data": "^0.40.0",
-		"pretty-hrtime": "^1.0.3",
 		"replacestream": "^4.0.3",
 		"rimraf": "^3.0.2",
 		"semver": "^7.3.5",
-		"sourcemap-codec": "^1.4.8",
-		"terser": "^5.10.0",
+		"terser": "^5.14.2",
 		"xml2js": "^0.4.23",
 		"yazl": "^2.5.1"
 	},
 	"devDependencies": {
+		"@ui5/project": "github:SAP/ui5-project#178dedbfe80ab890d134ce12ccb29fc709595a52",
 		"ava": "^3.15.0",
 		"chai": "^4.3.4",
 		"chai-fs": "^2.0.0",

package/lib/builder/BuildContext.js

@@ -1,60 +0,0 @@
-const ResourceTagCollection = require("@ui5/fs").ResourceTagCollection;
-const ProjectBuildContext = require("./ProjectBuildContext");
-
-// Note: When adding standard tags, always update the public documentation in TaskUtil
-// (Type "module:@ui5/builder.tasks.TaskUtil~StandardBuildTags")
-const GLOBAL_TAGS = Object.freeze({
-	IsDebugVariant: "ui5:IsDebugVariant",
-	HasDebugVariant: "ui5:HasDebugVariant",
-});
-
-/**
- * Context of a build process
- *
- * @private
- * @memberof module:@ui5/builder.builder
- */
-class BuildContext {
-	constructor({rootProject, options = {}}) {
-		if (!rootProject) {
-			throw new Error(`Missing parameter 'rootProject'`);
-		}
-		this.rootProject = rootProject;
-		this.projectBuildContexts = [];
-		this._resourceTagCollection = new ResourceTagCollection({
-			allowedTags: Object.values(GLOBAL_TAGS)
-		});
-		this.options = options;
-	}
-
-	getRootProject() {
-		return this.rootProject;
-	}
-
-	getOption(key) {
-		return this.options[key];
-	}
-
-	createProjectContext({project, resources}) {
-		const projectBuildContext = new ProjectBuildContext({
-			buildContext: this,
-			globalTags: GLOBAL_TAGS,
-			project,
-			resources
-		});
-		this.projectBuildContexts.push(projectBuildContext);
-		return projectBuildContext;
-	}
-
-	async executeCleanupTasks() {
-		await Promise.all(this.projectBuildContexts.map((ctx) => {
-			return ctx.executeCleanupTasks();
-		}));
-	}
-
-	getResourceTagCollection() {
-		return this._resourceTagCollection;
-	}
-}
-
-module.exports = BuildContext;

package/lib/builder/ProjectBuildContext.js

@@ -1,61 +0,0 @@
-const ResourceTagCollection = require("@ui5/fs").ResourceTagCollection;
-
-// Note: When adding standard tags, always update the public documentation in TaskUtil
-// (Type "module:@ui5/builder.tasks.TaskUtil~StandardBuildTags")
-const STANDARD_TAGS = {
-	OmitFromBuildResult: "ui5:OmitFromBuildResult",
-	IsBundle: "ui5:IsBundle",
-};
-
-/**
- * Build context of a single project. Always part of an overall
- * [Build Context]{@link module:@ui5/builder.builder.BuildContext}
- *
- * @private
- * @memberof module:@ui5/builder.builder
- */
-class ProjectBuildContext {
-	constructor({buildContext, globalTags, project, resources}) {
-		if (!buildContext || !globalTags || !project || !resources) {
-			throw new Error(`One or more mandatory parameters are missing`);
-		}
-		this._buildContext = buildContext;
-		this._project = project;
-		// this.resources = resources;
-		this.queues = {
-			cleanup: []
-		};
-
-		this.STANDARD_TAGS = Object.assign({}, STANDARD_TAGS, globalTags);
-		Object.freeze(this.STANDARD_TAGS);
-
-		this._resourceTagCollection = new ResourceTagCollection({
-			allowedTags: Object.values(this.STANDARD_TAGS),
-			superCollection: this._buildContext.getResourceTagCollection()
-		});
-	}
-
-	isRootProject() {
-		return this._project === this._buildContext.getRootProject();
-	}
-
-	getOption(key) {
-		return this._buildContext.getOption(key);
-	}
-
-	registerCleanupTask(callback) {
-		this.queues.cleanup.push(callback);
-	}
-
-	async executeCleanupTasks() {
-		await Promise.all(this.queues.cleanup.map((callback) => {
-			return callback();
-		}));
-	}
-
-	getResourceTagCollection() {
-		return this._resourceTagCollection;
-	}
-}
-
-module.exports = ProjectBuildContext;