Skip to content

Conversation

@isaacs
Copy link
Member

@isaacs isaacs commented Nov 26, 2025

Fix JS-1216

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

@linear
Copy link

linear bot commented Nov 26, 2025

@isaacs isaacs requested a review from RulaKhaled November 26, 2025 07:02
isaacs added a commit that referenced this pull request Nov 26, 2025
@isaacs isaacs force-pushed the isaacschlueter/js-1216-js-vercel-ai-sdk-attributes-missing branch from e14c708 to 830aa2e Compare November 26, 2025 07:03
@github-actions
Copy link
Contributor

github-actions bot commented Nov 26, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 24.8 kB - -
@sentry/browser - with treeshaking flags 23.31 kB - -
@sentry/browser (incl. Tracing) 41.54 kB - -
@sentry/browser (incl. Tracing, Profiling) 46.13 kB - -
@sentry/browser (incl. Tracing, Replay) 79.96 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 69.69 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 84.64 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 96.88 kB - -
@sentry/browser (incl. Feedback) 41.48 kB - -
@sentry/browser (incl. sendFeedback) 29.49 kB - -
@sentry/browser (incl. FeedbackAsync) 34.43 kB - -
@sentry/react 26.52 kB - -
@sentry/react (incl. Tracing) 43.74 kB - -
@sentry/vue 29.25 kB - -
@sentry/vue (incl. Tracing) 43.34 kB - -
@sentry/svelte 24.82 kB - -
CDN Bundle 27.17 kB - -
CDN Bundle (incl. Tracing) 42.16 kB - -
CDN Bundle (incl. Tracing, Replay) 78.7 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 84.16 kB - -
CDN Bundle - uncompressed 79.84 kB - -
CDN Bundle (incl. Tracing) - uncompressed 125.22 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 241.25 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 254.01 kB - -
@sentry/nextjs (client) 45.96 kB - -
@sentry/sveltekit (client) 41.9 kB - -
@sentry/node-core 51.19 kB - -
@sentry/node 159.48 kB +0.08% +121 B 🔺
@sentry/node - without tracing 92.82 kB - -
@sentry/aws-serverless 108.08 kB +0.01% +1 B 🔺

View base workflow run

@github-actions
Copy link
Contributor

github-actions bot commented Nov 26, 2025

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.

Scenario Requests/s % of Baseline Prev. Requests/s Change %
GET Baseline 8,597 - 8,923 -4%
GET With Sentry 1,652 19% 1,727 -4%
GET With Sentry (error only) 5,958 69% 6,129 -3%
POST Baseline 1,201 - 1,204 -0%
POST With Sentry 566 47% 602 -6%
POST With Sentry (error only) 1,037 86% 1,075 -4%
MYSQL Baseline 3,174 - 3,375 -6%
MYSQL With Sentry 420 13% 503 -17%
MYSQL With Sentry (error only) 2,615 82% 2,737 -4%

View base workflow run

Also, sort the fields in the integration test, in order to more easily
se which fields were missing/misnamed in the fixture objects.

Fix JS-1216
@isaacs isaacs force-pushed the isaacschlueter/js-1216-js-vercel-ai-sdk-attributes-missing branch from 830aa2e to f5b9b31 Compare November 26, 2025 22:39
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.

2 participants