ghostlink_core/

health.rs

//! Network Health Monitoring for Ghost-Link Cluster
//!
//! This module provides:
//! - Ping/pong latency tracking per node
//! - Delivery ratio monitoring
//! - Automatic quantization fallback triggers
//! - Fault detection and recovery
//! - Dynamic reconfiguration on hardware changes via `SystemProfileWatcher`.

use std::collections::{HashMap, VecDeque};
use std::net::{SocketAddr, TcpStream};
use std::sync::{Arc, Mutex};
use std::time::{Duration, Instant};

use crate::cluster::{ClusterState, NodeStatus};
use crate::host::{AccelerationMode, RuntimeProfile};
use crate::system_profile::SystemProfile;
use crate::watcher::ProfileChange;

/// Health check configuration
#[derive(Clone, Copy, Debug)]
pub struct HealthConfig {
    /// Interval between health checks
    pub check_interval: Duration,
    /// Timeout for health check response
    pub timeout: Duration,
    /// Minimum number of successful checks before considering node healthy
    pub min_successes: usize,
    /// Maximum allowed failures before marking node degraded
    pub max_failures: usize,
    /// Upper latency bound for a healthy node.
    pub healthy_latency_us: f32,
    /// Upper latency bound for a degraded-but-usable node.
    pub degraded_latency_us: f32,
    /// Minimum delivery ratio for a healthy node.
    pub healthy_delivery_ratio: f32,
    /// Minimum delivery ratio for a degraded node.
    pub degraded_delivery_ratio: f32,
}

impl Default for HealthConfig {
    fn default() -> Self {
        Self {
            check_interval: Duration::from_secs(5),
            timeout: Duration::from_secs(3),
            min_successes: 2,
            max_failures: 3,
            healthy_latency_us: 10.0,
            degraded_latency_us: 20.0,
            healthy_delivery_ratio: 0.95,
            degraded_delivery_ratio: 0.80,
        }
    }
}

impl HealthConfig {
    /// Tune health thresholds from the detected runtime profile.
    pub fn autotuned(profile: &RuntimeProfile) -> Self {
        let (healthy_latency_us, degraded_latency_us, timeout, max_failures) =
            match profile.acceleration_mode {
                AccelerationMode::Gpu => (6.0, 14.0, Duration::from_secs(2), 4),
                AccelerationMode::Avx512 => (8.0, 18.0, Duration::from_secs(2), 3),
                _ => (10.0, 20.0, Duration::from_secs(3), 3),
            };

        Self {
            check_interval: Duration::from_secs(5),
            timeout,
            min_successes: if profile.recommended_workers >= 8 {
                3
            } else {
                2
            },
            max_failures,
            healthy_latency_us,
            degraded_latency_us,
            healthy_delivery_ratio: if profile.node_resources.vram_gb > 0.0 {
                0.97
            } else {
                0.95
            },
            degraded_delivery_ratio: if profile.node_resources.vram_gb > 0.0 {
                0.88
            } else {
                0.80
            },
        }
    }
}

/// Health status for a node
#[derive(Clone, Copy, Debug, Default, PartialEq, Eq)]
pub enum HealthStatus {
    /// Node is healthy
    Healthy,
    /// Node is degraded (performance issues)
    Degraded,
    /// Node has failed
    Failed,
    /// No health data available yet
    #[default]
    Unknown,
}

/// Health check result
#[derive(Clone, Debug)]
pub struct HealthCheckResult {
    /// Node ID
    pub node_id: String,
    /// Latency in microseconds
    pub latency_us: f32,
    /// Delivery ratio (0.0 to 1.0)
    pub delivery_ratio: f32,
    /// Status
    pub status: HealthStatus,
    /// Timestamp of check
    pub timestamp: Instant,
}

/// Network health monitor
#[derive(Clone, Debug)]
pub struct NetworkHealthMonitor {
    /// Cluster state
    cluster: Arc<ClusterState>,
    /// Configuration (behind Mutex for atomic re-tune from watcher)
    config: Arc<Mutex<HealthConfig>>,
    /// Last check timestamp
    last_check: Arc<Mutex<Option<Instant>>>,
    /// Recent check results per node
    recent_checks: Arc<Mutex<HashMap<String, VecDeque<HealthCheckResult>>>>,
    /// Optional per-node TCP probe targets for active liveness checks.
    tcp_probe_targets: Arc<Mutex<HashMap<String, SocketAddr>>>,
}

