-
Notifications
You must be signed in to change notification settings - Fork 5.5k
feat: trezor web connect library upgrade. #33834
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
Conversation
|
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. |
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring alerts on:
|
|
@metamaskbot update-policies |
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
✨ Files requiring CODEOWNER review ✨🧩 @MetaMask/extension-devs (4 files, +120 -16)
📜 @MetaMask/policy-reviewers (4 files, +120 -16)
Tip Follow the policy review process outlined in the LavaMoat Policy Review Process doc before expecting an approval from Policy Reviewers. 🔗 @MetaMask/supply-chain (4 files, +120 -16)
|
|
@metamaskbot update-policies |
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
All those alerts from Trezor/connect-web library upgrade and those network access libraries is required to access trezor dapp external to complete the trezor funciton. therefore we need those libraries otherwise, trezor feature will not work. @SocketSecurity ignore-all |
|
@SocketSecurity ignore-all |
|
@metamaskbot update-policies |
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
Builds ready [e0ae2b0]
UI Startup Metrics (1195 ± 64 ms)
Benchmark value 34 exceeds gate value 33 for chrome browserify home p95 getState Benchmark value 2223 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 264 exceeds gate value 235 for chrome webpack home mean firstReactRender Benchmark value 2648 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 58 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 412 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 1986 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 116 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 26 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 17 exceeds gate value 11 for firefox browserify home mean getState Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 67 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 31 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 1406 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1406 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 107 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 1386 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 16 exceeds gate value 13 for firefox webpack home mean setupStore Benchmark value 2001 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1756 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1755 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 280 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 1738 exceeds gate value 1630 for firefox webpack home p95 loadScripts Benchmark value 61 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 191ms | Sum of p95 exceeds: 823ms Sum of all benchmark exceeds: 1014ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
|
@metamaskbot update-policies |
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
Builds ready [57f8ab7]
UI Startup Metrics (1274 ± 56 ms)
Benchmark value 1096 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1088 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 24 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 18 exceeds gate value 15 for chrome browserify home mean getState Benchmark value 1377 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1182 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 1183 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 27 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 23 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 269 exceeds gate value 235 for chrome webpack home mean firstReactRender Benchmark value 2790 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 2071 exceeds gate value 2030 for chrome webpack home p95 load Benchmark value 2055 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded Benchmark value 63 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 413 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 2044 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 125 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 16 exceeds gate value 11 for firefox browserify home mean getState Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1668 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 318 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 72 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 50 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1665 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1427 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1427 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 28 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 44 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 1406 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 2140 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1763 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1762 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 307 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 51 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 1741 exceeds gate value 1630 for firefox webpack home p95 loadScripts Benchmark value 36 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 426ms | Sum of p95 exceeds: 1469ms Sum of all benchmark exceeds: 1895ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Builds ready [791d8dd]
UI Startup Metrics (1234 ± 60 ms)
Benchmark value 17 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 18 exceeds gate value 17 for chrome browserify home p95 setupStore Benchmark value 2231 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 266 exceeds gate value 235 for chrome webpack home mean firstReactRender Benchmark value 11 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 2683 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 401 exceeds gate value 370 for chrome webpack home p95 firstReactRender Benchmark value 34 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 1984 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 1412 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 118 exceeds gate value 110 for firefox browserify home mean domInteractive 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 5 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 1752 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 309 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 42 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 18 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 51 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1655 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1431 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1431 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 113 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 6 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1412 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 2034 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1687 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1686 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 303 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 25 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 1667 exceeds gate value 1630 for firefox webpack home p95 loadScripts Sum of mean exceeds: 325ms | Sum of p95 exceeds: 948.8ms Sum of all benchmark exceeds: 1273.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
791d8dd to
cab3a04
Compare
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
|
❌ test-e2e-chrome-api-specs failed. View the html report here. |
Builds ready [f79ee0c]
UI Startup Metrics (1315 ± 59 ms)
Benchmark value 1133 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1122 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 260 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 4 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 873 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1424 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1241 exceeds gate value 1190 for chrome browserify home p95 load Benchmark value 1234 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 42 exceeds gate value 41 for chrome browserify home p95 domInteractive Benchmark value 1221 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 287 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 982 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 2418 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 1919 exceeds gate value 1711 for chrome webpack home mean load Benchmark value 1908 exceeds gate value 1704 for chrome webpack home mean domContentLoaded Benchmark value 58 exceeds gate value 40 for chrome webpack home mean backgroundConnect Benchmark value 1900 exceeds gate value 1699 for chrome webpack home mean loadScripts Benchmark value 2739 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 2277 exceeds gate value 2030 for chrome webpack home p95 load Benchmark value 2208 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded Benchmark value 61 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 287 exceeds gate value 90 for chrome webpack home p95 backgroundConnect Benchmark value 235 exceeds gate value 195 for chrome webpack home p95 getState Benchmark value 13 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 2193 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 1490 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1287 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1287 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 117 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 26 exceeds gate value 25 for firefox browserify home mean backgroundConnect Benchmark value 29 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 9 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 1265 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1799 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 455 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 58 exceeds gate value 55 for firefox browserify home p95 firstReactRender Benchmark value 23 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 30 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1791 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1514 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1513 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 29 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 53 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 9 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1489 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 16 exceeds gate value 13 for firefox webpack home mean setupStore Benchmark value 2172 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1765 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1764 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 208 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 51 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 65 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 192 exceeds gate value 32 for firefox webpack home p95 getState Benchmark value 16 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 1737 exceeds gate value 1630 for firefox webpack home p95 loadScripts Sum of mean exceeds: 2202ms | Sum of p95 exceeds: 2957.8ms Sum of all benchmark exceeds: 5159.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Co-authored-by: David Murdoch <[email protected]>
…ls for console.assert and process.platform
|
@metamaskbot update-policies |
Builds ready [60288f6]
UI Startup Metrics (1327 ± 52 ms)
Benchmark value 1141 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1128 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 264 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 24 exceeds gate value 23 for chrome browserify home mean firstReactRender Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 877 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1424 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1227 exceeds gate value 1190 for chrome browserify home p95 load Benchmark value 1221 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 1214 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 359 exceeds gate value 18 for chrome browserify home p95 backgroundConnect Benchmark value 47 exceeds gate value 45 for chrome browserify home p95 firstReactRender Benchmark value 7 exceeds gate value 1.2 for chrome browserify home p95 initialActions Benchmark value 967 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 2446 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 1952 exceeds gate value 1711 for chrome webpack home mean load Benchmark value 1937 exceeds gate value 1704 for chrome webpack home mean domContentLoaded Benchmark value 50 exceeds gate value 40 for chrome webpack home mean backgroundConnect Benchmark value 8 exceeds gate value 7 for chrome webpack home mean initialActions Benchmark value 1929 exceeds gate value 1699 for chrome webpack home mean loadScripts Benchmark value 2872 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 2479 exceeds gate value 2030 for chrome webpack home p95 load Benchmark value 2277 exceeds gate value 2005 for chrome webpack home p95 domContentLoaded Benchmark value 63 exceeds gate value 57 for chrome webpack home p95 domInteractive Benchmark value 270 exceeds gate value 90 for chrome webpack home p95 backgroundConnect Benchmark value 240 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 2263 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 1459 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1259 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1259 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 111 exceeds gate value 110 for firefox browserify home mean domInteractive Benchmark value 28 exceeds gate value 25 for firefox browserify home mean firstReactRender 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 1238 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1744 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 314 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 58 exceeds gate value 55 for firefox browserify home p95 firstReactRender Benchmark value 69 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 31 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 41 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1772 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1497 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1496 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 108 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 29 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 9 exceeds gate value 1 for firefox webpack home mean initialActions Benchmark value 1473 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 19 exceeds gate value 13 for firefox webpack home mean setupStore Benchmark value 2115 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1753 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1752 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 221 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 58 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 66 exceeds gate value 50 for firefox webpack home p95 firstReactRender Benchmark value 16 exceeds gate value 2 for firefox webpack home p95 initialActions Benchmark value 1723 exceeds gate value 1630 for firefox webpack home p95 loadScripts Benchmark value 51 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 2159ms | Sum of p95 exceeds: 3095.8ms Sum of all benchmark exceeds: 5254.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
|
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
Builds ready [45021d8]
UI Startup Metrics (1296 ± 59 ms)
Benchmark value 1121 exceeds gate value 1070 for chrome browserify home mean load Benchmark value 1113 exceeds gate value 1061 for chrome browserify home mean domContentLoaded Benchmark value 252 exceeds gate value 10 for chrome browserify home mean backgroundConnect Benchmark value 3 exceeds gate value 1 for chrome browserify home mean initialActions Benchmark value 869 exceeds gate value 830 for chrome browserify home mean loadScripts Benchmark value 1396 exceeds gate value 1365 for chrome browserify home p95 uiStartup Benchmark value 1221 exceeds gate value 1190 for chrome browserify home p95 load Benchmark value 1213 exceeds gate value 1180 for chrome browserify home p95 domContentLoaded Benchmark value 1198 exceeds gate value 1180 for chrome browserify home p95 firstPaint Benchmark value 271 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 959 exceeds gate value 940 for chrome browserify home p95 loadScripts Benchmark value 2398 exceeds gate value 2192 for chrome webpack home mean uiStartup Benchmark value 1911 exceeds gate value 1711 for chrome webpack home mean load Benchmark value 1899 exceeds gate value 1704 for chrome webpack home mean domContentLoaded Benchmark value 44 exceeds gate value 40 for chrome webpack home mean backgroundConnect Benchmark value 1889 exceeds gate value 1699 for chrome webpack home mean loadScripts Benchmark value 2780 exceeds gate value 2454 for chrome webpack home p95 uiStartup Benchmark value 2299 exceeds gate value 2030 for chrome webpack home p95 load Benchmark value 2216 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 286 exceeds gate value 90 for chrome webpack home p95 backgroundConnect Benchmark value 14 exceeds gate value 7 for chrome webpack home p95 initialActions Benchmark value 2195 exceeds gate value 1970 for chrome webpack home p95 loadScripts Benchmark value 1556 exceeds gate value 1405 for firefox browserify home mean uiStartup Benchmark value 1337 exceeds gate value 1245 for firefox browserify home mean load Benchmark value 1337 exceeds gate value 1239 for firefox browserify home mean domContentLoaded Benchmark value 117 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 28 exceeds gate value 25 for firefox browserify home mean firstReactRender Benchmark value 14 exceeds gate value 11 for firefox browserify home mean getState Benchmark value 10 exceeds gate value 1 for firefox browserify home mean initialActions Benchmark value 1313 exceeds gate value 1230 for firefox browserify home mean loadScripts Benchmark value 11 exceeds gate value 9 for firefox browserify home mean setupStore Benchmark value 1822 exceeds gate value 1660 for firefox browserify home p95 uiStartup Benchmark value 1536 exceeds gate value 1495 for firefox browserify home p95 load Benchmark value 1536 exceeds gate value 1495 for firefox browserify home p95 domContentLoaded Benchmark value 320 exceeds gate value 195 for firefox browserify home p95 domInteractive Benchmark value 56 exceeds gate value 55 for firefox browserify home p95 firstReactRender Benchmark value 37 exceeds gate value 24 for firefox browserify home p95 getState Benchmark value 30 exceeds gate value 2 for firefox browserify home p95 initialActions Benchmark value 1516 exceeds gate value 1475 for firefox browserify home p95 loadScripts Benchmark value 41 exceeds gate value 27 for firefox browserify home p95 setupStore Benchmark value 1760 exceeds gate value 1615 for firefox webpack home mean uiStartup Benchmark value 1503 exceeds gate value 1380 for firefox webpack home mean load Benchmark value 1503 exceeds gate value 1380 for firefox webpack home mean domContentLoaded Benchmark value 112 exceeds gate value 100 for firefox webpack home mean domInteractive Benchmark value 29 exceeds gate value 26 for firefox webpack home mean backgroundConnect Benchmark value 53 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 1479 exceeds gate value 1360 for firefox webpack home mean loadScripts Benchmark value 14 exceeds gate value 13 for firefox webpack home mean setupStore Benchmark value 2110 exceeds gate value 1935 for firefox webpack home p95 uiStartup Benchmark value 1787 exceeds gate value 1660 for firefox webpack home p95 load Benchmark value 1787 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded Benchmark value 299 exceeds gate value 156 for firefox webpack home p95 domInteractive Benchmark value 53 exceeds gate value 49 for firefox webpack home p95 backgroundConnect Benchmark value 61 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 1757 exceeds gate value 1630 for firefox webpack home p95 loadScripts Benchmark value 33 exceeds gate value 28 for firefox webpack home p95 setupStore Sum of mean exceeds: 2242ms | Sum of p95 exceeds: 2838.8ms Sum of all benchmark exceeds: 5080.8ms Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
davidmurdoch
left a comment
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.
Note: I didn't test trezor integration, but I did make sure it compiled the new libraries
|
Tested with the physical Trezor device: account import and Send flow works as expected. |
Description
This PR will upgrade trezor/connect-web library based on trezor team suggestion. please refer to plan https://github.com/MetaMask/accounts-planning/issues/936 for details.
This PR will add a appName parameter to manifest to initialise TrezorSDK.
Related issues
Fixes: https://github.com/MetaMask/accounts-planning/issues/936
Manual testing steps
All trezor flows will need to tested to make sure nothing broken.
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist