refactor: replace sp1,risc0 features for .env variables

Author: MarcosNicolau

aggregation_mode/Cargo.toml

@@ -40,6 +40,4 @@ opt-level = 3
 [features]
 default = []
 prove = []
-sp1 = []
-risc0 = []
 gpu = ["risc0-zkvm/cuda"]

aggregation_mode/src/aggregators/mod.rs

@@ -23,18 +23,16 @@ impl Display for ZKVMEngine {
 }
 
 impl ZKVMEngine {
-    pub fn from_rust_features() -> Option<Self> {
-        #[cfg(feature = "sp1")]
-        {
-            return Some(ZKVMEngine::SP1);
-        }
-
-        #[cfg(feature = "risc0")]
-        {
-            return Some(ZKVMEngine::RISC0);
-        }
+    pub fn from_env() -> Option<Self> {
+        let key = "AGGREGATOR";
+        let value = std::env::var(key).ok()?;
+        let engine = match value.as_str() {
+            "sp1" => ZKVMEngine::SP1,
+            "risc0" => ZKVMEngine::RISC0,
+            _ => panic!("Invalid AGGREGATOR, possible options are: sp1|risc0"),
+        };
 
-        None
+        Some(engine)
     }
 }
 

aggregation_mode/src/backend/mod.rs

@@ -64,7 +64,8 @@ impl ProofAggregator {
             rpc_provider,
         );
 
-        let engine = ZKVMEngine::from_rust_features().expect("A feature defining zkvm engine");
+        let engine =
+            ZKVMEngine::from_env().expect("AGGREGATOR env variable to be set to one of sp1|risc0");
         let fetcher = ProofsFetcher::new(config);
 
         Self {
@@ -75,7 +76,7 @@ impl ProofAggregator {
     }
 
     pub async fn start(&mut self, config: &Config) {
-        info!("Starting proof aggregator service",);
+        info!("Starting proof aggregator service");
 
         info!("About to aggregate and submit proof to be verified on chain");
         let res = self.aggregate_and_submit_proofs_on_chain().await;