fixes

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Features
 
 - Feat: Screenshot Attachment ([#1088](https://github.com/getsentry/sentry-dart/pull/1088))
+
 ### Fixes
 
 - Merging of integrations and packages ([#1111](https://github.com/getsentry/sentry-dart/pull/1111))

flutter/lib/src/integrations/screenshot_integration.dart

@@ -16,8 +16,9 @@ class ScreenshotIntegration implements Integration<SentryFlutterOptions> {
       options.clientAttachmentProcessor =
           ScreenshotAttachmentProcessor(options);
       _options = options;
+
+      options.sdk.addIntegration('screenshotIntegration');
     }
-    options.sdk.addIntegration('screenshotIntegration');
   }
 
   @override

flutter/lib/src/screenshot/screenshot_attachment_processor.dart

@@ -16,15 +16,15 @@ class ScreenshotAttachmentProcessor implements SentryClientAttachmentProcessor {
   ScreenshotAttachmentProcessor(this._options);
 
   /// This is true when the SentryWidget is in the view hierarchy
-  bool get _attachScreenshot =>
+  bool get _hasSentryScreenshotWidget =>
       sentryScreenshotWidgetGlobalKey.currentContext != null;
 
   @override
   Future<List<SentryAttachment>> processAttachments(
       List<SentryAttachment> attachments, SentryEvent event) async {
     if (event.exceptions == null &&
         event.throwable == null &&
-        _attachScreenshot) {
+        _hasSentryScreenshotWidget) {
       return attachments;
     }
     final renderer = _options.rendererWrapper.getRenderer();
@@ -60,8 +60,8 @@ class ScreenshotAttachmentProcessor implements SentryClientAttachmentProcessor {
         }
 
         final byteData = await image.toByteData(format: ui.ImageByteFormat.png);
-        final bytes = byteData?.buffer.asUint8List() ?? Uint8List(0);
-        if (bytes.isNotEmpty) {
+        final bytes = byteData?.buffer.asUint8List();
+        if (bytes?.isNotEmpty == true) {
           return bytes;
         } else {
           _options.logger(SentryLevel.debug,

flutter/lib/src/screenshot/sentry_screenshot_widget.dart

@@ -1,6 +1,8 @@
 import 'package:flutter/material.dart';
+import 'package:meta/meta.dart';
 
 /// Key which is used to identify the [RepaintBoundary]
+@internal
 final sentryScreenshotWidgetGlobalKey =
     GlobalKey(debugLabel: 'sentry_screenshot_widget');
 

flutter/lib/src/sentry_flutter_options.dart

@@ -181,6 +181,8 @@ class SentryFlutterOptions extends SentryOptions {
   bool autoAppStart = true;
 
   /// Automatically attaches a screenshot when capturing an error or exception.
+  ///
+  /// Requires adding the [SentryScreenshotWidget] to the widget tree.
   bool attachScreenshot = false;
 
   @internal

flutter/test/sentry_flutter_test.dart

@@ -323,7 +323,7 @@ void main() {
       List<Integration> integrations = [];
       Transport transport = MockTransport();
 
-      // Tests that iOS || macOS integrations aren't added on a browswer which
+      // Tests that iOS || macOS integrations aren't added on a browser which
       // runs on iOS or macOS
       await SentryFlutter.init(
         (options) async {