Skip to content

Conversation

@n3ps
Copy link
Contributor

@n3ps n3ps commented Nov 13, 2025

Description

Fix the edit-account-modal circular dependency

Open in GitHub Codespaces

Changelog

CHANGELOG entry: null

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.

Note

Removes a circular dependency by updating AccountListItem import in edit-accounts-modal and reflecting the change in development/circular-deps.jsonc.

  • Circular dependencies:
    • Removed cycle involving ui/components/multichain/edit-accounts-modal/* from development/circular-deps.jsonc.
  • Multichain UI:
    • ui/components/multichain/edit-accounts-modal/edit-accounts-modal.tsx: Update AccountListItem import from .. to ../account-list-item to break the cycle.

Written by Cursor Bugbot for commit 77c4c44. This will update automatically on new commits. Configure here.

@n3ps n3ps added the team-core-extension-ux Core Extension UX team label Nov 13, 2025
@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
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

👨‍🔧 @MetaMask/core-extension-ux (1 files, +1 -1)
  • 📁 ui/
    • 📁 components/
      • 📁 multichain/
        • 📁 edit-accounts-modal/
          • 📄 edit-accounts-modal.tsx +1 -1

🔒 @MetaMask/extension-security-team (1 files, +0 -5)
  • 📁 development/
    • 📄 circular-deps.jsonc +0 -5

👨‍🔧 @dbrans (1 files, +0 -5)
  • 📁 development/
    • 📄 circular-deps.jsonc +0 -5

👨‍🔧 @HowardBraham (1 files, +0 -5)
  • 📁 development/
    • 📄 circular-deps.jsonc +0 -5

@metamaskbot
Copy link
Collaborator

Builds ready [77c4c44]
UI Startup Metrics (1221 ± 84 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyStandard HomeuiStartup1221107214758412891358
load105693212077811161189
domContentLoaded105092711987711111182
domInteractive221486161767
firstPaint626145121542110641187
backgroundConnect2312162648234248
firstReactRender2418120112531
getState2076392436
initialActions001001
loadScripts82569796876893946
setupStore1072421114
numNetworkReqs1367519671
BrowserifyPower User HomeuiStartup17581517298235919632982
load1082930164521013001645
domContentLoaded1074926161720612931617
domInteractive471617650101176
firstPaint70011816704359821670
backgroundConnect24322330520248305
firstReactRender45406454664
getState18516024022205240
initialActions001001
loadScripts841703134919810711349
setupStore15103082630
numNetworkReqs1417235484210354
WebpackStandard HomeuiStartup84372112351048351170
load623568102396612920
domContentLoaded616564100392606890
domInteractive171256111548
firstPaint208621010210206876
backgroundConnect261372142763
firstReactRender29175683538
getState1255051318
initialActions001001
loadScripts61356299390604880
setupStore1365171328
numNetworkReqs1367719871
WebpackPower User HomeuiStartup1006836164326411381643
load596527867101656867
domContentLoaded58152082589641825
domInteractive281390266290
firstPaint36756868287645868
backgroundConnect581019970131199
firstReactRender42374734447
getState1246017430142174
initialActions001011
loadScripts57851881786634817
setupStore1462692526
numNetworkReqs1207223155167231
FirefoxBrowserifyStandard HomeuiStartup14361262186611614591700
load1216109014868412481404
domContentLoaded1216109014858412481404
domInteractive993332447104220
firstPaint------
backgroundConnect4226131185078
firstReactRender26215262642
getState1174451117
initialActions001001
loadScripts1191107214347712281364
setupStore147207211232
numNetworkReqs1267016758
BrowserifyPower User HomeuiStartup22111806286231225642862
load13741168167215315181672
domContentLoaded13731167167115315171671
domInteractive19564536181491536
firstPaint------
backgroundConnect1122332281154322
firstReactRender645192127492
getState1209417824136178
initialActions101111
loadScripts13421144163815114951638
setupStore3751424157142
numNetworkReqs1467223560203235
WebpackStandard HomeuiStartup16161442222916016421992
load13761222168510914191655
domContentLoaded13761222168510914191654
domInteractive983139655102188
firstPaint------
backgroundConnect51252413451135
firstReactRender3325101133277
getState1285871120
initialActions002001
loadScripts1348120416189813921585
setupStore167109181268
numNetworkReqs1366917766
WebpackPower User HomeuiStartup24162002352847327343528
load15551301223825617662238
domContentLoaded15551301223825617652238
domInteractive19139801204265801
firstPaint------
backgroundConnect1113433091164330
firstReactRender74531131586113
getState1477028156188281
initialActions106116
loadScripts15211283220325217102203
setupStore3071032850103
numNetworkReqs14557334103250334
📊 Page Load Benchmark Results

Current Commit: 77c4c44 | Date: 11/13/2025

📄 Localhost MetaMask Test Dapp

Samples: 100

Summary

  • pageLoadTime-> current mean value: 1.06s (±43ms) 🟡 | historical mean value: 1.03s ⬆️ (historical data)
  • domContentLoaded-> current mean value: 741ms (±61ms) 🟢 | historical mean value: 721ms ⬆️ (historical data)
  • firstContentfulPaint-> current mean value: 91ms (±128ms) 🟢 | historical mean value: 77ms ⬆️ (historical data)

📈 Detailed Results

Metric Mean Std Dev Min Max P95 P99
pageLoadTime 1.06s 43ms 1.03s 1.41s 1.09s 1.41s
domContentLoaded 741ms 61ms 714ms 1.30s 755ms 1.30s
firstPaint 91ms 128ms 64ms 1.37s 88ms 1.37s
firstContentfulPaint 91ms 128ms 64ms 1.37s 88ms 1.37s
largestContentfulPaint 0ms 0ms 0ms 0ms 0ms 0ms
Bundle size diffs [🚨 Warning! Bundle size has increased!]
  • background: 58 Bytes (0%)
  • ui: 36 Bytes (0%)
  • common: 20 Bytes (0%)

Copy link
Contributor

@HowardBraham HowardBraham left a comment

Choose a reason for hiding this comment

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

Yay!

@n3ps n3ps marked this pull request as ready for review November 13, 2025 15:58
@n3ps n3ps requested review from a team and dbrans as code owners November 13, 2025 15:58
@n3ps n3ps enabled auto-merge November 13, 2025 15:59
Copy link
Contributor

@MajorLift MajorLift left a comment

Choose a reason for hiding this comment

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

Nice

@n3ps n3ps added this pull request to the merge queue Nov 14, 2025
Merged via the queue into main with commit 1f1806a Nov 14, 2025
178 checks passed
@n3ps n3ps deleted the n3ps/fix-edit-account branch November 14, 2025 16:26
@github-actions github-actions bot locked and limited conversation to collaborators Nov 14, 2025
@metamaskbot metamaskbot added the release-13.11.0 Issue or pull request that will be included in release 13.11.0 label Nov 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-13.11.0 Issue or pull request that will be included in release 13.11.0 size-XS team-core-extension-ux Core Extension UX team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants