@sap/cds-dk 10.0.6 → 10.1.0

package/CHANGELOG.md

@@ -6,17 +6,110 @@ This project adheres to [Semantic Versioning](https://semver.org/).
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/).
 
+## Version 10.1.0 - 2026-09-03
+
+### Added
+
+- `cds add hana-serverless` sets up SAP HANA Cloud serverless as the database, provisioning one HDI container per tenant through the serverless Tenant API.
+- `cds add ai` now supports a `--plan` option to set the AI Core service plan in `mta.yaml` (default: `standard`).
+- `cds add data` now supports a `--keys-only` option that, together with `--records`, fills only key columns in the generated CSV files and leaves all other columns empty.
+- `cds add data` now generates values for `Binary`/`LargeBinary` (base64) and `Vector` (JSON array) elements instead of leaving them empty.
+- `cds add data` now supports a `--composition-records` option to control how many records are generated per to-many composition (default 2).
+- `cds add` now accepts space-separated features in addition to comma-separated, e.g. `cds add xsuaa hana nodejs`.
+- `cds add extensibility` now adds an `ExtensionDeveloper` role collection to `mta.yaml` (suffixed `${org}-${space}`) and `chart/values.yaml` (suffixed with the Kyma namespace), so it can be assigned immediately after deployment.
+- `cds version` now also shows the npm version and executable path.
+
+### Fixed
+
+- `cds add data` now generates values across the full `@assert.range` when the range lies outside the default `[0,100]`, instead of emitting the lower bound for every record (integers) or exceeding the upper bound (decimals).
+- `cds add data` now generates varying `Time` values instead of the same fixed `22:59:53` for every record.
+- `cds add data` now honors `@assert.range` on `Double` elements instead of generating values across the default `[0,100]` regardless of the annotation.
+- `cds add data` now generates whole numbers for `Integer` elements whose `@assert.range` uses fractional bounds, instead of emitting fractional values.
+- `cds add data` now generates varied values for `Decimal` elements whose scale equals their precision (such as `Decimal(2,2)`), instead of emitting `0` for every record.
+- `cds add data` now honors `@assert.range` on `Timestamp` elements instead of always generating dates in the default `2000-2024` window.
+- `cds add data` no longer crashes with `Invalid time value` when a `Date` element has a single-day `@assert.range`.
+- `cds add data` now generates varied dates within `@assert.range` bounds on `Date`, `DateTime` and `Timestamp` elements when the range lies entirely outside the default `2000-2024` window, instead of emitting the same out-of-range value for every record.
+- `cds add data` now honors open-interval and infinity `@assert.range` bounds (such as `[(0),_]` or `[(5),(10)]`) on numeric and date elements, instead of ignoring them and generating values across the default range.
+- `cds add data` now generates the exact day for `Date` elements regardless of the local timezone, instead of occasionally emitting the day before the intended date.
+- `cds add data` now generates multiple distinct rows for link entities whose key consists of two or more associations, instead of collapsing them to a single row.
+- `cds add data` now writes the foreign-key column for to-one compositions into the parent CSV, instead of leaving the parent unlinked to its generated child.
+- `cds add data` now honors an `@assert.range` on an `array of` scalar element, so the generated array elements stay within the annotated bounds.
+- `cds add data` now generates the requested number of distinct rows for a standalone link entity whose key associations point to entities created on demand, instead of collapsing them to a single row.
+- `cds add data` now pairs two distinct target rows in a self-referential link entity (two key associations to the same entity), instead of pointing both foreign keys at the same row.
+- `cds build --ws-pack` now also packs the workspace root package into a module's `gen/srv` when the module depends on it via a `file:` spec, so monorepos whose root package is itself a dependency no longer need to pack it by hand.
+- `cds bind --to-app-services` no longer classifies a machine-to-machine `identity` binding that consumes another service (carrying `consumed-services`) as the application's user authentication, so it no longer overwrites the real `xsuaa` auth binding.
+- `cds build` now copies the workspace root `package-lock.json` into a module's `gen/srv` when the module has none, so npm workspaces monorepo deployments get frozen dependencies instead of freshly resolved ranges.
+- `cds build` now copies `.cdsrc.yaml` into the deployment output folder alongside `.cdsrc.json`, so YAML-based CDS configuration works correctly when deployed to Cloud Foundry.
+- `cds import --from asyncapi` now correctly imports types referenced via `$ref` in AsyncAPI schemas, including `allOf` composition, and no longer loses types from earlier events when a service has multiple events.
+- `cds build` now correctly resolves `manifest.json` for UI5 CLI v5 component projects, which use a `src/` folder instead of `webapp/`.
+- `cds build --for fiori` now honors custom `resources.configuration.paths` mappings in `ui5.yaml` when locating a UI module's `manifest.json`.
+- `cds add html5-repo`, `cds add workzone`, and `cds add portal` no longer emit an HTML5 application content deployer module into `mta.yaml` when the project has no UI apps, avoiding the `could not find the ".../gen/app" path` build failure.
+- `cds up` now pushes buildpack images directly to the registry via `pack build --publish`, avoiding the `manifest invalid` error on containerd-backed Docker daemons.
+- `cds add app-front` avoids duplicates for `srv-api` in `xs-app.json` if entries pre-exist.
+- `cds add ias` (and `cds add ams`) with an approuter no longer leaves a stray `properties.name: srv-api` on the approuter's `srv-api` destination in `mta.yaml`, which uses certificate-based forwarding instead.
+- `cds add sample` fixes cds files such that `const { Books } = cds.entities` works, i.e. the default namespace is set.
+
+### Changed
+
+- `cds watch` now ignores the `.cds` folder by default, alongside the existing defaults like `node_modules` and `gen`. `.cds` model files and `.cdsrc.json` are unaffected.
+- `cds compile -2 asyncapi` and `cds import --asyncapi` are now provided by `@cap-js/asyncapi` as a CDS plugin. The plugin is currently still bundled with `@sap/cds-dk`. Add it explicitly to your `devDependencies` as it will be removed in a future version.
+- `cds build` now packages npm workspace dependencies as tarballs by default, use `--no-ws-pack` to disable.
+- `cds completion` now comes with pre-calculated shell scripts for faster tab-completion.  Also, it no longer disturbs shell completion in VS Code debug terminals.
+- `cds add approuter` now uses `@sap/approuter@^23.0.0` instead of `@sap/approuter@^22.0.0`.
+
+## Version 10.0.9 - tbd
+
+### Fixed
+
+- `cds login`, `cds pull`, and `cds push` again support non-interactive machine login with `--client` and `--mtls`, now performed directly against the authorization server. See `cds login --help` for the credentials and environment variables.
+
+## Version 10.0.8 - 2026-08-24
+
+### Changed
+
+- `cds login` now uses the renamed login endpoints `/-/cds/login/grant` and `/-/cds/login/grant-metadata` with a legacy fallback, and updates saved legacy token URLs.
+
+## Version 10.0.7 - 2026-08-07
+
+### Fixed
+
+- Bundle `@sap/cds-mtxs` 4 again, instead of 3.
+- `cds import --from asyncapi` now correctly imports types referenced via `$ref` in AsyncAPI schemas, including `allOf` composition, and no longer loses types from earlier events when a service has multiple events.
+- `cds add html5-repo`, `cds add workzone`, and `cds add portal` no longer emit an HTML5 application content deployer module into `mta.yaml` when the project has no UI apps, avoiding the `could not find the ".../gen/app" path` build failure.
+- `cds init --nodejs` creates the ESLint and VS Code config files again
+
+### Changed
+
+- `cds init --nodejs` creates a cleaned-up .gitignore file
+
 ## Version 10.0.6 - 2026-07-24
 
 ### Added
 
 - `cds add xsuaa` now generates namespace-suffixed XSUAA `role-collections` for Kyma.
 