impl NetworkHealthMonitor {
    /// Create new health monitor
    pub fn new(cluster: Arc<ClusterState>, config: HealthConfig) -> Self {
        Self {
            cluster,
            config: Arc::new(Mutex::new(config)),
            last_check: Arc::new(Mutex::new(None)),
            recent_checks: Arc::new(Mutex::new(HashMap::new())),
            tcp_probe_targets: Arc::new(Mutex::new(HashMap::new())),
        }
    }

    /// Create a monitor with thresholds derived from runtime auto-detection.
    pub fn with_runtime_profile(cluster: Arc<ClusterState>, profile: &RuntimeProfile) -> Self {
        Self::new(cluster, HealthConfig::autotuned(profile))
    }

    /// Read the current configuration.
    pub fn config(&self) -> HealthConfig {
        self.config
            .lock()
            .ok()
            .as_deref()
            .copied()
            .unwrap_or_default()
    }

    /// Register a node-specific TCP probe target.
    pub fn register_tcp_probe_target(&self, node_id: impl Into<String>, target: SocketAddr) {
        self.tcp_probe_targets
            .lock()
            .unwrap_or_else(|poison| poison.into_inner())
            .insert(node_id.into(), target);
    }

    /// Remove a TCP probe target for a node.
    pub fn clear_tcp_probe_target(&self, node_id: &str) -> bool {
        self.tcp_probe_targets
            .lock()
            .unwrap_or_else(|poison| poison.into_inner())
            .remove(node_id)
            .is_some()
    }

    /// Run health check on all nodes
    pub fn check_all(&self) {
        let cfg = self.config();
        let now = Instant::now();
        let nodes_snapshot = self.cluster.nodes_snapshot();

        // Phase 1: Collect probe target addresses under a brief lock read
        let targets: Vec<(String, SocketAddr)> = {
            let probe_targets = self
                .tcp_probe_targets
                .lock()
                .unwrap_or_else(|poison| poison.into_inner());
            nodes_snapshot
                .iter()
                .filter_map(|node| {
                    probe_targets
                        .get(&node.id)
                        .map(|addr| (node.id.clone(), *addr))
                })
                .collect()
        };

        // Phase 2: Execute active TCP probes OUTSIDE of any state locks
        let mut probe_results = HashMap::with_capacity(targets.len());
        for (node_id, addr) in targets {
            let ok = TcpStream::connect_timeout(&addr, cfg.timeout).is_ok();
            probe_results.insert(node_id, ok);
        }

        // Phase 3: Acquire cluster metrics and recent_checks locks ONCE to update state
        let mut metrics_guard = self
            .cluster
            .metrics
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());

