@sap/cds-compiler 2.12.0 → 2.13.6

package/CHANGELOG.md

@@ -7,6 +7,101 @@
 Note: `beta` fixes, changes and features are usually not listed in this ChangeLog but [here](doc/CHANGELOG_BETA.md).
 The compiler behavior concerning `beta` features can change at any time without notice.
 
+## Version 2.13.6 - 2022-25-03
+
+### Fixed
+
+- to.hdbcds/hdi/sql: Correctly handle `localized` in conjunction with `@cds.persistence.exists`
+
+## Version 2.13.4 - 2022-22-03
+
+No changes compared to Version 2.13.0; fixes latest NPM tag
+
+## Version 2.13.2 - 2022-22-03
+
+No changes compared to Version 2.13.0; fixes latest NPM tag
+
+## Version 2.13.0 - 2022-22-03
+
+### Added
+
+- CDL syntax:
+  + Allow to `extend E:elem` and `annotate E:elem` instead of having to write deeply nested statements.
+  + Enable `default` values as part of scalar type definitions.
+  + The following `extend` syntax variants are now possible:
+    ```cds
+    extend … with elements { … }
+    extend … with definitions { … }
+    extend … with columns { … }
+    extend … with enum { … }
+    extend … with actions { … }
+    ```
+    This syntax expresses _how_ an artifact is extended instead of _what_ is extended.
+  + Using `ORDER BY` in generic functions such as SAP HANA's `first_value` is now possible.
+- Make API function `compileSources` accept CSN objects as file content
+- to.edm(x): Annotate view parameters with `@sap.parameter: mandatory` (V2) and `@Common.FieldControl: #Mandatory` (V4).
+- to.sql/hdi/hdbcds: Introduce the annotations `@sql.prepend` and `@sql.append` that allow inserting user-written SQL
+  snippets into the compiler generated content. Changes in annotations `@sql.prepend` and `@sql.append` are now reflected
+  in the output of `to.hdi.migration`. This enables CDS Build to produce `.hdbmigrationtable` files translating such model
+  changes into schema changes.
+- API: Lists of keywords for various backends are available as `to.<backend>[.<config>].keywords`, e.g. `to.sql.sqlite.keywords`.
+- for.odata/to.edm(x): The draft composition hull is now also taking into account compositions in subelements.
+
+### Changed
+
+- In query entities inside services, only auto-redirect associations and compositions
+  in the main query of the entity.
+- An element now inherits the property `notNull` from its query source (as
+  before) or its type (like it does for most other properties);
+  `notNull` is then not further propagated to its sub elements anymore.
+- A structure element inherits the property `virtual` from its query source (as
+  before), but does not further propagate `virtual` to its sub elements
+  (semantically of course, but the CSN is not cluttered with it);
+  there is a new warning if a previously `virtual` query entity
+  element is now considered to be non-virtual.
+- Do not propagate annotation value `null`.
+  The value `null` of an annotation (and `doc`) is used to stop the inheritance
+  of an annotation value.  This means than other than that, a value `null` should
+  not be handled differently to not having set that annotation.
+- In the effective CSN, the structure type is only expanded if something has changed
+  for associations: the `target` (`keys` does not change if the `target` does not change)
+  unmanaged associations as sub elements are not supported anyway.
+- In the effective CSN, “simple” type properties like `length`, `precision`,
+  `scale` and `srid` are propagated even for a propagation via type.
+- Update OData Vocabularies: 'Capabilities', 'Common', 'Core', 'UI'.
+- to.sql:
+  + For SQL dialect `hana` referential constraints are now appended
+    as `ALTER TABLE ADD CONSTRAINT` clause to the end of `schema.sql`.
+    With option `constraintsInCreateTable` constraints are rendered into the
+    `CREATE TABLE` statement.
+  + Referential constraint names are now prefixed with `c__`.
+
+### Fixed
+
+- Properly resolve references inside anonymous aspects:
+  + references starting with `$self.` made the compiler dump.
+  + a simple `$self` did  not always work as expected (it represents the entity created via the anonymous aspect).
+  + other references inside deeply nested anonymous aspects induced a compilation error.
+- compiler: `()` inside `ORDER BY` clause was not correctly set.
+- parse.cdl: References in `ORDER BY` and filters are now correctly resolved.
+- Issue error when trying to introduce managed compositions of aspects in `mixin`s
+- Issue error in all cases for type references to unmanaged associations.
+- Avoid dump when extending an illegal definition with a name starting with `cds.`.
+- to.sql/to.cdl/to.hdbcds/to.hdi: Render `cast()` inside `ORDER BY`, `GROUP BY` and `HAVING` properly.
+- to.sql/hdi/hdbcds:
+  + `$self` was incorrectly treated as a structured path step.
+  + Correctly handle table alias in on-condition of mixin in `exists` expansion.
+  + Correctly handle table `$self` references to aliased fields in on-condition of mixin association
+    during `exists` expansion.
+- to.edm: Don't escape `&` as `&amp;`.
+- to.edmx: Escaping compliant to XML specification:
+  + `&` and `<` are always escaped.
+  + `>` is not escaped, unless it appears in text values as `]]>`.
+  + `"` is escaped in attribute values only.
+  + Control characters are always escaped.
+- Ellipsis (`...`) in annotations in different layers but without base annotation now produces an error.
+  The old but incorrect behavior can be re-enabled with option `anno-unexpected-ellipsis-layers`.
+
 ## Version 2.12.0 - 2022-01-25
 
 ### Added
