Minidump attached to envelope results in cleared `exceptions`

[timfish]

If a minidump is attached to an event, any existing exceptions are removed:

relay/relay-server/src/utils/native.rs

Line 68 in 6c8f3be

exceptions.clear(); // clear previous errors if any

This is an issue for a new feature in the Electron SDK:
getsentry/sentry-electron#1382

Essentially, Electron now includes JavaScript stack traces in the crashpad annotations for renderer OOM crashes. The Electron SDK parses these out of the minidump and includes them in the exceptions of the event.

Currently the JavaScript stack trace is lost:
https://sentry-sdks.sentry.io/issues/7524402832/?project=5650507

Why do we need to clear previous exceptions when adding a minidump stack trace? Can this line be removed?

[jjbayer]

Thanks @timfish, I'm not sure whether leaving pre-existing exceptions in the event would break the product. I will look into it and get back to you!

[jjbayer]

Got a draft PR open, I think we could try allowing additional exceptions if we guard it with a feature flag and only enable it for internal orgs initially.

We just need to make sure that the placeholder added by relay is the first exception, because that is what the sentry consumer code expects: https://github.com/getsentry/sentry/blob/f949db3155fcb6b79d3ee5e875b542460ed7c2c4/src/sentry/lang/native/utils.py#L149