Skip to content

feat: bump @metamask/chain-agnostic-permission to 0.6.0#32152

Merged
HowardBraham merged 39 commits intomainfrom
jl/adopt-sip-26-post-merge-core-refactor
May 21, 2025
Merged

feat: bump @metamask/chain-agnostic-permission to 0.6.0#32152
HowardBraham merged 39 commits intomainfrom
jl/adopt-sip-26-post-merge-core-refactor

Conversation

@jiexi
Copy link
Copy Markdown
Member

@jiexi jiexi commented Apr 21, 2025

Description

Moves caip-25 helpers added in sip-26 branch into the core package

Open in GitHub Codespaces

Related issues

See: MetaMask/core#5609

Manual testing steps

No user facing changes

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.

@jiexi jiexi requested a review from a team as a code owner April 21, 2025 19:11
@github-actions
Copy link
Copy Markdown
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-wallet-api-platform-deprecated DEPRECATED: please use "team-wallet-integrations" instead label Apr 21, 2025
@socket-security
Copy link
Copy Markdown

socket-security bot commented Apr 21, 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/​chain-agnostic-permission@​0.3.0 ⏵ 0.6.096 +110075 +196100

View full report

@metamaskbot
Copy link
Copy Markdown
Collaborator

metamaskbot commented Apr 21, 2025

✨ Files requiring CODEOWNER review ✨

✅ @MetaMask/confirmations

  • ui/pages/confirmations/confirmation/stories/success.stories.js

