Skip to content

feat(extensibility): clean up ORD integration#134

Open
sophiaabi wants to merge 1 commit into
SAP:mainfrom
sophiaabi:updateExtensibility
Open

feat(extensibility): clean up ORD integration#134
sophiaabi wants to merge 1 commit into
SAP:mainfrom
sophiaabi:updateExtensibility

Conversation

@sophiaabi
Copy link
Copy Markdown
Member

Disclaimer: Do not include SAP-internal or customer-specific information in this PR (e.g. internal system URLs, customer names, tenant IDs, or confidential configurations). This is a public repository.

Description

Remove the feature to add dynamic extended integration dependencies to agent ORD as it is no longer a requirement for extensibility to work for net new MCP servers

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

Breaking Changes

If this PR introduces breaking changes, please describe:

Any consumer using:
from sap_cloud_sdk.extensibility import add_extension_integration_dependencies
will get an ImportError after upgrading.

Users should remove the usage of this method from their ORD implementation as it is no longer necessary for extensibility.

@sophiaabi sophiaabi requested a review from a team as a code owner May 21, 2026 18:31
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented May 21, 2026

CLA assistant check
All committers have signed the CLA.

@sophiaabi sophiaabi changed the title Clean up extensibility ORD integration feat(extensibility): clean up ORD integration May 21, 2026
@sophiaabi sophiaabi force-pushed the updateExtensibility branch from bd43e5f to a02365e Compare May 21, 2026 18:34
@tiagoek tiagoek self-requested a review May 21, 2026 18:35
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.

3 participants