Skip to content

Drop the cds add warning: 0.3.1 writes the project's module format - #32

Merged
oblomov-dev merged 1 commit into
mainfrom
claude/drop-cds-add-caveat
Sep 29, 2026
Merged

oblomov-dev merged 1 commit into
mainfrom
claude/drop-cds-add-caveat

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Merge only once @cap2ui5/cds-plugin@0.3.1 is on npm (cap2UI5/cap2UI5#93, then the v0.3.1 tag). Until then the warning is still true for what npm add installs.

What changes

  • Quickstart and Project Structure: the warning that cds add cap2ui5 writes require (rename the file to .cjs or rewrite its first line) is removed. Since 0.3.1 (cds add cap2ui5: write the first app in the project's module format cap2UI5#92), the file is written with import or require, whichever module format the project's package.json declares.
  • Current release named as 0.3.1 on ecosystem, configuration, deployment and in AGENTS.md. The deployment page's dependency line becomes "@cap2ui5/cds-plugin": "^0.3.1", the first version with the fix.
  • Left alone: the roadmap's "0.3.0 added npx cap2ui5 abap2js" (history), and the samples' peerDependencies example (^0.3.0, which is what @cap2ui5/samples declares).

Checks

npm run check against the release branch (plugin 0.3.1) and abap2UI5 main: verify-refs OK, build OK.

🤖 Generated with Claude Code

https://claude.ai/code/session_01NwpbkojNzGmuJrFdbwo5JF


Generated by Claude Code

@cap2ui5/cds-plugin 0.3.1 (cap2UI5/cap2UI5#92, #93) makes `cds add cap2ui5`
write its first app with import in an ES module project and with require
otherwise. The quickstart and the project-structure page told readers to
rename the file or rewrite its first line; that is no longer needed. The
pages that name the current release say 0.3.1, and the deployment page's
dependency asks for ^0.3.1, the first version with the fix.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NwpbkojNzGmuJrFdbwo5JF
@oblomov-dev
oblomov-dev marked this pull request as ready for review September 29, 2026 11:46
@oblomov-dev
oblomov-dev merged commit ec4c42f into main Sep 29, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants