@ui5/task-adaptation 1.1.0 → 1.1.2

package/dist/i18nManager.d.ts

@@ -1,4 +1,5 @@
-import { IDiffProperty, IJsonContent } from "./util/jsonDiffUtil";
+import { IDiffProperty } from "./annotations/comparator/comparator";
+import { IJsonPerLanguage, IJsonPromisePerLanguage } from "./model/types";
 import Language from "./model/language";
 export declare class PropertyValue {
     value: string;
@@ -31,12 +32,9 @@ export default class I18nManager {
     constructor(modelName: string, appVariantId: string, languages: Language[]);
     processDiff(properties: Set<IDiffProperty>, previousLanguage: Language, currentLanguage: Language): void;
     createFiles(i18nPathName: string): Map<string, string>;
-    populateTranslations(annotationJsons: Map<Language, any>): IJsonContent;
-    populate(annotationJsons: [Language, any][], defaultAnnotation: IJsonContent): {
-        language: Language;
-        json: any;
-    };
-    static extractDefaultLanguageAnnotation(annotationJsons: Map<Language, any>): IJsonContent;
+    populateTranslations(annotationJsons: Map<Language, Promise<any>>): Promise<IJsonPromisePerLanguage>;
+    populate(annotationJsons: [Language, Promise<any>][], defaultAnnotation: IJsonPerLanguage): Promise<IJsonPerLanguage>;
+    static extractDefaultLanguageAnnotation(annotationJsons: Map<Language, Promise<any>>): IJsonPromisePerLanguage;
     private replaceWithModelReference;
     private createReference;
     getUniqueKeyForValue(value: string): string;

package/dist/i18nManager.js

@@ -1,8 +1,9 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.I18nFileContent = exports.PropertyValue = void 0;
-const jsonDiffUtil_1 = require("./util/jsonDiffUtil");
+const comparator_1 = require("./annotations/comparator/comparator");
 const annotationDiffStructureError_1 = require("./model/annotationDiffStructureError");
+const commonUtil_1 = require("./util/commonUtil");
 const posix_1 = require("path/posix"); // Ensure standardized dir separators to ensure Windows compatibility
 // To generate keys, english language is more common, so compare all other
 // languages with it
@@ -103,7 +104,7 @@ class I18nManager {
     createFiles(i18nPathName) {
         return this.i18nFileContent.createFiles(i18nPathName);
     }
-    populateTranslations(annotationJsons) {
+    async populateTranslations(annotationJsons) {
         /* We compare annotations. Diff gives us:
         {
             "string": {
@@ -127,26 +128,27 @@ class I18nManager {
         and so on */
         let defaultAnnotation = I18nManager.extractDefaultLanguageAnnotation(annotationJsons);
         if (annotationJsons.size > 0 && defaultAnnotation) {
-            defaultAnnotation = this.populate([...annotationJsons], defaultAnnotation);
+            defaultAnnotation = await this.populate([...annotationJsons], defaultAnnotation);
         }
         return defaultAnnotation;
     }
-    populate(annotationJsons, defaultAnnotation) {
-        return annotationJsons
-            .map(([language, json]) => ({ language, json }))
-            .reduce((p, c) => {
+    async populate(annotationJsons, defaultAnnotation) {
+        let p = defaultAnnotation;
+        for (const c of annotationJsons.map(([language, json]) => ({ language, json }))) {
             try {
-                const pDiff = jsonDiffUtil_1.default.diff(p.json, c.json);
+                const comparator = new comparator_1.default(await p.json, await c.json);
+                const pDiff = comparator.compare();
                 this.processDiff(pDiff.properties, p.language, c.language);
-                return { json: pDiff.json, language: p.language };
+                p.json = pDiff.json;
             }
             catch (error) {
                 if (error instanceof annotationDiffStructureError_1.default) {
-                    throw new Error(`The structure of the oData annotation xml of language '${p.language.sap}' is different from xml of language '${c.language.sap}' near element: ${error.message}`);
+                    throw new Error(`The structure of the OData annotation xml of language '${p.language}' is different from xml of language '${c.language}' near element: ${error.message}`);
                 }
                 throw error;
             }
-        }, defaultAnnotation);
+        }
+        return p;
     }
     static extractDefaultLanguageAnnotation(annotationJsons) {
         let json = null;
@@ -161,20 +163,18 @@ class I18nManager {
         annotationJsons.delete(language);
         return { json, language };
     }
-    replaceWithModelReference({ object, property, type }, propertyValues) {
-        if (type === jsonDiffUtil_1.DiffTypeEnum.DELTA) {
-            const diff = object[property];
-            this.initPropertyValues(diff, propertyValues);
-            // If there are already generated key, like on step 3. above comment, we
-            // take it (from __old), so we don't need to generate new
-            const propReference = propertyValues.find(prop => prop.isReference);
-            let reference = propReference?.value ?? this.createReference(diff.__old);
-            object[property] = reference;
-            // Other values, which are not references, are extracted to .properties
-            const key = this.references.get(reference);
-            if (key) {
-                propertyValues.filter(prop => !prop.isReference).forEach(prop => this.i18nFileContent.add(prop, key));
-            }
+    replaceWithModelReference({ object, property }, propertyValues) {
+        const diff = object[property];
+        this.initPropertyValues(diff, propertyValues);
+        // If there are already generated key, like on step 3. above comment, we
+        // take it (from __old), so we don't need to generate new
+        const propReference = propertyValues.find(prop => prop.isReference);
+        let reference = propReference?.value ?? this.createReference(diff.__old);
+        object[property] = reference;
+        // Other values, which are not references, are extracted to .properties
+        const key = this.references.get(reference);
+        if (key) {
+            propertyValues.filter(prop => !prop.isReference).forEach(prop => this.i18nFileContent.add(prop, key));
         }
     }
     createReference(value) {
@@ -184,14 +184,11 @@ class I18nManager {
         return reference;
     }
     getUniqueKeyForValue(value) {
+        if (typeof value !== "string") {
+            throw new Error("Failed to create unique key from: " + JSON.stringify(value));
+        }
         const propertyName = value.replace(/\W/gi, "_").toUpperCase();
-        let suffix = -1;
-        let suffixString;
-        do {
-            suffixString = suffix === -1 ? "" : suffix;
-            suffix++;
-        } while (this.existingKeys.has(propertyName + suffixString));
-        const key = propertyName + suffixString;
+        const key = (0, commonUtil_1.getUniqueName)([...this.existingKeys.keys()], propertyName);
         this.existingKeys.add(key);
         return key;
     }

package/dist/index.js

@@ -1,11 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 const dotenv = require("dotenv");
+const commonUtil_1 = require("./util/commonUtil");
 const appVariantManager_1 = require("./appVariantManager");
 const baseAppManager_1 = require("./baseAppManager");
-const processor_1 = require("./processors/processor");
 const i18nMerger_1 = require("./util/i18nMerger");
-const commonUtil_1 = require("./util/commonUtil");
+const processor_1 = require("./processors/processor");
 /**
  * Creates an appVariant bundle from the provided resources.
  */
@@ -13,8 +13,9 @@ module.exports = ({ workspace, options, taskUtil }) => {
     dotenv.config();
     async function process(workspace, taskUtil) {
         await (0, commonUtil_1.logBuilderVersion)();
+        (0, commonUtil_1.logBetaUsage)();
         const processor = (0, processor_1.determineProcessor)(options.configuration);
-        const appVariantResources = await appVariantManager_1.default.getAppVariantResources(workspace);
+        const appVariantResources = await appVariantManager_1.default.getAppVariantResourcesToProcess(workspace);
         const appVariantInfo = await appVariantManager_1.default.process(appVariantResources, options.projectNamespace, taskUtil);
         const baseAppFiles = await processor.getBaseAppFiles(appVariantInfo.reference);
         const { resources, manifestInfo } = await baseAppManager_1.default.process(baseAppFiles, appVariantInfo, options, processor);

package/dist/model/annotationDiffStructureError.js

@@ -3,6 +3,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 class AnnotationDiffStructureError extends Error {
     constructor(json) {
         super(JSON.stringify(json));
+        this.name = "AnnotationDiffStructureError";
+        this.message = `The structure of the OData annotation xml is different near element: ${JSON.stringify(json)}`;
     }
 }
 exports.default = AnnotationDiffStructureError;

package/dist/model/language.js

@@ -17,7 +17,7 @@ class Language {
         let configLanguages = [];
         if (languages !== undefined) {
             configLanguages = languages.map(item => {
-                if (typeof item === 'string') {
+                if (typeof item === "string") {
                     // For legacy language format support which is just a string and doesn't contain i18n
                     return new Language(item, item.toLowerCase());
                 }

package/dist/model/serverError.d.ts

@@ -0,0 +1,3 @@
+export default class ServerError extends Error {
+    constructor(uri: string, error: any);
+}

package/dist/model/serverError.js

@@ -0,0 +1,9 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+class ServerError extends Error {
+    constructor(uri, error) {
+        super(`Request ${uri} failed with Server error: ${error.response.status}${error.response.data ?? ""}`);
+    }
+}
+exports.default = ServerError;
+//# sourceMappingURL=serverError.js.map

package/dist/model/types.d.ts

@@ -1,4 +1,6 @@
+import Language from "./language";
 export interface IConfiguration {
+    writeTempFiles?: any;
     appHostId?: string;
     appId?: string;
     appName?: string;
@@ -11,6 +13,7 @@ export interface IConfiguration {
     type?: "cf" | "abap";
     languages?: any[] | undefined;
     enableAnnotationCache?: boolean;
+    enableBetaFeatures?: boolean;
 }
 export interface IProjectOptions {
     configuration: IConfiguration;
@@ -58,8 +61,8 @@ export interface IUaa {
 export interface IAppVariantInfo {
     id: string;
     reference: string;
-    manifest: IAppVariantManifest;
-    manifestChanges: Array<IChange>;
+    layer?: string;
+    changes: IChange[];
 }
 export interface IAppVariantManifest {
     id: string;
@@ -106,3 +109,11 @@ export interface IMetadata {
 export type KeyedMap<T, K extends keyof T, V> = {
     [k in K]: V;
 };
+export interface IJsonPerLanguage {
+    language: Language;
+    json: any;
+}
+export interface IJsonPromisePerLanguage {
+    language: Language;
+    json: Promise<any>;
+}

package/dist/repositories/abapRepoManager.js

@@ -6,12 +6,12 @@ const log = require("@ui5/logger").getLogger("@ui5/task-adaptation::AbapRepoMana
 const REQUEST_OPTIONS_XML = {
     responseType: "text",
     headers: {
-        "Content-Type": "text/xml"
+        "Accept": "text/html,application/xhtml+xml,application/xml"
     }
 };
 const REQUEST_OPTIONS_JSON = {
     headers: {
-        "Content-Type": "application/json"
+        "Accept": "application/json"
     }
 };
 class AbapRepoManager {

package/dist/util/commonUtil.d.ts

@@ -1,6 +1,13 @@
+import { IConfiguration } from "../model/types";
+import Language from "../model/language";
 export declare function dotToUnderscore(value: string): string;
 export declare function validateObject<T extends Object>(options: T, properties: Array<keyof T>, message: string): void;
 export declare function escapeRegex(update: string): string;
 export declare function renameResources(files: Map<string, string>, search: string, replacement: string): Map<string, string>;
+export declare function insertInArray<T>(array: T[], index: number, insert: T): void;
+export declare function writeTempAnnotations({ writeTempFiles }: IConfiguration, name: string, language: Language, content: string): void;
 export declare function removePropertiesExtension(filePath: string): string;
+export declare function traverse(json: any, paths: string[], callback: (json: any, key: string | number, paths: string[]) => void): void;
 export declare function logBuilderVersion(): Promise<void>;
+export declare function logBetaUsage(): Promise<void>;
+export declare function getUniqueName(existingNames: string[], template: string): string;

package/dist/util/commonUtil.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.logBuilderVersion = exports.removePropertiesExtension = exports.renameResources = exports.escapeRegex = exports.validateObject = exports.dotToUnderscore = void 0;
+exports.getUniqueName = exports.logBetaUsage = exports.logBuilderVersion = exports.traverse = exports.removePropertiesExtension = exports.writeTempAnnotations = exports.insertInArray = exports.renameResources = exports.escapeRegex = exports.validateObject = exports.dotToUnderscore = void 0;
+const fs = require("fs");
+const path_1 = require("path");
 const log = require("@ui5/logger").getLogger("rollup-plugin-ui5-resolve-task-adaptation");
 function dotToUnderscore(value) {
     return value.replace(/\./g, "_");
@@ -35,32 +37,70 @@ function renameResources(files, search, replacement) {
         escapedSearch = escapeRegex(search);
     }
     const dotToSlash = (update) => update.replaceAll(".", "\/");
-    const replace = (content) => content.replace(new RegExp(escapedSearch, "g"), replacement);
-    const replaceWithSlashesOnly = (content) => {
-        if (!search.includes(".")) {
-            return content;
+    const replaces = [
+        {
+            regexp: new RegExp(escapedSearch, "g"),
+            replacement
+        },
+        {
+            regexp: new RegExp(dotToSlash(escapedSearch), "g"),
+            replacement: dotToSlash(replacement)
         }
-        let searchWithSlashes = dotToSlash(escapedSearch);
-        return content.replace(new RegExp(searchWithSlashes, "g"), dotToSlash(replacement));
-    };
-    const renamed = new Map();
-    files.forEach((content, filepath) => {
-        // Finds the id with dots (test.id) or without dots (id) and replaces it
-        content = replace(content);
-        // Only if the id has dots, these dots will be replaced with slashes
-        // first, and then it will search for the id with slashes and replace
-        // with the appVariantId also with slashes
-        content = replaceWithSlashesOnly(content);
-        renamed.set(filepath, content);
+    ];
+    files.forEach((content, filepath, map) => {
+        map.set(filepath, replaces.reduce((p, c) => p.replace(c.regexp, c.replacement), content));
     });
-    return renamed;
+    return files;
 }
 exports.renameResources = renameResources;
+function insertInArray(array, index, insert) {
+    array.splice(index, 0, insert);
+}
+exports.insertInArray = insertInArray;
+function writeTempAnnotations({ writeTempFiles }, name, language, content) {
+    const TEMP_DIST_FOLDER = path_1.posix.join(process.cwd(), "dist-debug", name);
+    if (writeTempFiles) {
+        if (!fs.existsSync(TEMP_DIST_FOLDER)) {
+            fs.mkdirSync(TEMP_DIST_FOLDER, { recursive: true });
+        }
+        if (language) {
+            name += "-" + language.i18n;
+        }
+        fs.writeFileSync(path_1.posix.join(TEMP_DIST_FOLDER, name + ".xml"), content);
+    }
+}
+exports.writeTempAnnotations = writeTempAnnotations;
 function removePropertiesExtension(filePath) {
     const lastIndexOf = filePath.lastIndexOf(".properties");
     return filePath.substring(0, lastIndexOf);
 }
 exports.removePropertiesExtension = removePropertiesExtension;
+function traverse(json, paths, callback) {
+    for (const key of Object.keys(json)) {
+        const internPaths = [...paths];
+        internPaths.push(key);
+        if (typeof json[key] === "object") {
+            if (Array.isArray(json[key])) {
+                const array = json[key];
+                for (let i = 0; i < array.length; i++) {
+                    if (typeof array[i] === "object") {
+                        traverse(array[i], internPaths, callback);
+                    }
+                    else {
+                        callback(array, i, internPaths);
+                    }
+                }
+            }
+            else {
+                traverse(json[key], internPaths, callback);
+            }
+        }
+        else {
+            callback(json, key, internPaths);
+        }
+    }
+}
+exports.traverse = traverse;
 async function logBuilderVersion() {
     try {
         // @ts-ignore
@@ -72,4 +112,18 @@ async function logBuilderVersion() {
     }
 }
 exports.logBuilderVersion = logBuilderVersion;
+async function logBetaUsage() {
+    log.info("Beta features enabled");
+}
+exports.logBetaUsage = logBetaUsage;
+function getUniqueName(existingNames, template) {
+    let suffix = -1;
+    let suffixString;
+    do {
+        suffixString = suffix === -1 ? "" : suffix;
+        suffix++;
+    } while (existingNames.includes(template + suffixString));
+    return template + suffixString;
+}
+exports.getUniqueName = getUniqueName;
 //# sourceMappingURL=commonUtil.js.map

package/dist/util/i18nMerger.d.ts

@@ -1,6 +1,6 @@
 import { IAppVariantInfo } from "../model/types";
 export default class I18NMerger {
-    static analyzeAppVariantManifestChanges(rootFolder: string, tranlsationRegexPattern: string, appVariantInfo: IAppVariantInfo): {
+    static analyzeAppVariantManifestChanges(rootFolder: string, tranlsationRegexPattern: string, { changes }: IAppVariantInfo): {
         mergePathsRegex: RegExp[];
         copyPathsRegex: RegExp[];
     };

package/dist/util/i18nMerger.js

@@ -5,13 +5,12 @@ const resourceUtil_1 = require("./resourceUtil");
 const path_1 = require("path");
 const Resource = require("@ui5/fs/lib/Resource");
 class I18NMerger {
-    static analyzeAppVariantManifestChanges(rootFolder, tranlsationRegexPattern, appVariantInfo) {
+    static analyzeAppVariantManifestChanges(rootFolder, tranlsationRegexPattern, { changes }) {
         // check which files need to be copied and which files need to be merged and copied
         // this is necessary because lrep does not support multiple enhanceWith with multiple locations
         const mergePaths = new Set();
         const copyPaths = new Set();
-        const { manifest, manifestChanges } = appVariantInfo;
-        manifestChanges.concat(manifest.content).forEach((change) => {
+        changes.forEach((change) => {
             const i18nPathWithExtension = change.content?.bundleUrl || change.texts?.i18n;
             if (i18nPathWithExtension) {
                 // build regex to match specific + language related files
@@ -46,7 +45,7 @@ class I18NMerger {
                     let baseAppI18NPath = `${rootFolder}/${baseAppManifestI18NPath}${mergePathMatch[1] || ""}.properties`;
                     await this.mergePropertiesFiles(aggregatedResourceFilesMap, appVariantResource, baseAppI18NPath);
                 }
-                // Resource for to be copied file already exists so we only have to adjust path 
+                // Resource for to be copied file already exists so we only have to adjust path
                 // Otherwise we have to omit it. We always change the path to avoid omitting a base app file
                 this.moveToAppVarSubfolder(appVariantResource, rootFolder, i18nTargetFolder);
                 if (!shouldCopyFile) {

package/dist/util/requestUtil.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 const axios_1 = require("axios");
 const noAuthorizationProvidedError_1 = require("../model/noAuthorizationProvidedError");
+const serverError_1 = require("../model/serverError");
 class RequestUtil {
     static async head(url, auth) {
         return this.request(url, axios_1.default.head, auth);
@@ -24,6 +25,9 @@ class RequestUtil {
             if (error.response.status === 401) {
                 throw new noAuthorizationProvidedError_1.default(uri);
             }
+            else if (error.response.status >= 500) {
+                throw new serverError_1.default(uri, error);
+            }
             else {
                 throw new Error(`Unexpected response received from '${uri}': ${error.response.status} ${error.response.data ?? ""}`);
             }

package/dist/util/resourceUtil.d.ts

@@ -1,8 +1,11 @@
 export default class ResourceUtil {
     static getRootFolder(projectNamespace?: string): string;
+    static relativeToRoot(resourcePath: string, projectNamespace?: string): string;
     static getResourcePath(projectNamespace?: string, ...paths: string[]): string;
     static write(dir: string, files: Map<string, string>): Promise<void[]>;
     static read(rootFolder: string, folder: string, files: Map<string, string>, exclude?: string[]): void;
     static getString(resource: any): Promise<string>;
+    static getJson(resource: any): Promise<any>;
     static setString(resource: any, str: string): void;
+    static createResource(filename: string, projectNamespace: string, content: string): any;
 }

package/dist/util/resourceUtil.js

@@ -12,6 +12,10 @@ class ResourceUtil {
         }
         return path_1.posix.join(...newPath);
     }
+    static relativeToRoot(resourcePath, projectNamespace) {
+        const rootFolder = ResourceUtil.getRootFolder(projectNamespace);
+        return path_1.posix.relative(rootFolder, resourcePath);
+    }
     static getResourcePath(projectNamespace, ...paths) {
         return path_1.posix.join(this.getRootFolder(projectNamespace), ...paths);
     }
@@ -33,7 +37,7 @@ class ResourceUtil {
             const entryPath = path_1.posix.join(folder, entry);
             const stats = fs.lstatSync(entryPath);
             if (stats.isFile() && !exclude.some(filepath => entryPath.endsWith(filepath))) {
-                const normalized = entryPath.substring(rootFolder.length);
+                const normalized = entryPath.substring(rootFolder.length + 1);
                 files.set(normalized, fs.readFileSync(entryPath, { encoding: "utf-8" }));
             }
             else if (stats.isDirectory()) {
@@ -44,9 +48,18 @@ class ResourceUtil {
     static getString(resource) {
         return resource.getBuffer().then((buffer) => buffer.toString(UTF8));
     }
+    static getJson(resource) {
+        return resource.getBuffer().then((buffer) => JSON.parse(buffer.toString(UTF8)));
+    }
     static setString(resource, str) {
         resource.setBuffer(Buffer.from(str, UTF8));
     }
+    static createResource(filename, projectNamespace, content) {
+        return resourceFactory.createResource({
+            path: this.getResourcePath(projectNamespace, filename),
+            string: content
+        });
+    }
 }
 exports.default = ResourceUtil;
 //# sourceMappingURL=resourceUtil.js.map

package/dist/util/urlUtil.d.ts

@@ -0,0 +1,4 @@
+export default class UrlUtil {
+    static join(relativeUrl: string, parentUrl: string): any;
+    static getResourcePath(url: string | undefined): any;
+}

package/dist/util/urlUtil.js

@@ -0,0 +1,21 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const { URI } = require("../../dist/bundle-odata");
+class UrlUtil {
+    static join(relativeUrl, parentUrl) {
+        // Remove trailing slash, otherwise url join can be incorrect Remove
+        // trailing slash, otherwise url join can be incorrect Annotation URLs
+        // defined in manifest might end with .../$value/ or .../$value and both
+        // are accepted by Gateway and produce the same content with same
+        // relative URLs. The first case is actually incorrect and we have to
+        // sanitize the same way as UI5. We also trim urls domain/host, because
+        // we need to substitute it with destination to download later.
+        return new URI(relativeUrl).absoluteTo(parentUrl.replace(/\/$/, "")).toString();
+    }
+    static getResourcePath(url) {
+        // Trim urls domain/host.
+        return url && URI.parse(url).path;
+    }
+}
+exports.default = UrlUtil;
+//# sourceMappingURL=urlUtil.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@ui5/task-adaptation",
-    "version": "1.1.0",
+    "version": "1.1.2",
     "description": "Custom task for ui5-builder which allows building UI5 Flexibility Adaptation Projects for SAP BTP, Cloud Foundry environment",
     "main": "index.js",
     "scripts": {
@@ -14,7 +14,7 @@
         "prepublishOnly": "git push --follow-tags",
         "release-note": "git-chglog -c .chglog/release-config.yml v$npm_package_version",
         "rollup": "npx ts-node scripts/rollup.ts",
-        "build": "npm run rollup && tsc -p ./",
+        "build": "npm run rollup && rimraf dist/resources && tsc -p ./",
         "download-metadata": "npx ts-node scripts/metadataDownloadHelper.ts"
     },
     "repository": {
@@ -43,10 +43,12 @@
         "amdextract": "^3.0.0",
         "axios": "^1.6.2",
         "builtin-modules": "^3.2.0",
+        "crc": "^4.3.2",
         "dotenv": "^16.0.3",
+        "glob": "^10.3.10",
         "js-yaml": "^4.1.0",
-        "json-diff": "^1.0.6",
-        "rimraf": "^3.0.2",
+        "jsdom": "^23.0.1",
+        "rimraf": "^5.0.5",
         "rollup": "^2.56.3",
         "semver": "^7.3.5",
         "temp-dir": "^2.0.0",
@@ -58,15 +60,16 @@
         "@types/chai": "^4.2.21",
         "@types/chai-as-promised": "^7.1.4",
         "@types/js-yaml": "^4.0.3",
-        "@types/json-diff": "^0.9.1",
+        "@types/jsdom": "^21.1.6",
         "@types/lodash": "^4.14.196",
         "@types/mocha": "^9.1.0",
-        "@types/rimraf": "^3.0.2",
+        "@types/rimraf": "^4.0.5",
         "@types/semver": "^7.3.8",
         "@types/sinon": "^10.0.16",
         "chai": "^4.3.4",
         "chai-as-promised": "^7.1.1",
         "chalk": "^4.1.2",
+        "minimatch": "^9.0.3",
         "mocha": "^9.2.0",
         "mock-require": "^3.0.3",
         "nyc": "^15.1.0",

package/scripts/bundler.ts

@@ -1,16 +1,12 @@
-import * as fs from "fs";
-import { posix as path } from "path";
-import * as semver from "semver";
 import * as rollup from "rollup";
 import * as builtins from "builtin-modules";
 import ui5 from "./rollup/ui5Resolve";
 import { nodeResolve } from "@rollup/plugin-node-resolve";
 
-const log = require("@ui5/logger").getLogger("rollup-plugin-ui5-resolve-task-adaptation");
 
 export default abstract class Bundler {
 
-    static async run(project: any, input: string, output: string, namespace: string,
+    static async run(project: any, input: string, output: string,
         assets: string[], skipTransformation?: string[]): Promise<void> {
         if (skipTransformation == null) {
             skipTransformation = [];
@@ -18,20 +14,11 @@ export default abstract class Bundler {
         if (skipTransformation.includes(input) === false) {
             skipTransformation.push(input);
         }
-        const bundledUI5Version = this.getBundledUI5Version(output);
-        const ui5version = this.getVersion(project, namespace);
-        if (bundledUI5Version == null || ui5version && semver.neq(bundledUI5Version, ui5version)) {
-            if (ui5version) {
-                log.info(`Using UI5 version ${ui5version.toString()} to bundle`);
-            }
-            await this.bundle(project, input, output, assets, skipTransformation, ui5version);
-        } else {
-            log.info(`UI5 version ${bundledUI5Version!.toString()} is already bundled`);
-        }
+        await this.bundle(project, input, output, assets, skipTransformation);
     }
 
     private static async bundle(project: any, input: string, output: string,
-        assets: string[], skipTransformation: string[], ui5version: string): Promise<void> {
+        assets: string[], skipTransformation: string[]): Promise<void> {
         const inputOptions = <rollup.RollupOptions>{
             input,
             plugins: [
@@ -39,8 +26,7 @@ export default abstract class Bundler {
                     assets,
                     skipTransformation,
                     output,
-                    project,
-                    ui5version
+                    project
                 }),
                 nodeResolve({
                     preferBuiltins: true
@@ -57,19 +43,4 @@ export default abstract class Bundler {
         await bundle.write(outputOptions);
         await bundle.close();
     }
-
-    private static getBundledUI5Version(output: string) {
-        const bundleFilePath = path.join(process.cwd(), output);
-        if (fs.existsSync(bundleFilePath)) {
-            const bundle = fs.readFileSync(bundleFilePath, { encoding: "utf-8" });
-            const version = bundle.substring(2, bundle.indexOf("\n"));
-            return semver.coerce(version);
-        }
-    }
-
-    private static getVersion(project: any, namespace: string): string {
-        const isModule = (dependency: any) => dependency.id.endsWith(namespace);
-        const sapUiFlDependency = project.dependencies.find(isModule);
-        return sapUiFlDependency && sapUiFlDependency.version;
-    }
 }