Skip to content

feat: initial wrapper for plugins hooks. - #22

Merged
Castorche merged 3 commits into
mainfrom
feat/add-wrapper-on-hooks-from-other-plugins
Jun 16, 2026
Merged

feat: initial wrapper for plugins hooks.#22
Castorche merged 3 commits into
mainfrom
feat/add-wrapper-on-hooks-from-other-plugins

Conversation

@Castorche

@Castorche Castorche commented Apr 23, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR introduces the wrapper for hooks of other plugins.
It currently handles a tool call that is blocked by another plugin, by creating the tool span and setting it in error state, and setting the error.type SpanAttribute to tool_blocked.

Type of Change

  • Bugfix
  • New Feature
  • Breaking Change
  • Refactor
  • Documentation
  • Other (please describe)

Checklist

  • I have read the contributing guidelines
  • Existing issues have been referenced (where applicable)
  • I have verified this change is not present in other open pull requests
  • Functionality is documented
  • All code style checks pass
  • New code contribution is covered by automated tests
  • All new and existing tests pass

@Castorche
Castorche requested a review from a team as a code owner April 23, 2026 15:59
Comment thread observability-plugin/src/wrapper.ts
@Castorche
Castorche force-pushed the feat/add-wrapper-on-hooks-from-other-plugins branch 2 times, most recently from 15e26a9 to f773337 Compare April 24, 2026 20:40
Signed-off-by: Jacques Samain <jsamain@cisco.com>
Signed-off-by: Jacques Samain <jsamain@cisco.com>
@Castorche
Castorche force-pushed the feat/add-wrapper-on-hooks-from-other-plugins branch from f773337 to 1861ced Compare June 12, 2026 09:00
Signed-off-by: Jacques Samain <jsamain@cisco.com>
@Castorche
Castorche force-pushed the feat/add-wrapper-on-hooks-from-other-plugins branch from 113af36 to 05dedc4 Compare June 12, 2026 11:59
@Castorche
Castorche merged commit 3c2d57e into main Jun 16, 2026
4 checks 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