[INTERNAL] lib/processors/jsdoc: Enhance visualization of multiple types

Background:
The jsdoc allows to specify multiple types for:
- function parameters (including constructor parameters)
- function return values
- properties of typedefs e.g. typedef sap/ui/performance/Measurement.Entry
- UI5 metadata properties
Then transformApiJson.js adds those multiple types
into the "types" field of the entries of the api.json
files for each library

Problem:
The UI5 Demokit did not correctly display multiple types
for UI5 metadata properties e.g. it created a single
link for all types that opened the Not Found page

Solution:
- in transformApiJson.js, assign the "linkedEnabled"
flag per type (instead of per group of types).
Further, set "linkedEnabled"=true only if the type
is a UI5 symbol.
- in the UI5 Demokit ApiRef section, for each displayed
control property, bind the view to the "types"
model field, to ensure all types are covered.

Cherry-picked from UI5/openui5@4cd0cc12d.

Author: codeworrior

lib/processors/jsdoc/lib/transformApiJson.js

@@ -37,12 +37,6 @@ function normalizeToUI5GlobalNotation(sModuleName){
 }
 
 
-function fnCreateTypesArr(sTypes) {
-	return sTypes.split("|").map(function (sType) {
-		return { value: sType }
-	});
-}
-
 /**
  * Transforms the api.json as created by the JSDoc build into a pre-processed api.json file suitable for the SDK.
  *
@@ -116,83 +110,98 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 		return func;
 	}
 
-	/**
-	 * Transforms api.json file
-	 * @param {object} oChainObject chain object
-	 */
-	let transformApiJson = function (oChainObject) {
-		/**
-		 * Check if a type is a built-in type, handling both simple
-		 * and compound types gracefully.
-		 *
-		 * @param {string} type A type to check
-		 * @returns {boolean} true if the type is built-in, otherwise false
-		 */
-		function isBuiltInType(type) {
-			const builtInTypes = formatters._baseTypes;
-
-			// Early return if the type is directly in baseTypes
-			if (builtInTypes.includes(type)) {
-				return true;
+	function fnCreateTypesArr(sTypes) {
+		return sTypes.split("|").map(function (sType) {
+			const bLinkEnabled = !isBuiltInType(sType) && possibleUI5Symbol(sType.replace(/\[\]$/, ""));
+			const oTypeInfo = { value: sType };
+			if (bLinkEnabled) {  // default is false, so omit if not required
+				oTypeInfo.linkEnabled = true;
 			}
+			return oTypeInfo;
+		});
+	}
 
-			// Check if the type is a union type
-			if (type.includes("|")) {
-				const unionParts = type.split("|").map(part => part.trim());
-				return unionParts.every(part => isBuiltInType(part));
-			}
+	/**
+	 * Check if a type is a built-in type, handling both simple
+	 * and compound types gracefully.
+	 *
+	 * @param {string} type A type to check
+	 * @returns {boolean} true if the type is built-in, otherwise false
+	 */
+	function isBuiltInType(type) {
+		const builtInTypes = formatters._baseTypes;
 
-			// Handle array notation directly
-			if (type.endsWith("[]")) {
-				return builtInTypes.includes(type.slice(0, -2));
-			}
+		// Early return if the type is directly in baseTypes
+		if (builtInTypes.includes(type)) {
+			return true;
+		}
 
-			// Predefined regex patterns for reuse
-			const arrayRegex = /Array<(.+)>/;
-			const arrayDotRegex = /Array\.<(.+)>/;
-			const objectRegex = /Object<([^,]+),([^>]+)>/;
-			const objectDotRegex = /Object\.<([^,]+),([^>]+)>/;
-
-			// Check if the type is a generic Array type
-			const arrayMatch = arrayRegex.exec(type);
-			if (arrayMatch) {
-				const innerType = arrayMatch[1];
-				return isBuiltInType(innerType);
-			}
+		// Check if the type is a union type
+		if (type.includes("|")) {
+			const unionParts = type.split("|").map(part => part.trim());
+			return unionParts.every(part => isBuiltInType(part));
+		}
 
-			const arrayDotMatch = arrayDotRegex.exec(type);
-			if (arrayDotMatch) {
-				const innerType = arrayDotMatch[1];
-				return isBuiltInType(innerType);
-			}
+		// Handle array notation directly
+		if (type.endsWith("[]")) {
+			return builtInTypes.includes(type.slice(0, -2));
+		}
 
-			// Check if the type is a generic Object type
-			const objectMatch = objectRegex.exec(type);
-			if (objectMatch) {
-				const innerTypes = [objectMatch[1], objectMatch[2]].map(t => t.trim());
-				return innerTypes.every(innerType => isBuiltInType(innerType));
-			}
+		// Predefined regex patterns for reuse
+		const arrayRegex = /Array<(.+)>/;
+		const arrayDotRegex = /Array\.<(.+)>/;
+		const objectRegex = /Object<([^,]+),([^>]+)>/;
+		const objectDotRegex = /Object\.<([^,]+),([^>]+)>/;
+
+		// Check if the type is a generic Array type
+		const arrayMatch = arrayRegex.exec(type);
+		if (arrayMatch) {
+			const innerType = arrayMatch[1];
+			return isBuiltInType(innerType);
+		}
 
-			const objectDotMatch = objectDotRegex.exec(type);
-			if (objectDotMatch) {
-				const innerTypes = [objectDotMatch[1], objectDotMatch[2]].map(t => t.trim());
-				return innerTypes.every(innerType => isBuiltInType(innerType));
-			}
+		const arrayDotMatch = arrayDotRegex.exec(type);
+		if (arrayDotMatch) {
+			const innerType = arrayDotMatch[1];
+			return isBuiltInType(innerType);
+		}
 
-			// Fallback case: if none of the above matched, return false
-			return false;
+		// Check if the type is a generic Object type
+		const objectMatch = objectRegex.exec(type);
+		if (objectMatch) {
+			const innerTypes = [objectMatch[1], objectMatch[2]].map(t => t.trim());
+			return innerTypes.every(innerType => isBuiltInType(innerType));
 		}
 
-		/**
-		 * Heuristically determining if there is a possibility the given input string
-		 * to be a UI5 symbol
-		 * @param {string} sName
-		 * @returns {boolean}
-		 */
-		function possibleUI5Symbol(sName) {
-			return /^[a-zA-Z][a-zA-Z0-9.]*[a-zA-Z0-9]$/.test(sName);
+		const objectDotMatch = objectDotRegex.exec(type);
+		if (objectDotMatch) {
+			const innerTypes = [objectDotMatch[1], objectDotMatch[2]].map(t => t.trim());
+			return innerTypes.every(innerType => isBuiltInType(innerType));
 		}
 
+		// Fallback case: if none of the above matched, return false
+		return false;
+	}
+
+	/**
+	 * Heuristically determining if there is a possibility the given input string
+	 * to be a UI5 symbol
+	 * Examples of UI5 symbols:
+	 * -"sap.ui.core.date.UI5Date"
+	 * -"module:sap/ui/core/date/UI5Date"
+	 * @param {string} sName
+	 * @returns {boolean}
+	 */
+	 function possibleUI5Symbol(sName) {
+	 	 const ui5SymbolRegex = /^(module:)?[a-zA-Z][a-zA-Z0-9/.]*[a-zA-Z0-9]$/;
+	 	 return ui5SymbolRegex.test(sName);
+	 }
+
+	/**
+	 * Transforms api.json file
+	 * @param {object} oChainObject chain object
+	 */
+	let transformApiJson = function (oChainObject) {
 		// Function is a copy from: LibraryInfo.js => LibraryInfo.prototype._getActualComponent => "match" inline method
 		function matchComponent(sModuleName, sPattern) {
 			sModuleName = sModuleName.toLowerCase();
@@ -364,15 +373,7 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 						// Types
 						oParameter.types = [];
 						if (oParameter.type) {
-							let aTypes = oParameter.type.split("|");
-
-							for (let i = 0; i < aTypes.length; i++) {
-								oParameter.types.push({
-									name: aTypes[i],
-									linkEnabled: !isBuiltInType(aTypes[i])
-								});
-							}
-
+							oParameter.types = fnCreateTypesArr(oParameter.type);
 							// Keep file size in check
 							delete oParameter.type;
 						}
@@ -457,17 +458,9 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 					// Type
 					if (oSymbol.kind !== "enum") { // enum properties don't have an own type
 						if (oProperty.type) {
-							oProperty.types = oProperty.type.split("|").map((sType) => {
-								const oType = {
-									value: sType
-								};
-								// Link Enabled
-								if (!isBuiltInType(oType.value) && possibleUI5Symbol(oType.value)) {
-									oType.linkEnabled = true;
-									oType.href = "api/" + oType.value.replace("[]", "");
-								}
-								return oType;
-							});
+							oProperty.types = fnCreateTypesArr(oProperty.type);
+							// Keep file size in check
+							delete oProperty.type;
 						}
 					}
 
@@ -506,16 +499,13 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 						// Type
 						if (oProperty.type) {
 							oProperty.types = fnCreateTypesArr(oProperty.type);
+							// Keep file size in check
+							delete oProperty.type;
 						}
 
 						// Description
 						oProperty.description = formatters.formatDescriptionSince(oProperty.description, oProperty.since);
 
-						// Link Enabled
-						if (!isBuiltInType(oProperty.type)) {
-							oProperty.linkEnabled = true;
-						}
-
 						// Default value
 						oProperty.defaultValue = formatters.formatDefaultValue(oProperty.defaultValue);
 
@@ -756,19 +746,6 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 					if (oMethod.parameters) {
 						oMethod.parameters.forEach(oParameter => {
 
-							// Types
-							if (oParameter.types) {
-								oParameter.types.forEach(oType => {
-
-									// Link Enabled
-									if (!isBuiltInType(oType.value) && possibleUI5Symbol(oType.value)) {
-										oType.linkEnabled = true;
-										oType.href = "api/" + oType.value.replace("[]", "");
-									}
-
-								});
-							}
-
 							// Default value
 							oParameter.defaultValue = formatters.formatDefaultValue(oParameter.defaultValue);
 
@@ -789,19 +766,6 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 						// Description
 						oMethod.returnValue.description = formatters.formatDescription(oMethod.returnValue.description);
 
-						// Types
-						if (oMethod.returnValue.types) {
-							oMethod.returnValue.types.forEach(oType => {
-
-								// Link Enabled
-								if (!isBuiltInType(oType.value) && possibleUI5Symbol(oType.value)) {
-									oType.href = "api/" + oType.value.replace("[]", "");
-									oType.linkEnabled = true;
-								}
-
-							});
-						}
-
 					}
 
 					// Throws
@@ -2129,6 +2093,8 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 						// Handle types
 						if (oProperty.type) {
 							oProperty.types = fnCreateTypesArr(oProperty.type);
+							// Keep file size in check
+							delete oProperty.type;
 						}
 
 						// Phone name - available only for parameters
@@ -2159,11 +2125,10 @@ function transformer(sInputFile, sOutputFile, sLibraryFile, vDependencyAPIFiles,
 					oMethod.parameters.forEach(function (oParameter) {
 						if (oParameter.type) {
 							oParameter.types = fnCreateTypesArr(oParameter.type);
+							// Keep file size in check
+							delete oParameter.type;
 						}
 
-						// Keep file size in check
-						delete oParameter.type;
-
 						// Add the parameter before the properties
 						aParameters.push(oParameter);