Merge pull request #9239 from getsentry/prepare-release/7.74.0

meta: Update CHANGELOG for 7.74.0
getsentry · Oct 13, 2023 · 6a0f478 · 6a0f478
2 parents 02e3b9c + 85ac5e3
commit 6a0f478
Show file tree

Hide file tree

Showing 226 changed files with 13,048 additions and 1,319 deletions.
diff --git a/.craft.yml b/.craft.yml
@@ -88,6 +88,9 @@ targets:
   - name: npm
     id: '@sentry/gatsby'
     includeNames: /^sentry-gatsby-\d.*\.tgz$/
+  - name: npm
+    id: '@sentry/astro'
+    includeNames: /^sentry-astro-\d.*\.tgz$/
 
   ## 7. Other Packages
   ## 7.1
@@ -180,3 +183,5 @@ targets:
         onlyIfPresent: /^sentry-bun-\d.*\.tgz$/
       'npm:@sentry/vercel-edge':
         onlyIfPresent: /^sentry-vercel-edge-\d.*\.tgz$/
+      'npm:@sentry/ember':
+        onlyIfPresent: /^sentry-ember-\d.*\.tgz$/
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -356,7 +356,7 @@ jobs:
       - name: Pack
         run: yarn build:tarball
       - name: Archive artifacts
-        uses: actions/upload-artifact@v3.1.2
+        uses: actions/upload-artifact@v3.1.3
         with:
           name: ${{ github.sha }}
           path: |
@@ -379,7 +379,7 @@ jobs:
       - name: Set up Node
         uses: actions/setup-node@v3
         with:
-          node-version: ${{ env.DEFAULT_NODE_VERSION }}
+          node-version-file: 'package.json'
       - name: Restore caches
         uses: ./.github/actions/restore-cache
         env:
@@ -406,7 +406,7 @@ jobs:
       - name: Set up Node
         uses: actions/setup-node@v3
         with:
-          node-version: ${{ matrix.node }}
+          node-version-file: 'package.json'
       - name: Set up Bun
         uses: oven-sh/setup-bun@v1
       - name: Restore caches
@@ -419,6 +419,38 @@ jobs:
       - name: Compute test coverage
         uses: codecov/codecov-action@v3
 
+  job_deno_unit_tests:
+    name: Deno Unit Tests
+    needs: [job_get_metadata, job_build]
+    timeout-minutes: 10
+    runs-on: ubuntu-20.04
+    strategy:
+      fail-fast: false
+    steps:
+      - name: Check out current commit (${{ needs.job_get_metadata.outputs.commit_label }})
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ env.HEAD_COMMIT }}
+      - name: Set up Node
+        uses: actions/setup-node@v3
+        with:
+          node-version-file: 'package.json'
+      - name: Set up Deno
+        uses: denoland/setup-deno@v1.1.3
+        with:
+          deno-version: v1.37.1
+      - name: Restore caches
+        uses: ./.github/actions/restore-cache
+        env:
+          DEPENDENCY_CACHE_KEY: ${{ needs.job_build.outputs.dependency_cache_key }}
+      - name: Run tests
+        run: |
+          cd packages/deno
+          yarn build
+          yarn test
+      - name: Compute test coverage
+        uses: codecov/codecov-action@v3
+
   job_node_unit_tests:
     name: Node (${{ matrix.node }}) Unit Tests
     needs: [job_get_metadata, job_build]
@@ -823,6 +855,7 @@ jobs:
             'standard-frontend-react-tracing-import',
             'sveltekit',
             'generic-ts3.8',
+            'node-experimental-fastify-app',
           ]
         build-command:
           - false
@@ -894,6 +927,7 @@ jobs:
         job_browser_build_tests,
         job_browser_unit_tests,
         job_bun_unit_tests,
+        job_deno_unit_tests,
         job_node_unit_tests,
         job_nextjs_integration_test,
         job_node_integration_tests,
@@ -951,7 +985,7 @@ jobs:
           GITHUB_TOKEN: ${{ github.token }}
 
       - name: Upload results
