Skip to content

chore: bump @metamask/profile-sync-controller to ^20.0.0#34078

Merged
mathieuartu merged 6 commits intomainfrom
chore/bump-identity-notification-controllers
Jul 8, 2025
Merged

chore: bump @metamask/profile-sync-controller to ^20.0.0#34078
mathieuartu merged 6 commits intomainfrom
chore/bump-identity-notification-controllers

Conversation

@mathieuartu
Copy link
Copy Markdown
Contributor

@mathieuartu mathieuartu commented Jul 4, 2025

Description

This PR bumps:

  • @metamask/profile-sync-controller to ^20.0.0

Updates from the package changelog since ^18.0.0:

## [20.0.0]

### Added

- Add new `AuthenticationController:getUserProfileMetaMetrics` method ([#6068](https://github.com/MetaMask/core/pull/6068))
  - This method fetches data using the Authentication API, returning all MetaMetrics sessions related to the currently authenticated user, in the form of `typeof UserProfileMetaMetrics`

### Changed

- **BREAKING:** Bump peer dependency `@metamask/snaps-controllers` from `^12.0.0` to `^14.0.0` ([#6035](https://github.com/MetaMask/core/pull/6035))
- Bump `@metamask/snaps-sdk` from `^7.1.0` to `^9.0.0` ([#6035](https://github.com/MetaMask/core/pull/6035))
- Bump `@metamask/snaps-utils` from `^9.4.0` to `^11.0.0` ([#6035](https://github.com/MetaMask/core/pull/6035))

## [19.0.0]

### Changed

- **BREAKING:** Bump peer dependency `@metamask/accounts-controller` to `^31.0.0` ([#5999](https://github.com/MetaMask/core/pull/5999))
- **BREAKING:** Bump peer dependency `@metamask/network-controller` to `^24.0.0` ([#5999](https://github.com/MetaMask/core/pull/5999))

Changelog

CHANGELOG entry:

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.

@mathieuartu mathieuartu self-assigned this Jul 4, 2025
@mathieuartu mathieuartu added the team-identity-deprecated DEPRECATED: team no longer exists label Jul 4, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 4, 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.

@socket-security
Copy link
Copy Markdown

socket-security bot commented Jul 4, 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/​profile-sync-controller@​18.0.0 ⏵ 20.0.09710086 +1100 +1100

View full report

@mathieuartu
Copy link
Copy Markdown
Contributor Author

@metamaskbot update-policies

@metamaskbot
Copy link
Copy Markdown
Collaborator

Policies updated.
👀 Please review the diff for suspicious new powers.

🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [4ca89c5]
UI Startup Metrics (1207 ± 63 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1207110314446312441332
load103692212315510731117
domContentLoaded102991312255610641111
domInteractive18135781639
firstPaint61380115641010441113
backgroundConnect94718912
firstReactRender2516124122740
getState15688101930
initialActions40296616
loadScripts78665895754820866
setupStore94546820
WebpackHomeuiStartup22561725276424224052745
load16921290206419017862033
domContentLoaded16851285205318817802023
domInteractive2013112171559
firstPaint176641926188187305
backgroundConnect24115192942
firstReactRender26190416110373398
getState1654582033
initialActions153290391039
loadScripts16821283203918617782010
setupStore1773772032
FirefoxBrowserifyHomeuiStartup14031196185812914361699
load1213106114247712611349
domContentLoaded1213106114247712611349
domInteractive1133636055127253
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2913182272693
firstReactRender26216052634
getState15520327962
initialActions6010214328
loadScripts1190104814067512351323
setupStore16425635989
WebpackHomeuiStartup16501368229519917842017
load14241165192915715631675
domContentLoaded14241165192815715621675
domInteractive1053241760111255
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2717261252743
firstReactRender41345644346
getState146178191332
initialActions12023338562
loadScripts14021149171415115461649
setupStore95314820
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 16 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 20 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2256 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 261 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 15 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 2746 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2034 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2023 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 60 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 398 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 39 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2010 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 113 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 29 exceeds gate value 25 for firefox browserify home mean backgroundConnect
Benchmark value 26 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 15 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 6 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 16 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1699 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 253 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 93 exceeds gate value 70 for firefox browserify home p95 backgroundConnect
Benchmark value 62 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 28 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 89 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1650 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1425 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1424 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 106 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 28 exceeds gate value 26 for firefox webpack home mean backgroundConnect
Benchmark value 41 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 13 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1402 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2017 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1675 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1675 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 255 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 62 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1649 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 316ms | Sum of p95 exceeds: 970.8ms
Sum of all benchmark exceeds: 1286.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 916 Bytes (0.02%)
  • ui: 0 Bytes (0%)
  • common: 5.23 KiB (0.06%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [6c59f50]
UI Startup Metrics (1217 ± 54 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1217112513675412551316
load104495512145210821123
domContentLoaded103794312045210771115
domInteractive18134781742
firstPaint64798121442910681120
backgroundConnect94687823
firstReactRender24175672642
getState1565592231
initialActions40205717
loadScripts79270394951827874
setupStore95335820
WebpackHomeuiStartup22261637283424523592645
load16641232209618217491983
domContentLoaded16571226208218017441964
domInteractive1912127181465
firstPaint176631711170189322
backgroundConnect2711269262750
firstReactRender27991410117380400
getState185311301933
initialActions11227728925
loadScripts16531224207017817401952
setupStore197314302033
FirefoxBrowserifyHomeuiStartup14261255190312114881665
load1241110315338212971378
domContentLoaded1241110315338212971378
domInteractive1183744159125260
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2813170242781
firstReactRender26225452635
getState15421326950
initialActions5012414320
loadScripts1218108515168112741351
setupStore12420925838
WebpackHomeuiStartup17121357236821318152200
load14771181194417515951809
domContentLoaded14761181194317515951809
domInteractive1133438977104340
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect25177382742
firstReactRender42355444449
getState165234261239
initialActions8021529614
loadScripts14571166188417315741785
setupStore13529130928
Benchmark value 25 exceeds gate value 23 for chrome browserify home mean firstReactRender
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 42 exceeds gate value 41 for chrome browserify home p95 domInteractive
Benchmark value 23 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 17 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 20 exceeds gate value 17 for chrome browserify home p95 setupStore
Benchmark value 2227 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 279 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 12 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 2645 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 66 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 400 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 25 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 1427 exceeds gate value 1405 for firefox browserify home mean uiStartup
Benchmark value 1241 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 26 exceeds gate value 25 for firefox browserify home mean firstReactRender
Benchmark value 16 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 6 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 13 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1665 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 260 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 81 exceeds gate value 70 for firefox browserify home p95 backgroundConnect
Benchmark value 50 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 20 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 38 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1712 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1477 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1477 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 114 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 43 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 16 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 9 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1457 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2200 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1809 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1809 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 340 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 39 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 14 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1785 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 557ms | Sum of p95 exceeds: 1329.8ms
Sum of all benchmark exceeds: 1886.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 916 Bytes (0.02%)
  • ui: 0 Bytes (0%)
  • common: 5.23 KiB (0.06%)

fabiobozzo
fabiobozzo previously approved these changes Jul 4, 2025
@seaona
Copy link
Copy Markdown
Member

seaona commented Jul 7, 2025

hi there 👋 The failed test are legit. It seems that this PR broke the dapp connectivity in FF. You can reproduce this by building MM for FF ENABLE_MV3=false yarn start and then trying to connect to any dapp -- it fails with all dapps, not just our test-dapp.

Hopefully, you can debug from this point 🙏

dapp-connectivity-broken-ff.mp4

@mathieuartu mathieuartu force-pushed the chore/bump-identity-notification-controllers branch from 6c59f50 to ec38d00 Compare July 8, 2025 08:23
@mathieuartu mathieuartu changed the title chore: bump notification & identity controllers chore: bump @metamask/profile-sync-controller to ^20.0.0 Jul 8, 2025
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [fffedc5]
UI Startup Metrics (1254 ± 64 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1254112014636413031355
load107993212426311331177
domContentLoaded107292012316311251170
domInteractive18145171737
firstPaint78772121742511211176
backgroundConnect84263911
firstReactRender23166662433
getState1666792329
initialActions40648315
loadScripts82368497062876919
setupStore85243816
WebpackHomeuiStartup22901686279821323942717
load17311259210418718532052
domContentLoaded17241255209818618492045
domInteractive191382151565
firstPaint1676937775193345
backgroundConnect24105493141
firstReactRender26592421117378409
getState185283281834
initialActions112158171136
loadScripts17211253209518418472043
setupStore217326322235
FirefoxBrowserifyHomeuiStartup13801183202015714441697
load12011044167310912561397
domContentLoaded12011044167310912561397
domInteractive1193556580117316
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2412237242248
firstReactRender25215142530
getState12424125928
initialActions8020326344
loadScripts11811031153610012391375
setupStore13417223870
WebpackHomeuiStartup16881419266421417732085
load14241187186615315291711
domContentLoaded14241187186615415281710
domInteractive109585477996295
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2415119132549
firstReactRender59478266269
getState196400431536
initialActions7021323420
loadScripts14041171182815315091690
setupStore135227231032
Benchmark value 1254 exceeds gate value 1234 for chrome browserify home mean uiStartup
Benchmark value 1079 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 17 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions
Benchmark value 15 exceeds gate value 1.2 for chrome browserify home p95 initialActions
Benchmark value 2290 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1732 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1725 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 265 exceeds gate value 235 for chrome webpack home mean firstReactRender
Benchmark value 12 exceeds gate value 7 for chrome webpack home mean initialActions
Benchmark value 1721 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2717 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 2053 exceeds gate value 2030 for chrome webpack home p95 load
Benchmark value 2046 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded
Benchmark value 65 exceeds gate value 57 for chrome webpack home p95 domInteractive
Benchmark value 345 exceeds gate value 334 for chrome webpack home p95 firstPaint
Benchmark value 409 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 36 exceeds gate value 7 for chrome webpack home p95 initialActions
Benchmark value 2044 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 119 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 13 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 8 exceeds gate value 1 for firefox browserify home mean initialActions
Benchmark value 13 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 1697 exceeds gate value 1660 for firefox browserify home p95 uiStartup
Benchmark value 316 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 28 exceeds gate value 24 for firefox browserify home p95 getState
Benchmark value 44 exceeds gate value 2 for firefox browserify home p95 initialActions
Benchmark value 70 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 1689 exceeds gate value 1615 for firefox webpack home mean uiStartup
Benchmark value 1424 exceeds gate value 1380 for firefox webpack home mean load
Benchmark value 1424 exceeds gate value 1380 for firefox webpack home mean domContentLoaded
Benchmark value 110 exceeds gate value 100 for firefox webpack home mean domInteractive
Benchmark value 59 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 20 exceeds gate value 15 for firefox webpack home mean getState
Benchmark value 7 exceeds gate value 1 for firefox webpack home mean initialActions
Benchmark value 1404 exceeds gate value 1360 for firefox webpack home mean loadScripts
Benchmark value 2085 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1711 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1710 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 295 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 69 exceeds gate value 50 for firefox webpack home p95 firstReactRender
Benchmark value 36 exceeds gate value 32 for firefox webpack home p95 getState
Benchmark value 20 exceeds gate value 2 for firefox webpack home p95 initialActions
Benchmark value 1690 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Benchmark value 32 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 512ms | Sum of p95 exceeds: 1243.8ms
Sum of all benchmark exceeds: 1755.8ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 916 Bytes (0.02%)
  • ui: 0 Bytes (0%)
  • common: 961 Bytes (0.01%)

@mathieuartu mathieuartu marked this pull request as ready for review July 8, 2025 08:53
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.

Bug: Profile Sync Update Breaks Firefox Dapp Connectivity

Updating @metamask/profile-sync-controller to ^20.0.0 introduced a peer dependency mismatch. The new version requires @metamask/accounts-controller ^31.0.0, but the project still uses ^30.0.0. This incompatibility causes runtime issues, specifically broken dapp connectivity in Firefox.

package.json#L322-L323

"@metamask/preinstalled-example-snap": "^0.6.0",
"@metamask/profile-sync-controller": "^20.0.0",

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

@mathieuartu
Copy link
Copy Markdown
Contributor Author

Bug: Profile Sync Update Breaks Firefox Dapp Connectivity

Updating @metamask/profile-sync-controller to ^20.0.0 introduced a peer dependency mismatch. The new version requires @metamask/accounts-controller ^31.0.0, but the project still uses ^30.0.0. This incompatibility causes runtime issues, specifically broken dapp connectivity in Firefox.

package.json#L322-L323
Fix in CursorFix in Web

Was this report helpful? Give feedback by reacting with 👍 or 👎

Bumping @metamask/notification-services-controller to ^13.0.0 introduced the Firefox Dapp connectivity issues.
I fixed this by not bumping this controller in this PR.

@mathieuartu mathieuartu enabled auto-merge July 8, 2025 08:59
@mathieuartu mathieuartu added this pull request to the merge queue Jul 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 8, 2025
@mathieuartu mathieuartu added this pull request to the merge queue Jul 8, 2025
Merged via the queue into main with commit ab29298 Jul 8, 2025
156 checks passed
@mathieuartu mathieuartu deleted the chore/bump-identity-notification-controllers branch July 8, 2025 10:29
@github-actions github-actions bot locked and limited conversation to collaborators Jul 8, 2025
@metamaskbot metamaskbot added the release-13.0.0 Issue or pull request that will be included in release 13.0.0 (also called 12.24.0) label Jul 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.0.0 Issue or pull request that will be included in release 13.0.0 (also called 12.24.0) team-identity-deprecated DEPRECATED: team no longer exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants