Skip to content

chore: update transaction controller #33490

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 3 commits into
base: main
Choose a base branch
from

Conversation

vinistevam
Copy link
Contributor

@vinistevam vinistevam commented Jun 5, 2025

Description

This PR aims to update the transaction controller to 47.1.0 (minor).

Open in GitHub Codespaces

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/4991

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.

@vinistevam vinistevam added the team-confirmations Push issues to confirmations team label Jun 5, 2025
Copy link
Contributor

github-actions bot commented Jun 5, 2025

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.

Copy link

socket-security bot commented Jun 5, 2025

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

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​metamask/​transaction-controller@​57.0.0 ⏵ 57.1.096 +410079 +1100100

View full report

@vinistevam vinistevam marked this pull request as ready for review June 5, 2025 07:50
@metamaskbot
Copy link
Collaborator

Builds ready [7cda1ab]
UI Startup Metrics (1215 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1215110114786712521337
load105290612196310961163
domContentLoaded104590012146410901156
domInteractive16134051629
firstPaint78777121740410801156
backgroundConnect84284821
firstReactRender2216130122134
getState1565582131
initialActions001001
loadScripts79965196663844908
setupStore85172813
WebpackHomeuiStartup20471575252222921972422
load16001247206018417291904
domContentLoaded15951243205418317231890
domInteractive15115091340
firstPaint1646340364190282
backgroundConnect2410333322432
firstReactRender1244436196123332
getState134318311119
initialActions512842834
loadScripts15921241204418117201879
setupStore2873126317256
FirefoxBrowserifyHomeuiStartup1342118318079513871516
load11961058165410012531384
domContentLoaded1196105816539912531384
domInteractive973619929111165
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect211383112047
firstReactRender23205232328
getState7410189
initialActions001001
loadScripts11771029163610112351367
setupStore64334611
WebpackHomeuiStartup15661337205814916431831
load13621163184814214451618
domContentLoaded13611163184814214451618
domInteractive76362102277115
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22165772242
firstReactRender41275654447
getState95316927
initialActions002111
loadScripts13421147183114214281602
setupStore85717813
Benchmark value 21 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 256 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 41 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 3ms | Sum of p95 exceeds: 194ms
Sum of all benchmark exceeds: 197ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -5 Bytes (0%)
  • ui: -8 Bytes (0%)
  • common: 3 KiB (0.04%)

@metamaskbot
Copy link
Collaborator

Builds ready [64daf32]
UI Startup Metrics (1251 ± 69 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1251112114926912961371
load108397313216811221206
domContentLoaded107695913176811141194
domInteractive17145151729
firstPaint711144132543810931189
backgroundConnect84305923
firstReactRender21164252131
getState1463692032
initialActions001001
loadScripts826711105968865945
setupStore85374913
WebpackHomeuiStartup21591609256721023012474
load16741283197816118001894
domContentLoaded16681279196416017941882
domInteractive15114781339
firstPaint1586427650191252
backgroundConnect22107492543
firstReactRender15943371114307353
getState1244571323
initialActions316135
loadScripts16651277195115817921870
setupStore3073416518296
FirefoxBrowserifyHomeuiStartup13691181179310914231598
load1210105115579812681419
domContentLoaded1209105115579812671419
domInteractive1043553555117160
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect22136092248
firstReactRender24215232428
getState12419426827
initialActions001001
loadScripts1190103715319812521391
setupStore84798716
WebpackHomeuiStartup15431403195111215971772
load13321202169110113741539
domContentLoaded13321202169010113731538
domInteractive77323433281110
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2416227222341
firstReactRender42299174448
getState135200201031
initialActions001011
loadScripts13111185167410013511516
setupStore95739824
Benchmark value 1252 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1084 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1077 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 1372 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1206 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1194 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1190 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 24 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 945 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2474 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 296 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 27 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 42 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 53ms | Sum of p95 exceeds: 312ms
Sum of all benchmark exceeds: 365ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 0 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 3.04 KiB (0.04%)

@metamaskbot
Copy link
Collaborator

Builds ready [bac98f4]
UI Startup Metrics (1223 ± 68 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1223109314106812691327
load106493212406811051163
domContentLoaded105692812296910991159
domInteractive17139891529
firstPaint70973123743310921160
backgroundConnect94628824
firstReactRender19155652025
getState1363271925
initialActions001001
loadScripts81369598169857913
setupStore75122811
WebpackHomeuiStartup21141622256823022862475
load16341281197118317651924
domContentLoaded16271277196318217611914
domInteractive15114471340
firstPaint1697142671195304
backgroundConnect2511241242640
firstReactRender12243358100100348
getState175309431229
initialActions316134
loadScripts16241275195218117591903
setupStore5063219319307
FirefoxBrowserifyHomeuiStartup1358121717358914151528
load1203106014448812631368
domContentLoaded1202106014448812631368
domInteractive993527730112145
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21148292143
firstReactRender24215752428
getState9417617810
initialActions001001
loadScripts1184104614198812401352
setupStore9428428611
WebpackHomeuiStartup14791323195911515281693
load12821139169110913141504
domContentLoaded12821138169010913141503
domInteractive75341681977123
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20144352132
firstReactRender40345634245
getState85656913
initialActions102111
loadScripts12641124166810912981486
setupStore85657820
Benchmark value 25 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 51 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2475 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 307 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 40 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 22ms | Sum of p95 exceeds: 270ms
Sum of all benchmark exceeds: 292ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -3 Bytes (0%)
  • ui: -1 Bytes (0%)
  • common: 3.04 KiB (0.04%)

@vinistevam vinistevam added the DO-NOT-MERGE Pull requests that should not be merged label Jun 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DO-NOT-MERGE Pull requests that should not be merged team-confirmations Push issues to confirmations team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants