fix: scalafix violations in metrics check and health monitor

- Wrap asInstanceOf casts with scalafix:off/on in isResourceConstrained
- Replace var with immutable List in HealthMonitor.checkInstanceHealth

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

modules/coordinator/src/main/scala/de/nowchess/coordinator/service/AutoScaler.scala

@@ -72,6 +72,7 @@ class AutoScaler:
     }
   // scalafix:on DisableSyntax.asInstanceOf
 
+  // scalafix:off DisableSyntax.asInstanceOf
   private def isResourceConstrained(instanceId: String): Boolean =
     kubeClientOpt.fold(false) { kube =>
       try
@@ -106,6 +107,7 @@ class AutoScaler:
           log.debugf(ex, "Failed to check resource metrics for %s", instanceId)
           false
     }
+  // scalafix:on DisableSyntax.asInstanceOf
 
   def checkAndScale: Unit =
     if config.autoScaleEnabled then

modules/coordinator/src/main/scala/de/nowchess/coordinator/service/HealthMonitor.scala

@@ -89,17 +89,17 @@ class HealthMonitor:
       log.warnf("Evicted %d stale instances: %s", evicted.size, evicted.mkString(", "))
       evicted.foreach(deleteK8sPod)
       autoScaler.scaleUp()
-    val instances      = instanceRegistry.getAllInstances
+    val instances = instanceRegistry.getAllInstances
-    var instanceFailed = false
+    val failed = instances.collect { inst =>
-    instances.foreach { inst =>
       val isHealthy = checkHealth(inst.instanceId)
       if !isHealthy && inst.state == "HEALTHY" then
         log.warnf("Instance %s marked unhealthy", inst.instanceId)
         instanceRegistry.markInstanceDead(inst.instanceId)
         deleteK8sPod(inst.instanceId)
-        instanceFailed = true
+        Some(inst.instanceId)
-    }
+      else None
-    if instanceFailed then autoScaler.scaleUp()
+    }.flatten
+    if failed.nonEmpty then autoScaler.scaleUp()
 
   private def checkHealth(instanceId: String): Boolean =
     val redisHealthy = checkRedisHeartbeat(instanceId)