-
Notifications
You must be signed in to change notification settings - Fork 5.3k
fix: show effective gas in swaps QuoteCard cp-13.1.0 #35038
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 tasks
✨ Files requiring CODEOWNER review ✨🔄 @MetaMask/swaps-engineers (5 files, +69 -24)
|
Builds ready [57a97b9]
UI Startup Metrics (1177 ± 62 ms)
Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 241 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 15 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 2254 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 35 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2525 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 61 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 387 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 270 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 12 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 29 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 3 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 211 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 10 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 37 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1651 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 104 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 35 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 58 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 7 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1936 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 255 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 68 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 68 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 37 exceeds gate value 32 for firefox webpack home p95 getState Benchmark value 15 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 362ms | Sum of p95 exceeds: 597.8ms Sum of all benchmark exceeds: 959.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
This reverts commit 27c98e7.
Builds ready [0619a08]
UI Startup Metrics (1168 ± 74 ms)
Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 232 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 8 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 2524 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 65 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 387 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 9 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 31 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 4 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 245 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 13 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 53 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1648 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 101 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 35 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 59 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 5 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 16 exceeds gate value 13 for firefox webpack home mean setupStore Benchmark value 1989 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 254 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 52 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 70 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 39 exceeds gate value 32 for firefox webpack home p95 getState Benchmark value 17 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 40 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 292ms | Sum of p95 exceeds: 613.8ms Sum of all benchmark exceeds: 905.8ms Bundle size diffs [🚀 Bundle size reduced!]
|
This reverts commit 0619a08.
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Builds ready [4ef064f]
UI Startup Metrics (1193 ± 81 ms)
Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 239 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 12 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 947 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 2259 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 33 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 35 exceeds gate value 32 for chrome webpack home mean setupStore Benchmark value 2588 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 380 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 279 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 14 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 282 exceeds gate value 65 for chrome webpack home p95 setupStore Benchmark value 30 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 4 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 225 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 21 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 28 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 8 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 237 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 40 exceeds gate value 32 for firefox webpack home p95 getState Benchmark value 16 exceeds gate value 2 for firefox webpack home p95 initialActions Sum of mean exceeds: 320ms | Sum of p95 exceeds: 862.8ms Sum of all benchmark exceeds: 1182.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [1e2c0c9]
UI Startup Metrics (1169 ± 67 ms)
Benchmark value 5 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 236 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 15 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 37 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2500 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 380 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 284 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 8 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 28 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 3 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 10 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 243 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 25 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 12 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 34 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 29 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 56 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 3 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 244 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 68 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 13 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 53 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 248ms | Sum of p95 exceeds: 585.8ms Sum of all benchmark exceeds: 833.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [b07a97b]
UI Startup Metrics (1175 ± 64 ms)
Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 233 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 2201 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 32 exceeds gate value 29 for chrome webpack home mean getState Benchmark value 2562 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 75 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 382 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 277 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 9 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 288 exceeds gate value 65 for chrome webpack home p95 setupStore Benchmark value 29 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 3 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 12 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 45 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1634 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 104 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 35 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 57 exceeds gate value 38 for firefox webpack home mean firstReactRender Benchmark value 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1991 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 265 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 67 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 18 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 31 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 285ms | Sum of p95 exceeds: 900.8ms Sum of all benchmark exceeds: 1185.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
SteP-n-s
approved these changes
Aug 13, 2025
@@ -145,7 +145,7 @@ export const CHAIN_ID_TO_SECURITY_API_NAME: Record< | |||
[CHAIN_IDS.AVALANCHE]: 'avalanche', | |||
[CHAIN_IDS.BSC]: 'bsc', | |||
[CHAIN_IDS.ARBITRUM]: 'arbitrum', | |||
[CHAIN_IDS.OPTIMISM]: 'optimist', | |||
[CHAIN_IDS.OPTIMISM]: 'optimism', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
infiniteflower
approved these changes
Aug 13, 2025
@SteP-n-s thank you for keeping eyes on this!! I later realized it and deleted the message 🙈 I will re-test again once it's merged 🙏 just to confirm it works! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
release-13.2.0
Issue or pull request that will be included in release 13.2.0
size-S
team-swaps-and-bridge
Swaps and Bridge team
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Changes
totalNetworkFee
instead of thetotalMaxNetworkFee
in the QuoteCard for accuracyChangelog
CHANGELOG entry:
Related issues
Fixes: #35006
Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist