fix: reduce log verbosity - default to warn, keep only important info

- Default log level: warn (was info/debug)
- validator_node and platform_bittensor: info (epoch/weight events)
- Silence noisy P2P consensus, WASM runtime, libp2p, tower_http
- Demote per-weight Burn UID log to debug
- Fix pre-existing clippy lints (derivable_impls, unnecessary_to_owned, matches)

Author: echobt

bins/validator-node/src/main.rs

@@ -353,7 +353,7 @@ async fn main() -> Result<()> {
     tracing_subscriber::registry()
         .with(tracing_subscriber::fmt::layer().with_filter(
             tracing_subscriber::EnvFilter::try_from_default_env().unwrap_or_else(|_| {
-                "info,validator_node=debug,platform_p2p_consensus=debug".into()
+                "warn,validator_node=info,platform_bittensor=info,platform_p2p_consensus=warn,wasm_runtime_interface=warn,libp2p_kad=error,libp2p_gossipsub=error,tower_http=warn".into()
             }),
         ))
         .with(sentry_tracing::layer().event_filter(|meta| {
@@ -1817,7 +1817,7 @@ async fn main() -> Result<()> {
                 // Write all consensus-reached proposals
                 for p in to_write {
                     let storage_key = StorageKey::new(
-                        &p.challenge_id.to_string(),
+                        p.challenge_id.as_ref(),
                         hex::encode(&p.key),
                     );
                     let result = if p.value.is_empty() {
@@ -3413,7 +3413,7 @@ async fn handle_network_event(
                             .apply(|state| state.remove_storage_proposal(&proposal.proposal_id));
                         if let Some(p) = proposal_opt {
                             let storage_key =
-                                StorageKey::new(&p.challenge_id.to_string(), hex::encode(&p.key));
+                                StorageKey::new(p.challenge_id.as_ref(), hex::encode(&p.key));
                             let result = if p.value.is_empty() {
                                 storage.delete(&storage_key).await
                             } else {
@@ -3499,7 +3499,7 @@ async fn handle_network_event(
                             // Use same key format as ChallengeStorageBackend:
                             // namespace = challenge_id.to_string(), key = hex::encode(key_bytes)
                             let storage_key = StorageKey::new(
-                                &proposal.challenge_id.to_string(),
+                                proposal.challenge_id.as_ref(),
                                 hex::encode(&proposal.key),
                             );
 
@@ -4313,21 +4313,13 @@ async fn handle_network_event(
 
                         // During bootstrap, always accept data from the bootstrap validator
                         // (it is the source of truth). Otherwise, only apply if newer.
-                        let should_apply = if in_bootstrap && from_bootstrap {
-                            true
-                        } else {
-                            match storage
-                                .get(&key, platform_distributed_storage::GetOptions::default())
-                                .await
-                            {
-                                Ok(Some(existing))
-                                    if existing.metadata.version >= entry.version =>
-                                {
-                                    false
-                                }
-                                _ => true,
-                            }
-                        };
+                        let should_apply = in_bootstrap && from_bootstrap
+                            || !matches!(
+                                storage
+                                    .get(&key, platform_distributed_storage::GetOptions::default())
+                                    .await,
+                                Ok(Some(existing)) if existing.metadata.version >= entry.version
+                            );
 
                         if should_apply {
                             if let Err(e) =
@@ -5063,7 +5055,7 @@ async fn handle_block_event(
                                         uids.push(0);
                                         vals.push(burn_u16);
                                     }
-                                    info!("  Burn (UID 0): {:.4} = {}", burn_weight, burn_u16);
+                                    debug!("  Burn (UID 0): {:.4} = {}", burn_weight, burn_u16);
                                 }
 
                                 if !uids.is_empty() {

crates/challenge-sdk/src/types.rs

@@ -5,7 +5,7 @@ use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 
 /// Unique challenge identifier — a human-readable string name.
-#[derive(Clone, PartialEq, Eq, Hash, Serialize, Deserialize)]
+#[derive(Clone, Default, PartialEq, Eq, Hash, Serialize, Deserialize)]
 pub struct ChallengeId(pub String);
 
 impl ChallengeId {
@@ -32,12 +32,6 @@ impl ChallengeId {
     }
 }
 
-impl Default for ChallengeId {
-    fn default() -> Self {
-        Self(String::new())
-    }
-}
-
 impl std::fmt::Debug for ChallengeId {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         write!(f, "Challenge({})", self.0)

crates/core/src/types.rs

@@ -63,7 +63,7 @@ impl fmt::Display for Hotkey {
 }
 
 /// Challenge ID — a human-readable string name (e.g. "bounty-challenge").
-#[derive(Clone, PartialEq, Eq, Hash, Serialize, Deserialize, PartialOrd, Ord)]
+#[derive(Clone, Default, PartialEq, Eq, Hash, Serialize, Deserialize, PartialOrd, Ord)]
 pub struct ChallengeId(pub String);
 
 impl ChallengeId {
@@ -84,12 +84,6 @@ impl ChallengeId {
     }
 }
 
-impl Default for ChallengeId {
-    fn default() -> Self {
-        Self(String::new())
-    }
-}
-
 impl fmt::Debug for ChallengeId {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         write!(f, "Challenge({})", self.0)

docker-compose.yml

@@ -27,7 +27,7 @@ services:
       - validator-data:/data
 
     environment:
-      - RUST_LOG=${RUST_LOG:-info,validator_node=debug}
+      - RUST_LOG=${RUST_LOG:-warn,validator_node=info,platform_bittensor=info,platform_p2p_consensus=warn,wasm_runtime_interface=warn,libp2p_kad=error,libp2p_gossipsub=error,tower_http=warn}
       - DATA_DIR=/data
       - VALIDATOR_SECRET_KEY=${VALIDATOR_SECRET_KEY}
       - SUBTENSOR_ENDPOINT=${SUBTENSOR_ENDPOINT:-wss://entrypoint-finney.opentensor.ai:443}

vendor/bittensor-rs/src/subtensor.rs

@@ -325,6 +325,7 @@ impl Subtensor {
     /// Subtensor uses epoch-granular commit-reveal:
     /// - Commits are accepted during the ENTIRE epoch
     /// - Reveals are accepted during the ENTIRE next epoch (epoch + reveal_period)
+    ///
     /// Returns: "commit" (always - entire epoch is a commit window)
     pub async fn get_current_phase(&self, _netuid: u16) -> Result<String> {
         Ok("commit".to_string())