fix: validation of EIP-7702 signatures with odd-length chain ID

[matthewwalsh0]

Description

Bump @metamask/transaction-controller to pad chain IDs when validating signature.

Related issues

Fixes: #31338

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/transaction-controller@52.2.0 ➜ 52.2.0-preview-ed8163d	None	0	0 B

View full report↗︎

[metamaskbot]

Builds ready [fa57037]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• bundle size: Bundle Size Stats
• user-actions-benchmark: User Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all CircleCI 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

UI Startup Metrics (1178 ± 54 ms)

Platform	BuildType	Page	Metric	Mean (ms)	Min (ms)	Max (ms)	Std Dev (ms)	P 75 (ms)	P 95 (ms)
Chrome	Browserify	Home	uiStartup	1178	1075	1401	54	1203	1266
			load	1028	933	1263	53	934	982
			domContentLoaded	1022	927	1259	54	937	986
			domInteractive	16	13	31	4	15	25
			firstPaint	736	70	1155	400	239	975
			backgroundConnect	11	6	142	14	9	10
			firstReactRender	20	14	67	7	19	31
			getState	10	4	28	6	6	7
			initialActions	0	0	1	0	0	1
			loadScripts	811	703	1046	54	835	888
			setupStore	7	4	24	3	7	9
	Webpack	Home	uiStartup	943	678	1216	75	965	977
			load	800	544	920	72	842	881
			domContentLoaded	793	539	914	73	836	868
			domInteractive	15	12	41	7	14	38
			firstPaint	507	61	918	340	834	869
			backgroundConnect	16	11	52	9	15	39
			firstReactRender	14	12	36	4	13	24
			getState	7	4	13	2	7	8
			initialActions	0	0	1	0	0	0
			loadScripts	791	538	912	73	835	858
			setupStore	7	4	12	1	7	9
Firefox	Browserify	Home	uiStartup	1375	1205	1928	150	1397	1760
			load	1237	1074	1760	141	1266	1583
			domContentLoaded	1237	1073	1760	141	1266	1583
			domInteractive	101	42	202	27	89	97
			firstPaint	NaN	NaN	NaN	NaN	NaN	NaN
			backgroundConnect	23	16	94	9	24	39
			firstReactRender	22	19	35	3	25	28
			getState	7	4	31	3	7	8
			initialActions	0	0	1	0	0	1
			loadScripts	1215	1057	1725	138	1243	1556
			setupStore	6	3	20	2	6	7
	Webpack	Home	uiStartup	982	840	1510	137	901	964
			load	858	725	1363	120	849	989
			domContentLoaded	858	725	1363	120	849	989
			domInteractive	114	35	203	28	148	98
			firstPaint	NaN	NaN	NaN	NaN	NaN	NaN
			backgroundConnect	21	14	76	9	24	39
			firstReactRender	19	16	41	3	19	23
			getState	8	4	51	6	8	9
			initialActions	0	0	1	0	0	1
			loadScripts	841	709	1348	118	838	975
			setupStore	8	4	75	10	7	9

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

• background: 0 Bytes (0%)
• ui: 0 Bytes (0%)
• common: 990 Bytes (0.01%)

[matthewwalsh0]

Transaction controller bumped to 52.3.0 in #31335