fix: cp-12.13.0 Disable origin throttling middleware if cause is "rejectAllApprovals" (#30388)

<!--
Please submit this PR as a draft initially.
Do not mark it as "Ready for review" until the template has been
completely filled out, and PR status checks have passed at least once.
-->

<!--
Write a short description of the changes included in this pull request,
also include relevant motivation and context. Have in mind the following
questions:
1. What is the reason for the change?
2. What is the improvement/solution?
-->

This PR aims to fix `release-blocker` in `v12.13.0` which disables
origin throttling middleware if `cause` is `rejectAllApprovals`.

Patch note:
Currently bumping `transaction-controller` is blocked because it's
breaking e2e tests
https://consensys.slack.com/archives/CTQAGKY5V/p1739968847746929?thread_ts=1739869734.799669&cid=CTQAGKY5V
To avoid applying version newer version of `transaction-controller`
needed change patched in this PR. Required change is already in the core
repository MetaMask/core#5355. And will most
likely be released in version `46.0.1`. Then we will most likely bump
`transaction-controller` in `main`.

[![Open in GitHub
Codespaces](https://github.com/codespaces/badge.svg)](https://codespaces.new/MetaMask/metamask-extension/pull/30388?quickstart=1)

Fixes: #30333

See the task details

<!-- If applicable, add screenshots and/or recordings to visualize the
before and after of your change. -->

<!-- [screenshots/recordings] -->

<!-- [screenshots/recordings] -->

- [X] I've followed [MetaMask Contributor
Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask
Extension Coding
Standards](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/CODING_GUIDELINES.md).
- [X] I've completed the PR template to the best of my ability
- [X] I’ve included tests if applicable
- [X] I’ve documented my code using [JSDoc](https://jsdoc.app/) format
if applicable
- [X] I’ve applied the right labels on the PR (see [labeling
guidelines](https://github.com/MetaMask/metamask-extension/blob/main/.github/guidelines/LABELING_GUIDELINES.md)).
Not required for external contributors.

- [ ] 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.

---------

Co-authored-by: MetaMask Bot <[email protected]>

Author: 2 people

.yarn/patches/@metamask-transaction-controller-npm-45.0.0-010fef9da6.patch

@@ -0,0 +1,17 @@
+diff --git a/dist/TransactionController.cjs b/dist/TransactionController.cjs
+index c96d2962d81c4c63c7b626bc1ca18b196b20a154..236b451e5f94584d4ce42b6bfaf6b7225b8ae7f9 100644
+--- a/dist/TransactionController.cjs
++++ b/dist/TransactionController.cjs
+@@ -1152,7 +1152,11 @@ class TransactionController extends base_controller_1.BaseController {
+                 if (!isTxCompleted) {
+                     if (error?.code === rpc_errors_1.errorCodes.provider.userRejectedRequest) {
+                         this.cancelTransaction(transactionId, actionId);
+-                        throw rpc_errors_1.providerErrors.userRejectedRequest('MetaMask Tx Signature: User denied transaction signature.');
++                        throw rpc_errors_1.providerErrors.userRejectedRequest({
++                            message: 
++                                'MetaMask Tx Signature: User denied transaction signature.',
++                            data: error?.data,
++                        });
+                     }
+                     else {
+                         this.failTransaction(meta, error, actionId);

app/scripts/lib/approval/utils.ts

@@ -49,7 +49,14 @@ export function rejectAllApprovals({
 
       default:
         log('Rejecting pending approval', { id });
-        approvalController.reject(id, providerErrors.userRejectedRequest());
+        approvalController.reject(
+          id,
+          providerErrors.userRejectedRequest({
+            data: {
+              cause: 'rejectAllApprovals',
+            },
+          }),
+        );
         break;
     }
   }

app/scripts/lib/createOriginThrottlingMiddleware.test.ts

@@ -1,4 +1,4 @@
-import { errorCodes } from '@metamask/rpc-errors';
+import { errorCodes, providerErrors } from '@metamask/rpc-errors';
 import { JsonRpcResponse } from '@metamask/utils';
 import type { Json } from '@metamask/utils';
 import createOriginThrottlingMiddleware, {
@@ -118,4 +118,35 @@ describe('createOriginThrottlingMiddleware', () => {
     });
     expect(nextCallback).toHaveBeenCalled();
   });
+
+  it('does not update throttling state if response has userRejected error and rejectAllApprovals is in the error data', async () => {
+    const req = {
+      method: 'eth_sendTransaction',
+      origin: 'testOrigin',
+    } as unknown as ExtendedJSONRPCRequest;
+    const nextCallback = jest.fn();
+    const next = jest
+      .fn()
+      .mockImplementation((callback) => callback(nextCallback));
+    const end = jest.fn();
+    const responseWithUserRejectedError = {
+      error: providerErrors.userRejectedRequest({
+        data: {
+          cause: 'rejectAllApprovals',
+        },
+      }),
+      id: 1,
+      jsonrpc: '2.0',
+    } as unknown as JsonRpcResponse<Json>;
+
+    mockGetThrottledOriginState.mockReturnValueOnce({
+      rejections: 0,
+      lastRejection: 0,
+    });
+
+    await middleware(req, responseWithUserRejectedError, next, end);
+
+    expect(mockUpdateThrottledOriginState).not.toHaveBeenCalled();
+    expect(nextCallback).toHaveBeenCalled();
+  });
 });

app/scripts/lib/createOriginThrottlingMiddleware.ts

@@ -83,6 +83,13 @@ export default function createOriginThrottlingMiddleware({
 
     next((callback: () => void) => {
       if ('error' in res && res.error && isUserRejectedError(res.error)) {
+        const extraData = res.error?.data as { cause?: string };
+        // Any rejection caused by rejectAllApprovals is not evaluated as user rejection for now
+        if (extraData?.cause === 'rejectAllApprovals') {
+          callback();
+          return;
+        }
+
         // User rejected the request
         const throttledOriginState = getThrottledOriginState(origin) || {
           rejections: 0,

yarn.lock

@@ -2307,7 +2307,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@ethereumjs/tx@npm:^5.2.1, @ethereumjs/tx@npm:^5.3.0":
+"@ethereumjs/tx@npm:^5.2.1, @ethereumjs/tx@npm:^5.3.0, @ethereumjs/tx@npm:^5.4.0":
   version: 5.4.0
   resolution: "@ethereumjs/tx@npm:5.4.0"
   dependencies:
@@ -6492,16 +6492,16 @@ __metadata:
   linkType: hard
 
 "@metamask/transaction-controller@npm:^45.0.0":
-  version: 45.0.0
-  resolution: "@metamask/transaction-controller@npm:45.0.0"
+  version: 45.1.0
+  resolution: "@metamask/transaction-controller@npm:45.1.0"
   dependencies:
-    "@ethereumjs/common": "npm:^3.2.0"
-    "@ethereumjs/tx": "npm:^4.2.0"
+    "@ethereumjs/common": "npm:^4.4.0"
+    "@ethereumjs/tx": "npm:^5.4.0"
     "@ethereumjs/util": "npm:^8.1.0"
     "@ethersproject/abi": "npm:^5.7.0"
     "@ethersproject/contracts": "npm:^5.7.0"
     "@ethersproject/providers": "npm:^5.7.0"
-    "@metamask/base-controller": "npm:^7.1.1"
+    "@metamask/base-controller": "npm:^8.0.0"
     "@metamask/controller-utils": "npm:^11.5.0"
     "@metamask/eth-query": "npm:^4.0.0"
     "@metamask/metamask-eth-abis": "npm:^3.1.1"
@@ -6521,7 +6521,7 @@ __metadata:
     "@metamask/eth-block-tracker": ">=9"
     "@metamask/gas-fee-controller": ^22.0.0
     "@metamask/network-controller": ^22.0.0
-  checksum: 10/42fc91e4ebb0ad83f8230c2060c392d0ba5fd555a2d3f407862c4574f195f1cb2b57ea025ea332e55a1196605602209219a828c565bd6a13d3135557d7427e24
+  checksum: 10/4be40e2315e512db873c436f87cfb43f63623491c98cbb22c28c24091cf666c2428e9916f2ba1706743504ba8af505dc2c3fa5549c1ed99f50868284eb5fe07f
   languageName: node
   linkType: hard