-        uses: actions/upload-artifact@v3.1.2
+        uses: actions/upload-artifact@v3.1.3
         if: github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository
         with:
           name: ${{ steps.process.outputs.artifactName }}

diff --git a/.gitignore b/.gitignore
@@ -21,6 +21,8 @@ jest/transformers/*.js
 # node tarballs
 packages/*/sentry-*.tgz
 .nxcache
+# The Deno types are downloaded before building
+packages/deno/lib.deno.d.ts
 
 # logs
 yarn-error.log

diff --git a/.vscode/extensions.json b/.vscode/extensions.json
@@ -4,6 +4,7 @@
   "recommendations": [
     "esbenp.prettier-vscode",
     "dbaeumer.vscode-eslint",
-    "augustocdias.tasks-shell-input"
-  ],
+    "augustocdias.tasks-shell-input",
+    "denoland.vscode-deno"
+  ]
 }
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -28,5 +28,6 @@
     {
       "mode": "auto"
     }
-  ]
+  ],
+  "deno.enablePaths": ["packages/deno/test"]
 }
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,67 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 7.74.0
+
+### Important Changes
+
+- **feat(astro): Add `sentryAstro` integration (#9218)**
+
+This Release introduces the first alpha version of our new SDK for Astro.
+At this time, the SDK is considered experimental and things might break and change in future versions.
+
+The core of the SDK is an Astro integration which you easily add to your Astro config:
+
+```js
+// astro.config.js
+import { defineConfig } from "astro/config";
+import sentry from "@sentry/astro";
+
+export default defineConfig({
+  integrations: [
+    sentry({
+      dsn: "__DSN__",
+      sourceMapsUploadOptions: {
+        project: "astro",
+        authToken: process.env.SENTRY_AUTH_TOKEN,
+      },
+    }),
+  ],
+});
+```
+
+Check out the [README](./packages/astro/README.md) for usage instructions and what to expect from this alpha release.
+
+### Other Changes
+
+- feat(core): Add `addIntegration` utility (#9186)
+- feat(core): Add `continueTrace` method (#9164)
+- feat(node-experimental): Add NodeFetch integration (#9226)
+- feat(node-experimental): Use native OTEL Spans (#9161, #9214)
+- feat(node-experimental): Sample in OTEL Sampler (#9203)
+- feat(serverlesss): Allow disabling transaction traces (#9154)
+- feat(tracing): Allow direct pg module to enable esbuild support (#9227)
+- feat(utils): Move common node ANR code to utils (#9191)
+- feat(vue): Expose `VueIntegration` to initialize vue app later (#9180)
+- fix: Don't set `referrerPolicy` on serverside fetch transports (#9200)
+- fix: Ensure we never mutate options passed to `init` (#9162)
+- fix(ember): Avoid pulling in utils at build time (#9221)
+- fix(ember): Drop undefined config values (#9175)
+- fix(node): Ensure mysql integration works without callback (#9222)
+- fix(node): Only require `inspector` when needed (#9149)
+- fix(node): Remove ANR `debug` option and instead add logger.isEnabled() (#9230)
+- fix(node): Strip `.mjs` and `.cjs` extensions from module name (#9231)
+- fix(replay): bump rrweb to 2.0.1 (#9240)
+- fix(replay): Fix potential broken CSS in styled-components (#9234)
+- fix(sveltekit): Flush in server wrappers before exiting (#9153)
+- fix(types): Update signature of `processEvent` integration hook (#9151)
+- fix(utils): Dereference DOM events after they have servered their purpose (#9224)
+- ref(integrations): Refactor pluggable integrations to use `processEvent` (#9021)
+- ref(serverless): Properly deprecate `rethrowAfterCapture` option (#9159)
+- ref(utils): Deprecate `walk` method (#9157)
+
+Work in this release contributed by @aldenquimby. Thank you for your contributions!
+
 ## 7.73.0
 
 ### Important Changes

diff --git a/package.json b/package.json
@@ -27,23 +27,25 @@
     "postpublish": "lerna run --stream --concurrency 1 postpublish",
     "test": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test",
     "test:unit": "lerna run --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\" test:unit",
-    "test-ci-browser": "lerna run test --ignore \"@sentry/{bun,node,node-experimental,opentelemetry-node,serverless,nextjs,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
+    "test-ci-browser": "lerna run test --ignore \"@sentry/{bun,deno,node,node-experimental,opentelemetry-node,serverless,nextjs,remix,gatsby,sveltekit,vercel-edge}\" --ignore \"@sentry-internal/{browser-integration-tests,e2e-tests,integration-shims,node-integration-tests,overhead-metrics}\"",
     "test-ci-node": "ts-node ./scripts/node-unit-tests.ts",
     "test-ci-bun": "lerna run test --scope @sentry/bun",
     "test:update-snapshots": "lerna run test:update-snapshots",
     "yalc:publish": "lerna run yalc:publish"
   },
   "volta": {
-    "node": "16.19.0",
+    "node": "18.17.0",
     "yarn": "1.22.19"
   },
   "workspaces": [
     "packages/angular",
     "packages/angular-ivy",
+    "packages/astro",
     "packages/browser",
     "packages/browser-integration-tests",
     "packages/bun",
     "packages/core",
+    "packages/deno",
     "packages/e2e-tests",
     "packages/ember",
     "packages/eslint-config-sdk",
@@ -126,7 +128,8 @@
     "yalc": "^1.0.0-pre.53"
   },
   "resolutions": {
-    "**/agent-base": "5"
+    "**/agent-base": "5",
+    "**/terser/source-map": "0.7.4"
   },
   "version": "0.0.0",
   "name": "sentry-javascript"

diff --git a/packages/angular-ivy/src/sdk.ts b/packages/angular-ivy/src/sdk.ts
@@ -1,6 +1,7 @@
 import { VERSION } from '@angular/core';
 import type { BrowserOptions } from '@sentry/browser';
 import { defaultIntegrations, init as browserInit, SDK_VERSION, setContext } from '@sentry/browser';
+import type { SdkMetadata } from '@sentry/types';
 import { logger } from '@sentry/utils';
 
 import { IS_DEBUG_BUILD } from './flags';
@@ -9,8 +10,21 @@ import { IS_DEBUG_BUILD } from './flags';
  * Inits the Angular SDK
  */
 export function init(options: BrowserOptions): void {
-  options._metadata = options._metadata || {};
-  options._metadata.sdk = {
+  const opts = {
+    _metadata: {} as SdkMetadata,
+    // Filter out TryCatch integration as it interferes with our Angular `ErrorHandler`:
+    // TryCatch would catch certain errors before they reach the `ErrorHandler` and thus provide a
+    // lower fidelity error than what `SentryErrorHandler` (see errorhandler.ts) would provide.
+    // see:
+    //  - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097
+    //  - https://github.com/getsentry/sentry-javascript/issues/2744
+    defaultIntegrations: defaultIntegrations.filter(integration => {
+      return integration.name !== 'TryCatch';
+    }),
+    ...options,
+  };
+
+  opts._metadata.sdk = opts._metadata.sdk || {
     name: 'sentry.javascript.angular-ivy',
     packages: [
       {
@@ -21,20 +35,8 @@ export function init(options: BrowserOptions): void {
     version: SDK_VERSION,
   };
 
-  // Filter out TryCatch integration as it interferes with our Angular `ErrorHandler`:
-  // TryCatch would catch certain errors before they reach the `ErrorHandler` and thus provide a
-  // lower fidelity error than what `SentryErrorHandler` (see errorhandler.ts) would provide.
-  // see:
-  //  - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097
-  //  - https://github.com/getsentry/sentry-javascript/issues/2744
-  if (options.defaultIntegrations === undefined) {
-    options.defaultIntegrations = defaultIntegrations.filter(integration => {
-      return integration.name !== 'TryCatch';
-    });
-  }
-
   checkAndSetAngularVersion();
-  browserInit(options);
+  browserInit(opts);
 }
 
 function checkAndSetAngularVersion(): void {

diff --git a/packages/angular/src/sdk.ts b/packages/angular/src/sdk.ts
@@ -1,6 +1,7 @@
 import { VERSION } from '@angular/core';
 import type { BrowserOptions } from '@sentry/browser';
 import { defaultIntegrations, init as browserInit, SDK_VERSION, setContext } from '@sentry/browser';
+import type { SdkMetadata } from '@sentry/types';
 import { logger } from '@sentry/utils';
 
 import { IS_DEBUG_BUILD } from './flags';
@@ -9,8 +10,21 @@ import { IS_DEBUG_BUILD } from './flags';
  * Inits the Angular SDK
  */
 export function init(options: BrowserOptions): void {
-  options._metadata = options._metadata || {};
-  options._metadata.sdk = {
+  const opts = {
+    _metadata: {} as SdkMetadata,
+    // Filter out TryCatch integration as it interferes with our Angular `ErrorHandler`:
+    // TryCatch would catch certain errors before they reach the `ErrorHandler` and thus provide a
+    // lower fidelity error than what `SentryErrorHandler` (see errorhandler.ts) would provide.
+    // see:
+    //  - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097
+    //  - https://github.com/getsentry/sentry-javascript/issues/2744
+    defaultIntegrations: defaultIntegrations.filter(integration => {
+      return integration.name !== 'TryCatch';
+    }),
+    ...options,
+  };
+
+  opts._metadata.sdk = opts._metadata.sdk || {
     name: 'sentry.javascript.angular',
     packages: [
       {
@@ -21,20 +35,8 @@ export function init(options: BrowserOptions): void {
     version: SDK_VERSION,
   };
 
-  // Filter out TryCatch integration as it interferes with our Angular `ErrorHandler`:
-  // TryCatch would catch certain errors before they reach the `ErrorHandler` and thus provide a
-  // lower fidelity error than what `SentryErrorHandler` (see errorhandler.ts) would provide.
-  // see:
-  //  - https://github.com/getsentry/sentry-javascript/issues/5417#issuecomment-1453407097
-  //  - https://github.com/getsentry/sentry-javascript/issues/2744
-  if (options.defaultIntegrations === undefined) {
-    options.defaultIntegrations = defaultIntegrations.filter(integration => {
-      return integration.name !== 'TryCatch';
-    });
-  }
-
   checkAndSetAngularVersion();
-  browserInit(options);
+  browserInit(opts);
 }
 
 function checkAndSetAngularVersion(): void {

diff --git a/packages/astro/.eslintrc.cjs b/packages/astro/.eslintrc.cjs
@@ -0,0 +1,22 @@
+module.exports = {
+  env: {
+    browser: true,
+    node: true,
+  },
+  extends: ['../../.eslintrc.js'],
+  overrides: [
+    {
+      files: ['vite.config.ts'],
+      parserOptions: {
+        project: ['tsconfig.test.json'],
+      },
+    },
+    {
+      files: ['src/integration/**', 'src/server/**'],
+      rules: {
+        '@sentry-internal/sdk/no-optional-chaining': 'off',
+        '@sentry-internal/sdk/no-nullish-coalescing': 'off',
+      },
+    },
+  ],
+};
diff --git a/packages/astro/.npmignore b/packages/astro/.npmignore
@@ -0,0 +1,10 @@
+# The paths in this file are specified so that they align with the file structure in `./build` after this file is copied
+# into it by the prepack script `scripts/prepack.ts`.
+
+*
+
+!/cjs/**/*
+!/esm/**/*
+!/types/**/*
+!/types-ts3.8/**/*
+!/integration/**/*
diff --git a/packages/astro/LICENSE b/packages/astro/LICENSE
@@ -0,0 +1,14 @@
+Copyright (c) 2023 Sentry (https://sentry.io) and individual contributors. All rights reserved.
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated
+documentation files (the "Software"), to deal in the Software without restriction, including without limitation the
+rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit
+persons to whom the Software is furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all copies or substantial portions of the
+Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE
+WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
+COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR
+OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.