Skip to content

fix(settings): remove "Proposed nicknames" from settings search results#40772

Merged
cryptotavares merged 8 commits intomainfrom
fix/remove-proposed-nicknames-from-settings-search
Mar 18, 2026
Merged

fix(settings): remove "Proposed nicknames" from settings search results#40772
cryptotavares merged 8 commits intomainfrom
fix/remove-proposed-nicknames-from-settings-search

Conversation

@cryptotavares
Copy link
Copy Markdown
Contributor

@cryptotavares cryptotavares commented Mar 10, 2026

Description

"Proposed nicknames" (externalNameSourcesSetting) appeared as a search result in the Settings search, but clicking it did nothing because the corresponding UI toggle is conditionally rendered (gated behind petnamesEnabled) and the search-to-scroll ref mapping was misaligned.

This PR removes the search index entry from SETTINGS_CONSTANTS so the phantom result no longer appears. The actual toggle UI in the Security tab is left intact for users who have petnamesEnabled active.

Open in GitHub Codespaces

Changelog

CHANGELOG entry: Fixed "Proposed nicknames" appearing as a non-functional result in Settings search

Related issues

Fixes: #40656

Manual testing steps

  1. Open the MetaMask extension
  2. Navigate to Settings
  3. Type "names" in the search bar
  4. Verify "No matching results found" is displayed (previously "Proposed nicknames" appeared)
  5. Type "phishing" in the search bar to confirm search still works for other settings
  6. Navigate to Security & privacy tab and verify it renders correctly

Screenshots/Recordings

Before

Searching "names" shows "Proposed nicknames" as a search result under Security & privacy — clicking it does nothing:

02-search-names-results-1773160172975

After

Searching "names" now correctly shows "No matching results found":

02-search-names-no-results-1773171206690

Other search terms (e.g. "phishing") continue to work:

03-search-phishing-works-1773171230071

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
Adds a new state migration (v199) and updates fixtures/tests to match, which can affect persisted user state and E2E reliability if the migration or version bump is wrong. UI changes are limited to settings/search and removing a conditional toggle path.

Overview
Removes the "Proposed nicknames" entry from settings search (SETTINGS_CONSTANTS) and deletes the corresponding conditional section/toggle wiring from the legacy Security & Privacy tab so the settings search no longer links to a non-functional anchor.

Cleans up the deprecated petnamesEnabled preference across the app by removing it from PreferencesController types/defaults, fixtures, and tests, and introducing migration 199 to delete preferences.petnamesEnabled from persisted state (with fixture migration versions bumped to 199).

Written by Cursor Bugbot for commit aca8484. This will update automatically on new commits. 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.

@metamaskbot metamaskbot added the team-product-safety Push issues to Product Safety team label Mar 10, 2026
@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 10, 2026

Builds ready [0e8cf6d]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account28627530914281309
total28627530914281309
Confirm Txconfirm_tx6070602061113060836111
total6070602061113060836111
Bridge User Actionsbridge_load_page2011902138204213
bridge_load_asset_picker20117622619220226
bridge_search_token7147067288713728
total1126108011844211671184
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup14081176196612114791593
load1164955165310512351318
domContentLoaded1156951162210412251305
domInteractive2916112212581
firstPaint187631189160213317
backgroundConnect20318437921205229
firstReactRender19133752029
initialActions104113
loadScripts970770142410010371119
setupStore1374561623
numNetworkReqs3627101202790
Power User HomeuiStartup5111203011015204263568449
load12961124219414413311549
domContentLoaded12791121212713613171506
domInteractive36201572732109
firstPaint203841187133257318
backgroundConnect17692917306171727955029
firstReactRender28177092942
initialActions108114
loadScripts1053914186912710801273
setupStore1665191736
numNetworkReqs1356125838149214
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2212192252222225
srpButtonToSrpForm99971022100102
confirmSrpToPwForm23222512325
pwFormToMetricsScreen16151711617
metricsToWalletReadyScreen17161701717
doneButtonToHomeScreen67859476774762767
openAccountMenuToAccountListLoaded292729262928129282928
total3985389940787440614078
Onboarding New WalletcreateWalletToSocialScreen2182182190218219
srpButtonToPwForm1111081153112115
createPwToRecoveryScreen981111011
skipBackupToMetricsScreen38383903939
agreeButtonToOnboardingSuccess17161811818
doneButtonToAssetList58050075496613754
total9358921003419221003
Asset DetailsassetClickToPriceChart1091021145113114
total1091021145113114
Solana Asset DetailsassetClickToPriceChart84749379293
total84749379293
Import Srp HomeloginToHomeScreen2253218423174622752317
openAccountMenuAfterLogin563680166280
homeAfterImportWithNewWallet24002222267116424992671
total4642458947195246604719
Send TransactionsopenSendPageFromHome32293733237
selectTokenToSendFormLoaded27173473334
reviewTransactionToConfirmationPage1226844146923314671469
total1289892155124515411551
SwapopenSwapPageFromHome32313213232
fetchAndDisplaySwapQuotes269226862700526932700
total272227152731627272731
🌐 Dapp Page Load Benchmarks

