@sap/cds-dk 8.7.2 → 8.7.3

package/CHANGELOG.md

@@ -7,7 +7,17 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
-## Version 8.7.2 - 2024-02-14
+## Version 8.7.3 - 2025-02-19
+
+### Fixed
+
+- `cds compile -o` fixes the output file name in `-o <service>.json`.
+- `cds lint` won't stumble over scalar config objects anymore.
+- `cds init` uses latest Maven Java archetype version 3.7.2 for creating Java projects.
+- `cds deploy --to hana --dry` doesn't exit with a `TypeError` if there are no models.
+- `cds add ias` fixes a few scenarios in combination with multitenancy.
+
+## Version 8.7.2 - 2025-02-14
 
 ### Fixed
 
@@ -21,7 +31,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 - `cds bind -a` doesn't concurrently try to check the Cloud Foundry version or OAuth token.
 - `cds import` now resolves target of association/composition correctly for multiple schema files.
 
-## Version 8.7.1 - 2024-02-04
+## Version 8.7.1 - 2025-02-04
 
 ### Fixed
 

package/bin/compile/index.js

@@ -294,7 +294,7 @@ async function compile (models, options={}) {
   const write = require ('../../lib/util/write')
   chain = chain.then (write.to ({
     folder: options.dest,
-    file: options.file || options.service === 'all' ? path.basename(src) : options.service,
+    file: options.file || (options.service === 'all' ? path.basename(src) : options.service),
     suffix: options.suffix || suffix4(_suffix),
     [options.dest ? 'foreach' : 'log']: options.log || consoleLog
   }))

package/bin/lint.js

@@ -1,7 +1,13 @@
 const Linter = require('../lib/lint/Linter');
 const linter = new Linter();
 
-module.exports = Object.assign(
-    linter.lint,
-    linter.init()
-);
+// cds cli exec expects an executable object with various additional properties.
+// As most of the linting functionality is bound to a class instance, we need to wrap the instance
+// to not lose the this-context and attach the additional properties to the function.
+const lint = (...args) => linter.lint(...args)
+const additionalProperties = Object.entries(linter.init())
+    .map(([key, value]) => [key, { value }])
+
+Object.defineProperties(lint, Object.fromEntries(additionalProperties))
+
+module.exports = lint

package/lib/deploy/to-hana/hana.js

@@ -79,6 +79,7 @@ Therefore, specifying the CDS model is not required when calling 'cds deploy'.`;
     }
 
     if (dry) {
+      if (!buildResult) return
       const sortedResult = Array.from(buildResult.hana).sort();
       for (const file of sortedResult) {
         const srcPath = path.join(buildResult.dest, file);
@@ -87,7 +88,7 @@ Therefore, specifying the CDS model is not required when calling 'cds deploy'.`;
         console.log();
       }
     } else {
-      // the primary database is always built and deployed 
+      // the primary database is always built and deployed
       const dest = path.join(cds.env.build.target, cds.env.folders.db);
 
       let serviceName = pServiceName;
@@ -211,8 +212,7 @@ Therefore, specifying the CDS model is not required when calling 'cds deploy'.`;
       src: cds.env.folders.db,
       [OUTPUT_MODE]: OUTPUT_MODE_FILESYSTEM | OUTPUT_MODE_RESULT
     }
-    console.log(`Running build`);
-    
+
     // cds build throws an error if the src folder does not exist
     const buildResult = await build(buildTaskOptions);
     return buildResult[0].result;

package/lib/init/constants.js

@@ -5,7 +5,7 @@ module.exports = {
     COMMAND_ADD: 'add',
 
     JAVA_LTS_VERSIONS: [17, 21],
-    MAVEN_ARCHETYPE_VERSION: '3.7.1',
+    MAVEN_ARCHETYPE_VERSION: '3.7.2',
 
     OPTIONS: Object.freeze({
         COMPLETION: 'completion',

package/lib/init/merge.js

@@ -330,7 +330,7 @@ async function _mergeYAML(into, from, project, semantics = {}) {
                     })
                     .filter(item => !entryMap.get(item))
                     .forEach(item => {
-                        if (!templateEntryMap.get(item)) throw 'Error: did not find entry in template for ' + inspect(item, { colors }) + ' in ' + from
+                        if (!templateEntryMap.get(item)) throw 'Error: did not find entry in template for ' + inspect(item, { colors, depth:11 }) + ' in ' + from
                         const templateNode = templateEntryMap.get(item).node
                         item.at !== undefined ? targetNode.items.splice(item.at, 0, templateNode) : targetNode.add(templateNode)
                     })

package/lib/init/registries/mta.js

@@ -140,7 +140,7 @@ const requiredMtxSidecarApi4 = srv => ({
     where: { 'properties.CDS_MULTITENANCY_SIDECAR_URL': '~{mtx-url}' }
 })
 
-const providedJavaApprouterApi4 = srv => ({
+const requiredJavaApprouterApi4 = srv => ({
     in: [srv, 'requires'],
     where: { 'properties.CDS_MULTITENANCY_APPUI_URL': '~{url}' }
 })
@@ -216,7 +216,7 @@ module.exports = {
     requiredMtxSidecarApi4,
     providedAppApi,
     requiredAppApi4,
-    providedJavaApprouterApi4,
+    requiredJavaApprouterApi4,
     providedJavaApprouterApi,
     applicationLogging,
     approuterMTXRoute,

package/lib/init/template/approuter/files/mta.yaml.hbs

@@ -12,9 +12,9 @@ modules:
     requires:
       - name: {{appName}}-auth
     {{#isJava}}
-      - name: app-api
+      - name: app-api{{#hasMultitenancy}}
         properties:
-          CDS_MULTITENANCY_APPUI_URL: ~{url}
+          CDS_MULTITENANCY_APPUI_URL: ~{url}{{/hasMultitenancy}}
     {{/isJava}}
 
   - name: {{appName}}

package/lib/init/template/approuter/index.js

@@ -6,7 +6,7 @@ const { sort, merge } = require('../../merge')
 const {
     srv4, mtxSidecar4, approuter, // Modules
     xsuaa, // BTP Services
-    requiredMtxApi, providedJavaApprouterApi4, providedJavaApprouterApi, // APIs
+    requiredMtxApi, requiredJavaApprouterApi4, providedJavaApprouterApi, // APIs
     requiredAppApi4, providedAppApi,
     approuterMTXRoute,
     approuterMTXRouteJava // xs-app.json config
@@ -36,7 +36,7 @@ module.exports = class ApprouterTemplate extends require('../../plugin') {
 
         if (hasMta) {
             const srv = srv4(srvPath)
-            const apis = isJava ? [providedJavaApprouterApi4(srv), providedJavaApprouterApi] : []
+            const apis = isJava && hasMultitenancy ? [requiredJavaApprouterApi4(srv), providedJavaApprouterApi] : []
             const mtxSidecar = mtxSidecar4(isJava ? 'mtx/sidecar' : 'gen/mtx/sidecar')
             if (isNodejs && hasMultitenancy) apis.push(requiredMtxApi, requiredAppApi4(mtxSidecar), providedAppApi)
             if (isJava && hasMultitenancy) apis.push(requiredMtxApi)

package/lib/init/template/ias/files/mta.yaml.hbs

@@ -36,7 +36,7 @@ modules:
 
   - name: {{appName}}-mtx
     type: nodejs
-    path: gen/mtx/sidecar
+    path: {{#isNodejs}}gen/{{/isNodejs}}mtx/sidecar
     requires:
       - name: {{appName}}-auth
         parameters:

package/lib/init/template/ias/files/sidecar.package.json.hbs

@@ -0,0 +1,12 @@
+{
+  "dependencies": {
+    "@sap/xssec": "^4"
+  },
+  "cds": {
+    "requires": {
+      "auth": {
+        "kind": "ias"
+      }
+    }
+  }
+}

package/lib/init/template/ias/index.js

@@ -23,9 +23,6 @@ module.exports = class IasTemplate extends require('../../plugin') {
         const { configFile, isNodejs } = project
         await merge(__dirname, 'files/package.json.hbs').into(configFile, { project, forceOverwrite: true })
         if (isNodejs) await sort('package.json', 'dependencies')
-        if (project.hasMultitenancy) {
-            await merge(__dirname, 'files/package.json.hbs').into('mtx/sidecar/package.json', { project, forceOverwrite: true })
-        }
         // if (project.isJava) await mvn.add('ias')
     }
 
@@ -43,6 +40,7 @@ module.exports = class IasTemplate extends require('../../plugin') {
                 conf.cds.profiles.push('subscription-manager')
             }
             await write('mtx/sidecar/package.json', conf, { spaces: 2 })
+            await merge(__dirname, 'files/sidecar.package.json.hbs').into('mtx/sidecar/package.json')
         }
 
         if (hasMta) {

package/lib/init/template/multitenancy/files/mta.yaml.hbs

@@ -92,14 +92,14 @@ resources:
         applicationType: application
         appName: {{appName}}-${org}-${space}
         appCallbacks:
-            dependenciesCallbacks:
-              url: ~{mtx-api/mtx-cert-url}/-/cds/sms-provisioning/dependencies/{app_tid}
-            subscriptionCallbacks:
-              url: ~{mtx-api/mtx-cert-url}/-/cds/sms-provisioning/tenant/{app_tid}
-              async:
-                subscribeEnable: true
-                unSubscribeEnable: true
-                timeoutInMillis: 300000 # Increase if your deployments are taking longer than that
+          dependenciesCallbacks:
+            url: {{#isJava}}~{srv-api/srv-cert-url}/mt/sms/subscriptions/tenants/{app_tid}/dependencies{{/isJava}}{{#isNodejs}}~{mtx-api/mtx-cert-url}/-/cds/sms-provisioning/dependencies/{app_tid}{{/isNodejs}}
+          subscriptionCallbacks:
+            url: {{#isJava}}~{srv-api/srv-cert-url}/mt/sms/subscriptions/tenants/{app_tid}{{/isJava}}{{#isNodejs}}~{mtx-api/mtx-cert-url}/-/cds/sms-provisioning/tenant/{app_tid}{{/isNodejs}}
+            async:
+              subscribeEnable: true
+              unSubscribeEnable: true
+              timeoutInMillis: 300000 # Increase if your deployments are taking longer than that
         displayName: {{appName}}
         description: Description for {{appName}}
         category: Category

package/lib/lint/Linter.js

@@ -1,42 +1,185 @@
 const term = require("../util/term");
-
 const io = require("./io");
 const checks = require("./checks");
 const generators = require("./generators");
-
 const cds = require("../../lib/cds");
 const { path, exists, isdir, isfile, readdir } = cds.utils
 const { exit } = require("process");
+
 const LOG = cds.debug("lint");
 const LOG_CONFIG = cds.debug("lint:config");
-
 const GENERAL_ERROR = 1;
 
-class Linter {
-  constructor() {
-    this.eslintCmd = "";
-    this.eslintCmdShort = "eslint";
-    this.eslintCmdFileExpr = "";
-    this.isFile = false;
-    this.help = "";
-    this.debug = "";
-    this.flags = [];
-    this.configPath = "";
-    this.configContents = {};
-    this.cdsdkPath = path.join(__dirname, "../..");
-    this.globalPath = path.join(this.cdsdkPath, "../..");
-    this.extendsPlugin = false;
-    this.fileExtensions = [];
-    this.pluginPath = "";
-    this.pluginApi = {};
-    this.lintType = "global";
-    this.ruleOpts = {};
-    this.customRulesOpts = [];
-    this.pluginRules = [];
-    this.init = this.init.bind(this);
-    this.lint = this.lint.bind(this);
+/**
+ * Safely require a module, return empty object if module not found
+ * @param {string} module - module to require
+ * @param {boolean} esm - whether to use dynamic import to accomodate ESM
+ * @returns {Promise<ReturnType<NodeRequire> | {}>} module or empty object
+ */
+async function tryRequire(module, esm = false) {
+  try {
+    return esm ? import(module) : require(module)
+  } catch (e) {
+    if (e.code === 'MODULE_NOT_FOUND' && e.message.includes(module)) return {}
+    // target module is either .mjs (etc) or user project is ESM
+    if (['ERR_REQUIRE_ASYNC_MODULE', 'ERR_REQUIRE_ESM'].includes(e.code)) return tryRequire(module, true)
+    throw e
+  }
+}
+
+/**
+ * Load ESLint module
+ * @param {string} currentDir - current directory to start search
+ * @returns {Promise<ReturnType<NodeRequire>>} - ESLint module
+ */
+async function _loadEslint(currentDir) {
+  const { loadESLint, ESLint } = require('eslint')
+  const DefaultESLint = loadESLint
+    ? await loadESLint({ cwd: currentDir }) // ESLint 9
+    : new ESLint({ cwd: currentDir }) // ESLint 8
+  return DefaultESLint;
+}
+
+/**
+ * @param {string[]} configFiles - possible config file paths
+ * @param {string} currentDir - current directory to start search
+ * @returns {string} - path to config file
+ */
+async function findConfigPath (configFiles, currentDir = '.') {
+  let configDir = path.resolve(currentDir)
+  while (configDir && configDir !== path.dirname(configDir)) {
+    for (const configFile of configFiles) {
+      const configPath = path.join(configDir, configFile)
+      const config = await tryRequire(configPath)
+      if (configFile !== 'package.json' || config?.eslintConfig) {
+        if (exists(configPath) && isfile(configPath)) { // TODO exists neccessary ?
+          return configPath
+        }
+      }
+    }
+    configDir = path.dirname(configDir);
+  }
+  return ''
+}
+
+/** 
+ * This class and its subclasses are an attempt to smoothen out some of the
+ * workflow in the Linter class. It gets rid of some of the isESLintLegacy ? ... : ...
+ * assignments, although not all. The remaining branching is too closely coupled
+ * to some properties of the Linter class to be easily abstracted away.
+ * @abstract
+ */
+class ESLintLoader {
+  /** 
+   * Searches for ESLint config file types (in order or precedence)
+   * and returns corresponding directory (usually project's root dir)
+   * https://eslint.org/docs/user-guide/configuring#configuration-file-formats
+   * @param {string} currentDir start here and search until root dir
+   * @returns {string} dir containing ESLint config file (empty if not exists)
+   * @abstract
+   */
+  // eslint-disable-next-line no-unused-vars
+  async getConfigPath (currentDir = '.') { throw new Error("Not implemented") }
+  /** @type {import('./io').sanitizeEslintConfig} */
+  // eslint-disable-next-line no-unused-vars
+  sanitizeEslintConfig (configIn, customRuleExample, logger) { throw new Error("Not implemented") }
+  /** @type {import('./io').readEslintConfig} */
+  // eslint-disable-next-line no-unused-vars
+  readEslintConfig(configPath) { throw new Error("Not implemented") }
+}
+
+class ESLint8Loader extends ESLintLoader {
+  async getConfigPath(currentDir = '.') {
+    return findConfigPath([
+      ".eslintrc.js",
+      ".eslintrc.cjs",
+      ".eslintrc.yaml",
+      ".eslintrc.yml",
+      ".eslintrc.json",
+      ".eslintrc",
+      "package.json",
+    ], currentDir)
+  }
+
+  sanitizeEslintConfig (configIn, customRuleExample, logger) {
+    return io.sanitizeEslintConfigLegacy(configIn, customRuleExample, logger)
+  }
+
+  readEslintConfig(configPath) {
+    return io.readEslintConfigLegacy(configPath)
+  }
+}
+
+class ESLint9Loader extends ESLintLoader {
+  async getConfigPath(currentDir = '.') {
+    return findConfigPath([
+      "eslint.config.js",
+      "eslint.config.cjs",
+      "eslint.config.mjs",
+      // the following 3 formats require additional setup by the user as of today,
+      // see https://eslint.org/docs/latest/use/configure/configuration-files#typescript-configuration-files
+      "eslint.config.ts",
+      "eslint.config.mts",
+      "eslint.config.cts",
+    ], currentDir)
+  }
+
+  sanitizeEslintConfig (configIn, customRuleExample, logger) {
+    return io.sanitizeEslintConfig(configIn, customRuleExample, logger)
   }
 
+  readEslintConfig(configPath) {
+    return io.readEslintConfig(configPath)
+  }
+}
+
+class Linter {
+  /** @type {string | undefined} */
+  eslintCmd = "";
+  /** @type {string} */
+  eslintCmdShort = "eslint";
+  /** @type {string | string[]} */
+  eslintCmdFileExpr = "";
+  /** @type {boolean} */
+  isFile = false;
+  /** @type {string} */    
+  help = "";
+  /** @type {string} */
+  debug = "";
+  /** @type {string[]} */
+  flags = [];
+  /** @type {string} */
+  configPath = "";
+  /** @type {object} */
+  configContents = {};
+  /** @type {import("fs").PathLike} */
+  cdsdkPath = path.join(__dirname, "../..");
+  /** @type {import("fs").PathLike} */
+  globalPath = path.join(this.cdsdkPath, "../..");
+  /** @type {boolean} */
+  extendsPlugin = false;
+  /** @type {string[]} */
+  fileExtensions = [];
+  /** @type {string} */
+  pluginPath = "";
+  /** @type {object} */
+  pluginApi = {};
+  /** @type {'global' | 'local'} */
+  lintType = "global";
+  /** @type {object} */
+  ruleOpts = {};
+  /** @type {array} */
+  customRulesOpts = [];
+  /** @type {array} */
+  pluginRules = [];
+  /** @type {ESLintLoader | undefined} */
+  #loader = undefined;
+  get loader () {
+    return this.#loader ??= this.isESLintLegacy
+      ? new ESLint8Loader()
+      : new ESLint9Loader()
+  };
+
   /**
    * Initializes `cds lint` call and generates the required content
    * object for `cds` executable
@@ -59,146 +202,69 @@ class Linter {
   /**
    * Runner for 'cds lint' which is a wrapper for eslint cmd calls
    * that detects and adds the required cmd line arguments
-   * @param {*} args files/globs for eslint to lint
-   * @param {*} options options/flag passed by user
+   * @param {string[]} args files/globs for eslint to lint
+   * @param {{[key: string]: unknown}} options options/flag passed by user
    */
   async lint(args, options) {
     // Check for ESLint (legacy)
     const currentDir = process.cwd();
-    const DefaultESLint = await this._loadEslint(currentDir);
+    const DefaultESLint = await _loadEslint(currentDir)
     this.isESLintLegacy = DefaultESLint.configType === "eslintrc";
-
-    this.eslintCmdOpts = this._sanitizeUserOpts(options);
+    this.eslintCmdOpts = options;  // these should probably be passed on in #runEsLint!
     this.eslintCmdFileExpr = args.length ? args : ["."];
-    if (this.eslintCmdOpts.help) {
-      this._printHelp();
-    } else {
-      // Determine if case "local" or "global" and resolve plugin accordingly
-      await this._getConfigFileAndSetProjectPath(args);
-
-      if (this.pluginPath) {
-        const overwriteRules = checks.hasEslintConfigContent(this.configContents, "rules");
-        // Overwrite rules severities
-        if (overwriteRules) {
-          await this._overwriteRuleSeverities();
-        }
 
-        // Limit to CDS file extensions
-        this._addExtensions();
+    if (this.eslintCmdOpts.help) {
+      this.#printHelp();
+      return
+    }
+    // Determine if case "local" or "global" and resolve plugin accordingly
+    await this.#getConfigFileAndSetProjectPath(args);
+
+    if (this.pluginPath) {
+      const overwriteRules = checks.hasEslintConfigContent(this.configContents, "rules");
+      // Overwrite rules severities
+      if (overwriteRules) {
+        await this.#overwriteRuleSeverities();
       }
 
-      // Run ESLint with collected options
-      try {
-        await this._runEslint(currentDir, DefaultESLint);
-      } catch (err) {
-        term.error(err);
-      }
+      // Limit to CDS file extensions
+      this.#addExtensions();
     }
-  }
-
-  _printHelp() {
-    console.log(this.help.replace(/ \*([^*]+)\*/g, ` ${term.codes.bold}$1${term.codes.reset}`));
-  }
 
-  _sanitizeUserOpts(options) {
-    for (let key in options) {
-      let value = options[key];
-      if (value && typeof value === "string") {
-        value = value.replace(/'/g, "").replace(/"/g, "");
-      }
+    // Run ESLint with collected options
+    try {
+      await this.#runEslint(currentDir, DefaultESLint);
+    } catch (err) {
+      term.error(err);
     }
-    return options;
   }
 
   /**
-   * Searches for ESLint config file types (in order or precedence)
-   * and returns corresponding directory (usually project's root dir)
-   * https://eslint.org/docs/user-guide/configuring#configuration-file-formats
-   * @param {string} currentDir start here and search until root dir
-   * @returns {string} dir containing ESLint config file (empty if not exists)
+   * Prints help message
    */
-  async _getConfigPathLegacy(currentDir = ".") {
-    const configFiles = [
-      ".eslintrc.js",
-      ".eslintrc.cjs",
-      ".eslintrc.yaml",
-      ".eslintrc.yml",
-      ".eslintrc.json",
-      ".eslintrc",
-      "package.json",
-    ];
-    let configDir = path.resolve(currentDir);
-    while (configDir && configDir !== path.dirname(configDir)) {
-      for (let i = 0; i < configFiles.length; i++) {
-        const configPath = path.join(configDir, configFiles[i]);
-        const config = this.safeRequire(configPath)
-
-        if (configFiles[i] !== "package.json" || config?.eslintConfig) {
-          if (exists(configPath) && await isfile(configPath)) {
-            return configPath;
-          }
-        }
-      }
-
-      configDir = path.dirname(configDir);
-    }
-    return "";
-  }
-
-  /**
- * Searches for ESLint config file types (in order or precedence)
- * and returns corresponding directory (usually project's root dir)
- * https://eslint.org/docs/user-guide/configuring#configuration-file-formats
- * @param {string} currentDir start here and search until root dir
- * @returns {string} dir containing ESLint config file (empty if not exists)
- */
-  async _getConfigPath(currentDir = ".") {
-    const configFiles = [
-      "eslint.config.js",
-      "eslint.config.cjs",
-      "eslint.config.mjs"
-    ];
-    let configDir = path.resolve(currentDir);
-    while (configDir && configDir !== path.dirname(configDir)) {
-      for (let i = 0; i < configFiles.length; i++) {
-        const configPath = path.join(configDir, configFiles[i]);
-        const config = this.safeRequire(configPath)
-        if (configFiles[i] !== "package.json" || config?.eslintConfig) {
-          if (exists(configPath) && await isfile(configPath)) { // TODO exists neccessary ?
-            return configPath;
-          }
-        }
-      }
-      configDir = path.dirname(configDir);
-    }
-    return "";
+  #printHelp() {
+    console.log(this.help.replace(/ \*([^*]+)\*/g, ` ${term.codes.bold}$1${term.codes.reset}`));
   }
 
-  _addExtensions() {
+  #addExtensions() {
     // Add CDS file extensions to lint
-    this.fileExtensions = this.pluginApi.getFileExtensions().map((ext) => path.extname(ext));
+    this.fileExtensions = this.pluginApi.getFileExtensions()
+      .map((ext) => path.extname(ext));
     // Only lint file extensions prescribed by plugin
-    this.ignorePatterns = [];
-    this.fileExtensions.forEach((ext) => {
-      this.ignorePatterns.push(`!${ext}`);
-    });
+    this.ignorePatterns = this.fileExtensions
+      .map(ext => `!${ext}`)
   }
 
-  async _runEslint(currentDir, DefaultESLint) {
+  async #runEslint(currentDir, DefaultESLint) {
     try {
-
-      let eslintOpts = {
+      const eslintOpts = this.lintType === "global"
+      ? {
+        cwd: process.cwd(),
+        overrideConfig: {},
+      }
+      : {
         cwd: currentDir,
-        overrideConfig: {
-          ...this.configContents,
-        },
-      };
-
-      if (this.lintType === "global") {
-        eslintOpts = {
-          cwd: process.cwd(),
-          overrideConfig: {},
-        };
+        overrideConfig: {...this.configContents},
       }
 
       if (this.isESLintLegacy) {
@@ -213,7 +279,7 @@ class Linter {
       } else {
         LOG?.("Using flat ESLint configuration");
         const cdsPlugin = require(this.pluginPath)
-        // Exvlude our own apis from plugin to avoid wrapper errors
+        // Exclude our own apis from plugin to avoid wrapper errors
         delete cdsPlugin.configs.recommended?.plugins?.['@sap/cds']
         eslintOpts.overrideConfig = [cdsPlugin.configs.recommended]
       }
@@ -224,17 +290,22 @@ class Linter {
 
       LOG_CONFIG?.(eslintOpts);
 
-      let lintString = `eslint`;
-      if (this.fileExtensions) { lintString += ` --ext "${this.fileExtensions.join(",")}"` }
+      if (LOG) {
+        let lintString = `eslint`;
 
-      for (const ruleOpt of Object.entries(this.ruleOpts)) {
-        lintString += ` --rule ${ruleOpt[0]}:${ruleOpt[1]}`;
-      }
-      if (this.customRulesOpts && this.customRulesOpts.length > 0) {
-        lintString += ` --rulesdir "${this.customRulesOpts}"`
-      }
+        if (this.fileExtensions) {
+          lintString += ` --ext "${this.fileExtensions.join(",")}"`
+        }
 
-      LOG?.(lintString);
+        for (const [name, rule] of Object.entries(this.ruleOpts)) {
+          lintString += ` --rule ${name}:${rule}`
+        }
+        if (this.customRulesOpts && this.customRulesOpts.length > 0) {
+          lintString += ` --rulesdir "${this.customRulesOpts}"`
+        }
+
+        LOG(lintString);
+      }
 
       if (!process.env.isTest) {
         const eslint = new DefaultESLint(eslintOpts);
@@ -252,7 +323,7 @@ class Linter {
           results = results.concat(resultsModel);
         }
 
-        if (results && results.length > 0) {
+        if (results?.length > 0) {
           console.log(formatter.format(results));
         }
       }
@@ -272,31 +343,17 @@ class Linter {
     }
   }
 
-  async _getConfigFileAndSetProjectPath(args) {
-    let input;
+  async #getConfigFileAndSetProjectPath(args) {
     // Get config path
     if (args.length > 0) {
       const firstArg = args[0];
-      if (firstArg === "." || (isfile(firstArg) || isdir(firstArg))) {
-        this.isFile = true;
-      }
-      input = args.join(" ");
-      if (!input) {
-        input = ".";
-      }
+      this.isFile = firstArg === "." || isfile(firstArg) || isdir(firstArg)
     }
-    this.configPath = this.isESLintLegacy
-      ? await this._getConfigPathLegacy(process.cwd())
-      : await this._getConfigPath(process.cwd());
+    this.configPath = await this.loader.getConfigPath(process.cwd())
     if (!this.configPath) {
-      this.lintType === "global";
-      this.configContents = this.isESLintLegacy
-        ? await io.sanitizeEslintConfigLegacy({}, false, LOG)
-        : await io.sanitizeEslintConfig({}, false, LOG);
+      this.configContents = this.loader.sanitizeEslintConfig({}, false, LOG)
     } else {
-      const configContents = this.isESLintLegacy
-        ? await io.readEslintConfigLegacy(this.configPath)
-        : await io.readEslintConfig(this.configPath);
+      const configContents = this.loader.readEslintConfig(this.configPath)
       this.configContents = configContents;
       if (this.isESLintLegacy) {
         this.configContents = await io.sanitizeEslintConfig(configContents, false, LOG);
@@ -313,14 +370,16 @@ class Linter {
                   paths: [path.dirname(this.configPath)],
                 });
                 this.pluginApi = require(this.pluginPath.replace("index.js", "api"));
-              } catch (err) {
+              } catch {
                 // CLI will report (no locally installed plugin)
               }
             }
           }
         }
       } else {
-        const cdsConfig = configContents?.filter(c => c.files?.includes("*.cds"))[0] ?? {}
+        const cdsConfig = Array.isArray(configContents)
+          ? configContents?.find(c => c.files?.includes("*.cds")) ?? {}
+          : configContents
         this.configContents = cdsConfig.files
           ? {
             ...cdsConfig.plugins.configs?.recommended,
@@ -337,7 +396,7 @@ class Linter {
           paths: [this.cdsdkPath],
         });
         this.pluginApi = require(this.pluginPath.replace("index.js", "api"));
-      } catch (err) {
+      } catch {
         // CLI will report (no globally installed plugin)
       }
     }
@@ -345,46 +404,28 @@ class Linter {
     LOG?.(`ESLint CDS plugin resolved: ${this.pluginPath}`);
   }
 
-  async _overwriteRuleSeverities() {
+  async #overwriteRuleSeverities() {
     let configContents = {};
     if (this.configPath) {
       configContents = await io.readEslintConfigLegacy(this.configPath);
     } else {
-      configContents = this.isESLintLegacy
-        ? await io.readEslintConfigLegacy(await this._getConfigPath(this.cdsdkPath))
-        : await io.readEslintConfig(await this._getConfigPath(this.cdsdkPath));
+      configContents = this.loader.readEslintConfig(await this.loader.getConfigPath(this.cdsdkPath))
     }
-    let rules = configContents?.rules || {};
+    const rules = configContents?.rules ?? {};
     // Allow recommended plugin rules in cds-dk to be overwritten
     // by user by adding rule to cmd line (because of precedence)
     const pluginRules = require(this.pluginPath).configs.recommended.rules;
     if (rules && pluginRules) {
-      for (const rule of Object.keys(rules)) {
-        if (typeof rules[rule] !== "undefined" && rules[rule] != pluginRules[rule] ||
-          isfile(path.relative(".", ".eslint", "rules", rule))) {
-          pluginRules[rule] = rules[rule];
-          this.ruleOpts[rule] = rules[rule];
+      for (const [name, rule] of Object.entries(rules)) {
+        if (typeof rule !== "undefined" && rule != pluginRules[name] ||
+          isfile(path.relative(".", ".eslint", "rules", name))) {
+          pluginRules[name] = rule;
+          this.ruleOpts[name] = rule;
           this.pluginRules = pluginRules;
         }
       }
     }
   }
-
-  async _loadEslint(currentDir) {
-    const { loadESLint, ESLint } = require("eslint");
-    const DefaultESLint = loadESLint
-      ? await loadESLint({ cwd: currentDir }) // ESLint 9
-      : new ESLint({ cwd: currentDir }) // ESLint 8
-    return DefaultESLint;
-  }
-
-  safeRequire(module) {
-    try {
-      return require(module)
-    } catch (e) {
-      return {}
-    }
-  }
 }
 
 module.exports = Linter;

package/lib/lint/io.js

@@ -8,7 +8,7 @@ module.exports = {
   /**
    * Reads ESLint config contents from file
    * @param {*} configPath config file path to read from
-   * @returns config file contents
+   * @returns {object[] | object} config file contents
    */
   async readEslintConfigLegacy(configPath) {
     let config;
@@ -19,7 +19,7 @@ module.exports = {
       case ".eslintrc.cjs":
         try {
           config = JSONC.parse((configString.replace(/^module\.exports =/, "")));
-        } catch (err) {
+        } catch {
           config = {};
         }
         break;
@@ -102,7 +102,6 @@ module.exports = {
  * (2) Option "root": false to stop further config merging
  * (3) Add's custom rule example 'no-entity-moo' if requested
  * @param {*} configPath ESLint config file
- * @returns {void}
  */
   async sanitizeEslintConfigLegacy(configIn, customRuleExample = false, logger = console) {
     let configContents = {};
@@ -121,10 +120,8 @@ module.exports = {
         if (!configContents["extends"].includes(extendsValue)) {
           configContents["extends"].push(extendsValue);
         }
-      } else {
-        if (configContents["extends"] !== extendsValue) {
-          configContents["extends"] = [configContents["extends"], extendsValue];
-        }
+      } else if (configContents["extends"] !== extendsValue) {
+        configContents["extends"] = [configContents["extends"], extendsValue];
       }
     } else {
       configContents["extends"] = extendsValue;

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "@sap/cds-dk",
-  "version": "8.7.2",
+  "version": "8.7.3",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@sap/cds-dk",
-      "version": "8.7.2",
+      "version": "8.7.3",
       "license": "SEE LICENSE IN LICENSE",
       "dependencies": {
         "@cap-js/asyncapi": "^1.0.0",
@@ -166,26 +166,16 @@
       }
     },
     "node_modules/@eslint/plugin-kit": {
-      "version": "0.2.5",
+      "version": "0.2.6",
       "license": "Apache-2.0",
       "dependencies": {
-        "@eslint/core": "^0.10.0",
+        "@eslint/core": "^0.11.0",
         "levn": "^0.4.1"
       },
       "engines": {
         "node": "^18.18.0 || ^20.9.0 || >=21.1.0"
       }
     },
-    "node_modules/@eslint/plugin-kit/node_modules/@eslint/core": {
-      "version": "0.10.0",
-      "license": "Apache-2.0",
-      "dependencies": {
-        "@types/json-schema": "^7.0.15"
-      },
-      "engines": {
-        "node": "^18.18.0 || ^20.9.0 || >=21.1.0"
-      }
-    },
     "node_modules/@humanfs/core": {
       "version": "0.19.1",
       "license": "Apache-2.0",
@@ -901,6 +891,19 @@
         "node": ">= 0.4"
       }
     },
+    "node_modules/es-set-tostringtag": {
+      "version": "2.1.0",
+      "license": "MIT",
+      "dependencies": {
+        "es-errors": "^1.3.0",
+        "get-intrinsic": "^1.2.6",
+        "has-tostringtag": "^1.0.2",
+        "hasown": "^2.0.2"
+      },
+      "engines": {
+        "node": ">= 0.4"
+      }
+    },
     "node_modules/escape-html": {
       "version": "1.0.3",
       "license": "MIT"
@@ -1212,7 +1215,7 @@
       }
     },
     "node_modules/flatted": {
-      "version": "3.3.2",
+      "version": "3.3.3",
       "license": "ISC"
     },
     "node_modules/follow-redirects": {
@@ -1234,11 +1237,12 @@
       }
     },
     "node_modules/form-data": {
-      "version": "4.0.1",
+      "version": "4.0.2",
       "license": "MIT",
       "dependencies": {
         "asynckit": "^0.4.0",
         "combined-stream": "^1.0.8",
+        "es-set-tostringtag": "^2.1.0",
         "mime-types": "^2.1.12"
       },
       "engines": {
@@ -1382,6 +1386,19 @@
         "url": "https://github.com/sponsors/ljharb"
       }
     },
+    "node_modules/has-tostringtag": {
+      "version": "1.0.2",
+      "license": "MIT",
+      "dependencies": {
+        "has-symbols": "^1.0.3"
+      },
+      "engines": {
+        "node": ">= 0.4"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/ljharb"
+      }
+    },
     "node_modules/hasown": {
       "version": "2.0.2",
       "license": "MIT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds-dk",
-  "version": "8.7.2",
+  "version": "8.7.3",
   "description": "Command line client and development toolkit for the SAP Cloud Application Programming Model",
   "homepage": "https://cap.cloud.sap/",
   "keywords": [