wip: commit for test and review Koin refactored code

app-watch/app/build.gradle.kts

@@ -1,7 +1,6 @@
 plugins {
     alias(libs.plugins.composenews.android.application)
     alias(libs.plugins.composenews.android.application.compose)
-    alias(libs.plugins.composenews.android.hilt)
 }
 
 android {
@@ -43,9 +42,7 @@ dependencies {
     }
     libs.apply {
         implementation(androidx.ktx)
-        implementation(hilt.work)
         implementation(lifecycle.runtime.ktx)
-        implementation(hilt.navigation.compose)
         implementation(work.runtime.ktx)
         implementation(compose.activity)
         implementation(compose.ui.preview.wear)

app/build.gradle.kts

@@ -2,7 +2,7 @@
 plugins {
     alias(libs.plugins.composenews.android.application)
     alias(libs.plugins.composenews.android.application.compose)
-    alias(libs.plugins.composenews.android.hilt)}
+}
 
 android {
     namespace = libs.versions.projectApplicationId.get()
@@ -48,10 +48,10 @@ dependencies {
     libs.apply {
         implementation(compose.activity)
         implementation(androidx.ktx)
-        implementation(hilt.work)
+        implementation(koin.work)
         implementation(lifecycle.runtime.ktx)
         implementation(work.runtime.ktx)
-        implementation(hilt.navigation.compose)
+        implementation(bundles.koin)
         implementation(compose.material3.adaptive.navigation.suite)
         implementation(compose.material3.adaptive.navigation)
         implementation(espresso.core)

app/proguard-rules.pro

@@ -44,11 +44,6 @@
 
 -keep class * extends androidx.compose.runtime.Composable { *; }
 
--keepclassmembers class * {
-    @dagger.hilt.android.lifecycle.HiltViewModel *;
-}
--keep,@dagger.hilt.InstallIn class * { *; }
--keep class dagger.hilt.** { *; }
 -dontwarn dagger.internal.codegen.ComponentProcessor
 
 -keep class retrofit2.** { *; }

app/src/main/AndroidManifest.xml

@@ -29,6 +29,18 @@
                 <category android:name="android.intent.category.LAUNCHER" />
             </intent-filter>
         </activity>
+
+        <provider
+            android:name="androidx.startup.InitializationProvider"
+            android:authorities="${applicationId}.androidx-startup"
+            android:exported="false"
+            tools:node="merge">
+            <meta-data
+                android:name="androidx.work.WorkManagerInitializer"
+                android:value="androidx.startup"
+                tools:node="remove" />
+        </provider>
+
     </application>
 
 </manifest>

app/src/main/java/ir/composenews/ComposeNewsApplication.kt

@@ -1,15 +1,31 @@
+@file:OptIn(KoinExperimentalAPI::class)
+
 package ir.composenews
 
 import android.app.Application
-import dagger.hilt.android.HiltAndroidApp
+import ir.composenews.navigation.di.navigationRootModule
 import ir.composenews.sync.Sync
+import ir.composenews.sync.worker.workerModule
+import org.koin.android.ext.koin.androidContext
+import org.koin.androidx.workmanager.koin.workManagerFactory
+import org.koin.androix.startup.KoinStartup
+import org.koin.core.annotation.KoinExperimentalAPI
+import org.koin.dsl.KoinConfiguration
 
-@HiltAndroidApp
-class ComposeNewsApplication : Application() {
+class ComposeNewsApplication : Application(), KoinStartup {
 
     override fun onCreate() {
         super.onCreate()
-        Sync.init(this)
+//        Sync.init(this) // TODO causes crash. temporary commented so our workmanager initializer disabled
+    }
+
+        override fun onKoinStartup() = KoinConfiguration {
+        androidContext(this@ComposeNewsApplication)
+        workManagerFactory()
+        modules(
+            workerModule,
+            navigationRootModule,
+        )
     }
 
 }

app/src/main/java/ir/composenews/ui/MainActivity.kt

@@ -5,13 +5,11 @@ import androidx.activity.ComponentActivity
 import androidx.activity.compose.setContent
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.tooling.preview.Preview
-import dagger.hilt.android.AndroidEntryPoint
 import ir.composenews.designsystem.theme.ComposeNewsTheme
 import ir.composenews.permission.enum.PermissionType
 import ir.composenews.permission.manager.PermissionManager
 import ir.composenews.permission.manager.PermissionManagerImpl
 
-@AndroidEntryPoint
 class MainActivity : ComponentActivity(), PermissionManager by PermissionManagerImpl() {
 
     override fun onCreate(savedInstanceState: Bundle?) {

build-logic/convention/build.gradle.kts

@@ -46,10 +46,6 @@ gradlePlugin {
             implementationClass = "AndroidApplicationComposeConventionPlugin"
         }
 
-        register("androidHilt") {
-            id = libs.plugins.composenews.android.hilt.get().pluginId
-            implementationClass = "HiltConventionPlugin"
-        }
         register("androidLibrary") {
             id = libs.plugins.composenews.android.library.asProvider().get().pluginId
             implementationClass = "AndroidLibraryConventionPlugin"

build-logic/convention/src/main/kotlin/AndroidFeatureConventionPlugin.kt

@@ -21,13 +21,11 @@ class AndroidFeatureConventionPlugin : Plugin<Project> {
         pluginManager.apply {
             apply(libs.findPlugin("composenews-android-library").get().get().pluginId)
             apply(libs.findPlugin("composenews-android-library.compose").get().get().pluginId)
-            apply(libs.findPlugin("composenews-android-hilt").get().get().pluginId)
         }
     }
 
     private fun Project.applyDependencies() {
         dependencies {
-            "implementation"(libs.findLibrary("hilt.navigation.compose").get())
             "androidTestImplementation"(libs.findLibrary("runner").get())
 
             "testImplementation"(project(":core:test"))

build.gradle.kts

@@ -15,7 +15,6 @@ plugins {
     alias(libs.plugins.kotlin.android) apply false
     alias(libs.plugins.ksp) apply false
     alias(libs.plugins.kotlin.parcelize) apply false
-    alias(libs.plugins.hilt.android) apply false
     alias(libs.plugins.kotliner) apply false
     alias(libs.plugins.detekt) apply true // Needs to be applied at the root, unlike others.
     alias(libs.plugins.compose) apply false

core/sync/build.gradle.kts

@@ -1,6 +1,5 @@
 plugins {
     alias(libs.plugins.composenews.android.library)
-    alias(libs.plugins.composenews.android.hilt)
 }
 
 android {
@@ -10,7 +9,7 @@ android {
 dependencies {
     api(projects.domain.market)
     libs.apply {
-        implementation(hilt.work)
+        implementation(koin.work)
         implementation(startup.runtime)
         implementation(work.runtime.ktx)
     }

core/sync/src/main/java/ir/composenews/sync/worker/SyncWorker.kt

@@ -11,24 +11,22 @@
 package ir.composenews.sync.worker
 
 import android.content.Context
-import androidx.hilt.work.HiltWorker
 import androidx.work.CoroutineWorker
 import androidx.work.ForegroundInfo
 import androidx.work.OneTimeWorkRequestBuilder
 import androidx.work.OutOfQuotaPolicy
 import androidx.work.WorkerParameters
-import dagger.assisted.Assisted
-import dagger.assisted.AssistedInject
 import ir.composenews.domain.use_case.SyncMarketListUseCase
 import ir.composenews.sync.SyncConstraints
 import ir.composenews.sync.syncForegroundInfo
+import org.koin.core.component.KoinComponent
 
-@HiltWorker
-class SyncWorker @AssistedInject constructor(
-    @Assisted private val appContext: Context,
-    @Assisted workerParams: WorkerParameters,
+class SyncWorker(
+    private val appContext: Context,
+    workerParams: WorkerParameters,
     private val syncMarketListUseCase: SyncMarketListUseCase,
-) : CoroutineWorker(appContext, workerParams) {
+) : CoroutineWorker(appContext, workerParams),
+    KoinComponent {
 
     override suspend fun doWork(): Result {
         return try {
@@ -43,10 +41,9 @@ class SyncWorker @AssistedInject constructor(
         appContext.syncForegroundInfo()
 
     companion object {
-        fun startUpSyncWork() = OneTimeWorkRequestBuilder<DelegatingWorker>()
+        fun startUpSyncWork() = OneTimeWorkRequestBuilder<SyncWorker>()
             .setExpedited(OutOfQuotaPolicy.RUN_AS_NON_EXPEDITED_WORK_REQUEST)
             .setConstraints(SyncConstraints)
-            .setInputData(SyncWorker::class.delegatedData())
             .build()
     }
 }

core/sync/src/main/java/ir/composenews/sync/worker/WorkerModule.kt

@@ -0,0 +1,24 @@
+@file:Suppress(
+    "TopLevelPropertyNaming",
+    "Indentation",
+    "SwallowedException",
+    "TooGenericExceptionCaught",
+    "ktlint:standard:chain-method-continuation",
+    "ktlint:standard:no-empty-first-line-in-class-body",
+    "ktlint:standard:multiline-expression-wrapping",
+    "ktlint:standard:blank-line-before-declaration",
+    "ktlint:standard:trailing-comma-on-call-site"
+)
+
+package ir.composenews.sync.worker
+
+import ir.composenews.domain.use_case.SyncMarketListUseCase
+import org.koin.androidx.workmanager.dsl.workerOf
+import org.koin.core.module.dsl.factoryOf
+import org.koin.dsl.module
+
+val workerModule = module {
+
+    factoryOf(::SyncMarketListUseCase)
+    workerOf(::SyncWorker)
+}

core/test/build.gradle.kts

@@ -1,6 +1,5 @@
 plugins {
     alias(libs.plugins.composenews.android.library)
-    alias(libs.plugins.composenews.android.hilt)
 }
 
 android {
@@ -12,6 +11,7 @@ android {
 
 dependencies {
     libs.apply {
+        api(bundles.koin)
         api(bundles.kotest)
         api(coroutines.test)
         api(mockk)

core/test/src/main/java/ir/composenews/core_test/di/DispatcherModule.kt

@@ -2,16 +2,13 @@
 
 package ir.composenews.core_test.di
 
-import dagger.Binds
-import dagger.Module
-import dagger.hilt.InstallIn
-import dagger.hilt.components.SingletonComponent
 import ir.composenews.core_test.dispatcher.DispatcherProvider
 import ir.composenews.core_test.dispatcher.PlatformDispatcherProvider
+import org.koin.core.module.dsl.singleOf
+import org.koin.dsl.bind
+import org.koin.dsl.module
 
-@Module
-@InstallIn(SingletonComponent::class)
-interface DispatcherModule {
-    @Binds
-    fun bindDispatcherProvider(impl: PlatformDispatcherProvider): DispatcherProvider
-}
+val dispatcherModule =
+    module {
+        singleOf(::PlatformDispatcherProvider) bind DispatcherProvider::class
+    }

core/test/src/main/java/ir/composenews/core_test/dispatcher/PlatformDispatcherProvider.kt

@@ -9,9 +9,8 @@ package ir.composenews.core_test.dispatcher
 
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.Dispatchers
-import javax.inject.Inject
 
-class PlatformDispatcherProvider @Inject constructor() : DispatcherProvider {
+class PlatformDispatcherProvider : DispatcherProvider {
     override val ui: CoroutineDispatcher = Dispatchers.Main
     override val io: CoroutineDispatcher = Dispatchers.IO
     override val bg: CoroutineDispatcher = Dispatchers.Default