-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
[Javascript] Add LLM monitoring support #12981
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Currently difficult because we need to annotate the LLM spans with the pipeline span's name, and there is no nice way to pass context down in a runtime agnostic way In python this is done with ContextVar Maybe we can convert the pipeline span into a transaction, and pull down its name into the children that way? |
In Node.js we can use In fact the Sentry scope lives on-top of async local storage in the Node SDK, so you can just store stuff on the scope to use. |
Where did this net out? Can I roll it by hand using lower level primitives / patterns via the in the javascript SDK? |
@staticshock we have automatic instrumentation for Vercel's For other libraries, you have to add manual instrumentation. We documented the span conventions you can use for that here: https://develop.sentry.dev/sdk/telemetry/traces/modules/llm-monitoring/ |
Uh oh!
There was an error while loading. Please reload this page.
Problem Statement
Add support for https://docs.sentry.io/product/insights/llm-monitoring/ in JavaScript
Solution Brainstorm
Things we can support:
ai
#13679The text was updated successfully, but these errors were encountered: