Merge pull request #147 from MostroP2P/fix/null-check-operator-deep-link

Fixed Null check operator used on a null value err

Author: grunch

lib/core/app_routes.dart

@@ -31,6 +31,11 @@ GoRouter createRouter(WidgetRef ref) {
     navigatorKey: GlobalKey<NavigatorState>(),
     initialLocation: '/',
     redirect: (context, state) {
+      // Redirect custom schemes to home to prevent assertion failures
+      if (state.uri.scheme == 'mostro' || 
+          (!state.uri.scheme.startsWith('http') && state.uri.scheme.isNotEmpty)) {
+        return '/';
+      }
       final firstRunState = ref.read(firstRunProvider);
 
       return firstRunState.when(

lib/core/deep_link_handler.dart

@@ -72,16 +72,23 @@ class DeepLinkHandler {
     try {
       // Show loading indicator
       context = router.routerDelegate.navigatorKey.currentContext;
-      if (context != null) {
+      if (context != null && context.mounted) {
         _showLoadingDialog(context);
       }
 
       // Get the services
       final nostrService = _ref.read(nostrServiceProvider);
       final deepLinkService = _ref.read(deepLinkServiceProvider);
 
+      // Ensure we have a valid context for processing
+      final processingContext = context ?? router.routerDelegate.navigatorKey.currentContext;
+      if (processingContext == null || !processingContext.mounted) {
+        _logger.e('No valid context available for deep link processing');
+        return;
+      }
+
       // Process the mostro link
-      final result = await deepLinkService.processMostroLink(url, nostrService, context!);
+      final result = await deepLinkService.processMostroLink(url, nostrService, processingContext);
 
       // Get fresh context after async operation
       final currentContext = router.routerDelegate.navigatorKey.currentContext;

lib/core/deep_link_interceptor.dart

@@ -21,11 +21,11 @@ class DeepLinkInterceptor extends WidgetsBindingObserver {
   @override
   Future<bool> didPushRouteInformation(RouteInformation routeInformation) async {
     final uri = routeInformation.uri;
-    _logger.i('Route information received: $uri');
+    _logger.i('DeepLinkInterceptor: Route information received: $uri');
 
     // Check if this is a custom scheme URL
     if (_isCustomScheme(uri)) {
-      _logger.i('Custom scheme detected: ${uri.scheme}, intercepting');
+      _logger.i('DeepLinkInterceptor: Custom scheme detected: ${uri.scheme}, intercepting and preventing GoRouter processing');
 
       // Emit the custom URL for processing
       _customUrlController.add(uri.toString());
@@ -35,12 +35,32 @@ class DeepLinkInterceptor extends WidgetsBindingObserver {
       return true;
     }
 
+    _logger.i('DeepLinkInterceptor: Allowing normal URL to pass through: $uri');
     // Let normal URLs pass through to GoRouter
     return super.didPushRouteInformation(routeInformation);
   }
 
   // Note: didPushRoute is deprecated, but we keep it for compatibility
   // The main handling is done in didPushRouteInformation above
+  @override
+  // ignore: deprecated_member_use
+  Future<bool> didPushRoute(String route) async {
+    _logger.i('DeepLinkInterceptor: didPushRoute called with: $route');
+    
+    try {
+      final uri = Uri.parse(route);
+      if (_isCustomScheme(uri)) {
+        _logger.i('DeepLinkInterceptor: Custom scheme detected in didPushRoute: ${uri.scheme}, intercepting');
+        _customUrlController.add(route);
+        return true;
+      }
+    } catch (e) {
+      _logger.w('DeepLinkInterceptor: Error parsing route in didPushRoute: $e');
+    }
+    
+    // ignore: deprecated_member_use
+    return super.didPushRoute(route);
+  }
 
   /// Check if the URI uses a custom scheme
   bool _isCustomScheme(Uri uri) {