Improve metrics hook setup

[mattsse]

Describe the feature

currently we do:

reth/crates/node/metrics/src/hooks.rs

Lines 24 to 26 in 3408059

	pub struct Hooks {
	inner: Arc<Vec<Box<dyn Hook<Output = ()>>>>,
	}

which makes the metrics crate depend on

reth/crates/node/metrics/src/hooks.rs

Line 4 in 3408059

use reth_provider::providers::StaticFileProvider;

we can improve this api with a builder API

HooksBuilder {
  Vec<Box<dyn Hook<Output = ()>>>
}

and replace the new function

reth/crates/node/metrics/src/hooks.rs

Line 30 in 3408059

pub fn new<Metrics, N>(db: Metrics, static_file_provider: StaticFileProvider<N>) -> Self

with something like

fn with(mut self, hook: impl Hook)

then we need to change how this type is instantiated and effectively push these:

reth/crates/node/metrics/src/hooks.rs

Lines 36 to 41 in 3408059

	Box::new(move || db.report_metrics()),
	Box::new(move || {
	let _ = static_file_provider.report_metrics().map_err(
	|error| tracing::error!(%error, "Failed to report static file provider metrics"),
	);
	}),

to the callsite

Additional context

No response

[nils-mathieu]

May I take this?