@sap/cds-compiler 4.8.0 → 4.9.0

package/lib/json/from-csn.js

@@ -649,7 +649,8 @@ const schema = compileSchema( {
     noPrefix: false,            // just '@' is no CSN property
     prop: '@‹anno›',            // which property name do messages use for annotation assignments?
     type: annotation,
-    inKind: () => true,         // allowed in all definitions (including columns and extensions)
+    // allowed in all definitions except mixins (including columns and extensions)
+    inKind: _kind => true, // TODO(v5): (kind !== 'mixin')
     schema: {
       '-expr': { // '-expr' and '-' must not exist top-level
         prop: '@‹anno›',
@@ -745,7 +746,7 @@ const schema = compileSchema( {
     ignore: true, type: ignore, // TODO: do we need to do something?
   },
   namespace: {
-    type: stringRef,
+    type: namespace,
   },
   meta: {                       // meta information
     type: ignore,               // TODO: should we test s/th here?
@@ -1043,6 +1044,11 @@ function definition( def, spec, xsn, csn, name ) {
   }
 }
 
+function namespace( ref, spec ) {
+  const ns = stringRef(ref, spec);
+  return ns ? { kind: 'namespace', name: ns } : null;
+}
+
 function couldNotBeEnumProperty( prop ) {
   // returns true for `value` (which we allow with warning when extending an enum with `elements`)
   const inKind = schema[prop]?.inKind; // undefined for annotations, $location, …
@@ -1486,9 +1492,12 @@ function annoValue( val, spec ) {
 function annotation( val, spec, xsn, csn, name ) {
   // not used for the value
   const id = (xsn ? name.substring(1) : name);
-  if (!id) {              // `"@": …` is already syntax-unknown-property
+  if (!id)              // `"@": …` is already syntax-unknown-property
     message( 'syntax-invalid-name', location(true), { '#': '{}' } );
-  }
+
+  if (xsn && xsn.kind === 'mixin') // TODO(v5): Remove, adapt spec['@'].inKind
+    message('anno-unexpected-mixin', location(true));
+
   const n = { id, location: location() };
   const r = annoValue( val, spec );
   r.name = n;

package/lib/json/to-csn.js

@@ -232,15 +232,13 @@ function compactModel( model, options = model.options || {} ) {
     if (using.length)
       csn.requires = using;
   }
-  if (!isBetaEnabled(options, 'v5preview')) {
-    // 'namespace' for complete model is 'namespace' of first source
-    // (not a really useful property at all, avoids XSN inspection by Umbrella)
-    for (const first in srcDict) {
-      const { namespace } = srcDict[first];
-      if (namespace?.path)
-        csn.namespace = pathName( namespace.path );
-      break;
-    }
+  // 'namespace' for complete model is 'namespace' of first source
+  // (not a really useful property at all, avoids XSN inspection by Umbrella)
+  for (const first in srcDict) {
+    const { namespace } = srcDict[first];
+    if (namespace?.name?.path)
+      csn.namespace = pathName( namespace.name.path );
+    break;
   }
   set( 'definitions', csn, model );
   if (Object.keys(model.vocabularies || {}).length > 0)
@@ -1010,8 +1008,9 @@ function artifactRef( node, terse ) {
     return undefined;
   // Works also on XSN directly coming from parser and with XSN from CDL->CSN transformation
   // Shortcut for many cases:
-  if (terse && node._artifact && !node._artifact._main && terse !== '.path')
-    return node._artifact.name.id;
+  const art = node._artifact;
+  if (art && (!art._main || art.kind === '$self') && terse && terse !== '.path')
+    return art.name.id;
   let { path } = node;
   if (!path)
     return undefined;           // TODO: complain with strict

package/lib/language/genericAntlrParser.js

@@ -724,11 +724,11 @@ function fragileAlias( ast, safe = false ) {
     return ast;
   if (safe || ast.$delimited || !/^[a-zA-Z][a-zA-Z_]+$/.test( ast.id )) {
     this.warning( 'syntax-deprecated-auto-as', ast.location, { keyword: 'as' },
-                  'Please add the keyword $(KEYWORD) in front of the alias name' );
+                  'Add keyword $(KEYWORD) in front of the alias name' );
   }
   else {                         // configurable error
     this.message( 'syntax-missing-as', ast.location, { keyword: 'as' },
-                  'Please add the keyword $(KEYWORD) in front of the alias name' );
+                  'Add keyword $(KEYWORD) in front of the alias name' );
   }
   return ast;
 }
@@ -759,10 +759,16 @@ function identAst( token, category, noTokenTypeCheck = false ) {
 
     // $delimited is used to complain about ![$self] and other magic vars usage;
     // we might complain about that already here via @arg{category}
-    return { id, $delimited: true, location: this.tokenLocation( token ) };
+
+    const ast = { id, $delimited: true, location: this.tokenLocation( token ) };
+    ast.location.tokenIndex = token.tokenIndex;
+    return ast;
+  }
+  if (token.text[0] !== '"') {
+    const ast = { id, location: this.tokenLocation(token) };
+    ast.location.tokenIndex = token.tokenIndex;
+    return ast;
   }
-  if (token.text[0] !== '"')
-    return { id, location: this.tokenLocation( token ) };
   // delimited:
   id = id.slice( 1, -1 ).replace( /""/g, '"' );
   if (!id) {
@@ -773,7 +779,9 @@ function identAst( token, category, noTokenTypeCheck = false ) {
                   // eslint-disable-next-line max-len
                   'Deprecated delimited identifier syntax, use $(DELIMITED) - strings are delimited by single quotes' );
   }
-  return { id, $delimited: true, location: this.tokenLocation( token ) };
+  const ast = { id, $delimited: true, location: this.tokenLocation( token ) };
+  ast.location.tokenIndex = token.tokenIndex;
+  return ast;
 }
 
 function pushXprToken( args ) {

package/lib/main.d.ts

@@ -150,6 +150,22 @@ declare namespace compiler {
      * @since v2.12.1
      */
     $xsnObjects?: boolean
+    /**
+     * Internal option for LSP only!
+     * If set, each AST gets a `tokenStream` property containing all lexed tokens.
+     *
+     * @private
+     */
+    attachTokens?: boolean
+    /**
+     * Internal option for LSP only!
+     * If set, enables some extra checks/work for the CDS LSP.
+     * May be implicitly set by `$lsp.<api>` functions.
+     *
+     * @private
+     * @since v4.9
+     */
+    lspMode?: boolean
   }
 
   /**
@@ -260,13 +276,34 @@ declare namespace compiler {
   /**
    * Options used by the `for.effective()` CSN transformation.
    *
-   * WORK IN PROGRESS
-   *
    * @internal
    * @see _for.effective()
    */
   export interface EffectiveCsnOptions extends SqlOptions {
-    // TODO
+    /**
+     * If true, resolve simple type references to their simple base type.
+     *
+     * @default true
+     */
+    resolveSimpleTypes?: boolean
+    /**
+     * If true, transform projections into ordinary views with SELECT.
+     *
+     * @default true
+     */
+    resolveProjections?: boolean
+    /**
+     * If true, remap OData annotations to ABAP annotations.
+     *
+     * @default false
+     */
+    remapOdataAnnotations?: boolean
+    /**
+     * If true, keep '.localized' property in the CSN.
+     *
+     * @default false
+     */
+    keepLocalized?: boolean
   }
 
   /**
@@ -547,11 +584,6 @@ declare namespace compiler {
        * @internal
        */
       model?: CSN;
-      /**
-       * Used by `cdsc` to indicate whether the message was already printed to stderr.
-       * @private
-       */
-      hasBeenReported: boolean;
   }
 
   /**
@@ -666,16 +698,23 @@ declare namespace compiler {
    * @param config.noMessageId
    *     If true, will _not_ show the message ID (+ explanation hint) in the output.
    *
+   * @param config.idInBrackets
+   *     If true, the message ID (if there is one and noMessageId is falsey) will be put in brackets.
+   *     This will be the default in cds-compiler v5.
+   *
    * @param config.noHome
    *     If true, will _not_ show message's semantic location.
    *
-   * @param config.module
+   * @param config.moduleForMarker
    *     If set, downgradable error messages will get a '‹↓›' marker, depending on whether
-   *     the message can be downgraded for the given module.
+   *     the message can be downgraded for the given module.  A `‹↑›` is used if the message
+   *     will be an error in the next major cds-compiler release.
+   *     Was called `module` in v4.8.0 and earlier.
    */
   export function messageString(msg: CompileMessage, config?: {
     normalizeFilename?: boolean
     noMessageId?: boolean
+    idInBrackets?: boolean
     noHome?: boolean
     module?: string
   }): string;
@@ -713,9 +752,11 @@ declare namespace compiler {
    * @param config.hintExplanation
    *     If true, messages with explanations will get a "…" marker, see {@link hasMessageExplanation}.
    *
-   * @param config.module
+   * @param config.moduleForMarker
    *     If set, downgradable error messages will get a '‹↓›' marker, depending on whether
-   *     the message can be downgraded for the given module.
+   *     the message can be downgraded for the given module.  A `‹↑›` is used if the message
+   *     will be an error in the next major cds-compiler release.
+   *     Was called `module` in v4.8.0 and earlier.
    *
    * @param config.sourceMap
    *     A dictionary of filename<->source-code entries.  You can pass the fileCache that is used
@@ -742,7 +783,7 @@ declare namespace compiler {
     normalizeFilename?: boolean
     noMessageId?: boolean
     hintExplanation?: boolean
-    module?: string
+    moduleForMarker?: string
     sourceMap?: Record<string, string>
     sourceLineMap?: Record<string, number[]>
     cwd?: string
@@ -857,6 +898,12 @@ declare namespace compiler {
      * @internal
      */
     function effective(csn: CSN, options?: EffectiveCsnOptions): CSN;
+    /**
+     * Transform the given CSN into one that has the properties required for SEAL
+     *
+     * @internal
+     */
+    function seal(csn: CSN, options?: EffectiveCsnOptions): CSN;
   }
 
   export { _for as for };
@@ -1269,7 +1316,13 @@ declare namespace compiler {
    * The functions in `userFunctions` are usually transformer functions, which
    * change the input CSN destructively.
    */
-  export function traverseCsn(userFunctions: Record<string, Function>, csn: object|any[]): void;
+  export function traverseCsn(userFunctions: Record<string, TraverseCsnCallback>, csn: object|any[]): void;
+  export type TraverseCsnCallback = (
+    userFunctions: Record<string, TraverseCsnCallback>,
+    value: any,
+    csn: any,
+    prop: string
+  ) => any;
 
   /**
    * CSN Model related functions.

package/lib/main.js

@@ -128,6 +128,7 @@ module.exports = {
     odata: (...args) => snapi.odata(...args),
     java: (...args) => snapi.java(...args),
     effective: (...args) => snapi.for_effective(...args),
+    seal: (...args) => snapi.for_seal(...args),
   },
   to: {
     cdl: Object.assign((...args) => snapi.cdl(...args), {

package/lib/model/cloneCsn.js

@@ -47,9 +47,12 @@ const internalCsnPropertyNames = Object.keys(internalCsnProps);
  * @see cloneAnnotationValue()
  * @see cloneCsnDict()
  *
- * @param {object} csn Top-level CSN.  You can pass non-dictionary values.
- * @param {CSN.Options} options CSN Options, only used for `dictionaryPrototype`, `testMode`, and `testSortCsn`.
- * @param {boolean} sort Whether to sort CSN properties.
+ * @param {object} csn
+ *   Top-level CSN.  You can pass non-dictionary values.
+ * @param {CSN.Options} options
+ *   CSN Options, only used for `dictionaryPrototype`, `testMode`, and `testSortCsn`.
+ * @param {boolean} sort
+ *   Whether to sort CSN properties.
  */
 function cloneCsn( csn, options, sort ) {
   if (!csn || typeof csn !== 'object')

package/lib/model/csnRefs.js

@@ -230,7 +230,8 @@ const referenceSemantics = {
   orderBy_ref: { lexical: query => query, dynamic: 'query' },
   orderBy_expr: { lexical: query => query, dynamic: 'source' }, // ref in ORDER BY expression
   orderBy_set_ref: { lexical: query => query.$next, dynamic: 'query' }, // to outer SELECT (from UNION)
-  // refs in ORDER BY expr in UNION not really allowed - only with table alias (of outer queries) or $self
+  // refs in ORDER BY expr in UNION not really allowed
+  // only with table alias (of outer queries) or $self
   orderBy_set_expr: { lexical: query => query.$next, dynamic: false },
   // default: { lexical: query => query, dynamic: 'source' }
 };
@@ -409,8 +410,11 @@ function csnRefs( csn, universalReady ) {
   }
 
   function getOriginRaw( art ) {
-    if (art.type)               // TODO: make robust against "linked" = only direct
-      return artifactRef( art.type, BUILTIN_TYPE );
+    if (art.type) {               // TODO: make robust against "linked" = only direct
+      return (art.type !== '$self' || csn.definitions.$self)
+        ? artifactRef( art.type, BUILTIN_TYPE )
+        : getCache( boundActionOrMain( art ), '_parent' );
+    }
     if (typeof art.$origin === 'object') // null, […], {…}
       return getOriginExplicit( art.$origin );
 
@@ -698,7 +702,8 @@ function csnRefs( csn, universalReady ) {
     const links = path.map( (_v, idx) => ({ idx }) );
     // TODO: backends should be changed to enable uncommenting:
     // if (!art)    // does not work with test3/Associations/KeylessManagedAssociation/
-    //   throw new ModelError( `Path item 0=${ pathId( path[0] ) } refers to nothing, scope: ${ scope }`);
+    //   throw new ModelError( `Path item 0=${ pathId( path[0] )
+    //       } refers to nothing, scope: ${ scope }`);
     links[0].art = art;
     for (let i = 1; i < links.length; ++i) { // yes, starting at 1, links[0] is set above
       parent = navigationEnv( art, staticAssoc );
@@ -1010,8 +1015,8 @@ function queryOrMain( query, main ) {
  * - NOT on a `join` node inside `from`.
  *
  * @param {CSN.Query} query
- * @param {CSN.QuerySelect} fromSelect: for query in `from`
- * @param {CSN.Query} parentQuery: for a sub query (ex those in `from`)
+ * @param {CSN.QuerySelect} fromSelect for query in `from`
+ * @param {CSN.Query} parentQuery for a sub query (ex those in `from`)
  * @param {(query: CSN.Query&CSN.QueryFrom, select: CSN.QuerySelectEnriched, parentQuery: CSN.Query) => void} callback
  */
 function traverseQuery( query, fromSelect, parentQuery, callback ) {
@@ -1135,7 +1140,7 @@ function startCsnPath( csnPath, csn ) {
     };
   }
   if (csnPath.length < 2 || head !== 'definitions' && head !== 'vocabularies')
-    throw new CompilerAssertion( 'References outside definitions and vocabularies not supported yet');
+    throw new CompilerAssertion( 'References outside definitions and vocabularies not supported yet' );
   const art = csn[head][csnPath[1]];
   return {
     index: 2, main: art, parent: art, art, query: null,

package/lib/model/csnUtils.js

@@ -6,6 +6,7 @@ const {
   applyTransformations,
   applyTransformationsOnNonDictionary,
   applyTransformationsOnDictionary,
+  mergeTransformers,
 } = require('../transform/db/applyTransformations');
 const { isBuiltinType, isAnnotationExpression } = require('../base/builtins');
 const { ModelError, CompilerAssertion } = require('../base/error');
@@ -134,7 +135,8 @@ function getUtils( model, universalReady ) {
       }
       else if (arg.ref) {
         const art = artifactRef.from(arg);
-        elements = mergeElementsIntoMap(elements, art.elements, art.$location, arg.as || implicitAs(arg.ref), implicitAs(arg.ref) || arg.as);
+        elements = mergeElementsIntoMap(elements, art.elements, art.$location,
+                                        arg.as || implicitAs(arg.ref), implicitAs(arg.ref) || arg.as);
       }
       else if (arg.SELECT || arg.SET) {
         elements = mergeElementMaps(elements, getSources(arg, true));
@@ -670,11 +672,13 @@ function isEdmPropertyRendered( elementCsn, options ) {
   // V4 struct: on/off
   if (elementCsn == null)
     return false;
-  const renderForeignKey = (options.odataVersion === 'v4' && options.odataFormat === 'structured') ? !!options.odataForeignKeys : true;
+  const renderForeignKey = (options.odataVersion === 'v4' && options.odataFormat === 'structured')
+    ? !!options.odataForeignKeys : true;
   const isNotIgnored = !elementCsn.target ? !elementCsn['@cds.api.ignore'] : true;
   const isNavigable = elementCsn.target
     ? (elementCsn['@odata.navigable'] === undefined ||
-     elementCsn['@odata.navigable'] !== undefined && (elementCsn['@odata.navigable'] === null || elementCsn['@odata.navigable'] === true)) : true;
+     elementCsn['@odata.navigable'] !== undefined &&
+      (elementCsn['@odata.navigable'] === null || elementCsn['@odata.navigable'] === true)) : true;
   // Foreign Keys can be ignored
   if (elementCsn['@odata.foreignKey4'])
     return isNotIgnored && renderForeignKey;
@@ -1124,7 +1128,7 @@ function moveAnnotationsAndDoc( sourceNode, targetNode, overwrite = false ) {
  *       override: Whether to ignore existing annotations.
  *       filter:   Positive filter. If it returns true, annotations for the referenced artifact
  *                 will be applied.
- *       applyToElements: Wether to apply annotations to elements or only to artifacts
+ *       applyToElements: Whether to apply annotations to elements or only to artifacts
  */
 function applyAnnotationsFromExtensions( csn, config ) {
   if (!csn.extensions)
@@ -1189,7 +1193,8 @@ function isAspect( node ) {
  */
 function hasValidSkipOrExists( artifact ) {
   return artifact.kind === 'entity' &&
-         (hasAnnotationValue(artifact, '@cds.persistence.exists', true) || hasAnnotationValue(artifact, '@cds.persistence.skip', true));
+         (hasAnnotationValue(artifact, '@cds.persistence.exists', true) ||
+          hasAnnotationValue(artifact, '@cds.persistence.skip', true));
 }
 
 /**
@@ -1368,7 +1373,8 @@ function functionList( functions, thisArg ) {
  * @returns {boolean}
  */
 function isDollarSelfOrProjectionOperand( arg ) {
-  return arg.ref && arg.ref.length === 1 && (arg.ref[0] === '$self' || arg.ref[0] === '$projection');
+  return arg.ref && arg.ref.length === 1 &&
+    (arg.ref[0] === '$self' || arg.ref[0] === '$projection');
 }
 
 /**
@@ -1408,7 +1414,6 @@ function findAnnotationExpression( node, prop ) {
       },
     });
   }
-
   return isExpr;
 }
 
@@ -1430,6 +1435,7 @@ module.exports = {
   applyTransformations,
   applyTransformationsOnNonDictionary,
   applyTransformationsOnDictionary,
+  mergeTransformers,
   setDependencies,
   isPersistedOnDatabase,
   isPersistedAsView,

package/lib/model/enrichCsn.js

@@ -207,7 +207,9 @@ function enrichCsn( csn, options = {} ) {
   }
 
   function pathRef( parent, prop, path, inspectionPath = csnPath ) {
-    const inspection = handleError( err => ((err) ? { scope: err.toString() } : inspectRef( inspectionPath )));
+    const inspection = handleError( err => ((err)
+      ? { scope: err.toString() }
+      : inspectRef( inspectionPath )));
     const {
       links, art, scope, $env,
     } = inspection;

package/lib/model/revealInternalProperties.js

@@ -38,7 +38,8 @@ const kindsRepresentedAsLinks = {
   // represent table alias in from / join-args property as link:
   $tableAlias: tableAliasAsLink,
   // represent "navigation elements" in _combined as links:
-  $navElement: (art, parent) => art._parent && parent !== art._parent.elements && art._parent.kind !== 'aspect',
+  $navElement: (art, parent) => art._parent && parent !== art._parent.elements &&
+    art._parent.kind !== 'aspect',
   // represent mixin in $tableAliases as link:
   mixin: tableAliasAsLink,
   // represent $projection as link, as it is just another search name for $self:

package/lib/model/sortViews.js

@@ -5,8 +5,11 @@ const { ModelError } = require('../base/error');
 
 /**
  * @typedef {Object} Layers
- * @property {Array[]} layers - An array of arrays, each subarray encompassing one Layer - L0 being layers[0].
- * @property {CSN.Artifact[]} leftover - Any artifacts not sorted into a layer due to unmet dependencies - points to there being some error.
+ * @property {Array[]} layers
+ *   An array of arrays, each subarray encompassing one Layer - L0 being layers[0].
+ * @property {CSN.Artifact[]} leftover
+ *   Any artifacts not sorted into a layer due to unmet dependencies.
+ *   Points to there being some error.
  */
 
 /**
@@ -22,7 +25,8 @@ const { ModelError } = require('../base/error');
  */
 function sortTopologically( csn, _dependents, _dependencies ) {
   const layers = [];
-  let { zero, nonZero } = _calculateDepth(Object.entries(csn.definitions), _dependents, _dependencies);
+  let { zero, nonZero } = _calculateDepth(Object.entries(csn.definitions),
+                                          _dependents, _dependencies);
   while (zero.length !== 0) {
     const currentLayer = [];
     zero.forEach(([ artifactName, artifact ]) => {
@@ -87,8 +91,10 @@ function _findWithXPointers( definitionsArray, x, _dependents, _dependencies ) {
  * For ordering, only the FROM clause of views is checked - this requires A2J to
  * be run beforehand to resolve association usages.
  *
- * @param {{sql: string, csn: CSN.Model}} arg sql: Map of <object name>: "CREATE STATEMENT", csn: Model
- * @returns {{name: string, sql: string}[]} Sorted array of artifact name / "CREATE STATEMENTS" pairs
+ * @param {{sql: string, csn: CSN.Model}} arg
+ *   sql: Map of <object name>: "CREATE STATEMENT", csn: Model
+ * @returns {{name: string, sql: string}[]}
+ *   Sorted array of artifact name / "CREATE STATEMENTS" pairs
  */
 module.exports = function sortViews({ sql, csn }) {
   const { cleanup, _dependents, _dependencies } = setDependencies(csn);
@@ -98,7 +104,9 @@ module.exports = function sortViews({ sql, csn }) {
 
   const result = [];
   // keep the "artifact name" - needed for to.hdi sorting
-  layers.forEach(layer => layer.forEach(objName => result.push({ name: objName, sql: sql[objName], dependents: csn.definitions[objName][_dependents] })));
+  layers.forEach(layer => layer.forEach(objName => result.push({
+    name: objName, sql: sql[objName], dependents: csn.definitions[objName][_dependents],
+  })));
   // attach sql artifacts which are not considered during the view sorting algorithm
   // --> this is the case for "ALTER TABLE ADD CONSTRAINT" statements,
   //     because their identifiers are not part of the csn.definitions

package/lib/modelCompare/compare.js

@@ -13,6 +13,12 @@ const { forEachKey, forEach } = require('../utils/objectUtils');
 // used to mark a view as changed so we know to drop-create it
 const isChanged = Symbol('Marks a view as changed');
 
+const relevantProperties = {
+  doc: true,
+  '@sql.prepend': true,
+  '@sql.append': true,
+};
+
 /**
  * Compares two models, in HANA-transformed CSN format, to each other.
  *
@@ -82,6 +88,33 @@ function getExtensionAndMigrations(beforeModel, options, {
 }) {
   return function compareArtifacts(artifact, name) {
     let hasPrimaryKeyChange = false;
+    const otherArtifact = beforeModel.definitions[name];
+    const isPersisted = isPersistedAsTable(artifact);
+    const isPersistedOther = otherArtifact && isPersistedAsTable(otherArtifact);
+
+    // to make it easier to know which views to drop-create
+    if (isPersistedAsView(artifact) && isPersistedAsView(otherArtifact)) {
+      // TODO: Check only on artifact.query/projection BUT: Need to manually check for sql-snippets then!
+      artifact[isChanged] = JSON.stringify(artifact) !== JSON.stringify(otherArtifact);
+    }
+
+    // Looking for added entities and added/deleted/changed elements.
+    // Parameters: `artifact` from afterModel and `otherArtifact` from beforeModel.
+
+    if (!isPersisted) // Artifact not persisted in afterModel.
+      return;
+
+    if (!isPersistedOther) {
+      extensions[name] = artifact;
+      return;
+    }
+
+    // Artifact changed?
+
+    addElements();
+    changePropsOrRemoveOrChangeElements();
+    if (hasPrimaryKeyChange)
+      changedPrimaryKeys.push(name);
 
     function addElements() {
       const elements = {};
@@ -211,34 +244,6 @@ function getExtensionAndMigrations(beforeModel, options, {
       if (migration.properties || migration.remove || migration.change || migration.removeConstraints)
         migrations.push(migration);
     }
-
-    const otherArtifact = beforeModel.definitions[name];
-    const isPersisted = isPersistedAsTable(artifact);
-    const isPersistedOther = otherArtifact && isPersistedAsTable(otherArtifact);
-
-    // to make it easier to know which views to drop-create
-    if (isPersistedAsView(artifact) && isPersistedAsView(otherArtifact)) {
-      // TODO: Check only on artifact.query/projection BUT: Need to manually check for sql-snippets then!
-      artifact[isChanged] = JSON.stringify(artifact) !== JSON.stringify(otherArtifact);
-    }
-
-    // Looking for added entities and added/deleted/changed elements.
-    // Parameters: `artifact` from afterModel and `otherArtifact` from beforeModel.
-
-    if (!isPersisted) // Artifact not persisted in afterModel.
-      return;
-
-    if (!isPersistedOther) {
-      extensions[name] = artifact;
-      return;
-    }
-
-    // Artifact changed?
-
-    addElements();
-    changePropsOrRemoveOrChangeElements();
-    if (hasPrimaryKeyChange)
-      changedPrimaryKeys.push(name);
   };
 }
 /**
@@ -374,12 +379,6 @@ function docCommentChanged(element, otherElement) {
   return element.doc && !otherElement.doc || otherElement.doc && !element.doc || element.doc && element.doc !== otherElement.doc;
 }
 
-const relevantProperties = {
-  doc: true,
-  '@sql.prepend': true,
-  '@sql.append': true,
-};
-
 /**
  * Returns whether any type parameters differ between two given elements. Ignores whether types themselves differ (`type` property) and ignores
  * diff in doc comments.

package/lib/optionProcessor.js

@@ -155,7 +155,8 @@ optionProcessor
        manageConstraints [options] <files...>   (internal) Generate ALTER TABLE statements to
                                                            add / modify referential constraints.
        inspect [options] <files...>             (internal) Inspect the given CDS files.
-       toEffectiveCsn [options] <files...>      (internal) Get an effective CSN for SEAL; requires beta mode
+       toEffectiveCsn [options] <files...>      (internal) Get an effective CSN; requires beta mode
+       forSeal [options] <files...>             (internal) Get a SEAL CSN
 
   Environment variables
     NO_COLOR                If set, compiler messages (/output) will not be colored.
@@ -546,6 +547,8 @@ optionProcessor.command('toEffectiveCsn')
   .option('-h, --help')
   .option('--resolve-simple-types <val>', { valid: ['true', 'false'] } )
   .option('--resolve-projections <val>', { valid: ['true', 'false'] } )
+  .option('--remap-odata-annotations <val>', { valid: ['true', 'false'] } )
+  .option('--keep-localized <val>', { valid: ['true', 'false'] } )
   .positionalArgument('<files...>')
   .help(`
   Usage: cdsc toEffectiveCsn [options] <files...>
@@ -560,8 +563,30 @@ optionProcessor.command('toEffectiveCsn')
                                           true: (default) resolve simple type references to their simple base type
                                           false:          do not resolve simple type references
        --resolve-projections <val>       Resolve projections:
-                                          true: (default) resolve projections to ordinary views with SELECT
+                                          true: (default) transform projections into ordinary views with SELECT
                                           false:          leave them as real projections
+       --remap-odata-annotations <val>   Remap OData annotations to ABAP annotations:
+                                          true:           remap annotations
+                                          false:(default) leave them as is
+       --keep-localized <val>            Keep '.localized' property in the CSN:
+                                          true:           property is kept
+                                          false:(default) property is deleted
+`);
+
+optionProcessor.command('forSeal')
+  .option('-h, --help')
+  .option('--remap-odata-annotations <val>', { valid: ['true', 'false'] } )
+  .positionalArgument('<files...>')
+  .help(`
+  Usage: cdsc forSeal [options] <files...>
+
+  (internal): Get the SEAL CSN model compiled from the provided CDS files.
+
+  Options
+   -h, --help                            Show this help text
+       --remap-odata-annotations <val>   Remap OData annotations to ABAP annotations:
+                                          true: (default)   remap annotations
+                                          false:            leave them as is
 `);
 
 module.exports = {