refactor(common): remove confusing aliased re-exports

Remove unused aliased re-exports from common/src/lib.rs:
- BarEventFromMarketData, MarketDataEventFromMarketData,
  OrderBookEventFromMarketData, QuoteEventFromMarketData,
  TradeEventFromMarketData (all unused, conflicted with types:: exports)
- ResilienceCircuitBreaker (unused alias for resilience::CircuitBreaker
  struct, conflicted with traits::CircuitBreaker trait)
- Also removes BarInterval and NewsEvent root re-exports (unused from
  root, accessible via common::market_data::*)

Users should import market_data types via common::market_data::* and
the resilience CircuitBreaker struct via common::resilience::CircuitBreaker.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

common/src/lib.rs

@@ -66,15 +66,8 @@ pub use traits::{
     Metrics, RateLimitStatus, RateLimited, Reloadable, Service,
 };
 
-pub use market_data::{
-    BarEvent as BarEventFromMarketData, BarInterval,
-    MarketDataEvent as MarketDataEventFromMarketData, NewsEvent,
-    OrderBookEvent as OrderBookEventFromMarketData, QuoteEvent as QuoteEventFromMarketData,
-    TradeEvent as TradeEventFromMarketData,
-};
-
-// Import market data types for canonical use
-// Use common::market_data::{MarketDataEvent, TradeEvent, QuoteEvent, BarEvent} etc.
+// Market data types: use common::market_data::{MarketDataEvent, TradeEvent, QuoteEvent, BarEvent, etc.}
+// (Not re-exported at crate root to avoid name collisions with types:: re-exports)
 pub mod trading;
 
 // Re-export shared ML strategy types
@@ -98,7 +91,6 @@ pub use features::{
 pub use resilience::{
     BoundedExecutor,
     BoundedExecutorError,
-    CircuitBreaker as ResilienceCircuitBreaker,
     CircuitBreakerConfig,
     CircuitBreakerState,
     retry_with_backoff,