@ui5/task-adaptation 1.6.5 → 1.6.7

package/CHANGELOG.md

@@ -2,7 +2,13 @@
 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.6.5...HEAD).
+A list of unreleased changes can be found [here](https://github.com/SAP/ui5-task-adaptation/compare/v1.6.7...HEAD).
+
+<a name="v1.6.7"></a>
+## [v1.6.7] - 2026-07-28
+
+<a name="v1.6.6"></a>
+## [v1.6.6] - 2026-06-30
 
 <a name="v1.6.5"></a>
 ## [v1.6.5] - 2026-06-24
@@ -158,6 +164,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.6.7]: https://github.com/SAP/ui5-task-adaptation/compare/v1.6.6...v1.6.7
+[v1.6.6]: https://github.com/SAP/ui5-task-adaptation/compare/v1.6.5...v1.6.6
 [v1.6.5]: https://github.com/SAP/ui5-task-adaptation/compare/v1.6.4...v1.6.5
 [v1.6.4]: https://github.com/SAP/ui5-task-adaptation/compare/v1.6.3...v1.6.4
 [v1.6.3]: https://github.com/SAP/ui5-task-adaptation/compare/v1.6.2...v1.6.3

package/README.md

@@ -23,10 +23,6 @@ target:
   destination: abc
 ```
 
-### Local/Offline Build
-Specify environment variables `ADP_BUILDER_MODE=local` and `ADP_BUILDER_DIR=<path-to-base-app-files>` to build the Adaptation Project from local base app files offline - without accessing the remote repositories.
-
-### More
 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.

package/dist/adapters/abapAdapter.d.ts

@@ -1,5 +1,5 @@
 import { IAdapter } from "./adapter.js";
-import { AdaptCommandChain, PostCommandChain, SetupCommandChain } from "./commands/command.js";
+import { AdaptCommandChain, ManifestUpdateCommandChain, PostCommandChain, SetupCommandChain } from "./commands/command.js";
 import BaseApp from "../baseApp.js";
 import AppVariant from "../appVariant.js";
 import IAnnotationManager from "../annotations/annotationManager.js";
@@ -11,4 +11,5 @@ export default class AbapAdapter implements IAdapter {
     createSetupCommandChain(_appId: string, _repository: IRepository): SetupCommandChain;
     createAdaptCommandChain(baseApp: BaseApp, appVariant: AppVariant): AdaptCommandChain;
     createPostCommandChain(references: Map<string, string>, adaptationProject: AppVariant, ui5BuilderTools: UI5BuilderTools): PostCommandChain;
+    createManifestPreviewCommandChain(_baseApp: BaseApp, _appVariant: AppVariant): ManifestUpdateCommandChain;
 }

package/dist/adapters/abapAdapter.js

@@ -28,5 +28,8 @@ export default class AbapAdapter {
     createPostCommandChain(references, adaptationProject, ui5BuilderTools) {
         return new PostCommandChain(getCommonPostCommands(references, adaptationProject, ui5BuilderTools));
     }
+    createManifestPreviewCommandChain(_baseApp, _appVariant) {
+        throw new Error("previewManifest currently supports Cloud Foundry (CF) landscapes only");
+    }
 }
 //# sourceMappingURL=abapAdapter.js.map

package/dist/adapters/adapter.d.ts

@@ -3,11 +3,12 @@ import BaseApp from "../baseApp.js";
 import { IAppVariantIdHierarchyManifestItem } from "../model/appVariantIdHierarchyItem.js";
 import { UI5BuilderTools } from "../model/types.js";
 import IRepository from "../repositories/repository.js";
-import { AdaptCommandChain, IPromiseCommand, ManifestUpdateCommand, PostCommand, PostCommandChain, SetupCommandChain } from "./commands/command.js";
+import { AdaptCommandChain, IPromiseCommand, ManifestUpdateCommand, ManifestUpdateCommandChain, PostCommand, PostCommandChain, SetupCommandChain } from "./commands/command.js";
 export interface IAdapter {
     createSetupCommandChain(appId: string, repository: IRepository): SetupCommandChain;
     createAdaptCommandChain(baseApp: BaseApp, appVariant: AppVariant): AdaptCommandChain;
     createPostCommandChain(references: Map<string, string>, adaptationProject: AppVariant, ui5BuilderTools: UI5BuilderTools): PostCommandChain;
+    createManifestPreviewCommandChain(baseApp: BaseApp, appVariant: AppVariant): ManifestUpdateCommandChain;
 }
 export declare function getCommonManifestUpdateCommands(baseApp: BaseApp, appVariant: AppVariant, appVariantIdHierarchyItem: IAppVariantIdHierarchyManifestItem): ManifestUpdateCommand[];
 export declare function getCommonPostCommands(references: Map<string, string>, adaptationProject: AppVariant, ui5BuilderTools: UI5BuilderTools): PostCommand[];

package/dist/adapters/cfAdapter.d.ts

@@ -1,6 +1,6 @@
 import { IConfiguration } from "../model/configuration.js";
 import { IAdapter } from "./adapter.js";
-import { AdaptCommandChain, PostCommandChain, SetupCommandChain } from "./commands/command.js";
+import { AdaptCommandChain, ManifestUpdateCommandChain, PostCommandChain, SetupCommandChain } from "./commands/command.js";
 import BaseApp from "../baseApp.js";
 import AppVariant from "../appVariant.js";
 import { UI5BuilderTools } from "../model/types.js";
@@ -12,4 +12,5 @@ export default class CFAdapter implements IAdapter {
     createSetupCommandChain(_appId: string, _repository: IRepository): SetupCommandChain;
     createAdaptCommandChain(baseApp: BaseApp, appVariant: AppVariant): AdaptCommandChain;
     createPostCommandChain(references: Map<string, string>, adaptationProject: AppVariant, ui5BuilderTools: UI5BuilderTools): PostCommandChain;
+    createManifestPreviewCommandChain(baseApp: BaseApp, appVariant: AppVariant): ManifestUpdateCommandChain;
 }

package/dist/adapters/cfAdapter.js

@@ -1,7 +1,9 @@
 import { dependsOn, getCommonManifestUpdateCommands, getCommonPostCommands } from "./adapter.js";
 import { AdaptCommandChain, ManifestUpdateCommandChain, PostCommandChain, SetupCommandChain } from "./commands/command.js";
+import { getReferences } from "../util/commonUtil.js";
 import XsAppJsonEnhanceRoutesCommand from "./commands/xsAppJsonEnhanceRoutesCommand.js";
 import UpdateCloudPlatformCommand from "./commands/updateCloudPlatformCommand.js";
+import RenameManifestCommand from "./commands/renameManifestCommand.js";
 import I18nPropertiesMergeCommand from "./commands/i18nPropertiesMergeCommand.js";
 import XsAppJsonMergeCommand from "./commands/xsAppJsonMergeCommand.js";
 import FetchEndpointsCommand from "./commands/fetchEndpointsCommand.js";
@@ -39,5 +41,17 @@ export default class CFAdapter {
             ...getCommonPostCommands(references, adaptationProject, ui5BuilderTools)
         ]);
     }
+    createManifestPreviewCommandChain(baseApp, appVariant) {
+        const appVariantIdHierarchyItem = {
+            appVariantId: baseApp.id,
+            version: baseApp.version
+        };
+        const references = getReferences([appVariant], appVariant.id);
+        return new ManifestUpdateCommandChain([
+            ...getCommonManifestUpdateCommands(baseApp, appVariant, appVariantIdHierarchyItem),
+            new UpdateCloudPlatformCommand(this.configuration.sapCloudService),
+            new RenameManifestCommand(references),
+        ]);
+    }
 }
 //# sourceMappingURL=cfAdapter.js.map

package/dist/adapters/commands/command.d.ts

@@ -21,7 +21,7 @@ export declare abstract class AdaptCommand {
      * @param files - Map of all files being processed
      * @param filename - The current file being processed
      */
-    abstract execute(files: Map<string, string>, filename: string): Promise<void>;
+    abstract execute(files: Map<string, Buffer>, filename: string): Promise<void>;
 }
 export declare abstract class MergeCommand {
     readonly commandType = "merge";
@@ -32,7 +32,7 @@ export declare abstract class MergeCommand {
      * @param filename - The current file being processed
      * @param appVariantContent - Content from the app variant
      */
-    abstract execute(files: Map<string, string>, filename: string, appVariantContent: string): Promise<void>;
+    abstract execute(files: Map<string, Buffer>, filename: string, appVariantContent: Buffer): Promise<void>;
 }
 export declare abstract class ManifestUpdateCommand {
     readonly commandType = "manifestUpdate";
@@ -46,7 +46,7 @@ export declare class ManifestUpdateCommandChain extends AdaptCommand {
     private commands;
     constructor(commands?: ManifestUpdateCommand[]);
     accept: (filename: string) => filename is "manifest.json";
-    execute(files: Map<string, string>, filename: string): Promise<void>;
+    execute(files: Map<string, Buffer>, filename: string): Promise<void>;
 }
 export declare abstract class PostCommand {
     readonly commandType = "post";
@@ -54,7 +54,7 @@ export declare abstract class PostCommand {
      * Executes the command on the specified file
      * @param files - Map of all files being processed
      */
-    abstract execute(files: Map<string, string>): Promise<void>;
+    abstract execute(files: Map<string, Buffer>): Promise<void>;
 }
 /**
  * CommandChain implements the Chain of Responsibility pattern
@@ -67,15 +67,15 @@ export declare class PostCommandChain {
      * Executes all commands in the chain that accept the given filename
      * @param files - Map of all files being processed
      */
-    execute(files: ReadonlyMap<string, string>): Promise<ReadonlyMap<string, string>>;
+    execute(files: ReadonlyMap<string, Buffer>): Promise<ReadonlyMap<string, Buffer>>;
 }
 export declare class AdaptCommandChain {
     private files;
     private appVariantFiles;
     private adaptCommands;
     private mergeCommands;
-    constructor(files: ReadonlyMap<string, string>, appVariantFiles: ReadonlyMap<string, string>, commands: (AdaptCommand | MergeCommand)[]);
-    execute(): Promise<ReadonlyMap<string, string>>;
+    constructor(files: ReadonlyMap<string, Buffer>, appVariantFiles: ReadonlyMap<string, Buffer>, commands: (AdaptCommand | MergeCommand)[]);
+    execute(): Promise<ReadonlyMap<string, Buffer>>;
 }
 export interface IPromiseCommand<T> {
     result: Promise<T>;

package/dist/adapters/commands/command.js

@@ -1,3 +1,4 @@
+import { stringToBuffer, bufferToJson } from "../../util/commonUtil.js";
 import { getLogger } from "@ui5/logger";
 const log = getLogger("@ui5/task-adaptation::CommandChain");
 export class SetupCommand {
@@ -42,14 +43,14 @@ export class ManifestUpdateCommandChain extends AdaptCommand {
         if (!manifestContent) {
             throw new Error("Original application should have manifest.json in root folder");
         }
-        const manifest = JSON.parse(manifestContent);
+        const manifest = bufferToJson(manifestContent);
         const timings = [];
         for (const command of this.commands) {
             const start = performance.now();
             await command.execute(manifest);
             pushTiming(command, timings, start);
         }
-        files.set(filename, JSON.stringify(manifest));
+        files.set(filename, stringToBuffer(JSON.stringify(manifest)));
         logTimings(this, timings);
     }
 }

package/dist/adapters/commands/downloadAnnotationsCommand.d.ts

@@ -6,5 +6,5 @@ export default class DownloadAnnotationsCommand extends AdaptCommand {
     private annotationManager;
     constructor(appVariantId: string, prefix: string, annotationManager: IAnnotationManager);
     accept: (filename: string) => filename is "manifest.json";
-    execute(files: Map<string, string>, filename: string): Promise<void>;
+    execute(files: Map<string, Buffer>, filename: string): Promise<void>;
 }

package/dist/adapters/commands/downloadAnnotationsCommand.js

@@ -1,4 +1,5 @@
 import { AdaptCommand } from "./command.js";
+import { stringToBuffer, bufferToJson } from "../../util/commonUtil.js";
 export default class DownloadAnnotationsCommand extends AdaptCommand {
     appVariantId;
     prefix;
@@ -11,12 +12,12 @@ export default class DownloadAnnotationsCommand extends AdaptCommand {
     }
     accept = (filename) => filename === "manifest.json";
     async execute(files, filename) {
-        const baseAppManifest = JSON.parse(files.get(filename));
+        const baseAppManifest = bufferToJson(files.get(filename));
         let newFiles = await this.annotationManager.process(baseAppManifest, this.appVariantId, this.prefix);
         if (newFiles) {
-            newFiles.forEach((value, key) => files.set(key, value));
+            newFiles.forEach((value, key) => files.set(key, stringToBuffer(value)));
         }
-        files.set(filename, JSON.stringify(baseAppManifest));
+        files.set(filename, stringToBuffer(JSON.stringify(baseAppManifest)));
     }
 }
 //# sourceMappingURL=downloadAnnotationsCommand.js.map

package/dist/adapters/commands/fetchPreviewResourcesCommand.js

@@ -1,4 +1,5 @@
 import FsUtil from "../../util/fsUtil.js";
+import { stringToBuffer, bufferToString } from "../../util/commonUtil.js";
 import { posix as path } from "path";
 import { SetupCommand } from "./command.js";
 import { getLogger } from "@ui5/logger";
@@ -19,7 +20,7 @@ export default class FetchPreviewResourcesCommand extends SetupCommand {
     async preparePreview() {
         let ui5AppInfo = "";
         try {
-            ui5AppInfo = await FsUtil.readInProject("ui5AppInfo.json", "utf-8");
+            ui5AppInfo = bufferToString(await FsUtil.readInProject("ui5AppInfo.json"));
         }
         catch (error) {
             throw new Error(`ui5AppInfo.json is missing in project root, cannot process preview resources: ${error instanceof Error ? error.message : String(error)}`);
@@ -76,7 +77,8 @@ export default class FetchPreviewResourcesCommand extends SetupCommand {
             const newFilename = path.join(libraryName, filename);
             // Source path in xs-app.json needs to be adjusted to new location
             if (filename.includes("xs-app.json")) {
-                content = this.modifyRoutes(content, libraryName, libId);
+                const modified = this.modifyRoutes(bufferToString(content), libraryName, libId);
+                content = stringToBuffer(modified);
             }
             files.set(newFilename, content);
         });

package/dist/adapters/commands/filterFilesCommand.d.ts

@@ -1,4 +1,4 @@
 import { PostCommand } from "./command.js";
 export default class FilterFilesCommand extends PostCommand {
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
 }

package/dist/adapters/commands/filterFilesCommand.js

@@ -1,10 +1,11 @@
 import { PostCommand } from "./command.js";
+import { bufferToJson } from "../../util/commonUtil.js";
 export default class FilterFilesCommand extends PostCommand {
     async execute(files) {
         const IGNORE_FILES = ["manifest.appdescr_variant"];
         const shouldIgnore = (filename, content) => {
             if (filename.endsWith(".change")) {
-                return JSON.parse(content).changeType?.startsWith("appdescr_"); // validate JSON
+                return bufferToJson(content).changeType?.startsWith("appdescr_"); // validate JSON
             }
             return IGNORE_FILES.includes(filename);
         };

package/dist/adapters/commands/i18nPropertiesMergeCommand.d.ts

@@ -8,7 +8,7 @@ export default class I18nPropertiesMergeCommand extends MergeCommand {
     constructor(i18nPath: string, prefix: string, manifestChanges: ReadonlyArray<IChange>);
     accept: (filename: string) => boolean;
     private analyzeAppVariantManifestChanges;
-    execute(files: Map<string, string>, filename: string, appVariantContent: string): Promise<void>;
+    execute(files: Map<string, Buffer>, filename: string, appVariantContent: Buffer): Promise<void>;
     /**
      * Filters out specific lines from the given string.
      * Removes lines matching:

package/dist/adapters/commands/i18nPropertiesMergeCommand.js

@@ -1,4 +1,4 @@
-import { escapeRegex, trimExtension } from "../../util/commonUtil.js";
+import { stringToBuffer, bufferToString, bufferToNormalizedString, escapeRegex, trimExtension } from "../../util/commonUtil.js";
 import { posix as path } from "path";
 import { MergeCommand } from "./command.js";
 export default class I18nPropertiesMergeCommand extends MergeCommand {
@@ -45,11 +45,12 @@ export default class I18nPropertiesMergeCommand extends MergeCommand {
         // check if file matches with regex in merge/copy
         const mergePathMatch = this.mergePaths.map(path => filename.match(path)).find(match => match);
         const copyPathMatch = this.copyPaths.map(path => filename.match(path)).find(match => match);
+        const appVariantFileContent = bufferToNormalizedString(appVariantContent);
         if (mergePathMatch) {
-            this.mergePropertiesFiles(files, this.i18nPath, appVariantContent, mergePathMatch[1]);
+            this.mergePropertiesFiles(files, this.i18nPath, appVariantFileContent, mergePathMatch[1]);
         }
         if (copyPathMatch) {
-            files.set(path.join(this.prefix, filename), appVariantContent);
+            files.set(path.join(this.prefix, filename), stringToBuffer(appVariantFileContent));
         }
     }
     /**
@@ -76,12 +77,14 @@ export default class I18nPropertiesMergeCommand extends MergeCommand {
      */
     mergePropertiesFiles(files, i18nPath, appVariantFileContent, language = "") {
         const baseAppI18nPath = i18nPath + language + ".properties";
-        const baseAppFileContent = files.get(baseAppI18nPath);
-        const filteredBaseContent = baseAppFileContent ? this.filterTranslationMetaLines(baseAppFileContent) : "";
+        const baseAppFileBuffer = files.get(baseAppI18nPath);
+        const filteredBaseContent = baseAppFileBuffer
+            ? this.filterTranslationMetaLines(bufferToString(baseAppFileBuffer))
+            : "";
         const content = filteredBaseContent
             ? `${filteredBaseContent}\n\n#App variant specific text file\n\n${appVariantFileContent}`
             : appVariantFileContent;
