Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

chore: Bumpeslint-plugin-react-hooks from^5.2.0 to^7.0.1#38376

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Draft
MajorLift wants to merge5 commits intomain
base:main
Choose a base branch
Loading
fromjongsun/chore/lint/bump-eslint-plugin-react-hooks-to-v7

Conversation

@MajorLift
Copy link
Contributor

@MajorLiftMajorLift commentedNov 28, 2025
edited by github-actionsbot
Loading

Description

  • Replace deprecated pre-releasereact-compiler/react-compiler ruleset withreact-hooks
  • Handle breaking changes from updating two major versions.
  • UseadditionalEffectHooks config field to applyreact-hooks/exhaustive-deps rule to custom hooksuseAsync{Callback,Result,ResultOrThrow}

Open in GitHub Codespaces

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.

@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.

@metamaskbotmetamaskbot added the team-extension-platformExtension Platform team labelNov 28, 2025
@socket-security
Copy link

socket-securitybot commentedNov 28, 2025
edited
Loading

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

DiffPackageSupply Chain
Security
VulnerabilityQualityMaintenanceLicense
Updatedeslint-plugin-react-hooks@​5.2.0 ⏵ 7.0.110010010096+1100

View full report

@MajorLift
Copy link
ContributorAuthor

@metamaskbot update-policies

metamaskbot reacted with thumbs up emoji

@metamaskbot
Copy link
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
Collaborator

metamaskbot commentedNov 28, 2025
edited
Loading

✨ Files requiring CODEOWNER review ✨

🧩@MetaMask/extension-devs (1 files, +57 -1)
  • 📁 lavamoat/
    • 📁 build-system/
      • 📄policy.json+57 -1

📜@MetaMask/policy-reviewers (1 files, +57 -1)
  • 📁 lavamoat/
    • 📁 build-system/
      • 📄policy.json+57 -1

Tip

Follow the policy review process outlined in theLavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers.


🔗@MetaMask/supply-chain (1 files, +57 -1)
  • 📁 lavamoat/
    • 📁 build-system/
      • 📄policy.json+57 -1

@github-actionsgithub-actionsbot added size-S and removed size-XS labelsNov 28, 2025
@MajorLiftMajorLiftforce-pushed thejongsun/chore/lint/bump-eslint-plugin-react-hooks-to-v7 branch from1c090f1 todf0041fCompareNovember 28, 2025 13:10
@metamaskbot
Copy link
Collaborator

