Skip to content

fix: mf-6829 reduce savings rpc burst and use reserve symbols - #12496

Merged
guanbinrui merged 1 commit into
developfrom
fix/mf-6829-savings-rpc-reduction
Sep 14, 2026
Merged

guanbinrui merged 1 commit into
developfrom
fix/mf-6829-savings-rpc-reduction

Conversation

@swkatmask

Copy link
Copy Markdown
Contributor

Summary

  • MF-6829 follow-up: the Savings dialog re-fetched name/symbol/decimals for every Aave reserve and aToken via useFungibleTokens (~5 eth_calls × ~62 addresses ≈ 310 calls in one burst). On installs pinned to a rate-limited public RPC every read failed and symbols degraded to the hardcoded Unknown Token / UNKNOWN fallbacks (cached 10 min) — which QA saw as "unknown token symbol".
  • The reserves query now keeps the symbols already returned by the Aave ProtocolDataProvider (getAllReservesTokens / getAllATokens) and the pairs query only reads what is missing: name + nameBytes32 + decimals for the bare token, nothing extra for the aToken (it shares the reserve decimals — Aave invariant). ~310 → ~93 eth_calls (~70% cut) and symbols can no longer degrade to UNKNOWN.
  • Bare/aToken pairing is now explicit per reserve instead of positional flatten + chunk(2), so one unmatched aToken no longer shifts every subsequent pair.

@vercel

vercel Bot commented Sep 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
book Skipped Skipped Sep 14, 2026 2:57am UTC

Request Review

@guanbinrui
guanbinrui merged commit 5e1c880 into develop Sep 14, 2026
11 checks passed
@guanbinrui
guanbinrui deleted the fix/mf-6829-savings-rpc-reduction branch September 14, 2026 08:12

This branch was previously deployed

1 inactive deployment
Preview — 83b2ae4c Deployed Sep 14, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants