From 1879f148a960127ae6ad6c691aee8038a7f97a99 Mon Sep 17 00:00:00 2001
From: nadin-Starkware <nadin@starkware.co>
Date: Sun, 17 Nov 2024 20:05:47 +0200
Subject: [PATCH] refactor: restrict RemoteServers struct and field visibility

commit-id:143ae876
---
 crates/starknet_sequencer_node/src/servers.rs | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/crates/starknet_sequencer_node/src/servers.rs b/crates/starknet_sequencer_node/src/servers.rs
index f6626f248d..e0cafa304f 100644
--- a/crates/starknet_sequencer_node/src/servers.rs
+++ b/crates/starknet_sequencer_node/src/servers.rs
@@ -107,12 +107,11 @@ impl WrapperServers {
 }
 
 // Component servers that can run remotely.
-// TODO(Nadin): Remove pub from the struct and update the fields to be pub(crate).
-pub struct RemoteServers {
-    pub batcher: Option<Box<RemoteBatcherServer>>,
-    pub gateway: Option<Box<RemoteGatewayServer>>,
-    pub mempool: Option<Box<RemoteMempoolServer>>,
-    pub mempool_p2p_propagator: Option<Box<RemoteMempoolP2pPropagatorServer>>,
+struct RemoteServers {
+    pub(crate) batcher: Option<Box<RemoteBatcherServer>>,
+    pub(crate) gateway: Option<Box<RemoteGatewayServer>>,
+    pub(crate) mempool: Option<Box<RemoteMempoolServer>>,
+    pub(crate) mempool_p2p_propagator: Option<Box<RemoteMempoolP2pPropagatorServer>>,
 }
 
 impl RemoteServers {
@@ -329,7 +328,7 @@ fn create_local_servers(
     }
 }
 
-pub fn create_remote_servers(
+fn create_remote_servers(
     config: &SequencerNodeConfig,
     clients: &SequencerNodeClients,
 ) -> RemoteServers {