[INTERNAL] lib/processors/jsdoc: support property names that are not identifiers

JSDoc allows to quote property names when they contain characters that
are not valid in identifiers. So far, the UI5 template failed to support
such names, esp. when used in nested parameter properties.

With this change, support for such names is added.

Cherry-picked from UI5/openui5@9b2a1914b.

Author: codeworrior

lib/processors/jsdoc/lib/ui5/template/publish.js

@@ -889,6 +889,13 @@ function normalizeWS(text) {
 
 }
 
+const rStartsWithValidPropertyName = /^(?:[^'"][^\s.]+|'(?:[^'\\]|\\.)*'|"(?:[^"\\]|\\.)*")/;
+
+function isNestedPropertyName(name) {
+	const match = rStartsWithValidPropertyName.exec(name);
+	return match && match[0].length < name.length && name[match[0].length] === ".";
+}
+
 //---- add on: API JSON -----------------------------------------------------------------
 
 function createAPIJSON(symbols, filename) {
@@ -1411,7 +1418,7 @@ function createAPIJSON4Symbol(symbol, omitDefaults) {
 				continue;
 			}
 
-			if ( name.indexOf('.') >= 0 ) {
+			if ( isNestedPropertyName(name) ) {
 				continue;
 			}
 
@@ -2276,6 +2283,19 @@ function validateAPIJSON(api) {
 		}
 	}
 
+	/**
+	 * Checks the name of a property in an object-like structure.
+	 * As in JavaScript, the rules for such names are less strict than
+	 * for classes, interfaces or function declarations.
+	 */
+	function checkPropertyName(name, hint) {
+		const match = rStartsWithValidPropertyName.exec(name);
+		if ( match == null || match[0].length !== name.length ) {
+			naming[name] ??= [];
+			naming[name].push(hint);
+		}
+	}
+
 	function checkModuleName(name, hint) {
 		if ( !rValidModuleNames.test(name) ) {
 			naming[name] = naming[name] || [];
@@ -2386,7 +2406,12 @@ function validateAPIJSON(api) {
 	}
 
 	function checkParam(param, prefix, hint) {
-		checkName(param.name, `name of param ${prefix}${param.name} of ${hint}`);
+		if (prefix) {
+			// nested parameter properties follow the naming conventions for object properties
+			checkPropertyName(param.name, `name of param ${prefix}${param.name} of ${hint}`);
+		} else {
+			checkName(param.name, `name of param ${prefix}${param.name} of ${hint}`);
+		}
 		checkType(param, `param ${prefix}${param.name} of ${hint}`);
 		if ( param.parameterProperties ) {
 			Object.keys(param.parameterProperties).forEach((sub) =>
@@ -2508,7 +2533,7 @@ function validateAPIJSON(api) {
 		if ( symbol.properties ) {
 			symbol.properties.forEach((prop) => {
 				const qualifiedName = `${symbol.name}.${prop.name}`;
-				checkName(prop.name, `name of ${qualifiedName}`);
+				checkPropertyName(prop.name, `name of ${qualifiedName}`);
 				if ( prop.type ) {
 					checkType(prop, `type of ${qualifiedName}`);
 				}