-        files.set(baseAppI18nPath, content);
+        files.set(baseAppI18nPath, stringToBuffer(content));
     }
 }
 //# sourceMappingURL=i18nPropertiesMergeCommand.js.map

package/dist/adapters/commands/omitDeletedResourcesCommand.d.ts

@@ -6,5 +6,5 @@ export default class OmitDeletedResourcesCommand extends PostCommand {
     private resources;
     private projectNamespace;
     constructor(taskUtil: TaskUtil, movedFiles: Map<string, string>, resources: ReadonlyArray<Resource> | undefined, projectNamespace: string);
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
 }

package/dist/adapters/commands/omitDeletedResourcesCommand.js

@@ -21,7 +21,7 @@ export default class OmitDeletedResourcesCommand extends PostCommand {
                 if (newPath) {
                     const renamedContent = files.get(newPath);
                     if (renamedContent) {
-                        resource.setString(renamedContent);
+                        resource.setBuffer(renamedContent);
                     }
                 }
             }

package/dist/adapters/commands/processPreviewResourcesCommand.d.ts

@@ -4,6 +4,6 @@ export default class ProcessPreviewResourcesCommand extends PostCommand {
     private serviceCredentialsPromise;
     private previewPromise;
     constructor(serviceCredentialsPromise: Promise<ServiceCredentials | undefined>, previewPromise: Promise<FetchFilesPromise | undefined>);
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
     private collectReuseLibFiles;
 }

