sendTransaction and sendSignedTransaction Error Refactors

CHANGELOG.md

@@ -1155,11 +1155,15 @@ should use 4.0.1-alpha.0 for testing.
 #### web3-errors
 
 -   Added error class `InvalidMethodParamsError` and error code `ERR_INVALID_METHOD_PARAMS = 207` (#5824)
+-   `request` property to `ResponseError` (#5854)
+-   `data` property to `TransactionRevertInstructionError` (#5854)
+-   `TransactionRevertWithCustomError` was added to handle custom solidity errors (#5854)
 
 #### web3-eth
 
 -   Added `createAccessList` functionality ( #5780 )
 -   Added support of `safe` and `finalized` block tags (#5823)
+-   `contractAbi` option to `SendTransactionOptions` and `SendSignedTransactionOptions` to added the ability to parse custom solidity errors (#5854)
 
 #### web3-eth-abi
 
@@ -1211,10 +1215,12 @@ should use 4.0.1-alpha.0 for testing.
 #### web3-errors
 
 -   The abstract class `Web3Error` is renamed to `BaseWeb3Error` (#5771)
+-   Renamed `TransactionRevertError` to `TransactionRevertInstructionError` to remain consistent with `1.x` (#5854)
 
 #### web3-eth
 
 -   Update imports statements for objects that was moved between web3 packages (#5771)
+-   `sendTransaction` and `sendSignedTransaction` now errors with (and `error` event emits) the following possible errors: `TransactionRevertedWithoutReasonError`, `TransactionRevertInstructionError`, `TransactionRevertWithCustomError`, `InvalidResponseError`, or `ContractExecutionError` (#5854)
 
 #### web3-eth-contract
 
@@ -1257,3 +1263,4 @@ should use 4.0.1-alpha.0 for testing.
 #### web3-eth-contract
 
 -   Fix contract defaults (#5756)
+-   Fixed getPastEventsError (#5819)

docs/docs/guides/web3_migration_guide/providers_migration_guide.md

@@ -187,4 +187,4 @@ provider.on('error', error => {
 		// the `maxAttempts` is equal to the provided value by the user, or the default value `5`.
 	}
 });
-```
+```

docs/docs/guides/web3_providers_guide/index.md

@@ -207,4 +207,4 @@ provider.on('error', error => {
 		// the `maxAttempts` is equal to the provided value by the user, or the default value `5`.
 	}
 });
-```
+```

packages/web3-core/src/web3_request_manager.ts

@@ -369,7 +369,7 @@ export class Web3RequestManager<
 					throw new RpcError(rpcErrorResponse);
 				}
 			} else if (!Web3RequestManager._isReverted(response)) {
-				throw new InvalidResponseError<ErrorType>(response);
+				throw new InvalidResponseError<ErrorType, RequestType>(response, payload);
 			}
 		}
 

packages/web3-errors/CHANGELOG.md

@@ -76,8 +76,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 -   The abstract class `Web3Error` is renamed to `BaseWeb3Error` (#5771)
+-   Renamed TransactionRevertError to TransactionRevertInstructionError to remain consistent with 1.x
 -   Using `MaxAttemptsReachedOnReconnectingError` with the same message for 1.x but also adding the `maxAttempts` (#5894)
 
 ### Added
 
 -   Added error class `InvalidMethodParamsError` and error code `ERR_INVALID_METHOD_PARAMS = 207` (#5824)
+-   `request` property to `ResponseError` (#5854)
+-   `data` property to `TransactionRevertInstructionError` (#5854)
+-   `TransactionRevertWithCustomError` was added to handle custom solidity errors (#5854)

packages/web3-errors/src/error_codes.ts

@@ -86,6 +86,7 @@ export const ERR_TX_GAS_MISMATCH = 434;
 export const ERR_TX_CHAIN_MISMATCH = 435;
 export const ERR_TX_HARDFORK_MISMATCH = 436;
 export const ERR_TX_INVALID_RECEIVER = 437;
+export const ERR_TX_REVERT_TRANSACTION_CUSTOM_ERROR = 438;
 
 // Connection error codes
 export const ERR_CONN = 500;

packages/web3-errors/src/errors/response_errors.ts

@@ -16,7 +16,12 @@ along with web3.js.  If not, see <http://www.gnu.org/licenses/>.
 */
 
 // eslint-disable-next-line max-classes-per-file
-import { JsonRpcError, JsonRpcResponse, JsonRpcResponseWithError } from 'web3-types';
+import {
+	JsonRpcError,
+	JsonRpcPayload,
+	JsonRpcResponse,
+	JsonRpcResponseWithError,
+} from 'web3-types';
 import { BaseWeb3Error } from '../web3_error_base';
 import { ERR_INVALID_RESPONSE, ERR_RESPONSE } from '../error_codes';
 
@@ -36,11 +41,16 @@ const isResponseWithError = <Error = unknown, Result = unknown>(
 const buildErrorMessage = (response: JsonRpcResponse<unknown, unknown>): string =>
 	isResponseWithError(response) ? response.error.message : '';
 
-export class ResponseError<ErrorType = unknown> extends BaseWeb3Error {
+export class ResponseError<ErrorType = unknown, RequestType = unknown> extends BaseWeb3Error {
 	public code = ERR_RESPONSE;
 	public data?: ErrorType | ErrorType[];
+	public request?: JsonRpcPayload<RequestType>;
 
-	public constructor(response: JsonRpcResponse<unknown, ErrorType>, message?: string) {
+	public constructor(
+		response: JsonRpcResponse<unknown, ErrorType>,
+		message?: string,
+		request?: JsonRpcPayload<RequestType>,
+	) {
 		super(
 			message ??
 				`Returned error: ${
@@ -55,16 +65,24 @@ export class ResponseError<ErrorType = unknown> extends BaseWeb3Error {
 				? response.map(r => r.error?.data as ErrorType)
 				: response?.error?.data;
 		}
+
+		this.request = request;
 	}
 
 	public toJSON() {
-		return { ...super.toJSON(), data: this.data };
+		return { ...super.toJSON(), data: this.data, request: this.request };
 	}
 }
 
-export class InvalidResponseError<ErrorType = unknown> extends ResponseError<ErrorType> {
-	public constructor(result: JsonRpcResponse<unknown, ErrorType>) {
-		super(result);
+export class InvalidResponseError<ErrorType = unknown, RequestType = unknown> extends ResponseError<
+	ErrorType,
+	RequestType
+> {
+	public constructor(
+		result: JsonRpcResponse<unknown, ErrorType>,
+		request?: JsonRpcPayload<RequestType>,
+	) {
+		super(result, undefined, request);
 		this.code = ERR_INVALID_RESPONSE;
 
 		let errorOrErrors: JsonRpcError | JsonRpcError[] | undefined;

packages/web3-errors/src/errors/transaction_errors.ts

@@ -57,6 +57,7 @@ import {
 	ERR_TX_UNABLE_TO_POPULATE_NONCE,
 	ERR_TX_UNSUPPORTED_EIP_1559,
 	ERR_TX_UNSUPPORTED_TYPE,
+	ERR_TX_REVERT_TRANSACTION_CUSTOM_ERROR,
 } from '../error_codes';
 import { InvalidValueError, BaseWeb3Error } from '../web3_error_base';
 
@@ -84,16 +85,21 @@ export class RevertInstructionError extends BaseWeb3Error {
 	}
 }
 
-export class TransactionRevertError extends BaseWeb3Error {
+export class TransactionRevertInstructionError<
+	ReceiptType = TransactionReceipt,
+> extends BaseWeb3Error {
 	public code = ERR_TX_REVERT_TRANSACTION;
 
 	public constructor(
 		public reason: string,
 		public signature?: string,
-		public receipt?: TransactionReceipt,
+		public receipt?: ReceiptType,
+		public data?: string,
 	) {
 		super(
-			`Transaction has been reverted by the EVM:\n ${JSON.stringify(receipt, undefined, 2)}`,
+			`Transaction has been reverted by the EVM${
+				receipt === undefined ? '' : `:\n ${BaseWeb3Error.convertToString(receipt)}`
+			}`,
 		);
 	}
 
@@ -103,6 +109,43 @@ export class TransactionRevertError extends BaseWeb3Error {
 			reason: this.reason,
 			signature: this.signature,
 			receipt: this.receipt,
+			data: this.data,
+		};
+	}
+}
+
+/**
+ * This error is used when a transaction to a smart contract fails and
+ * a custom user error (https://blog.soliditylang.org/2021/04/21/custom-errors/)
+ * is able to be parsed from the revert reason
+ */
+export class TransactionRevertWithCustomError<
+	ReceiptType = TransactionReceipt,
+> extends TransactionRevertInstructionError<ReceiptType> {
+	public code = ERR_TX_REVERT_TRANSACTION_CUSTOM_ERROR;
+
+	public constructor(
+		public reason: string,
+		public customErrorName: string,
+		public customErrorDecodedSignature: string,
+		public customErrorArguments: Record<string, unknown>,
+		public signature?: string,
+		public receipt?: ReceiptType,
+		public data?: string,
+	) {
+		super(reason);
+	}
+
+	public toJSON() {
+		return {
+			...super.toJSON(),
+			reason: this.reason,
+			customErrorName: this.customErrorName,
+			customErrorDecodedSignature: this.customErrorDecodedSignature,
+			customErrorArguments: this.customErrorArguments,
+			signature: this.signature,
+			receipt: this.receipt,
+			data: this.data,
 		};
 	}
 }
@@ -125,10 +168,14 @@ export class ContractCodeNotStoredError extends TransactionError {
 	}
 }
 
-export class TransactionRevertedWithoutReasonError extends TransactionError {
-	public constructor(receipt: TransactionReceipt) {
+export class TransactionRevertedWithoutReasonError<
+	ReceiptType = TransactionReceipt,
+> extends TransactionError<ReceiptType> {
+	public constructor(receipt?: ReceiptType) {
 		super(
-			`Transaction has been reverted by the EVM:\n ${JSON.stringify(receipt, undefined, 2)}`,
+			`Transaction has been reverted by the EVM${
+				receipt === undefined ? '' : `:\n ${BaseWeb3Error.convertToString(receipt)}`
+			}`,
 			receipt,
 		);
 		this.code = ERR_TX_REVERT_WITHOUT_REASON;

packages/web3-errors/test/unit/__snapshots__/errors.test.ts.snap

@@ -172,6 +172,7 @@ Object {
   },
   "message": "Returned error: error message",
   "name": "InvalidResponseError",
+  "request": undefined,
 }
 `;
 
@@ -229,6 +230,7 @@ Object {
   "innerError": undefined,
   "message": "Returned error: error message",
   "name": "ResponseError",
+  "request": undefined,
 }
 `;
 
@@ -239,6 +241,7 @@ Object {
   "innerError": undefined,
   "message": "Returned error: error message",
   "name": "ResponseError",
+  "request": undefined,
 }
 `;
 
@@ -280,15 +283,14 @@ Object {
 }
 `;
 
-exports[`errors TransactionRevertError should have valid json structure 1`] = `
+exports[`errors TransactionRevertInstructionError should have valid json structure 1`] = `
 Object {
   "code": 402,
+  "data": undefined,
   "innerError": undefined,
   "message": "Transaction has been reverted by the EVM:
- {
-  \\"attr1\\": \\"attr1\\"
-}",
-  "name": "TransactionRevertError",
+ {\\"attr1\\":\\"attr1\\"}",
+  "name": "TransactionRevertInstructionError",
   "reason": "message",
   "receipt": Object {
     "attr1": "attr1",
@@ -297,14 +299,32 @@ Object {
 }
 `;
 
+exports[`errors TransactionRevertWithCustomError should have valid json structure 1`] = `
+Object {
+  "code": 438,
+  "customErrorArguments": Object {
+    "customErrorArgument": "customErrorArgument",
+  },
+  "customErrorDecodedSignature": "customErrorDecodedSignature",
+  "customErrorName": "customErrorName",
+  "data": "data",
+  "innerError": undefined,
+  "message": "Transaction has been reverted by the EVM",
+  "name": "TransactionRevertWithCustomError",
+  "reason": "reason",
+  "receipt": Object {
+    "attr1": "attr1",
+  },
+  "signature": "signature",
+}
+`;
+
 exports[`errors TransactionRevertedWithoutReasonError should have valid json structure 1`] = `
 Object {
   "code": 405,
   "innerError": undefined,
   "message": "Transaction has been reverted by the EVM:
- {
-  \\"attr1\\": \\"attr1\\"
-}",
+ {\\"attr1\\":\\"attr1\\"}",
   "name": "TransactionRevertedWithoutReasonError",
   "receipt": Object {
     "attr1": "attr1",

packages/web3-errors/test/unit/errors.test.ts

@@ -53,7 +53,7 @@ describe('errors', () => {
 			// To disable error for the abstract class
 
 			// eslint-disable-next-line @typescript-eslint/ban-ts-comment
-			const err = new ErrorClass({} as never, {} as never, {} as never);
+			const err = new ErrorClass({} as never, {} as never, {} as never, {} as never);
 			errorCodes.push(err.code);
 		}
 
@@ -162,16 +162,34 @@ describe('errors', () => {
 		});
 	});
 
-	describe('TransactionRevertError', () => {
+	describe('TransactionRevertInstructionError', () => {
 		it('should have valid json structure', () => {
 			expect(
-				new transactionErrors.TransactionRevertError('message', 'signature', {
+				new transactionErrors.TransactionRevertInstructionError('message', 'signature', {
 					attr1: 'attr1',
 				} as any).toJSON(),
 			).toMatchSnapshot();
 		});
 	});
 
+	describe('TransactionRevertWithCustomError', () => {
+		it('should have valid json structure', () => {
+			expect(
+				new transactionErrors.TransactionRevertWithCustomError(
+					'reason',
+					'customErrorName',
+					'customErrorDecodedSignature',
+					{ customErrorArgument: 'customErrorArgument' },
+					'signature',
+					{
+						attr1: 'attr1',
+					} as any,
+					'data',
+				).toJSON(),
+			).toMatchSnapshot();
+		});
+	});
+
 	describe('NoContractAddressFoundError', () => {
 		it('should have valid json structure', () => {
 			expect(

packages/web3-eth-contract/src/contract.ts

@@ -1050,10 +1050,13 @@ export class Contract<Abi extends ContractAbi>
 			options,
 			contractOptions: modifiedContractOptions,
 		});
-		const transactionToSend = sendTransaction(this, tx, DEFAULT_RETURN_FORMAT);
+		const transactionToSend = sendTransaction(this, tx, DEFAULT_RETURN_FORMAT, {
+			// TODO Should make this configurable by the user
+			checkRevertBeforeSending: false,
+		});
 
 		// eslint-disable-next-line no-void
-		void transactionToSend.on('contractExecutionError', (error: unknown) => {
+		void transactionToSend.on('error', (error: unknown) => {
 			if (error instanceof ContractExecutionError) {
 				// this will parse the error data by trying to decode the ABI error inputs according to EIP-838
 				decodeContractErrorData(errorsAbi, error.innerError);
@@ -1094,6 +1097,8 @@ export class Contract<Abi extends ContractAbi>
 				newContract.options.address = receipt.contractAddress;
 				return newContract;
 			},
+			// TODO Should make this configurable by the user
+			checkRevertBeforeSending: false,
 		});
 	}
 

packages/web3-eth-contract/test/integration/contract_defaults_extra.test.ts

@@ -186,6 +186,7 @@ describe('contract defaults (extra)', () => {
 				}),
 				expect.any(Object),
 				expect.any(Object),
+				expect.any(Object),
 			);
 		});