docs: update to CAP 10 and Node.js 22+ (24 LTS recommended)#166
Merged
Conversation
- Bump documented Node.js requirement 20.19.0 -> 22.0.0 (24 LTS recommended, following the SAP CAP 10 recommendation) - Reflect @sap/cds 10.x and corrected CI matrix (Node 22/24) in dev docs - Fix hardcoded Node version + broken import paths in agent-instructions generator (paths broke when script moved to scripts/ in 750bc5e) - Regenerate agent-instructions artifacts (v4.202607.0)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Aligns all documentation with the CAP 10 upgrade already reflected in
package.json(@sap/cds ^10.0.3,engines.node >=22.0.0) and the CI matrix (Node 22.x/24.x). The code was already on CAP 10 / Node 22+; the prose docs and generated agent-instruction artifacts had lagged behind at Node 20.19.0 / CAP 9.Consistent phrasing used throughout: minimum Node.js 22, Node.js 24 LTS highly recommended (following the SAP CAP 10 recommendation).
Changes
Hand-maintained docs
README.md,CLAUDE.mddocs/01-getting-started/{index,installation}.mddocs/03-features/vscode-extension.md,docs/03-features/mcp/{server-updates,server-implementation-complete}.mddocs/99-reference/faq-extended.md,docs/troubleshooting/index.mdDeveloper instructions
.github/instructions/project-overview.instructions.md— Node ≥22,@sap/cds9.7.1 → 10.x, version-validation note.github/instructions/ci-cd-workflows.instructions.md— corrected stale matrix to Node 22.x/24.x (was documenting a 20/22/24 matrix that no longer matches the workflow)Generator + regenerated artifacts
scripts/generate-agent-instructions.js:Node.js ≥ 22.0.0 (Node.js 24 LTS recommended)import()paths andgetVersion()still used./bin/…/./mcp-server/…/__dirname/package.jsonafter the script was moved intoscripts/in750bc5e, so the generator had been crashing on every run. Corrected to../.agent-instructions/(v4.202607.0) — version stamps4.202603.2→4.202607.0, generated date refreshed, Node requirement updated, plus command content that had drifted since the last successful generation.Not included
Stray dependency bumps (
package.json,npm-shrinkwrap.json) andvscode-extension/esbuild.config.mjspresent in the working tree were intentionally left out — they belong in a separate dependency-update change.Verification
npx eslint scripts/generate-agent-instructions.js→ cleannpm run generate:agent-instructions→ runs cleanly (173 commands, 29 files)20.19.0/≥ 20/4.202603references remain inagent-instructions/