-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(docs): Add Next.js technical issue to OpenAI documentation #16149
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
base: master
Are you sure you want to change the base?
feat(docs): Add Next.js technical issue to OpenAI documentation #16149
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
|
||
| </PlatformSection> | ||
|
|
||
| <PlatformSection supported={['javascript.nextjs']}> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@priscilawebdev As we discussed in DMs, you might want to update this section to cover manual instrumentation for all runtimes in Next.js
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated!
docs/platforms/javascript/common/configuration/integrations/anthropic.mdx
Outdated
Show resolved
Hide resolved
docs/platforms/javascript/common/configuration/integrations/openai.mdx
Outdated
Show resolved
Hide resolved
coolguyzone
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
…thropic.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…enai.mdx Co-authored-by: Alex Krawiec <alex.krawiec@sentry.io>
…r-open-ai-and-anthropic
Based on @RulaKhaled’s report and my own findings, when using Next.js with Sentry without adding any AI instrumentation to the Sentry initialization, AI spans are not generated. That is, even though Node.js is the server runtime, the Sentry integration does not work out of the box. Users must explicitly instrument the OpenAI client using the helper
Sentry.instrumentOpenAiClient. This PR reflects that requirement.TODO: Update this in the agent monitoring onboarding too