        let mut checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());

        for node in nodes_snapshot.iter() {
            let tcp_probe_ok = probe_results.get(&node.id).copied();

            let result = if let Some(m) = metrics_guard.get_mut(&node.id) {
                let timeout =
                    now.saturating_duration_since(m.last_heartbeat) >= m.heartbeat_timeout;
                let latency_us = if m.latency_samples > 0 {
                    m.avg_latency_us
                } else {
                    0.0
                };

                let delivery_ratio = if m.delivery_ratio_initialized {
                    m.delivery_ratio
                } else {
                    1.0
                };

                let status =
                    if tcp_probe_ok == Some(false) || timeout || m.status == NodeStatus::Failed {
                        HealthStatus::Failed
                    } else if m.latency_samples == 0 && !m.delivery_ratio_initialized {
                        HealthStatus::Unknown
                    } else {
                        self.get_health_status(latency_us, delivery_ratio)
                    };

                let measured_latency_us = if tcp_probe_ok == Some(false) {
                    cfg.timeout.as_secs_f32() * 1_000_000.0
                } else {
                    latency_us
                };

                // Keep cluster node status aligned with health checks directly in place
                m.status = match status {
                    HealthStatus::Healthy => NodeStatus::Active,
                    HealthStatus::Degraded => NodeStatus::Degraded,
                    HealthStatus::Failed => NodeStatus::Failed,
                    HealthStatus::Unknown => m.status,
                };

                HealthCheckResult {
                    node_id: node.id.clone(),
                    latency_us: measured_latency_us,
                    delivery_ratio,
                    status,
                    timestamp: now,
                }
            } else {
                HealthCheckResult {
                    node_id: node.id.clone(),
                    latency_us: 0.0,
                    delivery_ratio: 1.0,
                    status: HealthStatus::Unknown,
                    timestamp: now,
                }
            };

            // Store recent check results in O(1) ring buffer (keep last 10)
            let node_checks = checks.entry(node.id.clone()).or_default();
            node_checks.push_back(result);
            if node_checks.len() > 10 {
                node_checks.pop_front();
            }
        }

        *self
            .last_check
            .lock()
            .unwrap_or_else(|poison| poison.into_inner()) = Some(now);
    }

    /// Get health status based on metrics.
    ///
    /// Degraded requires BOTH metrics to still be within their (looser)
    /// degraded floor — using OR here previously meant a node was only ever
    /// classified Failed when *both* metrics were simultaneously catastrophic,
    /// since either one alone being within its floor was enough to satisfy the
    /// OR. A node with fine delivery but catastrophic latency (or vice versa)
    /// could never reach Failed. Either metric crossing its floor should be
    /// enough to fail the node.
    fn get_health_status(&self, latency_us: f32, delivery_ratio: f32) -> HealthStatus {
        let cfg = self.config();
        if delivery_ratio >= cfg.healthy_delivery_ratio && latency_us <= cfg.healthy_latency_us {
            HealthStatus::Healthy
        } else if delivery_ratio >= cfg.degraded_delivery_ratio
            && latency_us <= cfg.degraded_latency_us
        {
            HealthStatus::Degraded
        } else {
            HealthStatus::Failed
        }
    }

    /// Get health report for a specific node
    pub fn get_node_health(&self, node_id: &str) -> Option<HealthCheckResult> {
        let checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());
        checks
            .get(node_id)
            .and_then(|checks| checks.back())
            .cloned()
    }

    /// Get health report for all nodes
    pub fn get_all_health(&self) -> Vec<HealthCheckResult> {
        let checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());
        checks
            .values()
            .flat_map(|checks| checks.iter().cloned())
            .collect()
    }

    /// Get the number of nodes whose latest health sample is healthy.
    pub fn healthy_node_count(&self) -> usize {
        let checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());
        checks
            .values()
            .filter(|node_checks| {
                node_checks
                    .back()
                    .map(|c| c.status == HealthStatus::Healthy)
                    .unwrap_or(false)
            })
            .count()
    }

    /// Check if node needs quantization fallback
    pub fn needs_quantization_fallback(&self, node_id: &str) -> bool {
        let cfg = self.config();
        let checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());

        if let Some(node_checks) = checks.get(node_id) {
            // Check last 3 results for consistent degradation
            let recent: Vec<_> = node_checks.iter().rev().take(3).collect();

            if recent.len() < 3 {
                return false;
            }

            // Calculate average delivery ratio of last 3 checks
            let avg_delivery_ratio: f32 =
                recent.iter().map(|r| r.delivery_ratio).sum::<f32>() / 3.0;

            // Check average latency
            let avg_latency_us: f32 = recent.iter().map(|r| r.latency_us).sum::<f32>() / 3.0;

            // Need fallback if delivery ratio dropped below threshold or latency increased significantly
            avg_delivery_ratio < cfg.healthy_delivery_ratio
                || avg_latency_us > cfg.healthy_latency_us
        } else {
            false
        }
    }

    /// Get cluster-wide health summary
    pub fn get_health_summary(&self) -> String {
        let checks = self
            .recent_checks
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());

        let total_nodes = self.cluster.node_count();
        let healthy_count = checks
            .values()
            .filter(|node_checks| {
                node_checks
                    .back()
                    .map(|c| c.status == HealthStatus::Healthy)
                    .unwrap_or(false)
            })
            .count();

        let degraded_count = checks
            .values()
            .filter(|node_checks| {
                node_checks
                    .back()
                    .map(|c| c.status == HealthStatus::Degraded)
                    .unwrap_or(false)
            })
            .count();

        let failed_count = checks
            .values()
            .filter(|node_checks| {
                node_checks
                    .back()
                    .map(|c| c.status == HealthStatus::Failed)
                    .unwrap_or(false)
            })
            .count();

        format!(
            "Network Health Summary\n\
             =================\n\
             Total nodes: {total_nodes}\n\
             Healthy: {healthy_count}\n\
             Degraded: {degraded_count}\n\
             Failed: {failed_count}\n"
        )
    }

    /// Start periodic health checks in background
    pub fn start_periodic_checks(&self) {
        let check_interval = self.config().check_interval;
        let this = self.clone();

        std::thread::spawn(move || loop {
            this.check_all();
            std::thread::sleep(check_interval);
        });
    }

    // ------------------------------------------------------------------
    // SystemProfileWatcher integration
    // ------------------------------------------------------------------

    /// Re-tune health thresholds from a system profile detected at runtime.
    ///
    /// This is the main integration point with `SystemProfileWatcher`: when
    /// hardware changes (GPU hot-plug, CPU scaling), call this to keep the
    /// health monitor's expectations aligned with reality.
    pub fn reconfigure_from_system_profile(&self, profile: &SystemProfile) {
        let rp: RuntimeProfile = profile.into();
        if let Ok(mut guard) = self.config.lock() {
            *guard = HealthConfig::autotuned(&rp);
        }
    }

    /// Subscribe to a `SystemProfileWatcher`'s broadcast channel and
    /// automatically re-tune whenever hardware changes are detected.
    ///
    /// Returns a `JoinHandle` that can be cancelled by dropping it.
    pub fn subscribe_to_watcher(
        self: &Arc<Self>,
        watcher: &crate::watcher::SystemProfileWatcher,
    ) -> tokio::task::JoinHandle<()> {
        let this = self.clone();
        let mut rx = watcher.subscribe();
        tokio::spawn(async move {
            while let Ok(event) = rx.recv().await {
                if let ProfileChange::Updated(profile) = event {
                    this.reconfigure_from_system_profile(&profile);
                    tracing::info!(
                        "health monitor re-tuned from profile: {} workers, {:?} accel",
                        profile.recommended_workers,
                        profile.acceleration_mode,
                    );
                }
            }
        })
    }
}

