Skip to content

Conversation

@davidmurdoch
Copy link
Contributor

@davidmurdoch davidmurdoch commented Jul 25, 2025

I accidentally removed comments from .yarnrc.yml in #33463

This PR restores them.

@github-actions
Copy link
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-extension-platform Extension Platform team label Jul 25, 2025
@metamaskbot
Copy link
Collaborator

metamaskbot commented Jul 25, 2025

✨ Files requiring CODEOWNER review ✨

👨‍🔧 @MetaMask/extension-platform (1 files, +124 -22)
  • 📄 .yarnrc.yml +124 -22

@davidmurdoch davidmurdoch added the no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed label Jul 25, 2025
@davidmurdoch davidmurdoch changed the title ci: put comments back in .yarnrc.yml chore: put comments back in .yarnrc.yml Jul 25, 2025
@metamaskbot
Copy link
Collaborator

Builds ready [2c85db7]
UI Startup Metrics (1302 ± 55 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1302120515125513391391
load112099512685411601200
domContentLoaded111098912625611531194
domInteractive18144761734
firstPaint733150127144011241197
backgroundConnect25423637318254271
firstReactRender2113167162032
getState103113121125
initialActions30607213
loadScripts865745101956908946
setupStore74254717
WebpackHomeuiStartup24621967308218625392868
load19881409254221121412424
domContentLoaded19731404247720021302297
domInteractive2213256331561
firstPaint1656552576182313
backgroundConnect55113628433288
firstReactRender1419033357174295
getState2843946713233
initialActions7217417618
loadScripts19591401246220521162256
setupStore216239421630
FirefoxBrowserifyHomeuiStartup15001309214214415561787
load1291113116699513391492
domContentLoaded1291113116699513391492
domInteractive1183649389103347
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2814195212955
firstReactRender302066113060
getState113155221324
initialActions6018420324
loadScripts1267111216499113181446
setupStore11315722733
WebpackHomeuiStartup17621540237616718562067
load14911315196414215881799
domContentLoaded14911314196414215881799
domInteractive1153643675112351
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect29207893147
firstReactRender53426955764
getState153252331629
initialActions9024527529
loadScripts14671290194114115611762
setupStore145247271143
Benchmark value 1302 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1120 exceeds gate value 1070 for chrome browserify home mean load
Benchmark value 1110 exceeds gate value 1061 for chrome browserify home mean domContentLoaded
Benchmark value 255 exceeds gate value 10 for chrome browserify home mean backgroundConnect
Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 865 exceeds gate value 830 for chrome browserify home mean loadScripts
Benchmark value 1392 exceeds gate value 1365 for chrome browserify home p95 uiStartup
Benchmark value 1200 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 1198 exceeds gate value 1180 for chrome browserify home p95 firstPaint
Benchmark value 271 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 13 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 946 exceeds gate value 940 for chrome browserify home p95 loadScripts
Benchmark value 2463 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1988 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1973 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 23 exceeds gate value 21 for chrome webpack home mean domInteractive
Benchmark value 55 exceeds gate value 40 for chrome webpack home mean backgroundConnect
Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 1959 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2869 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2424 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2298 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 62 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 288 exceeds gate value 90 for chrome webpack home p95 backgroundConnect
Benchmark value 233 exceeds gate value 195 for chrome webpack home p95 getState
Benchmark value 18 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2257 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 1501 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1292 exceeds gate value 1245 for firefox browserify home mean load
Benchmark value 1291 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 118 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 28 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 31 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 12 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 7 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 1268 exceeds gate value 1230 for firefox browserify home mean loadScripts
Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1787 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 347 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 60 exceeds gate value 55 for firefox browserify home p95 firstReactRender
Benchmark value 24 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 33 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1763 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1491 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1491 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 115 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 29 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 54 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 9 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1467 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore
Benchmark value 2067 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1799 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1799 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 351 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 64 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 29 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1762 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 43 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 2324ms | Sum of p95 exceeds: 3085.8ms
Sum of all benchmark exceeds: 5409.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 38 Bytes (0%)
  • ui: 63 Bytes (0%)
  • common: 15 Bytes (0%)

@davidmurdoch davidmurdoch marked this pull request as ready for review July 25, 2025 21:23
@davidmurdoch davidmurdoch requested a review from a team as a code owner July 25, 2025 21:23
- 'safe-event-emitter (deprecation)' # via eth-block-tracker and others

npmRegistryServer: 'https://registry.npmjs.org/'
# @metamask-institutional relies upon crypto which is deprecated
Copy link

Choose a reason for hiding this comment

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

Bug: Yarn Configuration Loss Affects Package Management

The npmRegistryServer: 'https://registry.npmjs.org/' configuration line is accidentally removed from .yarnrc.yml. This line, previously located between the npmAuditIgnoreAdvisories and plugins sections, is not restored, which could disrupt Yarn's package resolution and registry selection.

Locations (2)

Fix in CursorFix in Web

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it was accidentally added. i'm removing it on purpose.

Copy link
Member

@Gudahtt Gudahtt left a comment

Choose a reason for hiding this comment

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

LGTM!

@HowardBraham HowardBraham added this pull request to the merge queue Aug 2, 2025
Merged via the queue into main with commit c9ccdea Aug 2, 2025
402 of 406 checks passed
@HowardBraham HowardBraham deleted the putcommentsbackinyarnyml branch August 2, 2025 00:51
@github-actions github-actions bot locked and limited conversation to collaborators Aug 2, 2025
@metamaskbot metamaskbot added the release-13.2.0 Issue or pull request that will be included in release 13.2.0 label Aug 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no-changelog no-changelog Indicates no external facing user changes, therefore no changelog documentation needed release-13.2.0 Issue or pull request that will be included in release 13.2.0 team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants