fix: re-add patch to nonce-tracker that was incorrectly removed

[dbrans]

Note

This PR will fix a release blocker in v12.0.0 and will be cherry-picked there once it has been merged here.

Description

This PR re-adds a patch to nonce-tracker that was incorrectly removed.

For reference, here is the original patch that was incorrectly removed.

Here is the patch in this PR.

• patch branch: patch(v5.0.0): create a fresh instance of Web3Provider with this.provider every time nonce-tracker#79
• The original patch was introduced 7 months ago by @matthewwalsh0 "because the underlying proxy stopped working, meaning it would no longer send calls to the latest chain but be stuck on the original one. The quick fix was to patch it to create a fresh a Web3Provider for each call."
• @jiexi and @shanejonas have since identified the root cause of the issue this patch fixes: ethersjs.Web3Provider is not Proxy-friendly because it stores a reference to the sendAsync method of the provider.
• the patch was incorrectly removed recently, is missing from v12 and develop, and has been causing issues (see below).

Long Term Fix

The Wallet API and Transactions teams are discussing a long-term fix.

Related issues

• Fixes: [Bug]: Unable to submit a transaction on BNB Chain (happens on any network, with BNB being more consistent to reproduce) #25134
• Fixes: [Bug]: When attempting to initiate a transaction either from the dApp or directly within the wallet, users may encounter an "Internal JSON-RPC error." #25198
• Fixes: [Bug]: Nonce issues on Sepolia #25273

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• 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]

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.

[sleepytanya]

All transactions, including swaps, are created with the correct nonce and can be successfully submitted on BNB and mainnet.

bnbFix.mov

Current PR also resolves the issue with Sepolia 0 nonce:

sepoliaFix.mov

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.37%. Comparing base (2aaf565) to head (73de7d7).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop   #25342   +/-   ##
========================================
  Coverage    65.37%   65.37%           
========================================
  Files         1377     1377           
  Lines        54623    54623           
  Branches     14320    14320           
========================================
  Hits         35709    35709           
  Misses       18914    18914           

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

[metamaskbot]

Builds ready [73de7d7]

• 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, 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 (46 ± 3 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	61	91	76	8	4
		domContentLoaded	9	22	11	3	2
		load	38	60	46	6	3
		domInteractive	9	22	11	3	2

Bundle size diffs [🚀 Bundle size reduced!]

• background: 0 Bytes (0.00%)
• ui: 0 Bytes (0.00%)
• common: -17 Bytes (-0.00%)

[metamaskbot]

Missing release label release-12.0.0 on PR. Adding release label release-12.0.0 on PR and removing other release labels(release-12.1.0), as PR was cherry-picked in branch 12.0.0.