/// Health metrics aggregation
#[derive(Clone, Debug, Default)]
pub struct HealthMetrics {
    /// Average latency across cluster
    pub avg_latency_us: f32,
    /// Minimum observed latency
    pub min_latency_us: f32,
    /// Maximum observed latency
    pub max_latency_us: f32,
    /// Average delivery ratio
    pub avg_delivery_ratio: f32,
    /// Minimum delivery ratio
    pub min_delivery_ratio: f32,
    /// Number of samples
    pub sample_count: usize,
}

impl HealthMetrics {
    /// Create new metrics aggregator
    pub fn new() -> Self {
        Self::default()
    }

    /// Record a health check result
    pub fn record(&mut self, latency_us: f32, delivery_ratio: f32) {
        if self.sample_count == 0 {
            self.min_latency_us = latency_us;
            self.max_latency_us = latency_us;
            self.min_delivery_ratio = delivery_ratio;
        } else {
            if latency_us < self.min_latency_us {
                self.min_latency_us = latency_us;
            }
            if latency_us > self.max_latency_us {
                self.max_latency_us = latency_us;
            }
            if delivery_ratio < self.min_delivery_ratio {
                self.min_delivery_ratio = delivery_ratio;
            }
        }

        // Running mean
        let n = self.sample_count as f32;
        self.avg_latency_us = (self.avg_latency_us * n + latency_us) / (n + 1.0);
        self.avg_delivery_ratio = (self.avg_delivery_ratio * n + delivery_ratio) / (n + 1.0);

        self.sample_count += 1;
    }

