`Sentry.captureTransaction` should return `Future<SentryId?>` instead of `Future<SentryId>`

[ueman]

Platform:

• Dart
• Flutter Android or iOS
• Flutter Web

IDE:

• VSCode
• IntelliJ/AS
• XCode
• Other, which one?

split-debug-info and obfuscate (Flutter Android or iOS) or CanvasKit (Flutter Web):

• Enabled
• Disabled

Platform installed with:

• pub.dev
• GitHub

The version of the SDK (See pubspec.lock):
6.3.0 beta3

---

I have the following issue:

Sentry.captureTransaction should return Future<SentryId?> instead of Future<SentryId>.
If the transport, for whatever reason, returns null, which is allowed, Sentry crashes because
it does a force unwrap here

sentry-dart/dart/lib/src/sentry_client.dart

Line 264 in ea5c733

return id!;

[marandaneto]

Indeed, it should be return id ?? SentryId.empty(); as captureEvent, a transaction always has an id or its empty so clients don't need to handle NPE.