MetaMask / metamask-extension

:globe_with_meridians: :electric_plug: The MetaMask browser extension enables browsing Ethereum blockchain enabled websites
https://metamask.io
Other
12.08k stars 4.93k forks source link

Remove additional MV3 builds #25231

Open hjetpoluru opened 5 months ago

hjetpoluru commented 5 months ago

What is this about?

We need to prioritize the removal of extra MV3 builds from our pipeline. This task involves identifying and removing these builds to streamline our build process.

All the build code currently runs twice: once for MV2 and MV3. Question is: can we modify the build scripts such that the same script can generate all builds we need (MV2, MV3, Chrome, Firefox)?

Answer: yes. What we need to do is replace all uses of the ENABLE_MV3 environement variable with code that relies on the manifest version. Then, in a few places in the build code, we will need to rely on the browser type instead of whether the build is mv3 or not (e.g. when building the offscreen document and the manifest).

Alternatively (or in addition), we shall evaluate if we can use the event pages on Firefox for Offscreen, as we currently do on Chrome? It's likely the answer is yes.

Out of scope for this issue: ship MV3 in Firefox.

Scenario

No response

Design

No response

Technical Details

No response

Threat Modeling Framework

No response

Acceptance Criteria

No response

Stakeholder review needed before the work gets merged

References

No response

Gudahtt commented 5 months ago

Related to #21072