chore: bump assets controller to v7.2.1#30067
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. |
f13172f to
6a6f03d
Compare
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 404957e. Configure here.
🔍 Smart E2E Test Selection
click to see 🤖 AI reasoning detailsE2E Test Selection: Performance Test Selection: |
|




Description
Bumps
@metamask/assets-controllersfrom v7.0.0 to v7.2.1 to pick up the latest bug fixes and improvements from the upstream package.Changelog
CHANGELOG entry: bump assets controller to v7.2.1
Related issues
Fixes: N/A
Manual testing steps
Screenshots/Recordings
N/A — dependency bump with no UI changes.
Before
N/A
After
N/A
Pre-merge author checklist
Performance checks (if applicable)
Pre-merge reviewer checklist
Note
Medium Risk
Dependency bump to a new major version of
@metamask/assets-controllerand updates to the AssetsController messenger event surface could affect asset updates on network add/remove flows.Overview
Updates the app to use
@metamask/assets-controller^7.1.2(from^6.2.1), with correspondingyarn.lockchanges.Aligns the AssetsController messenger typing and delegation to include the new
NetworkController:networkAddedandNetworkController:networkRemovedevents, and updates the associated unit test expectations.Adjusts the test
initial-background-state.jsonasset identifiers to use checksummed token addresses in theAssetsController.assetsInfokeys.Reviewed by Cursor Bugbot for commit a6f2b98. Bugbot is set up for automated code reviews on this repo. Configure here.