use @metamask/rpc-errors to standardize errors

packages/transaction-controller/package.json

@@ -35,10 +35,10 @@
     "@metamask/controller-utils": "^4.3.2",
     "@metamask/eth-query": "^3.0.1",
     "@metamask/network-controller": "^12.1.2",
+    "@metamask/rpc-errors": "^5.1.1",
     "@metamask/utils": "^6.2.0",
     "async-mutex": "^0.2.6",
     "eth-method-registry": "1.1.0",
-    "eth-rpc-errors": "^4.0.2",
     "ethereumjs-util": "^7.0.10",
     "fast-json-patch": "^3.1.1",
     "lodash": "^4.17.21",

packages/transaction-controller/src/TransactionController.test.ts

@@ -13,7 +13,7 @@ import type {
   Provider,
 } from '@metamask/network-controller';
 import { NetworkClientType, NetworkStatus } from '@metamask/network-controller';
-import { errorCodes } from 'eth-rpc-errors';
+import { errorCodes } from '@metamask/rpc-errors';
 import HttpProvider from 'ethjs-provider-http';
 import NonceTracker from 'nonce-tracker';
 

packages/transaction-controller/src/TransactionController.ts

@@ -30,10 +30,10 @@ import type {
   NetworkState,
   Provider,
 } from '@metamask/network-controller';
+import { errorCodes, rpcErrors, providerErrors } from '@metamask/rpc-errors';
 import type { Hex } from '@metamask/utils';
 import { Mutex } from 'async-mutex';
 import MethodRegistry from 'eth-method-registry';
-import { errorCodes, ethErrors } from 'eth-rpc-errors';
 import { addHexPrefix, bufferToHex } from 'ethereumjs-util';
 import { EventEmitter } from 'events';
 import { merge, pickBy } from 'lodash';
