Skip to content

chore: bump assets controllers to 99.3.0 to fix stock badges on homepage#25775

Closed
oscarwroche wants to merge 17 commits intomainfrom
chore/bump-assets-controllers-99.3.0
Closed

chore: bump assets controllers to 99.3.0 to fix stock badges on homepage#25775
oscarwroche wants to merge 17 commits intomainfrom
chore/bump-assets-controllers-99.3.0

Conversation

@oscarwroche
Copy link
Copy Markdown
Contributor

@oscarwroche oscarwroche commented Feb 6, 2026

Description

  • Bumping the version of assets-controllers to fix a cache issue that prevents stock badges from being displayed for RWA tokens

Changelog

CHANGELOG entry: Fixed cache issue for RWA token stock badges not appearing on home page

Related issues

Fixes:

Manual testing steps

Feature: my feature name

  Scenario: user [verb for user action]
    Given [describe expected initial app state]

    When user [verb for user action]
    Then [describe expected outcome]

Screenshots/Recordings

Before

After

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.

Note

Medium Risk
Dependency upgrade in the assets/controller layer could change token/asset caching and badge display behavior beyond the intended fix, though the PR itself is primarily version/lockfile churn.

Overview
Updates @metamask/assets-controllers from a locally patched 99.2.0 to ^99.3.0 (lockfile resolves to 99.3.1), removing the patch override.

Adds a changelog entry noting a fix for cached RWA token stock badges not appearing on the home/trending UI.

Written by Cursor Bugbot for commit acfbc22. This will update automatically on new commits. Configure here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 6, 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.

@metamaskbot metamaskbot added the team-be-trade BE Trade team label Feb 6, 2026
Akaryatrh
Akaryatrh previously approved these changes Feb 6, 2026
@Akaryatrh Akaryatrh enabled auto-merge February 6, 2026 16:53
asalsys
asalsys previously approved these changes Feb 6, 2026
@oscarwroche oscarwroche disabled auto-merge February 7, 2026 08:48
@Akaryatrh Akaryatrh enabled auto-merge February 9, 2026 08:56
@Akaryatrh Akaryatrh dismissed stale reviews from asalsys and themself via 3a78b3d February 9, 2026 11:57
Akaryatrh
Akaryatrh previously approved these changes Feb 9, 2026
Akaryatrh
Akaryatrh previously approved these changes Feb 10, 2026
Copy link
Copy Markdown

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

"@metamask/app-metadata-controller": "^2.0.0",
"@metamask/approval-controller": "^8.0.0",
"@metamask/assets-controllers": "patch:@metamask/assets-controllers@npm%3A99.2.0#~/.yarn/patches/@metamask-assets-controllers-npm-99.2.0-bf6ecbcde9.patch",
"@metamask/assets-controllers": "^99.3.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Resolutions still force old patched 99.2.0 for transitive deps

High Severity

The direct dependency @metamask/assets-controllers was bumped to ^99.3.0, but the resolutions section (lines 189–190) still redirects @metamask/assets-controllers@npm:^99.2.0 and @metamask/assets-controllers@npm:^97.0.0 to the old patched 99.2.0. This means transitive dependencies that request those ranges will still receive the old version without the cache fix. The project ends up with two installed versions of assets-controllers (99.3.1 and patched 99.2.0), potentially undermining the very fix this PR intends to ship.

Fix in Cursor Fix in Web

@github-actions
Copy link
Copy Markdown
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

@sonarqubecloud
Copy link
Copy Markdown

@Akaryatrh
Copy link
Copy Markdown
Contributor

Closed in favor of #25796

@Akaryatrh Akaryatrh closed this Feb 12, 2026
auto-merge was automatically disabled February 12, 2026 08:27

Pull request was closed

@github-actions github-actions bot locked and limited conversation to collaborators Feb 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants