apollo_proc_macros: re-export metrics crate for downstream usage (#13849)

Author: dorimedini-starkware

Cargo.lock

@@ -24,7 +24,6 @@ futures.workspace = true
 futures-util.workspace = true
 indexmap = { workspace = true, features = ["serde"] }
 lru.workspace = true
-metrics.workspace = true
 papyrus_base_layer.workspace = true
 papyrus_common.workspace = true
 reqwest = { workspace = true, features = ["blocking", "json"] }
@@ -42,13 +41,9 @@ assert_matches.workspace = true
 mockall.workspace = true
 pretty_assertions.workspace = true
 simple_logger.workspace = true
-starknet_api = { workspace = true, features = ["testing"] }
 starknet-types-core.workspace = true
+starknet_api = { workspace = true, features = ["testing"] }
 tokio-stream.workspace = true
 
-[package.metadata.cargo-machete]
-# `metrics` is used in `latency_histogram` but is falsely detected as unused.
-ignored = ["metrics"]
-
 [lints]
 workspace = true

crates/apollo_central_sync/Cargo.toml

@@ -8,8 +8,8 @@ description = "Peer-to-peer state synchronization for Starknet nodes."
 
 [dependencies]
 apollo_class_manager_types.workspace = true
-apollo_p2p_sync_config.workspace = true
 apollo_network.workspace = true
+apollo_p2p_sync_config.workspace = true
 apollo_proc_macros.workspace = true
 apollo_protobuf.workspace = true
 apollo_state_sync_metrics.workspace = true
@@ -23,7 +23,6 @@ enum-iterator.workspace = true
 futures.workspace = true
 indexmap.workspace = true
 lazy_static.workspace = true
-metrics.workspace = true
 papyrus_common.workspace = true
 rand.workspace = true
 rand_chacha.workspace = true
@@ -45,9 +44,5 @@ mockall.workspace = true
 static_assertions.workspace = true
 tokio = { workspace = true, features = ["test-util"] }
 
-# The `metrics` crate is used by `latency_histogram` proc macro, which is used in this crate.
-[package.metadata.cargo-machete]
-ignored = ["metrics"]
-
 [lints]
 workspace = true

crates/apollo_p2p_sync/Cargo.toml

@@ -8,6 +8,7 @@ description = "Wrapper crate for the apollo_proc_macros_lib"
 
 [dependencies]
 apollo_proc_macros_lib.workspace = true
+metrics.workspace = true
 
 [lints]
 workspace = true

crates/apollo_proc_macros/Cargo.toml

@@ -6,3 +6,5 @@ use std::sync::OnceLock;
 pub static COLLECT_PROFILING_METRICS: OnceLock<bool> = OnceLock::new();
 
 pub use apollo_proc_macros_lib::*;
+// Re-export for use in downstream crates.
+pub use metrics;

crates/apollo_proc_macros/src/lib.rs

@@ -168,7 +168,7 @@ pub fn latency_histogram(attr: TokenStream, input: TokenStream) -> TokenStream {
     // instead of boolean
 
     let metric_recording_logic = quote! {
-        ::metrics::histogram!(#metric_name).record(exec_time);
+        apollo_proc_macros::metrics::histogram!(#metric_name).record(exec_time);
     };
 
     let collect_metric_flag = quote! {