@@ -18,16 +113,16 @@ The compiler behavior concerning `beta` features can change at any time without
   for syntax highlighting, similar to markdown.  In difference to the former, text blocks require the
   opening and closing backticks to be on separate lines.
   Example:
+  ````
+  @annotation: `Multi
+   line\u{0020}strings`
 
-      @annotation: `Multi
-       line\u{0020}strings`
-  
-      @textblock: ```xml
-                  <summary>
-                    <detail>The root tag has no indentation in this example</detail>
-                  </summary>
-                  ```
-      ...
+  @textblock: ```xml
+              <summary>
+                <detail>The root tag has no indentation in this example</detail>
+              </summary>
+              ```
+  ````
 
 - Enhance the ellipsis operator `...` for array annotations by an `up to ‹val›`:
   only values in the array of the base annotation up to (including) the first match
@@ -38,12 +133,12 @@ The compiler behavior concerning `beta` features can change at any time without
   values in `‹val›` are equal to the corresponding property value in the array value;
   it is not necessary to specify all properties of the array value items in `‹val›`.
   Example
-
-      @Anno: [{name: one, val: 1}, {name: two, val: 2}, {name: four, val: 4}]
-      type T: Integer;
-      @Anno: [{name: zero, val: 0}, ... up to {name: two}, {name: three, val: 3}, ...]
-      annotate T;
-
+  ```
+  @Anno: [{name: one, val: 1}, {name: two, val: 2}, {name: four, val: 4}]
+  type T: Integer;
+  @Anno: [{name: zero, val: 0}, ... up to {name: two}, {name: three, val: 3}, ...]
+  annotate T;
+  ```
 - for.odata: Support `@cds.on {update|insert}` as replacement for deprecated `@odata.on { update|insert }` to
   set `@Core.Computed`.
 

package/bin/cdsc.js

@@ -163,14 +163,6 @@ try {
   )
     cmdLine.options.assertIntegrity = cmdLine.options.assertIntegrity === 'true';
 
-  // remap string values for `constraintsAsAlter` option to boolean
-  if (cmdLine.options.constraintsAsAlter &&
-    cmdLine.options.constraintsAsAlter === 'true' ||
-    cmdLine.options.constraintsAsAlter === 'false'
-  )
-    cmdLine.options.constraintsAsAlter = cmdLine.options.constraintsAsAlter === 'true';
-
-
   // Enable all beta-flags if betaMode is set to true
   if (cmdLine.options.betaMode)
     cmdLine.options.beta = availableBetaFlags;
