Skip to content

ref(openai)!: Move OpenAI AI integration to @sentry/server-utils - #22943

Closed
nicohrubec wants to merge 1 commit into
ref/move-ai-integrations-to-server-utilsfrom
ref/move-ai-openai
Closed

ref(openai)!: Move OpenAI AI integration to @sentry/server-utils#22943
nicohrubec wants to merge 1 commit into
ref/move-ai-integrations-to-server-utilsfrom
ref/move-ai-openai

Conversation

@nicohrubec

Copy link
Copy Markdown
Member

Moves the OpenAI gen-ai instrumentation from @sentry/core to @sentry/server-utils/src/ai/openai, consuming the shared ai/core base added in the parent PR.

The low-level instrumentOpenAiClient / OPENAI_INTEGRATION_NAME / attribute helpers are no longer exported from @sentry/core; the platform SDKs (@sentry/node, @sentry/cloudflare, @sentry/vercel-edge) re-export them from @sentry/server-utils, so their public API is unchanged. @sentry/vercel-edge gains a @sentry/server-utils dependency.

providerSkip and core's copy of the shared base remain in @sentry/core until the final cleanup PR.

Part of the v11 AI-move stack.

🤖 Generated with Claude Code

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 84d1532. Configure here.

Comment thread packages/vercel-edge/src/index.ts Outdated
@nicohrubec
nicohrubec force-pushed the ref/move-ai-openai branch 4 times, most recently from 5af16b8 to 020e9c8 Compare August 3, 2026 11:15
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.91 kB added added
@sentry/browser - with treeshaking flags 28.11 kB added added
@sentry/browser (incl. Tracing) 47.35 kB added added
@sentry/browser (incl. Tracing + Span Streaming) 47.36 kB added added
@sentry/browser (incl. Tracing, Profiling) 52.13 kB added added
@sentry/browser (incl. Tracing, Replay) 86.69 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 76.13 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 91.41 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 104.05 kB added added
@sentry/browser (incl. Feedback) 47.22 kB added added
@sentry/browser (incl. sendFeedback) 34.76 kB added added
@sentry/browser (incl. FeedbackAsync) 39.86 kB added added
@sentry/browser (incl. Metrics) 30.99 kB added added
@sentry/browser (incl. Logs) 31.22 kB added added
@sentry/browser (incl. Metrics & Logs) 31.89 kB added added
@sentry/react 31.73 kB added added
@sentry/react (incl. Tracing) 49.61 kB added added
@sentry/vue 34.98 kB added added
@sentry/vue (incl. Tracing) 49.34 kB added added
@sentry/svelte 29.94 kB added added
CDN Bundle 31.98 kB added added
CDN Bundle (incl. Tracing) 47.72 kB added added
CDN Bundle (incl. Logs, Metrics) 33.55 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) 49.07 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) 72.9 kB added added
CDN Bundle (incl. Tracing, Replay) 85.34 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.64 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 91.13 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.44 kB added added
CDN Bundle - uncompressed 95.49 kB added added
CDN Bundle (incl. Tracing) - uncompressed 143.08 kB added added
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.2 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 147.06 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 224.96 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 262.34 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 266.3 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 276.04 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 280 kB added added
@sentry/nextjs (client) 52.2 kB added added
@sentry/sveltekit (client) 47.8 kB added added
@sentry/core/server 78.53 kB added added
@sentry/core/browser 51.96 kB added added
@sentry/node 123.28 kB added added
@sentry/node/import (ESM hook with diagnostics-channel injection) 0 B added added
@sentry/node - without tracing 85.36 kB added added
@sentry/aws-serverless 94.01 kB added added
@sentry/cloudflare (withSentry) - minified 219.34 kB added added
@sentry/cloudflare (withSentry) 540.41 kB added added

Move the OpenAI gen-ai instrumentation from `@sentry/core` to
`@sentry/server-utils/src/ai/openai`, consuming the shared `ai/core` base. The
`instrumentOpenAiClient`/`OPENAI_INTEGRATION_NAME`/etc. symbols are no longer
exported from `@sentry/core`; node/cloudflare/vercel-edge re-export them from
`@sentry/server-utils`, so the platform SDK public API is unchanged.

`providerSkip` stays in `@sentry/core` until the final cleanup PR.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec

Copy link
Copy Markdown
Member Author

Superseded by #22953 (rebased onto develop, fresh branch to fix stacked-PR linkage). Closing in favor of the new stack: #22952#22953#22954.

@nicohrubec nicohrubec closed this Aug 3, 2026
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.

1 participant