Reduce UI lag: YAOB throttle, debounce, dedup, and balance cache

docs/wallet-cache.md

@@ -0,0 +1,73 @@
+# Wallet Cache Architecture
+
+Wallet caching provides instant UI on login by saving wallet state to an unencrypted JSON file and restoring it before currency engines load.
+
+## Overview
+
+On login, the account pixie checks for a cache file at `accountCache/<storageWalletId>/walletCache.json`. If found, it creates lightweight cached wallet objects that the GUI can display immediately. Real wallets load in the background and replace/supplement the cached ones.
+
+The cache file contains:
+
+- Token definitions (only tokens enabled by at least one wallet)
+- Wallet state: id, type, name, pluginId, fiatCurrencyCode, balances, enabledTokenIds, otherMethodNames, created date, publicWalletInfo
+- Config otherMethods names per plugin
+
+## Cached Wallet Delegation
+
+Cached wallets implement the full `EdgeCurrencyWallet` interface. Property getters return cached values as defaults, delegating to the real wallet when available via `tryGetRealWallet()`. Async methods delegate via a shared `delegate()` helper that checks for the real wallet synchronously first, then waits via a shared polling promise.
+
+Key design constraint: the cached wallet runs inside the WebView (edge-core-js), while the GUI reads properties through the yaob bridge. yaob caches getter values on the client side and only refreshes them when `update(object)` is called. Since no pixie calls `update()` on cached wallets, **any setter that changes a value the GUI reads back must call `update(wallet)` after mutation** to propagate through yaob. Four setters require this:
+
+- `changePaused` / `paused`
+- `renameWallet` / `name`
+- `setFiatCurrencyCode` / `fiatCurrencyCode`
+- `changeEnabledTokenIds` / `enabledTokenIds`
+
+Each setter: (1) awaits the delegate to the real wallet, (2) updates a local variable, (3) calls `update(wallet)`. If the delegate throws, no local state changes.
+
+## Shared Polling (`makeRealObjectPoller`)
+
+Both cached wallets and cached configs use `makeRealObjectPoller<T>` from `cache-utils.ts`. This creates a single shared promise per object -- all callers that need the real wallet share the same 500ms poll loop. This avoids N concurrent polling loops when N methods are called simultaneously. The poller times out after 60 seconds.
+
+## otherMethods Delegation
+
+Plugin `otherMethods` are cached by name in the cache file. `createDelegatingOtherMethods` creates stub functions for each cached name. When called, each stub checks if the real otherMethods are available synchronously, otherwise waits for the real wallet/config. Method names not in the cache return `undefined` until the real object loads. Wallet otherMethods are bridgified for yaob serialization.
+
+## Disklet Delegation
+
+Cached wallets expose delegating disklets that forward all operations (`getText`, `setText`, `getData`, `setData`, `list`, `delete`) to the real wallet's disklet. During the cache phase, operations wait for the real wallet. The GUI does not access wallet-level disklets during the cache window -- account-level disklets (for settings, referrals) come from the account's own storage wallet, not currency wallets.
+
+## Cache Saving
+
+`makeWalletCacheSaver` implements a dirty-triggered throttle. The account pixie's `cacheSaver` sub-pixie detects wallet state changes reactively in its `update()` method (triggered by Redux state changes) and calls `markDirty()`. The saver responds immediately or schedules a delayed save:
+
+- When `markDirty()` is called and >= throttleMs has elapsed since the last save, the save happens immediately.
+- When `markDirty()` is called within the throttle window, the save is scheduled for when the window expires.
+- Only one pending save is scheduled at a time; additional `markDirty()` calls during the window are coalesced.
+- If changes arrive during an active save, another save is scheduled after completion.
+
+Other features:
+
+- Max 3 consecutive failures before giving up (prevents infinite log spam)
+- Uses `account.loggedIn` to guard against writing after logout
+- Only caches tokens enabled by at least one wallet (avoids caching thousands of Ethereum tokens)
+- `walletCacheSaverConfig.throttleMs` can be overridden to 50ms in tests
+
+## Cache Loading
+
+`loadWalletCache` parses the JSON, validates through cleaners (`asWalletCacheFile`), creates one `EdgeCurrencyConfig` per plugin and one `EdgeCurrencyWallet` per cached wallet. Each gets a real-object lookup callback that reads from the pixie output. The loader also accepts `pauseWallets` from the login options so cached wallets match the real wallet's initial paused state.
+
+Cache loading happens before `loadAllFiles` / `ACCOUNT_KEYS_LOADED`. If the cache file doesn't exist or fails validation (expected on first login or after schema changes), login falls through to the normal flow.
+
+## paused State and WalletLifecycle
+
+The GUI's `WalletLifecycle` boots wallets in batches by checking `wallet.paused`. Cached wallets start with `paused = pauseWallets` (true when the GUI passes `pauseWallets: true`). When WalletLifecycle calls `changePaused(false)`, the cached wallet delegates to the real wallet and calls `update(wallet)` to propagate the change through yaob. Without the `update()` call, yaob's client-side proxy would cache the old `paused = true` indefinitely, causing WalletLifecycle to re-boot the same wallets in an infinite loop.
+
+## Testing
+
+Tests use two mechanisms for deterministic control:
+
+- **Engine gate**: `createEngineGate()` returns `{ gate, release }`. Setting `fakePluginTestConfig.engineGate = gate` blocks engine creation. Call `release()` to allow engines to load. This replaces timing-based delays with explicit control.
+- **Cache saver throttle**: `walletCacheSaverConfig.throttleMs = 50` reduces the save interval from 5 seconds to 50ms in tests. Cache save waits use `await snooze(100)` (2x the throttle).
+
+The fake currency plugin supports `fakePluginTestConfig.noOtherMethods = true` to test the empty-otherMethods code path.

