Merge branch 'develop' into timfish/build/dev-prod-browser-build

Author: timfish

.size-limit.js

@@ -190,14 +190,14 @@ module.exports = [
     path: createCDNPath('bundle.tracing.min.js'),
     gzip: false,
     brotli: false,
-    limit: '124.1 KB',
+    limit: '125 KB',
   },
   {
     name: 'CDN Bundle (incl. Tracing, Replay) - uncompressed',
     path: createCDNPath('bundle.tracing.replay.min.js'),
     gzip: false,
     brotli: false,
-    limit: '240 KB',
+    limit: '245 KB',
   },
   {
     name: 'CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed',

CHANGELOG.md

@@ -4,6 +4,37 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 10.23.0
+
+- feat(core): Send `user-agent` header with envelope requests in server SDKs ([#17929](https://github.com/getsentry/sentry-javascript/pull/17929))
+- feat(browser): Limit transport buffer size ([#18046](https://github.com/getsentry/sentry-javascript/pull/18046))
+- feat(core): Remove default value of `maxValueLength: 250` ([#18043](https://github.com/getsentry/sentry-javascript/pull/18043))
+- feat(react-router): Align options with shared build time options type ([#18014](https://github.com/getsentry/sentry-javascript/pull/18014))
+- fix(browser-utils): cache element names for INP ([#18052](https://github.com/getsentry/sentry-javascript/pull/18052))
+- fix(browser): Capture unhandled rejection errors for web worker integration ([#18054](https://github.com/getsentry/sentry-javascript/pull/18054))
+- fix(cloudflare): Ensure types for cloudflare handlers ([#18064](https://github.com/getsentry/sentry-javascript/pull/18064))
+- fix(nextjs): Update proxy template wrapping ([#18086](https://github.com/getsentry/sentry-javascript/pull/18086))
+- fix(nuxt): Added top-level fallback exports ([#18083](https://github.com/getsentry/sentry-javascript/pull/18083))
+- fix(nuxt): check for H3 error cause before re-capturing ([#18035](https://github.com/getsentry/sentry-javascript/pull/18035))
+- fix(replay): Linked errors not resetting session id ([#17854](https://github.com/getsentry/sentry-javascript/pull/17854))
+- fix(tracemetrics): Bump metrics buffer to 1k ([#18039](https://github.com/getsentry/sentry-javascript/pull/18039))
+- fix(vue): Make `options` parameter optional on `attachErrorHandler` ([#18072](https://github.com/getsentry/sentry-javascript/pull/18072))
+- ref(core): Set span status `internal_error` instead of `unknown_error` ([#17909](https://github.com/getsentry/sentry-javascript/pull/17909))
+
+<details>
+  <summary> <strong>Internal Changes</strong> </summary>
+
+- fix(tests): un-override nitro dep version for nuxt-3 test ([#18056](https://github.com/getsentry/sentry-javascript/pull/18056))
+- fix(e2e): Add p-map override to fix React Router 7 test builds ([#18068](https://github.com/getsentry/sentry-javascript/pull/18068))
+- feat: Add a note to save changes before starting ([#17987](https://github.com/getsentry/sentry-javascript/pull/17987))
+- test(browser): Add test for INP target name after navigation or DOM changes ([#18033](https://github.com/getsentry/sentry-javascript/pull/18033))
+- chore: Add external contributor to CHANGELOG.md ([#18032](https://github.com/getsentry/sentry-javascript/pull/18032))
+- chore(aws-serverless): Fix typo in timeout warning function name ([#18031](https://github.com/getsentry/sentry-javascript/pull/18031))
+- chore(browser): upgrade fake-indexeddb to v6 ([#17975](https://github.com/getsentry/sentry-javascript/pull/17975))
+- chore(tests): pass test flags through to the test command ([#18062](https://github.com/getsentry/sentry-javascript/pull/18062))
+
+</details>
+
 Work in this release was contributed by @hanseo0507. Thank you for your contribution!
 
 ## 10.22.0

dev-packages/browser-integration-tests/.eslintrc.js

@@ -3,7 +3,9 @@ module.exports = {
     browser: true,
     node: true,
   },
-  extends: ['../../.eslintrc.js'],
+  // todo: remove regexp plugin from here once we add it to base.js eslint config for the whole project
+  extends: ['../../.eslintrc.js', 'plugin:regexp/recommended'],
+  plugins: ['regexp'],
   ignorePatterns: [
     'suites/**/subject.js',
     'suites/**/dist/*',

dev-packages/browser-integration-tests/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry-internal/browser-integration-tests",
-  "version": "10.22.0",
+  "version": "10.23.0",
   "main": "index.js",
   "license": "MIT",
   "engines": {
@@ -43,7 +43,7 @@
     "@babel/preset-typescript": "^7.16.7",
     "@playwright/test": "~1.53.2",
     "@sentry-internal/rrweb": "2.34.0",
-    "@sentry/browser": "10.22.0",
+    "@sentry/browser": "10.23.0",
     "@supabase/supabase-js": "2.49.3",
     "axios": "^1.12.2",
     "babel-loader": "^8.2.2",
@@ -54,6 +54,7 @@
   "devDependencies": {
     "@types/glob": "8.0.0",
     "@types/node": "^18.19.1",
+    "eslint-plugin-regexp": "^1.15.0",
     "glob": "8.0.3"
   },
   "volta": {

dev-packages/browser-integration-tests/suites/integrations/webWorker/assets/worker.js

@@ -1,14 +1,34 @@
+// This worker manually replicates what Sentry.registerWebWorker() does
+// (In real code with a bundler, you'd import and call Sentry.registerWebWorker({ self }))
+
 self._sentryDebugIds = {
   'Error at http://sentry-test.io/worker.js': 'worker-debug-id-789',
 };
 
+// Send debug IDs
 self.postMessage({
   _sentryMessage: true,
   _sentryDebugIds: self._sentryDebugIds,
 });
 
+// Set up unhandledrejection handler (same as registerWebWorker)
+self.addEventListener('unhandledrejection', event => {
+  self.postMessage({
+    _sentryMessage: true,
+    _sentryWorkerError: {
+      reason: event.reason,
+      filename: self.location.href,
+    },
+  });
+});
+
 self.addEventListener('message', event => {
   if (event.data.type === 'throw-error') {
     throw new Error('Worker error for testing');
   }
+
+  if (event.data.type === 'throw-rejection') {
+    // Create an unhandled rejection
+    Promise.reject(new Error('Worker unhandled rejection'));
+  }
 });

dev-packages/browser-integration-tests/suites/integrations/webWorker/init.js

@@ -9,10 +9,17 @@ const worker = new Worker('/worker.js');
 
 Sentry.addIntegration(Sentry.webWorkerIntegration({ worker }));
 
-const btn = document.getElementById('errWorker');
+const btnError = document.getElementById('errWorker');
+const btnRejection = document.getElementById('rejectionWorker');
 
-btn.addEventListener('click', () => {
+btnError.addEventListener('click', () => {
   worker.postMessage({
     type: 'throw-error',
   });
 });
+
+btnRejection.addEventListener('click', () => {
+  worker.postMessage({
+    type: 'throw-rejection',
+  });
+});

dev-packages/browser-integration-tests/suites/integrations/webWorker/template.html

@@ -5,5 +5,6 @@
   </head>
   <body>
     <button id="errWorker">Throw error in worker</button>
+    <button id="rejectionWorker">Throw unhandled rejection in worker</button>
   </body>
 </html>

dev-packages/browser-integration-tests/suites/integrations/webWorker/test.ts

@@ -36,3 +36,32 @@ sentryTest('Assigns web worker debug IDs when using webWorkerIntegration', async
     expect(image.code_file).toEqual('http://sentry-test.io/worker.js');
   });
 });
+
+sentryTest('Captures unhandled rejections from web workers', async ({ getLocalTestUrl, page }) => {
+  const bundle = process.env.PW_BUNDLE;
+  if (bundle != null && !bundle.includes('esm') && !bundle.includes('cjs')) {
+    sentryTest.skip();
+  }
+
+  const url = await getLocalTestUrl({ testDir: __dirname });
+
+  const errorEventPromise = getFirstSentryEnvelopeRequest<Event>(page, url);
+
+  page.route('**/worker.js', route => {
+    route.fulfill({
+      path: `${__dirname}/assets/worker.js`,
+    });
+  });
+
+  const button = page.locator('#rejectionWorker');
+  await button.click();
+
+  const errorEvent = await errorEventPromise;
+
+  // Verify the unhandled rejection was captured
+  expect(errorEvent.exception?.values?.[0]?.value).toContain('Worker unhandled rejection');
+  expect(errorEvent.exception?.values?.[0]?.mechanism?.type).toBe('auto.browser.web_worker.onunhandledrejection');
+  expect(errorEvent.exception?.values?.[0]?.mechanism?.handled).toBe(false);
+  expect(errorEvent.contexts?.worker).toBeDefined();
+  expect(errorEvent.contexts?.worker?.filename).toContain('worker.js');
+});

dev-packages/browser-integration-tests/suites/manual-client/browser-context/test.ts

@@ -46,7 +46,7 @@ sentryTest('allows to setup a client manually & capture exceptions', async ({ ge
       },
     },
     contexts: {
-      trace: { trace_id: expect.stringMatching(/[a-f0-9]{32}/), span_id: expect.stringMatching(/[a-f0-9]{16}/) },
+      trace: { trace_id: expect.stringMatching(/[a-f\d]{32}/), span_id: expect.stringMatching(/[a-f\d]{16}/) },
     },
   });
 });

dev-packages/browser-integration-tests/suites/profiling/traceLifecycleMode_multiple-chunks/test.ts

@@ -58,9 +58,9 @@ sentryTest(
 
     // Required profile metadata (Sample Format V2)
     expect(typeof envelopeItemPayload1.profiler_id).toBe('string');
-    expect(envelopeItemPayload1.profiler_id).toMatch(/^[a-f0-9]{32}$/);
+    expect(envelopeItemPayload1.profiler_id).toMatch(/^[a-f\d]{32}$/);
     expect(typeof envelopeItemPayload1.chunk_id).toBe('string');
-    expect(envelopeItemPayload1.chunk_id).toMatch(/^[a-f0-9]{32}$/);
+    expect(envelopeItemPayload1.chunk_id).toMatch(/^[a-f\d]{32}$/);
     expect(envelopeItemPayload1.client_sdk).toBeDefined();
     expect(typeof envelopeItemPayload1.client_sdk.name).toBe('string');
     expect(typeof envelopeItemPayload1.client_sdk.version).toBe('string');
@@ -170,9 +170,9 @@ sentryTest(
     // Required profile metadata (Sample Format V2)
     // https://develop.sentry.dev/sdk/telemetry/profiles/sample-format-v2/
     expect(typeof envelopeItemPayload2.profiler_id).toBe('string');
-    expect(envelopeItemPayload2.profiler_id).toMatch(/^[a-f0-9]{32}$/);
+    expect(envelopeItemPayload2.profiler_id).toMatch(/^[a-f\d]{32}$/);
     expect(typeof envelopeItemPayload2.chunk_id).toBe('string');
-    expect(envelopeItemPayload2.chunk_id).toMatch(/^[a-f0-9]{32}$/);
+    expect(envelopeItemPayload2.chunk_id).toMatch(/^[a-f\d]{32}$/);
     expect(envelopeItemPayload2.client_sdk).toBeDefined();
     expect(typeof envelopeItemPayload2.client_sdk.name).toBe('string');
     expect(typeof envelopeItemPayload2.client_sdk.version).toBe('string');