Ethereum RPC Data Providers

client/consensus/src/mocks.rs

@@ -819,7 +819,7 @@ impl MockRuntimeApi {
                             if let Some(para_id) = self.0 {
                                 let _ =
                                     sender.send(Ok(vec![CoreState::Scheduled(ScheduledCore {
-                                        para_id: para_id,
+                                        para_id,
                                         collator: None,
                                     })]));
                             } else {
@@ -888,7 +888,7 @@ impl CollatorLookaheadTestBuilder {
         CancellationToken,
     ) {
         // Creation of keystore
-        let _ = sp_tracing::try_init_simple();
+        sp_tracing::try_init_simple();
         let keystore_path = tempfile::tempdir().expect("Creates keystore path");
         let keystore = LocalKeystore::open(keystore_path.path(), None).expect("Creates keystore.");
         let alice_public = keystore

client/consensus/src/tests.rs

@@ -153,7 +153,7 @@ async fn claim_slot_respects_min_slot_freq() {
 #[tokio::test]
 async fn collate_returns_correct_block() {
     let net = AuraTestNet::new(4);
-    let _ = sp_tracing::try_init_simple();
+    sp_tracing::try_init_simple();
 
     let keystore_path = tempfile::tempdir().expect("Creates keystore path");
     let keystore = LocalKeystore::open(keystore_path.path(), None).expect("Creates keystore.");

client/node-common/Cargo.toml

@@ -23,6 +23,7 @@ serde = { workspace = true, features = [ "derive" ] }
 
 # Local
 tc-consensus = { workspace = true }
+tp-traits = { workspace = true }
 
 # Nimbus
 nimbus-consensus = { workspace = true }

client/node-common/src/service.rs

@@ -65,6 +65,36 @@ use {
 #[allow(deprecated)]
 use sc_executor::NativeElseWasmExecutor;
 
+tp_traits::alias!(
+    pub trait MinimalRuntimeApi<
+        Block: (cumulus_primitives_core::BlockT),
+        Client: (sp_api::CallApiAt<Block>),
+    > :
+        ConstructRuntimeApi<
+            Block,
+            Client,
+            RuntimeApi:
+                TaggedTransactionQueue<Block>
+                + BlockBuilder<Block> + OffchainWorkerApi<Block>
+                + sp_api::Metadata<Block>
+                + sp_session::SessionKeys<Block>,
+        > + Send + Sync + 'static
+);
+
+tp_traits::alias!(
+    pub trait MinimalCumulusRuntimeApi<
+        Block: (cumulus_primitives_core::BlockT),
+        Client: (sp_api::CallApiAt<Block>),
+    > :
+        MinimalRuntimeApi<Block, Client> +
+        ConstructRuntimeApi<
+            Block,
+            Client,
+            RuntimeApi:
+                cumulus_primitives_core::CollectCollationInfo<Block>,
+        >
+);
+
 /// Trait to configure the main types the builder rely on, bundled in a single
 /// type to reduce verbosity and the amount of type parameters.
 pub trait NodeBuilderConfig {
@@ -83,10 +113,7 @@ pub trait NodeBuilderConfig {
         BlockOf<Self>: cumulus_primitives_core::BlockT,
         ExecutorOf<Self>:
             Clone + CodeExecutor + RuntimeVersionOf + TanssiExecutorExt + Sync + Send + 'static,
-        RuntimeApiOf<Self>:
-            ConstructRuntimeApi<BlockOf<Self>, ClientOf<Self>> + Sync + Send + 'static,
-        ConstructedRuntimeApiOf<Self>:
-            TaggedTransactionQueue<BlockOf<Self>> + BlockBuilder<BlockOf<Self>>,
+        RuntimeApiOf<Self>: MinimalRuntimeApi<BlockOf<Self>, ClientOf<Self>>,
     {
         NodeBuilder::<Self>::new(parachain_config, hwbench)
     }
@@ -137,8 +164,7 @@ pub struct NodeBuilder<
 > where
     BlockOf<T>: cumulus_primitives_core::BlockT,
     ExecutorOf<T>: Clone + CodeExecutor + RuntimeVersionOf + Sync + Send + 'static,
-    RuntimeApiOf<T>: ConstructRuntimeApi<BlockOf<T>, ClientOf<T>> + Sync + Send + 'static,
-    ConstructedRuntimeApiOf<T>: TaggedTransactionQueue<BlockOf<T>> + BlockBuilder<BlockOf<T>>,
+    RuntimeApiOf<T>: MinimalRuntimeApi<BlockOf<T>, ClientOf<T>>,
 {
     pub client: Arc<ClientOf<T>>,
     pub backend: Arc<BackendOf<T>>,
@@ -198,8 +224,7 @@ where
     BlockOf<T>: cumulus_primitives_core::BlockT,
     ExecutorOf<T>:
         Clone + CodeExecutor + RuntimeVersionOf + TanssiExecutorExt + Sync + Send + 'static,
-    RuntimeApiOf<T>: ConstructRuntimeApi<BlockOf<T>, ClientOf<T>> + Sync + Send + 'static,
-    ConstructedRuntimeApiOf<T>: TaggedTransactionQueue<BlockOf<T>> + BlockBuilder<BlockOf<T>>,
+    RuntimeApiOf<T>: MinimalRuntimeApi<BlockOf<T>, ClientOf<T>>,
 {
     /// Create a new `NodeBuilder` which prepare objects required to launch a
     /// node. However it only starts telemetry, and doesn't provide any
@@ -293,10 +318,7 @@ impl<T: NodeBuilderConfig, SNetwork, STxHandler, SImportQueueService>
 where
     BlockOf<T>: cumulus_primitives_core::BlockT,
     ExecutorOf<T>: Clone + CodeExecutor + RuntimeVersionOf + Sync + Send + 'static,
-    RuntimeApiOf<T>: ConstructRuntimeApi<BlockOf<T>, ClientOf<T>> + Sync + Send + 'static,
-    ConstructedRuntimeApiOf<T>: TaggedTransactionQueue<BlockOf<T>>
-        + BlockBuilder<BlockOf<T>>
-        + cumulus_primitives_core::CollectCollationInfo<BlockOf<T>>,
+    RuntimeApiOf<T>: MinimalCumulusRuntimeApi<BlockOf<T>, ClientOf<T>>,
 {
     pub async fn build_relay_chain_interface(
         &mut self,

client/service-container-chain/Cargo.toml

@@ -31,13 +31,15 @@ pallet-data-preservers = { workspace = true, features = [ "std" ] }
 services-payment-rpc = { workspace = true }
 stream-payment-rpc = { workspace = true }
 tc-consensus = { workspace = true }
+tp-traits = { workspace = true }
 
 # Dancekit
 dc-orchestrator-chain-interface = { workspace = true }
 dp-container-chain-genesis-data = { workspace = true, features = [ "json", "std" ] }
 dp-slot-duration-runtime-api = { workspace = true }
 
 # Substrate
+frame-support = { workspace = true }
 sc-basic-authorship = { workspace = true }
 sc-chain-spec = { workspace = true }
 sc-cli = { workspace = true }
@@ -55,6 +57,7 @@ sc-transaction-pool = { workspace = true }
 sc-transaction-pool-api = { workspace = true }
 sp-blockchain = { workspace = true }
 sp-consensus = { workspace = true }
+sp-transaction-pool = { workspace = true }
 substrate-frame-rpc-system = { workspace = true }
 substrate-prometheus-endpoint = { workspace = true }
 
@@ -79,6 +82,7 @@ cumulus-primitives-core = { workspace = true }
 cumulus-relay-chain-interface = { workspace = true }
 
 # Nimbus
+async-backing-primitives = { workspace = true }
 nimbus-consensus = { workspace = true }
 nimbus-primitives = { workspace = true }
 

client/service-container-chain/src/monitor.rs

@@ -16,11 +16,13 @@
 
 use {
     crate::{
-        service::{ContainerChainBackend, ContainerChainClient},
+        service::{ContainerChainBackend},
         spawner::{CcSpawnMsg, ContainerChainSpawnerState},
     },
     cumulus_primitives_core::ParaId,
+    frame_support::DefaultNoBound,
     std::{
+        any::Any,
         cell::Cell,
         collections::VecDeque,
         sync::{Arc, Mutex},
@@ -32,7 +34,7 @@ use {
     },
 };
 
-#[derive(Default)]
+#[derive(DefaultNoBound)]
 pub struct SpawnedContainersMonitor {
     /// List of the N most recently started container chains, with some statistics related to
     /// stopping time and reference count.
@@ -59,7 +61,7 @@ pub struct SpawnedContainer {
     /// Used to check the reference count, if it's 0 it means the database has been closed
     pub backend: std::sync::Weak<ContainerChainBackend>,
     /// Used to check the reference count, if it's 0 it means that the client has been closed.
-    pub client: std::sync::Weak<ContainerChainClient>,
+    pub client: std::sync::Weak<dyn Any + Send + Sync>,
 }
 
 impl SpawnedContainer {
@@ -295,7 +297,7 @@ mod tests {
             stop_task_manager_time: Default::default(),
             stop_refcount_time: Default::default(),
             backend: Default::default(),
-            client: Default::default(),
+            client: std::sync::Weak::<()>::new(),
         };
 
         // Truncating empty list does not panic