chore: update Trezor Connect to v9.4.0, remove workarounds #27112

vthomas13 · 2024-09-12T18:14:26Z

This PR includes changes from PR #26749 by @martykan.

Description

With MV3, MetaMask started to use Trezor Connect inside an offscreen environment, in a way that was previously unsupported and required a workaround by patching the Trezor Connect library.

In recent versions of Trezor Connect, the library can handle working in an offscreen environment correctly, without the need for the workaround, which could cause issues with compatibility.

This PR removes the patch and updates the Trezor Connect library to the latest version (v9.4.0).
The change in manifest.json is due to new URL parameters, Firefox needs the asterisk at the end to match the URL with them.
I haven't removed the WebUSB device request which was added on MetaMask's side in relation to the workaround, since it can improve UX of the pairing process, but it could be removed if desired.

The dependency update affects Lavamoat, I am including the policy changes in my commit, however let me know if you would like me to remove them and handle them using your own process.

Related issues

Fixes:

Manual testing steps

Open accounts dropdown, "Add hardware wallet"

Select Trezor

Follow prompts to connect the device

See a list of accounts from the Trezor

Screenshots/Recordings

Before

After

Pre-merge author checklist

[x ] I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
[x ] I've completed the PR template to the best of my ability
I’ve included tests if applicable
I’ve documented my code using JSDoc format if applicable
I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

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 · 2024-09-12T18:14:38Z

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.

