fix: use cached account for fetch accounts

[jaymengxy]

No description provided.

[github-actions]

PR Summary

Modified the wallet account fetching logic to preserve cached accounts when network requests return empty results, preventing account loss due to potential indexer latency issues. Added comprehensive logging to track cache loading and network fetch results across different blockchain networks.

Changes

File	Summary
Android/wallet/src/main/java/com/flow/wallet/wallet/BaseWallet.kt	Modified fetchAllNetworkAccounts() to preserve existing cached accounts instead of starting fresh. Added logic to keep cached accounts when network returns empty results, preventing data loss from indexer latency. Enhanced logging to track cache loading and network fetch results.
Android/wallet/src/main/java/com/flow/wallet/wallet/KeyWallet.kt	Enhanced debug logging in fetchAccountsForNetwork() to include network information when fetching accounts for both P256 and SECP256k1 public keys, improving debugging capabilities for network-specific operations.

---

autogenerated by presubmit.ai

[github-actions]

🚨 Pull request needs attention.

Review Summary

Commits Considered (1)

• df862a2: fix: use cached account for fetch accounts

Files Processed (2)

• Android/wallet/src/main/java/com/flow/wallet/wallet/BaseWallet.kt (3 hunks)
• Android/wallet/src/main/java/com/flow/wallet/wallet/KeyWallet.kt (2 hunks)

Actionable Comments (2)

• Android/wallet/src/main/java/com/flow/wallet/wallet/BaseWallet.kt [196-197]

  possible bug: "Cached accounts may not be preserved in new maps initialization."

• Android/wallet/src/main/java/com/flow/wallet/wallet/BaseWallet.kt [211-216]

  possible bug: "Cache preservation logic is incomplete."

Skipped Comments (0)

[github-actions]

The initialization of newAccounts and newFlowAccounts creates mutable copies of the current state, but when network fetches return empty results, these copies won't contain the cached data. Consider initializing these maps with the existing cached data to ensure cached accounts are preserved.

[github-actions]

The logic logs a warning about keeping cache but doesn't actually preserve the cached accounts in the newFlowAccounts and newAccounts maps. The cached data should be explicitly added to these maps to prevent account loss.