+### Changed
+
+- `cds init` and `cds add` now detect app folders by the UI framework in their `package.json` (`@ui5/cli`, `react`, `vue`, `svelte`, `@angular/core`, `vite`) or a `webapp/manifest.json`, instead of matching against well-known folder names.
+- `cds add approuter` now generates the approuter into `.deploy/app-router` by default instead of `app/router`, still respecting an existing `app/router` folder.
+- `cds add html5-repo`, `cds add app-frontend`, and `cds add portal` now place the HTML5 application content deployer in `.deploy/html5-deployer` by default instead of `app/html5-deployer`, still respecting an existing `app/html5-deployer` folder.
+
 ### Fixed
 
+- `cds add http` now respects capability restrictions, emitting only supported requests for `@Capabilities.*Restrictions: false`, `@odata.singleton`, and `@insertonly` entities.
+- `cds add http` now generates requests for bound actions and functions on an instance, including draft-enabled entities (e.g. `POST .../E(ID=…,IsActiveEntity=true)/S.approve`), and `GET` requests for OData functions.
+- `cds add http` now omits readonly, computed and managed fields (`@readonly`, `@Core.Computed`, `@cds.on.insert`/`@cds.on.update`) from `POST`/`PATCH` bodies, since clients cannot set them.
+- `cds add http` now slugs `# @name` handles and suffixes collisions, so delimited names (`![a.b]`) and entities that slug alike (`A.sub` vs. `Asub`) yield valid, unique handles.
+- `cds add http` now flattens nested composition entity-set names to underscores in URLs (e.g. `E_addr` instead of the unroutable `E.addr`).
+- `cds add http` now includes all keys in composite-key draft requests instead of only the first, and single-quotes single string keys (e.g. `Books/'abc'`), matching composite string keys.
+- `cds add http` now addresses parameterized entities as `Entity(p=…)/Set`, omitting the invalid `POST`/`PATCH`/`DELETE` requests it previously generated for them.
+- `cds add http` no longer emits `PATCH`/`DELETE` requests with `undefined` or `[object Object]` in the URL for keys without a sample value, structured keys, or entities without a scalar key.
+- `cds add http` no longer produces a double slash in request URLs for a service mounted at the root path (`@path: '/'`).
+- `cds add http` and `cds add data` no longer crash with `Cannot convert undefined or null to object` for a keyless entity whose composition's `on` condition references `$self`.
 - `cds add http` no longer throws `MULTIPLE_DOCS` when `application.yaml` uses `---` separators (Spring Boot multi-profile format).
 - `cds export --data` now reads seed data via in-memory SQLite instead of the configured db driver, so it no longer fails in projects with `requires.db: hana` (e.g. CAP Java).
 - `cds init --java --add mta` now writes `path: srv` for the Java `srv` module in `mta.yaml` instead of `gen/srv`, so `mbt build` no longer fails with `the "gen/srv" path of the "<app>-srv" module does not exist`.
