fix: change pre-installed snaps order#42814
Conversation
there is a long term issue with how we load in preinstalled snaps on extension initialisation
|
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
✨ Files requiring CODEOWNER review ✨🫰 @MetaMask/core-platform (1 files, +15 -15)
|
|
Builds ready [b96192a]
⚡ Performance Benchmarks (Total: 🟢 15 pass · 🟡 9 warn · 🔴 0 fail)
Bundle size diffs
|
NidhiKJha
left a comment
There was a problem hiding this comment.
Hey @Prithpal-Sooriya thanks for the fix, it resolves the issue on Tron but I am still not able to import on Solana. I think it's related?
https://github.com/user-attachments/assets/10e6a9d6-b8d8-4990-99f7-b157d4a1bb12
Thanks @NidhiKJha replied in our slack thread. Was another unrelated change due to a recent accounts-api network addition that caused a behaviour regression |
mcmire
left a comment
There was a problem hiding this comment.
Definitely strange, but seems like a benign change.



Description
This resolves a weird issue where the TRON snap is not loaded on startup causing issues with Asset Balances for TRON. This is first time install issue, closing and opening fixes. Convo here: https://consensys.slack.com/archives/CTQAGKY5V/p1779204701371199
But the underlying issue still needs debugging.
We have a longer term, improved, fix on our Assets Controllers side here: MetaMask/core#8862
EDIT
Doing some digging, the preinstalled snaps are installed correctly inside the
background.jswith no floating promises.However the
SnapController.init()- which processes these preinstalled snaps - is a floating promise.So while all other controllers are loading, the SnapController may not have finished its
init().Changelog
CHANGELOG entry: null
Related issues
Fixes: https://consensys.slack.com/archives/CTQAGKY5V/p1779204701371199
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist