feat(js-sdk): support release health

.github/workflows/flutter_test.yml

@@ -222,6 +222,6 @@ jobs:
 
           # Run the tests
           flutter drive \
-            --driver=integration_test/test_driver/web_driver.dart \
+            --driver=integration_test/test_driver/driver.dart \
             --target=integration_test/web_sdk_test.dart \
             -d chrome

dart/lib/sentry.dart

@@ -5,60 +5,60 @@
 /// A pure Dart client for Sentry.io crash reporting.
 library sentry_dart;
 
-export 'src/run_zoned_guarded_integration.dart';
+export 'src/event_processor.dart';
+export 'src/exception_cause.dart';
+// exception extraction
+export 'src/exception_cause_extractor.dart';
+export 'src/exception_stacktrace_extractor.dart';
+export 'src/exception_type_identifier.dart';
+export 'src/hint.dart';
+export 'src/http_client/sentry_http_client.dart';
+export 'src/http_client/sentry_http_client_error.dart';
 export 'src/hub.dart';
 // useful for tests
 export 'src/hub_adapter.dart';
-export 'src/platform_checker.dart';
+export 'src/integration.dart';
 export 'src/noop_isolate_error_integration.dart'
     if (dart.library.io) 'src/isolate_error_integration.dart';
+export 'src/performance_collector.dart';
+export 'src/platform_checker.dart';
 export 'src/protocol.dart';
+// feedback
+export 'src/protocol/sentry_feedback.dart';
+// proxy
+export 'src/protocol/sentry_proxy.dart';
+export 'src/run_zoned_guarded_integration.dart';
 export 'src/scope.dart';
 export 'src/scope_observer.dart';
 export 'src/sentry.dart';
+export 'src/sentry_attachment/sentry_attachment.dart';
+export 'src/sentry_baggage.dart';
+export 'src/sentry_client.dart';
 export 'src/sentry_envelope.dart';
 export 'src/sentry_envelope_item.dart';
-export 'src/sentry_client.dart';
 export 'src/sentry_options.dart';
+// ignore: invalid_export_of_internal_element
+export 'src/sentry_span_operations.dart';
+// ignore: invalid_export_of_internal_element
+export 'src/sentry_trace_origins.dart';
+export 'src/sentry_user_feedback.dart';
+// constants
+export 'src/span_data_convention.dart';
+// spotlight debugging
+export 'src/spotlight.dart';
 // useful for integrations
 export 'src/throwable_mechanism.dart';
-export 'src/transport/transport.dart';
-export 'src/integration.dart';
-export 'src/event_processor.dart';
-export 'src/http_client/sentry_http_client.dart';
-export 'src/http_client/sentry_http_client_error.dart';
-export 'src/sentry_attachment/sentry_attachment.dart';
-export 'src/sentry_user_feedback.dart';
-export 'src/utils/tracing_utils.dart';
-export 'src/performance_collector.dart';
 // tracing
 export 'src/tracing.dart';
-export 'src/hint.dart';
+export 'src/transport/transport.dart';
 export 'src/type_check_hint.dart';
-export 'src/sentry_baggage.dart';
-// exception extraction
-export 'src/exception_cause_extractor.dart';
-export 'src/exception_cause.dart';
-export 'src/exception_stacktrace_extractor.dart';
-export 'src/exception_type_identifier.dart';
-// URL
-// ignore: invalid_export_of_internal_element
-export 'src/utils/http_sanitizer.dart';
 // ignore: invalid_export_of_internal_element
-export 'src/utils/url_details.dart';
+export 'src/utils.dart';
 // ignore: invalid_export_of_internal_element
 export 'src/utils/http_header_utils.dart';
+// URL
 // ignore: invalid_export_of_internal_element
-export 'src/sentry_trace_origins.dart';
-// ignore: invalid_export_of_internal_element
-export 'src/sentry_span_operations.dart';
+export 'src/utils/http_sanitizer.dart';
+export 'src/utils/tracing_utils.dart';
 // ignore: invalid_export_of_internal_element