Current Commit: 0e8cf6d | Date: 3/10/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±45ms) 🟡 | historical mean value: 1.05s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 742ms (±65ms) 🟢 | historical mean value: 736ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 92ms (±132ms) 🟢 | historical mean value: 84ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 45ms 1.03s 1.39s 1.08s 1.39s
domContentLoaded 742ms 65ms 717ms 1.33s 767ms 1.33s
firstPaint 92ms 132ms 60ms 1.40s 88ms 1.40s
firstContentfulPaint 92ms 132ms 60ms 1.40s 88ms 1.40s
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs
  • background: 58 Bytes (0%)
  • ui: -241 Bytes (0%)
  • common: 20 Bytes (0%)

@cryptotavares cryptotavares marked this pull request as ready for review March 11, 2026 10:23
@cryptotavares
Copy link
Copy Markdown
Contributor Author

📝 Additional Context from Slack Discussion

Slack Thread Reference: https://consensys.slack.com/archives/C070G0RCX0T/p1773228147183789?thread_ts=1773224831.881679&cid=C070G0RCX0T

Root Cause Analysis

This PR addresses a symptom of an incomplete cleanup from PR #31207, which was intended to remove the petnamesEnabled feature. The original PR was incomplete and left behind:

  1. PreferencesController default state - The petnamesEnabled property was never removed from the default state at app/scripts/controllers/preferences-controller.ts#L189

  2. E2E fixtures - The property was not removed from e2e test fixtures

  3. Security tab rendering - Dead code remains at ui/pages/settings/security-tab/security-tab.component.js#L1397-L1405 that was attempting to render this entry (though it wasn't visible because the petnamesEnabled prop was removed)

  4. Recent regression - In PR #39975, the prop was re-added to address React console errors, but was incorrectly mapped - it was assumed to be on the metamask Redux state when it's actually on metamask.preferences state

Current Fix Scope

This PR fixes the user-facing symptom (phantom search result) by removing the externalNameSourcesSetting entry from SETTINGS_CONSTANTS.

Potential Follow-up Work

For a complete cleanup, consider:

Note on AI-Assisted Development

This PR was created as an experiment with autonomous agent-driven development workflow:

  1. Agent was given the issue
  2. Agent fixed the issue
  3. Agent validated the fix (built Extension + launched with MCP + visual validation)
  4. Agent committed and opened PR

This demonstrates AI agents can effectively fix user-facing issues, while also highlighting that deeper context about historical decisions and incomplete migrations may be needed for comprehensive fixes.


Context captured from Slack discussion by @joao.tavares

The "externalNameSourcesSetting" entry in the settings search index
pointed to a setting that was not consistently visible in the Security
tab, causing a phantom search result. Clicking it did nothing.

Remove the search entry from SETTINGS_CONSTANTS and the associated
settingsRef from the security tab component. The actual toggle UI
remains intact for users who have petnamesEnabled.

Fixes #40656
…d code

