@sap/cds-compiler 6.7.2 → 6.8.0

package/CHANGELOG.md

@@ -13,6 +13,40 @@ we might not list every change in its behavior here.
 Productive code should never require a `beta` flag to be set, and
 might use a deprecated flag only for a limited period of time.
 
+## Version 6.8.0 - 2026-03-05
+
+### Features
+
+- **api:** basic experimental support for code completion in `cds repl`
+- **compiler:**
+  + retain original meta property in CSN
+  + add compilerCsnFlavor to meta property
+- **sql:**
+  + Add support for `cds.Vector`
+  + Allow migration from cds.String to cds.LargeString
+  + Fiori Tree Views in the database backend
+  + For postgres: warning for identifiers that exceed the databases length limit
+
+### Bug Fixes
+
+- **sql:**
+  + do not report validation errors on entities marked as `@cds.persistence.skip`
+  + normalize named arguments for portable functions
+  + properly flatten associations for temporal unique constraints
+
+### Improvements
+
+- **compiler:**
+  + improve warning in case of invalid type for key
+  + simplify syntax error text when expecting Comparison and Operator tokens
+  + improve support for code completion with `case` expressions, queries, and subqueries
+
+## Version 6.7.3 - 2026-02-11
+
+### Fixed
+
+- **sql:** do not resolve path navigations to virtual elements which resulted in an internal error.
+
 ## Version 6.7.2 - 2026-02-04
 
 ### Fixed

package/bin/cdsc.js

@@ -25,9 +25,9 @@ const { toRename: _toRename } = require('../lib/render/toRename');
 const util = require('util');
 const fs = require('fs');
 const path = require('path');
