Skip to content

feat: removed feature flag for gns #33459

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Conversation

NidhiKJha
Copy link
Member

This PR is to remove GNS feature flag

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

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.

@NidhiKJha NidhiKJha requested a review from a team as a code owner June 3, 2025 19:22
@metamaskbot metamaskbot added team-wallet-ux INVALID-PR-TEMPLATE PR's body doesn't match template labels Jun 3, 2025
@metamaskbot
Copy link
Collaborator

metamaskbot commented Jun 3, 2025

✨ Files requiring CODEOWNER review ✨

💎 @MetaMask/metamask-assets (1 files, +5 -1)
  • 📁 ui/
    • 📁 components/
      • 📁 app/
        • 📁 assets/
          • 📁 token-cell/
            • 📁 __snapshots__/
              • 📄 token-cell.test.tsx.snap +5 -1

🖥️ @MetaMask/wallet-ux (7 files, +253 -378)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 account-picker/
          • 📁 __snapshots__/
            • 📄 account-picker.test.js.snap +1 -12
            • 📄 account-picker.js +2 -25
            • 📄 account-picker.test.js +0 -12
        • 📁 address-list-item/
          • 📁 __snapshots__/
            • 📄 address-list-item.test.tsx.snap +138 -87
            • 📄 address-list-item.tsx +13 -26
        • 📁 app-header/
          • 📁 __snapshots__/
            • 📄 app-header.test.js.snap +79 -110
            • 📄 app-header-unlocked-content.tsx +20 -106

@metamaskbot
Copy link
Collaborator

Builds ready [3ad8443]
UI Startup Metrics (1248 ± 72 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1248112315107212961385
load107995813257211331191
domContentLoaded107294813207211261186
domInteractive17133441729
firstPaint70989125644110931193
backgroundConnect94376823
firstReactRender18134041824
getState1567992029
initialActions001001
loadScripts822700105970870934
setupStore85263813
WebpackHomeuiStartup20461567265924321952463
load15941222202420017361993
domContentLoaded15851216201719717231981
domInteractive161179111346
firstPaint1606336261169297
backgroundConnect3210430572555
firstReactRender12740361104112352
getState1043141222
initialActions317134
loadScripts15821214201619617211971
setupStore2863166416294
FirefoxBrowserifyHomeuiStartup13741161207013114361606
load12181044162711012841435
domContentLoaded12181044162711012831435
domInteractive983636135108140
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2514341342248
firstReactRender22183032227
getState84354811
initialActions001001
loadScripts11971030161010512621400
setupStore947810722
WebpackHomeuiStartup15531358208215716601859
load13541171187814714521627
domContentLoaded13541170187814714521627
domInteractive75333373078103
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21157882230
firstReactRender38254844146
getState104849927
initialActions101011
loadScripts13351155185914614301602
setupStore95838814
Benchmark value 1249 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1080 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1072 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1385 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1192 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1186 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1193 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 2463 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 1971 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 294 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 26 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 39 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 38ms | Sum of p95 exceeds: 285ms
Sum of all benchmark exceeds: 323ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: -23.61 KiB (-0.44%)
  • ui: 2.16 KiB (0.03%)
  • common: -7.79 KiB (-0.09%)

@metamaskbot
Copy link
Collaborator

Builds ready [5baad98]
UI Startup Metrics (1202 ± 71 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1202109014597112321359
load104192012577010701202
domContentLoaded103491112537110631193
domInteractive16133751629
firstPaint71367122441510541194
backgroundConnect84295824
firstReactRender18124351831
getState1455781926
initialActions001001
loadScripts78766997269815944
setupStore85293817
WebpackHomeuiStartup20621650255921222092470
load16141293196316617251904
domContentLoaded16081289195016417201900
domInteractive161160101346
firstPaint1666460775198305
backgroundConnect2512273262639
firstReactRender111393549395344
getState1144671225
initialActions317134
loadScripts16051288193916317161898
setupStore3263137018302
FirefoxBrowserifyHomeuiStartup13621174171911314251648
load12101015152410312831412
domContentLoaded12101014152410312831411
domInteractive1003541740109152
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2114170162131
firstReactRender22198362226
getState11521524811
initialActions001001
loadScripts11911004150210112621394
setupStore74527621
WebpackHomeuiStartup21311640330835523492840
load18831360299833720582501
domContentLoaded18831360299833720582501
domInteractive1074940946114163
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect27177282943
firstReactRender513799115478
getState1063451123
initialActions102111
loadScripts18601335297233620302480
setupStore1062131118
cc: @HowardBraham
Benchmark value 1203 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1193 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1195 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 25 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 944 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2471 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 302 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 2131 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1884 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1884 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 108 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 27 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 52 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1861 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2840 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 2501 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 2501 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 163 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 78 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 2480 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 2048ms | Sum of p95 exceeds: 3778ms
Sum of all benchmark exceeds: 5826ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: -22.65 KiB (-0.42%)
  • ui: -888 Bytes (-0.01%)
  • common: -7.9 KiB (-0.09%)

@metamaskbot
Copy link
Collaborator

Builds ready [8561168]
UI Startup Metrics (1219 ± 74 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1219109514657412611382
load105994812626611051172
domContentLoaded105393712566611001166
domInteractive16133951629
firstPaint70069121143610991143
backgroundConnect74344812
firstReactRender17133131822
getState1465981928
initialActions001001
loadScripts808691100865854912
setupStore85162811
WebpackHomeuiStartup20871643260722022372470
load16361301198017617731928
domContentLoaded16291297196317517661919
domInteractive15115291340
firstPaint1576630449192244
backgroundConnect231056102750
firstReactRender12340371105120341
getState1044561223
initialActions317134
loadScripts16261296195217417641918
setupStore65732011123313
FirefoxBrowserifyHomeuiStartup13351160221914313921559
load11971040207514112601414
domContentLoaded11971040207514112591414
domInteractive110341078104109161
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect19134252028
firstReactRender20182822125
getState74757810
initialActions001001
loadScripts11801025205814012421398
setupStore64243611
WebpackHomeuiStartup16291360208816217072010
load14251184190615615151803
domContentLoaded14251184190515615141802
domInteractive81492872786117
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2416245232342
firstReactRender40255044347
getState11512313928
initialActions001011
loadScripts14041169188715414941783
setupStore10526827810
Benchmark value 1382 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 66 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2471 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 313 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 111 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 1629 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1426 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1425 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 40 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 1404 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2010 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1803 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1802 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 1783 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 186ms | Sum of p95 exceeds: 795ms
Sum of all benchmark exceeds: 981ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: -132.95 KiB (-2.49%)
  • ui: 12.04 KiB (0.17%)
  • common: -14.44 KiB (-0.17%)

@metamaskbot
Copy link
Collaborator

Builds ready [d446ed8]
UI Startup Metrics (1188 ± 54 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1188109213135412211298
load103293711414710621105
domContentLoaded102592911374910561100
domInteractive16133141629
firstPaint666129112640610361105
backgroundConnect84295823
firstReactRender17134031721
getState1464082027
initialActions0010101
loadScripts78368589649817858
setupStore85283814
WebpackHomeuiStartup20591582253023822052444
load16261256191819017331901
domContentLoaded16211252191318917291897
domInteractive14114771335
firstPaint1566332460199267
backgroundConnect21123662434
firstReactRender96413457388315
getState194309511221
initialActions315134
loadScripts16181250191118817271889
setupStore4273118716303
FirefoxBrowserifyHomeuiStartup1309120216488813591531
load1161101714157612211316
domContentLoaded1161101614147612211315
domInteractive91332072798135
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21135992148
firstReactRender22195352126
getState11417123816
initialActions001001
loadScripts1142100214007812031298
setupStore74334712
WebpackHomeuiStartup14901317178710715531714
load1297115115449913541495
domContentLoaded1296115115449913541495
domInteractive76542652478115
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20153732126
firstReactRender38266654046
getState11424625830
initialActions002111
loadScripts1279113315269913361476
setupStore75283812
Benchmark value 24 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 43 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 303 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Sum of mean exceeds: 12ms | Sum of p95 exceeds: 244ms
Sum of all benchmark exceeds: 256ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: -1.97 KiB (-0.03%)
  • common: 112 Bytes (0%)

@NidhiKJha NidhiKJha requested review from a team as code owners June 7, 2025 20:29
@MetaMask MetaMask deleted a comment from richardcacho Jun 7, 2025
Copy link

@richardcacho richardcacho left a comment

Choose a reason for hiding this comment

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

done/review

@metamaskbot
Copy link
Collaborator

Builds ready [b738456]
UI Startup Metrics (1213 ± 66 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1213108514506612501351
load104794612506210811159
domContentLoaded104092912456310751151
domInteractive17133441628
firstPaint724138123141210611150
backgroundConnect84295822
firstReactRender17132921821
getState1464991931
initialActions001001
loadScripts79469198462827901
setupStore85152814
WebpackHomeuiStartup20161567251425322192414
load15641252191918717041844
domContentLoaded15591248191318616991836
domInteractive15114571338
firstPaint1616455974191282
backgroundConnect2210263252332
firstReactRender1224035499120335
getState164310421122
initialActions813054234
loadScripts15561246191118416971825
setupStore247308551623
FirefoxBrowserifyHomeuiStartup1326117416389613761535
load1178101714428512381345
domContentLoaded1178101714428512381345
domInteractive1003434842103171
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect19128691928
firstReactRender20182822126
getState11418326813
initialActions001001
loadScripts1161100414238312221321
setupStore74718615
WebpackHomeuiStartup15431370196712416101787
load13421185172511414141571
domContentLoaded13421185172511414141570
domInteractive80551742288126
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22155682340
firstReactRender38284534043
getState95325926
initialActions102111
loadScripts13231166170111413971555
setupStore10525525818
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 9 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Sum of mean exceeds: 3ms | Sum of p95 exceeds: 5ms
Sum of all benchmark exceeds: 8ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -3.78 KiB (-0.05%)
  • common: -8 Bytes (0%)

@metamaskbot
Copy link
Collaborator

Builds ready [d51fb64]
UI Startup Metrics (1217 ± 74 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1217109715687412641335
load105593613576611011156
domContentLoaded104892813526710931151
domInteractive1913233231631
firstPaint74176137741810941134
backgroundConnect84325824
firstReactRender17142421821
getState1563682130
initialActions001000
loadScripts806687108965851905
setupStore85192913
WebpackHomeuiStartup21081592253824122992493
load16421247200418617821921
domContentLoaded16361242199418517741911
domInteractive15115481337
firstPaint1696454170188306
backgroundConnect21135362534
firstReactRender13342358107124340
getState134265261217
initialActions317134
loadScripts16331241198418317711900
setupStore4673269118310
FirefoxBrowserifyHomeuiStartup13301160169610913881579
load1180103914809012291367
domContentLoaded1180103914809012291367
domInteractive993434744107196
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2112132142131
firstReactRender21184942127
getState9415417811
initialActions002001
loadScripts1162102514668912141352
setupStore9417818711
WebpackHomeuiStartup15431370193410816251710
load13421192171410714261532
domContentLoaded13421192171310714261531
domInteractive77311851881112
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2415237222435
firstReactRender39276144145
getState95305923
initialActions102111
loadScripts13211175169910514041498
setupStore95687820
Benchmark value 24 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 47 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2493 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 310 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 196 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 39 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 16ms | Sum of p95 exceeds: 291ms
Sum of all benchmark exceeds: 307ms

Bundle size diffs [🚀 Bundle size reduced!]
  • background: 0 Bytes (0%)
  • ui: -3.75 KiB (-0.05%)
  • common: -8 Bytes (0%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
INVALID-PR-TEMPLATE PR's body doesn't match template team-wallet-ux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants