Fix light account/backup notification banner display behavior

Author: Jon-edge

CHANGELOG.md

@@ -11,6 +11,7 @@
 - changed: ramps: Infinite buy support according to new API
 - changed: Optimize login performance.
 - changed: Update Monero LWS server name to "Edge LWS"
+- fixed: Light account/backup reminder notification banner sometimes missing on login
 - fixed: ramps: Various Infinite UI/UX issues
 
 ## 4.41.1 (2025-12-29)

src/actions/LocalSettingsActions.ts

@@ -25,6 +25,16 @@ watchAccountSettings(s => {
 })
 
 let readSettingsFromDisk = false
+
+/**
+ * Resets the local account settings cache. Must be called on logout to prevent
+ * one account's settings from persisting to a subsequent account's session.
+ */
+export const resetLocalAccountSettingsCache = (): void => {
+  readSettingsFromDisk = false
+  localAccountSettings = asLocalAccountSettings({})
+}
+
 export const getLocalAccountSettings = async (
   account: EdgeAccount
 ): Promise<LocalAccountSettings> => {
@@ -214,7 +224,12 @@ const writeAccountNotifState = async (
   const localSettings = await getLocalAccountSettings(account)
   return await writeLocalAccountSettings(account, {
     ...localSettings,
-    notifState
+    // Merge with existing notifState to prevent concurrent writes from
+    // overwriting each other's keys
+    notifState: {
+      ...localSettings.notifState,
+      ...notifState
+    }
   })
 }
 
@@ -261,6 +276,13 @@ export const writeTokenWarningsShown = async (
 export const readLocalAccountSettings = async (
   account: EdgeAccount
 ): Promise<LocalAccountSettings> => {
+  // If we've already read from disk, return the cached settings.
+  // This prevents stale disk reads from overwriting newer in-memory writes
+  // that may not have been persisted to disk yet.
+  if (readSettingsFromDisk) {
+    return localAccountSettings
+  }
+
   try {
     const text = await account.localDisklet.getText(LOCAL_SETTINGS_FILENAME)
     const json = JSON.parse(text)
@@ -273,6 +295,7 @@ export const readLocalAccountSettings = async (
     // Defaults can be derived from cleaners. Only write when values change.
     const defaults = asLocalAccountSettings({})
     emitAccountSettings(defaults)
+    readSettingsFromDisk = true
     return defaults
   }
 }

src/actions/LoginActions.tsx

@@ -39,7 +39,10 @@ import {
   getDeviceSettings,
   writeIsSurveyDiscoverShown
 } from './DeviceSettingsActions'
-import { readLocalAccountSettings } from './LocalSettingsActions'
+import {
+  readLocalAccountSettings,
+  resetLocalAccountSettingsCache
+} from './LocalSettingsActions'
 import {
   registerNotificationsV2,
   updateNotificationSettings
@@ -333,6 +336,7 @@ export function logoutRequest(
     const { account } = state.core
     Keyboard.dismiss()
     Airship.clear()
+    resetLocalAccountSettingsCache()
 
     dispatch({ type: 'LOGOUT' })
     if (typeof account.logout === 'function') await account.logout()

src/components/services/NotificationService.ts

@@ -107,6 +107,7 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
 
   const wallets = useWatch(account, 'currencyWallets')
   const otpKey = useWatch(account, 'otpKey')
+  const username = useWatch(account, 'username')
 
   const detectedTokensRedux = useSelector(
     state => state.core.enabledDetectedTokens
@@ -124,7 +125,7 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
   // we only need referral-based promoId targeting.
   const accountFunded = useIsAccountFunded()
 
-  const isLightAccountReminder = account.id != null && account.username == null
+  const isLightAccountReminder = account.id != null && username == null
 
   const isOtpReminder =
     otpKey == null &&
@@ -146,6 +147,8 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
   // Update notification info with
   // 1. Date last received if transitioning from incomplete to complete
   // 2. Reset `isBannerHidden` if it's a new notification
+  // All notification writes are in a single effect with sequential awaits to
+  // prevent race conditions between concurrent writes.
   useAsyncEffect(
     async () => {
       // New token(s) detected
@@ -165,11 +168,6 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
       }
 
       await updateNotificationInfo(account, 'ip2FaReminder', isIp2faReminder)
-      await updateNotificationInfo(
-        account,
-        'lightAccountReminder',
-        isLightAccountReminder
-      )
       await updateNotificationInfo(
         account,
         'otpReminder',
@@ -198,6 +196,33 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
           )
         }
       }
+
+      // Handle lightAccountReminder separately using writeAccountNotifInfo
+      // directly (not updateNotificationInfo) to ensure isBannerHidden is
+      // always reset to false on login. This is placed last to avoid race
+      // conditions with other notification writes above.
+      if (isLightAccountReminder) {
+        // Light account: always show the backup reminder banner on login
+        await writeAccountNotifInfo(account, 'lightAccountReminder', {
+          isPriority: true,
+          isBannerHidden: false,
+          isCompleted: false
+        })
+      } else {
+        // Non-light account: mark complete if notification exists and isn't
+        // already complete (handles upgrade from light account)
+        const { notifState: currentNotifState } = await getLocalAccountSettings(
+          account
+        )
+        const existing = currentNotifState.lightAccountReminder
+        if (existing != null && !existing.isCompleted) {
+          await writeAccountNotifInfo(account, 'lightAccountReminder', {
+            isPriority: true,
+            isBannerHidden: true,
+            isCompleted: true
+          })
+        }
+      }
     },
     [
       isIp2faReminder,
@@ -206,7 +231,6 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
       isPwReminder,
       wallets,
       detectedTokensRedux,
-      notifState,
       accountReferral,
       accountReferralLoaded,
       countryCode,
@@ -215,21 +239,5 @@ export const NotificationService: React.FC<Props> = (props: Props) => {
     'NotificationServices'
   )
 
-  // Make sure the backup banner is always shown on login if needed. We do this
-  // separately in this effect so that we can hide the banner during current
-  // login session if they so choose.
-  useAsyncEffect(
-    async () => {
-      if (!isLightAccountReminder) return
-
-      await writeAccountNotifInfo(account, 'lightAccountReminder', {
-        isBannerHidden: false,
-        isCompleted: false
-      })
-    },
-    [isLightAccountReminder],
-    'NotificationServices'
-  )
-
   return null
 }