Add overload to the public API to take Action<Scope>

[bruno-garcia]

On the IHub methods like CaptureEvent, CaptureMessage and CaptureException should have an overload that takes: Action<Scope>.

The implementation should do what WithScope does, it pushes a scope, and passes the instance to the callback to be mutated. The result is then applied (normal code path of CaptureEvent(event, scope).

This serves as a replacement to the api:

SentrySdk.WithScope(s =>
{
	s...;
	// Odd call to static method in callback. Data passed magically through the callback
	SentrySdk.CaptureX 
});

This exists on sentry-cocoa:
https://github.com/getsentry/sentry-cocoa/blob/29d1e9f2353f1a9514b2549b1de56e83f136658d/Sources/Sentry/SentrySDK.m#L146-L151

[bruno-garcia]

Suggestion to tackle this:

1. Start with a draft PR touching only the main static API: SentrySdk.
2. Add the overloads to all methods
3. Add Obsolete to WithScope
4. Amend tests

The goal is to verify the effort to adding the thing and the impact in the API.
Even though this was discussed internally and added to some SDKs, seems like we don't have it on our develop docs:

• https://develop.sentry.dev/sdk/unified-api/#scope
• https://develop.sentry.dev/sdk/features/#scopes

Please also add to the docs: https://github.com/getsentry/develop

[tengl]

Hi @bruno-garcia

We just updated to Sentry 3.14.1 and changed from WithScope to CaptureMessege etc. where applicable. However, we had some lines with

SentrySdk.WithScope(scope =>
{
    scope.SetTag("ATag", "This is a tag");
    scope.SetExtra("extra", "This is extra");
    _logger.LogWarning("Printing a warning in the console and logging to Sentry");
});

ILogger _logger prints to output AND logs to Sentry including the tag and extra data.

How do we do that with the new API?