Skip to content

refactor: remove BIP-44 state 1 flag#40468

Open
gantunesr wants to merge 2 commits intomainfrom
gar/refactor/bip44/state1
Open

refactor: remove BIP-44 state 1 flag#40468
gantunesr wants to merge 2 commits intomainfrom
gar/refactor/bip44/state1

Conversation

@gantunesr
Copy link
Member

@gantunesr gantunesr commented Feb 27, 2026

Description

Remove usage of getIsMultichainAccountsState1Enabled feature flag

Open in GitHub Codespaces

Changelog

CHANGELOG entry: null

Related issues

Fixes: https://consensyssoftware.atlassian.net/browse/MUL-1486

Manual testing steps

Not applicable

Screenshots/Recordings

Not applicable

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

Low Risk
Low risk refactor primarily removing feature-flagged UI/selector code and unused i18n strings; main behavior change is the account options menu no longer exposes the remove-account action.

Overview
Removes multichain accounts state-1 flag support by deleting the enableMultichainAccounts selector/path and simplifying related UI logic to always use the state-2 behavior.

The account menu/button copy is standardized to addAccountOrWallet, and the account list item menu drops the Remove account menu item (keeping isRemovable only for API compatibility), with tests and locale strings updated to match.

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

@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Feb 27, 2026

✨ Files requiring CODEOWNER review ✨

🔑 @MetaMask/accounts-engineers (4 files, +15 -115)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 account-list-item-menu/
          • 📄 account-list-item-menu.js +9 -71
          • 📄 account-list-item-menu.test.js +4 -16
        • 📁 account-list-menu/
          • 📄 account-list-menu.test.tsx +1 -1
    • 📁 selectors/
      • 📁 multichain-accounts/
        • 📄 feature-flags.ts +1 -27

👨‍🔧 @MetaMask/core-extension-ux (5 files, +22 -126)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 account-list-item-menu/
          • 📄 account-list-item-menu.js +9 -71
          • 📄 account-list-item-menu.test.js +4 -16
        • 📁 account-list-menu/
          • 📄 account-list-menu.test.tsx +1 -1
        • 📁 account-menu/
          • 📄 account-menu.test.tsx +4 -16
          • 📄 account-menu.tsx +4 -22

@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Feb 27, 2026

Builds ready [f7addef]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account391271536106491536
total391271536106491536
Confirm Txconfirm_tx608260806085260856085
total608260806085260856085
Bridge User Actionsbridge_load_page24919427931278279
bridge_load_asset_picker20715128547232285
bridge_search_token73570676121751761
total1191113312725012201272
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup13231114159210613551563
load111392913429011411303
domContentLoaded110692413368911351295
domInteractive2616130202377
firstPaint167621075114202292
backgroundConnect19317541625193214
firstReactRender19125052029
initialActions105113
loadScripts9317461160909621120
setupStore1273151322
numNetworkReqs312298212287
Power User HomeuiStartup17501462211612417971971
load11701040170213811581563
domContentLoaded11551024167213411481544
domInteractive3820180263984
firstPaint188741622213214299
backgroundConnect29425946428302352
firstReactRender23145972637
initialActions106113
loadScripts94382414631349291331
setupStore1573651724
numNetworkReqs59391522457120
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2182172191219219
srpButtonToSrpForm89879018990
confirmSrpToPwForm21212102121
pwFormToMetricsScreen14141501515
metricsToWalletReadyScreen16131821718
doneButtonToHomeScreen879571136537213011365
openAccountMenuToAccountListLoaded7073699472079572077207
total82837825895347687648953
Onboarding New WalletcreateWalletToSocialScreen2222212231222223
srpButtonToPwForm1071061070107107
createPwToRecoveryScreen888088
skipBackupToMetricsScreen37353813738
agreeButtonToOnboardingSuccess16151601616
doneButtonToAssetList838559118326611341183
total1228943157126715261571
Asset DetailsassetClickToPriceChart533380197380
total533380197380
Solana Asset DetailsassetClickToPriceChart48474914949
total48474914949
Import Srp HomeloginToHomeScreen1956187520597220092059
openAccountMenuAfterLogin43355364653
homeAfterImportWithNewWallet21502008232711722022327
total41493946433414142724334
Send TransactionsopenSendPageFromHome28183993839
selectTokenToSendFormLoaded18181901919
reviewTransactionToConfirmationPage8628578674866867
total90588792413915924
SwapopenSwapPageFromHome1311281332133133
fetchAndDisplaySwapQuotes4663457047747547264774
total4795472149076948404907
🌐 Dapp Page Load Benchmarks

