null safety migration

dart/lib/src/http_client/sentry_http_client.dart

@@ -46,7 +46,7 @@ import '../../sentry.dart';
 // For example with Darts Stopwatch:
 // https://api.dart.dev/stable/2.10.4/dart-core/Stopwatch-class.html
 class SentryHttpClient extends BaseClient {
-  SentryHttpClient({Client/*?*/ client, Hub/*?*/ hub})
+  SentryHttpClient({Client? client, Hub? hub})
       : _hub = hub ?? HubAdapter(),
         _client = client ?? Client();
 

dart/lib/src/hub.dart

@@ -19,12 +19,11 @@ class Hub {
   final ListQueue<_StackItem> _stack = ListQueue();
 
   /// if stack is empty, it throws IterableElementError.noElement()
-  _StackItem _peek() => _stack.isNotEmpty ? _stack.first : null;
+  _StackItem? _peek() => _stack.isNotEmpty ? _stack.first : null;
 
   final SentryOptions _options;
 
   factory Hub(SentryOptions options) {
-    assert(options != null, 'SentryOptions is required.');
     _validateOptions(options);
 
     return Hub._(options);
@@ -36,7 +35,7 @@ class Hub {
   }
 
   static void _validateOptions(SentryOptions options) {
-    if (options.dsn?.isNotEmpty != true) {
+    if (options.dsn.isEmpty) {
       throw ArgumentError('DSN is required.');
     }
   }
@@ -53,7 +52,7 @@ class Hub {
 
   /// Captures the event.
   Future<SentryId> captureEvent(
-    SentryEvent /*?*/ event, {
+    SentryEvent? event, {
     dynamic stackTrace,
     dynamic hint,
   }) async {
@@ -146,10 +145,10 @@ class Hub {
 
   /// Captures the message.
   Future<SentryId> captureMessage(
-    String /*?*/ message, {
-    SentryLevel level = SentryLevel.info,
-    String template,
-    List<dynamic> params,
+    String? message, {
+    SentryLevel? level = SentryLevel.info,
+    String? template,
+    List<dynamic>? params,
     dynamic hint,
   }) async {
     var sentryId = SentryId.empty();
@@ -195,7 +194,7 @@ class Hub {
   }
 
   /// Adds a breacrumb to the current Scope
-  void addBreadcrumb(Breadcrumb /*?*/ crumb, {dynamic hint}) {
+  void addBreadcrumb(Breadcrumb? crumb, {dynamic hint}) {
     if (!_isEnabled) {
       _options.logger(
         SentryLevel.warning,

dart/lib/src/hub_adapter.dart

@@ -16,15 +16,15 @@ class HubAdapter implements Hub {
   }
 
   @override
-  void addBreadcrumb(Breadcrumb crumb, {dynamic hint}) =>
+  void addBreadcrumb(Breadcrumb? crumb, {dynamic hint}) =>
       Sentry.addBreadcrumb(crumb, hint: hint);
 
   @override
   void bindClient(SentryClient client) => Sentry.bindClient(client);
 
   @override
   Future<SentryId> captureEvent(
-    SentryEvent event, {
+    SentryEvent? event, {
     dynamic stackTrace,
     dynamic hint,
   }) =>
@@ -48,10 +48,10 @@ class HubAdapter implements Hub {
 
   @override
   Future<SentryId> captureMessage(
-    String message, {
-    SentryLevel level = SentryLevel.info,
-    String template,
-    List params,
+    String? message, {
+    SentryLevel? level = SentryLevel.info,
+    String? template,
+    List? params,
     dynamic hint,
   }) =>
       Sentry.captureMessage(

dart/lib/src/isolate_error_integration.dart

@@ -10,7 +10,7 @@ import 'sentry_options.dart';
 import 'throwable_mechanism.dart';
 
 class IsolateErrorIntegration extends Integration {
-  RawReceivePort _receivePort;
+  late RawReceivePort _receivePort;
 
   @override
   FutureOr<void> call(Hub hub, SentryOptions options) async {

dart/lib/src/noop_client.dart

@@ -24,38 +24,38 @@ class NoOpClient implements Client {
   @override
   Future<Response> delete(
     Uri url, {
-    Map<String, String> headers,
-    Object body,
-    Encoding encoding,
+    Map<String, String>? headers,
+    Object? body,
+    Encoding? encoding,
   }) =>
       _response;
 
   @override
-  Future<Response> get(url, {Map<String, String> headers}) => _response;
+  Future<Response> get(url, {Map<String, String>? headers}) => _response;
 
   @override
-  Future<Response> head(url, {Map<String, String> headers}) => _response;
+  Future<Response> head(url, {Map<String, String>? headers}) => _response;
 
   @override
   Future<Response> patch(url,
-          {Map<String, String> headers, body, Encoding encoding}) =>
+          {Map<String, String>? headers, body, Encoding? encoding}) =>
       _response;
 
   @override
   Future<Response> post(url,
-          {Map<String, String> headers, body, Encoding encoding}) =>
+          {Map<String, String>? headers, body, Encoding? encoding}) =>
       _response;
 
   @override
   Future<Response> put(url,
-          {Map<String, String> headers, body, Encoding encoding}) =>
+          {Map<String, String>? headers, body, Encoding? encoding}) =>
       _response;
 
   @override
-  Future<String> read(url, {Map<String, String> headers}) => _string;
+  Future<String> read(url, {Map<String, String>? headers}) => _string;
 
   @override
-  Future<Uint8List> readBytes(url, {Map<String, String> headers}) => _intList;
+  Future<Uint8List> readBytes(url, {Map<String, String>? headers}) => _intList;
 
   @override
   Future<StreamedResponse> send(BaseRequest request) => _streamedResponse;

dart/lib/src/noop_hub.dart

@@ -18,7 +18,7 @@ class NoOpHub implements Hub {
 
   @override
   Future<SentryId> captureEvent(
-    SentryEvent event, {
+    SentryEvent? event, {
     dynamic stackTrace,
     dynamic hint,
   }) =>
@@ -34,10 +34,10 @@ class NoOpHub implements Hub {
 
   @override
   Future<SentryId> captureMessage(
-    String message, {
-    SentryLevel level = SentryLevel.info,
-    String template,
-    List params,
+    String? message, {
+    SentryLevel? level = SentryLevel.info,
+    String? template,
+    List? params,
     dynamic hint,
   }) =>
       Future.value(SentryId.empty());
@@ -58,5 +58,5 @@ class NoOpHub implements Hub {
   SentryId get lastEventId => SentryId.empty();
 
   @override
-  void addBreadcrumb(Breadcrumb crumb, {dynamic hint}) {}
+  void addBreadcrumb(Breadcrumb? crumb, {dynamic hint}) {}
 }

dart/lib/src/noop_sentry_client.dart

@@ -17,7 +17,7 @@ class NoOpSentryClient implements SentryClient {
   Future<SentryId> captureEvent(
     SentryEvent event, {
     dynamic stackTrace,
-    Scope scope,
+    Scope? scope,
     dynamic hint,
   }) =>
       Future.value(SentryId.empty());
@@ -26,18 +26,18 @@ class NoOpSentryClient implements SentryClient {
   Future<SentryId> captureException(
     dynamic exception, {
     dynamic stackTrace,
-    Scope scope,
+    Scope? scope,
     dynamic hint,
   }) =>
       Future.value(SentryId.empty());
 
   @override
   Future<SentryId> captureMessage(
-    String message, {
-    SentryLevel level = SentryLevel.info,
-    String template,
-    List<dynamic> params,
-    Scope scope,
+    String? message, {
+    SentryLevel? level = SentryLevel.info,
+    String? template,
+    List<dynamic>? params,
+    Scope? scope,
     dynamic hint,
   }) =>
       Future.value(SentryId.empty());

dart/lib/src/platform_checker.dart

@@ -4,17 +4,17 @@ class PlatformChecker {
   const PlatformChecker();
 
   /// Check if running in release/production environment
-  bool/*!*/ isReleaseMode() {
+  bool isReleaseMode() {
     return const bool.fromEnvironment('dart.vm.product', defaultValue: false);
   }
 
   /// Check if running in debug environment
-  bool/*!*/ isDebugMode() {
+  bool isDebugMode() {
     return !isReleaseMode() && !isProfileMode();
   }
 
   /// Check if running in profile environment
-  bool/*!*/ isProfileMode() {
+  bool isProfileMode() {
     return const bool.fromEnvironment('dart.vm.profile', defaultValue: false);
   }
 }

dart/lib/src/protocol/app.dart

@@ -31,29 +31,29 @@ class App {
       );
 
   /// Human readable application name, as it appears on the platform.
-  final String name;
+  final String? name;
 
   /// Human readable application version, as it appears on the platform.
-  final String version;
+  final String? version;
 
   /// Version-independent application identifier, often a dotted bundle ID.
-  final String identifier;
+  final String? identifier;
 
   /// Internal build identifier, as it appears on the platform.
-  final String build;
+  final String? build;
 
   /// String identifying the kind of build, e.g. `testflight`.
-  final String buildType;
+  final String? buildType;
 
   /// When the application was started by the user.
-  final DateTime startTime;
+  final DateTime? startTime;
 
   /// Application specific device identifier.
-  final String deviceAppHash;
+  final String? deviceAppHash;
 
   /// Produces a [Map] that can be serialized to JSON.
   Map<String, dynamic> toJson() {
-    final json = <String, String>{};
+    final json = <String, String?>{};
 
     if (name != null) {
       json['app_name'] = name;
@@ -76,7 +76,7 @@ class App {
     }
 
     if (startTime != null) {
-      json['app_start_time'] = startTime.toIso8601String();
+      json['app_start_time'] = startTime!.toIso8601String();
     }
 
     if (deviceAppHash != null) {
@@ -97,13 +97,13 @@ class App {
       );
 
   App copyWith({
-    String name,
-    String version,
-    String identifier,
-    String build,
-    String buildType,
-    DateTime startTime,
-    String deviceAppHash,
+    String? name,
+    String? version,
+    String? identifier,
+    String? build,
+    String? buildType,
+    DateTime? startTime,
+    String? deviceAppHash,
   }) =>
       App(
         name: name ?? this.name,

dart/lib/src/protocol/breadcrumb.dart

@@ -24,7 +24,7 @@ class Breadcrumb {
   /// Creates a breadcrumb that can be attached to an [Event].
   Breadcrumb({
     this.message,
-    DateTime timestamp,
+    DateTime? timestamp,
     this.category,
     this.data,
     this.level = SentryLevel.info,
@@ -34,12 +34,12 @@ class Breadcrumb {
   /// Describes the breadcrumb.
   ///
   /// This field is optional and may be set to null.
-  final String message;
+  final String? message;
 
   /// A dot-separated string describing the source of the breadcrumb, e.g. "ui.click".
   ///
   /// This field is optional and may be set to null.
-  final String category;
+  final String? category;
 
   /// Data associated with the breadcrumb.
   ///
@@ -50,7 +50,7 @@ class Breadcrumb {
   /// See also:
   ///
   /// * https://docs.sentry.io/development/sdk-dev/event-payloads/breadcrumbs/#breadcrumb-types
-  final Map<String, dynamic> data;
+  final Map<String, dynamic>? data;
 
   /// Severity of the breadcrumb.
   ///
@@ -66,7 +66,7 @@ class Breadcrumb {
   /// See also:
   ///
   /// * https://docs.sentry.io/development/sdk-dev/event-payloads/breadcrumbs/#breadcrumb-types
-  final String type;
+  final String? type;
 
   /// The time the breadcrumb was recorded.
   ///
@@ -87,7 +87,7 @@ class Breadcrumb {
     if (category != null) {
       json['category'] = category;
     }
-    if (data != null && data.isNotEmpty) {
+    if (data != null && data!.isNotEmpty) {
       json['data'] = data;
     }
     if (level != null) {
@@ -100,12 +100,12 @@ class Breadcrumb {
   }
 
   Breadcrumb copyWith({
-    String message,
-    String category,
-    Map<String, dynamic> data,
-    SentryLevel level,
-    String type,
-    DateTime timestamp,
+    String? message,
+    String? category,
+    Map<String, dynamic>? data,
+    SentryLevel? level,
+    String? type,
+    DateTime? timestamp,
   }) =>
       Breadcrumb(
         message: message ?? this.message,