socket-security · 2024-09-12T18:14:39Z

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@babel/generator@7.25.5	None	`0`	489 kB	nicolo-ribaudo
npm/@babel/helper-annotate-as-pure@7.24.7	None	`0`	52.4 kB	nicolo-ribaudo
npm/@babel/helper-create-class-features-plugin@7.25.4	None	`+1`	1.23 MB	nicolo-ribaudo
npm/@babel/helper-module-imports@7.24.7	None	`0`	63.7 kB	nicolo-ribaudo
npm/@babel/helper-module-transforms@7.25.2	None	`0`	160 kB	nicolo-ribaudo
npm/@babel/helper-plugin-utils@7.24.8	None	`0`	114 kB	nicolo-ribaudo
npm/@babel/helper-replace-supers@7.25.0	None	`0`	106 kB	nicolo-ribaudo
npm/@babel/helper-simple-access@7.24.7	None	`0`	14.1 kB	nicolo-ribaudo
npm/@babel/helper-skip-transparent-expression-wrappers@7.24.7	None	`0`	58.5 kB	nicolo-ribaudo
npm/@babel/helper-validator-option@7.24.8	None	`0`	11.8 kB	nicolo-ribaudo
npm/@babel/parser@7.25.4	None	`0`	1.89 MB	nicolo-ribaudo
npm/@babel/plugin-syntax-jsx@7.24.7	None	`0`	70 kB	nicolo-ribaudo
npm/@babel/plugin-syntax-typescript@7.25.4	None	`0`	69.7 kB	existentialism, hzoo, jlhwung, ...1 more
npm/@babel/plugin-transform-modules-commonjs@7.24.8	None	`0`	107 kB	nicolo-ribaudo
npm/@babel/plugin-transform-typescript@7.25.2	None	`0`	200 kB	nicolo-ribaudo
npm/@babel/preset-typescript@7.24.7	None	`0`	90.8 kB	nicolo-ribaudo
npm/@babel/runtime@7.25.4	None	`0`	248 kB	nicolo-ribaudo
npm/@babel/template@7.25.0	None	`0`	70.4 kB	nicolo-ribaudo
npm/@babel/types@7.25.4	environment	`0`	2.48 MB	nicolo-ribaudo
npm/@emurgo/cardano-serialization-lib-browser@11.5.0	eval	`0`	3.35 MB	lisicky_emurgo
npm/@emurgo/cardano-serialization-lib-nodejs@11.5.0	eval, filesystem	`0`	3.35 MB	lisicky_emurgo
npm/@fivebinaries/coin-selection@2.2.1	None	`0`	149 kB	slowbackspace
npm/@mobily/ts-belt@3.13.1	None	`0`	381 kB	mobily
npm/@sinclair/typebox@0.31.28	None	`0`	536 kB	sinclair
npm/@solana/buffer-layout@4.0.1	None	`0`	197 kB	steveluscher
npm/@solana/web3.js@1.95.3	network	`+1`	11 MB	lorisleiva
npm/@trezor/analytics@1.2.0	network	`0`	14 kB	trezor-ci
npm/@trezor/blockchain-link-types@1.2.0	None	`0`	56.3 kB	trezor-ci
npm/@trezor/blockchain-link-utils@1.2.0	network	`0`	58.4 kB	trezor-ci
npm/@trezor/blockchain-link@2.3.0	Transitive: environment, network	`+2`	378 kB	trezor-ci
npm/@trezor/connect-analytics@1.2.0	environment	`0`	5.41 kB	trezor-ci
npm/@trezor/connect-common@0.2.0	None	`0`	180 kB	trezor-ci
npm/@trezor/connect-web@9.4.0	None	`0`	93.8 kB	trezor-ci
npm/@trezor/connect@9.4.0	filesystem Transitive: network	`+5`	2.23 MB	trezor-ci
npm/@trezor/env-utils@1.2.0	environment	`0`	17.7 kB	trezor-ci
npm/@trezor/protobuf@1.2.0	None	`0`	1.09 MB	trezor-ci
npm/@trezor/protocol@1.2.0	None	`0`	11.8 kB	trezor-ci
npm/@trezor/schema-utils@1.2.0	None	`0`	38.7 kB	trezor-ci
npm/@trezor/transport@1.3.0	network	`0`	165 kB	trezor-ci
npm/@trezor/type-utils@1.1.0	None	`0`	2.89 kB	trezor-ci
npm/@trezor/utils@9.2.0	None	`0`	64.2 kB	trezor-ci
npm/@trezor/utxo-lib@2.2.0	None	`0`	234 kB	trezor-ci
npm/@types/w3c-web-usb@1.0.10	None	`0`	8.92 kB	types
npm/@types/web@0.0.138	None	`0`	1.36 MB	types
npm/@types/ws@7.4.7	None	`0`	18.9 kB	types
npm/agentkeepalive@4.5.0	network	`0`	43.7 kB	fengmk2
npm/bchaddrjs@0.5.2	None	`+1`	1.57 MB	ealmansi
npm/bigint-buffer@1.1.5	None	`0`	55.7 kB	no2chem
npm/bip66@1.1.5	None	`0`	7.79 kB	dcousens
npm/bitcoin-ops@1.4.1	None	`0`	4.38 kB	dcousens
npm/blake-hash@2.0.0	None	`+1`	830 kB	fanatid
npm/borsh@0.7.0	None	`0`	34.6 kB	volovyk-s
npm/cashaddrjs@0.4.4	None	`+1`	274 kB	ealmansi
npm/delay@5.0.0	None	`0`	11.2 kB	sindresorhus
npm/es6-promise@4.2.8	None	`0`	315 kB	stefanpenner
npm/es6-promisify@5.0.0	None	`0`	7.76 kB	digitaldesignlabs
npm/eyes@0.1.8	None	`0`	14 kB	indexzero
npm/fast-stable-stringify@1.0.0	None	`0`	125 kB	nickyout
npm/humanize-ms@1.2.1	None	`0`	3.66 kB	dead_horse
npm/int64-buffer@1.0.1	None	`0`	22.2 kB	kawanet
npm/jayson@4.1.1	network	`+1`	922 kB	tedeh
npm/jsonschema@1.2.2	None	`0`	75.6 kB	tdegrunt
npm/long@4.0.0	None	`0`	177 kB	dcode
npm/nan@2.15.0	None	`0`	422 kB	kkoopa
npm/pushdata-bitcoin@1.0.1	None	`0`	4.28 kB	dcousens
npm/ripple-address-codec@4.2.3	None	`+1`	56.7 kB	jst5000
npm/ripple-binary-codec@1.3.0	None	`+1`	1.18 MB	jst5000
npm/ripple-keypairs@1.1.3	None	`0`	24.4 kB	jst5000
npm/ripple-lib-transactionparser@0.8.2	None	`0`	147 kB	intelliot
npm/ripple-lib@1.10.1	network	`+3`	5.09 MB	intelliot
npm/superstruct@2.0.2	None	`0`	182 kB	artmllr
npm/text-encoding-utf-8@1.0.2	None	`0`	79.4 kB	arv
npm/tiny-secp256k1@1.1.6	None	`0`	1.1 MB	junderw
npm/typeforce@1.18.0	None	`0`	19.1 kB	dcousens
npm/ua-parser-js@1.0.37	None	`0`	112 kB	faisalman
npm/usb@2.12.0	None	`+1`	7.3 MB	thegecko
npm/varuint-bitcoin@1.1.2	None	`0`	5.49 kB	junderw
npm/wif@4.0.0	None	`0`	4.67 kB	junderw

