hotfix for buildTemplate failing + remove duplicate keys

bad variable reference

Author: JoernBerkefeld

lib/Builder.js

@@ -141,10 +141,6 @@ saved
             /** @type {TYPE.MetadataTypeItemObj[]} */
             const result = await Promise.all(
                 keyArr.map((key) => {
-                    // with npx and powershell spaces are not parsed correctly as part of a string
-                    // we hence require users to put %20 in their stead and have to convert that back
-                    key = key.split('%20').join(' ');
-
                     MetadataTypeInfo[type].client = this.client;
                     MetadataTypeInfo[type].properties = this.properties;
                     MetadataTypeInfo[type].buObject = this.buObject;

lib/util/devops.js

@@ -328,15 +328,16 @@ const DevOps = {
                     // init array
                     buTypeDelta[buPath][file.type] = [];
                 }
-                buTypeDelta[buPath][file.type].push(file.key);
+                buTypeDelta[buPath][file.type].push(file.externalKey);
             }
 
             // Run buildTemplate for each business unit for each type
             Util.logger.info('Retrieve template from Git delta');
             // ! needs to be for (.. in ..) loop so that it gets executed in series
             for (const bu in buTypeDelta) {
                 for (const type in buTypeDelta[bu]) {
-                    const keyArr = buTypeDelta[bu][type];
+                    // get unique list (original search might include more than one entry for types with docs or extracted code)
+                    const keyArr = [...new Set(buTypeDelta[bu][type])];
                     Util.logger.info(
                         `⚡ mcdev bt ${bu} ${type} "${keyArr.join(',')}" ${sourceMarket}`
                     );