TF-4268 Enable sentry for Android

.github/workflows/release.yaml

@@ -89,3 +89,47 @@ jobs:
           APPLE_KEY_CONTENT: ${{ secrets.APPLE_KEY_CONTENT }}
         run: ../scripts/build-release.sh
         working-directory: ${{ matrix.os }}
+
+      # --- UPLOAD SENTRY MAPPING AND SYMBOLS FOR ANDROID ---
+      - name: Upload Android ProGuard Mapping & Dart Debug Symbols to Sentry
+        if: matrix.os == 'android'
+        working-directory: .
+        env:
+          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
+          SENTRY_ORG: ${{ secrets.SENTRY_ORG }}
+          SENTRY_PROJECT: ${{ secrets.SENTRY_PROJECT }}
+        run: |
+          # 1. Install Sentry CLI
+          curl -sL https://sentry.io/get-cli/ | bash
+
+          # 2. Get Version from pubspec.yaml
+          SENTRY_RELEASE=$(grep '^version:' pubspec.yaml | awk '{print $2}')
+          echo "Processing Sentry Release: $SENTRY_RELEASE"
+
+          # 3. Create Release
+          sentry-cli releases new "$SENTRY_RELEASE"
+
+          # 4. Upload ProGuard Mapping
+          MAPPING_FILE="build/app/outputs/mapping/release/mapping.txt"
+
+          if [ -f "$MAPPING_FILE" ]; then
+            echo "Found mapping.txt, uploading ProGuard mapping..."
+            sentry-cli upload-proguard "$MAPPING_FILE"
+          else
+            echo "::error::Could not find mapping.txt at $MAPPING_FILE."
+            exit 1
+          fi
+
+          # 5. Upload Dart Debug Symbols (Native Dart Stacktrace)
+          SYMBOLS_DIR="build/app/outputs/symbols"
+
+          if [ -d "$SYMBOLS_DIR" ]; then
+            echo "Uploading Dart debug symbols from $SYMBOLS_DIR..."
+            sentry-cli debug-files upload "$SYMBOLS_DIR"
+          else
+            echo "::error::Symbols directory not found at $SYMBOLS_DIR. Check your Fastfile build arguments."
+            exit 1
+          fi
+
+          # 6. Finalize
+          sentry-cli releases finalize "$SENTRY_RELEASE"

Dockerfile

@@ -20,8 +20,11 @@ ENV GITHUB_SHA=$GITHUB_SHA \
     SENTRY_RELEASE=$SENTRY_RELEASE
 
 RUN ./scripts/prebuild.sh && \
-    flutter build web --release --source-maps --dart-define=SENTRY_RELEASE=$SENTRY_RELEASE && \
-    if [ -n "$SENTRY_AUTH_TOKEN" ] && [ -n "$SENTRY_ORG" ] && [ -n "$SENTRY_PROJECT" ] && [ -n "$SENTRY_RELEASE" ]; then \
+    if [ -z "$GITHUB_SHA" ]; then echo "GITHUB_SHA is required for SENTRY_DIST"; exit 1; fi && \
+    flutter build web --release --source-maps  \
+    --dart-define=SENTRY_RELEASE=$SENTRY_RELEASE \
+    --dart-define=SENTRY_DIST=$GITHUB_SHA && \
+    if [ -n "$SENTRY_AUTH_TOKEN" ] && [ -n "$SENTRY_ORG" ] && [ -n "$SENTRY_PROJECT" ] && [ -n "$SENTRY_RELEASE" ] && [ -n "$GITHUB_SHA" ]; then \
         echo "Sentry configuration detected, uploading sourcemaps..." && \
         curl -sL https://sentry.io/get-cli/ | SENTRY_CLI_VERSION=2.20.7 bash && \
         sentry-cli releases new "$SENTRY_RELEASE" && \

android/fastlane/Fastfile

@@ -31,7 +31,9 @@ platform :android do
       json_key_data: ENV["PLAY_STORE_CONFIG_JSON"]
     )[0].to_i
     build_name = last_git_tag.gsub("v", "")
-    sh "flutter build appbundle --verbose --release --dart-define=SERVER_URL=$SERVER_URL --build-number=#{latest_build_number+1} --build-name=#{build_name}"
+    # Add: --obfuscate --split-debug-info=../build/app/outputs/symbols
+    # Note: The path starts with ../ because Fastlane runs in the android/ directory.
+    sh "cd .. && flutter build appbundle --verbose --release --obfuscate --split-debug-info=build/app/outputs/symbols --dart-define=SERVER_URL=$SERVER_URL --build-number=#{latest_build_number+1} --build-name=#{build_name}"
     upload_to_play_store(
       json_key_data: ENV["PLAY_STORE_CONFIG_JSON"],
       track: track,

core/lib/core.dart

@@ -25,6 +25,7 @@ export 'domain/exceptions/download_file_exception.dart';
 export 'data/extensions/options_extensions.dart';
 export 'domain/exceptions/web_session_exception.dart';
 export 'domain/exceptions/platform_exception.dart';
+export 'domain/exceptions/app_base_exception.dart';
 
 // Utils
 export 'presentation/utils/theme_utils.dart';

core/lib/domain/exceptions/address_exception.dart

@@ -1,13 +1,8 @@
-import 'package:equatable/equatable.dart';
+import 'package:core/domain/exceptions/app_base_exception.dart';
 
-class AddressException with EquatableMixin implements Exception {
-  final String message;
-
-  AddressException(this.message);
-
-  @override
-  String toString() => message;
+class AddressException extends AppBaseException {
+  const AddressException(super.message);
 
   @override
-  List<Object> get props => [message];
+  String get exceptionName => 'AddressException';
 }

core/lib/domain/exceptions/app_base_exception.dart

@@ -0,0 +1,20 @@
+import 'package:equatable/equatable.dart';
+
+abstract class AppBaseException with EquatableMixin implements Exception {
+  final String? message;
+
+  const AppBaseException([this.message]);
+
+  @override
+  String toString() {
+    if (message?.isNotEmpty == true) {
+      return '$exceptionName: $message';
+    }
+    return exceptionName;
+  }
+
+  String get exceptionName;
+
+  @override
+  List<Object?> get props => [message];
+}

core/lib/domain/exceptions/download_file_exception.dart

@@ -1,31 +1,27 @@
-import 'package:equatable/equatable.dart';
+import 'package:core/domain/exceptions/app_base_exception.dart';
 
-abstract class DownloadFileException with EquatableMixin implements Exception {
-  final String message;
-
-  DownloadFileException(this.message);
-
-  @override
-  String toString() => message;
-
-  @override
-  List<Object> get props => [message];
+abstract class DownloadFileException extends AppBaseException {
+  const DownloadFileException(super.message);
 }
 
 class CommonDownloadFileException extends DownloadFileException {
-  CommonDownloadFileException(message) : super(message);
+  const CommonDownloadFileException(super.message);
 
   @override
-  List<Object> get props => [message];
+  String get exceptionName => 'CommonDownloadFileException';
 }
 
 class CancelDownloadFileException extends DownloadFileException {
-  CancelDownloadFileException(cancelMessage) : super(cancelMessage);
+  const CancelDownloadFileException(super.message);
 
   @override
-  List<Object> get props => [message];
+  String get exceptionName => 'CancelDownloadFileException';
 }
 
 class DeviceNotSupportedException extends DownloadFileException {
-  DeviceNotSupportedException() : super('This device is not supported, please try on Android or iOS');
-}
+  const DeviceNotSupportedException()
+      : super('This device is not supported, please try on Android or iOS');
+
+  @override
+  String get exceptionName => 'DeviceNotSupportedException';
+}

core/lib/domain/exceptions/file_exception.dart

@@ -1,21 +1,19 @@
-import 'package:equatable/equatable.dart';
+import 'package:core/domain/exceptions/app_base_exception.dart';
 
-abstract class FileException with EquatableMixin implements Exception {
-  final String message;
-
-  FileException(this.message);
-
-  @override
-  String toString() => message;
-
-  @override
-  List<Object> get props => [message];
+abstract class FileException extends AppBaseException {
+  const FileException(super.message);
 }
 
 class NotFoundFileInFolderException extends FileException {
   NotFoundFileInFolderException() : super('No files found in the folder');
+
+  @override
+  String get exceptionName => 'NotFoundFileInFolderException';
 }
 
 class UserCancelShareFileException extends FileException {
   UserCancelShareFileException() : super('User cancel share file');
-}
+
+  @override
+  String get exceptionName => 'UserCancelShareFileException';
+}

core/lib/domain/exceptions/platform_exception.dart

@@ -1,14 +1,15 @@
-import 'package:equatable/equatable.dart';
+import 'package:core/domain/exceptions/app_base_exception.dart';
 
-class PlatformException with EquatableMixin implements Exception {
-  final String message;
-
-  PlatformException(this.message);
+class PlatformException extends AppBaseException {
+  const PlatformException(super.message);
 
   @override
-  List<Object> get props => [message];
+  String get exceptionName => 'PlatformException';
 }
 
 class NoSupportPlatformException extends PlatformException {
-  NoSupportPlatformException() : super('This platform is not supported');
-}
+  const NoSupportPlatformException() : super('This platform is not supported');
+
+  @override
+  String get exceptionName => 'NoSupportPlatformException';
+}

core/lib/domain/exceptions/string_exception.dart

@@ -1,7 +1,15 @@
-class UnsupportedCharsetException implements Exception {
-  const UnsupportedCharsetException();
+import 'package:core/domain/exceptions/app_base_exception.dart';
+
+class UnsupportedCharsetException extends AppBaseException {
+  const UnsupportedCharsetException([super.message]);
+
+  @override
+  String get exceptionName => 'UnsupportedCharsetException';
 }
 
-class NullCharsetException implements Exception {
-  const NullCharsetException();
-}
+class NullCharsetException extends AppBaseException {
+  const NullCharsetException([super.message]);
+
+  @override
+  String get exceptionName => 'NullCharsetException';
+}

core/lib/domain/exceptions/web_session_exception.dart

@@ -1,28 +1,44 @@
+import 'package:core/domain/exceptions/app_base_exception.dart';
 import 'package:equatable/equatable.dart';
 
-class NotFoundInWebSessionException with EquatableMixin implements Exception {
-  final String? errorMessage;
+class NotFoundInWebSessionException extends AppBaseException
+    with EquatableMixin {
+  const NotFoundInWebSessionException({String? errorMessage})
+      : super(errorMessage);
 
-  NotFoundInWebSessionException({this.errorMessage});
+  @override
+  String get exceptionName => 'NotFoundInWebSessionException';
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [message, exceptionName];
 }
 
-class NotMatchInWebSessionException with EquatableMixin implements Exception {
-  const NotMatchInWebSessionException();
+class NotMatchInWebSessionException extends AppBaseException
+    with EquatableMixin {
+  const NotMatchInWebSessionException() : super('Session data does not match');
+
+  @override
+  String get exceptionName => 'NotMatchInWebSessionException';
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [message, exceptionName];
 }
 
-class SaveToWebSessionFailException with EquatableMixin implements Exception {
-  final String? errorMessage;
+class SaveToWebSessionFailException extends AppBaseException
+    with EquatableMixin {
+  const SaveToWebSessionFailException({String? errorMessage})
+      : super(errorMessage);
 
-  SaveToWebSessionFailException({this.errorMessage});
+  @override
+  String get exceptionName => 'SaveToWebSessionFailException';
 
   @override
-  List<Object> get props => [];
+  List<Object?> get props => [message, exceptionName];
 }
 
-class CannotOpenNewWindowException implements Exception {}
+class CannotOpenNewWindowException extends AppBaseException {
+  const CannotOpenNewWindowException([super.message]);
+
+  @override
+  String get exceptionName => 'CannotOpenNewWindowException';
+}

core/lib/presentation/state/failure.dart

@@ -10,10 +10,12 @@ abstract class Failure with EquatableMixin {
 
 abstract class FeatureFailure extends Failure {
   final dynamic exception;
+  final StackTrace? stackTrace;
   final Stream<Either<Failure, Success>>? onRetry;
 
-  FeatureFailure({this.exception, this.onRetry});
+  FeatureFailure({this.exception, this.stackTrace, this.onRetry});
 
+  // Including stackTrace in props may break Equatable equality semantics.
   @override
   List<Object?> get props => [exception, onRetry];
 }

core/lib/utils/app_logger.dart

@@ -1,5 +1,3 @@
-import 'dart:async';
-
 import 'package:core/utils/build_utils.dart';
 import 'package:core/utils/platform_info.dart';
 import 'package:core/utils/sentry/sentry_manager.dart';
@@ -82,17 +80,13 @@ void _internalLog(
 
   if (shouldSentry) {
     if (level == Level.trace) {
-      unawaited(
-        SentryManager.instance.captureMessage(rawMessage, extras: extras),
-      );
+      SentryManager.instance.captureMessage(rawMessage, extras: extras);
     } else {
-      unawaited(
-        SentryManager.instance.captureException(
-          exception ?? rawMessage,
-          stackTrace: stackTrace,
-          message: rawMessage,
-          extras: extras,
-        ),
+      SentryManager.instance.captureException(
+        exception ?? rawMessage,
+        stackTrace: stackTrace,
+        message: rawMessage,
+        extras: extras,
       );
     }
   }

core/lib/utils/file_utils.dart

@@ -39,7 +39,7 @@ class FileUtils {
 
       return fileDirectory;
     } else {
-      throw DeviceNotSupportedException();
+      throw const DeviceNotSupportedException();
     }
   }
 

core/lib/utils/logger/log_tracking.dart

@@ -130,7 +130,7 @@ class LogTracking {
 
       return fileDirectory;
     } else {
-      throw DeviceNotSupportedException();
+      throw const DeviceNotSupportedException();
     }
   }