Fix bugs

lib/bootstrap.dart

@@ -10,7 +10,6 @@ import 'package:hiddify/core/prefs/prefs.dart';
 import 'package:hiddify/data/data_providers.dart';
 import 'package:hiddify/data/repository/app_repository_impl.dart';
 import 'package:hiddify/domain/environment.dart';
-import 'package:hiddify/domain/failures.dart';
 import 'package:hiddify/features/common/active_profile/active_profile_notifier.dart';
 import 'package:hiddify/features/common/window/window_controller.dart';
 import 'package:hiddify/features/system_tray/system_tray.dart';
@@ -108,7 +107,6 @@ Future<void> _lazyBootstrap(
   runApp(
     ProviderScope(
       parent: container,
-      observers: [SentryRiverpodObserver()],
       child: SentryUserInteractionWidget(
         child: const AppView(),
       ),

lib/data/data_providers.dart

@@ -12,7 +12,6 @@ import 'package:hiddify/domain/core_facade.dart';
 import 'package:hiddify/domain/profiles/profiles.dart';
 import 'package:hiddify/services/service_providers.dart';
 import 'package:riverpod_annotation/riverpod_annotation.dart';
-import 'package:sentry_dio/sentry_dio.dart';
 import 'package:shared_preferences/shared_preferences.dart';
 
 part 'data_providers.g.dart';
@@ -34,7 +33,7 @@ Dio dio(DioRef ref) => Dio(
           "User-Agent": ref.watch(appInfoProvider).userAgent,
         },
       ),
-    )..addSentry();
+    );
 
 @Riverpod(keepAlive: true)
 ProfilesDao profilesDao(ProfilesDaoRef ref) => ProfilesDao(

lib/data/repository/profiles_repository_impl.dart

@@ -136,7 +136,7 @@ class ProfilesRepositoryImpl
             },
           );
         } finally {
-          if (await File(tempPath).exists()) await File(tempPath).delete();
+          if (File(tempPath).existsSync()) File(tempPath).deleteSync();
         }
       },
       (error, stackTrace) {
@@ -277,7 +277,7 @@ class ProfilesRepositoryImpl
             },
           );
         } finally {
-          if (await File(tempPath).exists()) await File(tempPath).delete();
+          if (File(tempPath).existsSync()) File(tempPath).deleteSync();
         }
       },
     );

lib/domain/connectivity/connection_failure.dart

@@ -15,9 +15,11 @@ sealed class ConnectionFailure with _$ConnectionFailure, Failure {
     StackTrace? stackTrace,
   ]) = UnexpectedConnectionFailure;
 
+  @With<ExpectedMeasuredFailure>()
   const factory ConnectionFailure.missingVpnPermission([String? message]) =
       MissingVpnPermission;
 
+  @With<ExpectedMeasuredFailure>()
   const factory ConnectionFailure.missingNotificationPermission([
     String? message,
   ]) = MissingNotificationPermission;
