@sap-ux/adp-tooling 0.11.13 → 0.12.2

package/dist/base/abap.js

@@ -1,13 +1,4 @@
 "use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getManifest = void 0;
 const system_access_1 = require("@sap-ux/system-access");
@@ -20,32 +11,29 @@ const axios_extension_1 = require("@sap-ux/axios-extension");
  * @param {ToolsLogger} logger - The logger.
  * @returns {Promise<Manifest>} The manifest.
  */
-function getManifest(appId, adpConfig, logger) {
-    var _a, _b;
-    return __awaiter(this, void 0, void 0, function* () {
-        const provider = yield (0, system_access_1.createAbapServiceProvider)(adpConfig.target, {
-            ignoreCertErrors: (_a = adpConfig.ignoreCertErrors) !== null && _a !== void 0 ? _a : false
-        }, true, logger);
-        const appInfo = (yield provider.getAppIndex().getAppInfo(appId))[appId];
-        const manifestUrl = (_b = appInfo.manifestUrl) !== null && _b !== void 0 ? _b : appInfo.manifest;
-        if (!manifestUrl) {
-            throw new Error('Manifest URL not found');
+async function getManifest(appId, adpConfig, logger) {
+    const provider = await (0, system_access_1.createAbapServiceProvider)(adpConfig.target, {
+        ignoreCertErrors: adpConfig.ignoreCertErrors ?? false
+    }, true, logger);
+    const appInfo = (await provider.getAppIndex().getAppInfo(appId))[appId];
+    const manifestUrl = appInfo.manifestUrl ?? appInfo.manifest;
+    if (!manifestUrl) {
+        throw new Error('Manifest URL not found');
+    }
+    try {
+        const response = await provider.get(manifestUrl);
+        return JSON.parse(response.data);
+    }
+    catch (error) {
+        if ((0, axios_extension_1.isAxiosError)(error)) {
+            logger.error('Manifest fetching failed');
         }
-        try {
-            const response = yield provider.get(manifestUrl);
-            return JSON.parse(response.data);
+        else {
+            logger.error('Manifest parsing error: Manifest is not in expected format.');
         }
-        catch (error) {
-            if ((0, axios_extension_1.isAxiosError)(error)) {
-                logger.error('Manifest fetching failed');
-            }
-            else {
-                logger.error('Manifest parsing error: Manifest is not in expected format.');
-            }
-            logger.debug(error);
-            throw error;
-        }
-    });
+        logger.debug(error);
+        throw error;
+    }
 }
 exports.getManifest = getManifest;
 //# sourceMappingURL=abap.js.map

package/dist/base/change-utils.js

@@ -17,7 +17,6 @@ const project_access_1 = require("@sap-ux/project-access");
  * @returns {void}
  */
 function writeAnnotationChange(projectPath, data, change, fs) {
-    var _a;
     try {
         const { timestamp, annotation } = data;
         const changeFileName = `id_${timestamp}_addAnnotationsToOData.change`;
@@ -26,13 +25,13 @@ function writeAnnotationChange(projectPath, data, change, fs) {
         const annotationsFolderPath = path_1.default.join(changesFolderPath, project_access_1.DirName.Annotations);
         writeChangeToFile(changeFilePath, change, fs);
         if (!annotation.filePath) {
-            fs.write(path_1.default.join(annotationsFolderPath, (_a = annotation.fileName) !== null && _a !== void 0 ? _a : ''), '');
+            fs.write(path_1.default.join(annotationsFolderPath, annotation.fileName ?? ''), '');
         }
         else {
             const { filePath, fileName } = annotation;
             const selectedDir = path_1.default.dirname(filePath);
             if (selectedDir !== annotationsFolderPath) {
-                fs.copy(filePath, path_1.default.join(annotationsFolderPath, fileName !== null && fileName !== void 0 ? fileName : ''));
+                fs.copy(filePath, path_1.default.join(annotationsFolderPath, fileName ?? ''));
             }
         }
     }
@@ -127,7 +126,6 @@ exports.getParsedPropertyValue = getParsedPropertyValue;
  * @throws {Error} Throws an error if the change file cannot be read or if there's an issue accessing the directory.
  */
 function findChangeWithInboundId(projectPath, inboundId) {
-    var _a;
     let changeObj;
     let filePath = '';
     const pathToInboundChangeFiles = path_1.default.join(projectPath, project_access_1.DirName.Webapp, project_access_1.DirName.Changes, project_access_1.DirName.Manifest);
@@ -142,7 +140,7 @@ function findChangeWithInboundId(projectPath, inboundId) {
         for (const file of files) {
             const pathToFile = path_1.default.join(pathToInboundChangeFiles, file.name);
             const change = JSON.parse((0, fs_1.readFileSync)(pathToFile, 'utf-8'));
-            if (((_a = change.content) === null || _a === void 0 ? void 0 : _a.inboundId) === inboundId) {
+            if (change.content?.inboundId === inboundId) {
                 changeObj = change;
                 filePath = pathToFile;
                 break;

package/dist/base/prompt.js

@@ -1,13 +1,4 @@
 "use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -23,86 +14,83 @@ const helper_1 = require("./helper");
  * @param logger optional logger instance
  * @returns a configuration for the adp writer
  */
-function promptGeneratorInput(defaults, logger) {
-    var _a;
-    return __awaiter(this, void 0, void 0, function* () {
-        defaults = defaults !== null && defaults !== void 0 ? defaults : {};
-        const { target, apps, layer, customConfig } = yield promptTarget(defaults, logger);
-        const app = yield (0, prompts_1.default)([
-            {
-                type: 'autocomplete',
-                name: 'reference',
-                message: 'Original application:',
-                initial: defaults.reference,
-                choices: apps.map((app) => {
-                    var _a;
-                    return ({
-                        title: `${app['sap.app/title']} (${((_a = app['sap.fiori/registrationIds']) !== null && _a !== void 0 ? _a : []).join(',')})`,
-                        value: app['sap.app/id']
-                    });
-                }),
-                suggest: (input, choices) => Promise.resolve(choices.filter((i) => i.title.includes(input)))
-            },
-            {
-                type: 'text',
-                name: 'id',
-                message: (_prev) => {
-                    if (layer === 'CUSTOMER_BASE') {
-                        return 'New adaptation id (prefix "customer" will be automatically added to the id):';
-                    }
-                    else {
-                        return 'New adaptation id:';
-                    }
-                },
-                initial: defaults.id,
-                format: (input) => {
-                    if (layer === 'CUSTOMER_BASE' && !input.startsWith('customer.')) {
-                        return `customer.${input}`;
-                    }
-                    else {
-                        return input;
-                    }
-                },
-                validate: (input) => (input === null || input === void 0 ? void 0 : input.length) > 0
+async function promptGeneratorInput(defaults, logger) {
+    defaults = defaults ?? {};
+    const { target, apps, layer, customConfig } = await promptTarget(defaults, logger);
+    const app = await (0, prompts_1.default)([
+        {
+            type: 'autocomplete',
+            name: 'reference',
+            message: 'Original application:',
+            initial: defaults.reference,
+            choices: apps.map((app) => ({
+                title: `${app['sap.app/title']} (${(app['sap.fiori/registrationIds'] ?? []).join(',')})`,
+                value: app['sap.app/id']
+            })),
+            suggest: (input, choices) => Promise.resolve(choices.filter((i) => i.title.includes(input)))
+        },
+        {
+            type: 'text',
+            name: 'id',
+            message: (_prev) => {
+                if (layer === 'CUSTOMER_BASE') {
+                    return 'New adaptation id (prefix "customer" will be automatically added to the id):';
+                }
+                else {
+                    return 'New adaptation id:';
+                }
             },
-            {
-                type: 'text',
-                name: 'title',
-                message: 'Application title:'
-            }
-        ]);
-        const deploy = yield (0, prompts_1.default)([
-            {
-                type: 'text',
-                name: 'package',
-                message: 'Deployment package:',
-                initial: (_a = defaults.package) !== null && _a !== void 0 ? _a : '$TMP',
-                validate: (input) => (input === null || input === void 0 ? void 0 : input.length) > 0
+            initial: defaults.id,
+            format: (input) => {
+                if (layer === 'CUSTOMER_BASE' && !input.startsWith('customer.')) {
+                    return `customer.${input}`;
+                }
+                else {
+                    return input;
+                }
             },
-            {
-                type: 'text',
-                name: 'transport',
-                message: 'Transport request (optional):',
-                initial: defaults.transport
-            }
-        ]);
-        const options = yield (0, prompts_1.default)([
-            {
-                type: 'confirm',
-                name: 'fioriTools',
-                message: 'Enable Fiori tools?',
-                initial: defaults.ft !== false,
-                validate: (input) => (input === null || input === void 0 ? void 0 : input.length) > 0
-            }
-        ]);
-        return {
-            app: Object.assign(Object.assign({}, app), { layer }),
-            target,
-            options,
-            deploy,
-            customConfig
-        };
-    });
+            validate: (input) => input?.length > 0
+        },
+        {
+            type: 'text',
+            name: 'title',
+            message: 'Application title:'
+        }
+    ]);
+    const deploy = await (0, prompts_1.default)([
+        {
+            type: 'text',
+            name: 'package',
+            message: 'Deployment package:',
+            initial: defaults.package ?? '$TMP',
+            validate: (input) => input?.length > 0
+        },
+        {
+            type: 'text',
+            name: 'transport',
+            message: 'Transport request (optional):',
+            initial: defaults.transport
+        }
+    ]);
+    const options = await (0, prompts_1.default)([
+        {
+            type: 'confirm',
+            name: 'fioriTools',
+            message: 'Enable Fiori tools?',
+            initial: defaults.ft !== false,
+            validate: (input) => input?.length > 0
+        }
+    ]);
+    return {
+        app: {
+            ...app,
+            layer
+        },
+        target,
+        options,
+        deploy,
+        customConfig
+    };
 }
 exports.promptGeneratorInput = promptGeneratorInput;
 /**
@@ -112,51 +100,49 @@ exports.promptGeneratorInput = promptGeneratorInput;
  * @param logger logger instance
  * @returns apps, layer, target url and client
  */
-function promptTarget(defaults, logger) {
-    return __awaiter(this, void 0, void 0, function* () {
-        let count = 0;
-        let target = { url: defaults.url, client: defaults.client };
-        while (count < 3) {
-            try {
-                count++;
-                target = yield (0, prompts_1.default)([
-                    {
-                        type: 'text',
-                        name: 'url',
-                        message: 'Target system url:',
-                        initial: target.url,
-                        validate: helper_1.isNotEmptyString,
-                        format: (input) => input.trim()
-                    },
+async function promptTarget(defaults, logger) {
+    let count = 0;
+    let target = { url: defaults.url, client: defaults.client };
+    while (count < 3) {
+        try {
+            count++;
+            target = await (0, prompts_1.default)([
+                {
+                    type: 'text',
+                    name: 'url',
+                    message: 'Target system url:',
+                    initial: target.url,
+                    validate: helper_1.isNotEmptyString,
+                    format: (input) => input.trim()
+                },
+                {
+                    type: 'text',
+                    name: 'client',
+                    message: 'Client (optional):',
+                    initial: target.client,
+                    validate: helper_1.isValidSapClient
+                }
+            ]);
+            const systemInfo = await fetchSystemInformation(target, defaults.ignoreCertErrors, logger);
+            return { target, ...systemInfo };
+        }
+        catch (error) {
+            logger.error('Error while fetching system information. Please check your input.');
+            logger.debug(error.message);
+            if (error.code === 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY') {
+                logger.error('If you are using a self-signed certificate, please use the --ignore-cert-errors flag.');
+                const confirm = await (0, prompts_1.default)([
                     {
-                        type: 'text',
-                        name: 'client',
-                        message: 'Client (optional):',
-                        initial: target.client,
-                        validate: helper_1.isValidSapClient
+                        type: 'confirm',
+                        name: 'ignoreCertErrors',
+                        message: 'Do you want to ignore certificate errors?'
                     }
                 ]);
-                const systemInfo = yield fetchSystemInformation(target, defaults.ignoreCertErrors, logger);
-                return Object.assign({ target }, systemInfo);
-            }
-            catch (error) {
-                logger.error('Error while fetching system information. Please check your input.');
-                logger.debug(error.message);
-                if (error.code === 'UNABLE_TO_GET_ISSUER_CERT_LOCALLY') {
-                    logger.error('If you are using a self-signed certificate, please use the --ignore-cert-errors flag.');
-                    const confirm = yield (0, prompts_1.default)([
-                        {
-                            type: 'confirm',
-                            name: 'ignoreCertErrors',
-                            message: 'Do you want to ignore certificate errors?'
-                        }
-                    ]);
-                    defaults.ignoreCertErrors = confirm.ignoreCertErrors;
-                }
+                defaults.ignoreCertErrors = confirm.ignoreCertErrors;
             }
         }
-        throw new Error('Unable to fetch system information.');
-    });
+    }
+    throw new Error('Unable to fetch system information.');
 }
 exports.promptTarget = promptTarget;
 /**
@@ -167,33 +153,30 @@ exports.promptTarget = promptTarget;
  * @param logger logger instance
  * @returns app index and layer
  */
-function fetchSystemInformation(target, ignoreCertErrors, logger) {
-    var _a;
-    return __awaiter(this, void 0, void 0, function* () {
-        const provider = yield (0, system_access_1.createAbapServiceProvider)(target, {
-            ignoreCertErrors
-        }, true, logger);
-        logger.info('Fetching system information...');
-        const ato = yield provider.getAtoInfo();
-        const layer = ato.tenantType === 'SAP' ? 'VENDOR' : 'CUSTOMER_BASE';
-        const customConfig = {
-            adp: {
-                environment: (_a = ato.operationsType) !== null && _a !== void 0 ? _a : 'P',
-                safeMode: true
-            }
-        };
-        logger.info(`Target layer: ${layer}`);
-        logger.info('Fetching list of available applications... (it can take a moment)');
-        const appIndex = provider.getAppIndex();
-        const searchParams = {
-            'sap.ui/technology': 'UI5',
-            'sap.app/type': 'application'
-        };
-        if (customConfig.adp.environment === 'C') {
-            searchParams['sap.fiori/cloudDevAdaptationStatus'] = 'released';
+async function fetchSystemInformation(target, ignoreCertErrors, logger) {
+    const provider = await (0, system_access_1.createAbapServiceProvider)(target, {
+        ignoreCertErrors
+    }, true, logger);
+    logger.info('Fetching system information...');
+    const ato = await provider.getAtoInfo();
+    const layer = ato.tenantType === 'SAP' ? 'VENDOR' : 'CUSTOMER_BASE';
+    const customConfig = {
+        adp: {
+            environment: ato.operationsType ?? 'P',
+            safeMode: true
         }
-        const apps = yield appIndex.search(searchParams, ['sap.app/id', 'sap.app/title', 'sap.fiori/registrationIds']);
-        return { apps, layer, customConfig };
-    });
+    };
+    logger.info(`Target layer: ${layer}`);
+    logger.info('Fetching list of available applications... (it can take a moment)');
+    const appIndex = provider.getAppIndex();
+    const searchParams = {
+        'sap.ui/technology': 'UI5',
+        'sap.app/type': 'application'
+    };
+    if (customConfig.adp.environment === 'C') {
+        searchParams['sap.fiori/cloudDevAdaptationStatus'] = 'released';
+    }
+    const apps = await appIndex.search(searchParams, ['sap.app/id', 'sap.app/title', 'sap.fiori/registrationIds']);
+    return { apps, layer, customConfig };
 }
 //# sourceMappingURL=prompt.js.map

package/dist/i18n.js

@@ -1,13 +1,4 @@
 "use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -21,19 +12,17 @@ const adpI18nNamespace = 'adp-tooling';
  *
  * @returns {Promise<void>} A promise that resolves when the i18n initialization is completed.
  */
-function initI18n() {
-    return __awaiter(this, void 0, void 0, function* () {
-        yield i18next_1.default.init({
-            resources: {
-                en: {
-                    [adpI18nNamespace]: adp_tooling_i18n_json_1.default
-                }
-            },
-            lng: 'en',
-            fallbackLng: 'en',
-            defaultNS: adpI18nNamespace,
-            ns: [adpI18nNamespace]
-        });
+async function initI18n() {
+    await i18next_1.default.init({
+        resources: {
+            en: {
+                [adpI18nNamespace]: adp_tooling_i18n_json_1.default
+            }
+        },
+        lng: 'en',
+        fallbackLng: 'en',
+        defaultNS: adpI18nNamespace,
+        ns: [adpI18nNamespace]
     });
 }
 exports.initI18n = initI18n;
@@ -45,8 +34,8 @@ exports.initI18n = initI18n;
  * @returns {string} Localized string stored for the given key.
  */
 function t(key, options) {
-    if (!(options === null || options === void 0 ? void 0 : options.ns)) {
-        options = Object.assign(options !== null && options !== void 0 ? options : {}, { ns: adpI18nNamespace });
+    if (!options?.ns) {
+        options = Object.assign(options ?? {}, { ns: adpI18nNamespace });
     }
     return i18next_1.default.t(key, options);
 }

package/dist/preview/adp-preview.js

@@ -1,13 +1,4 @@
 "use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
@@ -21,6 +12,18 @@ const change_handler_1 = require("./change-handler");
  * Instance of an adaptation project handling requests and data transformation.
  */
 class AdpPreview {
+    config;
+    project;
+    util;
+    logger;
+    /**
+     * Merged descriptor variant with reference app manifest
+     */
+    mergedDescriptor;
+    /**
+     * Routes handler class to handle API requests
+     */
+    routesHandler;
     /**
      * @returns merged manifest.
      */
@@ -41,8 +44,7 @@ class AdpPreview {
                 [this.mergedDescriptor.name]: this.mergedDescriptor.url
             };
             this.mergedDescriptor.asyncHints.libs.forEach((lib) => {
-                var _a;
-                if ((_a = lib.url) === null || _a === void 0 ? void 0 : _a.url) {
+                if (lib.url?.url) {
                     resources[lib.name] = lib.url.url;
                 }
             });
@@ -73,21 +75,19 @@ class AdpPreview {
      * @param descriptorVariant descriptor variant from the project
      * @returns the UI5 flex layer for which editing is enabled
      */
-    init(descriptorVariant) {
-        return __awaiter(this, void 0, void 0, function* () {
-            const provider = yield (0, system_access_1.createAbapServiceProvider)(this.config.target, { ignoreCertErrors: this.config.ignoreCertErrors }, true, this.logger);
-            const lrep = provider.getLayeredRepository();
-            const zip = new adm_zip_1.default();
-            const files = yield this.project.byGlob('**/*.*');
-            for (const file of files) {
-                zip.addFile(file.getPath().substring(1), yield file.getBuffer());
-            }
-            const buffer = zip.toBuffer();
-            // fetch a merged descriptor from the backend
-            yield lrep.getCsrfToken();
-            this.mergedDescriptor = (yield lrep.mergeAppDescriptorVariant(buffer))[descriptorVariant.id];
-            return descriptorVariant.layer;
-        });
+    async init(descriptorVariant) {
+        const provider = await (0, system_access_1.createAbapServiceProvider)(this.config.target, { ignoreCertErrors: this.config.ignoreCertErrors }, true, this.logger);
+        const lrep = provider.getLayeredRepository();
+        const zip = new adm_zip_1.default();
+        const files = await this.project.byGlob('**/*.*');
+        for (const file of files) {
+            zip.addFile(file.getPath().substring(1), await file.getBuffer());
+        }
+        const buffer = zip.toBuffer();
+        // fetch a merged descriptor from the backend
+        await lrep.getCsrfToken();
+        this.mergedDescriptor = (await lrep.mergeAppDescriptorVariant(buffer))[descriptorVariant.id];
+        return descriptorVariant.layer;
     }
     /**
      * Proxy for the merged application manifest.json and blocking of preload files.
@@ -96,27 +96,25 @@ class AdpPreview {
      * @param res outgoing response object
      * @param next next middleware that is to be called if the request cannot be handled
      */
-    proxy(req, res, next) {
-        return __awaiter(this, void 0, void 0, function* () {
-            if (req.path === '/manifest.json') {
-                res.status(200);
-                res.send(JSON.stringify(this.descriptor.manifest, undefined, 2));
-            }
-            else if (req.path === '/Component-preload.js') {
-                res.status(404).send();
+    async proxy(req, res, next) {
+        if (req.path === '/manifest.json') {
+            res.status(200);
+            res.send(JSON.stringify(this.descriptor.manifest, undefined, 2));
+        }
+        else if (req.path === '/Component-preload.js') {
+            res.status(404).send();
+        }
+        else {
+            // check if the requested file exists in the file system (replace .js with .* for typescript)
+            const files = await this.project.byGlob(req.path.replace('.js', '.*'));
+            if (files.length === 1) {
+                // redirect to the exposed path so that other middlewares can handle it
+                res.redirect(302, req.path);
             }
             else {
-                // check if the requested file exists in the file system (replace .js with .* for typescript)
-                const files = yield this.project.byGlob(req.path.replace('.js', '.*'));
-                if (files.length === 1) {
-                    // redirect to the exposed path so that other middlewares can handle it
-                    res.redirect(302, req.path);
-                }
-                else {
-                    next();
-                }
+                next();
             }
-        });
+        }
     }
     /**
      * Add additional APIs to the router that are required for adaptation projects only.
@@ -160,24 +158,22 @@ class AdpPreview {
      * @param {Logger} logger - An instance of a logging interface for message logging.
      * @returns {Promise<void>} A promise that resolves when the change request has been processed.
      */
-    onChangeRequest(type, change, fs, logger) {
-        return __awaiter(this, void 0, void 0, function* () {
-            switch (type) {
-                case 'read':
-                    if (change_handler_1.moduleNameContentMap[change.changeType] && !change.moduleName) {
-                        (0, change_handler_1.tryFixChange)(change, logger);
-                    }
-                    break;
-                case 'write':
-                    if ((0, change_handler_1.isAddXMLChange)(change)) {
-                        (0, change_handler_1.addXmlFragment)(this.util.getProject().getSourcePath(), change, fs, logger);
-                    }
-                    break;
-                default:
-                    // no need to handle delete changes
-                    break;
-            }
-        });
+    async onChangeRequest(type, change, fs, logger) {
+        switch (type) {
+            case 'read':
+                if (change_handler_1.moduleNameContentMap[change.changeType] && !change.moduleName) {
+                    (0, change_handler_1.tryFixChange)(change, logger);
+                }
+                break;
+            case 'write':
+                if ((0, change_handler_1.isAddXMLChange)(change)) {
+                    (0, change_handler_1.addXmlFragment)(this.util.getProject().getSourcePath(), change, fs, logger);
+                }
+                break;
+            default:
+                // no need to handle delete changes
+                break;
+        }
     }
 }
 exports.AdpPreview = AdpPreview;