-const { reveal } = require('../lib/model/revealInternalProperties');
-const enrichCsn = require('../lib/model/enrichCsn');
-const { optionProcessor } = require('../lib/optionProcessor');
+const { reveal } = require('../lib/tool-lib/revealInternalProperties');
+const enrichCsn = require('../lib/tool-lib/enrichCsn');
+const { optionProcessor } = require('../lib/base/optionProcessor');
 const {
   explainMessage, hasMessageExplanation, sortMessages,
   messageIdsWithExplanation, makeMessageFunction,
@@ -35,7 +35,7 @@ const {
 const { term } = require('../lib/utils/term');
 const { addLocalizationViews } = require('../lib/transform/localized');
 const { addTenantFields } = require('../lib/transform/addTenantFields');
-const { availableBetaFlags } = require('../lib/base/model');
+const { availableBetaFlags } = require('../lib/base/specialOptions');
 const { alterConstraintsWithCsn } = require('../lib/render/manageConstraints');
 const { tmpFilePath, readStream } = require('../lib/utils/file');
 
@@ -548,7 +548,7 @@ async function executeCommandLine( command, options, args ) {
   }
 
   function inspect( model ) {
-    const inspectModel = require('../lib/inspect');
+    const inspectModel = require('../lib/tool-lib');
 
     if (options.statistics) {
       const result = inspectModel.inspectModelStatistics(model, options);

package/bin/cdsse.js

@@ -25,7 +25,7 @@ const path = require('path');
 const compiler = require('../lib/compiler');
 const main = require('../lib/main');
 const { locationString } = require('../lib/base/messages');
-const { availableBetaFlags: beta } = require('../lib/base/model');
+const { availableBetaFlags: beta } = require('../lib/base/specialOptions');
 
 const { argv } = process;
 const cmd = commands[argv[2]];

package/lib/api/main.js

@@ -2,10 +2,10 @@
 
 'use strict';
 
-const lazyload = require('../base/lazyload')( module );
+const lazyload = require('../utils/lazyload')( module );
 
 const prepareOptions = lazyload('./options');
-const baseModel = lazyload('../base/model');
+const specialOptions = lazyload('../base/specialOptions');
 const location = lazyload('../base/location');
 const messages = lazyload('../base/messages');
 const compiler = lazyload('../compiler/index');
@@ -27,8 +27,8 @@ const effective = lazyload('../transform/effective/main');
 const toHdbcds = lazyload('../render/toHdbcds');
 const baseError = lazyload('../base/error');
 const csnToEdm = lazyload('../edm/csn2edm');
-const trace = lazyload('./trace');
-const cloneCsn = lazyload('../model/cloneCsn');
+const trace = lazyload('../base/trace');
+const cloneCsn = lazyload('../base/cloneCsn');
 const objectUtils = lazyload('../utils/objectUtils');
 
 /**
@@ -56,6 +56,7 @@ const warnAboutMismatchOdata = [ 'odataVersion' ];
  * @param {string[]} relevantOptionNames Option names that are defining characteristics
  * @param {string[]} [optionalOptionNames] Option names that should be attached as a fyi
  */
+// TODO: move this function to some other place
 function attachTransformerCharacteristics( csn, transformation, options,
                                            relevantOptionNames, optionalOptionNames = [] ) {
   const relevant = {};
@@ -75,10 +76,10 @@ function attachTransformerCharacteristics( csn, transformation, options,
       relevant[name] = options[name];
   }
   if (!csn.meta)
-    baseModel.setProp(csn, 'meta', {});
+    objectUtils.setProp(csn, 'meta', {});
 
-  baseModel.setProp(csn.meta, 'options', relevant);
-  baseModel.setProp(csn.meta, 'transformation', transformation);
+  objectUtils.setProp(csn.meta, 'options', relevant);
+  objectUtils.setProp(csn.meta, 'transformation', transformation);
 }
 
 /**
@@ -1166,7 +1167,7 @@ function publishCsnProcessor( processor, _name ) {
     try {
       const messageFunctions = messages.makeMessageFunction(csn, options, _name);
       if (options.deprecated)
-        baseModel.checkRemovedDeprecatedFlags( options, messageFunctions );
+        specialOptions.checkRemovedDeprecatedFlags( options, messageFunctions );
 
       checkOutdatedOptions( options, messageFunctions );
       csn = ensureClientCsn( csn, options, messageFunctions, _name );

package/lib/api/options.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// The options are specified in ../optionProcessor.js (and some other files).
+// The options are specified in ../base/optionProcessor.js (and some other files).
 // Some backends feel the need to "translate" option, which require to list
 // all options also here (as “public" or "private" option).
 
@@ -36,6 +36,7 @@ const publicOptionsNewAPI = [
   'pre2134ReferentialConstraintNames',
   'betterSqliteSessionVariables',
   'fewerLocalizedViews',
+  'sqliteRealAffinityForDecimal',
   'withHanaAssociations',
   'standardDatabaseFunctions',
   'booleanEquality',

package/lib/api/validate.js

@@ -1,6 +1,6 @@
 'use strict';
 
-// The options are specified in ../optionProcessor.js (and some other files).
+// The options are specified in ../base/optionProcessor.js (and some other files).
 // Options with non-boolean values must also be listed in this file.
 
 const { forEach } = require('../utils/objectUtils');

package/lib/base/error.js

@@ -1,3 +1,5 @@
+// Special error classes for compiler
+
 'use strict';
 
 /**

package/lib/base/message-registry.js

@@ -36,7 +36,7 @@
 'use strict';
 
 const { CompilerAssertion } = require( './error' );
-const { isDeprecatedEnabled } = require( './model' );
+const { isDeprecatedEnabled } = require( './specialOptions' );
 
 const configurableForValidValues = {
   __proto__: null,
@@ -135,6 +135,7 @@ const centralMessages = {
   'type-ignoring-argument': { severity: 'Error', configurableFor: true },
   'type-expected-builtin': { severity: 'Error', configurableFor: true },
   'type-expecting-service-target': { severity: 'Error', configurableFor: true },
+  'type-invalid-for-key': { severity: 'Warning' },
   'ref-expecting-const': { severity: 'Error' },
   'ref-expecting-foreign-key': { severity: 'Error' },
   'ref-invalid-source': { severity: 'Error' },
@@ -154,6 +155,7 @@ const centralMessages = {
   'query-unexpected-structure-hdbcds': { severity: 'Error' },
   'query-ignoring-param-nullability': { severity: 'Info' },
   'query-expected-identifier': { severity: 'Error' },
+  'query-invalid-identifier': { severity: 'Warning' },
 
   'recalculated-localized': { severity: 'Info' }, // KEEP: Downgrade in lib/transform/translateAssocsToJoins.js
   'redirected-implicitly-ambiguous': { severity: 'Error', configurableFor: true }, // does not hurt us - TODO: ref-ambiguous-target
@@ -728,6 +730,7 @@ const centralMessageTexts = {
   'ref-undefined-def': {
     std: 'Artifact $(ART) has not been found',
     // TODO: proposal 'No definition of $(NAME) found',
+    hint: 'Artifact $(ART) has not been found. Did you mean to write $(CODE)?',
     element: 'Artifact $(ART) has no element $(MEMBER)',
   },
   'ref-undefined-param': 'Entity $(ART) has no parameter $(ID)',
@@ -866,6 +869,7 @@ const centralMessageTexts = {
     std: 'Unexpected arguments in path $(ELEMREF)', // unused
     'on-condition': 'ON-conditions must not contain parameters, step $(ID) of path $(ELEMREF)',
     calc: 'Unexpected arguments in path $(ELEMREF) of stored calculated element; only simple paths can be used here',
+    vector_length: 'Unexpected length of vector $(ELEMREF). The length must be divisble by 4',
   },
   'ref-unsupported-type': {
     std: 'Type $(TYPE) is not supported',
@@ -955,6 +959,7 @@ const centralMessageTexts = {
     missing: 'Expecting service entity $(TARGET); it does not have the key element $(ID) of the provided model target',
     order: 'Expecting service entity $(TARGET); its key elements are in a different order than those of the provided model target',
   },
+  'type-invalid-for-key': 'Type $(TYPE) is not supported for key elements',
 
   'anno-builtin': 'Builtin types should not be annotated nor extended. Use custom type instead',
   'ext-undefined-def': 'Artifact $(ART) has not been found',
@@ -1025,7 +1030,6 @@ const centralMessageTexts = {
     std: '$(KEYWORD) is not supported here', // unused variant
     kind: '$(KEYWORD) is only supported for elements in an entity or an aspect',
     sub: '$(KEYWORD) is only supported for top-level elements',
-    type: '$(KEYWORD) is not supported for elements of type $(TYPE)',
   },
   'def-unexpected-localized': {
     std: 'Unexpected $(KEYWORD)',
@@ -1199,6 +1203,7 @@ const centralMessageTexts = {
     std: 'Expected identifier for select item',
     assoc: 'Expected identifier as the association\'s name',
   },
+  'query-invalid-identifier': 'Expected identifier for SQL dialect $(NAME) to not exceed $(NUMBER) characters',
   'query-unsupported-calc': {
     std: 'Using nested projections next to calculated elements is not supported, yet',
     inside: 'Using calculated elements in nested projections is not supported, yet',

package/lib/base/messages.js

@@ -6,7 +6,7 @@
 
 const { term } = require('../utils/term');
 const { Location, locationString } = require('./location');
-const { isBetaEnabled } = require('./model');
+const { isBetaEnabled } = require('./specialOptions');
 const {
   centralMessages,
   configurableForValidValues,
@@ -14,7 +14,7 @@ const {
   oldMessageIds,
 } = require('./message-registry');
 const _messageIdsWithExplanation = require('../../share/messages/message-explanations.json').messages;
-const { analyseCsnPath, traverseQuery } = require('../model/csnRefs');
+const { analyseCsnPath, traverseQuery } = require('../base/csnRefs');
 const { CompilerAssertion } = require('./error');
 const { getArtifactName } = require('../compiler/base');
 const { cdlNewLineRegEx } = require('../language/textUtils');

package/lib/{optionProcessor.js → base/optionProcessor.js}

@@ -8,12 +8,12 @@
 // - Also list the option in the `help` text, used with `cdsc -h`.
 // - All options must also be added to ./api/options.js.
 // - Specify valid values for non-boolean options in ./api/validate.js.
-// - Beta and deprecated options are specified in ./base/model.js.
+// - Beta and deprecated options are specified in ./base/specialOptions.js.
 
 'use strict';
 
-const { createOptionProcessor } = require('./base/optionProcessorHelper');
-const { availableBetaFlags } = require('./base/model');
+const { createOptionProcessor } = require('../utils/optionProcessorHelper');
+const { availableBetaFlags } = require('./specialOptions');
 
 // This option processor is used both by the command line parser (to translate cmd line options
 // into an options object) and by the API functions (to verify options)

package/lib/base/{model.js → specialOptions.js}

@@ -1,15 +1,10 @@
-// module- and csn/XSN-independent definitions
+// Definitions for beta and deprecated options
 
-// TODO: move XSN-specific things to lib/compiler/utils/
-// TODO: move CSN-specific things to ???
+// Normal options are in ../model/optionProcessor.js (and some other files),
+// unfortunately partly non-grep-able (option `fooBar` is defined via `--foo-bar`)
 
 'use strict';
 
-const { forEach } = require('../utils/objectUtils');
-
-// Normal options are in ../optionProcessor.js (and some other files),
-// unfortunately partly non-grep-able (option `fooBar` is defined via `--foo-bar`)
-
 /**
  * Object of all available beta flags that will be enabled/disabled by `--beta-mode`
  * through cdsc.  Only intended for INTERNAL USE.
@@ -47,12 +42,12 @@ const availableDeprecatedFlags = {
   ignoreSpecifiedQueryElements: true,
 };
 
-// Deprecated flags that were removed in v5.
-const oldDeprecatedFlagsV5 = [
-  'includesNonShadowedFirst',
-  'eagerPersistenceForGeneratedEntities',
-  'noKeyPropagationWithExpansions',
-];
+// Deprecated flags that were removed in newer version and are complained about.
+const oldDeprecatedFlags = {
+  includesNonShadowedFirst: true,
+  eagerPersistenceForGeneratedEntities: true,
+  noKeyPropagationWithExpansions: true,
+};
 
 /**
  * Test for early-adaptor feature, stored in option `beta`(new-style) / `betaMode`(old-style)
@@ -98,48 +93,30 @@ function isDeprecatedEnabled( options, feature = null ) {
 }
 
 /**
- * In cds-compiler v3, we removed old v2 deprecated flags.  That can lead to silent
+ * In the current cds-compiler, we might have removed old deprecated flags.  That can lead to silent
  * errors such as entity/view names changing.  To ensure that the user is forced
  * to change their code, emit an error if one of such removed flags was used.
  *
  * @param {CSN.Options} options
  * @param error Error message function returned by makeMessageFunction().
  */
-function checkRemovedDeprecatedFlags( options, { error } ) {
+function checkRemovedDeprecatedFlags( { deprecated, messages }, { error } ) {
   // Assume that we emitted these errors once if a message with this ID was found.
-  if (!options.deprecated || options.messages?.some(m => m.messageId === 'api-invalid-deprecated'))
+  if (!deprecated || messages?.some( m => m.messageId === 'api-invalid-deprecated' ))
     return;
 
-  forEach(options.deprecated, (key, val) => {
-    if (val && oldDeprecatedFlagsV5.includes(key)) {
-      error('api-invalid-deprecated', null, { name: key },
-            'Deprecated flag $(NAME) has been removed in CDS compiler v6');
+  Object.keys( deprecated ).forEach( ( key ) => {
+    if (deprecated[key] && oldDeprecatedFlags[key]) { // why testing the value of the option?
+      error( 'api-invalid-deprecated', null, { name: key },
+             'Deprecated flag $(NAME) has been removed in CDS compiler v6' );
     }
   });
 }
 
-/**
- * Like `obj.prop = value`, but not contained in JSON / CSN
- * It's important to set enumerable explicitly to false (although 'false' is the default),
- * as else, if the property already exists, it keeps the old setting for enumerable.
- *
- * @param {object} obj
- * @param {string} prop
- * @param {any} value
- */
-function setProp( obj, prop, value ) {
-  const descriptor = {
-    value, configurable: true, writable: true, enumerable: false,
-  };
-  Object.defineProperty( obj, prop, descriptor );
-  return value;
-}
-
 
 module.exports = {
   isBetaEnabled,
   availableBetaFlags,
   isDeprecatedEnabled,
   checkRemovedDeprecatedFlags,
-  setProp,
 };

package/lib/checks/arrayOfs.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { setProp } = require('../base/model');
+const { setProp } = require('../utils/objectUtils');
 
 // Only to be used with validator.js - a correct `this` value needs to be provided!
 

package/lib/checks/elements.js

@@ -8,7 +8,7 @@ const {
 } = require('../model/csnUtils');
 const { isBuiltinType } = require('../base/builtins');
 const { isGeoTypeName } = require('../compiler/builtins');
-const { setProp } = require('../base/model');
+const { setProp } = require('../utils/objectUtils');
 // Only to be used with validator.js - a correct `this` value needs to be provided!
 
 /**

package/lib/checks/enricher.js

@@ -4,8 +4,8 @@
 
 'use strict';
 
-const { csnRefs } = require('../model/csnRefs');
-const { setProp } = require('../base/model');
+const { csnRefs } = require('../base/csnRefs');
+const { setProp } = require('../utils/objectUtils');
 const { isAnnotationExpression } = require('../base/builtins');
 
 /**

package/lib/checks/featureFlags.js

@@ -1,39 +1,69 @@
 'use strict';
 
-const { setProp } = require('../base/model');
+const { setProp, forEachValue } = require('../utils/objectUtils');
 const { featureFlags } = require('../transform/featureFlags');
 const { isSqlService, isDummyService, isDataProductProductionService } = require('../transform/db/processSqlServices');
 
 /**
+ * Set a feature flag on the CSN model.
  *
- * @param {string} flag
+ * @param {CSN.Model} csn CSN model to set flag on
+ * @param {string} flag Flag name to set
+ */
+function setFeatureFlag( csn, flag ) {
+  if (!csn.meta)
+    setProp(csn, 'meta', {});
+  if (!csn.meta[featureFlags])
+    csn.meta[featureFlags] = {};
+
+  csn.meta[featureFlags][flag] = true;
+}
+
+/**
+ * Get a function that detects and sets feature flags for expand/inline columns in a query.
  *
- * @returns {Function} Function to correctly set the given flag
+ * @param {CSN.Model} csn CSN model to set flags on
+ * @returns {Function} Function to call per query
  */
-function setFeatureFlag( flag ) {
-  return function setFlag() {
-    if (!this.csn.meta)
-      setProp(this.csn, 'meta', {});
-    if (!this.csn.meta[featureFlags])
-      this.csn.meta[featureFlags] = {};
+function getQueryFeatureFlagSetter( csn ) {
+  return function setQueryFeatureFlags(query) {
+    const queryProp = query.SELECT ? 'SELECT' : 'projection';
+    query[queryProp]?.columns?.forEach((column) => {
+      if (column.expand || column.inline)
+        setFeatureFlag(csn, '$expandInline');
+    });
+  };
+}
 
-    this.csn.meta[featureFlags][flag] = true;
+/**
+ * Get a function that detects and sets feature flags for entity/service definitions.
+ * That are calculated elements and SQL services.
+ *
+ * @param {CSN.Model} csn CSN model to set flags on
+ * @param {CSN.Options} options Compiler options
+ * @returns {Function} Function to call per definition
+ */
+function getDefinitionFeatureFlagSetter( csn, options ) {
+  return function setDefinitionFeatureFlags(def) {
+    if (def.kind === 'entity' && def.elements) {
+      forEachValue(def.elements, (element) => {
+        if (element.value)
+          setFeatureFlag(csn, '$calculatedElements');
+      });
+    }
+    else if (def.kind === 'service') {
+      if (isSqlService(def))
+        setFeatureFlag(csn, '$sqlService');
+      if (isDummyService(def, options))
+        setFeatureFlag(csn, '$dummyService');
+      if (isDataProductProductionService(def))
+        setFeatureFlag(csn, '$dataProductService');
+    }
   };
 }
 
-// Export a applyTransformations callback object that sets the feature flags if certain properties are present
+// Export factory functions that create bound feature flag setters for use in transformation loops
 module.exports = {
-  value: setFeatureFlag('$calculatedElements'),
-  expand: setFeatureFlag('$expandInline'),
-  inline: setFeatureFlag('$expandInline'),
-  kind: function setFeatureFlagForSqlService( artifact ) {
-    if (isSqlService(artifact))
-      setFeatureFlag( '$sqlService' ).call(this);
-
-    if (isDummyService(artifact, this.options))
-      setFeatureFlag( '$dummyService' ).call(this);
-
-    if (isDataProductProductionService(artifact, this.options))
-      setFeatureFlag( '$dataProductService' ).call(this);
-  },
+  getQueryFeatureFlagSetter,
+  getDefinitionFeatureFlagSetter,
 };

package/lib/checks/foreignKeys.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { setProp } = require('../base/model');
+const { setProp } = require('../utils/objectUtils');
 
 // Only to be used with validator.js - a correct this value needs to be provided!
 

package/lib/checks/invalidTarget.js

@@ -3,7 +3,7 @@
 // Only to be used with validator.js - a correct this value needs to be provided!
 
 const { ModelError } = require('../base/error');
-const { setProp } = require('../base/model');
+const { setProp } = require('../utils/objectUtils');
 
 /**
  * Assert that targets of associations and compositions are entities.

package/lib/checks/managedInType.js

@@ -1,6 +1,6 @@
 'use strict';
 
-const { setProp } = require('../base/model');
+const { setProp } = require('../utils/objectUtils');
 
 // Only to be used with validator.js - a correct this value needs to be provided!
 

package/lib/checks/onConditions.js

@@ -2,7 +2,7 @@
 
 const { forEachGeneric } = require('../model/csnUtils');
 const { otherSideIsExpandableStructure, resolveArtifactType } = require('./utils');
-const { pathId } = require('../model/csnRefs');
+const { pathId } = require('../base/csnRefs');
 
 // Only to be used with validator.js - a correct this value needs to be provided!
 

package/lib/checks/queryNoDbArtifacts.js

@@ -3,7 +3,7 @@
 const { isPersistedOnDatabase, applyTransformationsOnNonDictionary } = require('../model/csnUtils');
 const { isBuiltinType } = require('../base/builtins');
 const { requireForeignKeyAccess } = require('./onConditions');
-const { pathId } = require('../model/csnRefs');
+const { pathId } = require('../base/csnRefs');
 
 const generalQueryProperties = [ 'from', 'columns', 'where', 'groupBy', 'orderBy', 'having', 'limit' ];
 

package/lib/checks/validator.js

@@ -18,7 +18,6 @@ const validateHasPersistedElements = require('./hasPersistedElements');
 const checkForHanaTypes = require('./checkForTypes');
 const { checkAnnotationExpression } = require('./structuredAnnoExpressions');
 const checkForParams = require('./parameters');
-const checkAndRemoveEnums = require('./enums');
 // forOdata
 const { validateDefaultValues } = require('./defaultValues');
 const { checkActionOrFunction } = require('./actionsFunctions');
@@ -54,7 +53,6 @@ const {
   checkSqlAnnotationOnElement,
 } = require('./sql-snippets');
 const assertNoAssocUsageOutsideOfService = require('./assocOutsideService');
-const featureFlags = require('./featureFlags');
 const { timetrace } = require('../utils/timetrace');
 
 const forRelationalDBMemberValidators
@@ -94,7 +92,6 @@ const forRelationalDBCsnValidators = [
   navigationIntoMany,
   expandToMany,
   checkPathsInStoredCalcElement,
-  featureFlags,
 ];
 /**
  * @type {Array<(query: CSN.Query, path: CSN.Path) => void>}
@@ -168,7 +165,7 @@ function _validate( csn, that,
   // TODO: Don't know if that's feasible? Do we really need to enrich annotations always?
   // const { cleanup } = enrich(csn, { processAnnotations: that.options.tranformation === 'odata' });
 
-  applyTransformations(csn, mergeTransformers(csnValidators, that), [], { drillRef: true });
+  applyTransformations(csn, mergeTransformers(csnValidators, that), [], { drillRef: true, ...iterateOptions });
 
   forEachDefinition(csn, (artifact, artifactName, prop, path) => {
     artifactValidators.forEach((artifactValidator) => {
@@ -201,10 +198,8 @@ function _validate( csn, that,
 function getDBCsnValidators( options ) {
   const validations = [ ...forRelationalDBCsnValidators ];
 
-  if (options.transformation !== 'effective') {
-    validations.push(checkAndRemoveEnums);
+  if (options.transformation !== 'effective')
     validations.push(checkForParams.csnValidator);
-  }
 
   if (options.sqlDialect === 'h2' || options.sqlDialect === 'postgres')
     validations.push(checkForHanaTypes);
@@ -221,7 +216,13 @@ function forRelationalDB( csn, that ) {
   const memberValidators = [ ...forRelationalDBMemberValidators, ...commonMemberValidators ];
   if (that.options.transformation === 'hdbcds')
     memberValidators.push(checkForParams.memberValidator);
-
+  // skip artifacts / elements which are not persisted on the database from being validated
+  const iterateOptions = {
+    skipArtifact: artifact => artifact.abstract ||
+      hasPersistenceSkipAnnotation(artifact) ||
+      artifact['@cds.persistence.exists'] ||
+      [ 'action', 'function', 'event' ].includes(artifact.kind),
+  };
   return _validate(csn, that,
                    getDBCsnValidators(that.options),
                    memberValidators,
@@ -246,12 +247,7 @@ function forRelationalDB( csn, that ) {
                      }
                    ),
                    forRelationalDBQueryValidators.concat(commonQueryValidators),
-                   {
-                     skipArtifact: artifact => artifact.abstract ||
-                                                hasPersistenceSkipAnnotation(artifact) ||
-                                                artifact['@cds.persistence.exists'] ||
-                                                [ 'action', 'function', 'event' ].includes(artifact.kind),
-                   });
+                   iterateOptions);
 }
 
 /**

package/lib/compiler/builtins.js

@@ -9,7 +9,7 @@
 
 const { builtinLocation } = require('../base/location');
 const { setLink } = require('./utils');
-const { isBetaEnabled } = require('../base/model');
+const { isBetaEnabled } = require('../base/specialOptions');
 
 // TODO: make type parameters a dict
 const core = {

package/lib/compiler/checks.js

@@ -9,7 +9,7 @@
 
 'use strict';
 
-const { isDeprecatedEnabled } = require('../base/model');
+const { isDeprecatedEnabled } = require('../base/specialOptions');
 const { propagationRules, acceptsExprValues } = require('../base/builtins');
 const { typeParameters } = require('./builtins');
 const {
@@ -132,8 +132,8 @@ function check( model ) {
         typeName === 'cds.Vector' ||
         typeName === 'cds.hana.CLOB' ||
         typeName === 'cds.LargeBinary') {
-      warning( 'def-unsupported-key', [ elem.type?.location || elem.location, elem ],
-               { '#': 'type', keyword: 'key', type: typeName } );
+      warning( 'type-invalid-for-key', [ elem.type?.location || elem.location, elem ],
+               { type: typeName } );
     }
   }