From 5a628bb44a659884cdad88f636cd6bdd2a57380c Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Mon, 11 Aug 2025 09:37:50 +0000
Subject: [PATCH] Fix network monitoring initialization - NetworkMonitor
 service now properly registers and listens for connectivity changes

Co-authored-by: monkeyWie <13160176+monkeyWie@users.noreply.github.com>
---
 .../app/modules/app/bindings/app_binding.dart    | 14 --------------
 .../app/modules/root/bindings/root_binding.dart  |  2 ++
 ui/flutter/lib/util/network_monitor.dart         | 16 ++++++++++++++--
 3 files changed, 16 insertions(+), 16 deletions(-)
 delete mode 100644 ui/flutter/lib/app/modules/app/bindings/app_binding.dart

diff --git a/ui/flutter/lib/app/modules/app/bindings/app_binding.dart b/ui/flutter/lib/app/modules/app/bindings/app_binding.dart
deleted file mode 100644
index 3f109299..00000000
--- a/ui/flutter/lib/app/modules/app/bindings/app_binding.dart
+++ /dev/null
@@ -1,14 +0,0 @@
-import 'package:get/get.dart';
-
-import '../../../../util/network_monitor.dart';
-import '../controllers/app_controller.dart';
-
-class AppBinding extends Bindings {
-  @override
-  void dependencies() {
-    Get.lazyPut<AppController>(
-      () => AppController(),
-    );
-    Get.put<NetworkMonitor>(NetworkMonitor(), permanent: true);
-  }
-}
diff --git a/ui/flutter/lib/app/modules/root/bindings/root_binding.dart b/ui/flutter/lib/app/modules/root/bindings/root_binding.dart
index 2a106a1d..9ae0a49f 100644
--- a/ui/flutter/lib/app/modules/root/bindings/root_binding.dart
+++ b/ui/flutter/lib/app/modules/root/bindings/root_binding.dart
@@ -1,10 +1,12 @@
 import 'package:get/get.dart';
 
+import '../../../../util/network_monitor.dart';
 import '../controllers/root_controller.dart';
 
 class RootBinding extends Bindings {
   @override
   void dependencies() {
     Get.lazyPut<RootController>(() => RootController(), fenix: true);
+    Get.put<NetworkMonitor>(NetworkMonitor(), permanent: true);
   }
 }
diff --git a/ui/flutter/lib/util/network_monitor.dart b/ui/flutter/lib/util/network_monitor.dart
index fe476868..10417f21 100644
--- a/ui/flutter/lib/util/network_monitor.dart
+++ b/ui/flutter/lib/util/network_monitor.dart
@@ -26,8 +26,11 @@ class NetworkMonitor extends GetxService {
   Future<void> onInit() async {
     super.onInit();
     
+    logInfo('NetworkMonitor.onInit() called');
+    
     // Only monitor network on mobile platforms
     if (!Platform.isAndroid && !Platform.isIOS) {
+      logInfo('NetworkMonitor: Not on mobile platform, skipping initialization');
       return;
     }
     
@@ -35,8 +38,10 @@ class NetworkMonitor extends GetxService {
     
     // Check initial connectivity state
     final initialResult = await _connectivity.checkConnectivity();
+    logInfo('NetworkMonitor: Initial connectivity result: $initialResult');
     _updateNetworkStatus(initialResult);
     _previousNetworkType = _getNetworkType(initialResult);
+    logInfo('NetworkMonitor: Initial network type: $_previousNetworkType');
     _isInitialized = true;
     
     // Listen to connectivity changes
@@ -47,7 +52,7 @@ class NetworkMonitor extends GetxService {
       },
     );
     
-    logInfo('Network monitor initialized');
+    logInfo('Network monitor initialized successfully');
   }
 
   @override
@@ -75,11 +80,18 @@ class NetworkMonitor extends GetxService {
   }
 
   void _onConnectivityChanged(List<ConnectivityResult> result) {
-    if (!_isInitialized) return;
+    logInfo('NetworkMonitor: Connectivity changed event received: $result');
+    
+    if (!_isInitialized) {
+      logInfo('NetworkMonitor: Not initialized yet, ignoring connectivity change');
+      return;
+    }
     
     // Check if network auto control is enabled
     final isNetworkAutoControlEnabled = Database.instance.getNetworkAutoControl();
+    logInfo('NetworkMonitor: Network auto control enabled: $isNetworkAutoControlEnabled');
     if (!isNetworkAutoControlEnabled) {
+      logInfo('NetworkMonitor: Network auto control disabled, ignoring connectivity change');
       return;
     }