fix(tokens): only update reserve balance while buy/sell is idle

Also proactively update USDF token accounts post finalization when buying with reserves or selling

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

Author: bmc08gt

apps/flipcash/shared/tokens/src/main/kotlin/com/flipcash/app/tokens/ui/BuySellSwapTokenViewModel.kt

@@ -37,6 +37,7 @@ import com.getcode.utils.trace
 import com.getcode.view.BaseViewModel2
 import com.getcode.view.LoadingSuccessState
 import dagger.hilt.android.lifecycle.HiltViewModel
+import kotlinx.coroutines.delay
 import kotlinx.coroutines.flow.combine
 import kotlinx.coroutines.flow.distinctUntilChanged
 import kotlinx.coroutines.flow.filter
@@ -367,6 +368,8 @@ class BuySellSwapTokenViewModel @Inject constructor(
                 usdf = balance,
                 nativeAmount = balance.convertingTo(rate),
             )
+        }.filter {
+            stateFlow.value.buyProgress.isIdle && stateFlow.value.sellProgress.isIdle
         }.onEach {
             dispatchEvent(Event.OnReservesUpdated(TokenWithBalance(Token.usdf, it.nativeAmount)))
         }.launchIn(viewModelScope)
@@ -540,9 +543,9 @@ class BuySellSwapTokenViewModel @Inject constructor(
                     of = token,
                 ).onSuccess { swapId ->
                     dispatchEvent(Event.OnPurchaseSubmitted(token, swapId))
+                    dispatchEvent(Event.UpdateBuyState(loading = false, success = true))
                     // buy submitted from reserves, drop reserves balance
                     tokenController.subtract(Token.usdf, amount)
-                    dispatchEvent(Event.UpdateBuyState(loading = false, success = true))
                 }.onFailure {
                     dispatchEvent(Event.UpdateBuyState(loading = false, success = false))
                     BottomBarManager.showError(
@@ -569,9 +572,9 @@ class BuySellSwapTokenViewModel @Inject constructor(
                     of = token,
                 ).onSuccess { swapId ->
                     dispatchEvent(Event.OnSellSubmitted(token, swapId))
+                    dispatchEvent(Event.UpdateSellState(loading = false, success = true))
                     // sell submitted, drop from balance
                     tokenController.subtract(token, amount)
-                    dispatchEvent(Event.UpdateSellState(loading = false, success = true))
                 }.onFailure {
                     dispatchEvent(Event.UpdateSellState(loading = false, success = false))
                     BottomBarManager.showError(
@@ -598,15 +601,19 @@ class BuySellSwapTokenViewModel @Inject constructor(
             }.onResult(
                 onSuccess = {
                     val token = stateFlow.value.tokenWithBalance!!.token
+                    val isUsingReserves = stateFlow.value.purpose is TokenSwapPurpose.Buy ||
+                            stateFlow.value.purpose is TokenSwapPurpose.Sell
                     viewModelScope.launch { tokenController.updateTokenAccount(token) }
+                    if (isUsingReserves) {
+                        viewModelScope.launch { tokenController.updateTokenAccount(Mint.usdf) }
+                    }
                     viewModelScope.launch {
                         // update activity feed to grab the tx as a result of this buy/sell
                         feedCoordinator.fetchSinceLatest()
                     }
                     dispatchEvent(Event.UpdateProcessingState(loading = false, success = true))
                 },
                 onError = {
-                    // TODO: show error
                     dispatchEvent(Event.UpdateProcessingState(loading = false, success = false, error = true))
                 }
             ).launchIn(viewModelScope)

services/opencode/src/main/kotlin/com/getcode/opencode/controllers/TokenController.kt

@@ -320,7 +320,7 @@ class TokenController @Inject constructor(
 
         trace(
             tag = TAG,
-            message = "Token metadata cache miss for ${mint.base58()}..., fetching from network",
+            message = "Token metadata cache miss for ${mint.base58()}, fetching from network",
             type = TraceType.Process
         )
 
@@ -338,7 +338,7 @@ class TokenController @Inject constructor(
             .onFailure { error ->
                 trace(
                     tag = TAG,
-                    message = "Failed to fetch token metadata for ${mint.base58()}...: ${error.message}",
+                    message = "Failed to fetch token metadata for ${mint.base58()}: ${error.message}",
                     type = TraceType.Error
                 )
             }
@@ -357,7 +357,7 @@ class TokenController @Inject constructor(
         val token = _state.value.tokens[mint]
         trace(
             tag = TAG,
-            message = "Token selected: ${token?.symbol ?: mint.base58()}...",
+            message = "Token selected: ${token?.symbol ?: mint.base58()}",
             type = TraceType.User
         )
         selectedToken.edit { it[mintPreferenceKey] = mint.base58() }
@@ -382,7 +382,7 @@ class TokenController @Inject constructor(
     ): Result<List<HistoricalMintData>> {
         trace(
             tag = TAG,
-            message = "Fetching historical market cap for ${mint.base58()}..., range=$windowedRange",
+            message = "Fetching historical market cap for ${mint.base58()}, range=$windowedRange",
             type = TraceType.Process
         )
 
@@ -451,7 +451,7 @@ class TokenController @Inject constructor(
         if (!fetchingMints.add(mint)) {
             trace(
                 tag = TAG,
-                message = "Skipping duplicate fetch for ${mint.base58()}...",
+                message = "Skipping duplicate fetch for ${mint.base58()}",
                 type = TraceType.Process
             )
             return
@@ -460,7 +460,7 @@ class TokenController @Inject constructor(
         try {
             trace(
                 tag = TAG,
-                message = "Fetching token account for ${mint.base58()}...",
+                message = "Fetching token account for ${mint.base58()}",
                 type = TraceType.Process
             )
 
@@ -473,7 +473,7 @@ class TokenController @Inject constructor(
             if (account == null) {
                 trace(
                     tag = TAG,
-                    message = "No primary account found for ${mint.base58()}...",
+                    message = "No primary account found for ${mint.base58()}",
                     type = TraceType.Error
                 )
                 return