@ui5/task-adaptation 1.4.2 → 1.5.0

package/CHANGELOG.md

@@ -2,7 +2,20 @@
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
-A list of unreleased changes can be found [here](https://github.com/SAP/ui5-task-adaptation/compare/v1.4.2...HEAD).
+A list of unreleased changes can be found [here](https://github.com/SAP/ui5-task-adaptation/compare/v1.5.0...HEAD).
+
+<a name="v1.5.0"></a>
+## [v1.5.0] - 2025-07-17
+
+<a name="v1.4.3"></a>
+## [v1.4.3] - 2025-04-17
+### Bug Fixes
+- Handling undefined oAuthScopes ([#122](https://github.com/SAP/ui5-task-adaptation/issues/122)) [`ea6296b`](https://github.com/SAP/ui5-task-adaptation/commit/ea6296b143e7fcbf8ce3902499be2d4c2fea61a5)
+- Better error handling in CF ([#121](https://github.com/SAP/ui5-task-adaptation/issues/121)) [`8ce564f`](https://github.com/SAP/ui5-task-adaptation/commit/8ce564fa847a512f4de341f14b1788bca3075da8)
+- Get space guid correctly ([#120](https://github.com/SAP/ui5-task-adaptation/issues/120)) [`64a28ee`](https://github.com/SAP/ui5-task-adaptation/commit/64a28eefc19ecded18bd363c65a8417180200945)
+- Create html5 repo service instance ([#118](https://github.com/SAP/ui5-task-adaptation/issues/118)) [`92a6769`](https://github.com/SAP/ui5-task-adaptation/commit/92a6769ea8704ad81bf7e65766c260bb710ab043)
+- Audit and eslint issues [`34508cb`](https://github.com/SAP/ui5-task-adaptation/commit/34508cb013f951d68527d7a733a409ed343fe43c)
+
 
 <a name="v1.4.2"></a>
 ## [v1.4.2] - 2025-02-25
@@ -106,6 +119,8 @@ A list of unreleased changes can be found [here](https://github.com/SAP/ui5-task
 <a name="v1.0.0"></a>
 ## v1.0.0 - 2020-12-09
 
+[v1.5.0]: https://github.com/SAP/ui5-task-adaptation/compare/v1.4.3...v1.5.0
+[v1.4.3]: https://github.com/SAP/ui5-task-adaptation/compare/v1.4.2...v1.4.3
 [v1.4.2]: https://github.com/SAP/ui5-task-adaptation/compare/v1.4.0...v1.4.2
 [v1.4.0]: https://github.com/SAP/ui5-task-adaptation/compare/v1.3.3...v1.4.0
 [v1.3.3]: https://github.com/SAP/ui5-task-adaptation/compare/v1.3.2...v1.3.3

package/README.md

@@ -4,28 +4,29 @@
 [![npm version](https://badge.fury.io/js/@ui5%2Ftask-adaptation.svg)](https://badge.fury.io/js/@ui5%2Ftask-adaptation)
 
 ## Description
-A custom task for [ui5-builder](https://github.com/SAP/ui5-builder) that allows building SAPUI5 adaptation projects for [SAP S/4HANA Cloud](https://help.sap.com/docs/bas/584e0bcbfd4a4aff91c815cefa0bce2d/6fc4e11a4b1941efa8e37a428d046f8f.html?locale=en-US&state=PRODUCTION&version=Cloud) and [SAP BTP, Cloud Foundry environment](https://help.sap.com/viewer/584e0bcbfd4a4aff91c815cefa0bce2d/Cloud/en-US/019b0c38a6b043d1a66b11d992eed290.html).
+A custom task for [ui5-builder](https://github.com/SAP/ui5-builder) that allows you to build SAPUI5 adaptation projects for [SAP S/4HANA Cloud](https://help.sap.com/docs/bas/584e0bcbfd4a4aff91c815cefa0bce2d/6fc4e11a4b1941efa8e37a428d046f8f.html?locale=en-US&state=PRODUCTION&version=Cloud) and [SAP BTP, Cloud Foundry environment](https://help.sap.com/viewer/584e0bcbfd4a4aff91c815cefa0bce2d/Cloud/en-US/019b0c38a6b043d1a66b11d992eed290.html).
 
 ### Configuration
-#### ABAP Connection
-The following connection configuration format is used to connect to ABAP repository in IDE:
+#### Connection
+When creating an adaptation project, the Yeoman generator automatically generates the following connection configurations in the `ui5.yaml` file, depending on your IDE:
+
+For local IDEs like VS Code:
 ```yaml
 target:
   url: example.com,
-  authenticationType: basic | reentranceTicket,
+  authenticationType: reentranceTicket,
   ignoreCertErrors: true | false
 ```
-and in SAP Business Application Studio:
+In SAP Business Application Studio:
 ```yaml
 target:
   destination: abc
 ```
-OnPremise ABAP repository requires `authenticationType: basic`, credentials should be provided in project root `.env` file:
-```
-FIORI_TOOLS_USER=<username>
-FIORI_TOOLS_PASSWORD=<password>
-```
-Whereas S4/Hana uses `authenticationType: reentranceTicket` to authentificate. A browser window will be opened.
+
+For more information, see [Create an Adaptation Project](https://help.sap.com/docs/bas/developing-sap-fiori-app-in-sap-business-application-studio/create-project) (on SAP S/4HANA Cloud) or [Create an Adaptation Project](https://help.sap.com/docs/bas/developing-sap-fiori-app-in-sap-business-application-studio/create-adaptation-project-c7b455d488bc4229af7efe0311546752) (on SAP BTP, Cloud Foundry environment).
+> [!NOTE]  
+> For SAP S/4HANA (on-premise) systems, a different builder is used.
+
 
 ## How to obtain support
 In case you need any support, please create a [GitHub issue](https://github.com/SAP/ui5-task-adaptation/issues).
@@ -34,4 +35,7 @@ In case you need any support, please create a [GitHub issue](https://github.com/
 Copyright (c) 2020 SAP SE or an SAP affiliate company. All rights reserved. This file and all other files in this repository are licensed under the Apache License, v 2.0 except as noted otherwise in the [LICENSE file](LICENSE).
 
 ## Release History
-See [CHANGELOG.md](CHANGELOG.md).
+See [CHANGELOG.md](CHANGELOG.md).
+
+## Contributing
+Please refer to the [SAP Contribution Guidelines](https://github.com/SAP/.github/blob/main/CONTRIBUTING.md) for instructions on how to contribute to ui5-task-adaptation.

package/REUSE.toml

@@ -0,0 +1,10 @@
+version = 1
+SPDX-PackageName = "ui5-task-adaptation"
+SPDX-PackageSupplier = "Aleksandr Suvorov <aleksandr.suvorov@sap.com>, Matthias Schmalz <matthias.schmalz@sap.com>"
+SPDX-PackageDownloadLocation = "https://github.com/SAP/ui5-task-adaptation"
+
+[[annotations]]
+path = "**.**"
+precedence = "aggregate"
+SPDX-FileCopyrightText = "2009-2025 SAP SE or an SAP affiliate company"
+SPDX-License-Identifier = "Apache-2.0"

package/dist/annotationManager.d.ts

@@ -10,7 +10,7 @@ export default class AnnotationManager {
     private configuration;
     constructor(configuration: IConfiguration, abapRepoManager: AbapRepoManager);
     ANNOTATIONS_FOLDER: string;
-    process(renamedBaseAppManifest: any, languages: Language[]): Promise<Map<string, string>>;
+    process(baseAppManifest: any, languages: Language[], id: string): Promise<Map<string, string>>;
     private normalizeAppVariantId;
     private updateManifestModel;
     private createManifestModel;

package/dist/annotationManager.js

@@ -1,4 +1,3 @@
-import BaseAppManager from "./baseAppManager.js";
 import DataSourceManager from "./annotations/dataSource/dataSourceManager.js";
 import I18nManager from "./i18nManager.js";
 import ServiceRequestor from "./annotations/serviceRequestor.js";
@@ -14,9 +13,7 @@ export default class AnnotationManager {
         this.abapRepoManager = abapRepoManager;
     }
     ANNOTATIONS_FOLDER = "annotations";
-    async process(renamedBaseAppManifest, languages) {
-        const { id } = BaseAppManager.getIdVersion(renamedBaseAppManifest);
-        BaseAppManager.validateProperty(id, "sap.app/id");
+    async process(baseAppManifest, languages, id) {
         const normalisedId = this.normalizeAppVariantId(id);
         //TODO: switch to this after resolving @i18n custom model
         const modelName = I18N_DEFAULT_MODEL_NAME; //`i18n_a9n_${normalisedId}`;
@@ -24,20 +21,20 @@ export default class AnnotationManager {
         const i18nManager = new I18nManager(modelName, id, languages);
         const serviceRequestor = new ServiceRequestor(this.configuration, this.abapRepoManager);
         const dataSourceManager = new DataSourceManager();
-        dataSourceManager.addDataSources(renamedBaseAppManifest["sap.app"]?.dataSources);
+        dataSourceManager.addDataSources(baseAppManifest["sap.app"]?.dataSources);
         const annotationFiles = await dataSourceManager.createAnnotationFiles(languages, i18nManager, serviceRequestor);
         const i18nFiles = i18nManager.createFiles(i18nPathName);
         if (i18nManager.hasTranslations()) {
-            this.updateManifestModel(renamedBaseAppManifest, modelName, i18nPathName);
+            this.updateManifestModel(baseAppManifest, modelName, i18nPathName);
         }
         return new Map([...annotationFiles, ...i18nFiles]);
     }
     normalizeAppVariantId(id, replaceWith = "") {
         return id.replace(/[.\W]+/gi, replaceWith);
     }
-    updateManifestModel(renamedBaseAppManifest, modelName, i18nPathName) {
+    updateManifestModel(baseAppManifest, modelName, i18nPathName) {
         const uri = `${i18nPathName}/i18n.properties`;
-        this.enhanceManifestModel(renamedBaseAppManifest, modelName, uri);
+        this.enhanceManifestModel(baseAppManifest, modelName, uri);
         //TODO: switch to this after resolving @i18n custom model
         //this.createManifestModel(renamedBaseAppManifest, modelName, uri);
     }

package/dist/annotations/dataSource/dataSource.js

@@ -10,7 +10,6 @@ export default class DataSource {
     /**
      * Update the json of the dataSources in manifest.json
      */
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
     updateManifest(_) {
         // to be overriden in children
     }

package/dist/annotations/serviceRequestor.js

@@ -4,7 +4,6 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
-import AnnotationsCacheManager from "../cache/annotationsCacheManager.js";
 import ServerError from "../model/serverError.js";
 import { getLogger } from "@ui5/logger";
 import { writeTempAnnotations } from "../util/commonUtil.js";
@@ -46,20 +45,9 @@ export default class ServiceRequestor {
     //old decorators are already subject of compiler error, but it works. So we
     //wait till esbuild implement it correctly.
     async downloadAnnotation(uri, name, language) {
-        let cacheName = name;
-        if (language.sap) {
-            uri += `?sap-language=${language.sap}`;
-            cacheName += `-${language.sap}`;
-        }
-        const cacheManager = new AnnotationsCacheManager(this.configuration, cacheName);
-        log.verbose(`Getting annotation '${cacheName}' ${language} by '${uri}'`);
-        let files;
-        if (this.configuration.enableAnnotationCache) {
-            files = await cacheManager.getFiles(() => this.abapRepoManager.getAnnotationMetadata(uri), () => this.abapRepoManager.downloadAnnotationFile(uri));
-        }
-        else {
-            files = await this.abapRepoManager.downloadAnnotationFile(uri);
-        }
+        uri += `?sap-language=${language.sap}`;
+        log.verbose(`Getting annotation '${name}' ${language} by '${uri}'`);
+        let files = await this.abapRepoManager.downloadAnnotationFile(uri);
         if (!files || files.size === 0) {
             throw new Error(`No files were fetched for '${name}' by '${uri}'`);
         }

package/dist/appVariantManager.d.ts

@@ -1,12 +1,17 @@
-import { IAppVariantInfo } from "./model/types.js";
-export default class AppVariantManager {
-    static process(appVariantResources: any[], projectNamespace: string, taskUtil: any): Promise<IAppVariantInfo>;
-    static getAppVariantResourcesToProcess(workspace: any): Promise<any[]>;
-    static updateChanges(appVariantResources: any[], projectNamespace: string): Promise<void>;
-    private static isManifestChange;
-    private static isManifestAppVariant;
-    static getAppVariantInfo(appVariantResources: any[]): Promise<IAppVariantInfo>;
-    private static validateManifest;
-    private static updateRelativePaths;
-    private static omitFiles;
+import { IChange } from "./model/types.js";
+import TaskUtil from "@ui5/project/build/helpers/TaskUtil";
+export default class AppVariant {
+    readonly files: ReadonlyMap<string, string>;
+    readonly resources?: ReadonlyArray<Resource>;
+    readonly id: string;
+    readonly reference: string;
+    readonly layer: any;
+    readonly content: any;
+    static fromWorkspace(workspace: IWorkspace, projectNamespace: string): Promise<AppVariant>;
+    static fromFiles(files: ReadonlyMap<string, string>): AppVariant;
+    private constructor();
+    getProcessedManifestChanges(): IChange[];
+    private validateManifest;
+    private updateRelativePaths;
+    omitDeletedResources(files: ReadonlyMap<string, string>, projectNamespace: string, taskUtil: TaskUtil): void;
 }

package/dist/appVariantManager.js

@@ -1,101 +1,74 @@
 import ResourceUtil from "./util/resourceUtil.js";
 import { posix as path } from "path";
-import { renameResources } from "./util/commonUtil.js";
-const EXTENSIONS_TO_PROCESS = "js,json,xml,html,properties,change,appdescr_variant,ctrl_variant,ctrl_variant_change,ctrl_variant_management_change,variant,fioriversion,codeChange,xmlViewChange,context";
-export default class AppVariantManager {
-    static async process(appVariantResources, projectNamespace, taskUtil) {
-        for (const resource of appVariantResources) {
-            this.omitFiles(resource, taskUtil);
-        }
-        await this.updateChanges(appVariantResources, projectNamespace);
-        return this.getAppVariantInfo(appVariantResources);
+const MANIFEST_CHANGES_DIR = "changes/manifest/";
+export default class AppVariant {
+    files;
+    resources;
+    id;
+    reference;
+    layer;
+    content;
+    static async fromWorkspace(workspace, projectNamespace) {
+        const EXTENSIONS_TO_PROCESS = "js,json,xml,html,properties,change,appdescr_variant,ctrl_variant,ctrl_variant_change,ctrl_variant_management_change,variant,fioriversion,codeChange,xmlViewChange,context";
+        const resources = await workspace.byGlob(`/**/*.{${EXTENSIONS_TO_PROCESS}}`);
+        const files = await ResourceUtil.toFileMap(resources, projectNamespace);
+        return new AppVariant(files, resources);
     }
-    static getAppVariantResourcesToProcess(workspace) {
-        return workspace.byGlob(`/**/*.{${EXTENSIONS_TO_PROCESS}}`);
+    static fromFiles(files) {
+        return new AppVariant(files);
     }
-    static async updateChanges(appVariantResources, projectNamespace) {
-        const changesFolder = ResourceUtil.getResourcePath(projectNamespace, "changes");
-        const changes = new Map();
-        const resourcesByPath = new Map();
-        let manifest;
-        for (const resource of appVariantResources) {
-            if (this.isManifestAppVariant(resource)) {
-                manifest = await ResourceUtil.getJson(resource);
-            }
-            const resourcePath = resource.getPath();
-            const basename = path.dirname(resourcePath);
-            if (basename.startsWith(changesFolder)) {
-                changes.set(resourcePath, await ResourceUtil.getString(resource));
-                resourcesByPath.set(resourcePath, resource);
-            }
-        }
-        this.updateRelativePaths(changes, projectNamespace);
-        this.validateManifest(manifest);
-        const renamedChanges = renameResources(changes, manifest.reference, manifest.id);
-        renamedChanges.forEach((renamedContent, resourcePath) => {
-            const resource = resourcesByPath.get(resourcePath);
-            ResourceUtil.setString(resource, renamedContent);
-        });
+    constructor(files, resources) {
+        this.files = files;
+        this.resources = resources;
+        const manifestString = files.get("manifest.appdescr_variant");
+        this.validateManifest(manifestString);
+        const { reference, id, layer, content } = JSON.parse(manifestString);
+        this.reference = reference;
+        this.id = id;
+        this.layer = layer;
+        this.content = content;
     }
-    static isManifestChange(resource) {
-        const changesManifestFolder = path.join("changes", "manifest");
-        const resourcePath = typeof resource === "string" ? resource : resource.getPath();
-        const dirname = path.dirname(resourcePath);
-        return dirname.endsWith(changesManifestFolder);
-    }
-    static isManifestAppVariant(resource) {
-        const MANIFEST_APP_VARIANT = "manifest.appdescr_variant";
-        const basename = path.basename(resource.getPath());
-        return basename === MANIFEST_APP_VARIANT;
-    }
-    static async getAppVariantInfo(appVariantResources) {
-        let manifest;
-        const manifestChanges = [];
-        for (const resource of appVariantResources) {
-            if (this.isManifestAppVariant(resource)) {
-                manifest = await ResourceUtil.getJson(resource);
-            }
-            else if (this.isManifestChange(resource)) {
-                const content = await ResourceUtil.getString(resource);
-                manifestChanges.push(JSON.parse(content));
-            }
-        }
-        this.validateManifest(manifest);
+    getProcessedManifestChanges() {
         // Order is important: apply manifest.json changes first, then *.change
         // files. UI5 does the same.
-        const changes = (manifest.content ?? []).concat(manifestChanges);
-        return {
-            id: manifest.id,
-            reference: manifest.reference,
-            layer: manifest.layer,
-            changes
-        };
+        const manifestChanges = structuredClone(this.content) ?? [];
+        this.files.forEach((content, filename) => {
+            if (filename.startsWith(MANIFEST_CHANGES_DIR)) {
+                const change = JSON.parse(content);
+                this.updateRelativePaths(change, filename);
+                manifestChanges.push(change);
+            }
+        });
+        if (this.layer) {
+            manifestChanges.forEach(change => change.layer = this.layer ?? change.layer);
+        }
+        return manifestChanges;
     }
-    static validateManifest(manifest) {
+    validateManifest(manifest) {
         if (!manifest) {
             throw new Error("Adaptation project should contain manifest.appdescr_variant");
         }
     }
-    // TODO In future this should be handled by merger which needs to know change and target location
-    static updateRelativePaths(changes, projectNamespace) {
-        changes.forEach((jsonString, resourcePath) => {
-            if (this.isManifestChange(resourcePath)) {
-                const change = JSON.parse(jsonString);
-                if (change.changeType === "appdescr_app_addAnnotationsToOData") {
-                    for (const dataSource of Object.values(change.content.dataSource)) {
-                        if (!dataSource.uri.startsWith("/")) {
-                            const basepath = path.dirname(ResourceUtil.relativeToRoot(resourcePath, projectNamespace));
-                            dataSource.uri = path.join(basepath.replace(/^\//, ""), dataSource.uri);
-                        }
-                    }
-                    changes.set(resourcePath, JSON.stringify(change));
+    updateRelativePaths(change, filename) {
+        // TODO In future this should be handled by merger which needs to know change and target location
+        if (change.changeType === "appdescr_app_addAnnotationsToOData") {
+            for (const dataSource of Object.values(change?.content?.dataSource)) {
+                if (!dataSource.uri.startsWith("/")) {
+                    const basepath = path.dirname(filename);
+                    dataSource.uri = path.join(basepath.replace(/^\//, ""), dataSource.uri);
                 }
             }
-        });
+        }
     }
-    static omitFiles(resource, taskUtil) {
-        if (this.isManifestAppVariant(resource) || this.isManifestChange(resource)) {
-            taskUtil.setTag(resource, taskUtil.STANDARD_TAGS.OmitFromBuildResult, true);
+    omitDeletedResources(files, projectNamespace, taskUtil) {
+        if (!this.resources) {
+            return;
+        }
+        for (const resource of this.resources) {
+            const relativePath = ResourceUtil.relativeToRoot(resource.getPath(), projectNamespace);
+            if (!files.has(relativePath)) {
+                taskUtil.setTag(resource, taskUtil.STANDARD_TAGS.OmitFromBuildResult, true);
+            }
         }
     }
 }

package/dist/baseAppManager.d.ts

@@ -1,4 +1,4 @@
-import { IAppVariantInfo, IProjectOptions } from "./model/types.js";
+import AppVariant from "./appVariantManager.js";
 import IProcessor from "./processors/processor.js";
 export interface IBaseAppResources {
     resources: any[];
@@ -11,19 +11,19 @@ export interface IManifestIdVersion {
 export interface IManifestInfo extends IManifestIdVersion {
     i18nPath: string;
 }
-export default class BaseAppManager {
-    static process(baseAppFiles: Map<string, string>, appVariantInfo: IAppVariantInfo, options: IProjectOptions, processor: IProcessor): Promise<IBaseAppResources>;
-    private static updateAdaptationProperties;
-    static getIdVersion(manifest: any): IManifestIdVersion;
-    static getManifestInfo(manifest: any): IManifestInfo;
-    private static extractI18nPathFromManifest;
-    private static extractI18NFromBundleName;
-    private static extractI18NFromBundleUrl;
-    private static getBaseAppManifest;
-    private static fillAppVariantIdHierarchy;
-    private static VALIDATION_RULES;
-    static validateProperty(value: string, property: string): void;
-    static applyDescriptorChanges(baseAppManifest: any, { layer, changes, id }: IAppVariantInfo): Promise<void>;
-    private static adjustAddNewModelEnhanceWith;
-    static writeToWorkspace(baseAppFiles: Map<string, string>, projectNamespace: string): any[];
+export default class BaseApp {
+    readonly id: string;
+    readonly version: string;
+    readonly i18nPath: string;
+    readonly files: ReadonlyMap<string, string>;
+    static fromFiles(files: ReadonlyMap<string, string>): BaseApp;
+    private constructor();
+    adapt(appVariant: AppVariant, processor: IProcessor): Promise<ReadonlyMap<string, string>>;
+    private updateAdaptationProperties;
+    private extractI18nPathFromManifest;
+    private fillAppVariantIdHierarchy;
+    private VALIDATION_RULES;
+    private validateProperty;
+    private applyDescriptorChanges;
+    private adjustAddNewModelEnhanceWith;
 }

package/dist/baseAppManager.js

@@ -1,29 +1,45 @@
-import { Applier, Change, RegistrationBuild } from "../dist/bundle.js";
-import { dotToUnderscore, removePropertiesExtension } from "./util/commonUtil.js";
+import { AppDescriptorChange, Applier, RegistrationBuild } from "../dist/bundle.js";
+import { dotToUnderscore, trimExtension } from "./util/commonUtil.js";
 import BuildStrategy from "./buildStrategy.js";
-import ResourceUtil from "./util/resourceUtil.js";
 import { getLogger } from "@ui5/logger";
-import { renameResources } from "./util/commonUtil.js";
 const log = getLogger("@ui5/task-adaptation::BaseAppManager");
-export default class BaseAppManager {
-    static async process(baseAppFiles, appVariantInfo, options, processor) {
-        const baseAppManifest = this.getBaseAppManifest(baseAppFiles);
-        const { id, version } = this.getIdVersion(baseAppManifest.content);
-        this.validateProperty(id, "sap.app/id");
-        this.validateProperty(version, "sap.app/applicationVersion/version");
-        const renamedBaseAppFiles = renameResources(baseAppFiles, appVariantInfo.reference, appVariantInfo.id);
-        const { filepath, content } = this.getBaseAppManifest(renamedBaseAppFiles);
-        await processor.updateLandscapeSpecificContent(content, renamedBaseAppFiles);
-        this.fillAppVariantIdHierarchy(processor, id, version, content);
-        this.updateAdaptationProperties(content);
-        await this.applyDescriptorChanges(content, appVariantInfo);
-        renamedBaseAppFiles.set(filepath, JSON.stringify(content));
-        return {
-            resources: this.writeToWorkspace(renamedBaseAppFiles, options.projectNamespace),
-            manifestInfo: this.getManifestInfo(content)
-        };
+const IGNORE_FILES = [
+    "manifest-bundle.zip",
+    "Component-preload.js",
+    "sap-ui-cachebuster-info.json"
+];
+export default class BaseApp {
+    id;
+    version;
+    i18nPath;
+    files;
+    static fromFiles(files) {
+        return new BaseApp(files);
     }
-    static updateAdaptationProperties(content) {
+    constructor(files) {
+        this.files = new Map([...files].filter(([filename]) => !IGNORE_FILES.includes(filename)));
+        const manifestString = files.get("manifest.json");
+        if (!manifestString) {
+            throw new Error("Original application should have manifest.json in root folder");
+        }
+        const manifest = JSON.parse(manifestString);
+        this.id = manifest["sap.app"]?.id;
+        this.version = manifest["sap.app"]?.applicationVersion?.version;
+        this.validateProperty(this.id, "sap.app/id");
+        this.validateProperty(this.version, "sap.app/applicationVersion/version");
+        this.i18nPath = this.extractI18nPathFromManifest(this.id, manifest["sap.app"]?.i18n);
+    }
+    async adapt(appVariant, processor) {
+        const files = new Map(this.files);
+        const manifest = JSON.parse(files.get("manifest.json"));
+        await processor.updateLandscapeSpecificContent(manifest, files, appVariant.id);
+        this.fillAppVariantIdHierarchy(processor, this.id, this.version, manifest);
+        this.updateAdaptationProperties(manifest);
+        await this.applyDescriptorChanges(manifest, appVariant);
+        files.set("manifest.json", JSON.stringify(manifest));
+        return files;
+    }
+    updateAdaptationProperties(content) {
         if (content["sap.fiori"]?.cloudDevAdaptationStatus) {
             delete content["sap.fiori"].cloudDevAdaptationStatus;
         }
@@ -32,42 +48,21 @@ export default class BaseAppManager {
         }
         content["sap.ui5"].isCloudDevAdaptation = true;
     }
-    static getIdVersion(manifest) {
-        const id = manifest["sap.app"]?.id;
-        const version = manifest["sap.app"]?.applicationVersion?.version;
-        return { id, version };
-    }
-    static getManifestInfo(manifest) {
-        const { id, version } = this.getIdVersion(manifest);
-        const i18nNode = manifest["sap.app"]?.i18n;
-        const i18nPath = this.extractI18nPathFromManifest(id, i18nNode);
-        return { id, version, i18nPath };
-    }
-    static extractI18nPathFromManifest(sapAppId, i18nNode) {
-        if (typeof i18nNode === "object") {
-            return i18nNode["bundleUrl"] ? this.extractI18NFromBundleUrl(i18nNode) : this.extractI18NFromBundleName(i18nNode, sapAppId);
-        }
-        else {
-            return `${sapAppId?.replaceAll(".", "/")}/${i18nNode}`;
-        }
-    }
-    static extractI18NFromBundleName(i18nNode, sapAppId) {
-        return i18nNode["bundleName"].replace(sapAppId, "").replaceAll(".", "/").substring(1);
-    }
-    static extractI18NFromBundleUrl(i18nNode) {
-        return removePropertiesExtension(i18nNode["bundleUrl"]);
-    }
-    static getBaseAppManifest(baseAppFiles) {
-        const manifestContent = baseAppFiles.get("manifest.json");
-        if (manifestContent) {
-            return {
-                content: JSON.parse(manifestContent),
-                filepath: "manifest.json"
-            };
+    extractI18nPathFromManifest(sapAppId, i18nNode) {
+        if (i18nNode) {
+            if (i18nNode["bundleUrl"]) {
+                return trimExtension(i18nNode["bundleUrl"]);
+            }
+            else if (i18nNode["bundleName"]) {
+                return i18nNode["bundleName"].replace(sapAppId, "").replaceAll(".", "/").substring(1);
+            }
+            else if (typeof i18nNode === "string") {
+                return trimExtension(i18nNode);
+            }
         }
-        throw new Error("Original application should have manifest.json in root folder");
+        return "i18n/i18n";
     }
-    static fillAppVariantIdHierarchy(processor, id, version, baseAppManifest) {
+    fillAppVariantIdHierarchy(processor, id, version, baseAppManifest) {
         log.verbose("Filling up app variant hierarchy in manifest.json");
         if (baseAppManifest["sap.ui5"] == null) {
             baseAppManifest["sap.ui5"] = {};
@@ -78,12 +73,12 @@ export default class BaseAppManager {
         const appVariantIdHierarchyItem = processor.createAppVariantHierarchyItem(id, version);
         baseAppManifest["sap.ui5"].appVariantIdHierarchy.unshift(appVariantIdHierarchyItem);
     }
-    static VALIDATION_RULES = new Map([["sap.app/id", (value) => {
+    VALIDATION_RULES = new Map([["sap.app/id", (value) => {
                 if (!value.includes(".")) {
                     throw new Error(`The original application id '${value}' should consist of multiple segments split by dot, e.g.: original.id`);
                 }
             }]]);
-    static validateProperty(value, property) {
+    validateProperty(value, property) {
         if (!value) {
             throw new Error(`Original application manifest should have ${property}`);
         }
@@ -92,15 +87,12 @@ export default class BaseAppManager {
             validatationRule(value);
         }
     }
-    static async applyDescriptorChanges(baseAppManifest, { layer, changes, id }) {
+    async applyDescriptorChanges(baseAppManifest, appVariant) {
         log.verbose("Applying appVariant changes");
         const changesContent = new Array();
-        const i18nBundleName = dotToUnderscore(id);
-        for (const change of structuredClone(changes)) {
-            if (layer) {
-                change.layer = layer;
-            }
-            changesContent.push(new Change(change));
+        const i18nBundleName = dotToUnderscore(appVariant.id);
+        for (const change of appVariant.getProcessedManifestChanges()) {
+            changesContent.push(new AppDescriptorChange(change));
             this.adjustAddNewModelEnhanceWith(change, i18nBundleName);
         }
         if (changesContent.length > 0) {
@@ -108,7 +100,7 @@ export default class BaseAppManager {
             await Applier.applyChanges(baseAppManifest, changesContent, strategy);
         }
     }
-    static adjustAddNewModelEnhanceWith(change, i18nBundleName) {
+    adjustAddNewModelEnhanceWith(change, i18nBundleName) {
         if (change.changeType === "appdescr_ui5_addNewModelEnhanceWith") {
             if (change.texts == null) {
                 // We need to add texts properties to changes because not all
@@ -120,20 +112,5 @@ export default class BaseAppManager {
             change.texts.i18n = i18nBundleName + "/" + change.texts.i18n;
         }
     }
-    static writeToWorkspace(baseAppFiles, projectNamespace) {
-        const IGNORE_FILES = [
-            "manifest-bundle.zip",
-            "Component-preload.js",
-            "sap-ui-cachebuster-info.json"
-        ];
-        const resources = [];
-        for (let filename of baseAppFiles.keys()) {
-            if (!IGNORE_FILES.includes(filename)) {
-                const resource = ResourceUtil.createResource(filename, projectNamespace, baseAppFiles.get(filename));
-                resources.push(resource);
-            }
-        }
-        return resources;
-    }
 }
 //# sourceMappingURL=baseAppManager.js.map

package/dist/bundle.d.ts

@@ -1,13 +1,12 @@
 export declare const RegistrationBuild: () => void;
 
 export declare class Applier {
-    static applyChanges(manifest: any, changes: Change[], strategy: any): Promise<void>;
+    static applyChanges(manifest: any, changes: AppDescriptorChange[], strategy: any): Promise<void>;
 }
 
-export declare class Change {
+export declare class AppDescriptorChange {
     constructor(change: any);
     getLayer(): string;
-    _oDefinition: any;
 }
 
 export declare class V2MetadataConverter {