Skip to content

ref(core): Add registry in Vercel ai integration - #20098

Merged
nicohrubec merged 5 commits into
developfrom
nh/vercel-ops-registry
Apr 15, 2026
Merged

ref(core): Add registry in Vercel ai integration#20098
nicohrubec merged 5 commits into
developfrom
nh/vercel-ops-registry

Conversation

@nicohrubec

@nicohrubecnicohrubec commented Apr 3, 2026

Copy link
Copy Markdown
Member

We have mappings for essentially the same thing in 3 places instead of one map to encode what we need

@github-actions

github-actionsBot commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Deps

  • Bump lodash.template from 4.5.0 to 4.18.1 by dependabot in #20085
  • Bump @xmldom/xmldom from 0.8.3 to 0.8.12 by dependabot in #20066

Other

  • (core) Support embeddings in langchain by nicohrubec in #20017
  • (core, node) Portable Express integration by isaacs in #19928
  • (deno) Add denoRuntimeMetricsIntegration by chargome in #20023
  • (node, bun) Enforce minimum collection interval in runtime metrics integrations by chargome in #20068

Bug Fixes 🐛

  • (aws-serverless) Add timeout to _endSpan forceFlush to prevent Lambda hanging by logaretm in #20064
  • (cloudflare) Ensure every request instruments functions by JPeer264 in #20044
  • (gatsby) Fix errorHandler signature to match bundler-plugin-core API by JPeer264 in #20048

Internal Changes 🔧

Core

  • Add registry in Vercel ai integration by nicohrubec in #20098
  • Do not emit spans for chats.create in google-genai by nicohrubec in #19990
  • Unify .do* span ops to gen_ai.generate_content by nicohrubec in #20074
  • Simplify addResponseAttributes in openai integration by nicohrubec in #20013
  • Extract shared endStreamSpan for AI integrations by nicohrubec in #20021
  • Remove provider-specific AI span attributes in favor of gen_ai attributes in sentry conventions by nicohrubec in #20011

Deps

  • Bump mshick/add-pr-comment from dd126dd8c253650d181ad9538d8b4fa218fc31e8 to e7516d74559b5514092f5b096ed29a629a1237c6 by dependabot in #20078
  • Bump getsentry/craft/.github/workflows/changelog-preview.yml from 2.24.1 to 2.25.2 by dependabot in #20081

Other

  • (node) Add node integration tests for Vercel ToolLoopAgent by nicohrubec in #20087
  • Update validate-pr workflow by stephanie-anderson in #20072
  • Remove unused tsconfig-template folder by mydea in #20067

🤖 This preview updates automatically when you update the PR.

@github-actions

github-actionsBot commented Apr 3, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

PathSize% ChangeChange
@sentry/browser25.78 kB--
@sentry/browser - with treeshaking flags24.27 kB--
@sentry/browser (incl. Tracing)42.77 kB--
@sentry/browser (incl. Tracing, Profiling)47.4 kB--
@sentry/browser (incl. Tracing, Replay)81.69 kB--
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags71.22 kB--
@sentry/browser (incl. Tracing, Replay with Canvas)86.39 kB--
@sentry/browser (incl. Tracing, Replay, Feedback)98.6 kB--
@sentry/browser (incl. Feedback)42.59 kB--
@sentry/browser (incl. sendFeedback)30.45 kB--
@sentry/browser (incl. FeedbackAsync)35.45 kB--
@sentry/browser (incl. Metrics)27.07 kB--
@sentry/browser (incl. Logs)27.2 kB--
@sentry/browser (incl. Metrics & Logs)27.89 kB--
@sentry/react27.53 kB--
@sentry/react (incl. Tracing)45.09 kB--
@sentry/vue30.61 kB--
@sentry/vue (incl. Tracing)44.63 kB--
@sentry/svelte25.8 kB--
CDN Bundle28.46 kB--
CDN Bundle (incl. Tracing)43.82 kB--
CDN Bundle (incl. Logs, Metrics)29.83 kB--
CDN Bundle (incl. Tracing, Logs, Metrics)44.89 kB--
CDN Bundle (incl. Replay, Logs, Metrics)68.73 kB--
CDN Bundle (incl. Tracing, Replay)80.78 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)81.83 kB--
CDN Bundle (incl. Tracing, Replay, Feedback)86.31 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)87.34 kB--
CDN Bundle - uncompressed83.12 kB--
CDN Bundle (incl. Tracing) - uncompressed129.95 kB--
CDN Bundle (incl. Logs, Metrics) - uncompressed87.27 kB--
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed133.36 kB--
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed210.63 kB--
CDN Bundle (incl. Tracing, Replay) - uncompressed247.21 kB--
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed250.6 kB--
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed260.12 kB--
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed263.51 kB--
@sentry/nextjs (client)47.52 kB--
@sentry/sveltekit (client)43.24 kB--
@sentry/node-core57.94 kB+0.02%+7 B 🔺
@sentry/node174.76 kB-0.08%-131 B 🔽
@sentry/node - without tracing97.8 kB+0.01%+8 B 🔺
@sentry/aws-serverless115.06 kB+0.01%+6 B 🔺

View base workflow run

@github-actions

Copy link
Copy Markdown
Contributor

node-overhead report 🧳

Note: This is a synthetic benchmark with a minimal express app and does not necessarily reflect the real-world performance impact in an application.

ScenarioRequests/s% of BaselinePrev. Requests/sChange %
GET Baseline8,815-11,137-21%
GET With Sentry1,71719%1,952-12%
GET With Sentry (error only)6,15570%7,510-18%
POST Baseline1,194-1,142+5%
POST With Sentry58549%561+4%
POST With Sentry (error only)1,04688%1,020+3%
MYSQL Baseline3,212-3,939-18%
MYSQL With Sentry41313%435-5%
MYSQL With Sentry (error only)2,60981%3,093-16%

View base workflow run

@nicohrubec
nicohrubec marked this pull request as ready for review April 9, 2026 13:24
Comment threadpackages/core/src/tracing/vercel-ai/index.ts

@cursorcursorBot 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 fc90b27. Configure here.

Comment threadpackages/core/src/tracing/ai/gen-ai-attributes.ts Outdated
@nicohrubec
nicohrubec merged commit 49fb4df into developApr 15, 2026
473 of 475 checks passed
@nicohrubec
nicohrubec deleted the nh/vercel-ops-registry branch April 15, 2026 09:14
mydea pushed a commit that referenced this pull request Apr 15, 2026
We have mappings for essentially the same thing in 3 places instead of
one map to encode what we need
Sign up for freeto 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.

2 participants

@nicohrubec@andreiborza