🖥️ @MetaMask/wallet-ux

  • ui/components/multichain/account-list-menu/account-list-menu.test.tsx
  • ui/components/multichain/connect-accounts-modal/connect-accounts-modal.stories.tsx
  • ui/components/multichain/connected-status/connected-status.tsx
  • ui/components/multichain/edit-accounts-modal/edit-accounts-modal.stories.tsx
  • ui/components/multichain/pages/review-permissions-page/review-permissions-page.tsx
  • ui/components/multichain/pages/send/send.test.js

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [1e6c8b6]
UI Startup Metrics (1188 ± 52 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1188109513735212231282
load102489611725110531116
domContentLoaded101888911655010491109
domInteractive16135151626
firstPaint68584114540610421112
backgroundConnect6411179
firstReactRender21155362234
getState1253171728
initialActions001001
loadScripts78466491949815878
setupStore75162812
WebpackHomeuiStartup21041705272517622282303
load16231329200712917291776
domContentLoaded16161326199612717231768
domInteractive161172111350
firstPaint206671568205232325
backgroundConnect24988142556
firstReactRender20155382116331356
getState1144361426
initialActions316145
loadScripts16121324197312717211764
setupStore216279302135
FirefoxBrowserifyHomeuiStartup1370118917149814221556
load12161059156010512811414
domContentLoaded12161058155910512811413
domInteractive1063523236127176
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect221380122052
firstReactRender24197782433
getState84687812
initialActions001001
loadScripts11961038154310712641396
setupStore64344612
WebpackHomeuiStartup15701368199113716401850
load13431185176312514121587
domContentLoaded13421185176212514121587
domInteractive83561512290138
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2314125122440
firstReactRender36305753945
getState95325920
initialActions102111
loadScripts13211170167612113931564
setupStore1056310943
Benchmark value 207 exceeds gate value 199 for chrome webpack home mean firstPaint
Benchmark value 43 exceeds gate value 28 for firefox webpack home p95 setupStore
Sum of mean exceeds: 8ms | Sum of p95 exceeds: 15ms
Sum of all benchmark exceeds: 23ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: -679 Bytes (-0.01%)
  • common: 1.18 KiB (0.01%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [e1efe73]
UI Startup Metrics (1215 ± 67 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1215110214056712561351
load105495512355910881197
domContentLoaded104893412215910801191
domInteractive17135861627
firstPaint67481122741910551124
backgroundConnect74334813
firstReactRender20164852132
getState1453382027
initialActions001001
loadScripts80970096056841938
setupStore75162813
WebpackHomeuiStartup20881695250216222022308
load16071259190411817091756
domContentLoaded16001255188411717051747
domInteractive15115491341
firstPaint1796434462217292
backgroundConnect239101152657
firstReactRender18954417119331362
getState134249241422
initialActions316135
loadScripts15961253186211817031745
setupStore206271301930
FirefoxBrowserifyHomeuiStartup13961228172710514501616
load12431048156111113051459
domContentLoaded12421047156011113041458
domInteractive1153836043130196
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2414135182345
firstReactRender23205242429
getState85606811
initialActions001001
loadScripts12211031153911112891440
setupStore7434468
WebpackHomeuiStartup15311327215414915871947
load13081144194713613541664
domContentLoaded13071144194613613541663
domInteractive79381662282139
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2414111132541
firstReactRender36295863950
getState1053061030
initialActions101111
loadScripts12861122187713313321636
setupStore95477926
cc: @HowardBraham
Benchmark value 1197 exceeds gate value 1190 for chrome browserify home p95 load
Benchmark value 1191 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded
Benchmark value 1243 exceeds gate value 1239 for firefox browserify home mean domContentLoaded
Benchmark value 115 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 196 exceeds gate value 195 for firefox browserify home p95 domInteractive
Benchmark value 1947 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1664 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1663 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 1636 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 9ms | Sum of p95 exceeds: 44ms
Sum of all benchmark exceeds: 53ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: 1.54 KiB (0.02%)
  • common: 1.18 KiB (0.01%)

@jiexi
Copy link
Copy Markdown
Member Author

jiexi commented Apr 22, 2025

TODO: use account comparison helper after the following is merged

#32169 (review)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [79a0d1d]
UI Startup Metrics (1204 ± 59 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1204110314035912371328
load103994512005210641143
domContentLoaded103494211735110591138
domInteractive17136061730
firstPaint790159119738810601134
backgroundConnect74272710
firstReactRender19153632027
getState1444191930
initialActions001001
loadScripts79770793349819901
setupStore74162812
WebpackHomeuiStartup22281835261016223422481
load17191407219014218111992
domContentLoaded17111403218514018061960
domInteractive16115791444
firstPaint1796536259208289
backgroundConnect29895183674
firstReactRender22155405117333373
getState164199211732
initialActions317146
loadScripts17061401218313918041935
setupStore247301332644
FirefoxBrowserifyHomeuiStartup13431150170610214191512
load12001003154010612731368
domContentLoaded11991003154010612731368
domInteractive973719528104142
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2012102111933
firstReactRender22185142328
getState74273810
initialActions002001
loadScripts1181989152310612571350
setupStore6453567
WebpackHomeuiStartup14711275191811215421689
load1261111315369513211464
domContentLoaded1261111215359513211464
domInteractive82462572893134
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect19145972131
firstReactRender33285443543
getState10419419828
initialActions002111
loadScripts1244109915209413051450
setupStore84284814
cc: @HowardBraham
Benchmark value 2228 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1719 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1712 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 1707 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2481 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 373 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Sum of mean exceeds: 60ms | Sum of p95 exceeds: 30ms
Sum of all benchmark exceeds: 90ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: 1.54 KiB (0.02%)
  • common: 1.18 KiB (0.01%)

@jiexi jiexi changed the title feat: bump @metamask/chain-agnostic-permission to 0.5.0 feat: bump @metamask/chain-agnostic-permission to 0.6.0 May 7, 2025
vinnyhoward
vinnyhoward previously approved these changes May 15, 2025
Copy link
Copy Markdown
Contributor

@vinnyhoward vinnyhoward left a comment

Choose a reason for hiding this comment

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

Nice refactor LGTM

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [8668abf]
UI Startup Metrics (1204 ± 66 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1204109014166612431334
load104393212536210681163
domContentLoaded103692812476210631158
domInteractive16132531624
firstPaint774133125439010541157
backgroundConnect84234818
firstReactRender19163332027
getState15595112030
initialActions001001
loadScripts800704100662822925
setupStore75162810
WebpackHomeuiStartup21641672251317122812384
load16811310196313117731879
domContentLoaded16741307194013117691875
domInteractive161165121356
firstPaint1576132257184295
backgroundConnect329237343863
firstReactRender1454333999253321
getState1344671726
initialActions317136
loadScripts16691305191613217671869
setupStore187204202034
FirefoxBrowserifyHomeuiStartup13431154189313913811626
load11851024157711112471411
domContentLoaded11851024157711112471411
domInteractive1044229037116163
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect20138492028
firstReactRender23195162340
getState184248468195
initialActions0029301
loadScripts11671008155111012271394
setupStore74647611
WebpackHomeuiStartup15991377215714916901890
load13651180190313514481605
domContentLoaded13641179190213514471605
domInteractive84373823691128
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21155672240
firstReactRender35305243742
getState13523731915
initialActions102111
loadScripts13461159188513514301583
setupStore1057811828
Benchmark value 19 exceeds gate value 11 for firefox browserify home mean getState
Benchmark value 195 exceeds gate value 24 for firefox browserify home p95 getState
Sum of mean exceeds: 8ms | Sum of p95 exceeds: 171ms
Sum of all benchmark exceeds: 179ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: -637 Bytes (-0.01%)
  • common: 1.21 KiB (0.01%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [a31d3c1]
UI Startup Metrics (1224 ± 56 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1224112614095612661327
load106198012005311031165
domContentLoaded105496111915410961158
domInteractive16143131625
firstPaint74278116541810821145
backgroundConnect84285822
firstReactRender20164152036
getState1656392231
initialActions001001
loadScripts81072394552846905
setupStore85192810
WebpackHomeuiStartup21841757254117522992445
load16941340209415217761966
domContentLoaded16861337209114917681935
domInteractive161160101446
firstPaint1646567878176297
backgroundConnect319196253658
firstReactRender17544428111288335
getState1445491833
initialActions318145
loadScripts16801335206814817651912
setupStore246278352348
FirefoxBrowserifyHomeuiStartup13411174189212213901573
load11911027177711712451427
domContentLoaded11911026177711712441427
domInteractive1043727338121170
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21137182140
firstReactRender24196382251
getState941941989
initialActions001001
loadScripts11731009175111612231406
setupStore73657616
WebpackHomeuiStartup15451347194112716141814
load13231167168111414011549
domContentLoaded13221167168111414011548
domInteractive84372643292127
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21144562338
firstReactRender34284443642
getState94457928
initialActions102111
loadScripts13041152166211413831532
setupStore95748924
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 22 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Sum of mean exceeds: 1ms | Sum of p95 exceeds: 4ms
Sum of all benchmark exceeds: 5ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: -637 Bytes (-0.01%)
  • common: 1.21 KiB (0.01%)

adonesky1
adonesky1 previously approved these changes May 20, 2025
Copy link
Copy Markdown
Contributor

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

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

LGTM!

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [c9937c4]
UI Startup Metrics (1203 ± 65 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1203107414576512261317
load103892513016110671142
domContentLoaded103092112986310601137
domInteractive16133641624
firstPaint776127116138510451132
backgroundConnect84295824
firstReactRender19153332023
getState155104112027
initialActions003001
loadScripts791689102660815897
setupStore85202811
WebpackHomeuiStartup21651752270121623142564
load16871371208915017861959
domContentLoaded16801366207014817761949
domInteractive1812235241348
firstPaint1696648563208284
backgroundConnect231061102845
firstReactRender15343371111295356
getState1244771426
initialActions612812846
loadScripts16771365205814617741937
setupStore3873037424282
FirefoxBrowserifyHomeuiStartup13171150187314013611590
load11641019160411512131404
domContentLoaded11631019160311512121403
domInteractive963845649104151
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2412190252083
firstReactRender24206662331
getState84365814
initialActions001001
loadScripts11421006152010911921375
setupStore11421228637
WebpackHomeuiStartup15031292195814615711856
load13001144175012713741540
domContentLoaded12991144175012713741539
domInteractive76591461680124
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect21136072333
firstReactRender37274944145
getState84325919
initialActions001011
loadScripts12811124172912613591520
setupStore14522935823
Benchmark value 16 exceeds gate value 15 for chrome browserify home mean getState
Benchmark value 24 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 39 exceeds gate value 32 for chrome webpack home mean setupStore
Benchmark value 2564 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 282 exceeds gate value 65 for chrome webpack home p95 setupStore
Benchmark value 12 exceeds gate value 9 for firefox browserify home mean setupStore
Benchmark value 83 exceeds gate value 70 for firefox browserify home p95 backgroundConnect
Benchmark value 37 exceeds gate value 27 for firefox browserify home p95 setupStore
Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore
Sum of mean exceeds: 12ms | Sum of p95 exceeds: 356ms
Sum of all benchmark exceeds: 368ms

Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: -530 Bytes (-0.01%)
  • ui: -651 Bytes (-0.01%)
  • common: 1.21 KiB (0.01%)

@adonesky1 adonesky1 added this pull request to the merge queue May 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 21, 2025
@adonesky1 adonesky1 added this pull request to the merge queue May 21, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 21, 2025
@HowardBraham HowardBraham added this pull request to the merge queue May 21, 2025
Merged via the queue into main with commit d48277a May 21, 2025
157 checks passed
@HowardBraham HowardBraham deleted the jl/adopt-sip-26-post-merge-core-refactor branch May 21, 2025 18:44
@github-actions github-actions bot locked and limited conversation to collaborators May 21, 2025
@metamaskbot metamaskbot added the release-12.20.0 Issue or pull request that will be included in release 12.20.0 label May 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-12.20.0 Issue or pull request that will be included in release 12.20.0 team-wallet-api-platform-deprecated DEPRECATED: please use "team-wallet-integrations" instead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants