Skip to content

Server mocha #33524

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

Draft
wants to merge 1 commit into
base: feat/detect-updates-and-prompt-restart
Choose a base branch
from

Conversation

davidmurdoch
Copy link
Contributor

@davidmurdoch davidmurdoch commented Jun 6, 2025

Description

Open in GitHub Codespaces

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.

@metamaskbot metamaskbot added the team-extension-platform Extension Platform team label Jun 6, 2025
Copy link
Contributor

github-actions bot commented Jun 6, 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.

@davidmurdoch davidmurdoch changed the base branch from main to feat/detect-updates-and-prompt-restart June 6, 2025 21:22
@metamaskbot
Copy link
Collaborator

Builds ready [caf00b9]
UI Startup Metrics (1211 ± 70 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1211109114407012511319
load105192712306410921156
domContentLoaded104392312256510861150
domInteractive16133241628
firstPaint76771116240410761137
backgroundConnect94275923
firstReactRender21164762135
getState14682101830
initialActions001000
loadScripts80368699364843906
setupStore85233814
WebpackHomeuiStartup21081595256523722662547
load16311257197018017671900
domContentLoaded16241249196017817621891
domInteractive15115181342
firstPaint1607546759173270
backgroundConnect22136292540
firstReactRender15044363116308353
getState174315421336
initialActions317134
loadScripts16211248194917717571880
setupStore4873229219310
FirefoxBrowserifyHomeuiStartup13221165163510013721540
load1170102413938812201345
domContentLoaded1169102413938812201344
domInteractive923320026101133
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2314115132348
firstReactRender23205142330
getState11417518836
initialActions001001
loadScripts115098113608612031326
setupStore949513725
WebpackHomeuiStartup15271314190111816031761
load13241150159210913951555
domContentLoaded13241150159110913951555
domInteractive78321402086122
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21145362231
firstReactRender40354834346
getState11511714929
initialActions102111
loadScripts13061136157811013781540
setupStore9519819813
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 48 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2547 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 36 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 41 exceeds gate value 38 for firefox webpack home mean firstReactRender
Sum of mean exceeds: 19ms | Sum of p95 exceeds: 355ms
Sum of all benchmark exceeds: 374ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 896 Bytes (0.02%)
  • ui: 2.39 KiB (0.03%)
  • common: 890 Bytes (0.01%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-extension-platform Extension Platform team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants