refactor(svm): align supports_svm with supports_window_control pattern

Per cdnninja review (2026-08-06): supports_svm was a probe method on
VehicleManager/ApiImpl (GET getSVMDetails, check image_bytes), deviating
from the library's capability convention. supports_window_control and
supports_valet_mode are per-region class attributes on the region API,
stamped onto Vehicle by VehicleManager.initialize_vehicles — no API call.

Align supports_svm to the same pattern:
- ApiImpl: class attr supports_svm = False (was a probe stub method)
- HyundaiBlueLinkApiUSA: class attr supports_svm = True (was a probe method)
- VehicleManager: stamp vehicle.supports_svm = api.supports_svm in
  initialize_vehicles; drop the supports_svm(vehicle_id) wrapper
- Vehicle.supports_svm field unchanged (canonical source, now stamped)

Tradeoff: capability is per-region, not per-vehicle. Models without SVM
hardware will show the entity; capture fails at runtime — same as
supports_window_control today. ijojog (tester) accepted flag-on-Vehicle.

Also fix latent ruff violations exposed by removing .ruff.toml (prev
commit, which unmasked pyproject.toml [tool.ruff] py312 rule set):
- I001 import sort (ApiImpl, VehicleManager, us_svm_test)
- PYI041 svm._parse_float: int | float -> float
- PIE790 SafetyAcknowledgmentError unnecessary pass
- UP017 dt.timezone.utc -> dt.UTC (tests)
- SIM117 collapse nested with (tests)
- TRY004 noqa on _FakeResponse.json ValueError (JSONDecodeError semantic)

Tests: drop 10 probe tests, add 3 class-attr/stamp tests. 450 pass, ruff clean.

Author: blka

hyundai_kia_connect_api/ApiImpl.py

@@ -25,10 +25,10 @@
     VEHICLE_LOCK_ACTION,
     WINDOW_STATE,
 )
+from .svm import SVMDetails
 from .Token import Token
 from .utils import get_child_value, to_int_enum
 from .Vehicle import Vehicle
-from .svm import SVMDetails
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -165,6 +165,7 @@ class ApiImpl:
     previous_longitude: float = None
     supports_window_control: bool = False
     supports_valet_mode: bool = False
+    supports_svm: bool = False
 
     def __init__(self) -> None:
         """Initialize."""
@@ -460,14 +461,3 @@ def request_svm_capture(
         raise NotImplementedError(
             "request_svm_capture is not implemented for this region"
         )
-
-    def supports_svm(self, token: Token, vehicle: Vehicle) -> bool:
-        """Return whether this vehicle supports SVM.
-
-        The base implementation returns a cached value if present, otherwise
-        False without mutating the vehicle. Region subclasses that support SVM
-        should override and probe.
-        """
-        if vehicle.supports_svm is not None:
-            return vehicle.supports_svm
-        return False

hyundai_kia_connect_api/HyundaiBlueLinkApiUSA.py

@@ -141,6 +141,11 @@ class HyundaiBlueLinkApiUSA(ApiImpl):
     _SVM_POLL_TIMEOUT_SECONDS = 120
     _SVM_INITIAL_WAIT_SECONDS = 15
 
+    # Hyundai BlueLink USA exposes SVM / Find My Car for supported vehicles.
+    # Capability is declared per-region (like supports_window_control); the
+    # actual image is fetched on demand via get_svm_details / request_svm_capture.
+    supports_svm: bool = True
+
     # Maps transaction IDs to service_type values for action status polling.
     # Horn/hazard commands need HORN_AND_LIGHTS or LIGHTS_ONLY instead of
     # the default REMOTE_POLL.
@@ -1028,21 +1033,6 @@ def request_svm_capture(
             f"{self._SVM_POLL_TIMEOUT_SECONDS} seconds"
         )
 
-    def supports_svm(self, token: Token, vehicle: Vehicle) -> bool:
-        """Probe whether this USA Hyundai vehicle supports SVM.
-
-        Caches the result on the vehicle. Any API or auth failure is treated
-        as "not supported" so that consumers do not have to handle exceptions.
-        """
-        if vehicle.supports_svm is not None:
-            return vehicle.supports_svm
-        try:
-            details = self.get_svm_details(token, vehicle)
-            vehicle.supports_svm = bool(details.image_bytes)
-        except Exception:
-            vehicle.supports_svm = False
-        return vehicle.supports_svm
-
     @staticmethod
     def _svm_is_fresh(
         details: SVMDetails,

hyundai_kia_connect_api/VehicleManager.py

@@ -14,7 +14,6 @@
     ScheduleChargingClimateRequestOptions,
     WindowRequestOptions,
 )
-from .svm import SVMDetails
 from .const import (
     BRAND_GENESIS,
     BRAND_HYUNDAI,
@@ -45,6 +44,7 @@
 from .KiaUvoApiEU import KiaUvoApiEU
 from .KiaUvoApiIN import KiaUvoApiIN
 from .KiaUvoApiUSA import KiaUvoApiUSA
+from .svm import SVMDetails
 from .Token import Token
 from .Vehicle import Vehicle
 
@@ -131,6 +131,7 @@ def initialize_vehicles(self):
         for vehicle in vehicles:
             vehicle.supports_window_control = self.api.supports_window_control
             vehicle.supports_valet_mode = self.api.supports_valet_mode
+            vehicle.supports_svm = self.api.supports_svm
             self.vehicles[vehicle.id] = vehicle
 
     def get_vehicle(self, vehicle_id: str) -> Vehicle:
@@ -192,20 +193,6 @@ def force_refresh_vehicle_state(self, vehicle_id: str) -> None:
         else:
             _LOGGER.debug(f"{DOMAIN} - Vehicle Disabled, skipping.")
 
-    def supports_svm(self, vehicle_id: str) -> bool:
-        """Return whether the given vehicle supports SVM.
-
-        Delegates to the region-specific API implementation. Missing vehicles
-        and any API failures are reported as False.
-        """
-        vehicle = self.vehicles.get(vehicle_id)
-        if vehicle is None:
-            return False
-        try:
-            return self.api.supports_svm(self.token, vehicle)
-        except Exception:
-            return False
-
     def get_svm_details(self, vehicle_id: str) -> SVMDetails:
         """Return the latest cached SVM composite image and metadata.
 

hyundai_kia_connect_api/exceptions.py

@@ -71,8 +71,6 @@ class SafetyAcknowledgmentError(APIError):
     before it can be executed (e.g. SVM/Find My Car capture).
     """
 
-    pass
-
 
 class UnsupportedControlError(APIError):
     """

hyundai_kia_connect_api/svm.py

@@ -29,7 +29,7 @@ def _parse_int(value: str | int | None) -> int | None:
         return None
 
 
-def _parse_float(value: str | float | int | None) -> float | None:
+def _parse_float(value: str | float | None) -> float | None:
     if value is None:
         return None
     try: