fix(android): Fix auto-start service not launching on boot (#36)

Author: Suyunjing

.gitignore

@@ -9,6 +9,7 @@
 .history
 .svn/
 migrate_working_dir/
+docs/
 
 # IntelliJ related
 *.iml

android/app/src/main/AndroidManifest.xml

@@ -82,26 +82,21 @@
             android:name="flutterEmbedding"
             android:value="2" />
 
-        <!-- OpenList后台服务 -->
+        <!-- OpenList background service -->
         <service
             android:name=".OpenListService"
-            android:exported="true"
+            android:exported="false"
             android:enabled="true"
-            android:foregroundServiceType="dataSync"
-            tools:ignore="ExportedService" />
+            android:foregroundServiceType="dataSync" />
 
-        <!-- 开机启动接收器 -->
+        <!-- Boot receiver -->
         <receiver
             android:name=".BootReceiver"
             android:exported="true"
             android:enabled="true">
-            <intent-filter android:priority="1000">
+            <intent-filter>
                 <action android:name="android.intent.action.BOOT_COMPLETED" />
-                <action android:name="android.intent.action.QUICKBOOT_POWERON" />
-                <action android:name="com.htc.intent.action.QUICKBOOT_POWERON" />
                 <action android:name="android.intent.action.MY_PACKAGE_REPLACED" />
-                <action android:name="android.intent.action.PACKAGE_REPLACED" />
-                <data android:scheme="package" />
             </intent-filter>
         </receiver>
 

android/app/src/main/kotlin/com/openlist/mobile/App.kt

@@ -1,13 +1,16 @@
 package com.openlist.mobile
 
 import android.app.Application
+import android.util.Log
+import com.openlist.mobile.model.openlist.OpenList
 import com.openlist.mobile.utils.ToastUtils.longToast
 import io.flutter.app.FlutterApplication
 
 val app by lazy { App.app }
 
 class App : FlutterApplication() {
     companion object {
+        private const val TAG = "App"
         lateinit var app: Application
     }
 
@@ -17,18 +20,27 @@ class App : FlutterApplication() {
 
         app = this
 
-        // 设置全局异常处理器来捕获未处理的异常
+        // Early initialization of OpenList to prepare for boot startup
+        try {
+            Log.d(TAG, "Performing early OpenList initialization")
+            OpenList.init()
+            Log.d(TAG, "OpenList early initialization completed")
+        } catch (e: Exception) {
+            Log.e(TAG, "Failed to initialize OpenList early", e)
+        }
+        
+        // Set global exception handler to catch uncaught exceptions
         Thread.setDefaultUncaughtExceptionHandler { thread, throwable ->
-            android.util.Log.e("App", "Uncaught exception in thread ${thread.name}", throwable)
+            Log.e(TAG, "Uncaught exception in thread ${thread.name}", throwable)
 
-            // 如果是 JNI 相关的错误，记录详细信息
+            // Log detailed info for JNI related errors
             if (throwable.message?.contains("JNI") == true || 
                 throwable.message?.contains("native") == true ||
                 throwable is UnsatisfiedLinkError) {
-                android.util.Log.e("App", "Native/JNI related crash detected")
+                Log.e(TAG, "Native/JNI related crash detected")
             }
 
-            // 调用默认的异常处理器
+            // Call default exception handler
             val defaultHandler = Thread.getDefaultUncaughtExceptionHandler()
             defaultHandler?.uncaughtException(thread, throwable)
         }

android/app/src/main/kotlin/com/openlist/mobile/BootReceiver.kt

@@ -8,87 +8,62 @@ import android.util.Log
 import com.openlist.mobile.config.AppConfig
 
 /**
- * 开机启动接收器 - 处理开机启动和包更新事件
+ * Boot receiver - handles device boot and package update events
  */
 class BootReceiver : BroadcastReceiver() {
     companion object {
         private const val TAG = "BootReceiver"
     }
 
-    override fun onReceive(context: Context, intent: Intent) {
-        val action = intent.action
-        Log.d(TAG, "Received broadcast: $action")
+    override fun onReceive(context: Context?, intent: Intent?) {
+        if (context == null || intent?.action == null) return
+        
+        Log.d(TAG, "Received broadcast: ${intent.action}")
 
-        try {
-            when (action) {
-                Intent.ACTION_BOOT_COMPLETED,
-                "android.intent.action.QUICKBOOT_POWERON",
-                "com.htc.intent.action.QUICKBOOT_POWERON" -> {
-                    Log.d(TAG, "Boot completed")
-                    handleBootCompleted(context)
-                }
-                
-                Intent.ACTION_MY_PACKAGE_REPLACED,
-                Intent.ACTION_PACKAGE_REPLACED -> {
-                    Log.d(TAG, "Package replaced")
-                    handlePackageReplaced(context, intent)
-                }
-                
-                else -> {
-                    Log.d(TAG, "Unknown action: $action")
-                }
+        when (intent.action) {
+            Intent.ACTION_BOOT_COMPLETED -> {
+                handleBootCompleted(context)
+            }
+            Intent.ACTION_MY_PACKAGE_REPLACED -> {
+                handlePackageReplaced(context)
             }
-        } catch (e: Exception) {
-            Log.e(TAG, "Error handling boot broadcast", e)
         }
     }
 
-    /**
-     * 处理开机完成事件
-     */
     private fun handleBootCompleted(context: Context) {
         if (!AppConfig.isStartAtBootEnabled) {
-            Log.d(TAG, "Auto start is disabled")
+            Log.d(TAG, "Auto-start disabled, skipping")
             return
         }
 
-        // 开机时清除手动停止标志，因为设备重启了
+        // Clear manual stop flag on boot
         AppConfig.isManuallyStoppedByUser = false
-        Log.d(TAG, "Manual stop flag cleared on boot")
-
-        Log.d(TAG, "Starting services after boot")
-        startServices(context)
+        
+        Log.d(TAG, "Starting OpenList service")
+        startService(context)
     }
 
-    /**
-     * 处理包更新事件
-     */
-    private fun handlePackageReplaced(context: Context, intent: Intent) {
-        val packageName = intent.dataString
-        if (packageName?.contains(context.packageName) == true) {
-            Log.d(TAG, "Our package was replaced, restarting services")
-            if (AppConfig.isStartAtBootEnabled) {
-                startServices(context)
-            }
+    private fun handlePackageReplaced(context: Context) {
+        if (!AppConfig.isStartAtBootEnabled) {
+            Log.d(TAG, "Auto-start disabled, skipping package update restart")
+            return
         }
+        
+        Log.d(TAG, "Starting OpenList service after package update")
+        startService(context)
     }
 
-    /**
-     * 启动所有必要的服务
-     */
-    private fun startServices(context: Context) {
+    private fun startService(context: Context) {
         try {
-            // 启动主服务
-            val mainServiceIntent = Intent(context, OpenListService::class.java)
+            val serviceIntent = Intent(context, OpenListService::class.java)
             if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.O) {
-                context.startForegroundService(mainServiceIntent)
+                context.startForegroundService(serviceIntent)
             } else {
-                context.startService(mainServiceIntent)
+                context.startService(serviceIntent)
             }
-            Log.d(TAG, "Main service start command sent")
-
+            Log.d(TAG, "Service start command sent")
         } catch (e: Exception) {
-            Log.e(TAG, "Failed to start services", e)
+            Log.e(TAG, "Failed to start service", e)
         }
     }
 }