Remove the petnamesEnabled preference that was re-introduced after migration
150 due to it remaining in the PreferencesController default state. This adds
migration 199 to clean up persisted state, removes the property from the
controller type/defaults, deletes dead rendering code in the security-tab
(renderExternalNameSourcesToggle and its petnamesEnabled gate), and cleans
up all fixtures and test data.
@cryptotavares cryptotavares force-pushed the fix/remove-proposed-nicknames-from-settings-search branch from 0e8cf6d to b67f5d3 Compare March 13, 2026 14:25
@cryptotavares cryptotavares requested review from a team as code owners March 13, 2026 14:25
@github-actions github-actions bot added size-M and removed size-S labels Mar 13, 2026
@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 13, 2026

Builds ready [b67f5d3]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account32925747178349471
total32925747178349471
Confirm Txconfirm_tx6013599760241160236024
total6013599760241160236024
Bridge User Actionsbridge_load_page24620828728265287
bridge_load_asset_picker17215119920182199
bridge_search_token70769272816724728
total1117108911482411311148
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup16261337226414017021832
load13401092185213214311529
domContentLoaded13301082183813014231523
domInteractive3318133242896
firstPaint17174604104221405
backgroundConnect24021331116243272
firstReactRender21125062332
initialActions106124
loadScripts1111861159812912081305
setupStore1574061823
numNetworkReqs403186164280
Power User HomeuiStartup5496218311397190764009121
load12901089335924612941561
domContentLoaded12681085333524212701516
domInteractive3819272333785
firstPaint218811522159271354
backgroundConnect18192868915160227504528
firstReactRender28176683143
initialActions108114
loadScripts1056882309023510571290
setupStore1656691630
numNetworkReqs1528529143158261
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2192162222220222
srpButtonToSrpForm91899319193
confirmSrpToPwForm22212302223
pwFormToMetricsScreen15151501515
metricsToWalletReadyScreen16161701717
doneButtonToHomeScreen61159064927649649
openAccountMenuToAccountListLoaded2927290229612029372961
total3895388339121239123912
Onboarding New WalletcreateWalletToSocialScreen2182172201219220
srpButtonToPwForm1071071070107107
createPwToRecoveryScreen889089
skipBackupToMetricsScreen39384013940
agreeButtonToOnboardingSuccess16161601616
doneButtonToAssetList53548168084490680
total9268761069838821069
Asset DetailsassetClickToPriceChart54496365363
total54496365363
Solana Asset DetailsassetClickToPriceChart13211314612138146
total13211314612138146
Import Srp HomeloginToHomeScreen2307226623442723122344
openAccountMenuAfterLogin62517297172
homeAfterImportWithNewWallet1606479238190723772381
total39502875469587546624695
Send TransactionsopenSendPageFromHome26242812628
selectTokenToSendFormLoaded32293633436
reviewTransactionToConfirmationPage1207111213147612361314
total1247122712822512821282
SwapopenSwapPageFromHome74241384095138
fetchAndDisplaySwapQuotes2699268127201327042720
total2774274428202727852820
🌐 Dapp Page Load Benchmarks

Current Commit: b67f5d3 | Date: 3/13/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.08s (±47ms) 🟡 | historical mean value: 1.06s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 760ms (±60ms) 🟢 | historical mean value: 745ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 97ms (±127ms) 🟢 | historical mean value: 85ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.08s 47ms 1.03s 1.40s 1.13s 1.40s
domContentLoaded 760ms 60ms 718ms 1.29s 809ms 1.29s
firstPaint 97ms 127ms 72ms 1.36s 96ms 1.36s
firstContentfulPaint 97ms 127ms 72ms 1.36s 96ms 1.36s
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 841 Bytes (0.02%)
  • ui: -3.1 KiB (-0.04%)
  • common: -212 Bytes (0%)

@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 15, 2026

Builds ready [63acb87]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account2812712928281292
total2812712928281292
Confirm Txconfirm_tx601860126023460206023
total601860126023460206023
Bridge User Actionsbridge_load_page2192152212221221
bridge_load_asset_picker24317929446293294
bridge_search_token76070682545796825
total1226110613358712851335
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup14651234193611314811709
load1211101915559912291405
domContentLoaded1205101315459812251391
domInteractive3017117222594
firstPaint164661558158213282
backgroundConnect21319627411216229
firstReactRender20125862128
initialActions106124
loadScripts100781213379610311194
setupStore1263451520
numNetworkReqs403184164080
Power User HomeuiStartup5216227510341174262457715
load13411120209714513831611
domContentLoaded13181108190512913621579
domInteractive40192103635124
firstPaint217821393187274339
backgroundConnect14662867628148225704225
firstReactRender25174962838
initialActions106113
loadScripts1090902160411911271356
setupStore1673971830
numNetworkReqs26312337961311346
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2202192211221221
srpButtonToSrpForm99951012100101
confirmSrpToPwForm23232302323
pwFormToMetricsScreen17161701717
metricsToWalletReadyScreen17161701717
doneButtonToHomeScreen61659466931602669
openAccountMenuToAccountListLoaded29862903316710230283167
total40193880413211141044132
Onboarding New WalletcreateWalletToSocialScreen2182182191219219
srpButtonToPwForm1111081142113114
createPwToRecoveryScreen889089
skipBackupToMetricsScreen39374114041
agreeButtonToOnboardingSuccess16161701617
doneButtonToAssetList50048152415499524
total89587391916898919
Asset DetailsassetClickToPriceChart81808218282
total81808218282
Solana Asset DetailsassetClickToPriceChart973117350120173
total973117350120173
Import Srp HomeloginToHomeScreen2336231923631923632363
openAccountMenuAfterLogin67637346973
homeAfterImportWithNewWallet1285522241087523002410
total36932914492491346904924
Send TransactionsopenSendPageFromHome25202832828
selectTokenToSendFormLoaded34293833438
reviewTransactionToConfirmationPage1014756114315711351143
total1161104412166811921216
SwapopenSwapPageFromHome673387228587
fetchAndDisplaySwapQuotes2700267327432626942743
total2777275528192527762819
🌐 Dapp Page Load Benchmarks

Current Commit: 63acb87 | Date: 3/15/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±40ms) 🟡 | historical mean value: 1.06s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 740ms (±38ms) 🟢 | historical mean value: 746ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 83ms (±15ms) 🟢 | historical mean value: 85ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 40ms 1.02s 1.36s 1.08s 1.36s
domContentLoaded 740ms 38ms 713ms 1.03s 765ms 1.03s
firstPaint 83ms 15ms 68ms 220ms 92ms 220ms
firstContentfulPaint 83ms 15ms 68ms 220ms 92ms 220ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 841 Bytes (0.02%)
  • ui: -1.91 KiB (-0.02%)
  • common: -212 Bytes (0%)

@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 16, 2026

Builds ready [788bc46]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account28726232122300321
total28726232122300321
Confirm Txconfirm_tx602160186022260226022
total602160186022260226022
Bridge User Actionsbridge_load_page21920623410220234
bridge_load_asset_picker22718228939256289
bridge_search_token70569072312714723
total1165109512766812041276
🔌 Startup Benchmarks

⚠️ Missing data: chrome/browserify/startupStandardHome

BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Power User HomeuiStartup5695222514821230764099307
load13081148171212213511593
domContentLoaded12871139167611213261533
domInteractive3520162243275
firstPaint237901538212273390
backgroundConnect196129612507224926126329
firstReactRender24184452533
initialActions107113
loadScripts1065919142910510951315
setupStore177134151931
numNetworkReqs1948742459213313
🧭 User Journey Benchmarks

⚠️ Missing data: chrome/browserify/userJourneyAccountManagement, chrome/browserify/userJourneyTransactions

BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2192192190219219
srpButtonToSrpForm96959719697
confirmSrpToPwForm23222302323
pwFormToMetricsScreen16161601616
metricsToWalletReadyScreen16151711717
doneButtonToHomeScreen61460064117613641
openAccountMenuToAccountListLoaded292229072931929302931
total3908389439391839013939
Onboarding New WalletcreateWalletToSocialScreen2192172212220221
srpButtonToPwForm1081071101108110
createPwToRecoveryScreen989099
skipBackupToMetricsScreen38383803838
agreeButtonToOnboardingSuccess16161601616
doneButtonToAssetList48647349910497499
total87786189913883899
Asset DetailsassetClickToPriceChart684795208995
total684795208995
Solana Asset DetailsassetClickToPriceChart1066417246150172
total1066417246150172
🌐 Dapp Page Load Benchmarks