🚮 Removed packages: npm/@babel/generator@7.24.10, npm/@babel/helper-annotate-as-pure@7.22.5, npm/@babel/helper-create-class-features-plugin@7.24.5, npm/@babel/helper-module-imports@7.24.3, npm/@babel/helper-module-transforms@7.24.5, npm/@babel/helper-plugin-utils@7.24.5, npm/@babel/helper-skip-transparent-expression-wrappers@7.22.5, npm/@babel/helper-validator-option@7.23.5, npm/@babel/parser@7.24.8, npm/@babel/plugin-syntax-jsx@7.24.1, npm/@babel/plugin-syntax-typescript@7.24.1, npm/@babel/plugin-transform-modules-commonjs@7.24.1, npm/@babel/plugin-transform-typescript@7.24.5, npm/@babel/preset-typescript@7.24.1, npm/@babel/runtime@7.24.8, npm/@babel/traverse@7.24.8, npm/@babel/types@7.24.9

View full report↗︎

socket-security · 2024-09-12T18:14:40Z

👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring: npm/tiny-secp256k1@1.1.6, npm/varuint-bitcoin@1.1.2, npm/bs58@5.0.0, npm/bs58check@3.0.1, npm/@solana/buffer-layout@4.0.1, npm/agentkeepalive@4.5.0, npm/bigint-buffer@1.1.5, npm/ripple-lib@1.10.1, npm/blake-hash@2.0.0, npm/wif@4.0.0, npm/node-addon-api@7.1.0, npm/usb@2.12.0, npm/jayson@4.1.1, npm/@solana/web3.js@1.95.3, npm/@trezor/transport@1.3.0, npm/@trezor/blockchain-link-utils@1.2.0, npm/@trezor/analytics@1.2.0

View full report↗︎

Next steps

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/foo@1.0.0 or ignore all packages with @SocketSecurity ignore-all

metamaskbot · 2024-09-12T19:22:17Z

Builds ready [e3aa795]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1516 ± 98 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	225	2294	1391	419	201
		domContentLoaded	1356	2191	1497	189	91
		load	1364	2283	1516	204	98
		domInteractive	19	57	31	13	6

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

metamaskbot · 2024-09-12T22:18:29Z

Builds ready [34f33b3]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1805 ± 132 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	225	2737	1727	446	214
		domContentLoaded	1396	2681	1776	278	133
		load	1448	2690	1805	275	132
		domInteractive	12	75	28	14	7

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

codecov · 2024-09-12T22:18:48Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 69.95%. Comparing base (9459903) to head (9ce641e).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #27112   +/-   ##
========================================
  Coverage    69.95%   69.95%           
========================================
  Files         1441     1441           
  Lines        50099    50099           
  Branches     14007    14007           
========================================
  Hits         35046    35046           
  Misses       15053    15053

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

danjm · 2024-09-13T16:41:02Z

@metamaskbot update-policies

metamaskbot · 2024-09-13T16:50:11Z

Policies updated

vthomas13 · 2024-09-13T20:13:32Z

@metamaskbot update-policies

metamaskbot · 2024-09-13T20:23:01Z

No policy changes

metamaskbot · 2024-09-13T20:42:43Z

Builds ready [a840f33]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1727 ± 72 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	222	2224	1521	553	265
		domContentLoaded	1510	2215	1715	153	73
		load	1523	2224	1727	151	72
		domInteractive	13	102	35	19	9

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

sonarcloud · 2024-09-16T13:32:07Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

metamaskbot · 2024-09-16T14:28:42Z

Builds ready [9ce641e]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1751 ± 154 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	209	2426	1344	659	316
		domContentLoaded	1434	2459	1736	317	152
		load	1443	2466	1751	320	154
		domInteractive	19	98	38	20	10

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

danjm · 2024-09-19T09:12:17Z

@metamaskbot update-policies

metamaskbot · 2024-09-19T09:23:57Z

No policy changes

metamaskbot · 2024-09-19T10:02:23Z

Builds ready [8094f79]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1844 ± 180 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1519	3255	1841	369	177
		domContentLoaded	1512	3115	1812	346	166
		load	1520	3277	1844	375	180
		domInteractive	13	187	47	37	18

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

