From e660f6933d181bb0d301f9c3bcf00c777863df78 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=C3=96mer=20Faruk=20IRMAK?= <omerfirmak@gmail.com>
Date: Mon, 2 Oct 2023 15:35:20 +0300
Subject: [PATCH] Update ContractError to have the expected data field

---
 rpc/handlers.go | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

diff --git a/rpc/handlers.go b/rpc/handlers.go
index f8bc9bb5b6..0eb7f1d87f 100644
--- a/rpc/handlers.go
+++ b/rpc/handlers.go
@@ -1077,13 +1077,21 @@ func (h *Handler) Call(call FunctionCall, id BlockID) ([]*felt.Felt, *jsonrpc.Er
 
 	res, err := h.vm.Call(&call.ContractAddress, &call.EntryPointSelector, call.Calldata, blockNumber, header.Timestamp, state, h.network)
 	if err != nil {
-		contractErr := *ErrContractError
-		contractErr.Data = err.Error()
-		return nil, &contractErr
+		return nil, makeContractError(err)
 	}
 	return res, nil
 }
 
+func makeContractError(err error) *jsonrpc.Error {
+	contractErr := *ErrContractError
+	contractErr.Data = struct {
+		RevertError string `json:"revert_error"`
+	}{
+		RevertError: err.Error(),
+	}
+	return &contractErr
+}
+
 func (h *Handler) TransactionStatus(ctx context.Context, hash felt.Felt) (*TransactionStatus, *jsonrpc.Error) {
 	var status *TransactionStatus
 
@@ -1279,9 +1287,7 @@ func (h *Handler) simulateTransactions(id BlockID, transactions []BroadcastedTra
 	overallFees, traces, err := h.vm.Execute(txns, classes, blockNumber, header.Timestamp, sequencerAddress,
 		state, h.network, paidFeesOnL1, skipFeeCharge, header.GasPrice, legacyTraceJSON)
 	if err != nil {
-		rpcErr := *ErrContractError
-		rpcErr.Data = err.Error()
-		return nil, &rpcErr
+		return nil, makeContractError(err)
 	}
 
 	var result []SimulatedTransaction
@@ -1378,9 +1384,7 @@ func (h *Handler) traceBlockTransactions(block *core.Block, numTxns int, legacyT
 	_, traces, err := h.vm.Execute(transactions, classes, blockNumber, header.Timestamp,
 		sequencerAddress, state, h.network, paidFeesOnL1, false, header.GasPrice, legacyTraceJSON)
 	if err != nil {
-		rpcErr := *ErrContractError
-		rpcErr.Data = err.Error()
-		return nil, &rpcErr
+		return nil, makeContractError(err)
 	}
 
 	var result []TracedBlockTransaction