fix(tokens): remove aggressive keying that was prevent needed recomps

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>

Author: bmc08gt

apps/flipcash/features/tokens/src/main/kotlin/com/flipcash/app/tokens/internal/TokenInfoScreen.kt

@@ -148,7 +148,7 @@ private fun TokenInfoScreen(
                     // market cap
                     state.marketCap?.let { mcap ->
                         val loadable = state.historicalMarketCapData[state.selectedPeriod] ?: Loadable.Loaded(emptyList())
-                        item(key = "mcap") {
+                        item {
                              MarketCapSection(
                                 modifier = Modifier
                                     .fillParentMaxWidth(),

apps/flipcash/features/tokens/src/main/kotlin/com/flipcash/app/tokens/internal/components/marketcap/MarketCapChart.kt

@@ -96,14 +96,9 @@ internal fun MarketCapChart(
         }
     }
 
-    var trend by remember { mutableStateOf<LineTrend>(LineTrend.Up) }
-
     // Update the model when the window changes
     LaunchedEffect(windowedData) {
         if (windowedData.isNotEmpty()) {
-            // Update trend BEFORE the model transaction
-            trend = trendType.determineTrend(windowedData.yValues)
-
             modelProducer.runTransaction {
                 lineSeries {
                     series(
@@ -115,6 +110,12 @@ internal fun MarketCapChart(
         }
     }
 
+    val trend by remember(windowedData) {
+        derivedStateOf {
+            trendType.determineTrend(windowedData.yValues)
+        }
+    }
+
     MarketCapChart(
         modelProducer = modelProducer,
         trend = trend,

apps/flipcash/shared/tokens/src/main/kotlin/TokenInfoViewModel.kt

@@ -49,6 +49,7 @@ import kotlinx.coroutines.flow.combine
 import kotlinx.coroutines.flow.debounce
 import kotlinx.coroutines.flow.distinctUntilChanged
 import kotlinx.coroutines.flow.drop
+import kotlinx.coroutines.flow.filter
 import kotlinx.coroutines.flow.filterIsInstance
 import kotlinx.coroutines.flow.flatMapLatest
 import kotlinx.coroutines.flow.flowOf
@@ -137,7 +138,6 @@ class TokenInfoViewModel @Inject constructor(
 
         eventFlow
             .filterIsInstance<Event.OnMintProvided>()
-            .distinctUntilChanged()
             .onEach {
                 tokenController.getTokenMetadata(it.mint)
                     .onSuccess { result ->
@@ -155,7 +155,7 @@ class TokenInfoViewModel @Inject constructor(
         eventFlow
             .filterIsInstance<Event.OnTokenChanged>()
             .distinctUntilChanged()
-            .flatMapLatest { (token, needsFunds) ->
+            .flatMapLatest { (token, _) ->
                 combine(
                     tokenController.balanceForToken(token.address),
                     tokenController.appreciationForToken(token.address),
@@ -179,21 +179,22 @@ class TokenInfoViewModel @Inject constructor(
                         null
                     }
 
-                    localizedBalance to localizedAppreciation to  needsFunds
+                    localizedBalance to localizedAppreciation
                 }
-            }.map { (balance, appreciation, needsFunds) ->
+            }.onEach { (balance, appreciation) ->
                 dispatchEvent(Event.OnBalanceUpdated(balance))
                 dispatchEvent(Event.OnAppreciationUpdated(appreciation))
-
-                needsFunds
-            }.take(1)
-            .onEach {
-                if (it) {
-                    dispatchEvent(Event.OpenPurchaseMethods(true))
-                }
             }
             .launchIn(viewModelScope)
 
+        eventFlow
+            .filterIsInstance<Event.OnTokenChanged>()
+            .distinctUntilChanged()
+            .filter { it.forNeededFunds }
+            .onEach {
+                 dispatchEvent(Event.OpenPurchaseMethods(true))
+            }.launchIn(viewModelScope)
+
         combine(
             tokenController.observeReservesBalance(),
             exchange.observeBalanceRate(),
@@ -202,7 +203,7 @@ class TokenInfoViewModel @Inject constructor(
                 usdf = balance,
                 nativeAmount = balance.convertingTo(rate),
             )
-        }.distinctUntilChanged().onEach {
+        }.onEach {
             dispatchEvent(Event.OnReservesUpdated(it))
         }.launchIn(viewModelScope)
 
@@ -386,7 +387,6 @@ class TokenInfoViewModel @Inject constructor(
 
     companion object {
         val updateStateForEvent: (Event) -> ((State) -> State) = { event ->
-            println("TOKEN INFO EVENT: ${event.javaClass.simpleName}")
             when (event) {
                 is Event.CashReservesEnabled -> { state -> state.copy(cashReservesEnabled = event.enabled) }
                 is Event.MarketCapChartEnabled -> { state -> state.copy(marketCapChartEnabled = event.enabled) }