@sap/cds 9.8.0 → 9.8.2

package/CHANGELOG.md

@@ -4,6 +4,20 @@
 - The format is based on [Keep a Changelog](https://keepachangelog.com/).
 - This project adheres to [Semantic Versioning](https://semver.org/).
 
+## Version 9.8.2 - 2026-03-10
+
+### Fixed
+
+- Compatibility with `@eslint/js^10`
+
+## Version 9.8.1 - 2026-03-09
+
+### Fixed
+
+- OData batch parallel processing: Preserve request sequence for OData v2
+- In inbound messaging, only load the extended model if there is a tenant
+- Ensure `cds.fiori.direct_crud` is considered during `cds build`
+
 ## Version 9.8.0 - 2026-03-06
 
 ### Added

package/lib/compile/cds-compile.js

@@ -6,6 +6,7 @@ const compile = module.exports = Object.assign (cds_compile, {
   for: new class {
     get java(){ return super.java = require('./for/java') }
     get nodejs() { return super.nodejs = require('./for/nodejs') }
+    get direct_crud() { return super.direct_crud = require('./for/direct_crud') }
     get lean_drafts() { return super.lean_drafts = require('./for/lean_drafts') }
     get flows() { return super.flows = require('./for/flows') }
     get assert() { return super.assert = require('./for/assert') }

package/lib/compile/etc/properties.js

@@ -9,6 +9,7 @@ exports.read = function read (res, ext = '.properties', options) {
     const src = fs.readFileSync(path.resolve(res),'utf-8')
     return Object.defineProperty (exports.parse(src,options), '_source', {value:res})
   } catch (e) {
+    // eslint-disable-next-line preserve-caught-error
     if (e.code !== 'ENOENT') throw new Error (`Corrupt ${ext} file: ${res+ext}`)
   }
 }

package/lib/compile/for/direct_crud.js

@@ -0,0 +1,23 @@
+const $compiled_for_direct_crud = Symbol('compiled_for_direct_crud')
+const DRAFT_NEW = 'draftNew'
+
+module.exports = function cds_compile_for_direct_crud(csn) {
+  if (csn[$compiled_for_direct_crud]) return csn
+  csn[$compiled_for_direct_crud] = true
+
+  for (const each in csn.definitions) {
+    const def = csn.definitions[each]
+    if (!def['@Common.DraftRoot.NewAction'] && def['@odata.draft.enabled']) {
+      const srvName = Object.keys(csn.definitions)
+        .filter(k => csn.definitions[k].kind === 'service')
+        .find(k => each.startsWith(`${k}.`))
+      def['@Common.DraftRoot.NewAction'] = `${srvName}.${DRAFT_NEW}`
+      const params = { in: { items: { type: '$self' } } }
+      // for UI pop-up asking for values for non-UUID keys
+      Object.keys(def.elements)
+        .filter(k => k !== 'IsActiveEntity' && def.elements[k].key && def.elements[k].type !== 'cds.UUID')
+        .forEach(k => (params[k] = { type: def.elements[k].type }))
+      def.actions[DRAFT_NEW] = { kind: 'action', params, returns: { type: each } }
+    }
+  }
+}

package/lib/compile/for/odata.js

@@ -9,24 +9,7 @@ module.exports = function cds_compile_for_odata (csn,_o) {
   let dsn = compile.for.odata (csn,o)
   if (o.sql_mapping) dsn['@sql_mapping'] = o.sql_mapping //> compat4 old Java stack
 
-  if (cds.env.fiori.direct_crud) {
-    const DRAFT_NEW = 'draftNew'
-    for (const each in dsn.definitions) {
-      const def = dsn.definitions[each]
-      if (!def['@Common.DraftRoot.NewAction'] && def['@Common.DraftRoot.ActivationAction']) {
-        const srvName = Object.keys(dsn.definitions)
-          .filter(k => dsn.definitions[k].kind === 'service')
-          .find(k => each.startsWith(`${k}.`))
-        def['@Common.DraftRoot.NewAction'] = `${srvName}.${DRAFT_NEW}`
-        const params = { in: { items: { type: '$self' } } }
-        // for UI pop-up asking for values for non-UUID keys
-        Object.keys(def.elements)
-          .filter(k => k !== 'IsActiveEntity' && def.elements[k].key && def.elements[k].type !== 'cds.UUID')
-          .forEach(k => (params[k] = { type: def.elements[k].type }))
-        def.actions[DRAFT_NEW] = { kind: 'action', params, returns: { type: each } }
-      }
-    }
-  }
+  if (cds.env.fiori.direct_crud) cds.compile.for.direct_crud(dsn)
 
   Object.defineProperty (csn, '_4odata', {value:dsn})
   Object.defineProperty (dsn, '_4odata', {value:dsn})

package/lib/compile/to/edm.js

@@ -42,6 +42,7 @@ function cds_compile_to_edmx (csn,_o) {
     const next = () => {
       if (!result) {
         if (cds.env.features.annotate_for_flows) enhanceCSNwithFlowAnnotations4FE(csn)
+        if (cds.env.fiori.direct_crud) cds.compile.for.direct_crud(csn)
         result = o.service === 'all' ? _many('.xml', cdsc.to.edmx.all(csn, o)) : cdsc.to.edmx(csn, o)
       }
       return result

package/lib/env/cds-env.js

@@ -295,7 +295,7 @@ class Config {
       const any = this._add_vcap_services_to (vcaps)
       if (any)  this._sources.push ('process.env.VCAP_SERVICES')
     } catch(e) {
-      throw new Error ('[cds.env] - failed to parse VCAP_SERVICES:\n  '+ e.message)
+      throw new Error ('[cds.env] - failed to parse VCAP_SERVICES:\n  '+ e.message, {cause: e})
     }
   }
 

package/lib/i18n/index.js

@@ -1,3 +1,5 @@
+/* eslint-disable no-useless-assignment */
+
 const I18nBundle = require ('./bundles')
 const I18nFiles = require ('./files')
 const cds = require('../index')

package/lib/ql/cds.ql-infer.js

@@ -1,3 +1,5 @@
+/* eslint-disable no-useless-assignment */
+
 const cds = require('../index')
 const infer = exports = module.exports = (..._) => infer.target(..._)
 

package/lib/req/response.js

@@ -1,3 +1,5 @@
+/* eslint-disable no-useless-assignment */
+
 const cds = require ('../index')
 const placeholders = [...'x'.repeat(9)].map((x,i) => `{${i+1}}`)
 

package/lib/srv/middlewares/auth/mocked-users.js

@@ -39,7 +39,7 @@ const _configured = (u,x) => {
     u.attr = { ...u.attr, ...x }
   }
   if (u.jwt) {
-    if ((x = _deprecated (u.jwt.zid, 'jwt.zid','tenant'))) {
+    if ((    _deprecated (u.jwt.zid, 'jwt.zid','tenant'))) {
       u.tenant = u.jwt.zid
     }
     if ((x = _deprecated (u.jwt.attributes, 'jwt.attributes','attr'))) {

package/lib/utils/cds-utils.js

@@ -189,7 +189,7 @@ exports.read = async function read (file, _encoding) {
   if (_encoding === 'json' || !_encoding && f.endsWith('.json')) try {
     return JSON.parse(src)
   } catch(e) {
-    throw new Error (`Failed to parse JSON in ${f}: ${e.message}`)
+    throw new Error (`Failed to parse JSON in ${f}: ${e.message}`, { cause: e })
   }
   else return process.platform === 'win32' ? src?.replace(/\r\n/g, '\n') : src
 }

package/libx/_runtime/messaging/redis-messaging.js

@@ -36,7 +36,7 @@ class RedisMessaging extends cds.MessagingService {
     try {
       await this.client.connect()
     } catch (e) {
-      throw new Error('Connection to Redis could not be established: ' + e)
+      throw new Error('Connection to Redis could not be established: ' + e, { cause: e })
     }
 
     this._ready = true

package/libx/_runtime/messaging/service.js

@@ -77,9 +77,11 @@ module.exports = class MessagingService extends cds.Service {
     if (!cds.context) cds.context = {}
     if (ctx.tenant) cds.context.tenant = ctx.tenant
     if (!ctx.user) ctx.user = cds.User.privileged
+    // REVISIT: is cds.context.model really needed?
     // this.tx expects cds.context.model
-    if (cds.model && (cds.env.requires.extensibility || cds.env.requires.toggles))
+    if (ctx.tenant && cds.model && (cds.env.requires.extensibility || cds.env.requires.toggles))
       cds.context.model = await ExtendedModels.model4(ctx.tenant, ctx.features || {})
+    else if (cds.model) cds.context.model = cds.model
     const me = this.options.inboxed || this.options.inbox ? cds.queued(this) : this
     return await me.tx(ctx, tx => tx.emit(msg))
   }

package/libx/odata/middleware/batch.js

@@ -441,7 +441,7 @@ const _processBatch = async (srv, router, req, res, next, body, ct, boundary) =>
 
     const queue = []
     let _continue = true
-    const _queued_exec = (atomicityGroup, responses = []) => {
+    const _queued_exec = (atomicityGroup, agIndex, responses = []) => {
       const groupId = atomicityGroup[0].atomicityGroup
 
       return new Promise(resolve => queue.push(resolve)).then(() => {
@@ -487,16 +487,23 @@ const _processBatch = async (srv, router, req, res, next, body, ct, boundary) =>
 
             // ensure all subrequests run in this tx
             // (if first subrequest fails without really opening the tx, the rest are executed in a "dangling tx")
-            return Promise.allSettled(subrequests).then(ress => {
-              const failed = ress.filter(({ status }) => status === 'rejected')
-              if (!failed.length) return
-              // throw first error and call srv.on('error') for the others
-              const first = failed.shift()
-              if (srv.handlers._error?.length)
-                for (const other of failed)
-                  for (const each of srv.handlers._error) each.handler.call(srv, other.reason, cds.context)
-              throw first.reason
-            })
+            return Promise.allSettled(subrequests)
+              .then(ress => {
+                // wait for all previous atomicity groups (ignoring errors via allSettled) for odata v2
+                const prevs = []
+                for (let i = 0; i < agIndex; i++) prevs.push(promises[i])
+                return Promise.allSettled(prevs).then(() => ress)
+              })
+              .then(ress => {
+                const failed = ress.filter(({ status }) => status === 'rejected')
+                if (!failed.length) return
+                // throw first error and call srv.on('error') for the others
+                const first = failed.shift()
+                if (srv.handlers._error?.length)
+                  for (const other of failed)
+                    for (const each of srv.handlers._error) each.handler.call(srv, other.reason, cds.context)
+                throw first.reason
+              })
           })
           .catch(err => {
             responses._has_failure = true
@@ -528,7 +535,7 @@ const _processBatch = async (srv, router, req, res, next, body, ct, boundary) =>
 
     // queue execution of atomicity groups
     const promises = []
-    for (const atomicityGroup of atomicityGroups) promises.push(_queued_exec(atomicityGroup))
+    for (let i = 0; i < atomicityGroups.length; i++) promises.push(_queued_exec(atomicityGroups[i], i))
 
     // trigger first max_parallel in queue
     for (let i = 0; i < max_parallel; i++) if (queue.length) queue.shift()()

package/libx/odata/parse/afterburner.js

@@ -463,7 +463,6 @@ function _processSegments(from, model, namespace, cqn, protocol) {
 
         // > navigation
         one = !!(current.is2one || ref[i].where)
-        incompleteKeys = one || i === ref.length - 1 ? false : true
         current = model.definitions[current.target]
         target = current
 

package/libx/queue/index.js

@@ -259,7 +259,7 @@ const _processTasks = (target, tenant, _opts = {}) => {
               LOG.error(`${service.name}: Programming error detected:`, e)
               task.updateData = { attempts: opts.maxAttempts }
               toBeUpdated.push(task)
-              throw new Error(`${service.name}: Programming error detected.`)
+              throw new Error(`${service.name}: Programming error detected.`, { cause: e })
             }
             if (e.unrecoverable) {
               LOG.error(`${service.name}: Unrecoverable error:`, e)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds",
-  "version": "9.8.0",
+  "version": "9.8.2",
   "description": "SAP Cloud Application Programming Model - CDS for Node.js",
   "homepage": "https://cap.cloud.sap/",
   "keywords": [