Skip to content

chore: bump assets controllers to v100.2.0#27228

Merged
salimtb merged 6 commits intomainfrom
chore/bump-assets-controllers-v100.3.0
Mar 10, 2026
Merged

chore: bump assets controllers to v100.2.0#27228
salimtb merged 6 commits intomainfrom
chore/bump-assets-controllers-v100.3.0

Conversation

@salimtb
Copy link
Contributor

@salimtb salimtb commented Mar 9, 2026

Description

Bumps @metamask/assets-controllers to v100.2.0 and wires up the two new features introduced in that release:

  1. AccountTrackerController — homepage sections v1 flag (core#8117): When isHomepageSectionsV1Enabled is true, the controller uses all popular EVM networks (via NetworkEnablementController:listPopularEvmNetworks) for balance refresh on account change and keyring unlock, instead of only the enabled networks. The AccountTrackerController messenger is extended to delegate the two new NetworkEnablementController actions it requires.

  2. selectBalanceForAllWallets — network configurations: Passes merged EVM + non-EVM (core#8141): networkConfigurationsByChainId (via selectNetworkConfigurations) as an additional argument to calculateBalanceForAllWallets, enabling the balance calculation to be aware of all configured networks.

Changelog

CHANGELOG entry: null

Related issues

Fixes: #23862

Manual testing steps

Feature: Balance display with aggregated homepage

  Scenario: user views wallet balance with homepage sections v1 enabled
    Given the user has the homepageSectionsV1 remote feature flag enabled
    And the user has accounts on multiple EVM networks

    When the user opens the app
    Then balances are refreshed across all popular EVM networks
    And the total balance is displayed correctly

  Scenario: user views wallet balance with homepage sections v1 disabled
    Given the user has the homepageSectionsV1 remote feature flag disabled

    When the user opens the app
    Then balances are refreshed only for enabled networks
    And the total balance is displayed correctly

Screenshots/Recordings

N/A — no UI changes.

Before

N/A

After

N/A

Pre-merge author checklist

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.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

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.

@socket-security
Copy link

socket-security bot commented Mar 9, 2026

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatednpm/​@​metamask/​assets-controllers@​100.1.0 ⏵ 100.2.072 +110091 +199 +1100
Updatednpm/​@​metamask/​multichain-account-service@​7.0.0 ⏵ 7.1.09810079 +198 +1100

View full report

@salimtb salimtb marked this pull request as ready for review March 10, 2026 12:58
@salimtb salimtb requested review from a team as code owners March 10, 2026 12:58
Copy link

@cursor cursor bot left a comment

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.

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

@salimtb salimtb enabled auto-merge March 10, 2026 13:44
@salimtb salimtb changed the title chore: bump assets controllers to v100.3.0 chore: bump assets controllers to v100.2.0 Mar 10, 2026
Copy link
Contributor

@NicolasMassart NicolasMassart left a comment

Choose a reason for hiding this comment

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

Before merge: check all template checkboxes in description, update issue number with N/A if none, and confirm/fix Cursor bot issues and test suggestions. After that, this looks good.

@github-actions
Copy link
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: SmokeAccounts, SmokeConfirmations, SmokeIdentity, SmokeNetworkAbstractions, SmokeNetworkExpansion, SmokeTrade, SmokeWalletPlatform, SmokeCard, SmokePerps, SmokeRamps, SmokeMultiChainAPI, SmokePredictions, FlaskBuildTests
  • 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-controllers. Running all tests.

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

View GitHub Actions results

@salimtb salimtb requested a review from NicolasMassart March 10, 2026 14:36
@salimtb salimtb disabled auto-merge March 10, 2026 14:39
@salimtb salimtb enabled auto-merge March 10, 2026 14:39
@github-actions
Copy link
Contributor

E2E Fixture Validation — Schema is up to date
11 value mismatches detected (expected — fixture represents an existing user).
View details

@sonarqubecloud
Copy link

@salimtb salimtb added this pull request to the merge queue Mar 10, 2026
Merged via the queue into main with commit 289f897 Mar 10, 2026
130 checks passed
@salimtb salimtb deleted the chore/bump-assets-controllers-v100.3.0 branch March 10, 2026 15:41
@github-actions github-actions bot locked and limited conversation to collaborators Mar 10, 2026
@metamaskbot metamaskbot added the release-7.70.0 Issue or pull request that will be included in release 7.70.0 label Mar 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-7.70.0 Issue or pull request that will be included in release 7.70.0 size-M team-assets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: BEP20 token display $ value for incoming transaction and not for outgoing despite there is no $ value on the on-chain metric

4 participants