@@ -28,9 +30,9 @@ sealed class ConnectionFailure with _$ConnectionFailure, Failure {
   @override
   ({String type, String? message}) present(TranslationsEn t) {
     return switch (this) {
-      UnexpectedConnectionFailure(:final error) => (
+      UnexpectedConnectionFailure() => (
           type: t.failure.connectivity.unexpected,
-          message: t.mayPrintError(error),
+          message: null,
         ),
       MissingVpnPermission(:final message) => (
           type: t.failure.connectivity.missingVpnPermission,

lib/domain/core_service_failure.dart

@@ -14,7 +14,7 @@ sealed class CoreServiceFailure with _$CoreServiceFailure, Failure {
     StackTrace? stackTrace,
   ) = UnexpectedCoreServiceFailure;
 
-  @With<ExpectedException>()
+  @With<ExpectedFailure>()
   const factory CoreServiceFailure.serviceNotRunning([String? message]) =
       CoreServiceNotRunning;
 
@@ -22,6 +22,7 @@ sealed class CoreServiceFailure with _$CoreServiceFailure, Failure {
     String? message,
   ]) = InvalidConfigOptions;
 
+  @With<ExpectedMeasuredFailure>()
   const factory CoreServiceFailure.invalidConfig([
     String? message,
   ]) = InvalidConfig;
@@ -51,9 +52,9 @@ sealed class CoreServiceFailure with _$CoreServiceFailure, Failure {
   @override
   ({String type, String? message}) present(TranslationsEn t) {
     return switch (this) {
-      UnexpectedCoreServiceFailure(:final error) => (
+      UnexpectedCoreServiceFailure() => (
           type: t.failure.singbox.unexpected,
-          message: t.mayPrintError(error),
+          message: null,
         ),
       CoreServiceNotRunning(:final message) => (
           type: t.failure.singbox.serviceNotRunning,

lib/domain/failures.dart

@@ -1,68 +1,73 @@
 import 'package:dio/dio.dart';
 import 'package:hiddify/core/prefs/prefs.dart';
 
+typedef PresentableError = ({String type, String? message});
+
 mixin Failure {
   ({String type, String? message}) present(TranslationsEn t);
 }
 
+/// failures that are not expected to happen but depending on [error] type might not be relevant (eg network errors)
 mixin UnexpectedFailure {
   Object? get error;
   StackTrace? get stackTrace;
 }
 
+/// failures that are expected to happen and should be handled by the app
+/// and should be logged, eg missing permissions
+mixin ExpectedMeasuredFailure {}
+
 /// failures ignored by analytics service etc.
-mixin ExpectedException {}
+mixin ExpectedFailure {}
 
 extension ErrorPresenter on TranslationsEn {
-  String? _errorToMessage(Object error) {
-    switch (error) {
-      case Failure():
-        final err = error.present(this);
-        return err.type + (err.message == null ? "" : ": ${err.message}");
-      case DioException():
-        return error.present(this);
-      default:
-        return null;
-    }
-  }
+  PresentableError errorToPair(Object error) => switch (error) {
+        UnexpectedFailure(error: final nestedErr?) => errorToPair(nestedErr),
+        Failure() => error.present(this),
+        DioException() => error.present(this),
+        _ => (type: failure.unexpected, message: null),
+      };
 
-  String printError(Object error) =>
-      _errorToMessage(error) ?? failure.unexpected;
-
-  String? mayPrintError(Object? error) =>
-      error != null ? _errorToMessage(error) : null;
-
-  ({String type, String? message}) presentError(
+  PresentableError presentError(
     Object error, {
     String? action,
   }) {
-    final ({String type, String? message}) presentable;
-    if (error case Failure()) {
-      presentable = error.present(this);
-    } else {
-      presentable = (type: failure.unexpected, message: null);
-    }
+    final pair = errorToPair(error);
+    if (action == null) return pair;
     return (
-      type: action == null ? presentable.type : "$action: ${presentable.type}",
-      message: presentable.message,
+      type: action,
+      message: pair.type + (pair.message == null ? "" : "\n${pair.message!}"),
     );
   }
+
+  String presentShortError(
+    Object error, {
+    String? action,
+  }) {
+    final pair = errorToPair(error);
+    if (action == null) return pair.type;
+    return "$action: ${pair.type}";
+  }
 }
 
 extension DioExceptionPresenter on DioException {
-  String presentType(TranslationsEn t) => switch (type) {
+  PresentableError present(TranslationsEn t) => switch (type) {
         DioExceptionType.connectionTimeout ||
         DioExceptionType.sendTimeout ||
         DioExceptionType.receiveTimeout =>
-          t.failure.connection.timeout,
-        DioExceptionType.badCertificate => t.failure.connection.badCertificate,
-        DioExceptionType.badResponse => t.failure.connection.badResponse,
-        DioExceptionType.connectionError =>
-          t.failure.connection.connectionError,
-        _ => t.failure.unexpected,
+          (type: t.failure.connection.timeout, message: null),
+        DioExceptionType.badCertificate => (
+            type: t.failure.connection.badCertificate,
+            message: message,
+          ),
+        DioExceptionType.badResponse => (
+            type: t.failure.connection.badResponse,
+            message: message,
+          ),
+        DioExceptionType.connectionError => (
+            type: t.failure.connection.connectionError,
+            message: message,
+          ),
+        _ => (type: t.failure.connection.unexpected, message: message),
       };
-
-  String present(TranslationsEn t) {
-    return presentType(t) + (message == null ? "" : "\n$message");
-  }
 }

lib/domain/profiles/profile.dart

@@ -34,6 +34,7 @@ sealed class Profile with _$Profile {
     required DateTime lastUpdate,
   }) = LocalProfile;
 
+  // ignore: prefer_constructors_over_static_methods
   static RemoteProfile fromResponse(
     String url,
     Map<String, List<String>> headers,

lib/domain/profiles/profiles_failure.dart

@@ -16,18 +16,19 @@ sealed class ProfileFailure with _$ProfileFailure, Failure {
 
   const factory ProfileFailure.notFound() = ProfileNotFoundFailure;
 
-  @With<ExpectedException>()
+  @With<ExpectedFailure>()
   const factory ProfileFailure.invalidUrl() = ProfileInvalidUrlFailure;
 
+  @With<ExpectedFailure>()
   const factory ProfileFailure.invalidConfig([String? message]) =
       ProfileInvalidConfigFailure;
 
   @override
   ({String type, String? message}) present(TranslationsEn t) {
     return switch (this) {
-      ProfileUnexpectedFailure(:final error) => (
+      ProfileUnexpectedFailure() => (
           type: t.failure.profiles.unexpected,
-          message: t.mayPrintError(error),
+          message: null,
         ),
       ProfileNotFoundFailure() => (
           type: t.failure.profiles.notFound,

lib/features/about/view/about_page.dart

@@ -31,7 +31,7 @@ class AboutPage extends HookConsumerWidget {
               canIgnore: false,
             ).show(context);
           case AppUpdateStateError(:final error):
-            return CustomToast.error(t.printError(error)).show(context);
+            return CustomToast.error(t.presentShortError(error)).show(context);
           case AppUpdateStateNotAvailable():
             return CustomToast.success(t.appUpdate.notAvailableMsg)
                 .show(context);

lib/features/common/common_controllers.dart

@@ -1,4 +1,3 @@
-import 'package:dartx/dartx.dart';
 import 'package:hiddify/core/prefs/general_prefs.dart';
 import 'package:hiddify/features/common/app_update_notifier.dart';
 import 'package:hiddify/features/common/connectivity/connectivity_controller.dart';
@@ -20,8 +19,7 @@ void commonControllers(CommonControllersRef ref) {
     introCompletedProvider,
     (_, completed) async {
       if (completed) {
-        await Future.delayed(5.seconds)
-            .then((_) async => ref.read(cronServiceProvider).startScheduler());
+        await ref.read(cronServiceProvider).startScheduler();
       }
     },
     fireImmediately: true,

lib/features/common/new_version_dialog.dart

@@ -7,23 +7,30 @@ import 'package:hiddify/utils/utils.dart';
 import 'package:hooks_riverpod/hooks_riverpod.dart';
 
 // TODO add release notes
-class NewVersionDialog extends HookConsumerWidget {
-  const NewVersionDialog(
+class NewVersionDialog extends HookConsumerWidget with PresLogger {
+  NewVersionDialog(
     this.currentVersion,
     this.newVersion, {
-    super.key,
+    // super.key,
     this.canIgnore = true,
-  });
+  }) : super(key: _dialogKey);
 
   final String currentVersion;
   final RemoteVersionInfo newVersion;
   final bool canIgnore;
 
-  Future<void> show(BuildContext context) {
-    return showDialog(
-      context: context,
-      builder: (context) => this,
-    );
+  static final _dialogKey = GlobalKey(debugLabel: 'new version dialog');
+
+  Future<void> show(BuildContext context) async {
+    if (_dialogKey.currentContext == null) {
+      return showDialog(
+        context: context,
+        useRootNavigator: true,
+        builder: (context) => this,
+      );
+    } else {
+      loggy.warning("new version dialog is already open");
+    }
   }
 
   @override

lib/features/common/profile_tile.dart

@@ -32,7 +32,7 @@ class ProfileTile extends HookConsumerWidget {
 
     final selectActiveMutation = useMutation(
       initialOnFailure: (err) {
-        CustomToast.error(t.printError(err)).show(context);
+        CustomToast.error(t.presentShortError(err)).show(context);
       },
       initialOnSuccess: () {
         if (context.mounted) context.pop();

lib/features/home/view/home_page.dart

@@ -68,7 +68,7 @@ class HomePage extends HookConsumerWidget {
                     _ => const EmptyProfilesHomeBody(),
                   },
                 AsyncError(:final error) =>
-                  SliverErrorBodyPlaceholder(t.printError(error)),
+                  SliverErrorBodyPlaceholder(t.presentShortError(error)),
                 _ => const SliverToBoxAdapter(),
               },
             ],

lib/features/intro/view/intro_page.dart

@@ -69,8 +69,16 @@ class IntroPage extends HookConsumerWidget with PresLogger {
                   child: FilledButton(
                     onPressed: () async {
                       if (!ref.read(enableAnalyticsProvider)) {
-                        loggy.debug("disabling analytics per user request");
-                        await Sentry.close();
+                        loggy.info("disabling analytics per user request");
+                        try {
+                          await Sentry.close();
+                        } catch (error, stackTrace) {
+                          loggy.warning(
+                            "could not disable analytics",
+                            error,
+                            stackTrace,
+                          );
+                        }
                       }
                       await ref
                           .read(introCompletedProvider.notifier)

lib/features/logs/view/logs_page.dart

@@ -138,7 +138,7 @@ class LogsPage extends HookConsumerWidget with PresLogger {
               NestedTabAppBar(
                 title: Text(t.logs.pageTitle),
               ),
-              SliverErrorBodyPlaceholder(t.printError(error)),
+              SliverErrorBodyPlaceholder(t.presentShortError(error)),
             ],
           ),
         );

lib/features/profile_detail/view/profile_detail_page.dart

@@ -29,7 +29,15 @@ class ProfileDetailPage extends HookConsumerWidget with PresLogger {
         if (asyncSave case AsyncData(value: final save)) {
           switch (save) {
             case MutationFailure(:final failure):
-              CustomAlertDialog.fromErr(t.presentError(failure)).show(context);
+              final String action;
+              if (ref.read(provider) case AsyncData(value: final data)
+                  when data.isEditing) {
+                action = t.profile.save.failureMsg;
+              } else {
+                action = t.profile.add.failureMsg;
+              }
+              CustomAlertDialog.fromErr(t.presentError(failure, action: action))
+                  .show(context);
             case MutationSuccess():
               CustomToast.success(t.profile.save.successMsg).show(context);
               WidgetsBinding.instance.addPostFrameCallback(
@@ -62,7 +70,7 @@ class ProfileDetailPage extends HookConsumerWidget with PresLogger {
         if (asyncDelete case AsyncData(value: final delete)) {
           switch (delete) {
             case MutationFailure(:final failure):
-              CustomToast.error(t.printError(failure)).show(context);
+              CustomToast.error(t.presentShortError(failure)).show(context);
             case MutationSuccess():
               CustomToast.success(t.profile.delete.successMsg).show(context);
               WidgetsBinding.instance.addPostFrameCallback(
@@ -261,7 +269,7 @@ class ProfileDetailPage extends HookConsumerWidget with PresLogger {
                 title: Text(t.profile.detailsPageTitle),
                 pinned: true,
               ),
-              SliverErrorBodyPlaceholder(t.printError(error)),
+              SliverErrorBodyPlaceholder(t.presentShortError(error)),
             ],
           ),
         );

lib/features/profiles/view/add_profile_modal.dart

@@ -35,7 +35,9 @@ class AddProfileModal extends HookConsumerWidget {
             t.failure.profiles.invalidUrl,
           ).show(context);
         } else {
-          CustomAlertDialog.fromErr(t.presentError(err)).show(context);
+          CustomAlertDialog.fromErr(
+            t.presentError(err, action: t.profile.add.failureMsg),
+          ).show(context);
         }
       },
       initialOnSuccess: () {

lib/features/profiles/view/profiles_modal.dart

@@ -37,7 +37,7 @@ class ProfilesModal extends HookConsumerWidget {
                   itemCount: profiles.length,
                 ),
               AsyncError(:final error) => SliverErrorBodyPlaceholder(
-                  t.printError(error),
+                  t.presentShortError(error),
                 ),
               AsyncLoading() => const SliverLoadingBodyPlaceholder(),
               _ => const SliverToBoxAdapter(),

lib/features/proxies/view/proxies_page.dart

@@ -20,7 +20,7 @@ class ProxiesPage extends HookConsumerWidget with PresLogger {
 
     final selectActiveProxyMutation = useMutation(
       initialOnFailure: (error) =>
-          CustomToast.error(t.printError(error)).show(context),
+          CustomToast.error(t.presentShortError(error)).show(context),
     );
 
     switch (asyncProxies) {
@@ -144,7 +144,7 @@ class ProxiesPage extends HookConsumerWidget with PresLogger {
                 title: Text(t.proxies.pageTitle),
               ),
               SliverErrorBodyPlaceholder(
-                t.printError(error),
+                t.presentShortError(error),
                 icon: null,
               ),
             ],

lib/services/cron_service.dart

@@ -55,9 +55,9 @@ class CronService with InfraLogger {
     int runCount = 0;
     _scheduler = NeatPeriodicTaskScheduler(
       name: "cron job scheduler",
-      interval: const Duration(minutes: 5),
-      timeout: const Duration(seconds: 15),
-      minCycle: const Duration(minutes: 1),
+      interval: const Duration(minutes: 10),
+      timeout: const Duration(minutes: 5),
+      minCycle: const Duration(minutes: 2),
       task: () {
         loggy.debug("in run ${runCount++}");
         return Future.wait(jobs.values.map(run));

lib/utils/alerts.dart

@@ -20,6 +20,7 @@ class CustomAlertDialog extends StatelessWidget {
   Future<void> show(BuildContext context) async {
     await showDialog(
       context: context,
+      useRootNavigator: true,
       builder: (context) => this,
     );
   }

lib/utils/sentry_loggy_integration.dart

@@ -34,7 +34,7 @@ class SentryLoggyIntegration extends LoggyPrinter
 
   @override
   Future<void> onLog(LogRecord record) async {
-    if (!canSendEvent(record.error)) return;
+    if (!canLogEvent(record.error)) return;
 
     if (_shouldLog(record.level, _minEventLevel)) {
       await _hub.captureEvent(

lib/utils/sentry_utils.dart

@@ -15,7 +15,13 @@ bool canSendEvent(dynamic throwable) {
     UnexpectedFailure(:final error) => canSendEvent(error),
     DioException _ => false,
     SocketException _ => false,
-    ExpectedException _ => false,
+    ExpectedFailure _ => false,
+    ExpectedMeasuredFailure _ => false,
     _ => true,
   };
 }
+
+bool canLogEvent(dynamic throwable) => switch (throwable) {
+      ExpectedMeasuredFailure _ => true,
+      _ => canSendEvent(throwable),
+    };