Merge pull request #16544 from getsentry/prepare-release/9.28.1

meta(changelog): Update changelog for 9.28.1

Author: Lms24

.github/actions/install-playwright/action.yml

@@ -34,7 +34,9 @@ runs:
       working-directory: ${{ inputs.cwd }}
 
     - name: Install Playwright system dependencies only (cached)
-      run: npx playwright install-deps ${{ inputs.browsers || 'chromium webkit firefox' }}
+      env:
+        PLAYWRIGHT_BROWSERS: ${{ inputs.browsers || 'chromium webkit firefox' }}
+      run: npx playwright install-deps "$PLAYWRIGHT_BROWSERS"
       if: steps.playwright-cache.outputs.cache-hit == 'true'
       shell: bash
       working-directory: ${{ inputs.cwd }}

CHANGELOG.md

@@ -4,6 +4,14 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 9.28.1
+
+- feat(deps): Bump @sentry/cli from 2.45.0 to 2.46.0 ([#16516](https://github.com/getsentry/sentry-javascript/pull/16516))
+- fix(nextjs): Avoid tracing calls to symbolication server on dev ([#16533](https://github.com/getsentry/sentry-javascript/pull/16533))
+- fix(sveltekit): Add import attribute for node exports ([#16528](https://github.com/getsentry/sentry-javascript/pull/16528))
+
+Work in this release was contributed by @eltigerchino. Thank you for your contribution!
+
 ## 9.28.0
 
 ### Important Changes

dev-packages/e2e-tests/test-applications/sveltekit-2-svelte-5/package.json

@@ -23,7 +23,7 @@
     "@sentry-internal/test-utils": "link:../../../test-utils",
     "@sentry/core": "latest || *",
     "@sveltejs/adapter-auto": "^3.0.0",
-    "@sveltejs/kit": "2.21.2",
+    "@sveltejs/kit": "^2.21.3",
     "@sveltejs/vite-plugin-svelte": "^3.0.0",
     "svelte": "^5.0.0-next.115",
     "svelte-check": "^3.6.0",

dev-packages/e2e-tests/test-applications/sveltekit-2/package.json

@@ -23,7 +23,7 @@
     "@sentry/core": "latest || *",
     "@sveltejs/adapter-auto": "^3.0.0",
     "@sveltejs/adapter-node": "^2.0.0",
-    "@sveltejs/kit": "2.21.2",
+    "@sveltejs/kit": "^2.21.3",
     "@sveltejs/vite-plugin-svelte": "^3.0.0",
     "svelte": "^4.2.8",
     "svelte-check": "^3.6.0",

dev-packages/e2e-tests/test-applications/sveltekit-cloudflare-pages/package.json

@@ -20,7 +20,7 @@
   "devDependencies": {
     "@playwright/test": "^1.45.3",
     "@sveltejs/adapter-cloudflare": "^5.0.3",
-    "@sveltejs/kit": "2.21.2",
+    "@sveltejs/kit": "^2.21.3",
     "@sveltejs/vite-plugin-svelte": "^5.0.3",
     "svelte": "^5.20.2",
     "svelte-check": "^4.1.4",

packages/core/src/types-hoist/options.ts

@@ -356,7 +356,6 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
 
   /**
    * This function can be defined to modify a child span before it's sent.
-   * Returning `null` will cause this span to be dropped.
    *
    * Note that this function is only called for child spans and not for the root span (formerly known as transaction).
    * If you want to modify or drop the root span, use {@link Options.beforeSendTransaction} instead.

packages/nextjs/src/common/devErrorSymbolicationEventProcessor.ts

@@ -192,22 +192,24 @@ async function resolveStackFrames(
     const controller = new AbortController();
     const timer = setTimeout(() => controller.abort(), 3000);
 
-    const res = await fetch(
-      `${
-        // eslint-disable-next-line no-restricted-globals
-        typeof window === 'undefined' ? 'http://localhost:3000' : '' // TODO: handle the case where users define a different port
-      }${basePath}/__nextjs_original-stack-frames`,
-      {
-        method: 'POST',
-        headers: {
-          'Content-Type': 'application/json',
+    const res = await suppressTracing(() =>
+      fetch(
+        `${
+          // eslint-disable-next-line no-restricted-globals
+          typeof window === 'undefined' ? 'http://localhost:3000' : '' // TODO: handle the case where users define a different port
+        }${basePath}/__nextjs_original-stack-frames`,
+        {
+          method: 'POST',
+          headers: {
+            'Content-Type': 'application/json',
+          },
+          signal: controller.signal,
+          body: JSON.stringify(postBody),
         },
-        signal: controller.signal,
-        body: JSON.stringify(postBody),
-      },
-    ).finally(() => {
-      clearTimeout(timer);
-    });
+      ).finally(() => {
+        clearTimeout(timer);
+      }),
+    );
 
     if (!res.ok || res.status === 204) {
       return null;

packages/react-router/package.json

@@ -39,7 +39,7 @@
     "@opentelemetry/instrumentation": "0.57.2",
     "@opentelemetry/semantic-conventions": "^1.34.0",
     "@sentry/browser": "9.28.0",
-    "@sentry/cli": "^2.45.0",
+    "@sentry/cli": "^2.46.0",
     "@sentry/core": "9.28.0",
     "@sentry/node": "9.28.0",
     "@sentry/react": "9.28.0",

packages/remix/package.json

@@ -68,7 +68,7 @@
     "@opentelemetry/instrumentation": "^0.57.2",
     "@opentelemetry/semantic-conventions": "^1.34.0",
     "@remix-run/router": "1.x",
-    "@sentry/cli": "^2.45.0",
+    "@sentry/cli": "^2.46.0",
     "@sentry/core": "9.28.0",
     "@sentry/node": "9.28.0",
     "@sentry/opentelemetry": "9.28.0",

packages/sveltekit/package.json

@@ -28,7 +28,10 @@
         "import": "./build/esm/index.client.js",
         "require": "./build/cjs/index.client.js"
       },
-      "node": "./build/cjs/index.server.js"
+      "node": {
+        "import": "./build/esm/index.server.js",
+        "require": "./build/cjs/index.server.js"
+      }
     }
   },
   "publishConfig": {
@@ -83,5 +86,6 @@
   },
   "volta": {
     "extends": "../../package.json"
-  }
+  },
+  "sideEffects": false
 }