Merge pull request #15703 from getsentry/prepare-release/9.6.0

meta(changelog): Update changelog for 9.6.0

Author: lforst

.craft.yml

@@ -111,6 +111,9 @@ targets:
   - name: npm
     id: '@sentry/tanstackstart'
     includeNames: /^sentry-tanstackstart-\d.*\.tgz$/
+  - name: npm
+    id: '@sentry/tanstackstart-react'
+    includeNames: /^sentry-tanstackstart-react-\d.*\.tgz$/
   - name: npm
     id: '@sentry/gatsby'
     includeNames: /^sentry-gatsby-\d.*\.tgz$/

.github/CANARY_FAILURE_TEMPLATE.md

@@ -1,7 +1,6 @@
 ---
 title: '{{ env.TITLE }}'
 labels: 'Type: Tests, Waiting for: Product Owner'
-type: 'task'
 ---
 
 Canary tests failed: {{ env.RUN_LINK }}

.github/ISSUE_TEMPLATE/bug.yml

@@ -50,6 +50,7 @@ body:
         - '@sentry/solidstart'
         - '@sentry/svelte'
         - '@sentry/sveltekit'
+        - '@sentry/tanstackstart-react'
         - '@sentry/vue'
         - '@sentry/wasm'
         - Sentry Browser Loader

CHANGELOG.md

