@sap/cds 8.9.1 → 8.9.3

package/CHANGELOG.md

@@ -4,6 +4,23 @@
 - The format is based on [Keep a Changelog](https://keepachangelog.com/).
 - This project adheres to [Semantic Versioning](https://semver.org/).
 
+## Version 8.9.3 - 2025-05-06
+
+### Fixed
+
+- OData: `$value` access of primitive properties returned by custom handler
+- UCL: Add missing declaration of variable `$input` in mutation for creating an application template
+- Purge of `servers` from `.cds-services.json` file
+- Loading of relative service implementations in plugins
+- `cds.compile.to.edmx` in case the model was manipulated in a plugin
+
+## Version 8.9.2 - 2025-04-14
+
+### Fixed
+
+- `forUpdate` will not consider `wait` if `ignoreLocked` is set
+- Do not crash in case of custom `DraftAdministrativeData` table
+
 ## Version 8.9.1 - 2025-04-03
 
 ### Fixed

package/lib/compile/for/lean_drafts.js

@@ -144,7 +144,7 @@ module.exports = function cds_compile_for_lean_drafts(csn) {
       }
       if (e.name === 'DraftAdministrativeData') {
         // redirect to DraftAdministrativeData service entity
-        _redirect(newEl, active._service.entities.DraftAdministrativeData)
+        if (active._service?.entities.DraftAdministrativeData) _redirect(newEl, active._service.entities.DraftAdministrativeData)
       }
       Object.defineProperty (newEl,'parent',{value:draft,enumerable:false, configurable: true, writable: true})
 

package/lib/compile/to/edm.js

@@ -71,7 +71,7 @@ function _4odata (csn,o) {
   }
 
   // o.service is specified now
-  return cds.compile.for.odata(csn,o)
+  return cds.env.features.compile_to_edmx_compat ? cds.compile.for.odata(csn,o) : csn //> leave that to cdsc: cds.compile.for.odata(csn,o)
 }
 
 

package/lib/ql/SELECT.js

@@ -152,8 +152,8 @@ class SELECT extends Whereable {
   forUpdate ({ of, wait = cds.env.sql.lock_acquire_timeout || -1, ignoreLocked } = {}) {
     const sfu = this.SELECT.forUpdate = {}
     if (of) sfu.of = of.map (c => ({ref:c.split('.')}))
-    if (wait >= 0) sfu.wait = wait
     if (ignoreLocked) sfu.ignoreLocked = true
+    else if (wait >= 0) sfu.wait = wait
     return this
   }
 

package/lib/srv/bindings.js

@@ -99,11 +99,12 @@ class Bindings {
   }
 
   purge() {
-    if (this.done) return; else this.done = true
+    if (this.done) return;
     this.load (fs.readFileSync)
     LOG.debug ('purging bindings from:', registry)
     this.cleanup()
     this.store (fs.writeFileSync)
+    this.done = true
   }
 
   /**
@@ -113,7 +114,7 @@ class Bindings {
     const { servers, provides } = this, { pid } = process
     for (let [key,srv] of Object.entries (provides))
       if (srv.server === pid || url && srv.credentials?.url?.startsWith(url)) delete provides [key]
-    delete servers [pid]
+    delete servers [pid.toString()]
     return this
   }
 }

package/lib/srv/factory.js

@@ -23,9 +23,9 @@ function ServiceFactory (name, model, options) {
       return _use (impl[name] || impl.default || _kind())
     case 'string':
       if (impl.startsWith('@sap/cds/')) impl = cds.home + impl.slice(8)  //> for local tests in @sap/cds dev
-      if (impl.startsWith('./')) impl = path.resolve (cds.root, _source4(def) || '.', '..', impl.slice(2))
-      try { var resolved = require.resolve(impl, {paths:[ cds.root, cds.home ]}) } catch {
-        try { resolved = require.resolve(path.join(cds.root, impl)) } catch (e) { // compatibility
+      if (impl.startsWith('./')) impl = path.join (_source4(def) || 'x', '.'+impl)
+      try { var resolved = require.resolve (path.join (cds.root, impl)) } catch (e) {       // fetch local paths
+        try { resolved = require.resolve (impl, {paths:[ cds.root, cds.home ]}) } catch {  // fetch in node_modules
           throw cds.error(`Failed loading service implementation from ` + impl, { cause: e })
         }
       }
@@ -51,7 +51,7 @@ const _sibling = d => {
   for (let subdir of ['/', '/lib/', '/handlers/']) {
     for (let ext of exts) try {
       const impl = dir + subdir + name + ext
-      return isfile(impl) || require.resolve (impl)
+      return isfile(impl) || require.resolve (impl, {paths:[ cds.root, cds.home ]})
     } catch(e) {
       if (e.code !== 'MODULE_NOT_FOUND') throw e
     }

package/libx/_runtime/ucl/Service.js

@@ -155,7 +155,7 @@ const READ_QUERY = `
   }`
 
 const CREATE_MUTATION = `
-  mutation {
+  mutation ($input: ApplicationTemplateInput!) {
     result: createApplicationTemplate (
       in: $input
     ) {

package/libx/odata/middleware/read.js

@@ -246,7 +246,7 @@ module.exports = adapter => {
 
         // 204
         if (result == null && query.SELECT.one) return res.sendStatus(204)
-        if (_propertyAccess && result[_propertyAccess] === null) return res.sendStatus(204)
+        if (_propertyAccess && result[_propertyAccess] == null) return res.sendStatus(204)
 
         // 304
         if (validateIfNoneMatch(cdsReq.target, req.headers?.['if-none-match'], result)) return res.sendStatus(304)

package/package.json

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