@sap/cds-compiler 6.7.1 → 6.7.3

package/CHANGELOG.md

@@ -13,6 +13,22 @@ 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.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
+
+- **compiler:** Just issue warning for `using` declaration referring to nothing
+  (fixes regression introduced with v6.7.0 if there is a file containing
+  a `namespace` declaration, but no definitions)
+- **effective:** clean up internal Symbols from meta section
+- **sql:** clean up internal Symbols from meta section
+
 ## Version 6.7.1 - 2026-01-28
 
 ### Fixed
@@ -20,7 +36,6 @@ might use a deprecated flag only for a limited period of time.
 - compiler: Properly accept aspects as composition targets in an `extend`
   (fixes regression introduced with v6.7.0)
 
-
 ## Version 6.7.0 - 2026-01-23
 
 ### Added

package/lib/base/message-registry.js

@@ -160,10 +160,12 @@ const centralMessages = {
   'type-ambiguous-target': { severity: 'Warning' },
 
   'ref-unexpected-autoexposed': { severity: 'Error' },
+  'ref-unexpected-many-expand': { severity: 'Error' },
   'ref-unexpected-many-navigation': { severity: 'Error' },
   // Published! Used in @sap/cds-lsp; if renamed, add to oldMessageIds and contact colleagues
   'ref-undefined-art':    { severity: 'Error' },
   'ref-undefined-def':    { severity: 'Error' },
+  'ref-undefined-using':  { severity: 'Warning' },
   'ref-undefined-var':    { severity: 'Error' },
   'ref-undefined-element': { severity: 'Error' },
   'anno-undefined-element': { severity: 'Error' },
@@ -738,6 +740,7 @@ const centralMessageTexts = {
     std: 'No artifact has been found with name $(ART)',
     localized: 'Can\'t extend localized definitions, only annotate them using an $(KEYWORD) statement',
   },
+  'ref-undefined-using': 'There is no definition in the model whose name starts with $(ART)',
   // TODO: proposal 'No definition found for $(NAME)',
   'ref-undefined-element': {
     std: 'Element $(ART) has not been found',
@@ -786,6 +789,10 @@ const centralMessageTexts = {
     std: '$(ART) can\'t be extended because it originates from an include',
     elements: '$(ART) can\'t be extended by elements/enums because it originates from an include',
   },
+  'ref-unexpected-many-expand': {
+    std: 'Unexpected expand with to-many association',
+    composition: 'Unexpected expand with to-many composition',
+  },
   'ref-unexpected-many-navigation': {
     std: 'Unexpected navigation into arrayed structure',
   },

package/lib/checks/manyExpand.js

@@ -0,0 +1,28 @@
+'use strict';
+
+const { applyTransformationsOnNonDictionary } = require('../model/csnUtils');
+
+/**
+ * Check for nested projections (expand) on to-many associations or compositions.
+ * Nested projections are only valid for to-one relationships.
+ *
+ * @param {object} parent Object with the expression as a property
+ * @param {string} propOnParent Name of the expression property on parent
+ * @param {Array} _e Expression to check (unused)
+ * @param {CSN.Path} path
+ */
+function expandToMany( parent, propOnParent, _e, path ) {
+  applyTransformationsOnNonDictionary(parent, propOnParent, {
+    expand: (_parent, _prop, _expand, _path) => {
+      const last = _parent._links?.at(-1).art;
+      if (last?.cardinality && last.cardinality.max !== 1) {
+        this.message('ref-unexpected-many-expand', _path,
+                     { '#': last.type === 'cds.Composition' ? 'composition' : 'std' });
+      }
+    },
+  }, null, path);
+}
+
+module.exports = {
+  columns: expandToMany,
+};

package/lib/checks/validator.js

@@ -12,6 +12,7 @@ const { validateSelectItems } = require('./selectItems');
 const { rejectParamDefaultsInHanaCds, warnAboutDefaultOnAssociationForHanaCds } = require('./defaultValues');
 const validateCdsPersistenceAnnotation = require('./cdsPersistence');
 const navigationIntoMany = require('./manyNavigations');
+const expandToMany = require('./manyExpand');
 const checkUsedTypesForAnonymousAspectComposition = require('./managedInType');
 const validateHasPersistedElements = require('./hasPersistedElements');
 const checkForHanaTypes = require('./checkForTypes');
@@ -91,6 +92,7 @@ const forRelationalDBCsnValidators = [
   existsMustNotStartWithDollarSelf,
   assertFilterOfExists,
   navigationIntoMany,
+  expandToMany,
   checkPathsInStoredCalcElement,
   featureFlags,
 ];

package/lib/compiler/shared.js

@@ -43,11 +43,15 @@ function fns( model ) {
   // Map `exprCtx` (is a param of traversal functions) to reference semantics
   const referenceSemantics = {
     // global: ------------------------------------------------------------------
-    using: {                    // only used to produce error message
+    using: {                    // only used to produce warnings
       isMainRef: 'no-leaf-gap',
       lexical: null,
       dynamic: modelDefinitions,
       notFound: undefinedDefinition,
+      messageMap: {
+        'ref-undefined-art': 'ref-undefined-using',
+        'ref-undefined-def': 'ref-undefined-using',
+      },
     },
     // scope:'global': for cds.Association and auto-redirected targets
     $global: {

package/lib/edm/edmAnnoPreprocessor.js

@@ -49,7 +49,7 @@ function addToSetAttr( carrier, propName, propValue, removeFromType = true ) {
 
 function applyAppSpecificLateCsnTransformationOnElement( options, element, struct, error ) {
   if (options.isV2() && struct['@Aggregation.ApplySupported.PropertyRestrictions'])
-    mapAnnotationAssignment(element, struct, AnalyticalAnnotations());
+    mapAnnotationAssignment(element, struct, GenerateAnalyticalAnnotations());
 
 
   // etag requires Core.OptimisticConcurrency to be set in V4 (cap/issues#2641)
@@ -66,7 +66,7 @@ function applyAppSpecificLateCsnTransformationOnElement( options, element, struc
                               (struct['@Core.OptimisticConcurrency'] || [])/* .push(element.name) */);
   }
 
-  function AnalyticalAnnotations() {
+  function GenerateAnalyticalAnnotations() {
     function mapCommonAttributes( elt, structure, prop ) {
       const CommonAttributes = elt[prop];
       if (!Array.isArray(CommonAttributes)) {

package/lib/render/toSql.js

@@ -104,7 +104,7 @@ class SqlRenderEnvironment {
  *   }
  * }
  *
- * @param {CSN.Model} csn HANA transformed CSN
+ * @param {CSN.Model} csn for SQL transformed CSN
  * @param {CSN.Options} options Transformation options
  * @param {object} messageFunctions Message functions such as `error()`, `info()`, …
  * @returns {object} Dictionary of artifact-type:artifacts, where artifacts is a dictionary of name:content

package/lib/transform/db/expansion.js

@@ -2,14 +2,12 @@
 
 const {
   applyTransformations,
-  setDependencies,
   walkCsnPath,
   getUtils,
   forEachDefinition,
 } = require('../../model/csnUtils');
 const { implicitAs, columnAlias, pathId } = require('../../model/csnRefs');
 const { setProp } = require('../../base/model');
-const { forEach } = require('../../utils/objectUtils');
 const { killNonrequiredAnno } = require('./killAnnotations');
 const { featureFlags } = require('../featureFlags');
 const { applyTransformationsOnNonDictionary } = require('./applyTransformations');
@@ -128,15 +126,11 @@ function expandStructureReferences( csn, options, pathDelimiter, messageFunction
    * For such skipped things, error for usage of assoc pointing to them and ignore publishing of assoc pointing to them.
    */
   function rewriteExpandInline() {
-    let cleanup = [];
-    let _dependents;
-
     const entity = findAnEntity();
     const toDummify = [];
 
     applyTransformations(csn, {
-      columns: (parent, name, columns, path) => {
-        const artifact = csn.definitions[path[1]];
+      columns: (parent) => {
         // get$combined expects a SET/SELECT - so we wrap the parent
         // (which is the thing inside SET/SELECT)
         // We can directly use SELECT here, as only projections and SELECT can have .columns
@@ -146,18 +140,7 @@ function expandStructureReferences( csn, options, pathDelimiter, messageFunction
           root[key] = root[key][0].element;
         });
         const rewritten = rewrite(root, parent.columns, parent.excluding);
-        /*
-         * Do not remove unexpandable many columns in OData
-         */
-        if (rewritten.toMany.length > 0 && !options.toOdata) {
-          markAsToDummify(artifact, path[1]);
-          rewritten.toMany.forEach(({ art }) => {
-            error( null, art.$path || [ 'definitions', path[1] ], { name: `${ art.$env || path[1] }:${ art.ref.map(r => r.id || r) }` }, 'Unexpected .expand with to-many association $(NAME)');
-          });
-        }
-        else {
-          parent.columns = rewritten.columns;
-        }
+        parent.columns = rewritten.columns;
       },
     });
 
@@ -166,8 +149,6 @@ function expandStructureReferences( csn, options, pathDelimiter, messageFunction
     if (!options.toOdata)
       dummyfy();
 
-    cleanup.forEach(fn => fn());
-
     csnUtils = getUtils(csn);
 
     const publishing = [];
@@ -267,29 +248,6 @@ function expandStructureReferences( csn, options, pathDelimiter, messageFunction
       info(null, path, { name: last, target }, 'Ignoring association $(NAME) with target $(TARGET), because it was skipped because of .expand in conjunction with to-many');
     }
 
-    /**
-     * Mark the given artifact and all (transitively) dependent artifacts as `toDummify`.
-     * This means that they will be replaced with simple dummy views in `@dummify`
-     *
-     * @param {CSN.Artifact} artifact
-     * @param {string} name
-     */
-    function markAsToDummify( artifact, name ) {
-      if (!_dependents && cleanup.length === 0)
-        ({ cleanup, _dependents } = setDependencies(csn, csnUtils));
-
-      const stack = [ [ artifact, name ] ];
-      while (stack.length > 0) {
-        const [ a, n ] = stack.pop();
-        if (a[_dependents]) {
-          forEach(a[_dependents], (dependentName, dependent) => {
-            stack.push([ dependent, dependentName ]);
-          });
-        }
-        toDummify.push(n);
-      }
-    }
-
     /**
      * Replace the artifacts in `toDummify` with simple dummy views as produced by createDummyView.
      */

package/lib/transform/effective/main.js

@@ -17,7 +17,7 @@ const misc = require('./misc');
 const annotations = require('./annotations');
 const { rewriteCalculatedElementsInViews, processCalculatedElementsInEntities } = require('../db/rewriteCalculatedElements');
 const { cloneFullCsn } = require('../../model/cloneCsn');
-const { featureFlags } = require('../featureFlags');
+const { featureFlags, removeFeatureFlags } = require('../featureFlags');
 const getServiceFilterFunction = require('./service');
 const { traverseQuery } = require('../../model/csnRefs');
 const { expandWildcard } = require('../db/expansion');
@@ -136,6 +136,7 @@ function effectiveCsn( model, options, messageFunctions ) {
   }
 
   messageFunctions.throwWithError();
+  removeFeatureFlags(csn);
 
   return csn;
 }

package/lib/transform/featureFlags.js

@@ -2,4 +2,9 @@
 
 const featureFlags = Symbol.for('Feature flags');
 
-module.exports = { featureFlags };
+function removeFeatureFlags(csn) {
+  if (csn.meta)
+    delete csn.meta[featureFlags];
+}
+
+module.exports = { featureFlags, removeFeatureFlags };

package/lib/transform/forRelationalDB.js

@@ -34,7 +34,7 @@ const temporal = require('./db/temporal');
 const associations = require('./db/associations');
 const backlinks = require('./db/backlinks');
 const { getDefaultTypeLengths } = require('../render/utils/common');
-const { featureFlags } = require('./featureFlags');
+const { featureFlags, removeFeatureFlags } = require('./featureFlags');
 const { cloneCsnNonDict, cloneFullCsn } = require('../model/cloneCsn');
 const { processSqlServices, createServiceDummy } = require('./db/processSqlServices');
 const { expandWildcard } = require('./db/expansion');
@@ -465,6 +465,7 @@ function transformForRelationalDBWithCsn(csn, options, messageFunctions) {
 
   killTypes.forEach(fn => fn());
   redoProjections.forEach(fn => fn());
+  removeFeatureFlags(csn);
 
   timetrace.stop('Transform CSN');
   timetrace.stop('HANA transformation');

package/lib/transform/translateAssocsToJoins.js

@@ -331,10 +331,16 @@ function translateAssocsToJoins(model, inputOptions = {}) {
         pos++;
       // QA + tail is the rewritten path
       tail = tail.slice(pos);
+
+      // leave virtual paths as is
+      if ( tail.at(-1)._artifact.virtual?.val ) {
+        replaceNodeContent(pathNode, constructPathNode([ constructTableAliasPathStep(QA), ...tail ]));
+        return;
+      }
+
       // check from left to right (longest match) if a subsequent QAT is $njr
       // if so, substitute path with pregenerated foreign key, prepend by optional
       // (to be flattened) prefix
-
       for (let i = 0; i < tail.length - 1; i++) {
         if (tail[i]._navigation) {
           // the correct flattened foreign key must match the leaf artifact and access path prefix of this path
@@ -345,36 +351,30 @@ function translateAssocsToJoins(model, inputOptions = {}) {
             throw new CompilerAssertion('Debug me: No FK found for FK rewriting');
         }
       }
-      // leave virtual paths as is
-      if ( tail.at(-1)._artifact.virtual?.val === true ) {
-        replaceNodeContent(pathNode, constructPathNode([ constructTableAliasPathStep(QA), ...tail ]));
-      }
-      else {
-        const newTail = [];
-        // construct the tail (segment after last join relevant navigation) and preserve association steps
-        // each association may have a structured prefix
-        // `struct.foo.bar.assoc. otherStruct.baz.otherAssoc.fk`
-        // --> `[otherStruct_baz_otherAssoc, fk]
-        for (let i = 0; i < tail.length; i++) {
-          if (tail[i]._navigation) {
-            const nextNavigation = tail.slice(i + 1).findIndex(ps => ps._navigation);
-            if (nextNavigation >= 0) {
-              newTail.push({
-                id: tail.slice(i, i + 1 + nextNavigation).map(ps => ps.id).join(pathDelimiter),
-                _artifact: tail[i + nextNavigation]._artifact,
-              });
-            }
-            else {
-              newTail.push({
-                id: tail.slice(i).map(ps => ps.id).join(pathDelimiter),
-                _artifact: tail.at(-1)._artifact,
-              });
-            }
+      const newTail = [];
+      // construct the tail (segment after last join relevant navigation) and preserve association steps
+      // each association may have a structured prefix
+      // `struct.foo.bar.assoc. otherStruct.baz.otherAssoc.fk`
+      // --> `[otherStruct_baz_otherAssoc, fk]
+      for (let i = 0; i < tail.length; i++) {
+        if (tail[i]._navigation) {
+          const nextNavigation = tail.slice(i + 1).findIndex(ps => ps._navigation);
+          if (nextNavigation >= 0) {
+            newTail.push({
+              id: tail.slice(i, i + 1 + nextNavigation).map(ps => ps.id).join(pathDelimiter),
+              _artifact: tail[i + nextNavigation]._artifact,
+            });
+          }
+          else {
+            newTail.push({
+              id: tail.slice(i).map(ps => ps.id).join(pathDelimiter),
+              _artifact: tail.at(-1)._artifact,
+            });
           }
         }
-        replaceNodeContent(pathNode,
-                           constructPathNode([ constructTableAliasPathStep(QA), ...newTail ]));
       }
+      replaceNodeContent(pathNode,
+                         constructPathNode([ constructTableAliasPathStep(QA), ...newTail ]));
     }
 
     function findForeignKey(assoc, fk) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds-compiler",
-  "version": "6.7.1",
+  "version": "6.7.3",
   "description": "CDS (Core Data Services) compiler and backends",
   "homepage": "https://cap.cloud.sap/",
   "author": "SAP SE (https://www.sap.com)",