Merge pull request #13284 from getsentry/prepare-release/8.25.0

meta(changelog): Update changelog for 8.25.0
getsentry · Aug 9, 2024 · ccdc390 · ccdc390
2 parents 6e94661 + 476a51b
commit ccdc390
Show file tree

Hide file tree

Showing 88 changed files with 2,261 additions and 573 deletions.
diff --git a/.craft.yml b/.craft.yml
@@ -114,6 +114,9 @@ targets:
   - name: npm
     id: '@sentry/remix'
     includeNames: /^sentry-remix-\d.*\.tgz$/
+  - name: npm
+    id: '@sentry/solidstart'
+    includeNames: /^sentry-solidstart-\d.*\.tgz$/
   - name: npm
     id: '@sentry/sveltekit'
     includeNames: /^sentry-sveltekit-\d.*\.tgz$/

diff --git a/.github/ISSUE_TEMPLATE/bug.yml b/.github/ISSUE_TEMPLATE/bug.yml
@@ -1,6 +1,6 @@
 name: 🐞 Bug Report
 description: Tell us about something that's not working the way we (probably) intend.
-labels: ['Type: Bug']
+type: 'bug'
 body:
   - type: checkboxes
     attributes:
@@ -31,20 +31,23 @@ body:
         setup.
       options:
         - '@sentry/browser'
-        - '@sentry/astro'
+        - '@sentry/node'
         - '@sentry/angular'
+        - '@sentry/astro'
         - '@sentry/aws-serverless'
         - '@sentry/bun'
+        - '@sentry/cloudflare'
         - '@sentry/deno'
         - '@sentry/ember'
         - '@sentry/gatsby'
         - '@sentry/google-cloud-serverless'
         - '@sentry/nestjs'
         - '@sentry/nextjs'
-        - '@sentry/node'
+        - '@sentry/nuxt'
         - '@sentry/react'
         - '@sentry/remix'
         - '@sentry/solid'
+        - '@sentry/solidstart'
         - '@sentry/svelte'
         - '@sentry/sveltekit'
         - '@sentry/vue'

diff --git a/.github/ISSUE_TEMPLATE/feature.yml b/.github/ISSUE_TEMPLATE/feature.yml
@@ -1,6 +1,6 @@
 name: 💡 Feature Request
 description: Create a feature request for a sentry-javascript SDK.
-labels: ['Type: Improvement']
+type: 'enhancement'
 body:
   - type: markdown
     attributes:

diff --git a/.github/ISSUE_TEMPLATE/flaky.yml b/.github/ISSUE_TEMPLATE/flaky.yml
@@ -1,6 +1,7 @@
 name: ❅ Flaky Test
 description: Report a flaky test in CI
 title: '[Flaky CI]: '
+type: 'task'
 labels: ['Type: Tests']
 body:
   - type: dropdown

diff --git a/.github/ISSUE_TEMPLATE/internal.yml b/.github/ISSUE_TEMPLATE/internal.yml
@@ -1,6 +1,10 @@
 name: 💡 [Internal] Blank Issue
 description: Only for Sentry Employees! Create an issue without a template.
+type: 'task'
 body:
+  - type: markdown
+    attributes:
+      value: Make sure to apply relevant labels and issue types before submitting.
   - type: textarea
     id: description
     attributes:

diff --git a/.github/actions/install-playwright/action.yml b/.github/actions/install-playwright/action.yml
@@ -1,5 +1,9 @@
 name: "Install Playwright dependencies"
 description: "Installs Playwright dependencies and caches them."
+inputs:
+  browsers:
+    description: 'What browsers to install.'
+    default: 'chromium webkit firefox'
 
 runs:
   using: "composite"
@@ -17,12 +21,13 @@ runs:
             ~/.cache/ms-playwright
           key: playwright-${{ runner.os }}-${{ steps.playwright-version.outputs.version }}
 
+      # We always install all browsers, if uncached
       - name: Install Playwright dependencies (uncached)
         run: npx playwright install chromium webkit firefox --with-deps
         if: steps.playwright-cache.outputs.cache-hit != 'true'
         shell: bash
 
       - name: Install Playwright system dependencies only (cached)
