From e39c51cc83c5c57ff4e1b1df2a96f2216e0bda75 Mon Sep 17 00:00:00 2001
From: Brandon McAnsh <git@bmcreations.dev>
Date: Fri, 26 Jun 2026 16:27:31 -0400
Subject: [PATCH] fix(auth): skip network retries during offline cold launch
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Check connectivity before retrying getUserFlags(), updateUserProfile(),
and savePrefs() in login(). When offline during a soft login (app
restart), flags resolve to null immediately and the existing local
fallback sets AuthState.Ready from cached DataStore — eliminating the
~6s spinner caused by 3 failed retries with 2s delays. Interactive
logins still always attempt the network fetch. Data refreshes on
reconnect via RealSessionController.

Signed-off-by: Brandon McAnsh <git@bmcreations.dev>
---
 .../com/flipcash/app/auth/AuthManager.kt       | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/apps/flipcash/shared/authentication/src/main/kotlin/com/flipcash/app/auth/AuthManager.kt b/apps/flipcash/shared/authentication/src/main/kotlin/com/flipcash/app/auth/AuthManager.kt
index 6464b85d2..a7ff230ab 100644
--- a/apps/flipcash/shared/authentication/src/main/kotlin/com/flipcash/app/auth/AuthManager.kt
+++ b/apps/flipcash/shared/authentication/src/main/kotlin/com/flipcash/app/auth/AuthManager.kt
@@ -22,6 +22,7 @@ import com.getcode.opencode.model.core.ID
 import com.getcode.utils.TraceManager
 import com.getcode.utils.TraceType
 import com.flipcash.libs.coroutines.DispatcherProvider
+import com.getcode.utils.network.NetworkConnectivityListener
 import com.getcode.utils.network.retryable
 import com.getcode.utils.trace
 import kotlinx.coroutines.CoroutineScope
@@ -49,6 +50,7 @@ class AuthManager @Inject constructor(
     private val appSettings: AppSettingsCoordinator,
     private val userFlags: UserFlagsCoordinator,
     private val contactCoordinator: ContactCoordinator,
+    private val networkObserver: NetworkConnectivityListener,
     private val dispatchers: DispatcherProvider,
 //    private val analytics: AnalyticsService,
 ) : CoroutineScope by CoroutineScope(dispatchers.IO) {
@@ -192,8 +194,12 @@ class AuthManager @Inject constructor(
 
                 coroutineScope {
                     launch {
-                        val flags = retryable(maxRetries = 3) {
-                            accountController.getUserFlags().getOrNull()
+                        val flags = if (!isSoftLogin || networkObserver.isConnected) {
+                            retryable(maxRetries = 3) {
+                                accountController.getUserFlags().getOrNull()
+                            }
+                        } else {
+                            null
                         }
 
                         val seenAccessKey = credentialManager.hasSeenAccessKey()
@@ -228,9 +234,13 @@ class AuthManager @Inject constructor(
                             }
                         }
 
-                        profileController.updateUserProfile()
+                        if (networkObserver.isConnected) {
+                            profileController.updateUserProfile()
+                        }
+                    }
+                    if (networkObserver.isConnected) {
+                        launch { savePrefs() }
                     }
-                    launch { savePrefs() }
                 }
             }.onFailure {
                 logout()