@@ -452,7 +444,7 @@ function executeCommandLine(command, options, args) {
    * Print the model's messages to stderr in a human readable way.
    *
    * @param {CSN.Model | XSN.Model} model
-   * @param {CSN.Message[]} messages
+   * @param {CompileMessage[]} messages
    */
   function displayMessages( model, messages = options.messages ) {
     if (!Array.isArray(messages))
@@ -483,7 +475,6 @@ function executeCommandLine(command, options, args) {
         hasAtLeastOneExplanation = hasAtLeastOneExplanation || main.hasMessageExplanation(msg.messageId);
         const name = msg.location && msg.location.file;
         const fullFilePath = name ? path.resolve('', name) : undefined;
-        const context = fullFilePath ? sourceLines(fullFilePath) : [];
         log(main.messageStringMultiline(msg, {
           normalizeFilename,
           noMessageId: !options.showMessageId,
@@ -491,8 +482,12 @@ function executeCommandLine(command, options, args) {
           hintExplanation: true,
           color: options.color,
         }));
-        if (context)
+        if (fullFilePath && msg.location.col) {
+          // A message context only makes sense, if we have at least a medium precision,
+          // i.e. line and column for start position.
+          const context = sourceLines(fullFilePath);
           log(main.messageContext(context, msg, { color: options.color }));
+        }
         log(); // newline
       });
       if (options.showMessageId && hasAtLeastOneExplanation)
@@ -557,8 +552,13 @@ function executeCommandLine(command, options, args) {
       content = JSON.stringify(content, null, 2);
 
     if (dir === '-') {
-      if (!omitHeadline)
-        process.stdout.write(`// ------------------- ${fileName} -------------------\n`);
+      if (!omitHeadline) {
+        const sqlTypes = {
+          sql: true, hdbconstraint: true, hdbtable: true, hdbview: true,
+        };
+        const commentStarter = fileName.split('.').pop() in sqlTypes ? '--$' : '//';
+        process.stdout.write(`${commentStarter} ------------------- ${fileName} -------------------\n`);
+      }
 
       process.stdout.write(`${content}\n`);
       if (!omitHeadline)

package/bin/cdsse.js

@@ -144,8 +144,8 @@ function lint( err, buf ) {
   }
 }
 
-function tokensAt( buf, offset, col, symbol ) {
-  const src = `${buf.substring( 0, offset )}≠${buf.substring( offset )}`;
+function tokensAt( buf, _offset, col, symbol ) {
+  const src = `${buf.substring( 0, _offset )}≠${buf.substring( _offset )}`;
   const et = messageAt( compiler.parseX( src, frel, { messages: [] } ), 'expectedTokens', col ) || [];
   for (const n of et) {
     if (typeof symbol === 'string') {

package/doc/CHANGELOG_BETA.md

@@ -8,23 +8,30 @@ Note: `beta` fixes, changes and features are listed in this ChangeLog just for i
 The compiler behavior concerning `beta` features can change at any time without notice.
 **Don't use `beta` fixes, changes and features in productive mode.**
 
+## Version 2.XX.YY
+
+### Removed `assocsWithParams`
+
+Instead, of using the beta flag `assocsWithParams`, you can change the severity of the messages
+`def-unexpected-paramview-assoc` and `def-unexpected-calcview-assoc`.
+
 ## Version 2.12.0 - 2022-01-25
 
 ### Added `sqlSnippets`
 
 - to.sql/hdi/hdbcds: Introduce the annotations `@sql.prepend` and `@sql.append` that allow inserting user-written SQL snippets into the compiler generated content.
 
-## Version 2.11.0
+## Version 2.11.0 - 2021-12-02
 
 ### Removed `foreignKeyConstraints`
 
-## Version 2.10.4
+## Version 2.10.4 - 2021-11-05
 
 ### Fixed `nestedProjections`
 
 - to.sql/hdi/hdbcds: Correctly handle a `*` at the not-first place in the query
 
-## Version 2.6.0
+## Version 2.6.0 - 2021-08-23
 
 ### Removed `pretransformedCSN`
 
@@ -48,7 +55,7 @@ This is now the default - see CHANGELOG entry for 2.6.0
 - Composition of one w/o backlink will result in a constraint in
   the entity where the composition is defined
 
-## Version 2.4.4
+## Version 2.4.4 - 2021-07-02
 
 ### Added `nestedProjections`
 
@@ -59,7 +66,7 @@ This is now the default - see CHANGELOG entry for 2.6.0
 - _Some checks are missing and will be added! Minor changes might occur._
 - **The SQL backends might not work properly yet if nested projections are used!**
 
-## Version 2.4.2
+## Version 2.4.2 - 2021-07-01
 
 ### Added `keylessManagedAssoc`
 
@@ -71,7 +78,7 @@ This is now the default - see CHANGELOG entry for 2.6.0
   Consequently, these associations are not added to the `WITH ASSOCIATIONS` clause or forwarded to HANA CDS.
   Managed Associations without foreign keys must be enabled with `--beta: keylessManagedAssoc`
 
-## Version 2.4.0
+## Version 2.4.0 - 2021-06-28
 
 ### Changed `foreignKeyConstraints`
 

package/doc/CHANGELOG_DEPRECATED.md

@@ -11,9 +11,25 @@ Note: `deprecated` features are listed in this ChangeLog just for information.
 **When the `deprecated` option is set, the `beta` option is ignored,
 and several new features are not available.**
 
+## Version 2.XX.YY - 2022-MM-DD
+
+### Added `redirectInSubQueries`
+
+When this option is set, we auto-redirect associations and composition also in
+non-main queries, sometimes without rewriting the `keys`/`on` (there will be no
+fix for this).
+
+### Added `oldVirtualNotNullPropagation`
+
+When this option is set, we do not propagate `notNull` along types.
+Additionally, we propagate `notNull` and `virtual` from a query source element
+to the sub elements of a query entity element, even if the property is not
+propagated to the query entity element itself (like with type references).
+
+
 ## Version 2.2.0
 
-## Added `noScopedRedirections`
+### Added `noScopedRedirections`
 
 When this option is set, the definition scope is not taken into account when
 trying to find an implicit redirection target.  Setting the following
@@ -21,18 +37,18 @@ deprecated options also switches off scoped redirections (additionally to their
 other effect): `noElementsExpansion`, `generatedEntityNameWithUnderscore`,
 `shortAutoexposed`, `longAutoexposed`, `noInheritedAutoexposeViaComposition`.
 
-## Added `noInheritedAutoexposeViaComposition`
+### Added `noInheritedAutoexposeViaComposition`
 
 When this option is set, only entities directly specified after `Composition of` are
 auto-exposed, not entities used as target via explicit or implicit `redirected to`.
 
 ## Version 2.0.16
 
-## Added `downgradableErrors`
+### Added `downgradableErrors`
 
 Allow to change the severity of some errors which should stay to be an error.
 
-## Added `shortAutoexposed`
+### Added `shortAutoexposed`
 
 When this option is set (and `generatedEntityNameWithUnderscore`), the names of
 autoexposed entities are calculated according to the default compiler v1
@@ -40,13 +56,13 @@ behavior (without v1 options `dependentAutoexposed` and `longAutoexposed`).
 
 ## Version 2.0.10
 
-## Added `longAutoexposed`
+### Added `longAutoexposed`
 
 When this option is set (and `generatedEntityNameWithUnderscore`),
 the names of autoexposed entities are calculated according to the
 compiler v1 option `longAutoexposed`.
 
-## Added `generatedEntityNameWithUnderscore`
+### Added `generatedEntityNameWithUnderscore`
 
 Keep using `_` is separator for generated autoexposed entities and for entities
 created for managed compositions.  It also disables a definition `A.B.C` if `A`

package/doc/NameResolution.md

@@ -22,22 +22,27 @@ others might want to [skip the introduction](#design-principles).
 
 ## Table of Contents
 
- 1. [Introduction](#introduction)
-    <br/>  –  [Background: SQL](#background-sql)
-    <br/>  –  [Background: modern programming languages](#background-modern-programming-languages)
- 2. [Design Principles](#design-principles)
- 3. [Name Resolution - the Basics](#name-resolution---the-basics)
-    <br/>  –  [Common rules](#common-rules)
-    <br/>  –  [Resolving paths](#resolving-paths)
-    <br/>  –  [Navigation environment](#navigation-environment)
- 4. [References to main artifacts](#references-to-main-artifacts)
- 5. [Values and references to elements](#values-and-references-to-elements)
-    <br/>  –  [References in queries](#references-in-queries)
-    <br/>  –  [References to sibling elements](#references-to-sibling-elements)
-    <br/>  –  [Other element references](#other-element-references)
- 6. [Paths as Annotation Values](#paths-as-annotation-values)
- 7. [Differences to HANA-CDS](#differences-to-hana-cds)
- 8. [Summary](#summary)
+<!-- toc: start -->
+
+1. [Table of Contents](#table-of-contents)
+2. [Introduction](#introduction)
+   1. [Background: SQL](#background-sql)
+   2. [Background: modern programming languages](#background-modern-programming-languages)
+3. [Design Principles](#design-principles)
+4. [Name Resolution - the Basics](#name-resolution---the-basics)
+   1. [Common rules](#common-rules)
+   2. [Resolving paths](#resolving-paths)
+   3. [Navigation environment](#navigation-environment)
+5. [References to main artifacts](#references-to-main-artifacts)
+6. [Values and references to elements](#values-and-references-to-elements)
+   1. [References in queries](#references-in-queries)
+   2. [References to sibling elements](#references-to-sibling-elements)
+   3. [Other element references](#other-element-references)
+7. [Paths as annotation values](#paths-as-annotation-values)
+8. [Differences to HANA-CDS](#differences-to-hana-cds)
+9. [Summary](#summary)
+
+<!-- toc: end -->
 
 
 ## Introduction

package/lib/api/main.js

@@ -35,6 +35,7 @@ const propertyToCheck = {
 
 const { cloneCsn } = require('../model/csnUtils');
 const { toHdbcdsSource } = require('../render/toHdbcds');
+const { ModelError } = require('../base/error');
 
 const relevantGeneralOptions = [ /* for future generic options */ ];
 const relevantOdataOptions = [ 'sqlMapping', 'odataFormat' ];
@@ -115,7 +116,7 @@ function checkPreTransformedCsn(csn, options, relevantOptionNames, warnAboutMism
  * @returns {boolean} Return true if it is pre-transformed
  */
 function isPreTransformed(csn, transformation) {
-  return csn.meta && csn.meta.transformation === transformation;
+  return csn && csn.meta && csn.meta.transformation === transformation;
 }
 
 /**
@@ -173,7 +174,7 @@ function forSql(csn, options = {}) {
  * Transform a CSN like to.hdi
  *
  * @param {CSN.Model} csn Plain input CSN
- * @param {hdiOptions} [options={}] Options
+ * @param {HdiOptions} [options={}] Options
  * @returns {CSN.Model} CSN transformed like to.hdi
  * @private
  */
@@ -186,7 +187,7 @@ function forHdi(csn, options = {}) {
  * Transform a CSN like to.hdbcds
  *
  * @param {CSN.Model} csn Plain input CSN
- * @param {hdbcdsOptions} [options={}] Options
+ * @param {HdbcdsOptions} [options={}] Options
  * @returns {CSN.Model} CSN transformed like to.hdbcds
  * @private
  */
@@ -224,7 +225,7 @@ function sql(csn, options = {}) {
  * Process the given CSN into HDI artifacts.
  *
  * @param {CSN.Model} csn A clean input CSN
- * @param {hdiOptions} [options={}] Options
+ * @param {HdiOptions} [options={}] Options
  * @returns {HDIArtifacts} { <filename>:<content>, ...}
  */
 function hdi(csn, options = {}) {
@@ -247,19 +248,19 @@ function hdi(csn, options = {}) {
     const flat = flattenResultStructure(intermediateResult);
 
     const nameMapping = Object.create(null);
-    const sqlsWithCSNNamesToSort = Object.create(null);
-    const sqlsNotToSort = Object.create(null);
+    const sqlArtifactsWithCSNNamesToSort = Object.create(null);
+    const sqlArtifactsNotToSort = Object.create(null);
 
     Object.keys(flat).forEach((key) => {
       const artifactNameLikeInCsn = key.replace(/\.[^/.]+$/, '');
       nameMapping[artifactNameLikeInCsn] = key;
       if (key.endsWith('.hdbtable') || key.endsWith('.hdbview'))
-        sqlsWithCSNNamesToSort[artifactNameLikeInCsn] = flat[key];
+        sqlArtifactsWithCSNNamesToSort[artifactNameLikeInCsn] = flat[key];
       else
-        sqlsNotToSort[key] = flat[key];
+        sqlArtifactsNotToSort[key] = flat[key];
     });
 
-    const sorted = sortViews({ sql: sqlsWithCSNNamesToSort, csn: sqlCSN })
+    const sorted = sortViews({ sql: sqlArtifactsWithCSNNamesToSort, csn: sqlCSN })
       .filter(obj => obj.sql)
       .reduce((previous, current) => {
         const hdiArtifactName = remapName(nameMapping[current.name], sqlCSN, k => !k.endsWith('.hdbindex'));
@@ -267,9 +268,9 @@ function hdi(csn, options = {}) {
         return previous;
       }, Object.create(null));
 
-    // now add the not-sorted stuff, like indizes
-    Object.keys(sqlsNotToSort).forEach((key) => {
-      sorted[remapName(key, sqlCSN, k => !k.endsWith('.hdbindex'))] = sqlsNotToSort[key];
+    // now add the not-sorted stuff, like indices
+    Object.keys(sqlArtifactsNotToSort).forEach((key) => {
+      sorted[remapName(key, sqlCSN, k => !k.endsWith('.hdbindex'))] = sqlArtifactsNotToSort[key];
     });
 
     return sorted;
@@ -325,7 +326,7 @@ function remapName(key, csn, filter = () => true) {
  * Note: Only supports changes in entities (not views etc.) compiled/rendered as HANA-CSN/SQL.
  *
  * @param {CSN.Model}  csn          A clean input CSN representing the desired "after-image"
- * @param {hdiOptions} options      Options
+ * @param {HdiOptions} options      Options
  * @param {CSN.Model}  beforeImage  A HANA-transformed CSN representing the "before-image", or null in case no such image
  *                                  is known, i.e. for the very first migration step
  * @returns {object} - afterImage:  The desired after-image in HANA-CSN format
@@ -346,8 +347,8 @@ function hdiMigration(csn, options, beforeImage) {
    * This is for backward compatibility with @sap/cds@4.5.(2…3).
    *
    * @todo Remove in cds-compiler@2.x
-   * @param {hdiOptions|CSN.Model} inputOptions Options or CSN image
-   * @param {CSN.Model|hdiOptions} inputBeforeImage CSN image or options
+   * @param {HdiOptions|CSN.Model} inputOptions Options or CSN image
+   * @param {HdiOptions|CSN.Model} inputBeforeImage CSN image or options
    * @returns {Array} Array where the real options come first
    */
   function backwardCompatible(inputOptions, inputBeforeImage) {
@@ -438,7 +439,7 @@ hdi.migration = hdiMigration;
  * Process the given CSN into HDBCDS artifacts.
  *
  * @param {any} csn A clean input CSN
- * @param {hdbcdsOptions} [options={}] Options
+ * @param {HdbcdsOptions} [options={}] Options
  * @returns {HDBCDS} { <filename>:<content>, ...}
  */
 function hdbcds(csn, options = {}) {
@@ -643,12 +644,18 @@ function publishCsnProcessor( processor, _name ) {
       return processor( csn, options, ...args );
     }
     catch (err) {
-      if (err instanceof CompilationError || options.noRecompile)
-      // options.testMode && err instanceof RangeError) // stack overflow
+      if (err instanceof CompilationError || options.noRecompile || isPreTransformed(csn, 'odata')) // we cannot recompile a pre-transformed CSN
+        throw err;
+
+      if (options.testMode && !(err instanceof TypeError) &&
+        !(err instanceof ModelError))
         throw err;
 
       const { info } = makeMessageFunction( csn, options, 'compile' );
-      info( 'api-recompiled-csn', emptyLocation('csn.json'), {}, 'CSN input had to be recompiled' );
+      const msg = info( 'api-recompiled-csn', emptyLocation('csn.json'), {}, 'CSN input had to be recompiled' );
+      if (options.internalMsg)
+        msg.error = err; // Attach original error
+
       // next line to be replaced by CSN parser call which reads the CSN object
       const xsn = recompileX(csn, options);
       const recompiledCsn = compactModel(xsn);
@@ -682,12 +689,6 @@ function publishCsnProcessor( processor, _name ) {
  * @typedef {'plain' | 'quoted' | 'hdbcds' } NamingMode
  */
 
-/**
- * Available SQL change modes
- *
- * @typedef {'alter' | 'drop' } SqlChangeMode
- */
-
 /**
  * Available oData versions
  *
@@ -700,42 +701,6 @@ function publishCsnProcessor( processor, _name ) {
  * @typedef { 'structured' | 'flat' } oDataFormat
  */
 
-/**
- * Generally available options
- *
- * @typedef {object} Options
- * @property {object} [beta] Enable experimental features - not for productive use!
- * @property {boolean} [dependentAutoexposed=false] For dependent autoexposed entities (managed compositions, texts entity), follow name of base entity
- * @property {boolean} [longAutoexposed=false] Deprecated: Produce long names (with underscores) for autoexposed entities
- * @property {Map<string, number>} [severities={}] Map of message-id and severity that allows setting the severity for the given message
- * @property {Array} [messages] Allows collecting all messages in the options instead of printing them to stderr.
- */
-
-/**
- * Options available for to.hdi
- *
- * @typedef {object} hdiOptions
- * @property {NamingMode} [sqlMapping='plain'] Naming mode to use
- * @property {SqlChangeMode} [sqlChangeMode='alter'] SQL change mode to use (for changed columns)
- * @property {boolean} [allowCsnDowngrade=false] Allow downgrades of CSN major version (for modelCompare)
- * @property {object} [beta] Enable experimental features - not for productive use!
- * @property {boolean} [longAutoexposed=false] Deprecated: Produce long names (with underscores) for autoexposed entities
- * @property {Map<string, number>} [severities={}] Map of message-id and severity that allows setting the severity for the given message
- * @property {Array} [messages] Allows collecting all messages in the options instead of printing them to stderr.
- */
-
-/**
- * Options available for to.hdbcds
- *
- * @typedef {object} hdbcdsOptions
- * @property {NamingMode} [sqlMapping='plain'] Naming mode to use
- * @property {object} [beta] Enable experimental features - not for productive use!
- * @property {boolean} [longAutoexposed=false] Deprecated: Produce long names (with underscores) for autoexposed entities
- * @property {Map<string, number>} [severities={}] Map of message-id and severity that allows setting the severity for the given message
- * @property {Array} [messages] Allows collecting all messages in the options instead of printing them to stderr.
- */
-
-
 /**
  * A fresh (just compiled, not transformed) CSN
  *
@@ -755,13 +720,13 @@ function publishCsnProcessor( processor, _name ) {
  */
 
 /**
- * A map of { <file.hdbcds>:<content> }.
+ * A map of { <file.hdbcds|hdbconstraint>:<content> }.
  *
  * @typedef {object} HDBCDS
  */
 
 /**
- * A map of { <file.hdbtable/view...>:<content> }.
+ * A map of { <file.hdbtable|hdbview|hdbconstraint...>:<content> }.
  *
  * @typedef {object} HDIArtifacts
  */

package/lib/api/options.js

@@ -49,7 +49,7 @@ const privateOptions = [
   'traceParserAmb',
   'testMode',
   'testSortCsn',
-  'constraintsAsAlter',
+  'constraintsInCreateTable',
   'integrityNotEnforced',
   'integrityNotValidated',
   'assertIntegrity',
@@ -158,7 +158,7 @@ module.exports = {
     sql: (options) => {
       const hardOptions = { src: 'sql' };
       const defaultOptions = { sqlMapping: 'plain', sqlDialect: 'plain' };
-      const processed = translateOptions(options, defaultOptions, hardOptions, undefined, [ 'sql-dialect-and-naming', 'constraints-as-alter-sqlite' ], 'to.sql');
+      const processed = translateOptions(options, defaultOptions, hardOptions, undefined, [ 'sql-dialect-and-naming' ], 'to.sql');
 
       const result = Object.assign({}, processed);
       result.toSql = Object.assign({}, processed);
@@ -166,7 +166,7 @@ module.exports = {
       return result;
     },
     hdi: (options) => {
-      const hardOptions = { src: 'hdi', constraintsAsAlter: false };
+      const hardOptions = { src: 'hdi' };
       const defaultOptions = { sqlMapping: 'plain', sqlDialect: 'hana' };
       const processed = translateOptions(options, defaultOptions, hardOptions, { sqlDialect: generateStringValidator([ 'hana' ]) }, undefined, 'to.hdi');
 

package/lib/api/validate.js

@@ -142,11 +142,6 @@ const allCombinationValidators = {
     severity: 'warning',
     getMessage: () => 'Option "beta" was used. This option should not be used in productive scenarios!',
   },
-  'constraints-as-alter-sqlite': {
-    validate: options => options.constraintsAsAlter && options.sqlDialect && options.sqlDialect === 'sqlite',
-    severity: 'warning',
-    getMessage: options => `Option 'constraintsAsAlter' is ignored for sqlDialect '${ options.sqlDialect }'`,
-  },
 };
 /* eslint-disable jsdoc/no-undefined-types */
 /**