Current Commit: 788bc46 | Date: 3/16/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±43ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 736ms (±40ms) 🟢 | historical mean value: 742ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 80ms (±12ms) 🟢 | historical mean value: 92ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 43ms 1.02s 1.37s 1.08s 1.37s
domContentLoaded 736ms 40ms 712ms 1.04s 761ms 1.04s
firstPaint 80ms 12ms 60ms 184ms 88ms 184ms
firstContentfulPaint 80ms 12ms 60ms 184ms 88ms 184ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 841 Bytes (0.02%)
  • ui: 20.46 KiB (0.24%)
  • common: 1.42 KiB (0.01%)

@cryptotavares cryptotavares force-pushed the fix/remove-proposed-nicknames-from-settings-search branch from 788bc46 to cc14bda Compare March 16, 2026 23:49
@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 17, 2026

Builds ready [cc14bda]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account27826330113283301
total27826330113283301
Confirm Txconfirm_tx6010598960331860216033
total6010598960331860216033
Bridge User Actionsbridge_load_page22118724727247247
bridge_load_asset_picker25017831747277317
bridge_search_token7597517677766767
total1255122012752212711275
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup15541285210912716241757
load12841038175611813461485
domContentLoaded12761033174311613411478
domInteractive3218127242793
firstPaint181711309182211416
backgroundConnect22419828316231262
firstReactRender2412213212246
initialActions209225
loadScripts1069826152211411301259
setupStore157137131724
numNetworkReqs403186164080
Power User HomeuiStartup5376202614913245262658512
load13061119221315113341619
domContentLoaded12851108219414412951543
domInteractive37201573033127
firstPaint234921563191279359
backgroundConnect184329711848230125705246
firstReactRender24164542630
initialActions104113
loadScripts1063888188813210741300
setupStore1575181729
numNetworkReqs2109034758250317
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2212192221222222
srpButtonToSrpForm93939409494
confirmSrpToPwForm22222202222
pwFormToMetricsScreen15151601516
metricsToWalletReadyScreen16161601616
doneButtonToHomeScreen62359466430664664
openAccountMenuToAccountListLoaded2909289029251429182925
total3894386439292338923929
Onboarding New WalletcreateWalletToSocialScreen2192192190219219
srpButtonToPwForm1081061112107111
createPwToRecoveryScreen888088
skipBackupToMetricsScreen39383903939
agreeButtonToOnboardingSuccess16161701617
doneButtonToAssetList638491746119733746
total112511161138811261138
Asset DetailsassetClickToPriceChart684696208396
total684696208396
Solana Asset DetailsassetClickToPriceChart643896247996
total643896247996
Import Srp HomeloginToHomeScreen2283226523031822982303
openAccountMenuAfterLogin63517386873
homeAfterImportWithNewWallet2329228523764323692376
total4682466146931346924693
Send TransactionsopenSendPageFromHome482099327399
selectTokenToSendFormLoaded482894266294
reviewTransactionToConfirmationPage102794211267310971126
total1123100912608411521260
SwapopenSwapPageFromHome935016140107161
fetchAndDisplaySwapQuotes268726822694426892694
total2780273928433727932843
🌐 Dapp Page Load Benchmarks

Current Commit: cc14bda | Date: 3/17/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.06s (±39ms) 🟡 | historical mean value: 1.05s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 747ms (±36ms) 🟢 | historical mean value: 742ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 83ms (±10ms) 🟢 | historical mean value: 92ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.06s 39ms 1.04s 1.36s 1.08s 1.36s
domContentLoaded 747ms 36ms 730ms 1.02s 761ms 1.02s
firstPaint 83ms 10ms 68ms 172ms 92ms 172ms
firstContentfulPaint 83ms 10ms 68ms 172ms 92ms 172ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 841 Bytes (0.02%)
  • ui: 20.46 KiB (0.24%)
  • common: 1.42 KiB (0.01%)

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.

There are 2 total unresolved issues (including 1 from previous review).

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.

