chore: bump assets-controller - #45236
Conversation
|
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/metamask-assets (2 files, +63 -151)
📜 @MetaMask/policy-reviewers (8 files, +256 -96)
Tip Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers. 🧪 @MetaMask/qa (1 files, +19 -0)
|
LavaMoat grants — Range Four new resources, all first-party, all under The remainder of the diff is canonical-path relocations, where the generator renamed a resource because its shortest dependency path changed. No capability moved. Not covered by this run: whether each grant has a live call site at the upgraded versions. This reads the policy delta, not the packages' source, so it establishes that no capability arrived without a package to explain it — not that every capability is exercised. |
Bring in the AssetsController v13 messenger actions required by the Tron snap CoreAssetsAdapter path, while keeping the Accounts API core-backend patches.
Bring in the AssetsController v13 messenger actions required by the Tron snap CoreAssetsAdapter path, while keeping the Accounts API core-backend patches.
|
@metamaskbot update-policies |
|
Policies updated. Tip Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers. 👀 lavamoat/webpack/mv2/beta/policy.json changes differ from lavamoat/webpack/mv2/main/policy.json changes |
|
Builds ready [fbee8cb]
⚡ Performance Benchmarks (Total: 🟢 6 pass · 🟡 17 warn · 🔴 1 fail)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|


Description
Bump assets-controller and resolve multiple breaking changes.
Changelog
CHANGELOG entry: null
Related issues
Fixes:
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist
Note
Medium Risk
Touches balance/send flows and core AssetsController integration; messenger changes could affect real-time balances if miswired, but scope is mostly dependency alignment and test hardening.
Overview
Bumps
@metamask/assets-controllerto ^13.1.1 and@metamask/config-registry-controllerto ^2.0.1, with matching LavaMoat policy updates for nestedassets-controllerdependencies (e.g. scopedkeyring-snap-client/keyring-api).AssetsController messenger delegation is updated for the new package:
BackendWebSocketServiceactions andconnectionStateChangedare removed;ConfigRegistryController:getNetworkConfigByCaip2ChainIdandAccountActivityService:statusChangedare added. Messenger tests are consolidated withit.eachover the delegated action/event lists.Send E2E adds
checkAvailableBalanceon the send page and uses it before Max in the ERC-20 max-balance test so the amount screen does not race async balance updates.Reviewed by Cursor Bugbot for commit fbee8cb. Bugbot is set up for automated code reviews on this repo. Configure here.