@sap/cds-dk 7.3.2 → 7.4.0

package/bin/build.js

@@ -1,7 +1,7 @@
 module.exports = Object.assign(build, {
     options: ['--project', '--src', '--dest', '--for', '--use', '--opts', '--log-level', '--options-model'],
     shortcuts: ['-in', '-s', '-o', '-4'],
-    flags: ['--ws', '--clean'],
+    flags: ['--clean', '--no-clean', '--ws'], // REVISIT: --clean is deprecated, backward compatibility
     help: `
 # SYNOPSIS
 
@@ -16,7 +16,6 @@ module.exports = Object.assign(build, {
 # OPTIONS
 
      *-4 | --for* hana | java | nodejs | mtx | mtx-sidecar | mtx-extension
-
         Name of the build task that is applied to the source folder, either
         explicitly configured in the build task list or generated out of the
         project setup
@@ -25,7 +24,6 @@ module.exports = Object.assign(build, {
         otherwise.
 
     *-s | --src* <folder>
-
         The module folder to build using the build task defined by option
         '--for'.
         This option is only supported when building an individual module of the
@@ -36,7 +34,6 @@ module.exports = Object.assign(build, {
         *hana* build task, 'srv' for *nodejs* or *java* build task.
 
     *-o | --dest* <folder>
-
         Output folder of the build task. The type of build task is defined by
         option '--for'.
         The folder is resolved based on '<project>/<cds.env.build.target>'.
@@ -49,14 +46,12 @@ module.exports = Object.assign(build, {
         '<project>/*<dest>*/*'.
 
     *--opts* <build task specific options>
-
         Add a comma-separated list of build task specific options in the format
         <opt1>=[value1,value2],<opt2>=value.
         For example, --opts model=[db,srv,schema.cds] loads and compiles the CDS
         model based on the given sources.
 
     *--profile* <profile,...>
-
         Loads the configuration data for the specified profile(s) and executes
         'cds build' based on this profile settings.
         E.g. you may want to create the HANA deployment artefacts only if the
@@ -64,18 +59,28 @@ module.exports = Object.assign(build, {
         The *development* profile is used by default.
 
     *--production*
-
         Shortcut for '--profile production'
 
     *--log-level* <debug | info | warn | error>
-
         Choose which level of cds build messages to log, default is *warn*.
 
     *-in | --project* [<project>]
-
         Runs 'cds build' for the specified project, default is current working
         directory.
-
+        
+    *--no-clean* (beta)
+        Skips cleaning the build output folder before running the actual build
+        operation. Clients need to clean the output folder upfront to ensure
+        consistent build results.
+
+    *--ws* (beta)
+        Enable tarball based packaging of npm workspace dependencies for Node.js apps.
+        This feature can be used to package npm workspace dependencies as part of your 
+        mono or multi-repo deployment.
+        Tarball files are created for all 'dependencies' and 'devDependencies' 
+        of your project for which a npm workspace exists. The '*' dependency is replaced
+        by the correct tarball file URL.
+        See 'npm workspaces' for further details.
 
 # EXAMPLES
 
@@ -105,13 +110,17 @@ async function build([project], options = {}) {
     if (Number(major) < 6) {
         throw "This application uses @sap/cds version < 6, which is not compatible with the installed @sap/cds-dk version >= 7. Update @sap/cds to version >= 6."
     }
-    if (options.clean === "false") {
-        throw "cds build option '--clean' is deprecated - build output folders are always cleaned before build"
-    }
     if (typeof options["options-model"] === "string") {
         options.opts = "model=[" + options["options-model"].split(",").map((entry) => entry.trim()) + "]"
         delete options["options-model"]
     }
+    if (options["no-clean"]) {
+        // mapping to the existing 'clean' option
+        if (options["no-clean"] === "true" || options["no-clean"] === true) {
+            options.clean = false
+        }
+        delete options["no-clean"]
+    }
 
     await build(options)
 }

package/bin/cds.js

@@ -93,8 +93,8 @@ const cli = { //NOSONAR
 
     function add (k,v) { options[k.slice(2)] = v || true }
     function add_global (k,v='') {
-      if (k === '--production') return process.env.CDS_ENV = [...process.env.CDS_ENV?.split(',') ?? [], 'production']
-      if (k === '--profile') return process.env.CDS_ENV = [...process.env.CDS_ENV?.split(',') ?? [], ...v.split(',')]
+      if (k === '--production') return process.env.CDS_ENV = Array.from(new Set([...process.env.CDS_ENV?.split(',') ?? [], 'production']))
+      if (k === '--profile')    return process.env.CDS_ENV = Array.from(new Set([...process.env.CDS_ENV?.split(',') ?? [], ...v.split(',')]))
       if (k === '--odata') v = { flavor:v }
       let e=env || (env={}), path = k.slice(2).split('-')
       while (path.length > 1) { let p = path.shift(); e = e[p]||(e[p]={}) }
@@ -103,7 +103,7 @@ const cli = { //NOSONAR
 
     // if (env) require('../lib/cds').env.add (env) //> avoid loading env
     // unless it has already explicitly been set by the user
-    process.env.CDS_CONFIG ??= JSON.stringify(env)
+    if (env) process.env.CDS_CONFIG ??= JSON.stringify(env)
     return [ args, options ]
   },
 

package/bin/migrate.js

@@ -43,11 +43,11 @@ module.exports = Object.assign(migrate, {
         Skip verification step. Does not check for potential deployment
         differences.
 
-    *--cleanup* (beta)
+    *--cleanup*
 
         Removes all metadata containers created by @sap/cds-mtx.
 
-    *--resolve-bindings* (beta)
+    *--resolve-bindings*
 
         Resolve remote service bindings configured via *cds bind*.
 

package/bin/repl.js

@@ -17,8 +17,6 @@ async function repl (_, options={}) {
   const cds = require('../lib/cds')
   console.log('\x1b[32m\x1b[1m'+'Welcome to cds repl v', cds.version, '\x1b[0m')
 
-  await cds.plugins
-
   const { inspect } = require('util')
   inspect.defaultOptions.colors = true
   inspect.defaultOptions.depth = 11

package/bin/subscribe.js

@@ -1,8 +1,8 @@
 module.exports = Object.assign(subscribe, {
     init: cmd => new (require('../lib/client/tenant_action'))(cmd),
     options: ['--to', '--user'],
-    flags: ['--local'],
-    shortcuts: ['-2', '-u', '-l'],
+    flags: ['--local', '--resolve-bindings'],
+    shortcuts: ['-2', '-u', '-l', '-b'],
     help: `
 # SYNOPSIS
 
@@ -24,10 +24,10 @@ module.exports = Object.assign(subscribe, {
 
         Username and optionally password for authentication with Basic Auth in
         test scenarios.
-        
+
         Only relevant if <url> is given. May be omitted in case of dummy
         authentication.
-        
+
         Tokens saved with *cds login* are not used for this command.
 
     *-l* | *--local*
@@ -35,11 +35,15 @@ module.exports = Object.assign(subscribe, {
         Run a locally-installed @sap/cds-mtxs and connect to it instead of a
         URL. Ignore settings saved with *cds login*.
 
+    *-b* | *--resolve-bindings*
+
+        Resolve remote service bindings configured via *cds bind*.
+
 
 # SEE ALSO
 
     *cds login* for more information on authentication options.
-    
+
     *cds login* to save project settings and Basic-Auth data, simplifying multiple runs of this command.
 
 `});
@@ -63,6 +67,7 @@ async function subscribe([tenant], options = {}) {
     }
     const [username, password] = options.user?.split(':') ?? [];
     delete options.user;
+    delete options['resolve-bindings'];
 
     await subscribe.init('subscribe').run({
         ...options,

package/bin/unsubscribe.js

@@ -1,8 +1,8 @@
 module.exports = Object.assign(unsubscribe, {
     init: cmd => new (require('../lib/client/tenant_action'))(cmd),
     options: ['--from', '--user'],
-    flags: ['--local'],
-    shortcuts: ['-f', '-u', '-l'],
+    flags: ['--local', '--resolve-bindings'],
+    shortcuts: ['-f', '-u', '-l', '-b'],
     help: `
 # SYNOPSIS
 
@@ -25,10 +25,10 @@ module.exports = Object.assign(unsubscribe, {
 
         Username and optionally password for authentication with Basic Auth in
         test scenarios.
-        
+
         Only relevant if <url> is given. May be omitted in case of dummy
         authentication.
-        
+
         Tokens saved with *cds login* are not used for this command.
 
     *-l* | *--local*
@@ -36,11 +36,15 @@ module.exports = Object.assign(unsubscribe, {
         Run a locally-installed @sap/cds-mtxs and connect to it instead of a
         URL. Ignore settings saved with *cds login*.
 
+    *-b* | *--resolve-bindings*
+
+        Resolve remote service bindings configured via *cds bind*.
+
 
 # SEE ALSO
 
     *cds login* for more information on authentication options.
-    
+
     *cds login* to save project settings and Basic-Auth data, simplifying multiple runs of this command.
 
 `});
@@ -64,6 +68,7 @@ async function unsubscribe([tenant], options = {}) {
     }
     const [username, password] = options.user?.split(':') ?? [];
     delete options.user;
+    delete options['resolve-bindings'];
 
     await unsubscribe.init('unsubscribe').run({
         ...options,

package/bin/upgrade.js

@@ -1,8 +1,8 @@
 module.exports = Object.assign(upgrade, {
     init: cmd => new (require('../lib/client/tenant_action'))(cmd),
     options: ['--at', '--user'],
-    flags: ['--local'],
-    shortcuts: ['-a', '-u', '-l'],
+    flags: ['--local', '--resolve-bindings'],
+    shortcuts: ['-a', '-u', '-l', '-b'],
     help: `
 # SYNOPSIS
 
@@ -25,10 +25,10 @@ module.exports = Object.assign(upgrade, {
 
         Username and optionally password for authentication with Basic Auth in
         test scenarios.
-        
+
         Only relevant if <url> is given. May be omitted in case of dummy
         authentication.
-        
+
         Tokens saved with *cds login* are not used for this command.
 
     *-l* | *--local*
@@ -36,11 +36,15 @@ module.exports = Object.assign(upgrade, {
         Run a locally-installed @sap/cds-mtxs and connect to it instead of a
         URL. Ignore settings saved with *cds login*.
 
+    *-b* | *--resolve-bindings*
+
+        Resolve remote service bindings configured via *cds bind*.
+
 
 # SEE ALSO
 
     *cds login* for more information on authentication options.
-    
+
     *cds login* to save project settings and Basic-Auth data, simplifying multiple runs of this command.
 
 `});
@@ -64,6 +68,7 @@ async function upgrade([tenant], options = {}) {
     }
     const [username, password] = options.user?.split(':') ?? [];
     delete options.user;
+    delete options['resolve-bindings'];
 
     await upgrade.init('upgrade').run({
         ...options,

package/bin/watch.js

@@ -37,6 +37,11 @@ module.exports = Object.assign ( watch, {
     Open the given URL (suffix) in the browser after starting.
     If none is given, the default application URL will be opened.
 
+  *--profile* <profile,...>
+
+    Specify from which profile(s) the binding information is taken.
+    *Example:* cds w --profile hybrid,production
+
 # SEE ALSO
 
   *cds serve --help* for the different start options.

package/bin/watched.js

@@ -7,8 +7,8 @@
 
   // run through cli in order to use its error handling
   const args = JSON.parse(process.env._args)
+  const cds = require('../lib/cds'); cds.watched = true
   const bootstrap = require('./cds')(...args)
-  const {cds} = global; cds.watched = true
 
   cds.once('listening', ({server}) => {
     process.send ({ code:'listening', port: server.address().port })

package/lib/build/buildTaskFactory.js

@@ -230,7 +230,7 @@ class BuildTaskFactory {
 
     static _scanTaskOptionParams(optsParams) {
         // need to create new regex every call since a constant would keep the match state
-        const quoteRegex = /([\w-]+)=([\w/.]+|\[([\w/,.]+)\])/g
+        const quoteRegex = /([\w-]+)=([\w/.-]+|\[([\w/,.-]+)\])/g
 
         // captures a=1             => a:1
         //          a=[x,y,z]       => a:[x,y,z]

package/lib/build/constants.js

@@ -34,6 +34,7 @@ exports.CONTENT_NODE_MODULES = "contentNodeModules"             // copy node_mod
 exports.CONTENT_ENV = "contentEnv"                              // copy .env file if existing into deployment folder - DISABLED by default
 exports.CONTENT_DEFAULT_ENV_JSON = "contentDefaultEnvJson"      // copy default-env.json file if existing into deployment folder - DISABLED by default
 
+exports.CSV_FILE_TARGET = "csvFileTarget"                       // target folder when copying CSV files to the deployment target folder ./db/src/gen/*, default is 'data'
 exports.CSV_FILE_DETECTION = "csvFileDetection"                 // detect CSV files based on CDS model location - ENABLED by default
 exports.EXTENSION_POINT_VALIDATION = "extensionPointValidation" // validate extension point restrictions defined by the extended SaaS app - ENABLED by default
 exports.MTX_SIDECAR_DB_VALIDATION = "mtxSidecarDbValidation"    // validate consistency of app and sidecar db configuration - ENABLED by default

package/lib/build/provider/hana/2migration.js

@@ -13,8 +13,7 @@ module.exports = async (model, lastDevVersion, srcPath, options = {}) => {
     logger = options.logger
   }
 
-  const journalFileNames = _getJournalFileNames(model)
-  const { definitions, deletions, migrations, afterImage } = _toHdiMigration(model, lastDevVersion, journalFileNames, options)
+  const { definitions, deletions, migrations, afterImage, journalFileNames } = _toHdiMigration(model, lastDevVersion, options)
   const definitionResult = []
 
   for (const { name, suffix, sql } of definitions) {
@@ -33,8 +32,9 @@ module.exports = async (model, lastDevVersion, srcPath, options = {}) => {
   return { definitions: definitionResult, deletions, afterImage: _filterJournalArtifacts(afterImage) }
 }
 
-function _toHdiMigration(model, lastDevVersion, journalFileNames, options) {
+function _toHdiMigration(model, lastDevVersion, options) {
   const result = cdsc.to.hdi.migration(cds.minify(model), options, lastDevVersion);
+  const journalFileNames = _getJournalFileNames(result.afterImage);
   if (logger._debug) {
     logger.debug('cdsc.to.hdi.migration returned')
     for (const { name, suffix, sql } of result.definitions) {
@@ -50,7 +50,7 @@ ${migration ? migration.changeset.map(change => change.sql).join('\n') : 'Empty
       }
     }
   }
-  return result
+  return { ...result, journalFileNames };
 }
 
 /**

package/lib/build/provider/hana/index.js

@@ -6,7 +6,7 @@ const InternalBuildPlugin = require('../internalBuildPlugin')
 const { BuildError, relativePaths, BuildMessage } = require('../../util')
 
 const { OUTPUT_MODE_NONE, OUTPUT_MODE, CONTENT_PACKAGE_JSON, CONTENT_HDBTABLEDATA, CSV_FILE_DETECTION,
-    CONTENT_ENV, CONTENT_DEFAULT_ENV_JSON, CONTENT_NODE_MODULES, OUTPUT_MODE_RESULT, CONTINUE_UNRESOLVED_SCHEMA_CHANGES } = require('../../constants')
+    CONTENT_ENV, CONTENT_DEFAULT_ENV_JSON, CONTENT_NODE_MODULES, OUTPUT_MODE_RESULT, CONTINUE_UNRESOLVED_SCHEMA_CHANGES, CSV_FILE_TARGET } = require('../../constants')
 const { WARNING } = InternalBuildPlugin
 
 const DEFAULT_COMPILE_DEST_FOLDER = path.normalize("src/gen")
@@ -90,18 +90,28 @@ class HanaBuildPlugin extends InternalBuildPlugin {
      */
     async _copyResourcesExt(src, dest, model) {
         const resources = Object.keys(await cds.deploy.resources(model)).reverse() // reverse to get reuse resources first and app resources last
-        const csvPath = path.join(src, "data")
-        // determine subfolder name used by the application for backward compatibility
-        const csvFolder = resources.some(res => res && res.startsWith(csvPath)) ? "data" : "csv"
+        const dbSrc = path.join(src, 'src')
 
         // 1. copy csv files into 'src/gen/data' or 'src/gen/csv' subfolder
-        const dbSrc = path.join(src, 'src')
+        if (resources.length) {
+            // determine the CSV folder from the location of the CSVs defined in this build task db folder, use subfolder data by default
+            const dbCSVs = resources.filter(res => res.startsWith(src) && !res.startsWith(dbSrc))
+            let csvFolder = this.getBuildOption(CSV_FILE_TARGET)
+            if (csvFolder && !["data", "csv"].includes(csvFolder)) {
+                throw new BuildError("Invalid value for option 'csvFileTarget'. Valid values are 'data' or 'csv'.")
+            }
+            if (!csvFolder) {
+                const csvPath = path.join(src, "data")
+                // determine subfolder name used by the application for backward compatibility
+                csvFolder = !dbCSVs.length || dbCSVs.some(res => res && res.startsWith(csvPath)) ? "data" : "csv"
+            }
 
-        // await each copy as order is important - reuse resources first and app resources last
-        for (const res of resources) {
-            // do not duplicate resources that are already contained in db/src/**
-            if (res && /\.csv$/.test(res) && !res.startsWith(dbSrc)) {
-                await this.copy(res).to(path.join(this.task.options.compileDest, csvFolder, path.basename(res)))
+            // await each copy as order is important - reuse resources first and app resources last
+            for (const res of resources) {
+                // do not duplicate resources that are already contained in db/src/**
+                if (res && /\.csv$/.test(res) && !res.startsWith(dbSrc)) {
+                    await this.copy(res).to(path.join(this.task.options.compileDest, csvFolder, path.basename(res)))
+                }
             }
         }
 

package/lib/build/provider/java/index.js

@@ -24,12 +24,12 @@ class JavaBuildPlugin extends EdmxBuildPlugin {
 
         if (!this.hasBuildOption(CONTENT_EDMX, false)) {
             // generate edmx files containing all features
-            // localized edmx for backward compatibility
+            // REVISIT: localized edmx for backward compatibility
             if (!this.hasBuildOption(CONTENT_LOCALIZED_EDMX, false)) {
                 await this.compileToEdmx(odata, compileDest, { [FLAVOR_LOCALIZED_EDMX]: true })
             }
             // non-localized edmx
-            await this.compileToEdmx(odata, path.join(compileDest, 'odata', cds.env.odata.version), { [FLAVOR_LOCALIZED_EDMX]: false })
+            await this.compileToEdmx(odata, path.join(compileDest, 'odata', cds.env.odata.version), { [FLAVOR_LOCALIZED_EDMX]: this.hasBuildOption(FLAVOR_LOCALIZED_EDMX, true) })
         }
 
         if (!this.hasBuildOption(CONTENT_LANGUAGE_BUNDLES, false)) {

package/lib/build/provider/nodejs/index.js

@@ -1,8 +1,9 @@
 const fs = require('fs')
 const path = require('path')
 const cds = require('../../../cds')
+const cmd = require('../../../util/command');
 const EdmxBuildPlugin = require('../edmxBuildPlugin')
-const { BuildError } = require('../../util')
+const { BuildError, pathExists } = require('../../util')
 const { OUTPUT_MODE, OUTPUT_MODE_FILESYSTEM, ODATA_VERSION_V2, FOLDER_GEN, CONTENT_EDMX, CONTENT_PACKAGELOCK_JSON,
     CONTENT_NPMRC, CONTENT_CDSRC_JSON, CONTENT_ENV, CONTENT_DEFAULT_ENV_JSON, FLAVOR_LOCALIZED_EDMX } = require('../../constants')
 const { WARNING } = EdmxBuildPlugin
@@ -61,59 +62,6 @@ class NodejsBuildPlugin extends EdmxBuildPlugin {
         return this._result
     }
 
-    // REVISIT: fix for simple mono repos using npm workspaces
-    // Support for HANA native artifacts defined in sub-modules required
-    async _addWorkspaceDependencies(dest) {
-        try {
-            const workspaceRoot = path.resolve(cds.root, this.getBuildOption("workspaceRoot") ?? '.')
-            if (workspaceRoot !== cds.root && !fs.existsSync(workspaceRoot)) {
-                throw new BuildError(`The given workspaceRoot path ${workspaceRoot} does not exist`)
-            }
-            const packageJson = require(path.join(dest, 'package.json'))
-            const workspaces = workspaceRoot === cds.root ? packageJson.workspaces : require(path.join(workspaceRoot, 'package.json')).workspaces
-            if (workspaces?.length) {
-                this.logger._debug && this.logger.debug(`Existing npm workspace configuration '${JSON.stringify(workspaces, ' ')}'`)
-                const { tar } = require('../../../../lib').utils
-                const { dependencies } = packageJson
-                const folders = fs.readdirSync(workspaceRoot).filter(res => {
-                    try {
-                        return fs.lstatSync(path.join(workspaceRoot, res)).isDirectory()
-                    } catch (e) {
-                        return false
-                    }
-                })
-                this.logger._debug && this.logger.debug(`Existing workspace subfolders '${JSON.stringify(folders, ' ')}'`)
-
-                const workspaceNames = new Set()
-                let changed
-                for (const workspace of workspaces) {
-                    //REVISIT: workspaces must only use simple glob like 'foo*', nested folders are not supported
-                    const regex = new RegExp('^' + workspace.split('/').pop().replace(/\*/g, '.*') + '$')
-                    for (const folder of folders) {
-                        if (!workspaceNames.has(folder) && folder.match(regex)) {
-                            workspaceNames.add(folder)
-                            const plugin = require(require.resolve(path.join(workspaceRoot, folder, 'package.json')))
-                            const tarFile = `${plugin.name.replace(/\//g, '-').replace(/@/, '')}-${plugin.version}.tgz`
-                            if (dependencies[plugin.name] === '*') {
-                                const tarFilePath = path.join(dest, tarFile)
-                                dependencies[plugin.name] = `file:${tarFile}`
-                                await tar.czfd(tarFilePath, path.join(workspaceRoot, folder))
-                                this.pushFile(tarFilePath)
-                                changed = true
-                            }
-                        }
-                    }
-                }
-                if (changed) {
-                    return this.write(JSON.stringify(packageJson, null, '  ')).to(path.join(dest, 'package.json'))
-                }
-            }
-        } catch (e) {
-            this.pushMessage(`Failed to add npm workspace dependencies: ${e.message}`)
-            this.logger.debug(e)
-        }
-    }
-
     async clean() {
         // staging build content is deleted by BuildTaskEngine
         if (cds.env.build.target === '.') {
@@ -209,6 +157,78 @@ class NodejsBuildPlugin extends EdmxBuildPlugin {
             return srvAllowList.test(entry) && (!filter || filter.call(this, entry))
         })
     }
+
+    async _addWorkspaceDependencies(dest) {
+        let packageDescriptors
+        try {
+             packageDescriptors = await this._execNpmPack(dest)
+        } catch (e) {
+            this.logger.debug(e)
+            throw new BuildError(`Failed to package npm workspace dependencies\n${e.message}`)
+        }
+        if (packageDescriptors?.length) {
+            const packageJson = require(path.join(dest, 'package.json'))
+            const dependencies = packageJson.dependencies
+            const devDependencies = packageJson.devDependencies
+            let changed
+            const promises = []
+            for (const packageDescriptor of packageDescriptors) {
+                const {name, filename: fileName} = packageDescriptor
+                const filePath = path.join(dest, fileName)                        
+                if (dependencies?.[name] === '*' || devDependencies?.[name] === '*') {
+                    if (dependencies?.[name] === '*') {
+                        dependencies[name] = `file:${fileName}`
+                    } else {
+                        devDependencies[name] = `file:${fileName}`
+                    }
+                    this.pushFile(filePath)
+                    changed |= true
+                } else {
+                    if (await pathExists(filePath)) {
+                        promises.push(fs.promises.rm(filePath))
+                    }
+                }
+            }
+            if (changed) {
+                promises.push(this.write(JSON.stringify(packageJson, 0, 2)).to(path.join(dest, 'package.json')))
+            } else {
+                throw new BuildError('No matching workspaces found!')
+            }
+            await Promise.all(promises)
+        }
+    }
+    
+    async _execNpmPack(dest) {
+        function findWorkspaceRoot(dir) {
+            if(fs.existsSync(path.join(dir, 'package.json'))) {
+                if(require(path.join(dir, 'package.json')).workspaces) {
+                    return dir
+                }
+            }
+            if(fs.existsSync(path.join(dir, '.gitmodules')) || fs.existsSync(path.join(dir, 'node_modules'))) {
+                return // project root reached
+            }
+            if (path.dirname(dir) !== dir) {
+                return findWorkspaceRoot(path.dirname(dir))
+            }
+        }
+        const wsRoot = findWorkspaceRoot(cds.root)
+        if (!wsRoot) {
+            throw new BuildError('No workspace configuration found!')
+        }
+        const args = ['pack', '-ws', '--json', `--pack-destination=${dest}`]
+        if (this.task.options?.workspaces) {
+            let {workspaces} = this.task.options
+            workspaces = Array.isArray(workspaces) ? workspaces : [workspaces]
+            workspaces.forEach(w => args.push(`-w=${w}`))
+        }
+        this.logger._debug && this.logger.debug(`execute command: npm ${args}`)
+
+        const result = await cmd.spawnCommand( 'npm', args, { cwd: wsRoot || cds.root }, true, !!this.logger._debug)
+
+        this.logger._debug && this.logger.debug(result)
+        return JSON.parse(result)
+    }
 }
 
 module.exports = NodejsBuildPlugin

package/lib/client/auth_manager.js

@@ -66,7 +66,7 @@ async function retrieveTokenOrPasscodeUrl(params) {
         let response;
         for (const { method, url, data } of reqParams(params)) {
             try {
-                DEBUG?.(`Getting authentication token or passcode URL from ${method.toUpperCase()} ${(params.obfuscateQueryParams(url))}`);
+                DEBUG?.(`Getting authentication token or passcode URL from ${method.toUpperCase()} ${params.obfuscateQueryParams(url)}`);
                 response = await axios[method](url, data);
             } catch (error) {
                 if (error.status === 404) { // may represent 405 in case of wrong method

package/lib/client/pull.js

@@ -49,6 +49,14 @@ module.exports = class Pull extends BaseCommand {
             .catch(error => this.handleHttpError(error, params, { url: pullUrl }));
     }
 
+    static getPackageJsonIndentation(packageContents) {
+        return /^ +/m.exec(packageContents)?.[0].length ?? 2;
+    }
+
+    static formatPackageJson(newObj, oldContents) {
+        return JSON.stringify(newObj, undefined, this.getPackageJsonIndentation(oldContents));
+    }
+
     static amendPackageJson(projectFolder, env) {
         if ('extends' in env) {
             return;
@@ -61,7 +69,7 @@ module.exports = class Pull extends BaseCommand {
         } catch (error) {
             throw getMessage('package.json missing or unreadable', { error });
         }
-        fs.writeFileSync(packageJson, JSON.stringify({
+        fs.writeFileSync(packageJson, this.formatPackageJson({
             ...packageContents,
             cds: {
                 ...(packageContents.cds ?? {}),

package/lib/compile/asyncapi/components/schemas/annotations.js

@@ -1,3 +1,17 @@
+
+const odmAnnotationsMapping = Object.freeze({
+    '@ODM.entityName': 'x-sap-odm-entity-name',
+    '@ODM.oid': 'x-sap-odm-oid'
+});
+
+function addOdmExtensions(definition, object) {
+    for (const [annotation, asyncApiExtension] of Object.entries(odmAnnotationsMapping)) {
+        if (definition[annotation]) {
+            object[asyncApiExtension] = definition[annotation];
+        }
+    }
+}
+
 module.exports = function addAnnotations(definition, object) {
     if (typeof (definition['@title']) === 'string') {
         object.title = definition['@title'];
@@ -9,5 +23,11 @@ module.exports = function addAnnotations(definition, object) {
         object.description = definition.doc.replace(/\n/g, ' ');
     }
 
+    if (definition['@ODM.root']) {
+        object['x-sap-root-entity'] = definition['@ODM.root']
+    }
+
+    addOdmExtensions(definition, object);
+
     return object;
 }