chore(sequencer_node): separate config pointers to required and optional

commit-id:0c02cd67

crates/mempool_node/src/config/config_test.rs

@@ -22,6 +22,7 @@ use crate::config::{
     SequencerNodeConfig,
     CONFIG_POINTERS,
     DEFAULT_CONFIG_PATH,
+    REQUIRED_PARAM_CONFIG_POINTERS,
 };
 
 /// Test the validation of the struct ComponentExecutionConfig.
@@ -93,7 +94,7 @@ fn test_default_config_file_is_up_to_date() {
 /// Tests parsing a node config without additional args.
 #[test]
 fn test_config_parsing() {
-    let args = create_test_config_load_args(&CONFIG_POINTERS);
+    let args = create_test_config_load_args(&REQUIRED_PARAM_CONFIG_POINTERS);
     let config = SequencerNodeConfig::load_and_process(args);
     let config = config.expect("Parsing function failed.");
 

crates/mempool_node/src/config/node_config.rs

@@ -2,6 +2,7 @@ use std::collections::BTreeMap;
 use std::fs::File;
 use std::path::Path;
 use std::sync::LazyLock;
+use std::vec::Vec;
 
 use clap::Command;
 use papyrus_config::dumping::{
@@ -34,12 +35,30 @@ pub const DEFAULT_CONFIG_PATH: &str = "config/mempool/default_config.json";
 
 // Configuration parameters that share the same value across multiple components.
 pub const DEFAULT_CHAIN_ID: ChainId = ChainId::Mainnet;
+
+// Required target parameters.
+pub static REQUIRED_PARAM_CONFIG_POINTERS: LazyLock<Vec<(ParamPath, SerializedParam)>> =
+    LazyLock::new(|| {
+        vec![ser_pointer_target_required_param(
+            "chain_id",
+            SerializationType::String,
+            "The chain to follow.",
+        )]
+    });
+
+// TODO(Tsabary): Create a struct detailing all the required parameters and their types. Add a macro
+// that verifies the correctness of the required parameters compared to
+// REQUIRED_PARAM_CONFIG_POINTERS.
+
+// Optional target parameters, i.e., target parameters with default values.
+pub static DEFAULT_PARAM_CONFIG_POINTERS: LazyLock<Vec<(ParamPath, SerializedParam)>> =
+    LazyLock::new(Vec::new);
+
+// All target parameters.
 pub static CONFIG_POINTERS: LazyLock<Vec<(ParamPath, SerializedParam)>> = LazyLock::new(|| {
-    vec![ser_pointer_target_required_param(
-        "chain_id",
-        SerializationType::String,
-        "The chain to follow.",
-    )]
+    let mut combined = REQUIRED_PARAM_CONFIG_POINTERS.clone();
+    combined.extend(DEFAULT_PARAM_CONFIG_POINTERS.clone());
+    combined
 });
 
 // TODO(Tsabary): Bundle required config values in a struct, detailing whether they are pointer