Update sentry dependency to ^9.0.0 (#367)

* bump

* remove deprecated

* feat(feedback): enhance Sentry feedback handling with attachment support

* revert + fix test

* refactor(feedback): remove deprecated method and update feedback handling

* feat(feedback): add user feedback context and attachment support in Sentry error handling

---------

Co-authored-by: Jonas Uekötter <[email protected]>

Author: eifr

feedback_sentry/lib/feedback_sentry.dart

@@ -26,10 +26,6 @@ extension SentryFeedbackX on FeedbackController {
   /// This method opens the feedback ui and the users feedback
   /// is uploaded to Sentry after the user submitted his feedback.
   /// [name] and [email] are optional. They are shown in the Sentry.io ui.s
-  @Deprecated(
-    'Sentry marked the underlying APIs for this method as deprecated. '
-    'Unfortunately, there is no replacement.',
-  )
   void showAndUploadToSentryWithException({
     Hub? hub,
     String? name,
@@ -59,20 +55,23 @@ OnFeedbackCallback sendToSentry({
 
   return (UserFeedback feedback) async {
     await realHub.captureFeedback(
-      SentryFeedback(
-        contactEmail: email,
-        name: name,
-        message: feedback.text,
-        unknown: feedback.extra,
-      ),
-      hint: Hint.withScreenshot(
-        SentryAttachment.fromUint8List(
-          feedback.screenshot,
-          'screenshot.png',
-          contentType: 'image/png',
+        SentryFeedback(
+          contactEmail: email,
+          name: name,
+          message: feedback.text,
         ),
-      ),
-    );
+        hint: Hint.withScreenshot(
+          SentryAttachment.fromUint8List(
+            feedback.screenshot,
+            'screenshot.png',
+            contentType: 'image/png',
+          ),
+        ), withScope: (scope) {
+      scope.setContexts(
+        "user_feedback",
+        feedback.extra,
+      );
+    });
   };
 }
 
@@ -93,19 +92,22 @@ OnFeedbackCallback sendToSentryWithException({
       exception,
       stackTrace: stackTrace,
       withScope: (scope) {
+        scope.setContexts(
+          "user_feedback",
+          feedback.extra,
+        );
         scope.addAttachment(SentryAttachment.fromUint8List(
           feedback.screenshot,
           'screenshot.png',
           contentType: 'image/png',
         ));
       },
     );
-    // ignore: deprecated_member_use
-    await realHub.captureUserFeedback(SentryUserFeedback(
-      eventId: id,
-      email: email,
+    await realHub.captureFeedback(SentryFeedback(
+      associatedEventId: id,
+      contactEmail: email,
       name: name,
-      comments: feedback.text + '\n${feedback.extra.toString()}',
+      message: feedback.text,
     ));
   };
 }

feedback_sentry/pubspec.yaml

@@ -9,15 +9,16 @@ funding:
   - https://github.com/sponsors/ueman
 
 environment:
-  sdk: '>=3.0.0 <4.0.0'
-  flutter: '>=3.10.0'
+  sdk: ">=3.5.0 <4.0.0"
+  flutter: ">=3.24.0"
 
 dependencies:
   flutter:
     sdk: flutter
   feedback: ^3.0.0
-  sentry: ^8.10.0
+  sentry: ^9.0.0
 
 dev_dependencies:
   flutter_test:
-    sdk: flutter
+    sdk: flutter
+

feedback_sentry/test/feedback_sentry_test.dart

@@ -23,29 +23,28 @@ void main() {
 
     expect(completed, true);
 
-    expect(mockHub.capturedMessage, 'foo bar');
-
-    expect(mockHub.capturedFeedback?.comments, 'foo bar\n{foo: bar}');
+    expect(mockHub.capturedFeedback?.message, 'foo bar');
     expect(mockHub.capturedFeedback?.name, 'foo');
-    expect(mockHub.capturedFeedback?.email, 'bar@foo.de');
-
-    expect(mockHub.scope.attachments.length, 1);
+    expect(mockHub.scope.contexts["user_feedback"], {'foo': 'bar'});
+    expect(mockHub.capturedFeedback?.contactEmail, '[email protected]');
+    expect(mockHub.captureHint?.screenshot?.filename, "screenshot.png");
   });
 }
 
 class MockHub implements Hub {
-  SentryUserFeedback? capturedFeedback;
+  SentryFeedback? capturedFeedback;
   String? capturedMessage;
   Scope scope = Scope(SentryOptions());
   Completer<bool> completer = Completer<bool>();
+  Hint? captureHint;
 
   @override
   Future<SentryId> captureMessage(
     String? message, {
     SentryLevel? level,
     String? template,
     List? params,
-    hint,
+    Hint? hint,
     ScopeCallback? withScope,
   }) async {
     capturedMessage = message;
@@ -54,9 +53,16 @@ class MockHub implements Hub {
   }
 
   @override
-  Future<void> captureUserFeedback(SentryUserFeedback userFeedback) async {
-    capturedFeedback = userFeedback;
+  Future<SentryId> captureFeedback(
+    SentryFeedback feedback, {
+    Hint? hint,
+    ScopeCallback? withScope,
+  }) async {
+    capturedFeedback = feedback;
+    withScope?.call(scope);
+    captureHint = hint;
     completer.complete(true);
+    return SentryId.newId();
   }
 
   // Used so that unneeded methods, do not need to be overridden.