mirror of
https://github.com/vitorpamplona/amethyst.git
synced 2026-08-09 16:14:40 +00:00
nrobi144
b91519157e
Follow-up to #3186, addressing the unresolved review feedback: - RelayHealthStore.schedulePersist() wrapped the blocking save() in withContext(ioDispatcher) so prefs.flush() no longer sits on the Compose composition thread on Desktop. - close() now fires the final save on a detached IO-bound scope instead of blocking the composition thread for ~50ms during account switch / app exit. - @Volatile on persistJob/tickJob and a closed-flag guard so the relay-network thread and composition thread no longer race on plain vars (and post-close work is dropped). - desktopApp/Main.kt passes Dispatchers.IO to RelayHealthStore so persistence flushes land on the IO dispatcher instead of Dispatchers.Default. Plus a separate-but-related fix to the offline-banner-stuck-after-Mac-sleep issue: NostrClient.keepAliveJob now tracks wall-clock overshoot of its scheduled tick. If the OS suspended us (laptop lid closed, system sleep), delay() returns far past its deadline and the OkHttp websockets we held are dead even though BasicRelayClient.isConnected() still reads true until the next ping fails. On a >5x interval overshoot, force relayPool.disconnect() + connect() instead of trusting needsToReconnect(), so feeds resume without an app restart.