Remove contracts dependency from ethrpc (cowprotocol#3796)

`ethrpc` only depends on 1 function from `contracts` which arguably
shouldn't have been re-exported to begin with since it's only needed for
`ethcontract` bindings.
Removing that function from `ethrpc` means it no longer depends on
`contracts` which allows `cargo` to compile it in parallel with
`contracts`.

Also where this `dummy` functionality lives become irrelevant when we
finally switch to `alloy` completely since `alloy` does not need these
dummy instances just to encode calldata. 🥳
- remove `contracts` dependency from `ethrpc`
compiler

Author: xdecentralix

Cargo.lock

@@ -79,7 +79,7 @@ fn to_interaction(
     output: &liquidity::ExactOutput,
     receiver: &eth::Address,
 ) -> eth::Interaction {
-    let web3 = ethrpc::dummy::web3();
+    let web3 = contracts::web3::dummy();
     let handler = balancer_v2::SettlementHandler::new(
         pool.id.into(),
         // Note that this code assumes `receiver == sender`. This assumption is

crates/driver/src/boundary/liquidity/balancer/v2/mod.rs

@@ -80,7 +80,7 @@ fn to_interaction(
     output: &liquidity::ExactOutput,
     receiver: &eth::Address,
 ) -> eth::Interaction {
-    let web3 = ethrpc::dummy::web3();
+    let web3 = contracts::web3::dummy();
     let handler = balancer_v3::SettlementHandler::new(
         pool.id.into(),
         // Note that this code assumes `receiver == sender`. This assumption is

crates/driver/src/boundary/liquidity/balancer/v3/mod.rs

@@ -99,7 +99,7 @@ pub fn to_interaction(
 ) -> eth::Interaction {
     let handler = uniswap_v2::Inner::new(
         pool.router.0.into_alloy(),
-        GPv2Settlement::at(&ethrpc::dummy::web3(), receiver.0),
+        GPv2Settlement::at(&contracts::web3::dummy(), receiver.0),
         Mutex::new(Allowances::empty(receiver.0)),
     );
 

crates/driver/src/boundary/liquidity/uniswap/v2.rs

@@ -78,7 +78,7 @@ pub fn to_interaction(
     output: &liquidity::ExactOutput,
     receiver: &eth::Address,
 ) -> eth::Interaction {
-    let web3 = ethrpc::dummy::web3();
+    let web3 = contracts::web3::dummy();
 
     let handler = UniswapV3SettlementHandler::new(
         UniswapV3SwapRouterV2::at(&web3, pool.router.0),

crates/driver/src/boundary/liquidity/uniswap/v3.rs

@@ -14,7 +14,6 @@ path = "src/lib.rs"
 alloy = { workspace = true, default-features = false, features = ["json-rpc", "providers", "rpc-client", "rpc-types", "transports", "reqwest", "signers", "signer-aws", "signer-local", "eips"] }
 anyhow = { workspace = true }
 async-trait = { workspace = true }
-contracts = { workspace = true }
 ethcontract = { workspace = true }
 futures = { workspace = true }
 hex = { workspace = true }

crates/ethrpc/Cargo.toml

@@ -1,7 +1,6 @@
 pub mod alloy;
 pub mod block_stream;
 pub mod buffered;
-pub mod dummy;
 pub mod extensions;
 pub mod http;
 pub mod instrumented;

crates/ethrpc/src/dummy.rs

@@ -79,7 +79,7 @@ fn to_interaction(
     output: &liquidity::ExactOutput,
     receiver: &eth::Address,
 ) -> eth::Interaction {
-    let web3 = ethrpc::dummy::web3();
+    let web3 = contracts::web3::dummy();
     let handler = balancer_v2::SettlementHandler::new(
         pool.id.into(),
         // Note that this code assumes `receiver == sender`. This assumption is

crates/ethrpc/src/lib.rs

@@ -80,7 +80,7 @@ fn to_interaction(
     output: &liquidity::ExactOutput,
     receiver: &eth::Address,
 ) -> eth::Interaction {
-    let web3 = ethrpc::dummy::web3();
+    let web3 = contracts::web3::dummy();
     let handler = balancer_v3::SettlementHandler::new(
         pool.id.into(),
         // Note that this code assumes `receiver == sender`. This assumption is