    /// Get health status recommendation
    pub fn get_recommendation(&self) -> String {
        if self.sample_count == 0 {
            return "No data available".to_string();
        }

        match (self.avg_delivery_ratio, self.avg_latency_us) {
            (ratio, latency) if ratio >= 0.95 && latency <= 10.0 => {
                "Cluster is healthy. No action needed.".to_string()
            }
            (ratio, latency) if ratio >= 0.90 && latency <= 20.0 => {
                "Cluster is degraded. Consider load balancing.".to_string()
            }
            (ratio, latency) if ratio >= 0.80 && latency <= 50.0 => {
                "Cluster performance is poor. Recommend quantization fallback.".to_string()
            }
            _ => "Cluster has failed nodes. Investigate and recover.".to_string(),
        }
    }
}

/// Fault detection and recovery system
#[derive(Clone, Debug)]
pub struct FaultDetector {
    /// Cluster state
    cluster: Arc<ClusterState>,
    /// Detection interval
    detection_interval: Duration,
}

impl FaultDetector {
    /// Create new fault detector
    pub fn new(cluster: Arc<ClusterState>, detection_interval: Duration) -> Self {
        Self {
            cluster,
            detection_interval,
        }
    }

    /// Detect failed nodes based on heartbeat timeouts
    ///
    /// OPTIMIZATION: Acquires the cluster metrics lock once and mutates status in-place
    /// instead of acquiring locks repeatedly (up to 3N times) and deep-cloning `NodeMetrics`
    /// (including heap-allocated latency history vectors) on every node check.
    pub fn detect_failures(&self) -> Vec<String> {
        let nodes_snapshot = self.cluster.nodes_snapshot();
        let now = Instant::now();
        let mut failed_nodes = Vec::new();

        let mut metrics = self
            .cluster
            .metrics
            .lock()
            .unwrap_or_else(|poison| poison.into_inner());

        for node in nodes_snapshot.iter() {
            if let Some(m) = metrics.get_mut(&node.id) {
                if now.saturating_duration_since(m.last_heartbeat) >= m.heartbeat_timeout {
                    // Any node whose heartbeat has timed out should be marked
                    // Failed, not just ones currently Active. This used to
                    // check `status == Active`, so a node already marked
                    // Degraded (e.g. by NetworkHealthMonitor) that then went
                    // completely silent would never be detected as failed
                    // here, and would never be reported in `failed_nodes`.
                    if m.status != NodeStatus::Failed {
                        m.status = NodeStatus::Failed;
                        failed_nodes.push(node.id.clone());
                    }
                }
            } else {
                // No metrics for this node - insert entry with Failed status so
                // it is only reported once on initial failure detection.
                let mut m = crate::cluster::NodeMetrics::new(
                    node.vram_gb,
                    node.system_memory_gb,
                    node.compute_capability.clone(),
                    Duration::from_secs(5),
                );
                m.name = node.id.clone();
                m.gpu_name = node.gpu_name.clone();
                m.status = NodeStatus::Failed;
                metrics.insert(node.id.clone(), m);
                failed_nodes.push(node.id.clone());
            }
        }

        failed_nodes
    }

    /// Recover a failed node
    pub fn recover_node(&self, node_id: &str) {
        self.cluster.get_metrics_mut(node_id, |metrics| {
            metrics.status = NodeStatus::Active;
            metrics.last_heartbeat = Instant::now();

            // Reset metrics
            metrics.avg_latency_us = 0.0;
            metrics.min_latency_us = f32::MAX;
            metrics.max_latency_us = 0.0;
            metrics.latency_samples = 0;
            metrics.delivery_ratio = 1.0;
            metrics.throughput_gbps = 0.0;
        });
    }

    /// Start periodic fault detection in background
    pub fn start_periodic_detection(&self) {
        let this = self.clone();

        std::thread::spawn(move || loop {
            let failed = this.detect_failures();
            if !failed.is_empty() {
                tracing::warn!(
                    "Detected {} failed nodes: {:?}",
                    failed.len(),
                    failed.iter().map(|s| s.as_str()).collect::<Vec<_>>()
                );
            }
            std::thread::sleep(this.detection_interval);
        });
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use crate::cluster::ClusterState;
    use crate::protocol::NodeResources;
    use std::net::TcpListener;
    use std::sync::Arc;
    use std::time::Duration;

    #[test]
    fn health_monitor_checks_all_nodes() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));
        cluster.register(NodeResources::new("node-b", 12.0, 32.0, "8.6", None));

        let monitor = NetworkHealthMonitor::new(cluster.clone(), HealthConfig::default());
        monitor.check_all();

        let summary = monitor.get_health_summary();
        assert!(summary.contains("Total nodes: 2"));
    }

    #[test]
    fn health_monitor_marks_fresh_nodes_unknown_without_samples() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        let monitor = NetworkHealthMonitor::new(cluster, HealthConfig::default());
        monitor.check_all();

        let node_health = monitor
            .get_node_health("node-a")
            .expect("expected health check result for node-a");
        assert_eq!(node_health.status, HealthStatus::Unknown);
    }

    #[test]
    fn health_monitor_detects_degraded_nodes() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        let monitor = NetworkHealthMonitor::new(cluster.clone(), HealthConfig::default());

        // Simulate multiple degraded health checks by seeding recent_checks directly
        // (check_all uses random healthy values, so we manually push degraded results)
        for _ in 0..3 {
            let result = HealthCheckResult {
                node_id: "node-a".to_string(),
                latency_us: 15.0,
                delivery_ratio: 0.85,
                status: HealthStatus::Degraded,
                timestamp: std::time::Instant::now(),
            };
            monitor
                .recent_checks
                .lock()
                .unwrap()
                .entry("node-a".to_string())
                .or_default()
                .push_back(result);
        }

        assert!(monitor.needs_quantization_fallback("node-a"));
    }

    #[test]
    fn health_metrics_aggregates() {
        let mut metrics = HealthMetrics::new();

        metrics.record(1.0, 0.98);
        metrics.record(2.0, 0.95);
        metrics.record(1.5, 0.97);

        assert!((metrics.avg_latency_us - 1.5).abs() < 0.1);
        assert!(metrics.min_latency_us <= 1.0);
        assert!(metrics.max_latency_us >= 2.0);
    }

    #[test]
    fn fault_detector_detects_failures() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        // Wait for timeout
        std::thread::sleep(Duration::from_secs(6));

        let detector = FaultDetector::new(cluster.clone(), Duration::from_secs(1));
        let failed = detector.detect_failures();

        assert!(failed.contains(&"node-a".to_string()));
    }

    #[test]
    fn fault_detector_detects_failures_for_degraded_nodes() {
        // Regression: detect_failures() used to only transition a node from
        // Active -> Failed on heartbeat timeout. A node already marked
        // Degraded (e.g. by NetworkHealthMonitor.check_all) that then went
        // completely silent was never picked up, so it stayed Degraded
        // forever and was never reported in the failed_nodes list.
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));
        cluster.get_metrics_mut("node-a", |metrics| {
            metrics.status = NodeStatus::Degraded;
        });

        std::thread::sleep(Duration::from_secs(6));

        let detector = FaultDetector::new(cluster.clone(), Duration::from_secs(1));
        let failed = detector.detect_failures();

        assert!(failed.contains(&"node-a".to_string()));
        assert_eq!(
            cluster.get_metrics("node-a").unwrap().status,
            NodeStatus::Failed
        );
    }

    #[test]
    fn fault_detector_recovers_nodes() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        std::thread::sleep(Duration::from_secs(6));

        let detector = FaultDetector::new(cluster.clone(), Duration::from_secs(1));
        detector.detect_failures();

        assert!(cluster.get_metrics("node-a").unwrap().status == NodeStatus::Failed);

        detector.recover_node("node-a");
        assert_eq!(
            cluster.get_metrics("node-a").unwrap().status,
            NodeStatus::Active
        );
    }

    #[test]
    fn health_monitor_gets_recommendation() {
        let mut metrics = HealthMetrics::new();

        // Good cluster
        metrics.record(1.0, 0.98);
        metrics.record(2.0, 0.95);

        let recommendation = metrics.get_recommendation();
        assert!(recommendation.contains("healthy"));
    }

    #[test]
    fn get_recommendation_uses_average_latency_not_minimum() {
        // Regression: get_recommendation() used to key off `min_latency_us`
        // instead of `avg_latency_us`, paired against `avg_delivery_ratio`.
        // Since min_latency_us only ever decreases (or holds), a single fast
        // sample early on would make the recommendation report "healthy"
        // forever, no matter how bad subsequent (averaged) latency became.
        let mut metrics = HealthMetrics::new();
        metrics.record(1.0, 0.98);
        metrics.record(200.0, 0.98);

        let recommendation = metrics.get_recommendation();
        assert!(!recommendation.contains("healthy"));
    }

    #[test]
    fn health_config_autotunes_for_gpu_profiles() {
        let profile = RuntimeProfile {
            node_resources: NodeResources::new("node-a", 24.0, 64.0, "8.9", None),
            logical_cores: 16,
            recommended_workers: 8,
            acceleration_mode: AccelerationMode::Gpu,
            gpu_backend: crate::host::GpuBackend::Cuda,
            xdp_supported: true,
            detection_source: String::from("test"),
            probe_mode: crate::host::ProbeMode::Fast,
        };

        let config = HealthConfig::autotuned(&profile);
        assert!(config.healthy_latency_us < 10.0);
        assert!(config.healthy_delivery_ratio > 0.95);
    }

    #[test]
    fn get_health_status_fails_on_either_metric_crossing_floor() {
        // Regression: Degraded used to require only ONE metric within its
        // floor (an OR), so a node could never reach Failed unless BOTH
        // metrics were simultaneously catastrophic. Either one alone crossing
        // its floor must be enough.
        let cluster = Arc::new(ClusterState::new());
        let monitor = NetworkHealthMonitor::new(cluster, HealthConfig::default());
        let cfg = HealthConfig::default();

        // Delivery ratio fine, latency catastrophic — must be Failed, not Degraded.
        assert_eq!(
            monitor.get_health_status(cfg.degraded_latency_us * 10.0, 0.99),
            HealthStatus::Failed
        );
        // Latency fine, delivery ratio catastrophic — must be Failed, not Degraded.
        assert_eq!(
            monitor.get_health_status(1.0, cfg.degraded_delivery_ratio / 2.0),
            HealthStatus::Failed
        );
        // Both within their degraded floor but not healthy — genuinely Degraded.
        assert_eq!(
            monitor.get_health_status(cfg.degraded_latency_us, cfg.degraded_delivery_ratio),
            HealthStatus::Degraded
        );
    }

    #[test]
    fn tcp_probe_target_can_mark_node_failed_when_unreachable() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        let listener = TcpListener::bind("127.0.0.1:0").expect("bind probe listener");
        let addr = listener.local_addr().expect("listener local addr");
        drop(listener);

        let monitor = NetworkHealthMonitor::new(cluster.clone(), HealthConfig::default());
        monitor.register_tcp_probe_target("node-a", addr);
        monitor.check_all();

        let node_health = monitor
            .get_node_health("node-a")
            .expect("expected health check result for node-a");
        assert_eq!(node_health.status, HealthStatus::Failed);
    }

    #[test]
    fn tcp_probe_target_healthy_when_reachable() {
        let cluster = Arc::new(ClusterState::new());
        cluster.register(NodeResources::new("node-a", 24.0, 64.0, "8.9", None));

        cluster.get_metrics_mut("node-a", |metrics| {
            metrics.record_latency(2.0);
            metrics.record_delivery_ratio(0.99);
        });

        let listener = TcpListener::bind("127.0.0.1:0").expect("bind probe listener");
        let addr = listener.local_addr().expect("listener local addr");

        let monitor = NetworkHealthMonitor::new(cluster.clone(), HealthConfig::default());
        monitor.register_tcp_probe_target("node-a", addr);
        monitor.check_all();

        let node_health = monitor
            .get_node_health("node-a")
            .expect("expected health check result for node-a");
        assert_eq!(node_health.status, HealthStatus::Healthy);
    }
}