[0.13] Update transaction hash calculation (#1214)

* Add GasPricess & FeeTokenAddresses structs

* Update code

* RpcStateReader::get_gas_price fetch both

* Update code

* Clippy

* Fix comment

* Add VersionSpecificAccountTxFields struct

* Integrate into TransactionExecutionContext

* Replace max_fee field with account_tx_fields for InvokeFunction tx

* Check tx account fields version when creating Invoke tx

* Replace max_fee field with account_tx_fields for DeclareV3 tx

* Replace max_fee field with account_tx_fields for DeployAccount tx

* Update get_transaction_context for Txs not affected by this change

* Fixes

* Fixes

* Fixes

* Fix handling of ignore_max_fee

* Improve placeholder method

* Handle TODOs

* Implement get_onchain_data_cost

* Refactor

* Generalize fn

* Start implementing check_fee_bounds

* Finish implementing check_fee_bounds

* Replace old code

* Fixes

* Implement FeeType for VersionSpecificAccountTxFields

* Update other txs

* Clippy

* Add test

* Simplify

* Move test

* Implement hash_fee_related_fields

* Finish hash impl

* Simplify logic

* Rename current tx hash functions to deprecated

* Refactor comments

* Fix fn name

* refactor transaction_hash module

* Use constants for hash prefixes

* Implement calculate_deploy_account_transaction_hash

* Add a general calculate_deploy_account_transaction_hash for use in DeployAccount

* Implement calculate_declare_v2_transaction_hash

* Refactor code

* Implement calculate_invoke_transaction_hash

* Move code

* Clippy

* Handle possible overflow

* Clippy

* Fix constants

* Fix tx prefix

* Fix constants

* Add test for invoke hash calculation

* Add test

* Use default value instead of ignoring if resource bounds are missing

* Clippy

* fmt

Author: fmoletta

cli/src/main.rs

@@ -7,8 +7,12 @@ use starknet_in_rust::{
         contract_address::compute_deprecated_class_hash,
         errors::{contract_address_errors::ContractAddressError, state_errors::StateError},
         transaction_hash::{
-            calculate_declare_transaction_hash, calculate_deploy_transaction_hash,
-            calculate_transaction_hash_common, TransactionHashPrefix,
+            deprecated::{
+                deprecated_calculate_declare_transaction_hash,
+                deprecated_calculate_deploy_transaction_hash,
+                deprecated_calculate_transaction_hash_common,
+            },
+            TransactionHashPrefix,
         },
     },
     definitions::{
@@ -118,7 +122,7 @@ fn declare_parser(
         &CompiledClass::Deprecated(Arc::new(contract_class.clone())),
     )?;
 
-    let tx_hash = calculate_declare_transaction_hash(
+    let tx_hash = deprecated_calculate_declare_transaction_hash(
         &contract_class,
         Felt252::ZERO,
         &Address(0.into()),
@@ -146,7 +150,7 @@ fn deploy_parser(
     )?;
 
     cached_state.deploy_contract(Address(address), string_to_hash(&args.class_hash))?;
-    let tx_hash = calculate_deploy_transaction_hash(
+    let tx_hash = deprecated_calculate_deploy_transaction_hash(
         0.into(),
         &Address(address),
         &constructor_calldata,
@@ -210,7 +214,7 @@ fn invoke_parser(
     )?;
     cached_state.apply_state_update(&StateDiff::from_cached_state(transactional_state.cache())?)?;
 
-    let tx_hash = calculate_transaction_hash_common(
+    let tx_hash = deprecated_calculate_transaction_hash_common(
         TransactionHashPrefix::Invoke,
         *TRANSACTION_VERSION,
         &contract_address,