Skip to content

Conversation

@kashishhora
Copy link
Member

Summary

Modified

  • Refactored tracing architecture from per-tool callback wrapping to handler-level wrapping in tracingV2.ts
  • Added setupToolsCallHandlerWrapping() to intercept setRequestHandler() calls for tools/call handler
  • Added createToolsCallWrapper() to centralize tracing logic at the protocol handler level
  • Simplified addTracingToToolCallback() to only remove context from args instead of handling full tracing

Test Plan

  • All existing tests pass
  • Type checks pass
  • New tests added for handler-level architecture
  • Tests verify no duplicate events when track() called multiple times
  • Tests verify context parameter is properly removed before callback execution

@kashishhora kashishhora requested a review from naji247 November 20, 2025 18:37
@naji247 naji247 self-requested a review November 24, 2025 19:47
@kashishhora kashishhora merged commit e40c7ed into main Nov 24, 2025
8 checks passed
@kashishhora kashishhora deleted the refactor/handler-level-tracing branch November 24, 2025 20: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