+- `cds build --for nodejs --production` now also bundles _messages.properties_ files from plugins found via neighborhood resolution. `cds build --for java` retains the previous behaviour and does not use neighborhood resolution for plugin message files.
 - `cds version` no longer reports dependency problems if no dependencies are installed at all. It prints an info message instead.
 - `cds add test` creates tests that are green by default. Sample data is provided as commented code.
 
@@ -84,6 +177,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 - `cds add ias` does not use the `HTML5.` prefix for HTML5 repo settings.
 - `cds add node` now adds a devDependency to `@cap-js/sqlite@^3`
 - `cds upgrade` now produces concise CLI output (count + title per rule) and a structured report with capire-aligned sections.
+- The preferred way of calling `cds export` for Maven is now `cds export --mvn groupId:artifactId`. When using the old `cds export -4 mvn` syntax, groupId and artifactId must be passed via the `--artifact` parameter instead.
 
 ### Fixed
 
@@ -137,6 +231,19 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/).
 - `cds migrate` is removed as `@sap/cds-mtx` to `@sap/cds-mtxs` migration is now effectively complete.
 - `cds login`, `cds logout`, `cds pull`, `cds push`: the project-folder argument and the `--directory` (`-d`) option are removed; run these commands from the project directory instead.
 
+## Version 9.9.5 - 2026-08-24
+
+### Changed
+
+- `cds login` now uses the renamed login endpoints `/-/cds/login/grant` and `/-/cds/login/grant-metadata` with a legacy fallback, and updates saved legacy token URLs.
+
+## Version 9.9.4 - 2026-07-31
+
+### Changed
+
+- `cds init` uses Maven Java archetype version 4.9.1 for creating Java projects.
+- Other dependency updates
+
 ## Version 9.9.3 - 2026-06-29
 
 ### Fixed

package/bin/add.js

@@ -5,7 +5,7 @@ const { join } = require('path')
 const fromDk = fs.readdirSync(join(__dirname, '../lib/init/template'))
 const fromPlugins = require('../lib/init/add').registered
 const conf = {}
