feat: support tracloop for GenAI observability #11
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add Tools field to ResponseSpanParams and ResponseSpanData for capturing tool schemas - Implement tool schema extraction and serialization in tracing framework - Add proper SDK tool conversion with full parameter schemas and descriptions - Update traceloop processor to instrument tool definitions in prompts - Support both function tools and hosted tools (web search, etc.) - Update to official go-openllmetry SDK v0.1.0 release - Add comprehensive tool definition extraction with reflection-based parsing 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Hi @nirga! Thanks for working on this library and contributing this PR! I noticed the PR includes some files and features beyond the Traceloop integration mentioned in the title, with some overlap with existing functionality (like LiteLLM). Would you mind updating it to focus specifically on the Traceloop support? This would help keep the changes focused and make the review process smoother. Happy to help if needed - just let me know! |
|
Done @matteo-grella |
- Remove langsmith tracing wrapper directory - Keep only traceloop integration for tool definitions instrumentation 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
|
@matteo-grella can you take a look? 🙏 |
|
Thanks @nirga this is under evaluation. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Instrument agents with go-openllmetry.
cc @afavi3tal
See: