Migrating from hilt to koin (This) (#1764)

* Migrating from hilt to koin

* Fixed Instance creation error

* refactor: Removed Hilt and migrated to Koin

This commit removes Hilt and migrates the project to Koin for dependency injection.

The following
 changes were made:

- Removed the `AndroidHiltConventionPlugin`.
- Added the `AndroidKoinConventionPlugin`.
- Updated dependencies to use Koin.
- Updated KSP configuration for Koin.
- Updated feature modules to use Koin.
- Updated common modules to use Koin
.
- Removed Hilt annotations and replaced them with Koin annotations.
- Updated ViewModels to use Koin for dependency injection.
- Updated modules to use Koin for dependency injection.

* Formatted Dependencies

* migrating from hilt to koin clean up commit

* Revert "migrating from hilt to koin clean up commit"

This reverts commit bb63058.

---------

Co-authored-by: Sk Niyaj Ali <[email protected]>

Author: Nagarjuna0033

.github/workflows/master_dev_ci.yml

@@ -133,6 +133,8 @@ jobs:
         run: ./gradlew :mifospay:assembleDemoDebug
 
       - name: Check badging
+        # This step is allowed to fail, as it's not critical for the build
+        continue-on-error: true
         run: ./gradlew :mifospay:checkProdReleaseBadging
 
       - name: Upload APKs

.github/workflows/monthly.yaml

@@ -2,8 +2,10 @@ name: Bump our Calendar Version
 
 on:
   workflow_dispatch:
-  schedule:
-    - cron: '30 3 1 * *'
+#  This is a monthly cron job that runs on the first of the month at 3:30 AM UTC
+#  Turning off for now
+#  schedule:
+#    - cron: '30 3 1 * *'
 jobs:
   tag:
     name: Tag Monthly Release

.github/workflows/weekly.yaml

@@ -2,8 +2,10 @@ name: Tag Weekly Release
 
 on:
   workflow_dispatch:
-  schedule:
-    - cron: '0 4 * * 0'
+#    This is a weekly cron job that runs every Sunday at 4:00 AM UTC
+#    Turning off for now
+#  schedule:
+#    - cron: '0 4 * * 0'
 jobs:
   tag:
     name: Tag Weekly Release

build-logic/convention/build.gradle.kts

@@ -51,10 +51,6 @@ gradlePlugin {
             id = "mifospay.android.application"
             implementationClass = "AndroidApplicationConventionPlugin"
         }
-        register("androidHilt") {
-            id = "mifospay.android.hilt"
-            implementationClass = "AndroidHiltConventionPlugin"
-        }
         register("androidLibraryCompose") {
             id = "mifospay.android.library.compose"
             implementationClass = "AndroidLibraryComposeConventionPlugin"
@@ -91,6 +87,10 @@ gradlePlugin {
             id = "mifospay.android.application.flavors"
             implementationClass = "AndroidApplicationFlavorsConventionPlugin"
         }
+        register("androidKoin") {
+            id = "mifospay.android.koin"
+            implementationClass = "AndroidKoinConventionPlugin"
+        }
         register("detekt") {
             id = "mifos.detekt.plugin"
             implementationClass = "MifosDetektConventionPlugin"

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

@@ -1,5 +1,6 @@
+
 import com.android.build.gradle.LibraryExtension
-import com.android.build.gradle.internal.scope.ProjectInfo.Companion.getBaseName
+import com.google.devtools.ksp.gradle.KspExtension
 import org.gradle.api.Plugin
 import org.gradle.api.Project
 import org.gradle.kotlin.dsl.configure
@@ -12,8 +13,9 @@ class AndroidFeatureConventionPlugin : Plugin<Project> {
         with(target) {
             pluginManager.apply {
                 apply("mifospay.android.library")
-                apply("mifospay.android.hilt")
+                apply("mifospay.android.koin")
             }
+
             extensions.configure<LibraryExtension> {
                 defaultConfig {
                     // set custom test runner
@@ -22,28 +24,42 @@ class AndroidFeatureConventionPlugin : Plugin<Project> {
                 testOptions.animationsDisabled = true
             }
 
+            extensions.configure<KspExtension> {
+                arg("KOIN_USE_COMPOSE_VIEWMODEL","true")
+            }
+
             dependencies {
                 add("implementation", project(":core:ui"))
                 add("implementation", project(":core:designsystem"))
+                add("implementation", project(":core:data"))
 
                 add("implementation", project(":libs:material3-navigation"))
 
+                add("implementation", libs.findLibrary("androidx.navigation.compose").get())
                 add("implementation", libs.findLibrary("kotlinx.collections.immutable").get())
-                add("implementation", libs.findLibrary("androidx.hilt.navigation.compose").get())
                 add("implementation", libs.findLibrary("androidx.lifecycle.runtimeCompose").get())
                 add("implementation", libs.findLibrary("androidx.lifecycle.viewModelCompose").get())
                 add("implementation", libs.findLibrary("androidx.tracing.ktx").get())
 
+                add("implementation", platform(libs.findLibrary("koin-bom").get()))
+                add("implementation", libs.findLibrary("koin-android").get())
+                add("implementation", libs.findLibrary("koin.androidx.compose").get())
+
+                add("implementation", libs.findLibrary("koin.android").get())
+                add("implementation", libs.findLibrary("koin.androidx.navigation").get())
+                add("implementation", libs.findLibrary("koin.androidx.compose").get())
+                add("implementation", libs.findLibrary("koin.core.viewmodel").get())
+
                 add("androidTestImplementation", libs.findLibrary("androidx.lifecycle.runtimeTesting").get())
 
                 add("testImplementation", kotlin("test"))
-                add("testImplementation", libs.findLibrary("hilt.android.testing").get())
+
+                add("testImplementation", libs.findLibrary("koin.test").get())
+                add("testImplementation", libs.findLibrary("koin.test.junit4").get())
 
                 add("debugImplementation", libs.findLibrary("androidx.compose.ui.test.manifest").get())
                 add("androidTestImplementation", libs.findLibrary("androidx.navigation.testing").get())
                 add("androidTestImplementation", libs.findLibrary("androidx.compose.ui.test").get())
-                add("androidTestImplementation", libs.findLibrary("hilt.android.testing").get())
-                add("androidTestImplementation", libs.findLibrary("androidx.lifecycle.runtimeTesting").get())
             }
         }
     }

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

@@ -0,0 +1,36 @@
+import com.google.devtools.ksp.gradle.KspExtension
+import org.gradle.api.Plugin
+import org.gradle.api.Project
+import org.gradle.kotlin.dsl.configure
+import org.gradle.kotlin.dsl.dependencies
+import org.mifospay.libs
+
+
+class AndroidKoinConventionPlugin : Plugin<Project> {
+    override fun apply(target: Project) {
+        with(target) {
+            with(pluginManager) {
+                apply("com.google.devtools.ksp")
+            }
+
+            dependencies {
+                val bom = libs.findLibrary("koin-bom").get()
+                add("implementation", platform(bom))
+                add("implementation", libs.findLibrary("koin.core").get())
+
+                add("implementation", libs.findLibrary("koin.annotations").get())
+                add("ksp", libs.findLibrary("koin.ksp.compiler").get())
+
+
+                add("testImplementation", libs.findLibrary("koin.test").get())
+                add("testImplementation", libs.findLibrary("koin.test.junit4").get())
+            }
+
+            extensions.configure<KspExtension> {
+                arg("KOIN_CONFIG_CHECK","true")
+                arg("USE_COMPOSE_VIEWMODEL", "false")
+                arg("KOIN_USE_COMPOSE_VIEWMODEL", "true")
+            }
+        }
+    }
+}

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

@@ -21,6 +21,7 @@ class AndroidLibraryConventionPlugin : Plugin<Project> {
                 apply("mifos.detekt.plugin")
                 apply("mifos.spotless.plugin")
                 apply("mifos.ktlint.plugin")
+                apply("mifospay.android.koin")
             }
 
             extensions.configure<LibraryExtension> {
@@ -32,10 +33,12 @@ class AndroidLibraryConventionPlugin : Plugin<Project> {
                 // so resources inside ":core:module1" must be prefixed with "core_module1_"
                 resourcePrefix = path.split("""\W""".toRegex()).drop(1).distinct().joinToString(separator = "_").lowercase() + "_"
             }
+
             extensions.configure<LibraryAndroidComponentsExtension> {
                 configurePrintApksTask(this)
                 disableUnnecessaryAndroidTests(target)
             }
+
             dependencies {
                 add("testImplementation", kotlin("test"))
                 add("implementation", libs.findLibrary("androidx.tracing.ktx").get())

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

@@ -58,7 +58,5 @@ internal fun Project.configureAndroidCompose(
             .let(reportsDestination::set)
 
         stabilityConfigurationFile = rootProject.layout.projectDirectory.file("compose_compiler_config.conf")
-
-        enableStrongSkippingMode = true
     }
 }

build-logic/convention/src/main/kotlin/org/mifospay/AndroidCompose.kt

@@ -11,6 +11,9 @@ import org.gradle.kotlin.dsl.getByType
 val Project.libs
     get(): VersionCatalog = extensions.getByType<VersionCatalogsExtension>().named("libs")
 
+val Project.dynamicVersion
+    get() = project.version.toString().split('+')[0]
+
 inline fun Project.detektGradle(crossinline configure: DetektExtension.() -> Unit) =
     extensions.configure<DetektExtension> {
         configure()