-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Closed
Description
Note
The pull request "ref(core): Avoid looking up openai integration options" was created by @mydea but did not reference an issue. Therefore this issue was created for better visibility in external tools like Linear.
This avoids looking up the integration for openai instrumentation, instead relying on this being passed in (which it already is). When manually instrumenting the client you need to pass in the options directly.
- Node: Options are passed from the integration to instrumentOpenAiClient anyhow, so nothing changes
- cloudflare/vercel-edge: There is no integration, users need to manually call instrumentOpenAiClient() and pass in the options anyhow (no integration to look anything up from exists there)
See also #17694
Metadata
Metadata
Assignees
Labels
No labels