Current Commit: f7addef | Date: 2/27/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.06s (±44ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 728ms (±43ms) 🟢 | historical mean value: 735ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 83ms (±13ms) 🟢 | historical mean value: 84ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.06s 44ms 1.02s 1.39s 1.09s 1.39s
domContentLoaded 728ms 43ms 700ms 1.05s 760ms 1.05s
firstPaint 83ms 13ms 68ms 200ms 92ms 200ms
firstContentfulPaint 83ms 13ms 68ms 200ms 92ms 200ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs
  • background: 58 Bytes (0%)
  • ui: 99 Bytes (0%)
  • common: -341 Bytes (0%)

@sonarqubecloud
Copy link

@metamaskbotv2
Copy link
Contributor

metamaskbotv2 bot commented Feb 27, 2026

Builds ready [dd4640d]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account2712692732272273
total2712692732272273
Confirm Txconfirm_tx604260346049560436049
total604260346049560436049
Bridge User Actionsbridge_load_page26223028921278289
bridge_load_asset_picker20111535189250351
bridge_search_token72270375221744752
total1185110713028312671302
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup14001212195111214321593
load11831008163010012141356
domContentLoaded117599616049712001331
domInteractive2816145202472
firstPaint162691140130205363
backgroundConnect20719027413207229
firstReactRender19123852028
initialActions108124
loadScripts98580614049610121137
setupStore1373551522
numNetworkReqs312288192283
Power User HomeuiStartup2864146310959194635704950
load12441042170713912901583
domContentLoaded12241035168113312641553
domInteractive3718195263693
firstPaint2009245790280353
backgroundConnect10202698793156010552716
firstReactRender26165472839
initialActions108113
loadScripts1010827145712910411337
setupStore1655081829
numNetworkReqs78441412289119
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2202172222221222
srpButtonToSrpForm94939519595
confirmSrpToPwForm22212202222
pwFormToMetricsScreen15151601616
metricsToWalletReadyScreen16161701617
doneButtonToHomeScreen81859011361918911136
openAccountMenuToAccountListLoaded74526859782935876877829
total8825873189437888428943
Onboarding New WalletcreateWalletToSocialScreen2222202242223224
srpButtonToPwForm1071001125111112
createPwToRecoveryScreen889099
skipBackupToMetricsScreen35343713637
agreeButtonToOnboardingSuccess16161601616
doneButtonToAssetList812558119027811111190
total1204939159328215011593
Asset DetailsassetClickToPriceChart39384014040
total39384014040
Solana Asset DetailsassetClickToPriceChart59585905959
total59585905959
Import Srp HomeloginToHomeScreen20101912219210420532192
openAccountMenuAfterLogin47474804748
homeAfterImportWithNewWallet23602146271520524022715
total44174124481526146054815
Send TransactionsopenSendPageFromHome19172111921
selectTokenToSendFormLoaded20182211922
reviewTransactionToConfirmationPage8618558654864865
total90689392110911921
SwapopenSwapPageFromHome1151131203120120
fetchAndDisplaySwapQuotes4702461548178147384817
total4821472849168048834916
🌐 Dapp Page Load Benchmarks

Current Commit: dd4640d | Date: 2/27/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±47ms) 🟡 | historical mean value: 1.04s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 728ms (±43ms) 🟢 | historical mean value: 731ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 79ms (±10ms) 🟢 | historical mean value: 83ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 47ms 1.01s 1.36s 1.05s 1.36s
domContentLoaded 728ms 43ms 708ms 1.03s 738ms 1.03s
firstPaint 79ms 10ms 60ms 160ms 88ms 160ms
firstContentfulPaint 79ms 10ms 60ms 160ms 88ms 160ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs
  • background: -136 Bytes (0%)
  • ui: 99 Bytes (0%)
  • common: -123 Bytes (0%)

@gantunesr gantunesr marked this pull request as ready for review February 27, 2026 13:09
@gantunesr gantunesr requested review from a team as code owners February 27, 2026 13:09
@gantunesr gantunesr added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed size-M team-accounts-framework Accounts team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants