@sap/cds-dk 6.0.3 → 6.1.2

package/lib/init/template/approuter/index.js

@@ -1,4 +1,3 @@
-const fs = require('fs')
 const { join } = require('path')
 const TemplateBase = require('../templateBase')
 const ProjectReader = require('../../util/projectReader')
@@ -9,7 +8,7 @@ const {
     requiredMtxAPI, requiredJavaApprouterAPI, providedJavaApprouterAPI, // APIs
     requiredAppAPI, providedAppAPI,
     approuterExtensibility, approuterExtensibilityJava // xs-app.json config
-} = require('../_merging/existences')
+} = require('../_merging/registry-mta')
 
 module.exports = class ApprouterTemplate extends TemplateBase {
 
@@ -24,50 +23,41 @@ module.exports = class ApprouterTemplate extends TemplateBase {
 
     async runDependentMerging() {
         const projectDescriptor = await this.projectReader.read(this.options)
-        const mtaYAMLPath = join(this.projectPath, 'mta.yaml')
-        const { isNodejs, isJava, isMultitenant } = projectDescriptor.cap
-        if (fs.existsSync(mtaYAMLPath)) {
+        const { isNodejs, isJava, isExtensible, isMultitenant, hasMtaDeployment } = projectDescriptor.cap
+
+        if (hasMtaDeployment) {
             const srv = isNodejs ? srvNode : srvJava
             const apis = isJava ? [requiredJavaApprouterAPI, providedJavaApprouterAPI] : []
             if (isNodejs && isMultitenant) apis.push(requiredMtxAPI, requiredAppAPI, providedAppAPI)
             if (isJava && isMultitenant) apis.push(requiredMtxAPI)
-            const existences = [srv, approuter, xsuaa, ...apis]
-            await this.templateUtil.mergeYAML(
-                mtaYAMLPath,
-                `${__dirname}/files/mta.yaml.hbs`,
-                projectDescriptor,
-                {
-                    existences,
-                    relationships: [{
-                        in: srv,
-                        inKeyPath: ["requires"],
-                        into: "name",
-                        existences: [xsuaa],
-                        existenceKeyPath: ["name"],
-                    }, {
-                        in: approuter,
-                        inKeyPath: ["requires"],
-                        into: "name",
-                        existences: [xsuaa],
-                        existenceKeyPath: ["name"],
-                    }],
-                }
-            )
+            const additions = [srv, approuter, xsuaa, ...apis]
+            const mtaYAMLPath = join(this.projectPath, 'mta.yaml')
+            await this.templateUtil.mergeYAML(mtaYAMLPath, `${__dirname}/files/mta.yaml.hbs`, projectDescriptor, {
+                additions,
+                relationships: [{
+                    insert: [xsuaa, 'name'],
+                    into: [srv, 'requires', 'name']
+                }, {
+                    insert: [xsuaa, 'name'],
+                    into: [approuter, 'requires', 'name']
+                }],
+            })
         }
 
         const appPath = join(this.projectPath, projectDescriptor.ui.appPath || 'app')
-        const existences = isMultitenant && isJava ? [approuterExtensibilityJava] :
-                           isMultitenant && isNodejs ? [approuterExtensibility] : []
+        const additions = isExtensible && isJava ? [{ ...approuterExtensibilityJava, at: 0 }] :
+                          isExtensible && isNodejs ? [{ ...approuterExtensibility, at: 0 }] : []
         await this.templateUtil.mergeJSON(
             `${appPath}/xs-app.json`,
             `${__dirname}/files/xs-app.json.hbs`,
             projectDescriptor,
-            { existences }
+            { additions }
         )
     }
 
     async run() {
         const projectDescriptor = await this.projectReader.read(this.options)
+        const { for: forProfile } = projectDescriptor.cap
         const appPath = join(this.projectPath, projectDescriptor.ui.appPath || 'app')
         const appPackageJSONPath = join(appPath, 'package.json')
 
@@ -83,14 +73,14 @@ module.exports = class ApprouterTemplate extends TemplateBase {
         switch (projectType) {
             case PROJECT_TYPE.java: {
                 const cdsrcJSONPath = join(this.projectPath, '.cdsrc.json')
-                const cdsTemplateFile = projectDescriptor.cap.for ? 'cds.cdsrc.json.hbs' : 'cds.cdsrc.json'
+                const cdsTemplateFile = forProfile ? 'cds.cdsrc.json.hbs' : 'cds.cdsrc.json'
                 const cdsPackageJSONPath = join(__dirname, 'files', cdsTemplateFile)
                 await this.templateUtil.mergeJSON(cdsrcJSONPath, cdsPackageJSONPath, projectDescriptor)
                 break
             }
             case PROJECT_TYPE.nodejs: {
                 const packageJSONPath = join(this.projectPath, 'package.json')
-                const cdsTemplateFile = projectDescriptor.cap.for ? 'cds.package.json.hbs' : 'cds.package.json'
+                const cdsTemplateFile = forProfile ? 'cds.package.json.hbs' : 'cds.package.json'
                 const cdsPackageJSONPath = join(__dirname, 'files', cdsTemplateFile)
                 await this.templateUtil.mergeJSON(packageJSONPath, cdsPackageJSONPath, projectDescriptor)
                 break

package/lib/init/template/auditlog/index.js

@@ -2,7 +2,7 @@ const fs = require('fs');
 const path = require('path');
 const TemplateBase = require('../templateBase');
 const ProjectReader = require('../../util/projectReader');
-const { srvNode, srvJava, auditlog } = require('../_merging/existences');
+const { srvNode, srvJava, auditlog } = require('../_merging/registry-mta');
 
 module.exports = class AuditlogTemplate extends TemplateBase {
     constructor(projectPath, generator) {
@@ -12,22 +12,20 @@ module.exports = class AuditlogTemplate extends TemplateBase {
 
     async runDependentMerging() {
         const projectDescriptor = await this.projectReader.read(this.options);
+        const { isNodejs } = projectDescriptor.cap;
         const mtaYAMLPath = path.join(this.projectPath, 'mta.yaml');
         if (!fs.existsSync(mtaYAMLPath)) return;
-        const srv = projectDescriptor.cap.isNodejs ? srvNode : srvJava
+        const srv = isNodejs ? srvNode : srvJava
         await this.templateUtil.mergeYAML(
             mtaYAMLPath,
             `${__dirname}/files/mta.yaml.hbs`,
             projectDescriptor,
             {
-                existences: [srv, auditlog],
+                additions: [srv, auditlog],
                 relationships: [{
-                    in: srv,
-                    inKeyPath: ["requires"],
-                    into: "name",
-                    existences: [auditlog],
-                    existenceKeyPath: ["name"],
-                }],
+                    insert: [auditlog, 'name'],
+                    into: [srv, 'requires', 'name']
+                }]
             }
         );
     }

package/lib/init/template/data/index.js

@@ -148,9 +148,8 @@ function getNamespace(csn, artifactName) {
 
   // First step is not a namespace (we faked those in the CSN)
   // No subsequent step can be a namespace then
-  if (art && art.kind !== 'namespace') {
+  if (art && art.kind !== 'namespace' && art.kind !== 'context')
     return null;
-  }
 
 
   for (let i = 1; i < parts.length; i++) {
@@ -159,9 +158,8 @@ function getNamespace(csn, artifactName) {
     seen = `${seen}.${parts[i]}`;
     // This might not be - if it isn't, return the result.
     const currentArtifact = csn.definitions[seen];
-    if (currentArtifact && currentArtifact.kind !== 'namespace') {
+    if (currentArtifact && currentArtifact.kind !== 'namespace' && currentArtifact.kind !== 'context')
       return previousArtifactName;
-    }
   }
   // We came till here - so the full artifactName is a namespace
   return artifactName;

package/lib/init/template/extensibility/files/cds.package.json

@@ -0,0 +1,7 @@
+{
+  "cds": {
+    "requires": {
+      "extensibility": true
+    }
+  }
+}

package/lib/init/template/extensibility/files/cds.package.json.hbs

@@ -0,0 +1,9 @@
+{
+  "cds": {
+    "requires": {
+      "[{{cap.for}}]": {
+        "extensibility": true
+      }
+    }
+  }
+}

package/lib/init/template/extensibility/files/xs-security.json.hbs

@@ -0,0 +1,26 @@
+{
+  "scopes": [
+    {
+      "name": "$XSAPPNAME.cds.ExtensionDeveloper",
+      "description": "Extend CAP applications via extension projects"
+    },
+    {
+      "name": "$XSAPPNAME.cds.UIFlexDeveloper",
+      "description": "Extend CAP applications via UIFlex extensibility"
+    }
+  ],
+  "role-templates": [
+    {
+      "name": "ExtensionDeveloper",
+      "description": "Extension development including UIFlex extensibility",
+      "scope-references": [
+        "$XSAPPNAME.cds.ExtensionDeveloper",
+        "$XSAPPNAME.cds.UIFlexDeveloper"
+      ]
+    }
+  ],
+  "authorities": [
+    "$XSAPPNAME.cds.ExtensionDeveloper",
+    "$XSAPPNAME.cds.UIFlexDeveloper"
+  ]
+}

package/lib/init/template/extensibility/index.js

@@ -0,0 +1,48 @@
+const cds = require('../../../cds'), { path } = cds.utils
+const TemplateBase = require('../templateBase')
+const ProjectReader = require('../../util/projectReader')
+
+module.exports = class ExtensibilityTemplate extends TemplateBase {
+  constructor(projectPath, generator) {
+    super(projectPath, generator, __dirname)
+    this.projectReader = new ProjectReader(this)
+  }
+
+  async runDependentMerging() {
+    const projectDescriptor = await this.projectReader.read()
+    const { hasXsuaa } = projectDescriptor.cap
+    if (hasXsuaa) {
+      await this.templateUtil.mergeJSON(
+        path.join(this.projectPath, 'xs-security.json'),
+        path.join(__dirname, 'files', 'xs-security.json.hbs'),
+        projectDescriptor,
+        {
+          additions: [{
+            ref: 'scope-extension-developer',
+            in: 'scopes',
+            where: [{ property: 'name', isEqualTo: '$XSAPPNAME.cds.ExtensionDeveloper' }],
+          }, {
+            ref: 'scope-uiflex',
+            in: 'scopes',
+            where: [{ property: 'name', isEqualTo: '$XSAPPNAME.cds.UIFlexDeveloper' }],
+          }, {
+            ref: 'template-extension-developer',
+            in: 'role-templates',
+            where: [{ property: 'name', isEqualTo: 'ExtensionDeveloper' }],
+          }]
+        }
+      )
+    }
+  }
+
+  async run() {
+    const projectDescriptor = await this.projectReader.read(this.options)
+    const { for: forProfile, isJava } = projectDescriptor.cap
+    const basePath = isJava ? path.join(this.projectPath, 'mtx', 'sidecar') : this.projectPath
+    const packageJSONPath = path.join(basePath, 'package.json')
+    const cdsTemplateFile = forProfile ? 'cds.package.json.hbs' : 'cds.package.json'
+    await this.templateUtil.mergeJSON(packageJSONPath, path.join(__dirname, 'files', cdsTemplateFile), projectDescriptor)
+    await this.templateUtil.sortDependencies(packageJSONPath)
+    await this.runDependentMerging()
+  }
+}

package/lib/init/template/hana/files/java/cdsrc.json.hbs

@@ -1,7 +1,7 @@
 {
   "requires": {
     "db": {
-      "kind": { "#overwrite": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}" }
+      "kind": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}"
     }
   }
 }

package/lib/init/template/hana/files/java/profile.cdsrc.json.hbs

@@ -2,11 +2,11 @@
   "requires": {
     "[{{cap.for}}]": {
       "db": {
-        "kind": { "#overwrite": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}" }
+        "kind": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}"
       }
     },
     "db": {
-      "kind": "sql"
+      "kind": "{{#cap.isMultitenant}}sqlite{{/cap.isMultitenant}}{{^cap.isMultitenant}}sql{{/cap.isMultitenant}}"
     }
   }
 }

package/lib/init/template/hana/files/mta.yaml.hbs

@@ -15,6 +15,16 @@ modules:
       - name: {{cap.appName}}-db
 {{/cap.isMultitenant}}
 
+{{#cap.isJava}}
+{{#cap.isMultitenant}}
+  - name: {{cap.appName}}-mtx-sidecar
+    type: nodejs
+    path: mtx/sidecar
+    requires:
+      - name: {{cap.appName}}-db
+{{/cap.isMultitenant}}
+{{/cap.isJava}}
+
 resources:
   - name: {{cap.appName}}-db
     {{^cap.isMultitenant}}

package/lib/init/template/hana/files/nodejs/cds.package.json.hbs

@@ -2,7 +2,7 @@
   "cds": {
     "requires": {
       "db": {
-        "kind": { "#overwrite": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}" }
+        "kind": "{{#cap.isMultitenant}}sql-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}"
       }
     }
   }

package/lib/init/template/hana/files/nodejs/cds.profile.package.json.hbs

@@ -3,11 +3,11 @@
     "requires": {
       "[{{cap.for}}]": {
         "db": {
-          "kind": { "#overwrite": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}" }
+          "kind": "{{#cap.isMultitenant}}hana-mt{{/cap.isMultitenant}}{{^cap.isMultitenant}}hana-cloud{{/cap.isMultitenant}}"
         }
       },
       "db": {
-        "kind": "sql"
+        "kind": "{{#cap.isMultitenant}}sqlite{{/cap.isMultitenant}}{{^cap.isMultitenant}}sql{{/cap.isMultitenant}}"
       }
     }
   }

package/lib/init/template/hana/index.js

@@ -7,10 +7,11 @@ const term = require('../../../util/term');
 const ProjectReader = require('../../util/projectReader');
 
 const TemplateBase = require('../templateBase');
+const versionCompare = require('../../util/versionCompare');
 
 const { PROJECT_TYPE, OPTION_JAVA, URLS, OPTION_HELM_UPDATE } = require('../../constants');
 const { nullLogger } = require('../../util/logger');
-const { srvNode, srvJava, hdbDeployer, serviceManager, hdiContainer } = require('../_merging/existences');
+const { srvNode, srvJava, hdbDeployer, serviceManager, hdiContainer, mtxSidecar } = require('../_merging/registry-mta');
 
 module.exports = class HanaTemplate extends TemplateBase {
     constructor(projectPath, generator) {
@@ -34,66 +35,70 @@ module.exports = class HanaTemplate extends TemplateBase {
     }
 
     async checkEnabled() {
-        const { lt } = require('semver')
         const cds = require('../../../cds')
-        if (lt(cds.version, '5.9.0')) {
+        if (versionCompare(cds.version, '5.9.0') < 0) {
             cds.error('The added configuration requires @sap/cds 5.9.0 or higher. Please update @sap/cds to the latest version and run \x1b[1mcds add hana\x1b[0m again.')
         }
         return true
     }
 
     async runDependentMerging() {
-        const projectDescriptor = await this.projectReader.read(this.options);
+        const projectDescriptor = await this.projectReader.read(this.options)
+        const { hasMtaDeployment, isNodejs, isJava, isMultitenant, hasHana } = projectDescriptor.cap
+
+        if (isNodejs && isMultitenant) {
+            await this._updateCdsConfiguration()
+        } else if (isJava && isMultitenant) {
+            await this._updateCdsrcJson()
+        }
 
-        const mtaYAMLPath = path.join(this.projectPath, 'mta.yaml');
+        if (hasMtaDeployment) {
+            const srv = isNodejs ? srvNode : srvJava
+            const db = isMultitenant ? serviceManager : hdiContainer
+            const deletions = isMultitenant ? [{
+                item: hdbDeployer,
+                relationships: [{
+                    removeProperty: 'name',
+                    allWithin: ['modules', 'requires', 'name'],
+                }]
+            }, {
+                item: hdiContainer,
+                relationships: [{
+                    removeProperty: 'name',
+                    allWithin: ['modules', 'requires', 'name'],
+                }]
+            }] : []
 
-        if (!fs.existsSync(mtaYAMLPath)) return
+            const mtaYAMLPath = path.join(this.projectPath, 'mta.yaml')
 
-        const srv = projectDescriptor.cap.isNodejs ? srvNode : srvJava
-        const deployer = projectDescriptor.cap.isMultitenant ? null : hdbDeployer
-        const db = projectDescriptor.cap.isMultitenant ? serviceManager : hdiContainer
-        const deletions = projectDescriptor.cap.isMultitenant ? [{
-            existence: hdbDeployer,
-            relationships: [{
-                allWithinKeyPath: ["modules"],
-                inKeyPath: ["requires"],
-                into: "name",
-                existenceKeyPath: ["name"],
-            }]
-        }, {
-            existence: hdiContainer,
-            relationships: [{
-                allWithinKeyPath: ["modules"],
-                inKeyPath: ["requires"],
-                into: "name",
-                existenceKeyPath: ["name"],
+            const relationships = [{
+                insert: [db, 'name'],
+                into: [srv, 'requires', 'name']
             }]
-        }] : []
-
-        await this.templateUtil.mergeYAML(
-            mtaYAMLPath,
-            `${__dirname}/files/mta.yaml.hbs`,
-            projectDescriptor,
-            {
-                existences: [srv, deployer, db],
-                relationships: [{
-                    in: srv,
-                    inKeyPath: ["requires"],
-                    into: "name",
-                    existences: [db],
-                    existenceKeyPath: ["name"],
-                },
-                !projectDescriptor.cap.isMultitenant && projectDescriptor.cap.isNodejs ?
-                    {
-                        in: deployer,
-                        inKeyPath: ["requires"],
-                        into: "name",
-                        existences: [db],
-                        existenceKeyPath: ["name"],
-                    } : null],
-                deletions
+
+            if (!isMultitenant) {
+                relationships.push({
+                    insert: [hdiContainer, 'name'],
+                    into: [hdbDeployer, 'requires', 'name']
+                })
+            }
+
+            if (isJava && isMultitenant) {
+                relationships.push({
+                    insert: [serviceManager, 'name'],
+                    into: [mtxSidecar, 'requires', 'name']
+                })
             }
-        );
+
+            const modules = !isMultitenant ? [srv, hdbDeployer] : [srv]
+
+            await this.templateUtil.mergeYAML(
+                mtaYAMLPath,
+                `${__dirname}/files/mta.yaml.hbs`,
+                projectDescriptor,
+                { additions: [...modules, db].filter(a => a), deletions, relationships }
+            )
+        }
     }
 
     async run() {
@@ -112,14 +117,13 @@ module.exports = class HanaTemplate extends TemplateBase {
             case PROJECT_TYPE.java: {
                 await this.templateUtil.copyFiles(path.join('java', 'db'), path.join(this.projectPath, env.folders.db), {}, this.options.force);
                 await this._updatePomXml();
-                const cdsrcJsonPath = path.join(this.projectPath, '.cdsrc.json');
-                await this._updateCdsrcJson(cdsrcJsonPath, env.folders.db, env);
+                await this._updateCdsrcJson();
                 break;
             }
-            case PROJECT_TYPE.nodejs:
-                await this._updatePackageJson(path.join(this.projectPath, 'package.json'));
+            case PROJECT_TYPE.nodejs: {
+                await this._updatePackageJson()
                 break;
-
+            }
             default: break;
         }
         await this.runDependentMerging()
@@ -142,29 +146,38 @@ module.exports = class HanaTemplate extends TemplateBase {
         }
     }
 
-    async _updatePackageJson(packageJsonPath) {
-        const projectDescriptor = await this.projectReader.read(this.options);
+    async _updatePackageJson() {
+        const packageJsonPath = path.join(this.projectPath, 'package.json')
         const dependenciesTemplatePath = path.join(__dirname, 'files', 'nodejs', 'dependencies.package.json');
         await this.templateUtil.mergeJSON(packageJsonPath, dependenciesTemplatePath);
 
-        const cdsTemplateFile = projectDescriptor.cap.for ? 'cds.profile.package.json.hbs' : 'cds.package.json.hbs';
-        const cdsTemplatePath = path.join(__dirname, 'files', 'nodejs', cdsTemplateFile);
-        await this.templateUtil.mergeJSON(packageJsonPath, cdsTemplatePath, projectDescriptor);
-
         const packageJson = await this.fsUtil.readJSON(packageJsonPath);
         if (packageJson.dependencies['@sap/hana-client']) {
             delete packageJson.dependencies['@sap/hana-client'];
             await this.fsUtil.writeJSON(packageJsonPath, packageJson);
         }
+
+        await this._updateCdsConfiguration(packageJsonPath)
+    }
+
+    async _updateCdsConfiguration() {
+        const packageJsonPath = path.join(this.projectPath, 'package.json')
+        const projectDescriptor = await this.projectReader.read(this.options);
+        const { for: forProfile } = projectDescriptor.cap;
+        const cdsTemplateFile = forProfile ? 'cds.profile.package.json.hbs' : 'cds.package.json.hbs';
+        const cdsTemplatePath = path.join(__dirname, 'files', 'nodejs', cdsTemplateFile);
+        await this.templateUtil.mergeJSON(packageJsonPath, cdsTemplatePath, projectDescriptor, { forceOverwrite: true })
     }
 
-    async _updateCdsrcJson(cdsrcJsonPath) {
+    async _updateCdsrcJson() {
+        const cdsrcJsonPath = path.join(this.projectPath, '.cdsrc.json');
         const projectDescriptor = await this.projectReader.read(this.options);
-        const cdsTemplateFile = projectDescriptor.cap.for ? 'profile.cdsrc.json.hbs' : 'cdsrc.json.hbs';
+        const { for: forProfile } = projectDescriptor.cap;
+        const cdsTemplateFile = forProfile ? 'profile.cdsrc.json.hbs' : 'cdsrc.json.hbs';
         const cdsTemplatePath = path.join(__dirname, 'files', 'java', cdsTemplateFile);
         if (!fs.existsSync(cdsrcJsonPath)) {
             this.fsUtil.writeJSON(cdsrcJsonPath, {})
         }
-        await this.templateUtil.mergeJSON(cdsrcJsonPath, cdsTemplatePath, projectDescriptor);
+        await this.templateUtil.mergeJSON(cdsrcJsonPath, cdsTemplatePath, projectDescriptor, { forceOverwrite: true });
     }
 }

package/lib/init/template/kibana-logging/index.js

@@ -1,59 +1,53 @@
 const cds = require('../../../cds')
-const fs = require('fs');
-const path = require('path');
+const path = require('path')
 
-const ProjectReader = require('../../util/projectReader');
-
-const TemplateBase = require('../templateBase');
-
-const { PROJECT_TYPE, } = require('../../constants');
-const { srvNode, srvJava, kibanaLogging } = require('../_merging/existences');
+const ProjectReader = require('../../util/projectReader')
+const TemplateBase = require('../templateBase')
+const { PROJECT_TYPE, } = require('../../constants')
+const { srvNode, srvJava, kibanaLogging } = require('../_merging/registry-mta')
 
 module.exports = class KibanaTemplate extends TemplateBase {
     constructor(projectPath, generator) {
-        super(projectPath, generator, __dirname);
-        this.projectReader = new ProjectReader(this);
+        super(projectPath, generator, __dirname)
+        this.projectReader = new ProjectReader(this)
     }
 
     async runDependentMerging() {
-        const mtaYAMLPath = path.join(this.projectPath, 'mta.yaml');
-
-        if (!fs.existsSync(mtaYAMLPath)) {
-            return;
+        const projectDescriptor = await this.projectReader.read(this.options)
+        const { hasMtaDeployment, isNodejs } = projectDescriptor.cap
+
+        if (hasMtaDeployment) {
+            const srv = isNodejs ? srvNode : srvJava
+
+            const mtaYAMLPath = path.join(this.projectPath, 'mta.yaml')
+            await this.templateUtil.mergeYAML(
+                mtaYAMLPath,
+                `${__dirname}/files/mta.yaml.hbs`,
+                projectDescriptor,
+                {
+                    additions: [srv, kibanaLogging],
+                    relationships: [{
+                        insert: [kibanaLogging, 'name'],
+                        into: [srv, 'requires', 'name']
+                    }],
+                }
+            )
         }
-
-        const projectDescriptor = await this.projectReader.read(this.options);
-        const srv = projectDescriptor.cap.isNodejs ? srvNode : srvJava
-
-        await this.templateUtil.mergeYAML(
-            mtaYAMLPath,
-            `${__dirname}/files/mta.yaml.hbs`,
-            projectDescriptor,
-            {
-                existences: [srv, kibanaLogging],
-                relationships: [{
-                    in: srv,
-                    inKeyPath: ["requires"],
-                    into: "name",
-                    existences: [kibanaLogging],
-                    existenceKeyPath: ["name"],
-                }],
-            }
-        );
     }
 
     async run() {
-        const projectType = await this.getProjectType();
+        const projectType = await this.getProjectType()
 
         switch (projectType) {
             case PROJECT_TYPE.java:
             default:
-                cds.error(`kibana-logging is not implemented for project type '${projectType}' yet`);
+                cds.error(`kibana-logging is not implemented for project type '${projectType}' yet`)
                 // TODO: implement
-                break;
+                break
             case PROJECT_TYPE.nodejs: {
                 const projectDescriptor = await this.projectReader.read(this.options);
-                const cdsTemplateFile = projectDescriptor.cap.for ? 'cds.profile.package.json.hbs' : 'cds.package.json'
+                const { for: forProfile } = projectDescriptor.cap
+                const cdsTemplateFile = forProfile ? 'cds.profile.package.json.hbs' : 'cds.package.json'
                 const cdsTemplatePath = path.join(__dirname, 'files', cdsTemplateFile)
                 await this.templateUtil.mergeJSON(path.join(this.projectPath, 'package.json'), cdsTemplatePath, projectDescriptor)
             }