Skip to content

fix(extensibility): improving spans - #344

Open
rishikunnath2747 wants to merge 7 commits into
SAP:mainfrom
yashmeet29:feat/extensibility-models
Open

rishikunnath2747 wants to merge 7 commits into
SAP:mainfrom
yashmeet29:feat/extensibility-models

Conversation

@rishikunnath2747

Copy link
Copy Markdown
Contributor

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

This PR adds improvements to extension spans

  1. Adds new spans for instructions
  2. Adds solutionId and isExtension to summary span

Type of Change

  • 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

Testing

This change was tested in dev-eu12 on extensibility agent, given below are screenshots captured from eu octobus

image image

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

@rishikunnath2747
rishikunnath2747 requested a review from a team as a code owner September 17, 2026 19:12
@rishikunnath2747 rishikunnath2747 changed the title fix(extensibility): Improving spans fix(extensibility): improving spans Sep 17, 2026
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