vthomas13 · 2024-10-04T23:48:32Z

@SocketSecurity ignore npm/wif@4.0.0 npm/usb@2.12.0 npm/jayson@4.1.1 npm/@solana/web3.js@1.95.3 npm/@trezor/transport@1.3.0

npm/wif@4.0.0: Verified the new author as a member of other open source projects
npm/usb@2.12.0, npm/jayson@4.1.1, npm/@solana/web3.js@1.95.3, npm/@trezor/transport@1.3.0: These are not used in trezor/connect-web, which is what we import.

vthomas13 · 2024-10-05T00:01:19Z

@SocketSecurity ignore npm/@trezor/blockchain-link-utils@1.2.0 npm/@trezor/analytics@1.2.0
npm/@trezor/blockchain-link-utils@1.2.0, npm/@trezor/analytics@1.2.0: These are not used in trezor/connect-web, which is what we import.

metamaskbot · 2024-10-07T15:59:54Z

Builds ready [ba14308]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1991 ± 112 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	309	2616	1913	434	208
		domContentLoaded	1651	2605	1963	235	113
		load	1699	2630	1991	233	112
		domInteractive	28	194	64	49	23

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

metamaskbot · 2024-10-09T14:45:57Z

Builds ready [fd765ae]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1871 ± 95 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1636	2300	1870	200	96
		domContentLoaded	1621	2291	1843	189	91
		load	1629	2299	1871	198	95
		domInteractive	24	76	44	15	7

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

vthomas13 · 2024-10-11T14:31:33Z

@SocketSecurity ignore npm/bs58@5.0.0 npm/bs58check@3.0.1 npm/node-addon-api@7.1.0
The new authors are members of other related open source projects

sonarcloud · 2024-10-11T14:58:37Z

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

metamaskbot · 2024-10-11T15:04:57Z

Builds ready [ebc4ff9]

builds: chrome, firefox
builds (beta): chrome
builds (flask): chrome, firefox
builds (MMI): chrome, firefox
builds (test): chrome, firefox
builds (test-flask): chrome, firefox
build viz: Build System
mv3: Background Module Init Stats
mv3: UI Init Stats
mv3: Module Load Stats
mv3: Bundle Size Stats
mv2: E2e Actions Stats
code coverage: Report
storybook: Storybook
typescript migration: Dashboard
all artifacts
bundle viz:
- background: 0, 1, 2, 3, 4, 5, 6
- common: 0, 1, 10, 11, 2, 3, 4, 5, 6, 7, 8, 9
- content-script: 0
- offscreen: 0
- ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1869 ± 121 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1510	2393	1862	253	122
		domContentLoaded	1503	2384	1831	252	121
		load	1511	2395	1869	252	121
		domInteractive	24	253	61	56	27

Bundle size diffs [🚨 Warning! Bundle size has increased!]

background: 0 Bytes (0.00%)
ui: 0 Bytes (0.00%)
common: 5.55 KiB (0.07%)

vthomas13 requested review from a team as code owners September 12, 2024 18:14

github-actions bot added the team-extension-platform label Sep 12, 2024

vthomas13 force-pushed the marty-trezor-update branch from fe14360 to 922368c Compare September 12, 2024 18:37

vthomas13 force-pushed the marty-trezor-update branch from d5c86db to 0676b39 Compare September 12, 2024 20:49

vthomas13 requested a review from danjm September 12, 2024 21:52

vthomas13 force-pushed the marty-trezor-update branch 3 times, most recently from 8804fcc to 60175c2 Compare September 13, 2024 16:26

vthomas13 force-pushed the marty-trezor-update branch from c8f5607 to a840f33 Compare September 13, 2024 20:12

vthomas13 force-pushed the marty-trezor-update branch from a840f33 to 9ce641e Compare September 16, 2024 13:30

legobeat mentioned this pull request Sep 17, 2024

chore: update Trezor Connect to v9.4.0, remove workarounds #26749

Closed

7 tasks

danjm force-pushed the marty-trezor-update branch from 9ce641e to c1350dd Compare September 19, 2024 09:09

vthomas13 force-pushed the marty-trezor-update branch from 8094f79 to 6448cf7 Compare September 24, 2024 14:34

vthomas13 force-pushed the marty-trezor-update branch 2 times, most recently from 145a9aa to ba14308 Compare October 7, 2024 14:48

