feat(add-new-user)

Author: hoc081098

app/src/main/java/com/hoc/flowmvi/core/DefaultCoroutineDispatchers.kt

@@ -8,6 +8,5 @@ import javax.inject.Inject
 internal class DefaultCoroutineDispatchers @Inject constructor() : AppCoroutineDispatchers {
   override val main: CoroutineDispatcher get() = Dispatchers.Main
   override val io: CoroutineDispatcher get() = Dispatchers.IO
-  override val mainImmediate: CoroutineDispatcher get() = Dispatchers.Main.immediate
   override val default: CoroutineDispatcher get() = Dispatchers.Default
 }

buildSrc/src/main/kotlin/deps.kt

@@ -130,6 +130,7 @@ inline val PDsS.androidLib: PDS get() = id("com.android.library")
 inline val PDsS.kotlinAndroid: PDS get() = kotlin("android")
 inline val PDsS.kotlin: PDS get() = kotlin("jvm")
 inline val PDsS.kotlinKapt: PDS get() = kotlin("kapt")
+inline val PDsS.kotlinParcelize: PDS get() = id("kotlin-parcelize")
 inline val PDsS.daggerHiltAndroid: PDS get() = id("dagger.hilt.android.plugin")
 
 inline val DependencyHandler.domain get() = project(":domain")
@@ -144,7 +145,9 @@ inline val DependencyHandler.mviBase get() = project(":mvi-base")
 inline val DependencyHandler.mviTesting get() = project(":mvi-testing")
 inline val DependencyHandler.testUtils get() = project(":test-utils")
 
-fun DependencyHandler.implementationCompose() {
+fun DependencyHandler.implementationCompose(
+  includeMaterial2: Boolean = false,
+) {
   arrayOf(
     platform(deps.compose.bom),
     // activity compose
@@ -160,7 +163,10 @@ fun DependencyHandler.implementationCompose() {
     deps.compose.layout,
     deps.compose.foundation,
     deps.compose.ui,
-    deps.compose.material,
+    *(
+      if (includeMaterial2) arrayOf(deps.compose.material)
+      else emptyArray()
+      ),
     deps.compose.material3,
     deps.compose.materialIconsExtended,
     deps.compose.runtime,

core-ui/build.gradle.kts

@@ -44,5 +44,8 @@ dependencies {
   implementation(deps.lifecycle.runtimeKtx)
   implementation(deps.androidx.material)
 
+  implementation(deps.coroutines.core)
+  implementation(deps.coroutines.android)
+
   addUnitTest()
 }

core-ui/src/main/java/com/hoc/flowmvi/core_ui/checkImmediateMainDispatcher.kt

@@ -0,0 +1,20 @@
+package com.hoc.flowmvi.core_ui
+
+import android.util.Log
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.currentCoroutineContext
+import kotlin.coroutines.ContinuationInterceptor
+
+suspend fun debugCheckImmediateMainDispatcher() {
+  if (BuildConfig.DEBUG) {
+    val interceptor = currentCoroutineContext()[ContinuationInterceptor]
+    Log.d(
+      "###",
+      "debugCheckImmediateMainDispatcher: $interceptor, ${Dispatchers.Main.immediate}, ${Dispatchers.Main}"
+    )
+
+    check(interceptor === Dispatchers.Main.immediate) {
+      "Expected ContinuationInterceptor to be Dispatchers.Main.immediate but was $interceptor"
+    }
+  }
+}

core-ui/src/main/java/com/hoc/flowmvi/core_ui/rememberFlowWithLifecycle.kt

@@ -8,7 +8,9 @@ import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.flowWithLifecycle
 import androidx.lifecycle.repeatOnLifecycle
 import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.flow.Flow
+import kotlinx.coroutines.withContext
 
 @Composable
 fun <T> rememberFlowWithLifecycle(
@@ -32,8 +34,10 @@ fun <T> Flow<T>.collectInLaunchedEffectWithLifecycle(
 ) {
   val flow = this
   LaunchedEffect(flow, lifecycle, minActiveState, *keys) {
-    lifecycle.repeatOnLifecycle(minActiveState) {
-      flow.collect { collector(it) }
+    withContext(Dispatchers.Main.immediate) {
+      lifecycle.repeatOnLifecycle(minActiveState) {
+        flow.collect { collector(it) }
+      }
     }
   }
 }

core/src/main/java/com/hoc/flowmvi/core/dispatchers/AppCoroutineDispatchers.kt

@@ -5,6 +5,5 @@ import kotlinx.coroutines.CoroutineDispatcher
 interface AppCoroutineDispatchers {
   val main: CoroutineDispatcher
   val io: CoroutineDispatcher
-  val mainImmediate: CoroutineDispatcher
   val default: CoroutineDispatcher
 }

data/src/test/java/com/hoc/flowmvi/data/UserRepositoryImplRealAPITest.kt

@@ -38,7 +38,6 @@ class UserRepositoryImplRealAPITest : KoinTest {
           object : AppCoroutineDispatchers {
             override val main: CoroutineDispatcher get() = Main
             override val io: CoroutineDispatcher get() = IO
-            override val mainImmediate: CoroutineDispatcher get() = Main.immediate
             override val default: CoroutineDispatcher get() = IO
           }
         }

feature-add/build.gradle.kts

@@ -2,6 +2,7 @@ plugins {
   androidLib
   kotlinAndroid
   kotlinKapt
+  kotlinParcelize
   daggerHiltAndroid
 }
 
@@ -62,7 +63,9 @@ dependencies {
 
   implementation(deps.androidx.material)
 
+  implementation(deps.arrow.core)
   implementation(deps.coroutines.core)
+  implementation(deps.flowExt)
 
   implementation(deps.daggerHilt.android)
   kapt(deps.daggerHilt.compiler)

feature-add/src/main/AndroidManifest.xml

@@ -1,13 +1,2 @@
 <?xml version="1.0" encoding="utf-8"?>
-<manifest xmlns:android="http://schemas.android.com/apk/res/android"
-  package="com.hoc.flowmvi.ui.add">
-
-  <application>
-
-    <activity
-      android:name=".AddActivity"
-      android:label="Add user" />
-
-  </application>
-
-</manifest>
+<manifest package="com.hoc.flowmvi.ui.add" />