@sap/cds 5.6.1 → 5.7.1

package/CHANGELOG.md

@@ -4,6 +4,138 @@
 - The format is based on [Keep a Changelog](http://keepachangelog.com/).
 - This project adheres to [Semantic Versioning](http://semver.org/).
 
+## Version 5.7.1 - 2021-12-06
+
+### Fixed
+
+- Draft (OData flavors `w4`, `x4` and `v4` with structs): Flags `HasActiveEntity`, `HasDraftEntity`, and `IsActiveEntity` are now included in the HTTP response for GET requests.
+- Instance-based restriction on entities using localized fields in draft
+- Results of actions/functions do not ignore nested data if query options are present
+
+## Version 5.7.0 - 2021-12-03
+
+### Added
+
+- Deferred emitting via persistent outbox, enabled through service `outbox` of kind `persistent-outbox`
+- Support for compiler-generated referential constraints (beta)
+  + Activate via `cds.env.features.assert_integrity: '<preset>'`
+  + Available presets:
+    + `off`: no database constraints and no runtime checks
+    + `app`: runtime checks by default
+    + `db`: database constraints by default
+  + "by default": if not excluded, the runtime check or database constraint applies
+  + "opt-in": if included, the runtime check or database constraint applies
+  + Behavior can be overridden via `@assert.integrity: <true/false/'RT'/'DB'>` on property, entity, or service level (lowest applies)
+- Allow `--with-mocks` in production via `cds.env.features.with_mocks = true`
+- Set media type from content type header while inbound streaming
+- Support for navigations with `$count` in `$filter`, for example `GET Entity?$filter=toMany/$count gt 0`
+- Draft: Generate UUIDs for request payloads to which extra data items are added (without the UUID keys) in a custom application handler.
+- Generate GraphQL schema via `cds compile -2 gql` (alpha)
+- Log requests to remote services if respective log level is set to `debug`
+- Beta OData URL to CQN parser (`cds.env.features.odata_new_parser`):
+  + support for `$skiptoken` query option
+  + limited support for `$apply` query option
+    + Supported are following transformations and their combinations: `aggregate`, `groupby`, `topcount`, `bottomcount`, `filter`, `search`
+    + Not supported:
+      + Transformations `topsum`, `bottomsum`, `toppercent`, `bottompercent`, `expand`, `concat`, `compute`, `identity`
+      + `rollup` and `$all` in `groupby` transformation
+      + Filter function `isdefined`
+      + Custom aggregation methods, arithmetic operators (`add`, `sub`, `mul` and so on) and keyword `from` in `aggregate` transformation
+      + OData vocabulary for Data Aggregation, `@Aggregation.default` annotation
+    + Out of scope:
+      + Draft handling
+- Out-of-the-box audit logging for draft enabled entities
+- Support for `@sap/instance-manager`'s hybrid mode
+  + Enable via `cds.env.features.hybrid_instance_manager`
+  + Respective version of `@sap/instance-manager` required
+- `cds.minify()` (alpha) as static method
+- Annotation `@open` for actions in new Rest Adapter
+- Audit logging (`cds.env.features.audit_personal_data`) supports annotation `@PersonalData.EntitySemantics: 'Other'` and allows an arbitrary composition of entities with respect to `@PersonalData.EntitySemantics` annotations
+
+### Changed
+
+- `if-match` and `if-none-match` headers are  ignored for entities without etags
+- Improve response time of `SELECT` queries that check referential integrity by adding an upper bound `LIMIT 1`
+- Leaner implementation for `sap-statistics`
+- Leading and trailing whitespaces are allowed for `$search` query parameter
+- Insert / Update of Composition of one with empty object is not allowed for non UUID keys
+- Search behavior of whitespaces changed as follow:
+  + Searches for plain whitespace, for example, `"$search= "` matches the complete data set.
+  + Searches for whitespace surrounded by double-quotes, for example, `$search=" "` matches all entries containing whitespaces.
+- In single tenant mode, the default SQLite database is used, regardless of `context.tenant`
+- `cds build` for Node.js projects now copies `package-lock.json` files into the staging folder (usually `gen/srv`). This allows the execution of `npm ci` there.
+- Relaxed UUIDs in beta URL to CQN parser (`cds.env.features.odata_new_parser`)
+- Authentication strategies `dummy` and `mock` no longer require `passport`
+- In production, debug logs of `cds.DatabaseService` and `cds.RemoteService` have sanitized values
+  + Deactivate via `cds.env.log.sanitize_values = false`
+
+### Fixed
+
+- Path resolution for references in sub selects
+- Where exists without infix filter, e.g., `@restrict.where: 'exists author'`
+- `@restrict.where: 'exists [...]'` in draft union scenario
+- Select query with path exists predicates e.g., `WHERE EXISTS books[year = 2000].pages[wordcount > 1000]`
+- Proper registration of audit log event handlers
+- Draft: Generate foreign keys for request payloads to which extra data items are added in a custom application handler.
+- `cds build` correctly merges `hdbmigrationtable` files that have multiple new migration versions defined.
+- `cds.test` converts response data of failed requests to JSON to prevent lost error details
+- Instance based restriction for draft enabled entities
+- Delete requests for localized with compositions
+- Ignore input for static and calculated fields during draft activate
+- Clear extension map entry on error during CSN fetching
+- Do not ignore errors during diff calculation
+- Requests to mocked remote service when using custom service name with `.service` property
+- Rollback on already backrolled or committed transactions are ignored
+- Rollback handling in spawned background job
+- `cds.spawn()` throws error if passed options is an instance of `cds.EventContext`
+- Delete `timestamp` from options passed to `cds.spawn()` (transactions create their own timestamp)
+- Type error during programmatic action/function call if no params defined
+- Fully qualified bound actions/ functions in beta URL to CQN parser (`cds.env.features.odata_new_parser`)
+- Draft handling: `GET` requests with navigation via `SiblingEntity` and expand to-one
+- No audit log if sensitive data not selected
+- Kibana formatter: do not log authorization header value
+- Audit logging (`cds.env.features.audit_personal_data`) no longer crashes the application
+  + when using the same entity as a composition child in different parent entities
+  + when accessing a not existing entity
+
+### Removed
+
+- Deprecated feature flags `cds.env.runtime.skipWithParameters` and `cds.env.features.skip_with_parameters`. Use `cds.env.features.with_parameters` instead.
+
+## Version 5.6.4 - 2021-11-23
+
+### Fixed
+
+- Preserve log level in Kibana formatter
+- RFC 3986 compliant segment recognition in beta URL to CQN parser (`cds.env.features.odata_new_parser`)
+- Support for `$skiptoken` OData query option when using beta URL to CQN parser (`cds.env.features.odata_new_parser`)
+- OData requests with `$skip` and without `$top` query option to services with defined default limit (`cds.query.limit.default`)
+- Creating entities with binary keys. Currently the binary keys may be provided only as Node.js `Buffer` instances using a custom handler.
+- Requests with payload containing nested arrayed elements no longer crash the application
+
+## Version 5.6.3 - 2021-11-15
+
+### Fixed
+
+- `cds run` does not fail if `cds.requires.multitenancy` is explicitly set to `false`
+- Calculation of `DraftIsCreatedByMe` or `DraftIsProcessedByMe` when expanding or navigating to `DraftAdministrativeData`
+- Nested `any` in `$filter` query option
+- Crash on draft activate after draft edit for not existing composition of one, if no explicit DB service is defined
+- Typescript definition of `srv.delete` no longer leads to a duplication error
+
+## Version 5.6.2 - 2021-11-08
+
+### Fixed
+
+- Handle arrayed elements using templating mechanism
+- OData requests to `$count` endpoint of ETag enabled entity
+- `cds.test` does no longer crash if executed in `cds repl` on a remote service call
+- Crash on draft activate after draft edit for not existing composition of one
+- Ensure request correlation (with default server)
+- `<entity>.texts` points to real text entity
+- Draft union with expand to to-one and to-many
+- No columns in draft lock statement (i.e., use `SELECT 1`)
+
 ## Version 5.6.1 - 2021-11-02
 
 ### Fixed
@@ -79,6 +211,9 @@
   + Deactivate during two month grace period via compat feature flag `cds.env.features.keys_into_where = true`
 - Removed duplicate integrity checks
 - Optimized search: Optimize queries for non-localized elements
+- OData to CQN parsing changed to enable remote service consumption. As a side effect, application code in `srv.on('READ', handler)` custom handlers relaying on CQN might need to be adapted. The following has changed:
+  - Previously, columns in `req.query.SELECT.columns` were always defined. Now, in case there is no `$select` and `$expand` query options in the OData query, `req.query.SELECT.columns` is `undefined`.
+  - Previously, if the `$expand` query option was specified in the OData query, all elements of the expanded navigation property were listed explicitly in the CQN query. Now, an `*` (asterisk) is listed instead.
 - Non-specified columns are resolved at database layer
 - `cds deploy` no longer enforces the presence of SAP CommonCryptoLib (checked with env variable `SECUDIR`) on Windows since it uses now the built-in security libraries
 - Target keys are not included into a body when sending `PATCH` requests to external services
@@ -551,6 +686,7 @@
 - `SELECT.where(...)` generates CQN with list of values for `in` operator
 - Always use flag `u` during input validation via `@assert.format`
 - Intermediate CQN format for lambda expressions with preceding navigation path
+- Better error messages for draft enabled entities
 
 ### Fixed
 

package/_i18n/i18n_fr.properties

@@ -26,16 +26,16 @@
 #----------------------------------------------------------------------------------------------------------------------
 
 #XTIT: Created By (Answer to: "Which user has created a certain entity?")
-CreatedBy=Cr\u00E9\u00E9(e) par
+CreatedBy=Auteur de la cr\u00E9ation
 
 #XTIT: Created On (Answer to: "When has a certain entity been created?")
-CreatedAt=Cr\u00E9\u00E9(e) le
+CreatedAt=Date de cr\u00E9ation
 
 #XTIT: Changed By (Answer to: "Which user has changed a certain entity?")
-ChangedBy=Modifi\u00E9(e) par
+ChangedBy=Auteur de la modification
 
 #XTIT: Changed On (Answer to: "When has a certain entity been changed?")
-ChangedAt=Modifi\u00E9(e) le
+ChangedAt=Date de modification
 
 #XTIT: Currency
 Currency=Devise

package/apis/cds.d.ts

@@ -1,14 +1,11 @@
-declare type cds_facade =
-import ('./core')
-& import ('./models')
-& import ('./connect')
-& import ('./serve')
-& import ('./ql').cds_ql
-& import ('./services').cds_facade
-& import ('./services').QueryAPI
-
 declare global {
-	const cds : cds_facade
+	const cds : import ('./core')
+		& import ('./models')
+		& import ('./connect')
+		& import ('./serve')
+		& import ('./ql').cds_ql
+		& import ('./services').cds
+		& import ('./services').QueryAPI
 	class SELECT<T> extends cds.ql.SELECT<T>{}
 	class INSERT<T> extends cds.ql.INSERT<T>{}
 	class UPDATE<T> extends cds.ql.UPDATE<T>{}

package/apis/connect.d.ts

@@ -1,8 +1,8 @@
 import { Service } from "./services"
-import * as cds from './cds'
+import * as cds_facade from './cds'
 
-export = cds_connect
-declare class cds_connect {
+export = cds
+declare class cds {
 
 	connect : {
 		/**

package/apis/core.d.ts

@@ -1,11 +1,9 @@
 // import types from '../lib/core'
 import { ReflectedModel, LinkedModel, LinkedDefinition } from './reflect'
 import { CSN as csn, Definition } from "./csn"
-import { Query as cqn } from "./cqn"
-import { EventEmitter } from "events";
 
-export = cds_core
-declare class cds_core extends EventEmitter {
+export = cds
+declare class cds {
 
 	/**
 	 * Turns the given plain CSN model into a linked model

package/apis/models.d.ts

@@ -1,4 +1,3 @@
-import edm from "@sap/cds-compiler/lib/edm/edm"
 import { Query, expr, _xpr } from "./cqn"
 import { CSN } from "./csn"
 
@@ -30,8 +29,8 @@ type EDMX = XML
 type filename = string
 
 
-export = cds_models
-declare class cds_models {
+export = cds
+declare class cds {
 
     /**
      * Provides a set of methods to parse a given model, query or expression.

package/apis/ql.d.ts

@@ -1,6 +1,5 @@
 import {Definition} from "./csn"
 import * as CQN from "./cqn"
-import { STATUS_CODES } from "http"
 
 export type Query = CQN.Query
 

package/apis/services.d.ts

@@ -64,8 +64,8 @@ export class QueryAPI {
 	 * Starts or joins a transaction
 	 * @see [docs](https://cap.cloud.sap/docs/node.js/services#srv-tx)
 	 */
-	tx (context : object) : Transaction
-	transaction (context : object) : Transaction
+	tx (context? : object) : Transaction
+	transaction (context? : object) : Transaction
 }
 
 /**
@@ -165,6 +165,10 @@ export class Service extends QueryAPI {
 	 * @see [capire docs](https://cap.cloud.sap/docs/node.js/services#srv-send)
 	 */
 	delete (entityOrPath: Target, data?: object) : Promise<this>
+	/**
+	 * @see [docs](https://cap.cloud.sap/docs/node.js/services#srv-run)
+	 */
+	delete <T>(entity : Definition | string, key?: any) : DELETE<T>
 
 	 // The central method to dispatch events
 	dispatch (msg: EventMessage): Promise<any>
@@ -200,7 +204,7 @@ export class DatabaseService extends Service {
 
 export interface ResultSet extends Array<{}> {}
 
-export class cds_facade {
+export class cds {
 
 	/**
 	 * @see [capire docs](https://cap.cloud.sap/docs/node.js/services)

package/bin/build/buildTaskFactory.js

@@ -31,7 +31,7 @@ class BuildTaskFactory {
             buildOptions.target = path.resolve(buildOptions.root, this.env.build.target)
 
             // log build tasks
-            this.logger.log(`[cds] - the following build tasks will be executed`)
+            this.logger.log(`the following build tasks will be executed`)
             let tasksOutput = "   {\n     \"build\": {\n       \"target\": \"" + this.env.build.target + "\",\n       \"tasks\": ["
             for (let i = 0; i < tasks.length; i++) {
                 tasksOutput += "\n         " + BuildTaskFactory._stringifyTask(tasks[i]) + (i + 1 < tasks.length ? "," : "")
@@ -63,7 +63,7 @@ class BuildTaskFactory {
 
     async _createTasks(providerFactory) {
         const buildOptions = providerFactory.buildOptions
-        this.logger.log(`[cds] - determining build tasks for project [${buildOptions.root}].`)
+        this.logger.log(`determining build tasks for project [${buildOptions.root}].`)
 
         let tasks = this._getExistingTasks()
         if (tasks.length === 0) {
@@ -118,7 +118,7 @@ class BuildTaskFactory {
 
         // Java projects use "." as the default build target folder
         if (this._hasJavaNature(buildOptions.root, srv) && this._adaptBuildTargetSettingForJava(buildOptions.root)) {
-            this.logger.debug("[cds] - using inplace build for java project instead of default staging build")
+            this.logger.debug("using inplace build for java project instead of default staging build")
         }
         buildOptions.target = path.resolve(buildOptions.root, this.env.build.target)
     }
@@ -130,7 +130,7 @@ class BuildTaskFactory {
         this.cds.resolve.cache = {}
 
         const modelPaths = this.cds.resolve("*", false)
-        return BuildTaskFactory._pushModelPaths(projectPath, [], modelPaths)
+        return this._pushModelPaths(projectPath, [], modelPaths)
     }
 
     /**
@@ -241,17 +241,23 @@ class BuildTaskFactory {
         return taskOptions
     }
 
-    static _pushModelPaths(projectPath, model, ...modelPaths) {
+    _pushModelPaths(projectPath, model, ...modelPaths) {
         model = new Set(model)
         // may contain nested arrays
         modelPaths = BuildTaskFactory._flatten(modelPaths)
-
+        const { roots } = this.cds.env
         modelPaths.forEach(m => {
             if (m && !model.has(m) && !model.has(m + "/")) {
-                const dir = path.resolve(projectPath, m)
-                if (fs.existsSync(dir)) {
-                    model.add(BuildTaskFactory._normalizePath(m))
-                } else if (fs.existsSync(dir + FILE_EXT_CDS)) { //might be cds file name, compability to old build configs
+                // filter root model paths that do not exist
+                // other entries are added as is, e.g. reuse model entries
+                if (roots.includes(m)) {
+                    const dir = path.resolve(projectPath, m)
+                    if (fs.existsSync(dir)) {
+                        model.add(BuildTaskFactory._normalizePath(m))
+                    } else if (fs.existsSync(dir + FILE_EXT_CDS)) { //might be cds file name, compability to old build configs
+                        model.add(m)
+                    }
+                } else {
                     model.add(m)
                 }
             }

package/bin/build/buildTaskProviderFactory.js

@@ -56,7 +56,7 @@ class BuildTaskProviderFactory {
     }
 
     async lookupTasks() {
-        this.logger.debug("[cds] - Determining CDS build tasks from CDS configuration - applying defaults")
+        this.logger.debug("determining CDS build tasks from CDS configuration - applying defaults")
         let tasks = []
         await Promise.all(this.providers.map(async (provider) => {
             let pluginTasks = await this._lookupTasks(provider)
@@ -81,14 +81,14 @@ class BuildTaskProviderFactory {
     createHandler(task) {
         const BuildTaskHandlerClass = this.loadHandler(task)
         const resolvedTask = this.resolveTask(task)
-        this.logger._debug && this.logger.debug(`[cds] - loaded build task handler [${resolvedTask.use}]`)
+        this.logger._debug && this.logger.debug(`loaded build task handler [${resolvedTask.use}]`)
 
         const handler = new BuildTaskHandlerClass()
         handler._task = resolvedTask
         handler._cds = this.cds
         handler._logger = this.logger
         handler._buildOptions = this.buildOptions
-        this.logger._debug && this.logger.debug(`[cds] - created BuildTaskHandler [${resolvedTask.use}]`)
+        this.logger._debug && this.logger.debug(`created BuildTaskHandler [${resolvedTask.use}]`)
         return handler
     }
 

package/bin/build/constants.js

@@ -31,7 +31,8 @@ exports.CONTENT_DEFAULT_CSN = "contentDefaultCsn"           // create default CS
 exports.CONTENT_EDMX = "contentEdmx"                        // create EDMX for required languages
 exports.CONTENT_MANIFEST = "contentManifest"                // create manifest.yml and .cfignore files
 exports.CONTENT_PACKAGE_JSON = "contentPackageJson"         // create package.json file if not existing, or modify existing package.json
-exports.CONTENT_HDBTABLEDATA = "contentHdbtabledata"        // create .hdbtabledata files for .csv files if not existing 
+exports.CONTENT_PACKAGELOCK_JSON = "contentPackageLockJson" // copy package-lock.json file if existing
+exports.CONTENT_HDBTABLEDATA = "contentHdbtabledata"        // create .hdbtabledata files for .csv files if not existing
 
 exports.FOLDER_GEN = "gen"
 exports.FILE_EXT_CDS = ".cds"

package/bin/build/provider/buildTaskProviderInternal.js

@@ -62,7 +62,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
     }
 
     async _createTasks(buildOptions) { // NOSONAR
-        this.logger.debug("[cds] - Determining CDS build tasks from CDS configuration - applying defaults")
+        this.logger.debug("determining CDS build tasks from CDS configuration - applying defaults")
         const { root: projectPath } = buildOptions
         let tasks = []
         let db = typeof this.env.folders.db === "string" ? [BuildTaskProviderInternal._normalizePath(this.env.folders.db)] : this.env.folders.db
@@ -87,7 +87,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
                 tasks.push(dbTask)
             }
         } else {
-            this.logger.log(`[cds] - project doesn't have a database module [${this.env.folders.db}]`)
+            this.logger.log(`project doesn't have a database module [${this.env.folders.db}]`)
         }
 
         if (srv) {
@@ -117,7 +117,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
                 }
             }
         } else {
-            this.logger.log(`[cds] - project doesn't have a service module '${this.env.folders.srv}'`)
+            this.logger.log(`project doesn't have a service module '${this.env.folders.srv}'`)
         }
 
         // create mtx build task for node applications
@@ -135,11 +135,11 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
     }
 
     _createDbTask(projectPath, src, taskOptions, buildOptions) {
-        this.logger.debug("[cds] - Determining database kind.")
+        this.logger.debug("determining database kind.")
         let task = null
 
         if (this._useHana(projectPath, buildOptions)) {
-            this.logger.debug("[cds] - Found HANA database.")
+            this.logger.debug("found HANA database.")
             // legacy build supports dest property
             const compileDest = this.env.get("data.dest")
             if (compileDest) {
@@ -154,7 +154,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
                 options: taskOptions
             }
         } else {
-            this.logger.debug("[cds] - Found sqlite database - skipping HANA build task")
+            this.logger.debug("found sqlite database - skipping HANA build task")
         }
         return task
     }
@@ -179,28 +179,28 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
     }
 
     _createMtxTask(srv) {
-        this.logger.debug("[cds] - Determining single or multi-tenant strategy.")
+        this.logger.debug("determining single or multi-tenant strategy.")
         let task = null
 
         if (this.env.get("requires.multitenancy") && (this.env.get("requires.db.kind") === "hana" || this.env.get("requires.db.dialect") === "hana")) {
-            this.logger.debug("[cds] - Found multi-tenant app.")
+            this.logger.debug("found multi-tenant app.")
             task = {
                 src: ".",
                 for: BUILD_TASK_MTX,
                 dest: srv
             }
         } else {
-            this.logger.debug("[cds] - Found single-tenant app - skipping mtx build task")
+            this.logger.debug("found single-tenant app - skipping mtx build task")
         }
         return task
     }
 
     _createSrvTask(projectPath, src, taskOptions) {
-        this.logger.debug("[cds] - Determining implementation technology")
+        this.logger.debug("fetermining implementation technology")
         let task = this._createJavaTask(projectPath, src, taskOptions)
 
         if (!task) {
-            this.logger.debug("[cds] - Found implementation technology node")
+            this.logger.debug("found implementation technology node")
             task = {
                 src: src,
                 for: BUILD_TASK_NODE,
@@ -212,7 +212,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
 
     _createJavaTask(projectPath, src, taskOptions) {
         if (this._hasJavaNature(projectPath, src)) {
-            this.logger.debug("[cds] - Found implementation technology java")
+            this.logger.debug("found implementation technology java")
             // legacy build supports dest property
             const compileDest = this.env.get("service.dest")
             if (compileDest) {
@@ -237,7 +237,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
      */
     _createFioriTasks(projectPath, fioriSrvOptions) {
         let tasks = []
-        this.logger.debug("[cds] - Determining fiori modules - matching modules */webapp/manifest.json")
+        this.logger.debug("determining fiori modules - matching modules */webapp/manifest.json")
         // fiori-app build-tasks
         let appDirs = this.env.ui && this.env.ui.apps ? this.env.ui.apps : undefined
         if (!appDirs) {
@@ -263,7 +263,7 @@ class BuildTaskProviderInternal extends BuildTaskProvider {
                 return appDir
             })
         }
-        this.logger._debug && this.logger.debug(`[cds] - Found fiori app paths [${appDirs}]`)
+        this.logger._debug && this.logger.debug(`found fiori app paths [${appDirs}]`)
         appDirs.forEach(appDir => {
             appDir = BuildTaskProviderInternal._normalizePath(appDir)
             let modelPaths = this._resolveModel(path.resolve(projectPath, appDir))

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

@@ -4,8 +4,7 @@ const { getProperty, BuildError } = require('../../util')
 const { getArtifactCdsPersistenceName } = require('@sap/cds-compiler')
 const { LOG_MODULE_NAMES } = require('../../constants')
 const cdscVersion = `-- generated by cds-compiler version ${require('@sap/cds-compiler/package.json').version}`
-const cds = require('../../cds'), minified = csn => cds.linked(csn).minified()
-const { compiler: cdsc } = cds
+const cds = require('../../cds'), { compiler: cdsc } = cds
 let logger = cds.log(LOG_MODULE_NAMES)
 
 module.exports = async (model, lastDevVersion, srcPath, options = {}) => {
@@ -35,7 +34,7 @@ module.exports = async (model, lastDevVersion, srcPath, options = {}) => {
 
 function _toHdiMigration(model, lastDevVersion, journalFileNames, options) {
   options.sqlChangeMode = getProperty(options, 'hana.journal.change-mode')
-  const result = cdsc.to.hdi.migration(minified(model), options, lastDevVersion);
+  const result = cdsc.to.hdi.migration(cds.minify(model), options, lastDevVersion);
   if (logger._debug) {
     logger.debug('cdsc.to.hdi.migration returned')
     for (const { name, suffix, sql } of result.definitions) {

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

@@ -22,6 +22,7 @@ const FILE_NAME_HDINAMESPACE = ".hdinamespace"
 const FILE_NAME_PACKAGE_JSON = "package.json"
 const FILE_NAME_MANIFEST_YML = "manifest.yml"
 const PATH_LAST_DEV_CSN = "last-dev/csn.json"
+const FILE_NAME_UNDEPLOY_JSON = "undeploy.json"
 
 // add well-known types supported by HANA Cloud Edition - see also https://github.wdf.sap.corp/cap/issues/issues/8056
 const REQUIRED_PLUGIN_TYPES = [FILE_EXT_CSV, FILE_EXT_HDBTABLEDATA, FILE_EXT_HDBTABLE, ".hdbview", ".hdbindex", ".hdbconstraint"]
@@ -64,6 +65,8 @@ class HanaModuleBuilder extends BuildTaskHandlerInternal {
             await this._copyNativeContent(src, dest)
             await this._writeHdiConfig(plugins)
             await this._writeHdiNamespace()
+            // TODO disabled as this contradicts the MTX domain concept which allows partial app deployments 
+            //await this._writeUndeployJson()
 
             if (this.hasBuildOption(CONTENT_HDBTABLEDATA, true)) {
                 await this._compileToHdbtabledata(model, dest)
@@ -344,6 +347,37 @@ class HanaModuleBuilder extends BuildTaskHandlerInternal {
         return await this.write(content).to(hdiNamespace)
     }
 
+    /**
+     * Create undeploy.json file in <dest> folder of db module.
+     */
+    async _writeUndeployJson() {
+        if (this.isStagingBuild()) {
+            // see issue #64 - add .hdinamespace file to prevent HDI from adding gen/ folder to the namespace.
+            const undeployJsonDest = path.join(this.task.dest, FILE_NAME_UNDEPLOY_JSON)
+            const undeployJsonSrc = path.join(this.task.src, FILE_NAME_UNDEPLOY_JSON)
+            const templateEntries = await this._readTemplateAsJson(FILE_NAME_UNDEPLOY_JSON)
+            let newEntries = []
+            if (await fs.pathExists(undeployJsonSrc)) {
+                newEntries = await fs.readJSON(undeployJsonSrc, 'utf-8')
+                newEntries = Array.isArray(newEntries) ? newEntries : []
+                templateEntries.forEach(entry => {
+                    if (!newEntries.includes(entry)) {
+                        newEntries.push(entry)
+                    }
+                })
+            } else {
+                newEntries = templateEntries
+            }
+            // formatted output
+            let content = '[\n'
+            for (let i = 0; i < newEntries.length; i++) {
+                content += `  "${newEntries[i]}"${i + 1 < newEntries.length ? ',' : ''}\n`
+            }
+            content += ']'
+            await this.write(content).to(undeployJsonDest)
+        }
+    }
+
     /**
      * Create .cfignore file only for staging build.
      * This is to have a fast-turnaround at development