-for (const plugin of [...fromDk, ...Object.keys(Object.fromEntries(fromPlugins))]) {
+for (const plugin of [...fromDk, ...fromPlugins.keys()]) {
   const Plugin = fromPlugins.get(plugin) ?? require(join(__dirname, '../lib/init/template', plugin))
   for (const [key, { type, help, short }] of Object.entries(new Plugin().options())) {
     conf[plugin] ??= { options: [], flags: [], help: [] }
@@ -22,15 +22,17 @@ const shortcuts = ['-4', '-p', ...Object.values(conf).flatMap(p => p.options).so
 const flags = ['--force', ...Object.values(conf).flatMap(p => p.flags).map(o => o.key)]
 DEBUG?.('cli config ', { options, shortcuts, flags })
 
+const { COMMA } = require('./completion')
+
 module.exports = Object.assign(add, {
-  handleCompletion,
+  staticCompletions: () => ({ '_': COMMA(require('./completion/completionFs').readTemplates([])) }),
   options,
   shortcuts,
   flags,
   help: `
 # SYNOPSIS
 
-    *cds add* <feature | comma-separated list of features>
+    *cds add* <feature> [<feature> ...]
 
     Add one or more features to an existing project - grow as you go.
 
@@ -66,6 +68,7 @@ ${Object.entries(conf).flatMap(([pluginName, { options, flags }]) => {
 # EXAMPLES
 
   *cds add* sample
+  *cds add* multitenancy hana xsuaa
   *cds add* multitenancy,hana,xsuaa
   *cds add* data --filter my.namespace.MyEntity
   *cds add* mta
@@ -77,29 +80,10 @@ ${Object.entries(conf).flatMap(([pluginName, { options, flags }]) => {
 
 # SUMMARY
 
-  *cds add* <feature | comma-separated list of features>
+  *cds add* <feature> [<feature> ...]
 
 `});
 
-async function handleCompletion(currentWord, previousWord, argv, util) {
-  const allOptionsFlags = [
-    ...options ?? [],
-    ...flags ?? []
-  ].filter(e => !argv.includes(e));
-
-  if (currentWord?.startsWith('-')) {
-    return allOptionsFlags;
-  }
-
-  if (!currentWord && options?.includes(previousWord)) {
-    return [];
-  }
-
-  const entries = await util.completionFs.readTemplates(argv);
-  entries.push(...allOptionsFlags);
-
-  return entries;
-}
 
 async function add(args) {
   const cds = require('../lib/cds')

package/bin/bind.js

@@ -1,6 +1,12 @@
 const term = require('../lib/util/term')
+const { FILE } = require('./completion')
 
 module.exports = Object.assign(cds_bind, {
+    staticCompletions: () => ({
+        '--on':          ['cf', 'k8s'],
+        '--out':         FILE(),
+        '--output-file': FILE(),
+    }),
     options: ['--to', '--for', '--on', '--kind', '--out', '--to-app-services', '--credentials', '--output-file'],
     flags: ['--exec', '--no-create-service-key'],
     shortcuts: ['-2', '-4', '-n', '-k', '-o', '-a', '-c'],
@@ -165,6 +171,10 @@ async function cds_bind(args, options) {
 
         services.forEach(service => {
             const type = service.label || service.type;
+            if (type === 'identity' && Array.isArray(service.credentials?.['consumed-services']) && service.credentials['consumed-services'].length > 0) {
+                console.log(`skipping ${term.bold(service.name)}: identity binding consuming another service, not the application's user authentication`)
+                return
+            }
             const requiredServiceEntry = Object.entries(cds.env.requires).find(([, requiredService]) => requiredService.kind === type && requiredService?.vcap?.name === service.name)
             if (requiredServiceEntry) {
                 resolvedServices.push({ options: { ...options, ...{ to: service.name, serviceArg: requiredServiceEntry[0] } } })

package/bin/build.js

@@ -1,8 +1,15 @@
+const { DIR } = require('./completion');
+
 module.exports = Object.assign(build, {
-    handleCompletion,
+    staticCompletions: () => ({
+        '--for':      ['hana', 'helm', 'java', 'mtx', 'mtx-extension', 'mtx-sidecar', 'nodejs'],
+        '--log-level':['debug', 'info', 'warn', 'error'],
+        '--dest':     DIR(),
+        '--src':      DIR(),
+    }),
     options: ['--project', '--src', '--dest', '--for', '--opts', '--log-level', '--options-model'],
     shortcuts: ['-in', '-s', '-o', '-4'],
-    flags: ['--no-clean', '--ws', '--ws-pack'],
+    flags: ['--no-clean', '--ws', '--ws-pack', '--no-ws-pack'],
     help: `
 # SYNOPSIS
 
@@ -70,7 +77,7 @@ module.exports = Object.assign(build, {
 
         Skips cleaning the output folder before running the build.
 
-    *--ws-pack* (beta)
+    *--ws-pack* (default)
 
         Enable tarball based packaging of npm workspace dependencies - typically defined for
         Node.js mono or multi-repo setups.
@@ -79,6 +86,10 @@ module.exports = Object.assign(build, {
         will be adapted to point to the correct tarball file URL.
         Contents is packaged based on the rules of the 'npm pack' command.
 
+    *--no-ws-pack*
+
+        Disable tarball based packaging of npm workspace dependencies.
+
     *--ws* (beta)
 
         NPM workspaces are typically used in mono or multi-repo setups whereas each workspace
@@ -101,36 +112,6 @@ module.exports = Object.assign(build, {
    *cds* build --for java --dest ../srv/target --opts model=[db/model,srv,schema]
 `})
 
-async function handleCompletion(currentWord, previousWord, argv, util) {
-    const allOptionsFlags = [
-        ...build.options ?? [],
-        ...build.flags ?? []
-    ].filter(e => !argv.includes(e));
-
-    if (currentWord?.startsWith('-')) {
-        return allOptionsFlags;
-    }
-
-    switch (previousWord) {
-        case '--log-level':
-            return ['debug', 'info', 'warn', 'error'];
-        case '-4':
-        case '--for':
-            return ['hana', 'java', 'mtx', 'mtx-extension', 'mtx-sidecar', 'nodejs'];
-        case '--dest':
-        case '-o':
-        case '-s':
-        case '--src': {
-            return util.completionFs.readdir(currentWord, { files: false });
-        }
-    }
-
-    if (!currentWord && build.options?.includes(previousWord)) {
-        return [];
-    }
-
-    return allOptionsFlags;
-}
 
 async function build([project], options = {}) {
     const path = require('path')
@@ -160,6 +141,12 @@ async function build([project], options = {}) {
         }
         delete options["no-clean"]
     }
+    if (options["no-ws-pack"] === "true" || options["no-ws-pack"] === true) {
+        options["ws-pack"] = false
+        delete options["no-ws-pack"]
+    } else if (typeof options["ws-pack"] === 'undefined') {
+        options["ws-pack"] = true
+    }
     // convert opts string to task options object
     if (options.opts) {
         options.taskOptions = scanTaskOptionParams(options.opts)
@@ -198,4 +185,3 @@ function scanTaskOptionParams(optsParams) {
     }
     return taskOptions
 }
-

package/bin/cds.js

@@ -229,7 +229,7 @@ function _dieForUnknownCommand(cmd) {
 }
 
 function getCommands() {
-  const excludeList = new Set(['cds.js', 'cds-ts.js', 'fix-redirects.js']);
+  const excludeList = new Set(['cds.js', 'cds-ts.js', 'cds-tsx.js', 'fix-redirects.js']);
 
   const files = fs.readdirSync(__dirname, { withFileTypes: true, encoding: 'utf8' })
   const result = new Set()

package/bin/compile/index.js

@@ -1,6 +1,19 @@
+const { FILE, DIR } = require('../completion');
 
 module.exports = Object.assign ( compile, {
-  handleCompletion,
+  staticCompletions: () => ({
+    '--to':           ['json', 'yml', 'edm', 'edmx', 'edmx-v2', 'edmx-v4', 'sql', 'hdbtable', 'hana', 'cdl', 'xsuaa', 'openapi', 'asyncapi'],
+    '--for':          ['odata', 'sql'],
+    '--dialect':      ['sqlite', 'h2', 'postgres', 'hana'],
+    '--sql-dialect':  ['sqlite', 'h2', 'postgres', 'hana'],
+    '--log-level':    ['debug', 'info', 'warn', 'error'],
+    '--odata-version':['4.0', '4.01'],
+    '--flavor':       ['sources', 'files', 'parsed', 'xtended', 'inferred'],
+    '--service':      ['all'],
+    '--lang':         ['all'],
+    '--dest':         DIR(),
+    '_':              FILE('cds,json'),
+  }),
   options: [
     '--from',
     '--service',
@@ -77,7 +90,7 @@ module.exports = Object.assign ( compile, {
     - cdl
     - xsuaa
     - openapi
-    - asyncapi
+    - asyncapi  (requires @cap-js/asyncapi as a project dependency, currently bundled as a fallback)
 
   *-4* | *--for* <target>
 
@@ -188,45 +201,6 @@ module.exports = Object.assign ( compile, {
 
 const path = require('path')
 
-async function handleCompletion(currentWord, previousWord, argv, util) {
-  const allOptionsFlags = [
-    ...compile.options ?? [],
-    ...compile.flags ?? []
-  ].filter(e => !argv.includes(e))
-
-  if (currentWord?.startsWith('-')) { return allOptionsFlags }
-
-  switch (previousWord) {
-    case '-2':
-    case '--to':
-      return ['json', 'yml', 'edm', 'edmx', 'edmx-v2', 'edmx-v4', 'sql', 'hdbtable', 'hana', 'cdl', 'xsuaa', 'openapi', 'asyncapi'];
-    case '-4':
-    case '--for':
-      return ['odata', 'sql'];
-    case '--dialect':
-      return ['sqlite', 'h2', 'postgres', 'hana'];
-    case '--log-level':
-      return ['debug', 'info', 'warn', 'error'];
-    case '--odata-version':
-      return ['4.0', '4.01'];
-    case '-f':
-    case '--flavor':
-      return ['sources', 'files', 'parsed', 'xtended', 'inferred'];
-    case '-s':
-    case '--service':
-    case '-l':
-    case '--lang':
-      return ['all'];
-    case '-o':
-    case '--dest':
-      return util.completionFs.readdir(currentWord, { files: false })
-    default:
-  }
-
-  if (!currentWord && compile.options?.includes(previousWord)) { return [] }
-
-  return util.completionFs.readdir(currentWord, { fileRegex: /\.(cds|json)/ })
-}
 
 async function compile_all (root='.') {
 

package/bin/completion/completionFs.js

@@ -1,4 +1,6 @@
 module.exports = new class CompletionFS {
+    // Returns a list of file/folder names under currentFolder that match the given options.
+    // currentFolder may be a partial path typed by the user (e.g. 'src/', 'lib/u').
     async readdir(currentFolder, options) {
         options = {
             files: true,
@@ -10,6 +12,8 @@ module.exports = new class CompletionFS {
         const fsp = require('node:fs/promises');
         const path = require('node:path');
 
+        // Detect the path separator from the typed prefix so completions use the same style.
+        // Group 1 captures '\' (Windows), group 0 captures '/' (Unix); default to '/'.
         const match = /\/|(\\)[^\s]/.exec(currentFolder);
         const separationChar = match?.[1] || match?.[0] || '/';
         let folder = currentFolder || process.cwd();
@@ -18,9 +22,11 @@ module.exports = new class CompletionFS {
         try {
             entries = await fsp.readdir(folder, { withFileTypes: true });
         } catch {
+            // currentFolder was a partial filename (not a directory) — list its parent instead.
             folder = path.dirname(folder);
             entries = await fsp.readdir(folder, { withFileTypes: true });
         }
+        // When listing the cwd, use relative names (e.g. 'src/') rather than absolute paths.
         const isCwd = folder === '.' || folder === process.cwd();
 
         entries = entries
@@ -28,15 +34,10 @@ module.exports = new class CompletionFS {
                 options.folders && e.isDirectory() && e.name.match(options.folderRegex)
                 || options.files && e.isFile() && e.name.match(options.fileRegex))
             .map(e => {
-                let result = isCwd
-                    ? path.relative(folder, e.name)
+                const name = isCwd
+                    ? e.name
                     : (folder + separationChar + e.name).replace(/\\{2,}/g, '\\').replace(/\/{2,}/g, '/');
-
-                if (e.isDirectory()) {
-                    result += separationChar;
-                }
-
-                return result;
+                return e.isDirectory() ? name + separationChar : name;
             });
 
         if (entries.length === 0) {
@@ -45,6 +46,8 @@ module.exports = new class CompletionFS {
         return entries;
     }
 
+    // Returns the names of available project template directories (used by `cds init --add`).
+    // argv: template names to exclude in addition to the built-in 'nodejs' and 'java' ones.
     async readTemplates(argv) {
         const fsp = require('node:fs/promises');
         const path = require('node:path');