Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@MarioAslau
Copy link
Contributor

@MarioAslau MarioAslau commented Dec 19, 2025

Description

Bumping version of remote-feature-flag-controller to support independent A/B tests introduced in:

Changelog

CHANGELOG entry:

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

Bumps @metamask/remote-feature-flag-controller to 4.0.0 and refreshes lockfile, updating related @metamask/utils and @metamask/controller-utils versions.

  • Dependencies:
    • Upgrade @metamask/remote-feature-flag-controller from ^3.1.0 to ^4.0.0 in package.json.
    • Update yarn.lock resolutions accordingly, including @metamask/utils to 11.9.0 and @metamask/controller-utils to 11.17.0.

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

@MarioAslau MarioAslau self-assigned this Dec 19, 2025
@github-actions
Copy link
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.

@metamaskbot metamaskbot added the team-mobile-platform Mobile Platform team label Dec 19, 2025
@MarioAslau MarioAslau added team-mobile-delivery no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed no changelog required No changelog entry is required for this change and removed team-mobile-platform Mobile Platform team labels Dec 19, 2025
@MarioAslau MarioAslau marked this pull request as ready for review December 19, 2025 23:09
@MarioAslau MarioAslau requested a review from sethkfman December 19, 2025 23:09
@socket-security
Copy link

socket-security bot commented Dec 19, 2025

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​metamask/​remote-feature-flag-controller@​4.0.0991007696100
Updatednpm/​@​metamask/​utils@​11.8.1 ⏵ 11.9.099 +110094 +193 +6100

View full report

weitingsun
weitingsun previously approved these changes Dec 19, 2025
@github-actions
Copy link
Contributor

🔍 Smart E2E Test Selection

  • Selected E2E tags: SmokeCore, SmokeAccounts, SmokeConfirmationsRedesigned, SmokeIdentity, SmokeNetworkAbstractions, SmokeNetworkExpansion, SmokeTrade, SmokeWalletPlatform, SmokeWalletUX, SmokeAssets, SmokeSwaps, SmokeStake, SmokeCard, SmokeNotifications, SmokeRewards, SmokePerps, SmokeRamps, SmokeMultiChainPermissions, SmokeAnalytics, SmokeMultiChainAPI, SmokePredictions
  • Risk Level: high
  • AI Confidence: 90%
click to see 🤖 AI reasoning details

This PR upgrades @metamask/remote-feature-flag-controller from v3.1.0 to v4.0.0 - a MAJOR version bump. This controller is a critical core dependency that:

  1. Is deeply integrated into the Engine - Initialized in Engine.ts and used across many controllers and messengers

  2. Controls feature flags for virtually all features - The codebase search shows it's used in:

    • Confirmations (redesign flags)
    • Perps (perpetuals trading)
    • Predictions
    • Earn/Staking (TRX staking, etc.)
    • Card features
    • Ramps (buy/sell)
    • Network controller
    • Multichain accounts (Bitcoin, Tron, Solana)
    • Assets (trending tokens)
    • Token search/discovery
  3. API changes detected - The yarn.lock shows the new version uses @metamask/messenger (v0.3.0) instead of the previous messenger pattern, indicating potential breaking changes in controller communication.

  4. Affects state management - RemoteFeatureFlagControllerState is used across many selectors and components to determine feature availability.

Given this is a major version upgrade of a controller that gates features across the entire application, any regression could disable or break multiple features. All test tags should be run to ensure comprehensive coverage.

View GitHub Actions results

Copy link
Contributor

@sethkfman sethkfman left a comment

Choose a reason for hiding this comment

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

LGTM

@sonarqubecloud
Copy link

@MarioAslau MarioAslau added this pull request to the merge queue Dec 20, 2025
Merged via the queue into main with commit e03727e Dec 20, 2025
93 checks passed
@MarioAslau MarioAslau deleted the feat-bump-feature-flag-controller-4.0.0 branch December 20, 2025 00:35
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2025
@metamaskbot metamaskbot added the release-7.62.0 Issue or pull request that will be included in release 7.62.0 label Dec 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no changelog required No changelog entry is required for this change no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-7.62.0 Issue or pull request that will be included in release 7.62.0 size-XS team-mobile-delivery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants