Fix: use @sap/cds instead of @sap/cds-dk for build plugin (#324)

* refactor: use @sap/cds instead of @sap/cds-dk for build plugin

- Change build plugin to extend cds.build.Plugin from @sap/cds
- Remove @sap/cds-dk from peerDependencies
- Update tests to mock cds.build.Plugin before requiring build.js
- Aligns with other CAP plugins (notifications, captivator) pattern
- cds.build.Plugin is injected by cds-dk at build time

* remove cds-dk

* use npmx

* revert removing cds-dk from doc and move cds-dk to devDep

Author: zongqichen

__tests__/unit/build.test.js

@@ -1,33 +1,31 @@
-const cds = require("@sap/cds");
 const path = require("path");
-const OrdBuildPlugin = require("../../lib/build");
 const { BUILD_DEFAULT_PATH } = require("../../lib/constants");
 const index = require("../../lib/index");
 
-jest.mock("@sap/cds-dk", () => {
-    return {
-        build: {
-            Plugin: class {
-                constructor() {
-                    this.task = {
-                        dest: undefined,
-                        src: null,
-                    };
-                }
-                write() {
-                    return {
-                        to: () => {
-                            return Promise.resolve();
-                        },
-                    };
-                }
-                async model() {
-                    return Promise.resolve({});
-                }
+// Setup cds.build.Plugin mock BEFORE requiring build.js
+const cds = require("@sap/cds");
+cds.build = cds.build || {};
+cds.build.Plugin = class {
+    constructor() {
+        this.task = {
+            dest: undefined,
+            src: null,
+        };
+    }
+    write() {
+        return {
+            to: () => {
+                return Promise.resolve();
             },
-        },
-    };
-});
+        };
+    }
+    async model() {
+        return Promise.resolve({});
+    }
+};
+
+// Now require the build plugin after the mock is set up
+const OrdBuildPlugin = require("../../lib/build");
 
 jest.mock("../../lib/index", () => {
     return {

lib/build.js

@@ -1,13 +1,12 @@
 const cds = require("@sap/cds");
-const cds_dk = require("@sap/cds-dk");
 const path = require("path");
 const _ = require("lodash");
 const { ord, getMetadata } = require("./index");
 const cliProgress = require("cli-progress");
 const { BUILD_DEFAULT_PATH, ORD_SERVICE_NAME, ORD_DOCUMENT_FILE_NAME } = require("./constants");
 const { isMCPPluginInPackageJson } = require("./mcpAdapter");
 
-module.exports = class OrdBuildPlugin extends cds_dk.build.Plugin {
+module.exports = class OrdBuildPlugin extends cds.build.Plugin {
     static taskDefaults = { src: cds.env.folders.srv };
 
     init() {

memory-bank/techContext.md

@@ -101,7 +101,6 @@ cds compile srv/ --to ord -o ./custom-output/
 ### CAP Framework Dependencies
 
 - **Minimum CAP Version**: 8.9.4 (peer dependency)
-- **Minimum CDS-DK Version**: 8.9.5 (peer dependency)
 - **CSN Compatibility**: Must handle all CAP CSN model structures
 - **Interop CSN**: Generate interop-compatible CSN format
 - **Plugin Architecture**: Must follow CAP plugin conventions

package.json

@@ -32,11 +32,11 @@
         "jest": "^30.0.0",
         "prettier": "3.7.4",
         "supertest": "^7.0.0",
-        "@cap-js/sqlite": "^2"
+        "@cap-js/sqlite": "^2",
+        "@sap/cds-dk": ">=8.9.5"
     },
     "peerDependencies": {
-        "@sap/cds": ">=8.9.4",
-        "@sap/cds-dk": ">=8.9.5"
+        "@sap/cds": ">=8.9.4"
     },
     "dependencies": {
         "@cap-js/asyncapi": "^1.0.3",

xmpl_java/package.json

@@ -5,8 +5,5 @@
     "license": "ISC",
     "dependencies": {
         "@cap-js/ord": "file:../../ord"
-    },
-    "devDependencies": {
-        "@sap/cds-dk": "^9.1.3"
     }
 }