Fix: Remove Firebase/Timber from App.kt, simplify AppSyncWorker (no HiltWorker)

Author: risunCode

app/src/main/java/com/appcontrolx/App.kt

@@ -1,24 +1,19 @@
 package com.appcontrolx
 
 import android.app.Application
+import android.util.Log
 import androidx.appcompat.app.AppCompatDelegate
-import androidx.hilt.work.HiltWorkerFactory
 import androidx.preference.PreferenceManager
-import androidx.work.Configuration
 import com.appcontrolx.utils.Constants
-import com.google.firebase.crashlytics.FirebaseCrashlytics
 import com.topjohnwu.superuser.Shell
 import dagger.hilt.android.HiltAndroidApp
-import timber.log.Timber
-import javax.inject.Inject
 
 @HiltAndroidApp
-class App : Application(), Configuration.Provider {
-    
-    @Inject
-    lateinit var workerFactory: HiltWorkerFactory
+class App : Application() {
 
     companion object {
+        private const val TAG = "AppControlX"
+        
         init {
             Shell.enableVerboseLogging = BuildConfig.DEBUG
             Shell.setDefaultBuilder(
@@ -31,50 +26,13 @@ class App : Application(), Configuration.Provider {
 
     override fun onCreate() {
         super.onCreate()
-        initTimber()
-        initCrashlytics()
         applyTheme()
-    }
-    
-    private fun initTimber() {
-        if (BuildConfig.DEBUG) {
-            Timber.plant(Timber.DebugTree())
-        } else {
-            Timber.plant(CrashlyticsTree())
-        }
-        Timber.d("App initialized")
-    }
-    
-    private fun initCrashlytics() {
-        FirebaseCrashlytics.getInstance().apply {
-            setCrashlyticsCollectionEnabled(!BuildConfig.DEBUG)
-            setCustomKey("app_version", BuildConfig.VERSION_NAME)
-            setCustomKey("version_code", BuildConfig.VERSION_CODE)
-        }
+        Log.d(TAG, "App initialized")
     }
 
     private fun applyTheme() {
         val prefs = PreferenceManager.getDefaultSharedPreferences(this)
         val theme = prefs.getInt(Constants.PREFS_THEME, AppCompatDelegate.MODE_NIGHT_FOLLOW_SYSTEM)
         AppCompatDelegate.setDefaultNightMode(theme)
     }
-    
-    override val workManagerConfiguration: Configuration
-        get() = Configuration.Builder()
-            .setWorkerFactory(workerFactory)
-            .build()
-    
-    /**
-     * Custom Timber tree that logs to Crashlytics in production
-     */
-    private class CrashlyticsTree : Timber.Tree() {
-        override fun log(priority: Int, tag: String?, message: String, t: Throwable?) {
-            if (priority == android.util.Log.ERROR || priority == android.util.Log.WARN) {
-                FirebaseCrashlytics.getInstance().apply {
-                    log("$tag: $message")
-                    t?.let { recordException(it) }
-                }
-            }
-        }
-    }
 }

app/src/main/java/com/appcontrolx/worker/AppSyncWorker.kt

@@ -1,34 +1,30 @@
 package com.appcontrolx.worker
 
 import android.content.Context
-import androidx.hilt.work.HiltWorker
+import android.util.Log
 import androidx.work.CoroutineWorker
 import androidx.work.WorkerParameters
-import com.appcontrolx.data.repository.AppRepository
-import dagger.assisted.Assisted
-import dagger.assisted.AssistedInject
-import kotlinx.coroutines.flow.first
-import timber.log.Timber
+import com.appcontrolx.service.AppFetcher
 
-@HiltWorker
-class AppSyncWorker @AssistedInject constructor(
-    @Assisted context: Context,
-    @Assisted params: WorkerParameters,
-    private val repository: AppRepository
+class AppSyncWorker(
+    context: Context,
+    params: WorkerParameters
 ) : CoroutineWorker(context, params) {
 
+    private val appFetcher = AppFetcher(context)
+    
     override suspend fun doWork(): Result {
         return try {
-            Timber.d("Starting app sync worker")
+            Log.d(TAG, "Starting app sync worker")
 
             // Pre-fetch apps to cache
-            repository.getUserApps().first()
-            repository.getSystemApps().first()
+            appFetcher.getUserApps()
+            appFetcher.getSystemApps()
 
-            Timber.d("App sync completed successfully")
+            Log.d(TAG, "App sync completed successfully")
             Result.success()
         } catch (e: Exception) {
-            Timber.e(e, "App sync failed")
+            Log.e(TAG, "App sync failed", e)
             if (runAttemptCount < 3) {
                 Result.retry()
             } else {
@@ -38,6 +34,7 @@ class AppSyncWorker @AssistedInject constructor(
     }
 
     companion object {
+        private const val TAG = "AppSyncWorker"
         const val WORK_NAME = "app_sync_worker"
     }
 }