src/core/account/account-api.ts

@@ -30,6 +30,7 @@ import {
 } from '../../types/types'
 import { makeEdgeResult } from '../../util/edgeResult'
 import { base58 } from '../../util/encoding'
+import { WalletCacheSetup } from '../cache/cache-wallet-loader'
 import { getPublicWalletInfo } from '../currency/wallet/currency-wallet-pixie'
 import {
   finishWalletCreation,
@@ -72,10 +73,21 @@ import { makeLobbyApi } from './lobby-api'
 import { makeMemoryWalletInner } from './memory-wallet'
 import { CurrencyConfig, SwapConfig } from './plugin-api'
 
+export interface AccountApiOptions {
+  /** Optional cached wallet data for instant UI on login */
+  cacheSetup?: WalletCacheSetup
+}
+
 /**
  * Creates an unwrapped account API object around an account state object.
+ * If cacheSetup is provided, cached wallets are used until real wallets load.
  */
-export function makeAccountApi(ai: ApiInput, accountId: string): EdgeAccount {
+export function makeAccountApi(
+  ai: ApiInput,
+  accountId: string,
+  opts: AccountApiOptions = {}
+): EdgeAccount {
+  const { cacheSetup } = opts
   // We don't want accountState to be undefined when we log out,
   // so preserve a snapshot of our last state:
   let lastState = ai.props.state.accounts[accountId]
@@ -613,23 +625,57 @@ export function makeAccountApi(ai: ApiInput, accountId: string): EdgeAccount {
     // ----------------------------------------------------------------
 
     get activeWalletIds(): string[] {
-      return ai.props.state.accounts[accountId].activeWalletIds
+      const accountState = ai.props.state.accounts[accountId]
+      // Use cached IDs until keys are loaded:
+      if (
+        accountState != null &&
+        !accountState.keysLoaded &&
+        cacheSetup != null
+      ) {
+        return cacheSetup.activeWalletIds
+      }
+      return accountState?.activeWalletIds ?? []
     },
 
     get archivedWalletIds(): string[] {
-      return ai.props.state.accounts[accountId].archivedWalletIds
+      return ai.props.state.accounts[accountId]?.archivedWalletIds ?? []
     },
 
     get hiddenWalletIds(): string[] {
-      return ai.props.state.accounts[accountId].hiddenWalletIds
+      return ai.props.state.accounts[accountId]?.hiddenWalletIds ?? []
     },
 
     get currencyWallets(): { [walletId: string]: EdgeCurrencyWallet } {
-      return ai.props.output.accounts[accountId].currencyWallets
+      // Get real wallets from pixie output
+      const pixieWallets =
+        ai.props.output.accounts[accountId]?.currencyWallets ?? {}
+
+      // If no cache, just return pixie wallets (stable reference)
+      if (cacheSetup == null) {
+        return pixieWallets
+      }
+
+      // Once all active wallets have loaded, return the stable pixie
+      // reference to avoid allocating a new object on every access:
+      const activeIds = this.activeWalletIds
+      if (activeIds.every(id => pixieWallets[id] != null)) {
+        return pixieWallets
+      }
+
+      // Merge: real wallets take priority, cached fill gaps
+      const result: { [walletId: string]: EdgeCurrencyWallet } = {}
+      for (const walletId of activeIds) {
+        const wallet =
+          pixieWallets[walletId] ?? cacheSetup.currencyWallets[walletId]
+        if (wallet != null) {
+          result[walletId] = wallet
+        }
+      }
+      return result
     },
 
     get currencyWalletErrors(): { [walletId: string]: Error } {
-      return ai.props.state.accounts[accountId].currencyWalletErrors
+      return ai.props.state.accounts[accountId]?.currencyWalletErrors ?? {}
     },
 
     async createCurrencyWallet(
@@ -804,7 +850,7 @@ export function makeAccountApi(ai: ApiInput, accountId: string): EdgeAccount {
             : undefined,
 
         // Added for backward compatibility for plugins using core 1.x
-        // @ts-expect-error
+        // @ts-expect-error - paymentTokenId/paymentWallet are deprecated but still used by old plugins
         paymentTokenId: paymentInfo?.tokenId,
         paymentWallet: wallet
       })
@@ -832,7 +878,9 @@ export function makeAccountApi(ai: ApiInput, accountId: string): EdgeAccount {
 
       // Close unused quotes:
       for (const otherQuote of otherQuotes) {
-        otherQuote.close().catch(() => undefined)
+        otherQuote.close().catch((error: unknown) => {
+          ai.props.log.warn('Failed to close unused swap quote:', error)
+        })
       }
 
       // Return the front quote:

src/core/account/account-cleaners.ts

@@ -22,7 +22,7 @@ const asEdgeDenomination = asObject<EdgeDenomination>({
   symbol: asOptional(asString)
 })
 
-const asEdgeToken = asObject<EdgeToken>({
+export const asEdgeToken = asObject<EdgeToken>({
   currencyCode: asString,
   denominations: asArray(asEdgeDenomination),
   displayName: asString,