vthomas13 requested a review from a team October 9, 2024 14:13

vthomas13 force-pushed the marty-trezor-update branch from ba14308 to fd765ae Compare October 9, 2024 14:13

danjm approved these changes Oct 10, 2024

View reviewed changes

mikesposito approved these changes Oct 10, 2024

View reviewed changes

vthomas13 added this pull request to the merge queue Oct 10, 2024

github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 10, 2024

vthomas13 added this pull request to the merge queue Oct 10, 2024

github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 10, 2024

vthomas13 force-pushed the marty-trezor-update branch from fd765ae to 2b63afb Compare October 11, 2024 12:52

removing trezor workaround

ebc4ff9

vthomas13 force-pushed the marty-trezor-update branch from 2b63afb to ebc4ff9 Compare October 11, 2024 14:31

vthomas13 added this pull request to the merge queue Oct 11, 2024

github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 11, 2024

vthomas13 added this pull request to the merge queue Oct 11, 2024

Merged via the queue into develop with commit 39e0251 Oct 11, 2024
79 checks passed

vthomas13 deleted the marty-trezor-update branch October 11, 2024 16:12

github-actions bot locked and limited conversation to collaborators Oct 11, 2024

metamaskbot added the release-12.7.0 Issue or pull request that will be included in release 12.7.0 label Oct 11, 2024

gauthierpetetin added release-12.6.0 Issue or pull request that will be included in release 12.6.0 and removed release-12.7.0 Issue or pull request that will be included in release 12.7.0 labels Oct 21, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: update Trezor Connect to v9.4.0, remove workarounds #27112

chore: update Trezor Connect to v9.4.0, remove workarounds #27112

vthomas13 commented Sep 12, 2024 •

edited

Loading

github-actions bot commented Sep 12, 2024

socket-security bot commented Sep 12, 2024 •

edited

Loading

socket-security bot commented Sep 12, 2024 •

edited

Loading

metamaskbot commented Sep 12, 2024

metamaskbot commented Sep 12, 2024

codecov bot commented Sep 12, 2024 •

edited

Loading

danjm commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

vthomas13 commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

sonarcloud bot commented Sep 16, 2024

metamaskbot commented Sep 16, 2024

danjm commented Sep 19, 2024

metamaskbot commented Sep 19, 2024

metamaskbot commented Sep 19, 2024

vthomas13 commented Oct 4, 2024

vthomas13 commented Oct 5, 2024

metamaskbot commented Oct 7, 2024

metamaskbot commented Oct 9, 2024

vthomas13 commented Oct 11, 2024

sonarcloud bot commented Oct 11, 2024

metamaskbot commented Oct 11, 2024

chore: update Trezor Connect to v9.4.0, remove workarounds #27112

chore: update Trezor Connect to v9.4.0, remove workarounds #27112

Conversation

vthomas13 commented Sep 12, 2024 • edited Loading

Description

Related issues

Manual testing steps

Screenshots/Recordings

Before

After

Pre-merge author checklist

Pre-merge reviewer checklist

github-actions bot commented Sep 12, 2024

socket-security bot commented Sep 12, 2024 • edited Loading

socket-security bot commented Sep 12, 2024 • edited Loading

Next steps

metamaskbot commented Sep 12, 2024

metamaskbot commented Sep 12, 2024

codecov bot commented Sep 12, 2024 • edited Loading

Codecov Report

danjm commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

vthomas13 commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

metamaskbot commented Sep 13, 2024

sonarcloud bot commented Sep 16, 2024

Quality Gate passed

metamaskbot commented Sep 16, 2024

danjm commented Sep 19, 2024

metamaskbot commented Sep 19, 2024

metamaskbot commented Sep 19, 2024

vthomas13 commented Oct 4, 2024

vthomas13 commented Oct 5, 2024

metamaskbot commented Oct 7, 2024

metamaskbot commented Oct 9, 2024

vthomas13 commented Oct 11, 2024

sonarcloud bot commented Oct 11, 2024

Quality Gate passed

metamaskbot commented Oct 11, 2024

vthomas13 commented Sep 12, 2024 •

edited

Loading

socket-security bot commented Sep 12, 2024 •

edited

Loading

socket-security bot commented Sep 12, 2024 •

edited

Loading

codecov bot commented Sep 12, 2024 •

edited

Loading