Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meta(changelog): Update changelog for 9.0.0-alpha.2 #15323

Merged
merged 25 commits into from
Feb 6, 2025

Conversation

lforst
Copy link
Member

@lforst lforst commented Feb 6, 2025

No description provided.

maxmaxme and others added 25 commits February 3, 2025 14:32
…ions` (#15246)

This pull request includes changes to enhance the configuration options
for managing Sentry source maps in a Nuxt.js project. The most important
changes include adding new options to suppress logs, handle errors
during release creation, and manage Sentry releases.

Enhancements to Sentry source maps configuration:

*
[`packages/nuxt/src/common/types.ts`](diffhunk://#diff-199725da81bbdbb2e85f3dfe2b1a2bf453b4c9755059e5050815d7fcde2f59a5R11-R41):
Added `silent`, `errorHandler`, and `release` options to the
`SourceMapsOptions` type to provide more control over logging, error
handling, and release management.
*
[`packages/nuxt/src/vite/sourceMaps.ts`](diffhunk://#diff-d511a0577f152ed6476e519722483c904cd2878586fdcd27f71c5587036bee37R96-R98):
Updated the `getPluginOptions` function to include the new `silent`,
`errorHandler`, and `release` options, allowing them to be used during
the source maps upload process.

Before submitting a pull request, please take a look at our

[Contributing](https://github.com/getsentry/sentry-javascript/blob/master/CONTRIBUTING.md)
guidelines and verify:

- [x] If you've added code that should be tested, please add tests.
- [x] Ensure your code lints and the test suite passes (`yarn lint`) &
(`yarn test`).

---------

Co-authored-by: Sigrid Huemer <[email protected]>
[Gitflow] Merge master into develop
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #15246

---------

Co-authored-by: s1gr1d <[email protected]>
Co-authored-by: Francesco Gringl-Novy <[email protected]>
This streamlines this a bit and should be good enough here.
- Related to #15008

Export helpers to avoid duplications in sentry/react-native.

- RN PR getsentry/sentry-react-native#4466
Due to a change in the lifecycle of Svelte
components in Svelte 5 (using Rune mode), our SDK can no longer leverage
the `(before|after)Update` hooks to track component update spans.
For v9, this patch therefore disables update tracking by default.
Removes a no longer necessary fallback check that we only needed in
SvelteKit 1.26.0 or older. For Kit 2.x, we can rely on the
`event.isSubRequest` flag to identify sub vs. actual requests in our
request handler.

fixes #15244
- adds a toolbar for cropping and annotations
- changes from inline styles to multiple class names in BEM format

With annotation option:
![Screenshot 2025-02-03 at 3 51
04 PM](https://github.com/user-attachments/assets/97e4ac38-4926-49e5-a6f3-d474174e3c38)

Without annotation option (to confirm that it looks the same as before):
![Screenshot 2025-02-03 at 5 09
01 PM](https://github.com/user-attachments/assets/8b614c38-3c1b-4d7e-986e-ead86a3f4349)

Closes #15252
…15288)

The debugging ability for Azure Functions when using Azure Static Web
Apps is very limited. But as I could see some Sentry-related logs I
**think** Sentry is generally initialized.

However, server-related logs don't show up in Sentry. This is
**probably** because the Azure Function finishes before the error can be
sent to Sentry.
By adding the Azure environment variable to check whether we should
flush with a timeout, this should possibly fix the issue.

Logs in Application Insights:

![image](https://github.com/user-attachments/assets/ed5ccd37-be87-4338-946b-94ec543e08c5)
…5299)

This integration is removed in v9 so we do not need this code anymore.
Noticed this by chance 😅
We use capitalized integration names everywhere, except here.

Noticed this by chance, this is kind of breaking, so probably also
should go into the migration guide - @lforst should I add this, but this
will conflict with the big PR, so maybe we just add it there directly?
This was still set differently in a few places. This PR unifies this so
we do this consistently.

For aws-serverless, we also still hard-coded the `AWSLambda` integration
name, but this is already reflected via the `Aws` and `AwsLambda`
integrations anyhow.
Rename the `hasTracingEnabled` utility function to
`hasSpansEnabled`. The original name is causing confusion and doesn't
represent what the function actually does: determining if the SDK is
configured to record spans. This function does not check if trace
propagation without span recording (or "Tracing without Performance") is
enabled (which it basically always is).
…#15314)

`trackUpdates: false` is already
default behaviour since #15265. We should rather test against this in
the e2e test app than explicitly opting out of update tracking.
When we first created the Svelte SDK, we wanted to measure how often it
was used in a SvelteKit application. Therefore we added some SvelteKit
detection logic to annotate events coming from the Svelte SDK. Since we
have a SvelteKit SDK now, we don't need to track this anymore. Saves a
few bytes, too.

This patch therefore removes the detection and event processing logic
…tegration CDN bundle (#15303)

Since we removed `debugIntegration` in v9, the `debug(.min).js` bundle
now only contains `spotlightIntegration`, which is confusing, and
doesn't follow the usual naming patter for integration CDN bundles.

This patch therefore renames the bundle to `spotlight(.min).js`.
Adds all stores to the state object. 
closes #15232

Before, it could only hold the state of the currently changed store
like:

```javascript
// before
{
    count: 0,
    name: 'Counter Store'
}
```

Now, when having a counter and a cart store, it looks like this (using
the store IDs as the object keys):

```javascript
// now
{
    cart: {
      rawItems: ['item'],
    },
    counter: {
      count: 0,
      name: 'Counter Store',
    },
  }
```
Ran both tests 100x in CI to check if the flakes are gone
([1](https://github.com/getsentry/sentry-javascript/actions/runs/13160819942/job/36729143461?pr=15311#step:14:80),
[2](https://github.com/getsentry/sentry-javascript/actions/runs/13160819942/job/36729144129?pr=15311#step:14:82)).
Seems so! The reason for the flake was that we didn't wait long enough
until the Sentry SDK was intiialized before checking for it.

closes #15269
)

I think it is fair to exclude this here, saving a few bytes...?
@lforst lforst changed the base branch from develop to master February 6, 2025 10:28
@lforst lforst requested a review from a team as a code owner February 6, 2025 10:28
@lforst lforst merged commit e714635 into master Feb 6, 2025
30 checks passed
@lforst lforst deleted the prepare-release/9.0.0-alpha.2 branch February 6, 2025 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants