@sap-ux/odata-service-writer 0.27.0 → 0.27.1

package/dist/data/defaults.d.ts

@@ -7,6 +7,7 @@ import type { Editor } from 'mem-fs-editor';
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {OdataService} service - the OData service instance
  * @param {Editor} fs - the memfs editor instance
+ * @param {boolean} update - whether the service update is running
  */
-export declare function enhanceData(basePath: string, service: OdataService, fs: Editor): Promise<void>;
+export declare function enhanceData(basePath: string, service: OdataService, fs: Editor, update?: boolean): Promise<void>;
 //# sourceMappingURL=defaults.d.ts.map

package/dist/data/defaults.js

@@ -147,9 +147,12 @@ async function setDefaultPreviewSettings(basePath, service, fs) {
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {OdataService} service - the OData service instance
  * @param {Editor} fs - the memfs editor instance
+ * @param {boolean} update - whether the service update is running
  */
-async function enhanceData(basePath, service, fs) {
-    setDefaultServicePath(service);
+async function enhanceData(basePath, service, fs, update = false) {
+    if (!update) {
+        setDefaultServicePath(service);
+    }
     await setDefaultServiceName(basePath, service, fs);
     await setDefaultServiceModel(basePath, service, fs);
     // set service type to EDMX if not defined

package/dist/index.d.ts

@@ -17,10 +17,11 @@ declare function generate(basePath: string, service: OdataService, fs?: Editor):
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {OdataService} service - the OData service instance
  * @param {Editor} [fs] - the memfs editor instance
+ * @param {boolean} updateMiddlewares - whether the YAML files for the service (mock-server and fiori-tools-proxy middlewares) should be updated
  * @throws {Error} - if required UI5 project files are not found
  * @returns {Promise<Editor>} the updated memfs editor instance
  */
-declare function update(basePath: string, service: OdataService, fs?: Editor): Promise<Editor>;
+declare function update(basePath: string, service: OdataService, fs?: Editor, updateMiddlewares?: boolean): Promise<Editor>;
 /**
  * Removes service related data from project files for an existing UI5 project specified by the base path.
  * How the method works:

package/dist/index.js

@@ -97,22 +97,23 @@ async function generate(basePath, service, fs) {
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {OdataService} service - the OData service instance
  * @param {Editor} [fs] - the memfs editor instance
+ * @param {boolean} updateMiddlewares - whether the YAML files for the service (mock-server and fiori-tools-proxy middlewares) should be updated
  * @throws {Error} - if required UI5 project files are not found
  * @returns {Promise<Editor>} the updated memfs editor instance
  */
-async function update(basePath, service, fs) {
+async function update(basePath, service, fs, updateMiddlewares = true) {
     if (!fs) {
         fs = (0, mem_fs_editor_1.create)((0, mem_fs_1.create)());
     }
     const paths = await findProjectFiles(basePath, fs);
     const webappPath = await (0, project_access_1.getWebappPath)(basePath, fs);
     ensureExists(webappPath, ['manifest.json'], fs);
-    await (0, data_1.enhanceData)(basePath, service, fs);
+    await (0, data_1.enhanceData)(basePath, service, fs, true);
     // Set isServiceTypeEdmx true if service is EDMX
     const isServiceTypeEdmx = service.type === types_1.ServiceType.EDMX;
     await (0, manifest_1.updateManifest)(basePath, service, fs, true);
     // Dont extend/update backend and mockserver middlewares if service type is CDS
-    if (isServiceTypeEdmx) {
+    if (isServiceTypeEdmx && updateMiddlewares) {
         await (0, update_1.updateServicesData)(basePath, paths, service, fs);
     }
     return fs;

package/dist/update.d.ts

@@ -15,7 +15,6 @@ export declare function addServicesData(basePath: string, paths: ProjectPaths, t
 /**
  * Updates services data in ui5-*.yaml files.
  * Mockserver configuration for services and annotations are updated using dataSources from manifest.json.
- * At the end, older XML files for service annotations are removed and new annotations are generated.
  *
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {ProjectPaths} paths - paths to the project files (package.json, ui5.yaml, ui5-local.yaml and ui5-mock.yaml)

package/dist/update.js

@@ -27,28 +27,36 @@ async function generateMockserverMiddlewareBasedOnUi5MockYaml(fs, ui5YamlPath, u
     }
 }
 /**
- * Extends backend middleware for UI5Config with service data.
+ * Extends backend middleware for UI5Config with the service data.
  *
  * @param {Editor} fs - the memfs editor instance
  * @param {OdataService} service - the OData service instance data
  * @param {UI5Config} ui5Config - UI5 configuration
  * @param {string} ui5ConfigPath - path to the YAML config file
+ * @param {boolean} update - whether the service update is running
  * @throws {Error} - if required UI5 project files are not found
  */
-function extendBackendMiddleware(fs, service, ui5Config, ui5ConfigPath) {
-    try {
-        ui5Config.addBackendToFioriToolsProxydMiddleware(service.previewSettings, service.ignoreCertError);
+function extendBackendMiddleware(fs, service, ui5Config, ui5ConfigPath, update = false) {
+    if (update) {
+        ui5Config.updateBackendToFioriToolsProxydMiddleware(service.previewSettings);
     }
-    catch (error) {
-        if ((error instanceof ui5_config_1.YAMLError && error.code === ui5_config_1.yamlErrorCode.nodeNotFound) ||
-            error.message === 'Could not find fiori-tools-proxy') {
-            ui5Config.addFioriToolsProxydMiddleware({
-                backend: [service.previewSettings],
-                ignoreCertError: service.ignoreCertError
-            });
+    else {
+        // Try to add backend
+        try {
+            ui5Config.addBackendToFioriToolsProxydMiddleware(service.previewSettings, service.ignoreCertError);
         }
-        else {
-            throw error;
+        catch (error) {
+            if ((error instanceof ui5_config_1.YAMLError && error.code === ui5_config_1.yamlErrorCode.nodeNotFound) ||
+                error.message === 'Could not find fiori-tools-proxy') {
+                // Middleware is missing, add it along with the service backend
+                ui5Config.addFioriToolsProxydMiddleware({
+                    backend: [service.previewSettings],
+                    ignoreCertError: service.ignoreCertError
+                });
+            }
+            else {
+                throw error;
+            }
         }
     }
     fs.write(ui5ConfigPath, ui5Config.toString());
@@ -108,7 +116,6 @@ async function addServicesData(basePath, paths, templateRoot, service, fs) {
 /**
  * Updates services data in ui5-*.yaml files.
  * Mockserver configuration for services and annotations are updated using dataSources from manifest.json.
- * At the end, older XML files for service annotations are removed and new annotations are generated.
  *
  * @param {string} basePath - the root path of an existing UI5 application
  * @param {ProjectPaths} paths - paths to the project files (package.json, ui5.yaml, ui5-local.yaml and ui5-mock.yaml)
@@ -118,37 +125,45 @@ async function addServicesData(basePath, paths, templateRoot, service, fs) {
 async function updateServicesData(basePath, paths, service, fs) {
     let ui5Config;
     let ui5LocalConfig;
+    let ui5MockConfig;
     if (paths.ui5Yaml) {
         ui5Config = await ui5_config_1.UI5Config.newInstance(fs.read(paths.ui5Yaml));
+        // Update ui5.yaml with backend middleware
+        extendBackendMiddleware(fs, service, ui5Config, paths.ui5Yaml, true);
     }
+    // Update ui5-local.yaml with backend middleware
     if (paths.ui5LocalYaml) {
         ui5LocalConfig = await ui5_config_1.UI5Config.newInstance(fs.read(paths.ui5LocalYaml));
+        extendBackendMiddleware(fs, service, ui5LocalConfig, paths.ui5LocalYaml, true);
     }
     // For update, updatable files should already exist
     if (service.metadata) {
         const webappPath = await (0, project_access_1.getWebappPath)(basePath, fs);
         // Generate mockserver only when ui5-mock.yaml already exists
-        if (paths.ui5MockYaml) {
-            if (paths.ui5Yaml && ui5Config) {
-                const config = {
-                    webappPath: webappPath,
-                    // Since ui5-mock.yaml already exists, set 'skip' to skip package.json file updates
-                    packageJsonConfig: {
-                        skip: true
-                    },
-                    // Set 'overwrite' to true to overwrite services data in YAML files
-                    ui5MockYamlConfig: {
-                        overwrite: true
-                    }
-                };
-                // Regenerate mockserver middleware for ui5-mock.yaml by overwriting
-                await (0, mockserver_config_writer_1.generateMockserverConfig)(basePath, config, fs);
-                // Update ui5-local.yaml with mockserver middleware from updated ui5-mock.yaml
-                await generateMockserverMiddlewareBasedOnUi5MockYaml(fs, paths.ui5Yaml, paths.ui5LocalYaml, ui5LocalConfig);
-                if (paths.ui5LocalYaml && ui5LocalConfig) {
-                    // write ui5 local yaml if service type is not CDS
-                    fs.write(paths.ui5LocalYaml, ui5LocalConfig.toString());
+        if (paths.ui5MockYaml && paths.ui5Yaml && ui5Config) {
+            const config = {
+                webappPath: webappPath,
+                // Since ui5-mock.yaml already exists, set 'skip' to skip package.json file updates
+                packageJsonConfig: {
+                    skip: true
+                },
+                // Set 'overwrite' to true to overwrite services data in YAML files
+                ui5MockYamlConfig: {
+                    overwrite: true
                 }
+            };
+            // Regenerate mockserver middleware for ui5-mock.yaml by overwriting
+            await (0, mockserver_config_writer_1.generateMockserverConfig)(basePath, config, fs);
+            // Update ui5-local.yaml with mockserver middleware from updated ui5-mock.yaml
+            await generateMockserverMiddlewareBasedOnUi5MockYaml(fs, paths.ui5Yaml, paths.ui5LocalYaml, ui5LocalConfig);
+            // Update ui5-mock.yaml with backend middleware
+            if (paths.ui5MockYaml) {
+                ui5MockConfig = await ui5_config_1.UI5Config.newInstance(fs.read(paths.ui5MockYaml));
+                extendBackendMiddleware(fs, service, ui5MockConfig, paths.ui5MockYaml, true);
+            }
+            if (paths.ui5LocalYaml && ui5LocalConfig) {
+                // write ui5 local yaml if service type is not CDS
+                fs.write(paths.ui5LocalYaml, ui5LocalConfig.toString());
             }
         }
         // Write metadata.xml file

package/package.json

@@ -9,7 +9,7 @@
   "bugs": {
     "url": "https://github.com/SAP/open-ux-tools/issues?q=is%3Aopen+is%3Aissue+label%3Abug+label%3Aodata-service-writer"
   },
-  "version": "0.27.0",
+  "version": "0.27.1",
   "license": "Apache-2.0",
   "main": "dist/index.js",
   "files": [
@@ -27,9 +27,9 @@
     "mem-fs-editor": "9.4.0",
     "prettify-xml": "1.2.0",
     "semver": "7.5.4",
-    "@sap-ux/mockserver-config-writer": "0.9.0",
-    "@sap-ux/project-access": "1.30.0",
-    "@sap-ux/ui5-config": "0.28.0"
+    "@sap-ux/mockserver-config-writer": "0.9.1",
+    "@sap-ux/project-access": "1.30.1",
+    "@sap-ux/ui5-config": "0.28.1"
   },
   "devDependencies": {
     "@types/ejs": "3.1.2",