@@ -10,6 +10,28 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 9.6.0
+
+### Important Changes
+
+- **feat(tanstackstart): Add `@sentry/tanstackstart-react` package and make `@sentry/tanstackstart` package a utility package ([#15629](https://github.com/getsentry/sentry-javascript/pull/15629))**
+
+  Since TanStack Start is supposed to be a generic framework that supports libraries like React and Solid, the `@sentry/tanstackstart` SDK package was renamed to `@sentry/tanstackstart-react` to reflect that the SDK is specifically intended to be used for React TanStack Start applications.
+  Note that the TanStack Start SDK is still in alpha status and may be subject to breaking changes in non-major package updates.
+
+### Other Changes
+
+- feat(astro): Accept all vite-plugin options ([#15638](https://github.com/getsentry/sentry-javascript/pull/15638))
+- feat(deps): bump @sentry/webpack-plugin from 3.2.1 to 3.2.2 ([#15627](https://github.com/getsentry/sentry-javascript/pull/15627))
+- feat(tanstackstart): Refine initial API ([#15574](https://github.com/getsentry/sentry-javascript/pull/15574))
+- fix(core): Ensure `fill` only patches functions ([#15632](https://github.com/getsentry/sentry-javascript/pull/15632))
+- fix(nextjs): Consider `pageExtensions` when looking for instrumentation file ([#15701](https://github.com/getsentry/sentry-javascript/pull/15701))
+- fix(remix): Null-check `options` ([#15610](https://github.com/getsentry/sentry-javascript/pull/15610))
+- fix(sveltekit): Correctly parse angle bracket type assertions for auto instrumentation ([#15578](https://github.com/getsentry/sentry-javascript/pull/15578))
+- fix(sveltekit): Guard process variable ([#15605](https://github.com/getsentry/sentry-javascript/pull/15605))
+
+Work in this release was contributed by @angelikatyborska and @nwalters512. Thank you for your contributions!
+
 ## 9.5.0
 
 ### Important Changes

MIGRATION.md

@@ -5,7 +5,7 @@ These docs walk through how to migrate our JavaScript SDKs through different maj
 - Upgrading from [SDK 4.x to 5.x/6.x](./docs/migration/v4-to-v5_v6.md)
 - Upgrading from [SDK 6.x to 7.x](./docs/migration/v6-to-v7.md)
 - Upgrading from [SDK 7.x to 8.x](./docs/migration/v7-to-v8.md)
-- Upgrading from [SDK 8.x to 9.x](./docs/migration/v8-to-v9.md#upgrading-from-7x-to-8x)
+- Upgrading from [SDK 8.x to 9.x](#upgrading-from-8x-to-9x)
 
 # Upgrading from 8.x to 9.x
 

README.md

@@ -60,6 +60,7 @@ package. Please refer to the README and instructions of those SDKs for more deta
 - [`@sentry/nestjs`](https://github.com/getsentry/sentry-javascript/tree/master/packages/nestjs): SDK for NestJS
 - [`@sentry/nextjs`](https://github.com/getsentry/sentry-javascript/tree/master/packages/nextjs): SDK for Next.js
 - [`@sentry/remix`](https://github.com/getsentry/sentry-javascript/tree/master/packages/remix): SDK for Remix
+- [`@sentry/tanstackstart-react`](https://github.com/getsentry/sentry-javascript/tree/master/packages/tanstackstart-react): SDK for TanStack Start React
 - [`@sentry/aws-serverless`](https://github.com/getsentry/sentry-javascript/tree/master/packages/aws-serverless): SDK
   for AWS Lambda Functions
 - [`@sentry/google-cloud-serverless`](https://github.com/getsentry/sentry-javascript/tree/master/packages/google-cloud-serverless):

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

@@ -43,7 +43,7 @@
     "@playwright/test": "~1.50.0",
     "@sentry-internal/rrweb": "2.34.0",
     "@sentry/browser": "9.5.0",
-    "axios": "1.7.7",
+    "axios": "1.8.2",
     "babel-loader": "^8.2.2",
     "fflate": "0.8.2",
     "html-webpack-plugin": "^5.5.0",

dev-packages/e2e-tests/test-applications/node-profiling-cjs/package.json

@@ -13,7 +13,7 @@
     "@playwright/test": "~1.50.0",
     "@sentry/node": "latest || *",
     "@sentry/profiling-node": "latest || *",
-    "esbuild": "0.20.0",
+    "esbuild": "0.25.0",
     "typescript": "^5.7.3"
   },
   "volta": {

dev-packages/e2e-tests/test-applications/nuxt-3-dynamic-import/package.json

@@ -18,7 +18,6 @@
     "nuxt": "^3.14.0"
   },
   "devDependencies": {
-    "@nuxt/test-utils": "^3.14.1",
     "@playwright/test": "~1.50.0",
     "@sentry-internal/test-utils": "link:../../../test-utils"
   },

dev-packages/e2e-tests/test-applications/nuxt-3-dynamic-import/playwright.config.ts

@@ -1,19 +1,10 @@
-import { fileURLToPath } from 'node:url';
-import type { ConfigOptions } from '@nuxt/test-utils/playwright';
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
-const nuxtConfigOptions: ConfigOptions = {
-  nuxt: {
-    rootDir: fileURLToPath(new URL('.', import.meta.url)),
-  },
-};
-
 /*  Make sure to import from '@nuxt/test-utils/playwright' in the tests
  *  Like this: import { expect, test } from '@nuxt/test-utils/playwright' */
 
 const config = getPlaywrightConfig({
   startCommand: `pnpm start`,
-  use: { ...nuxtConfigOptions },
 });
 
 export default config;

dev-packages/e2e-tests/test-applications/nuxt-3-dynamic-import/tests/errors.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForError } from '@sentry-internal/test-utils';
 
 test.describe('client-side errors', async () => {

dev-packages/e2e-tests/test-applications/nuxt-3-dynamic-import/tests/tracing.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
 import type { Span } from '@sentry/nuxt';
 

dev-packages/e2e-tests/test-applications/nuxt-3-min/package.json

@@ -20,7 +20,6 @@
     "nuxt": "3.7.0"
   },
   "devDependencies": {
-    "@nuxt/test-utils": "^3.14.1",
     "@playwright/test": "~1.50.0",
     "@sentry-internal/test-utils": "link:../../../test-utils"
   },

dev-packages/e2e-tests/test-applications/nuxt-3-min/playwright.config.ts

@@ -1,19 +1,7 @@
-import { fileURLToPath } from 'node:url';
-import type { ConfigOptions } from '@nuxt/test-utils/playwright';
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
-const nuxtConfigOptions: ConfigOptions = {
-  nuxt: {
-    rootDir: fileURLToPath(new URL('.', import.meta.url)),
-  },
-};
-
-/*  Make sure to import from '@nuxt/test-utils/playwright' in the tests
- *  Like this: import { expect, test } from '@nuxt/test-utils/playwright' */
-
 const config = getPlaywrightConfig({
   startCommand: `pnpm start:import`,
-  use: { ...nuxtConfigOptions },
 });
 
 export default config;

dev-packages/e2e-tests/test-applications/nuxt-3-min/tests/errors.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForError } from '@sentry-internal/test-utils';
 
 test.describe('client-side errors', async () => {

dev-packages/e2e-tests/test-applications/nuxt-3-min/tests/tracing.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
 import type { Span } from '@sentry/nuxt';
 

dev-packages/e2e-tests/test-applications/nuxt-3-top-level-import/package.json

@@ -18,7 +18,6 @@
     "nuxt": "^3.14.0"
   },
   "devDependencies": {
-    "@nuxt/test-utils": "^3.14.1",
     "@playwright/test": "~1.50.0",
     "@sentry-internal/test-utils": "link:../../../test-utils"
   }

dev-packages/e2e-tests/test-applications/nuxt-3-top-level-import/playwright.config.ts

@@ -1,19 +1,7 @@
-import { fileURLToPath } from 'node:url';
-import type { ConfigOptions } from '@nuxt/test-utils/playwright';
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
-const nuxtConfigOptions: ConfigOptions = {
-  nuxt: {
-    rootDir: fileURLToPath(new URL('.', import.meta.url)),
-  },
-};
-
-/*  Make sure to import from '@nuxt/test-utils/playwright' in the tests
- *  Like this: import { expect, test } from '@nuxt/test-utils/playwright' */
-
 const config = getPlaywrightConfig({
   startCommand: `pnpm start`,
-  use: { ...nuxtConfigOptions },
 });
 
 export default config;

dev-packages/e2e-tests/test-applications/nuxt-3-top-level-import/tests/errors.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForError } from '@sentry-internal/test-utils';
 
 test.describe('client-side errors', async () => {

dev-packages/e2e-tests/test-applications/nuxt-3-top-level-import/tests/tracing.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
 import type { Span } from '@sentry/nuxt';
 

dev-packages/e2e-tests/test-applications/nuxt-3/package.json

@@ -19,7 +19,6 @@
     "nuxt": "^3.14.0"
   },
   "devDependencies": {
-    "@nuxt/test-utils": "^3.14.1",
     "@playwright/test": "~1.50.0",
     "@sentry-internal/test-utils": "link:../../../test-utils"
   }

dev-packages/e2e-tests/test-applications/nuxt-3/playwright.config.ts

@@ -1,19 +1,7 @@
-import { fileURLToPath } from 'node:url';
-import type { ConfigOptions } from '@nuxt/test-utils/playwright';
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
-const nuxtConfigOptions: ConfigOptions = {
-  nuxt: {
-    rootDir: fileURLToPath(new URL('.', import.meta.url)),
-  },
-};
-
-/*  Make sure to import from '@nuxt/test-utils/playwright' in the tests
- *  Like this: import { expect, test } from '@nuxt/test-utils/playwright' */
-
 const config = getPlaywrightConfig({
   startCommand: `pnpm start:import`,
-  use: { ...nuxtConfigOptions },
 });
 
 export default config;

dev-packages/e2e-tests/test-applications/nuxt-3/tests/errors.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForError } from '@sentry-internal/test-utils';
 
 test.describe('client-side errors', async () => {

dev-packages/e2e-tests/test-applications/nuxt-3/tests/tracing.client.test.ts

@@ -1,4 +1,4 @@
-import { expect, test } from '@nuxt/test-utils/playwright';
+import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
 import type { Span } from '@sentry/nuxt';
 

dev-packages/e2e-tests/test-applications/nuxt-4/package.json

@@ -20,7 +20,6 @@
     "nuxt": "^3.13.2"
   },
   "devDependencies": {
-    "@nuxt/test-utils": "^3.14.2",
     "@playwright/test": "~1.50.0",
     "@sentry-internal/test-utils": "link:../../../test-utils"
   },

dev-packages/e2e-tests/test-applications/nuxt-4/playwright.config.ts

@@ -1,19 +1,7 @@
-import { fileURLToPath } from 'node:url';
-import type { ConfigOptions } from '@nuxt/test-utils/playwright';
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
-const nuxtConfigOptions: ConfigOptions = {
-  nuxt: {
-    rootDir: fileURLToPath(new URL('.', import.meta.url)),
-  },
-};
-
-/*  Make sure to import from '@nuxt/test-utils/playwright' in the tests
- *  Like this: import { expect, test } from '@nuxt/test-utils/playwright' */
-
 const config = getPlaywrightConfig({
   startCommand: `pnpm start:import`,
-  use: { ...nuxtConfigOptions },
 });
 
 export default config;

dev-packages/e2e-tests/test-applications/sveltekit-2/src/routes/type-assertion/+page.svelte

@@ -0,0 +1,14 @@
+<script lang="ts">
+  export let data
+</script>
+
+<h1>Type Assertion</h1>
+
+<p>
+  This route only exists to ensure we don't emit a build error because of the angle bracket type assertion in +page.ts
+  see https://github.com/getsentry/sentry-javascript/issues/9318
+</p>
+
+<p>
+  Message: {data.msg}
+</p>

dev-packages/e2e-tests/test-applications/sveltekit-2/src/routes/type-assertion/+page.ts

@@ -0,0 +1,8 @@
+export async function load() {
+  let x: unknown = 'foo';
+  return {
+    // this angle bracket type assertion threw an auto instrumentation error
+    // see: https://github.com/getsentry/sentry-javascript/issues/9318
+    msg: <string>x,
+  };
+}

dev-packages/e2e-tests/verdaccio-config/config.yaml

@@ -176,6 +176,12 @@ packages:
     unpublish: $all
     # proxy: npmjs # Don't proxy for E2E tests!
 
+  '@sentry/tanstackstart-react':
+    access: $all
+    publish: $all
+    unpublish: $all
+    # proxy: npmjs # Don't proxy for E2E tests!
+
   '@sentry/types':
     access: $all
     publish: $all

docs/new-sdk-release-checklist.md

@@ -62,6 +62,12 @@ differ slightly for other SDKs depending on how they are structured and how they
 - [ ] If the package you're adding is a dependency of fullstack framework (e.g. Remix or NextJS) SDKs, make sure that
       your package is added to the integration test apps' `"resolutions"` field in their `package.json`s.
 
+- [ ] Add the new package to the "root" README inside the repository.
+
+- [ ] Add the new package to the GitHub Issue bug template.
+
+- [ ] Create label inside the GitHub repo named "Package: foobar".
+
 ## Cutting the Release
 
 When you’re ready to make the first release, there are a couple of steps that need to be performed in the **correct

package.json

@@ -80,6 +80,7 @@
     "packages/svelte",
     "packages/sveltekit",
     "packages/tanstackstart",
+    "packages/tanstackstart-react",
     "packages/types",
     "packages/typescript",
     "packages/vercel-edge",

packages/astro/src/integration/index.ts

@@ -30,7 +30,7 @@ export const sentryAstro = (options: SentryOptions = {}): AstroIntegration => {
         };
 
         const sourceMapsNeeded = sdkEnabled.client || sdkEnabled.server;
-        const uploadOptions = options.sourceMapsUploadOptions || {};
+        const { unstable_sentryVitePluginOptions, ...uploadOptions } = options.sourceMapsUploadOptions || {};
         const shouldUploadSourcemaps = (sourceMapsNeeded && uploadOptions?.enabled) ?? true;
 
         // We don't need to check for AUTH_TOKEN here, because the plugin will pick it up from the env
@@ -68,23 +68,26 @@ export const sentryAstro = (options: SentryOptions = {}): AstroIntegration => {
                     project: uploadOptions.project ?? env.SENTRY_PROJECT,
                     authToken: uploadOptions.authToken ?? env.SENTRY_AUTH_TOKEN,
                     telemetry: uploadOptions.telemetry ?? true,
+                    _metaOptions: {
+                      telemetry: {
+                        metaFramework: 'astro',
+                      },
+                    },
+                    ...unstable_sentryVitePluginOptions,
+                    debug: options.debug ?? false,
                     sourcemaps: {
                       assets: uploadOptions.assets ?? [getSourcemapsAssetsGlob(config)],
                       filesToDeleteAfterUpload:
                         uploadOptions?.filesToDeleteAfterUpload ?? updatedFilesToDeleteAfterUpload,
+                      ...unstable_sentryVitePluginOptions?.sourcemaps,
                     },
                     bundleSizeOptimizations: {
                       ...options.bundleSizeOptimizations,
                       // TODO: with a future version of the vite plugin (probably 2.22.0) this re-mapping is not needed anymore
                       // ref: https://github.com/getsentry/sentry-javascript-bundler-plugins/pull/582
                       excludePerformanceMonitoring: options.bundleSizeOptimizations?.excludeTracing,
+                      ...unstable_sentryVitePluginOptions?.bundleSizeOptimizations,
                     },
-                    _metaOptions: {
-                      telemetry: {
-                        metaFramework: 'astro',
-                      },
-                    },
-                    debug: options.debug ?? false,
                   }),
                 ),
               ],