Skip to content

chore: bump assets controller to v7.2.1#30067

Merged
salimtb merged 8 commits into
mainfrom
chore/bump-assets-controller-v-7-0-0
May 15, 2026
Merged

chore: bump assets controller to v7.2.1#30067
salimtb merged 8 commits into
mainfrom
chore/bump-assets-controller-v-7-0-0

Conversation

@salimtb

@salimtb salimtb commented May 12, 2026

Copy link
Copy Markdown
Contributor

Description

Bumps @metamask/assets-controllers from 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

Feature: Assets display

  Scenario: user views their token balances
    Given the user has tokens on one or more networks

    When user opens the wallet
    Then token balances and asset info are displayed correctly

Screenshots/Recordings

N/A — dependency bump with no UI changes.

Before

N/A

After

N/A

Pre-merge author checklist

Performance checks (if applicable)

  • I've tested on Android
  • I've tested with a power user scenario
  • I've instrumented key operations with Sentry traces for production performance metrics

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

Note

Medium Risk
Dependency bump to a new major version of @metamask/assets-controller and 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 corresponding yarn.lock changes.

Aligns the AssetsController messenger typing and delegation to include the new NetworkController:networkAdded and NetworkController:networkRemoved events, and updates the associated unit test expectations.

Adjusts the test initial-background-state.json asset identifiers to use checksummed token addresses in the AssetsController.assetsInfo keys.

Reviewed by Cursor Bugbot for commit a6f2b98. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions

Copy link
Copy Markdown
Contributor

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.

@salimtb salimtb force-pushed the chore/bump-assets-controller-v-7-0-0 branch from f13172f to 6a6f03d Compare May 12, 2026 22:31
@socket-security

socket-security Bot commented May 12, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​metamask/​keyring-sdk@​2.0.2 ⏵ 2.1.175 +110085 +195 +1100

View full report

@salimtb salimtb changed the title chore: bump assets controller to v7.0.0 chore: bump assets controller to v7.2.1 May 13, 2026
@github-actions github-actions Bot added size-S and removed size-XS labels May 15, 2026
@salimtb salimtb marked this pull request as ready for review May 15, 2026 09:54
@salimtb salimtb requested a review from a team as a code owner May 15, 2026 09:54

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ 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.

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: SmokeAccounts, SmokeConfirmations, SmokeIdentity, SmokeNetworkAbstractions, SmokeNetworkExpansion, SmokeSwap, SmokeStake, SmokeWalletPlatform, SmokeMoney, SmokePerps, SmokeMultiChainAPI, SmokePredictions, SmokeSeedlessOnboarding, SmokeBrowser, SmokeSnaps
  • Selected Performance tags: @PerformanceAccountList, @PerformanceOnboarding, @PerformanceLogin, @PerformanceSwaps, @PerformanceLaunch, @PerformanceAssetLoading, @PerformancePredict, @PerformancePreps
  • Risk Level: high
  • AI Confidence: 100%
click to see 🤖 AI reasoning details

E2E Test Selection:
Hard rule (controller-version-update): @MetaMask controller package version updated in package.json: @metamask/assets-controller. Running all tests.

Performance Test Selection:
Hard rule (controller-version-update): @MetaMask controller package version updated in package.json: @metamask/assets-controller. Running all tests.

View GitHub Actions results

@sonarqubecloud

Copy link
Copy Markdown

@Cal-L Cal-L left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@salimtb salimtb added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit 1a13e37 May 15, 2026
203 of 209 checks passed
@salimtb salimtb deleted the chore/bump-assets-controller-v-7-0-0 branch May 15, 2026 16:38
@github-actions github-actions Bot locked and limited conversation to collaborators May 15, 2026
@metamaskbotv2 metamaskbotv2 Bot added the release-7.79.0 Issue or pull request that will be included in release 7.79.0 label May 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.79.0 Issue or pull request that will be included in release 7.79.0 size-S team-assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants