Skip to content

Bump dependencies#123

Merged
eric-pSAP merged 2 commits into
mainfrom
chore/bump-dependencies
Apr 14, 2026
Merged

Bump dependencies#123
eric-pSAP merged 2 commits into
mainfrom
chore/bump-dependencies

Conversation

@schiwekM
Copy link
Copy Markdown
Contributor

Bumping Cloud SDK dependencies. Remove chai as it was unused. Add cds-types for CAP types.

@schiwekM schiwekM requested a review from eric-pSAP April 14, 2026 09:30
@hyperspace-insights
Copy link
Copy Markdown
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:


Bump Dependencies and Clean Up Dev Dependencies

Chore

🔧 Updated SAP Cloud SDK dependencies to their latest major version, removed the unused chai library, and added @cap-js/cds-types for CAP TypeScript types support.

Changes

  • package.json:

    • Bumped @sap-cloud-sdk/connectivity, @sap-cloud-sdk/http-client, and @sap-cloud-sdk/util from ^3.13.0 to ^4.6.0.
    • Added @cap-js/cds-types: ^0.16.0 as a dev dependency for CAP types.
    • Removed unused chai dev dependency.
    • Upgraded jest from ^29.7.0 to ^30.3.0.
    • Minor formatting fix on the engines field indentation.
  • tests/unit/lib/notificationTypes.test.js: Removed the unused chai import.

  • tests/unit/lib/notifications.test.js: Added @sap/cds import to support test utilities.

  • tests/unit/srv/notifyToRest.test.js: Added @sap/cds import to support test utilities.


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.20.11 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • File Content Strategy: Full file content
  • LLM: anthropic--claude-4.6-sonnet
  • Correlation ID: 8db31130-37e4-11f1-9d15-a7f96093dfdf
  • Output Template: Default Template
  • Event Trigger: pull_request.opened
  • Summary Prompt: Default Prompt

💌 Have ideas or want to contribute? Create an issue and share your thoughts with us!
📑 Check out the documentation for more information.
📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!

Made with ❤️ by Hyperspace.

Copy link
Copy Markdown
Contributor

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR is mostly clean — bumping Cloud SDK from v3 to v4, removing the unused chai dependency, and upgrading Jest to v30 are all straightforward improvements. However, there is one substantive issue: @sap/cds was dropped from devDependencies while new test files that directly require('@sap/cds') were added in the same PR, which will cause test runs to fail in a clean install where the peer dependency is not present.

PR Bot Information

Version: 1.20.11 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback

  • LLM: anthropic--claude-4.6-sonnet
  • Correlation ID: 8db31130-37e4-11f1-9d15-a7f96093dfdf
  • File Content Strategy: Full file content
  • Event Trigger: pull_request.opened

Comment thread package.json
@eric-pSAP eric-pSAP merged commit bbcade8 into main Apr 14, 2026
2 checks passed
@eric-pSAP eric-pSAP deleted the chore/bump-dependencies branch April 14, 2026 09:36
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