@@ -1166,7 +1166,7 @@ export class TransactionController extends BaseController<
           if (error.code === errorCodes.provider.userRejectedRequest) {
             this.cancelTransaction(transactionId);
 
-            throw ethErrors.provider.userRejectedRequest(
+            throw providerErrors.userRejectedRequest(
               'User rejected the transaction',
             );
           } else {
@@ -1181,10 +1181,10 @@ export class TransactionController extends BaseController<
     switch (finalMeta?.status) {
       case TransactionStatus.failed:
         resultCallbacks?.error(finalMeta.error);
-        throw ethErrors.rpc.internal(finalMeta.error.message);
+        throw rpcErrors.internal(finalMeta.error.message);
 
       case TransactionStatus.cancelled:
-        const cancelError = ethErrors.rpc.internal(
+        const cancelError = rpcErrors.internal(
           'User cancelled the transaction',
         );
 
@@ -1196,7 +1196,7 @@ export class TransactionController extends BaseController<
         return finalMeta.hash as string;
 
       default:
-        const internalError = ethErrors.rpc.internal(
+        const internalError = rpcErrors.internal(
           `MetaMask Tx Signature: Unknown problem: ${JSON.stringify(
             finalMeta || transactionId,
           )}`,

packages/transaction-controller/src/external-transactions.test.ts

@@ -1,4 +1,4 @@
-import { ethErrors } from 'eth-rpc-errors';
+import { rpcErrors } from '@metamask/rpc-errors';
 
 import { validateConfirmedExternalTransaction } from './external-transactions';
 import type { TransactionMeta } from './types';
@@ -17,15 +17,15 @@ describe('validateConfirmedExternalTransaction', () => {
     expect(() =>
       validateConfirmedExternalTransaction(undefined, [], []),
     ).toThrow(
-      ethErrors.rpc.invalidParams(
+      rpcErrors.invalidParams(
         '"transactionMeta" or "transactionMeta.txParams" is missing',
       ),
     );
 
     expect(() =>
       validateConfirmedExternalTransaction({} as TransactionMeta, [], []),
     ).toThrow(
-      ethErrors.rpc.invalidParams(
+      rpcErrors.invalidParams(
         '"transactionMeta" or "transactionMeta.txParams" is missing',
       ),
     );
@@ -37,7 +37,7 @@ describe('validateConfirmedExternalTransaction', () => {
       '123',
     );
     expect(() => validateConfirmedExternalTransaction(transactionMeta)).toThrow(
-      ethErrors.rpc.invalidParams(
+      rpcErrors.invalidParams(
         'External transaction status should be "confirmed"',
       ),
     );
@@ -56,7 +56,7 @@ describe('validateConfirmedExternalTransaction', () => {
     expect(() =>
       validateConfirmedExternalTransaction(transactionMeta, [], pendingTxs),
     ).toThrow(
-      ethErrors.rpc.invalidParams(
+      rpcErrors.invalidParams(
         'External transaction nonce should not be in pending txs',
       ),
     );
@@ -75,7 +75,7 @@ describe('validateConfirmedExternalTransaction', () => {
     expect(() =>
       validateConfirmedExternalTransaction(transactionMeta, confirmedTxs, []),
     ).toThrow(
-      ethErrors.rpc.invalidParams(
+      rpcErrors.invalidParams(
         'External transaction nonce should not be in confirmed txs',
       ),
     );

packages/transaction-controller/src/external-transactions.ts

@@ -1,5 +1,5 @@
 // These utility functions are exclusively used by `confirmExternalTransaction` method in controller
-import { ethErrors } from 'eth-rpc-errors';
+import { rpcErrors } from '@metamask/rpc-errors';
 
 import { TransactionStatus } from './types';
 import type { TransactionMeta } from './types';
@@ -17,13 +17,13 @@ export function validateConfirmedExternalTransaction(
   pendingTxs?: TransactionMeta[],
 ) {
   if (!transactionMeta || !transactionMeta.txParams) {
-    throw ethErrors.rpc.invalidParams(
+    throw rpcErrors.invalidParams(
       '"transactionMeta" or "transactionMeta.txParams" is missing',
     );
   }
 
   if (transactionMeta.status !== TransactionStatus.confirmed) {
-    throw ethErrors.rpc.invalidParams(
+    throw rpcErrors.invalidParams(
       'External transaction status should be "confirmed"',
     );
   }
@@ -34,7 +34,7 @@ export function validateConfirmedExternalTransaction(
       (tx) => tx.txParams?.nonce === externalTxNonce,
     );
     if (foundPendingTxByNonce) {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         'External transaction nonce should not be in pending txs',
       );
     }
@@ -45,7 +45,7 @@ export function validateConfirmedExternalTransaction(
       (tx) => tx.txParams?.nonce === externalTxNonce,
     );
     if (foundConfirmedTxByNonce) {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         'External transaction nonce should not be in confirmed txs',
       );
     }

packages/transaction-controller/src/utils.test.ts

@@ -1,4 +1,4 @@
-import { ethErrors } from 'eth-rpc-errors';
+import { rpcErrors } from '@metamask/rpc-errors';
 import type { Transaction as NonceTrackerTransaction } from 'nonce-tracker/dist/NonceTracker';
 
 import type {
@@ -50,23 +50,23 @@ describe('utils', () => {
   describe('validateTxParams', () => {
     it('should throw if no from address', () => {
       expect(() => util.validateTxParams({} as any)).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "from" address: undefined must be a valid string.',
         ),
       );
     });
 
     it('should throw if non-string from address', () => {
       expect(() => util.validateTxParams({ from: 1337 } as any)).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "from" address: 1337 must be a valid string.',
         ),
       );
     });
 
     it('should throw if invalid from address', () => {
       expect(() => util.validateTxParams({ from: '1337' } as any)).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "from" address: 1337 must be a valid string.',
         ),
       );
@@ -79,7 +79,7 @@ describe('utils', () => {
           to: '0x',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "to" address: 0x must be a valid string.',
         ),
       );
@@ -89,7 +89,7 @@ describe('utils', () => {
           from: '0x3244e191f1b4903970224322180f1fbbc415696b',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "to" address: undefined must be a valid string.',
         ),
       );
@@ -112,7 +112,7 @@ describe('utils', () => {
           to: '1337',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "to" address: 1337 must be a valid string.',
         ),
       );
@@ -126,7 +126,7 @@ describe('utils', () => {
           value: '133-7',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "value": 133-7 is not a positive number.',
         ),
       );
@@ -138,7 +138,7 @@ describe('utils', () => {
           value: '133.7',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "value": 133.7 number must be denominated in wei.',
         ),
       );
@@ -150,7 +150,7 @@ describe('utils', () => {
           value: 'hello',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "value": hello number must be a valid number.',
         ),
       );
@@ -162,7 +162,7 @@ describe('utils', () => {
           value: 'one million dollar$',
         } as any),
       ).toThrow(
-        ethErrors.rpc.invalidParams(
+        rpcErrors.invalidParams(
           'Invalid "value": one million dollar$ number must be a valid number.',
         ),
       );

packages/transaction-controller/src/utils.ts

@@ -2,8 +2,8 @@ import {
   convertHexToDecimal,
   isValidHexAddress,
 } from '@metamask/controller-utils';
+import { rpcErrors } from '@metamask/rpc-errors';
 import type { Hex } from '@metamask/utils';
-import { ethErrors } from 'eth-rpc-errors';
 import { addHexPrefix, isHexString } from 'ethereumjs-util';
 import type { Transaction as NonceTrackerTransaction } from 'nonce-tracker/dist/NonceTracker';
 
@@ -61,7 +61,7 @@ export function validateTxParams(txParams: TransactionParams) {
     typeof txParams.from !== 'string' ||
     !isValidHexAddress(txParams.from)
   ) {
-    throw ethErrors.rpc.invalidParams(
+    throw rpcErrors.invalidParams(
       `Invalid "from" address: ${txParams.from} must be a valid string.`,
     );
   }
@@ -70,26 +70,26 @@ export function validateTxParams(txParams: TransactionParams) {
     if (txParams.data) {
       delete txParams.to;
     } else {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         `Invalid "to" address: ${txParams.to} must be a valid string.`,
       );
     }
   } else if (txParams.to !== undefined && !isValidHexAddress(txParams.to)) {
-    throw ethErrors.rpc.invalidParams(
+    throw rpcErrors.invalidParams(
       `Invalid "to" address: ${txParams.to} must be a valid string.`,
     );
   }
 
   if (txParams.value !== undefined) {
     const value = txParams.value.toString();
     if (value.includes('-')) {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         `Invalid "value": ${value} is not a positive number.`,
       );
     }
 
     if (value.includes('.')) {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         `Invalid "value": ${value} number must be denominated in wei.`,
       );
     }
@@ -100,7 +100,7 @@ export function validateTxParams(txParams: TransactionParams) {
       !isNaN(Number(value)) &&
       Number.isSafeInteger(intValue);
     if (!isValid) {
-      throw ethErrors.rpc.invalidParams(
+      throw rpcErrors.invalidParams(
         `Invalid "value": ${value} number must be a valid number.`,
       );
     }

yarn.lock

@@ -2048,6 +2048,16 @@ __metadata:
   languageName: node
   linkType: hard
 
+"@metamask/rpc-errors@npm:^6.0.0":
+  version: 6.0.0
+  resolution: "@metamask/rpc-errors@npm:6.0.0"
+  dependencies:
+    "@metamask/utils": ^8.0.0
+    fast-safe-stringify: ^2.0.6
+  checksum: 7e1ee1a98972266af4a34f0bbc842cdc11dc565056f0b8fbc93aa95663a7027eab8ff1fecbe3e09c38a1dc199f8219a6c69b2237015b2fdb8de0e5b35027c3f8
+  languageName: node
+  linkType: hard
+
 "@metamask/safe-event-emitter@npm:^2.0.0":
   version: 2.0.0
   resolution: "@metamask/safe-event-emitter@npm:2.0.0"
@@ -2142,14 +2152,14 @@ __metadata:
     "@metamask/controller-utils": ^4.3.2
     "@metamask/eth-query": ^3.0.1
     "@metamask/network-controller": ^12.1.2
+    "@metamask/rpc-errors": ^6.0.0
     "@metamask/utils": ^6.2.0
     "@types/jest": ^27.4.1
     "@types/node": ^16.18.24
     async-mutex: ^0.2.6
     babel-runtime: ^6.26.0
     deepmerge: ^4.2.2
     eth-method-registry: 1.1.0
-    eth-rpc-errors: ^4.0.2
     ethereumjs-util: ^7.0.10
     ethjs-provider-http: ^0.1.6
     fast-json-patch: ^3.1.1