-        run: npx playwright install-deps chromium webkit firefox
+        run: npx playwright install-deps ${{ inputs.browsers || 'chromium webkit firefox' }}
         if: steps.playwright-cache.outputs.cache-hit == 'true'
         shell: bash
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -528,7 +528,7 @@ jobs:
         run: yarn lerna run test --scope @sentry/profiling-node
 
   job_browser_playwright_tests:
-    name: Playwright (${{ matrix.bundle }}${{ matrix.shard && format(' {0}/{1}', matrix.shard, matrix.shards) || ''}}) Tests
+    name: Playwright ${{ matrix.bundle }}${{ matrix.project && matrix.project != 'chromium' && format(' {0}', matrix.project) || ''}}${{ matrix.shard && format(' ({0}/{1})', matrix.shard, matrix.shards) || ''}} Tests
     needs: [job_get_metadata, job_build]
     if: needs.job_build.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04-large-js
@@ -548,31 +548,30 @@ jobs:
         project:
           - chromium
         include:
-          # Only check all projects for esm & full bundle
+          # Only check all projects for full bundle
           # We also shard the tests as they take the longest
           - bundle: bundle_tracing_replay_feedback_min
-            project: ''
-            shard: 1
-            shards: 2
+            project: 'webkit'
           - bundle: bundle_tracing_replay_feedback_min
-            project: ''
-            shard: 2
-            shards: 2
+            project: 'firefox'
           - bundle: esm
-            project: ''
+            project: chromium
             shard: 1
-            shards: 3
+            shards: 4
           - bundle: esm
+            project: chromium
             shard: 2
-            shards: 3
+            shards: 4
           - bundle: esm
-            project: ''
+            project: chromium
             shard: 3
-            shards: 3
+            shards: 4
+          - bundle: esm
+            project: chromium
+            shard: 4
+            shards: 4
         exclude:
-          # Do not run the default chromium-only tests
-          - bundle: bundle_tracing_replay_feedback_min
-            project: 'chromium'
+          # Do not run the un-sharded esm tests
           - bundle: esm
             project: 'chromium'
 
@@ -592,12 +591,15 @@ jobs:
 
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: ${{ matrix.project }}
 
       - name: Run Playwright tests
         env:
           PW_BUNDLE: ${{ matrix.bundle }}
         working-directory: dev-packages/browser-integration-tests
         run: yarn test:ci${{ matrix.project && format(' --project={0}', matrix.project) || '' }}${{ matrix.shard && format(' --shard={0}/{1}', matrix.shard, matrix.shards) || '' }}
+
       - name: Upload Playwright Traces
         uses: actions/upload-artifact@v3
         if: always()
@@ -606,7 +608,7 @@ jobs:
           path: dev-packages/browser-integration-tests/test-results
 
   job_browser_loader_tests:
-    name: Playwright Loader (${{ matrix.bundle }}) Tests
+    name: PW ${{ matrix.bundle }} Tests
     needs: [job_get_metadata, job_build]
     if: needs.job_build.outputs.changed_browser_integration == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-20.04
@@ -639,6 +641,8 @@ jobs:
 
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: chromium
 
       - name: Run Playwright Loader tests
         env:
@@ -750,8 +754,12 @@ jobs:
         uses: ./.github/actions/restore-cache
         env:
           DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
+
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: chromium
+
       - name: Run integration tests
         env:
           NODE_VERSION: ${{ matrix.node }}
@@ -878,6 +886,7 @@ jobs:
             'react-router-5',
             'react-router-6',
             'solid',
+            'solidstart',
             'svelte-5',
             'sveltekit',
             'sveltekit-2',
@@ -952,6 +961,8 @@ jobs:
 
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: chromium
 
       - name: Get node version
         id: versions
@@ -1049,6 +1060,8 @@ jobs:
 
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: chromium
 
       - name: Get node version
         id: versions
@@ -1149,6 +1162,8 @@ jobs:
 
       - name: Install Playwright
         uses: ./.github/actions/install-playwright
+        with:
+          browsers: chromium
 
       - name: Get node version
         id: versions

diff --git a/.github/workflows/flaky-test-detector.yml b/.github/workflows/flaky-test-detector.yml
@@ -68,10 +68,10 @@ jobs:
           CHANGED_TEST_PATHS: ${{ steps.changed.outputs.browser_integration_files }}
           TEST_RUN_COUNT: 'AUTO'
 
-      - name: Artifacts upload
+      - name: Upload Playwright Traces
         uses: actions/upload-artifact@v4
         if: failure() && steps.test.outcome == 'failure'
         with:
           name: playwright-test-results
