fix

bin/reth/src/args/rpc_server_args.rs

@@ -193,14 +193,15 @@ impl RpcServerArgs {
         let module_config = self.transport_rpc_module_config();
         debug!(target: "reth::cli", http=?module_config.http(), ws=?module_config.ws(), "Using RPC module config");
 
-        let (mut modules, auth_module, registry) = RpcModuleBuilder::default()
+        let (mut modules, auth_module, mut registry) = RpcModuleBuilder::default()
             .with_provider(components.provider())
             .with_pool(components.pool())
             .with_network(components.network())
             .with_events(components.events())
             .with_executor(components.task_executor())
             .build_with_auth_server(module_config, engine_api);
-        let node_modules = &mut RethRpcComponents::<Reth> { registry, modules: &mut modules };
+        let node_modules =
+            &mut RethRpcComponents::<Reth> { registry: &mut registry, modules: &mut modules };
         // apply configured customization
         conf.extend_rpc_modules(self, components, node_modules)?;
 

bin/reth/src/cli/components.rs

@@ -74,10 +74,16 @@ pub trait RethNodeComponents {
 
 /// Helper function to encapsulate [RethModuleRegistry] and [TransportRpcModules]
 /// generic over [RethNodeComponents]
+#[allow(clippy::type_complexity)]
 pub struct RethRpcComponents<'a, Reth: RethNodeComponents> {
     /// A Helper type the holds instances of the configured modules.
-    pub registry:
-        RethModuleRegistry<Reth::Provider, Reth::Pool, Reth::Network, Reth::Tasks, Reth::Events>,
+    pub registry: &'a mut RethModuleRegistry<
+        Reth::Provider,
+        Reth::Pool,
+        Reth::Network,
+        Reth::Tasks,
+        Reth::Events,
+    >,
     /// Holds installed modules per transport type.
     pub modules: &'a mut TransportRpcModules,
 }

bin/reth/src/cli/ext.rs

@@ -7,10 +7,6 @@ use crate::cli::{
 use clap::Args;
 use reth_basic_payload_builder::{BasicPayloadJobGenerator, BasicPayloadJobGeneratorConfig};
 use reth_payload_builder::{PayloadBuilderHandle, PayloadBuilderService};
-#[doc(hidden)]
-#[allow(unused)]
-/// For doc purposes
-use reth_rpc_builder::TransportRpcModules;
 use reth_tasks::TaskSpawner;
 use std::fmt;
 
@@ -52,23 +48,22 @@ pub trait RethNodeCommandConfig: fmt::Debug {
 
     /// Allows for registering additional RPC modules for the transports.
     ///
-    /// This is expected to call the merge functions of [TransportRpcModules], for example
-    /// [TransportRpcModules::merge_configured]
+    /// This is expected to call the merge functions of [reth_rpc_builder::TransportRpcModules], for
+    /// example [reth_rpc_builder::TransportRpcModules::merge_configured]
     #[allow(clippy::type_complexity)]
     fn extend_rpc_modules<Conf, Reth>(
         &mut self,
         config: &Conf,
         components: &Reth,
-        node_components: &mut RethRpcComponents<'_, Reth>,
+        rpc_components: &mut RethRpcComponents<'_, Reth>,
     ) -> eyre::Result<()>
     where
         Conf: RethRpcConfig,
         Reth: RethNodeComponents,
     {
         let _ = config;
         let _ = components;
-        let _ = node_components.registry;
-        let _ = node_components.modules;
+        let _ = rpc_components;
         Ok(())
     }