fix: Restrict scope of most utils to pub(crate)...

...so that they can only be used from within the crate.
This way we can reduce the interface, resulting in fewer breaking changes if we were to modify these util methods.

src/evm/protocol/erc20.rs

@@ -6,7 +6,7 @@ use tycho_core::Bytes;
 ///
 /// Checks the length of the `Bytes` before attempting to convert, and returns a `SimulationError`
 /// if not 20 bytes long.
-pub fn bytes_to_erc20_address(address: &Bytes) -> Result<Address, SimulationError> {
+pub(crate) fn bytes_to_erc20_address(address: &Bytes) -> Result<Address, SimulationError> {
     if address.len() == 20 {
         Ok(Address::from_slice(address))
     } else {

src/evm/protocol/mod.rs

@@ -1,4 +1,4 @@
-pub mod erc20;
+pub(crate) mod erc20;
 pub mod safe_math;
 pub mod u256_num;
 pub mod uniswap_v2;

src/evm/protocol/vm/erc20_token.rs

@@ -33,9 +33,9 @@ impl ERC20Slots {
     }
 }
 
-pub type Overwrites = HashMap<SlotId, U256>;
+pub(crate) type Overwrites = HashMap<SlotId, U256>;
 
-pub struct ERC20OverwriteFactory {
+pub(crate) struct ERC20OverwriteFactory {
     token_address: Address,
     overwrites: Overwrites,
     balance_slot: SlotId,
@@ -132,7 +132,7 @@ type U256Return = U256;
 ///   testing both compiler configurations.
 /// - Once the balance slot is found, it uses the detected compiler to search for the allowance
 ///   slot, which is dependent on the balance slot.
-pub fn brute_force_slots<D: EngineDatabaseInterface + Clone + Debug>(
+pub(crate) fn brute_force_slots<D: EngineDatabaseInterface + Clone + Debug>(
     token_addr: &Address,
     block: &BlockHeader,
     engine: &SimulationEngine<D>,

src/evm/protocol/vm/mod.rs

@@ -6,4 +6,4 @@ pub mod state;
 pub mod state_builder;
 pub mod tycho_decoder;
 mod tycho_simulation_contract;
-pub mod utils;
+pub(crate) mod utils;