Builds ready [df0041f]
UI Startup Metrics (1239 ± 122 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12391041157112213301458
load1038861129910511071243
domContentLoaded1032858129310311011234
domInteractive261696202282
firstPaint53393126940210021171
backgroundConnect21219126912217235
firstReactRender31216293654
getState3816118194583
initialActions1011113
loadScripts82666010871008951024
setupStore1254361325
numNetworkReqs1257820573
BrowserifyPower User HomeuiStartup20621750267422321672537
load1033908141711010241318
domContentLoaded1018901140410910151311
domInteractive36191322535116
firstPaint55913214213879761281
backgroundConnect260201761106234554
firstReactRender6042107136684
getState18913272760204230
initialActions104112
loadScripts80769711921088021103
setupStore211076112446
numNetworkReqs102662735095259
WebpackStandard HomeuiStartup808699116179826976
load62656390466649787
domContentLoaded62156089666644778
domInteractive2615125212278
firstPaint27279784191319657
backgroundConnect1057281019
firstReactRender27204563138
getState271372113547
initialActions103111
loadScripts61955788664642769
setupStore1062031216
numNetworkReqs1257720573
WebpackPower User HomeuiStartup16731367259927119572183
load6935921241112696967
domContentLoaded6835841234112692957
domInteractive38171963433135
firstPaint27696929185273687
backgroundConnect47857911618488
firstReactRender6344101106982
getState18313073159189227
initialActions104112
loadScripts6805821225110690948
setupStore221070122448
numNetworkReqs1686742784219415
FirefoxBrowserifyStandard HomeuiStartup12921040188018014051725
load1058897137711011371276
domContentLoaded1057896137711011371276
domInteractive55311312969114
firstPaint------
backgroundConnect46212564142149
firstReactRender25185482545
getState12789111020
initialActions103112
loadScripts102788113289410971217
setupStore2162494311146
numNetworkReqs1156816662
BrowserifyPower User HomeuiStartup26242051484646427454016
load1195970271225211811628
domContentLoaded1195969271125211811628
domInteractive12733627130108468
firstPaint------
backgroundConnect115341527158110314
firstReactRender6138276276199
getState28487908232365825
initialActions40438234
loadScripts1150947252020511511532
setupStore1568935184150569
numNetworkReqs100592466079239
WebpackStandard HomeuiStartup14791265186214315641792
load12251065160411113021414
domContentLoaded12251065160311113011414
domInteractive55271393176117
firstPaint------
backgroundConnect45171542545107
firstReactRender27208392737
getState146151171340
initialActions102022
loadScripts11961049153810212741388
setupStore175256301445
numNetworkReqs1257317763
WebpackPower User HomeuiStartup31722157575684334085004
load16121203341551917083018
domContentLoaded16111203341551917083018
domInteractive135311124162111499
firstPaint------
backgroundConnect175231210216163426
firstReactRender70442323170152
getState262701113218322806
initialActions40941126
loadScripts15411176322844316772829
setupStore140101133186154606
numNetworkReqs101622516179243
📊 Page Load Benchmark Results

Current Commit:df0041f |Date: 11/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.03s (±56ms) 🟡 | historical mean value: 1.05s ⬇️ (historical data)
  • domContentLoaded-> current mean value: 722ms (±53ms) 🟢 | historical mean value: 728ms ⬇️ (historical data)
  • firstContentfulPaint-> current mean value: 76ms (±10ms) 🟢 | historical mean value: 78ms ⬇️ (historical data)

📈 Detailed Results

MetricMeanStd DevMinMaxP95P99
pageLoadTime1.03s56ms1.01s1.33s1.07s1.33s
domContentLoaded722ms53ms699ms1.02s752ms1.02s
firstPaint76ms10ms60ms160ms88ms160ms
firstContentfulPaint76ms10ms60ms160ms88ms160ms
largestContentfulPaint0ms0ms0ms0ms0ms0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 1.21 KiB (0.03%)
  • ui: 11.61 KiB (0.15%)
  • common: 9.58 KiB (0.11%)

@MajorLiftMajorLiftforce-pushed thejongsun/chore/lint/bump-eslint-plugin-react-hooks-to-v7 branch fromdf0041f to402c831CompareNovember 28, 2025 21:43
@MajorLift
Copy link
ContributorAuthor

@metamaskbot update-policies

metamaskbot reacted with thumbs up emoji

@metamaskbot
Copy link
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
Collaborator

Builds ready [c5d7a56]
UI Startup Metrics (1235 ± 107 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup12351016156010712951478
load103586113609910911219
domContentLoaded102885613539710851198
domInteractive2816184272181
firstPaint5298913004179911173
backgroundConnect21519726314223245
firstReactRender301982113456
getState3815138194466
initialActions106113
loadScripts820652109794875983
setupStore1363251324
numNetworkReqs1257821575
BrowserifyPower User HomeuiStartup20331681314426521142548
load1010882178213810001359
domContentLoaded99587617361379781336
domInteractive35181502934130
firstPaint55311617873939711296
backgroundConnect270197729126232599
firstReactRender5741104146193
getState17914124823197227
initialActions104113
loadScripts79068515241377751130
setupStore20104372339
numNetworkReqs1556640877206320
WebpackStandard HomeuiStartup8196931096888481019
load63956687881669820
domContentLoaded63556186980665815
domInteractive2614109212290
firstPaint21978874154204583
backgroundConnect1144581427
firstReactRender2920239223138
getState261468113345
initialActions106111
loadScripts63255886078663805
setupStore1062131217
numNetworkReqs1257620574
WebpackPower User HomeuiStartup16451257253028019132151
load6795811359115680944
domContentLoaded6695761349115666938
domInteractive37172453630127
firstPaint28496960213263903
backgroundConnect70766316119555
firstReactRender60428986577
getState18214365669183212
initialActions103112
loadScripts6675741340113664929
setupStore21858122446
numNetworkReqs1706642483220408
FirefoxBrowserifyStandard HomeuiStartup12831079172714713631615
load1069912142410011261275
domContentLoaded1068911142310011261275
domInteractive54311522982118
firstPaint------
backgroundConnect4221165234693
firstReactRender22184142335
getState137148191023
initialActions102012
loadScripts104289713449210971250
setupStore136143151134
numNetworkReqs1156616763
BrowserifyPower User HomeuiStartup25511909411848127023801
load1140935237119211481486
domContentLoaded1139929237019211481485
domInteractive1153446599110402
firstPaint------
backgroundConnect952440569107248
firstReactRender5838144166090
getState24168958188244682
initialActions2028323
loadScripts1109916223217711141416
setupStore1597789194197649
numNetworkReqs99612565879234
WebpackStandard HomeuiStartup14821280219115815541755
load12281077158010312811423
domContentLoaded12281076158010312801423
domInteractive63272293881139
firstPaint------
backgroundConnect48212483649126
firstReactRender26206862838
getState1276291222
initialActions102122
loadScripts1203105015509812591373
setupStore1255491435
numNetworkReqs1156515754
WebpackPower User HomeuiStartup28752087545066929214520
load14691184277239814862503
domContentLoaded14681184277239814862503
domInteractive101304738690355
firstPaint------
backgroundConnect12829932146137237
firstReactRender64411822265112
getState270571051235359835
initialActions2125223
loadScripts14131151272635414572371
setupStore1059676132105396
numNetworkReqs102612526179242
📊 Page Load Benchmark Results

Current Commit:c5d7a56 |Date: 11/28/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.05s (±53ms) 🟡 | historical mean value: 1.04s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 731ms (±50ms) 🟢 | historical mean value: 724ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 77ms (±11ms) 🟢 | historical mean value: 81ms ⬇️ (historical data)

📈 Detailed Results

MetricMeanStd DevMinMaxP95P99
pageLoadTime1.05s53ms1.02s1.34s1.09s1.34s
domContentLoaded731ms50ms702ms1.00s757ms1.00s
firstPaint77ms11ms60ms168ms88ms168ms
firstContentfulPaint77ms11ms60ms168ms88ms168ms
largestContentfulPaint0ms0ms0ms0ms0ms0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 20 Bytes (0%)

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size-Steam-extension-platformExtension Platform team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MajorLift@metamaskbot

[8]ページ先頭

©2009-2025 Movatter.jp