- Remove duplicate checkpoint existence checks in RealTFTModel and
RealMamba2Model (each had two identical `!checkpoint_path.exists()`
guards — reduced to one)
- Move unused ModelPerformanceMetrics struct into test module (only
consumer; was generating dead_code warning)
- Restrict visibility of internal types to pub(crate): RuntimeModelInfo,
FeatureNormStats, FeaturePreprocessor, EnsembleConfig,
ModelPerformanceMetrics — none have external consumers
- Add #[allow(dead_code)] with documentation on RuntimeModelInfo fields
(model_id, fallback_priority) that are stored for Debug output and
future fallback ordering but not yet read in hot paths
- Remove emoji from PPO checkpoint log message for consistency
- No proto contract changes; all gRPC method signatures preserved
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
41b872ca5a