Conversation
SummaryThe following content is AI-generated and provides a summary of the pull request: Use Default Import Options from cds-dkNew Feature✨ Leverages the new default options support introduced in Changes
PR Bot InformationVersion:
💌 Have ideas or want to contribute? Create an issue and share your thoughts with us! Made with ❤️ by Hyperspace. |
There was a problem hiding this comment.
The PR introduces default import options via the new cds-dk 9.8 API, removes the now-redundant package.json mutation logic, and updates the test fixture accordingly. One substantive issue was found: the process options entry uses plain assignment instead of ??=, which would silently overwrite any user-configured options, inconsistent with the defensive ??= pattern used throughout the rest of the registration code.
PR Bot Information
Version: 1.18 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback
- LLM:
anthropic--claude-4.6-sonnet - Correlation ID:
90de9380-1d49-11f1-8385-1f8e96505e34 - Event Trigger:
pull_request.opened
|
Somebody please fix the typescript *** |
|
overruled by #36 |
New cds-dk (9.8) has support for default options for import
Writing to package json is done automitacally by dk