feat: support updated EIP-5792 specification

[matthewwalsh0]

Description

Support updated EIP-5792 specification.

See #363 for specification changes.

Specifically:

• Add GetCallsStatus hook.
• Throw if any non-optional capabilities specified in wallet_sendCalls.
• Throw if version not supported in wallet_sendCalls.
• Locate transaction using batchId.
• Return empty object from wallet_getCapabilities.
• Return standardised status code from wallet_getCallsStatus.
• Update @metamask/transaction-controller to 49.0.0.
• Update @metamask/eth-json-rpc-middleware to 16.0.0.

Related issues

Manual testing steps

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.

[socket-security]

Updated dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@metamask/[email protected] ➜ 49.0.0	None	0	2.37 MB	danfinlay, gudahtt, kumavis, ...6 more

View full report↗︎

[metamaskbot]

Builds ready [6940043]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12
  • content-script: 0
  • offscreen: 0

Page Load Metrics (3290 ± 1452 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	1721	14956	3067	2871	1379
		domContentLoaded	1612	14186	2721	2741	1316
		load	1740	15628	3290	3024	1452
		domInteractive	25	246	73	74	35
		backgroundConnect	125	1512	510	408	196
		firstReactRender	50	178	91	38	18
		getState	49	907	283	254	122
		initialActions	0	1	0	0	0
		loadScripts	1165	12366	2059	2424	1164
		setupStore	20	842	144	188	90
		uiStartup	2368	18248	5440	3940	1892

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

• background: 93.51 KiB (1.50%)
• ui: 0 Bytes (0.00%)
• common: 145.19 KiB (1.41%)

[matthewwalsh0]

Upgrading @metamask/transaction-controller past 48.2.0 includes a fix that supports both gas and gasLimit in params.

Many of the E2E tests use the #sendButton to create legacy transactions via the test dApp, which includes gasLimit.

As this was previously ignored, the transaction controller would automatically estimate the gas limit via eth_estimateGas which would slightly delay the display of the confirmation.

Some of these tests were reliant on that timing, meaning they now break since the confirmations display quicker as no estimate is performed since the gasLimit is provided.

This is the quickest and least invasive fix as it doesn't require new releases, doesn't require refactoring multiple E2E tests, and doesn't require a patch which would be very large since the test dApp code is minimised to a single line.