...mockState.metamask,
preferences: {
...mockState.metamask.preferences,
petnamesEnabled: true,
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Misleading test name after removing feature flag

Low Severity

The test is named 'renders appropriately with PetNames enabled' but the petnamesEnabled property it used to set has been removed. The test body now just spreads mockState.metamask.preferences into an identical copy with no actual override, making the entire metamask state override a no-op. The test name no longer reflects what it actually verifies, and the redundant state spread adds confusion for future readers.

Fix in Cursor Fix in Web

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That test is still fine.. petnames are now always enabled. We are just removing the property to specify it or not. They are always on. It is part of the wallet

@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 17, 2026

Builds ready [1a734e8]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account2742732761276276
total2742732761276276
Confirm Txconfirm_tx599859966000260006000
total599859966000260006000
Bridge User Actionsbridge_load_page22921025617234256
bridge_load_asset_picker21314826039242260
bridge_search_token72970275623747756
total1190108412987712291298
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup14641233177011115011667
load1207100414269612471376
domContentLoaded1200100214189512421367
domInteractive2818101182677
firstPaint1536941172215249
backgroundConnect21319326412217236
firstReactRender2113212202129
initialActions106123
loadScripts100380512149510451175
setupStore1376671523
numNetworkReqs393189163979
Power User HomeuiStartup56432216178713364648513342
load13111130175012513621573
domContentLoaded12891125171511913261538
domInteractive40202303434119
firstPaint223861227135284338
backgroundConnect233831114484289725868810
firstReactRender3017310352741
initialActions105114
loadScripts1064924146310911031279
setupStore17793121932
numNetworkReqs1347132349137260
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2212202211221221
srpButtonToSrpForm93929419494
confirmSrpToPwForm22212202222
pwFormToMetricsScreen15151501515
metricsToWalletReadyScreen16161601616
doneButtonToHomeScreen5935905983598598
openAccountMenuToAccountListLoaded292929252931329312931
total3917389039432239393943
Onboarding New WalletcreateWalletToSocialScreen2192162243220224
srpButtonToPwForm1091071122110112
createPwToRecoveryScreen888088
skipBackupToMetricsScreen39384013940
agreeButtonToOnboardingSuccess16161601616
doneButtonToAssetList52350453412531534
total91389392112919921
Asset DetailsassetClickToPriceChart644685177685
total644685177685
Solana Asset DetailsassetClickToPriceChart58331062875106
total58331062875106
Import Srp HomeloginToHomeScreen2334222924889523792488
openAccountMenuAfterLogin977312322120123
homeAfterImportWithNewWallet2362231624083323672408
total47824668492011048594920
Send TransactionsopenSendPageFromHome482390256190
selectTokenToSendFormLoaded392456125156
reviewTransactionToConfirmationPage1033693122120012101221
total1120749135622113141356
SwapopenSwapPageFromHome14912018324160183
fetchAndDisplaySwapQuotes270026902714927052714
total2830274628814928752881
🌐 Dapp Page Load Benchmarks

Current Commit: 1a734e8 | Date: 3/17/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.04s (±42ms) 🟡 | historical mean value: 1.03s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 738ms (±64ms) 🟢 | historical mean value: 729ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 84ms (±44ms) 🟢 | historical mean value: 89ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.04s 42ms 1.02s 1.38s 1.08s 1.38s
domContentLoaded 738ms 64ms 710ms 1.33s 771ms 1.33s
firstPaint 84ms 44ms 68ms 516ms 88ms 516ms
firstContentfulPaint 84ms 44ms 68ms 516ms 88ms 516ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 841 Bytes (0.02%)
  • ui: -1.62 KiB (-0.02%)
  • common: 0 Bytes (0%)

@cryptotavares cryptotavares added team-confirmations Push issues to confirmations team and removed team-product-safety Push issues to Product Safety team labels Mar 17, 2026
@sonarqubecloud
Copy link
Copy Markdown

@metamaskbotv2
Copy link
Copy Markdown
Contributor

metamaskbotv2 bot commented Mar 18, 2026

Builds ready [aca8484]
⚡ Performance Benchmarks
👆 Interaction Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Load New Accountload_new_account28126730616282306
total28126730616282306
Confirm Txconfirm_tx5998598760161159956016
total5998598760161159956016
Bridge User Actionsbridge_load_page23820328026252280
bridge_load_asset_picker2512492521252252
bridge_search_token7737697794779779
total1274119713926913071392
🔌 Startup Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Standard HomeuiStartup14741231180211815411680
load12251009150510912941420
domContentLoaded12191005149710712901408
domInteractive3016111212683
firstPaint1626941274219283
backgroundConnect21719827614223238
firstReactRender19135072030
initialActions105124
loadScripts1018805128710410771199
setupStore1473651725
numNetworkReqs393189163780
Power User HomeuiStartup60752368176543485645615950
load12891138173012913231581
domContentLoaded12701129172012312891566
domInteractive39181913731148
firstPaint221871226171276410
backgroundConnect2844302150993432341411742
firstReactRender23184252632
initialActions104113
loadScripts1048918147711310651321
setupStore1573451725
numNetworkReqs1857832955214293
🧭 User Journey Benchmarks
BenchmarkMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P75 (ms)P95 (ms)
Onboarding Import WalletimportWalletToSocialScreen2172172181218218
srpButtonToSrpForm95949619696
confirmSrpToPwForm22212312223
pwFormToMetricsScreen16151711617
metricsToWalletReadyScreen16161811718
doneButtonToHomeScreen61860065622610656
openAccountMenuToAccountListLoaded293229312933129332933
total3919389539572339143957
Onboarding New WalletcreateWalletToSocialScreen2192182190219219
srpButtonToPwForm1081071091108109
createPwToRecoveryScreen989099
skipBackupToMetricsScreen39374224042
agreeButtonToOnboardingSuccess16161701617
doneButtonToAssetList50148151917516519
total89386991720908917
Asset DetailsassetClickToPriceChart694690157890
total694690157890
Solana Asset DetailsassetClickToPriceChart1191091267126126
total1191091267126126
Import Srp HomeloginToHomeScreen2225217523396621942339
openAccountMenuAfterLogin793812535118125
homeAfterImportWithNewWallet2326225424016024012401
total4645457447105647104710
Send TransactionsopenSendPageFromHome622595278595
selectTokenToSendFormLoaded532282216782
reviewTransactionToConfirmationPage1182113512063312061206
total1291124113323813321332
SwapopenSwapPageFromHome1013714444144144
fetchAndDisplaySwapQuotes268926882690126902690
total2780272728323628052832
🌐 Dapp Page Load Benchmarks

Current Commit: aca8484 | Date: 3/18/2026

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±42ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 741ms (±39ms) 🟢 | historical mean value: 729ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 81ms (±14ms) 🟢 | historical mean value: 84ms ⬇️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.05s 42ms 1.02s 1.36s 1.08s 1.36s
domContentLoaded 741ms 39ms 715ms 1.03s 768ms 1.03s
firstPaint 81ms 14ms 64ms 208ms 88ms 208ms
firstContentfulPaint 81ms 14ms 64ms 208ms 88ms 208ms
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚀 Bundle size reduced!]
  • background: 841 Bytes (0.02%)
  • ui: -1.62 KiB (-0.02%)
  • common: -91 Bytes (0%)

Copy link
Copy Markdown
Member

@seaona seaona left a comment

Choose a reason for hiding this comment

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

e2e files LGTM

Copy link
Copy Markdown
Member

@tuna1207 tuna1207 left a comment

Choose a reason for hiding this comment

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

LGTM

@cryptotavares cryptotavares added this pull request to the merge queue Mar 18, 2026
Merged via the queue into main with commit 34be7cc Mar 18, 2026
391 of 393 checks passed
@cryptotavares cryptotavares deleted the fix/remove-proposed-nicknames-from-settings-search branch March 18, 2026 18:33
@github-actions github-actions bot locked and limited conversation to collaborators Mar 18, 2026
@metamaskbot metamaskbot added the release-13.24.0 Issue or pull request that will be included in release 13.24.0 label Mar 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.24.0 Issue or pull request that will be included in release 13.24.0 size-M team-confirmations Push issues to confirmations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Settings - Proposed nicknames appears in settings search, but it's not present in Settings

6 participants