Skip to content

feat(observability): add tool call completion hook - #192

Merged
mcollina merged 1 commit into
platformatic:mainfrom
rozzilla:feat/observability/add-tool-call-completion-hook
Aug 21, 2026
Merged

feat(observability): add tool call completion hook#192
mcollina merged 1 commit into
platformatic:mainfrom
rozzilla:feat/observability/add-tool-call-completion-hook

Conversation

@rozzilla

Copy link
Copy Markdown
Contributor

Adds a single transport-neutral onToolCallComplete hook that fires once after every tool call settles, across JSON-RPC, mcpCallTool(), and tasks.

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina

Copy link
Copy Markdown
Member

conflicts

Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>

update

Signed-off-by: Roberto Bianchi <roberto.bianchi@spendesk.com>
@rozzilla
rozzilla force-pushed the feat/observability/add-tool-call-completion-hook branch from 33eb138 to bb23d48 Compare August 21, 2026 17:20
@rozzilla
rozzilla requested a review from mcollina August 21, 2026 17:20
@rozzilla

Copy link
Copy Markdown
Contributor Author

conflicts

@mcollina rebased and fixed

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina
mcollina merged commit 47f94bf into platformatic:main Aug 21, 2026
2 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