-export 'src/utils.dart';
-// spotlight debugging
-export 'src/spotlight.dart';
-// proxy
-export 'src/protocol/sentry_proxy.dart';
-// feedback
-export 'src/protocol/sentry_feedback.dart';
-// constants
-export 'src/span_data_convention.dart';
+export 'src/utils/url_details.dart';

dart/lib/src/client_reports/client_report.dart

@@ -1,10 +1,10 @@
 import 'package:meta/meta.dart';
 
+import '../../sentry.dart';
 import 'discarded_event.dart';
-import '../utils.dart';
 
 @internal
-class ClientReport {
+class ClientReport implements SentryEnvelopeItemPayload {
   ClientReport(this.timestamp, this.discardedEvents);
 
   final DateTime? timestamp;
@@ -27,4 +27,9 @@ class ClientReport {
 
     return json;
   }
+
+  @override
+  Future getPayload() {
+    return Future.value(toJson());
+  }
 }

dart/lib/src/platform_checker.dart

@@ -40,20 +40,13 @@ class PlatformChecker {
   }
 
   /// Indicates whether a native integration is available.
-  bool get hasNativeIntegration {
-    if (isWeb) {
-      return false;
-    }
-    // We need to check the platform after we checked for web, because
-    // the OS checks return true when the browser runs on the checked platform.
-    // Example: platform.isAndroid return true if the browser is used on an
-    // Android device.
-    return platform.isAndroid ||
-        platform.isIOS ||
-        platform.isMacOS ||
-        platform.isWindows ||
-        platform.isLinux;
-  }
+  bool get hasNativeIntegration =>
+      isWeb ||
+      platform.isAndroid ||
+      platform.isIOS ||
+      platform.isMacOS ||
+      platform.isWindows ||
+      platform.isLinux;
 
   static bool _isWebWithWasmSupport() {
     if (const bool.hasEnvironment(_jsUtil)) {

dart/lib/src/protocol/sentry_event.dart

@@ -1,14 +1,14 @@
-import 'package:meta/meta.dart';
 import 'package:collection/collection.dart';
+import 'package:meta/meta.dart';
 
-import '../protocol.dart';
-import '../throwable_mechanism.dart';
-import '../utils.dart';
+import '../../sentry.dart';
 import 'access_aware_map.dart';
 
 /// An event to be reported to Sentry.io.
 @immutable
-class SentryEvent with SentryEventLike<SentryEvent> {
+class SentryEvent
+    with SentryEventLike<SentryEvent>
+    implements SentryEnvelopeItemPayload {
   /// Creates an event.
   SentryEvent({
     SentryId? eventId,
@@ -418,4 +418,9 @@ class SentryEvent with SentryEventLike<SentryEvent> {
   SentryStackTrace? get stacktrace =>
       exceptions?.firstWhereOrNull((e) => e.stackTrace != null)?.stackTrace ??
       threads?.firstWhereOrNull((t) => t.stacktrace != null)?.stacktrace;
+
+  @override
+  Future<dynamic> getPayload() {
+    return Future.value(toJson());
+  }
 }

dart/lib/src/sentry_attachment/sentry_attachment.dart

@@ -1,16 +1,15 @@
 import 'dart:async';
 import 'dart:typed_data';
 
-import '../protocol/sentry_view_hierarchy.dart';
-import '../utils.dart';
+import '../../sentry.dart';
 
 // https://develop.sentry.dev/sdk/features/#attachments
 // https://develop.sentry.dev/sdk/envelopes/#attachment
 
 typedef ContentLoader = FutureOr<Uint8List> Function();
 
 /// Arbitrary content which gets attached to an event.
-class SentryAttachment {
+class SentryAttachment implements SentryEnvelopeItemPayload {
   /// Standard attachment without special meaning.
   static const String typeAttachmentDefault = 'event.attachment';
 
@@ -122,4 +121,9 @@ class SentryAttachment {
   /// If true, attachment should be added to every transaction.
   /// Defaults to false.
   final bool addToTransactions;
+
+  @override
+  Future<dynamic> getPayload() async {
+    return await bytes;
+  }
 }

dart/lib/src/sentry_client.dart

@@ -2,7 +2,6 @@ import 'dart:async';
 import 'dart:math';
 
 import 'package:meta/meta.dart';
-import 'type_check_hint.dart';
 
 import 'client_reports/client_report_recorder.dart';
 import 'client_reports/discard_reason.dart';
@@ -27,6 +26,7 @@ import 'transport/http_transport.dart';
 import 'transport/noop_transport.dart';
 import 'transport/rate_limiter.dart';
 import 'transport/spotlight_http_transport.dart';
+import 'type_check_hint.dart';
 import 'utils/isolate_utils.dart';
 import 'utils/regex_utils.dart';
 import 'utils/stacktrace_utils.dart';

dart/lib/src/sentry_envelope_item.dart

@@ -10,10 +10,14 @@ import 'sentry_item_type.dart';
 import 'sentry_user_feedback.dart';
 import 'utils.dart';
 
+abstract class SentryEnvelopeItemPayload {
+  Future<dynamic> getPayload();
+}
+
 /// Item holding header information and JSON encoded data.
 class SentryEnvelopeItem {
   /// The original, non-encoded object, used when direct access to the source data is needed.
-  Object? originalObject;
+  SentryEnvelopeItemPayload? originalObject;
 
   SentryEnvelopeItem(this.header, this.dataFactory, {this.originalObject});
 
@@ -102,7 +106,6 @@ class SentryEnvelopeItem {
     return SentryEnvelopeItem(
       header,
       dataFactory,
-      originalObject: buckets,
     );
   }
 

dart/lib/src/sentry_options.dart

@@ -276,7 +276,7 @@ class SentryOptions {
   /// breadcrumbs.
   /// In a Flutter environment, this setting also toggles recording of `debugPrint` calls.
   /// `debugPrint` calls are only recorded in release builds, though.
-  bool enablePrintBreadcrumbs = true;
+  bool enablePrintBreadcrumbs = false;
 
   /// If [platformChecker] is provided, it is used get the environment.
   /// This is useful in tests. Should be an implementation of [PlatformChecker].

dart/lib/src/sentry_user_feedback.dart

@@ -1,10 +1,10 @@
 import 'package:meta/meta.dart';
 
-import 'protocol.dart';
+import '../sentry.dart';
 import 'protocol/access_aware_map.dart';
 
 @Deprecated('Will be removed in a future version. Use [SentryFeedback] instead')
-class SentryUserFeedback {
+class SentryUserFeedback implements SentryEnvelopeItemPayload {
   SentryUserFeedback({
     required this.eventId,
     this.name,
@@ -66,4 +66,9 @@ class SentryUserFeedback {
       unknown: unknown,
     );
   }
+
+  @override
+  Future<dynamic> getPayload() {
+    return Future.value(toJson());
+  }
 }

dart/lib/src/transport/http_transport.dart

@@ -2,17 +2,17 @@ import 'dart:async';
 import 'dart:convert';
 
 import 'package:http/http.dart';
-import '../utils/transport_utils.dart';
-import 'http_transport_request_handler.dart';
 
+import '../http_client/client_provider.dart'
+    if (dart.library.io) '../http_client/io_client_provider.dart';
 import '../noop_client.dart';
 import '../protocol.dart';
-import '../sentry_options.dart';
 import '../sentry_envelope.dart';
-import 'transport.dart';
+import '../sentry_options.dart';
+import '../utils/transport_utils.dart';
+import 'http_transport_request_handler.dart';
 import 'rate_limiter.dart';
-import '../http_client/client_provider.dart'
-    if (dart.library.io) '../http_client/io_client_provider.dart';
+import 'transport.dart';
 
 /// A transport is in charge of sending the event to the Sentry server.
 class HttpTransport implements Transport {

flutter/example/integration_test/utils.dart

@@ -20,3 +20,5 @@ FutureOr<void> restoreFlutterOnErrorAfter(FutureOr<void> Function() fn) async {
     originalOnError?.call(details);
   };
 }
+
+const fakeDsn = 'https://abc@def.ingest.sentry.io/1234567';