-          path: test-results
+          path: dev-packages/browser-integration-tests/test-results
           retention-days: 5
diff --git a/.github/workflows/issue-package-label.yml b/.github/workflows/issue-package-label.yml
@@ -29,45 +29,66 @@ jobs:
           # Note: Since this is handled as a regex, and JSON parse wrangles slashes /, we just use `.` instead
           map: |
             {
+              "@sentry.angular": {
+                "label": "Package: angular"
+              },
               "@sentry.astro": {
-                "label": "Package: Astro"
+                "label": "Package: astro"
               },
-              "@sentry.browser": {
-                "label": "Package: Browser"
+              "@sentry.aws-serverless": {
+                "label": "Package: aws-serverless"
               },
-              "@sentry.angular": {
-                "label": "Package: Angular"
+              "@sentry.browser": {
+                "label": "Package: browser"
               },
               "@sentry.bun": {
-                "label": "Package: Bun"
+                "label": "Package: bun"
+              },
+              "@sentry.cloudflare": {
+                "label": "Package: cloudflare"
+              },
+              "@sentry.deno": {
+                "label": "Package: deno"
               },
               "@sentry.ember": {
                 "label": "Package: ember"
               },
               "@sentry.gatsby": {
                 "label": "Package: gatbsy"
               },
+              "@sentry.google-cloud-serverless": {
+                "label": "Package: google-cloud-serverless"
+              },
+              "@sentry.nestjs": {
+                "label": "Package: nestjs"
+              },
               "@sentry.nextjs": {
-                "label": "Package: Nextjs"
+                "label": "Package: nextjs"
               },
               "@sentry.node": {
-                "label": "Package: Node"
+                "label": "Package: node"
+              },
+              "@sentry.nuxt": {
+                "label": "Package: nuxt"
               },
               "@sentry.react": {
                 "label": "Package: react"
               },
               "@sentry.remix": {
                 "label": "Package: remix"
               },
-              "@sentry.serverless": {
-                "label": "Package: Serverless"
+              "@sentry.solid": {
+                "label": "Package: solid"
               },
-              "@sentry.sveltekit": {
-                "label": "Package: SvelteKit"
+              "@sentry.solid": {
+                "label": "Package: solidstart"
               },
               "@sentry.svelte": {
                 "label": "Package: svelte"
               },
+              "@sentry.sveltekit": {
+                "label": "Package: sveltekit"
+              },
               "@sentry.vue": {
                 "label": "Package: vue"
               },

diff --git a/.size-limit.js b/.size-limit.js
@@ -22,7 +22,7 @@ module.exports = [
     path: 'packages/browser/build/npm/esm/index.js',
     import: createImport('init', 'browserTracingIntegration', 'replayIntegration'),
     gzip: true,
-    limit: '72 KB',
+    limit: '73 KB',
   },
   {
     name: '@sentry/browser (incl. Tracing, Replay) - with treeshaking flags',

diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -10,6 +10,28 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 8.25.0
+
+### Important Changes
+
+- **Alpha release of Official Solid Start SDK**
+
+This release contains the alpha version of `@sentry/solidstart`, our SDK for [Solid Start](https://start.solidjs.com/)!
+For details on how to use it, please see the [README](./packages/solidstart/README.md). Any feedback/bug reports are
+greatly appreciated, please [reach out on GitHub](https://github.com/getsentry/sentry-javascript/issues/12538).
+
+### Other Changes
+
+- feat(astro): Add `bundleSizeOptimizations` vite options to integration (#13250)
+- feat(astro): Always add BrowserTracing (#13244)
+- feat(core): Add `getTraceMetaTags` function (#13201)
+- feat(nestjs): Automatic instrumentation of nestjs exception filters (#13230)
+- feat(node): Add `useOperationNameForRootSpan` to`graphqlIntegration` (#13248)
+- feat(sveltekit): Add `wrapServerRouteWithSentry` wrapper (#13247)
+- fix(aws-serverless): Extract sentry trace data from handler `context` over `event` (#13266)
+- fix(browser): Initialize default integration if `defaultIntegrations: undefined` (#13261)
+- fix(utils): Streamline IP capturing on incoming requests (#13272)
+
 ## 8.24.0
 
 - feat(nestjs): Filter RPC exceptions (#13227)