package/dist/adapters/commands/processPreviewResourcesCommand.js

@@ -1,9 +1,9 @@
 import { PostCommand } from "./command.js";
+import { stringToBuffer, bufferToString } from "../../util/commonUtil.js";
 import { getLogger } from "@ui5/logger";
-import { merge, XSAPP_JSON_FILENAME } from "../../util/cf/xsAppJsonUtil.js";
+import { fetchCredentialsAndEnhanceRoutes, merge, XSAPP_JSON_FILENAME } from "../../util/cf/xsAppJsonUtil.js";
 import ResourceUtil from "../../util/resourceUtil.js";
 import { REUSE_DIR } from "../../model/configuration.js";
-import XsAppJsonEnhanceRoutesCommand from "./xsAppJsonEnhanceRoutesCommand.js";
 const log = getLogger("@ui5/task-adaptation::ProcessPreviewResourcesCommand");
 export default class ProcessPreviewResourcesCommand extends PostCommand {
     serviceCredentialsPromise;
@@ -15,15 +15,21 @@ export default class ProcessPreviewResourcesCommand extends PostCommand {
     }
     async execute(files) {
         log.verbose(`Downloading reuse libraries to reuse folder`);
-        const baseAppXsAppJson = files.get(XSAPP_JSON_FILENAME);
+        const baseAppXsAppJsonBuffer = files.get(XSAPP_JSON_FILENAME);
+        const baseAppXsAppJson = baseAppXsAppJsonBuffer ? bufferToString(baseAppXsAppJsonBuffer) : undefined;
         const mergedFiles = await this.collectReuseLibFiles(await this.previewPromise, baseAppXsAppJson);
-        new XsAppJsonEnhanceRoutesCommand(this.serviceCredentialsPromise).execute(mergedFiles);
+        const xsAppJson = mergedFiles.get(XSAPP_JSON_FILENAME);
+        if (xsAppJson) {
+            const enhanced = await fetchCredentialsAndEnhanceRoutes(bufferToString(xsAppJson), this.serviceCredentialsPromise);
+            mergedFiles.set(XSAPP_JSON_FILENAME, stringToBuffer(enhanced));
+        }
         if (mergedFiles.size === 0) {
             return;
         }
         await ResourceUtil.writeInProject(REUSE_DIR, mergedFiles);
     }
-    async collectReuseLibFiles(fetchLibsPromises, mergedXsAppJson) {
+    async collectReuseLibFiles(fetchLibsPromises, baseAppXsAppJson) {
+        let mergedXsAppJson = baseAppXsAppJson;
         const mergedFiles = new Map();
         if (!fetchLibsPromises) {
             log.verbose("No reuse libraries defined in ui5AppInfo.json for preview");
@@ -37,13 +43,14 @@ export default class ProcessPreviewResourcesCommand extends PostCommand {
             }
             for (const [filename, content] of libFiles) {
                 if (filename.endsWith(XSAPP_JSON_FILENAME)) {
-                    mergedXsAppJson = mergedXsAppJson ? merge([content, mergedXsAppJson]) : content;
+                    const xsAppText = bufferToString(content);
+                    mergedXsAppJson = mergedXsAppJson ? merge([xsAppText, mergedXsAppJson]) : xsAppText;
                 }
                 mergedFiles.set(filename, content);
             }
         }
         if (mergedXsAppJson) {
-            mergedFiles.set(XSAPP_JSON_FILENAME, mergedXsAppJson);
+            mergedFiles.set(XSAPP_JSON_FILENAME, stringToBuffer(mergedXsAppJson));
         }
         return mergedFiles;
     }

package/dist/adapters/commands/renameFilesCommand.d.ts

@@ -2,7 +2,7 @@ import { PostCommand } from "./command.js";
 export default class RenameFilesCommand extends PostCommand {
     private references;
     constructor(references: Map<string, string>);
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
     /**
          * 1p. i18n.properties are not renamed and property keys are ignored during
          * the renaming across the files.
@@ -10,11 +10,19 @@ export default class RenameFilesCommand extends PostCommand {
          * @param references - Reference maps of the app variant
          * @returns A map of renamed files
          */
-    rename(files: Map<string, string>, references: Map<string, string>): void;
+    rename(files: Map<string, Buffer>, references: Map<string, string>): void;
     private getManifestSAPUI5DependencyIds;
     private getI18nPropertyKeys;
 }
 /**
  * We might rename appVariantIdHierarchy and dependencies, so we restore them after the renaming.
+ *
+ * The decorated method may receive either a Map of files (filename → Buffer) or
+ * a plain JSON object. Each handler is responsible for a single file (via its
+ * `accept`), operating on its parsed JSON content:
+ *  - Map case: every file a handler accepts is parsed, snapshotted before the
+ *    renaming, then restored and written back after.
+ *  - JSON case: the object is mutated in-place (its reference is preserved) and
+ *    every handler is applied to it.
  */
 export declare function restoreWhatShouldntBeRenamed(): (_target: any, _propertyKey: string, descriptor: PropertyDescriptor) => void;

package/dist/adapters/commands/renameFilesCommand.js

@@ -5,8 +5,15 @@ var __decorate = (this && this.__decorate) || function (decorators, target, key,
     return c > 3 && r && Object.defineProperty(target, key, r), r;
 };
 import ManifestRenamingHandler from "../../util/renamingHandlers/manifestRenamingHandler.js";
+import { stringToBuffer, bufferToString, bufferToJson, jsonToBuffer } from "../../util/commonUtil.js";
+import { getSAPUI5DependencyIds } from "../../util/manifestUtil.js";
 import { renameMap } from "../../util/renamingUtil.js";
+import { TEXT_EXTENSIONS } from "../../util/resourceUtil.js";
+import { posix as path } from "path";
 import { PostCommand } from "./command.js";
+function isTextFile(filename) {
+    return TEXT_EXTENSIONS.has(path.extname(filename).slice(1));
+}
 export default class RenameFilesCommand extends PostCommand {
     references;
     constructor(references) {
@@ -31,13 +38,17 @@ export default class RenameFilesCommand extends PostCommand {
         ];
         const updates = new Map();
         for (const [filename, content] of files) {
+            if (!isTextFile(filename)) {
+                continue;
+            }
             if (!IGNORE_EXTENSIONS.some(ext => filename.endsWith(ext))) {
                 // 5p. We pass replacements as ignores since we don't want to
                 // rename them again. E.g. we replace app.id with
                 // customer.app.id, but if we found customer.app.id somewhere,
                 // because we applied changes or something, we don't want to
                 // rename it again to customer.customer.app.id.
-                updates.set(filename, renameMap(content, references, [...references.values(), ...ignoreInString]));
+                const renamed = renameMap(bufferToString(content), references, [...references.values(), ...ignoreInString]);
+                updates.set(filename, stringToBuffer(renamed));
             }
         }
         for (const [filename, content] of updates) {
@@ -46,22 +57,13 @@ export default class RenameFilesCommand extends PostCommand {
     }
     getManifestSAPUI5DependencyIds(files) {
         const manifestFile = files.get("manifest.json");
-        if (manifestFile) {
-            const manifest = JSON.parse(manifestFile);
-            const dependencies = manifest["sap.ui5"]?.dependencies;
-            if (dependencies) {
-                const libs = dependencies.libs ? Object.keys(dependencies.libs) : [];
-                const components = dependencies.components ? Object.keys(dependencies.components) : [];
-                return [...libs, ...components];
-            }
-        }
-        return [];
+        return manifestFile ? getSAPUI5DependencyIds(bufferToJson(manifestFile)) : [];
     }
     getI18nPropertyKeys(files) {
         const keys = new Set();
         files.forEach((content, filename) => {
             if (filename.endsWith(".properties")) {
-                const lines = content.split("\n").filter(line => !line.startsWith("#"));
+                const lines = bufferToString(content).split("\n").filter(line => !line.startsWith("#"));
                 for (const line of lines) {
                     const [key] = line.split("=");
                     if (key) {
@@ -78,20 +80,46 @@ __decorate([
 ], RenameFilesCommand.prototype, "rename", null);
 /**
  * We might rename appVariantIdHierarchy and dependencies, so we restore them after the renaming.
+ *
+ * The decorated method may receive either a Map of files (filename → Buffer) or
+ * a plain JSON object. Each handler is responsible for a single file (via its
+ * `accept`), operating on its parsed JSON content:
+ *  - Map case: every file a handler accepts is parsed, snapshotted before the
+ *    renaming, then restored and written back after.
+ *  - JSON case: the object is mutated in-place (its reference is preserved) and
+ *    every handler is applied to it.
  */
 export function restoreWhatShouldntBeRenamed() {
     return function (_target, _propertyKey, descriptor) {
         const handlers = [new ManifestRenamingHandler()];
         const originalValue = descriptor.value;
         descriptor.value = function (...args) {
-            const files = args[0];
-            const references = args[1];
-            handlers.forEach(handler => handler.before(files));
-            originalValue.apply(this, [files, references]);
-            handlers.forEach(handler => handler.after(files));
+            const renaming = args[0];
+            const files = renaming;
+            forEachAccepted(files, handlers, (handler, content) => handler.before(bufferToJson(content)));
+            originalValue.apply(this, args);
+            forEachAccepted(files, handlers, (handler, content, filename) => {
+                const json = bufferToJson(content);
+                handler.after(json);
+                files.set(filename, jsonToBuffer(json));
+            });
             return files;
         };
     };
 }
 ;
+/**
+ * Parses each file that at least one handler accepts, hands the JSON to the
+ * given callback along with the accepting handlers, and — when `writeBack` is
+ * provided — serializes the (possibly mutated) JSON back into that map.
+ */
+function forEachAccepted(files, handlers, callback) {
+    for (const [filename, content] of files) {
+        for (const handler of handlers) {
+            if (handler.accept(filename)) {
+                callback(handler, content, filename);
+            }
+        }
+    }
+}
 //# sourceMappingURL=renameFilesCommand.js.map

package/dist/adapters/commands/renameManifestCommand.d.ts

@@ -0,0 +1,14 @@
+import { ManifestUpdateCommand } from "./command.js";
+/**
+ * Renames app variant references (base app id -> app variant id) inside the
+ * manifest.json object. This is the manifest scoped counterpart of
+ * {@link RenameFilesCommand}: it operates on the parsed manifest object rather
+ * than the raw file map, so it slots into the {@link ManifestUpdateCommandChain}
+ * alongside the other manifest updates instead of running as a post step.
+ */
+export default class RenameManifestCommand extends ManifestUpdateCommand {
+    private references;
+    private ignoreInStrings;
+    constructor(references: Map<string, string>, ignoreInStrings?: string[]);
+    execute(manifest: any): Promise<void>;
+}

package/dist/adapters/commands/renameManifestCommand.js

@@ -0,0 +1,32 @@
+import { ManifestUpdateCommand } from "./command.js";
+import { renameJson } from "../../util/renamingUtil.js";
+import { getSAPUI5DependencyIds } from "../../util/manifestUtil.js";
+import ManifestRenamingHandler from "../../util/renamingHandlers/manifestRenamingHandler.js";
+/**
+ * Renames app variant references (base app id -> app variant id) inside the
+ * manifest.json object. This is the manifest scoped counterpart of
+ * {@link RenameFilesCommand}: it operates on the parsed manifest object rather
+ * than the raw file map, so it slots into the {@link ManifestUpdateCommandChain}
+ * alongside the other manifest updates instead of running as a post step.
+ */
+export default class RenameManifestCommand extends ManifestUpdateCommand {
+    references;
+    ignoreInStrings;
+    constructor(references, ignoreInStrings = []) {
+        super();
+        this.references = references;
+        this.ignoreInStrings = ignoreInStrings;
+    }
+    async execute(manifest) {
+        const renamingHandler = new ManifestRenamingHandler();
+        renamingHandler.snapshot(manifest);
+        const ignoreInStrings = [
+            ...this.references.values(),
+            ...this.ignoreInStrings,
+            ...getSAPUI5DependencyIds(manifest)
+        ];
+        renameJson(manifest, this.references, ignoreInStrings);
+        renamingHandler.restore(manifest);
+    }
+}
+//# sourceMappingURL=renameManifestCommand.js.map

package/dist/adapters/commands/writeResourcesCommand.d.ts

@@ -3,5 +3,5 @@ export default class WriteResourcesCommand extends PostCommand {
     private workspace;
     private projectNamespace;
     constructor(workspace: any, projectNamespace: string);
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
 }

package/dist/adapters/commands/xsAppJsonEnhanceRoutesCommand.d.ts

@@ -3,5 +3,5 @@ import { PostCommand } from "./command.js";
 export default class XsAppJsonEnhanceRoutesCommand extends PostCommand {
     private serviceCredentialsPromise;
     constructor(serviceCredentialsPromise: Promise<ServiceCredentials | undefined>);
-    execute(files: Map<string, string>): Promise<void>;
+    execute(files: Map<string, Buffer>): Promise<void>;
 }

package/dist/adapters/commands/xsAppJsonEnhanceRoutesCommand.js

@@ -1,5 +1,6 @@
 import { fetchCredentialsAndEnhanceRoutes } from "../../util/cf/xsAppJsonUtil.js";
 import { PostCommand } from "./command.js";
+import { stringToBuffer, bufferToString } from "../../util/commonUtil.js";
 export default class XsAppJsonEnhanceRoutesCommand extends PostCommand {
     serviceCredentialsPromise;
     constructor(serviceCredentialsPromise) {
@@ -7,10 +8,10 @@ export default class XsAppJsonEnhanceRoutesCommand extends PostCommand {
         this.serviceCredentialsPromise = serviceCredentialsPromise;
     }
     async execute(files) {
-        let xsAppJson = files.get("xs-app.json");
+        const xsAppJson = files.get("xs-app.json");
         if (xsAppJson) {
-            xsAppJson = await fetchCredentialsAndEnhanceRoutes(xsAppJson, this.serviceCredentialsPromise);
-            files.set("xs-app.json", xsAppJson);
+            const enhanced = await fetchCredentialsAndEnhanceRoutes(bufferToString(xsAppJson), this.serviceCredentialsPromise);
+            files.set("xs-app.json", stringToBuffer(enhanced));
         }
     }
 }