Use actual modulation for burner flame state

Author: jfhautenauven

custom_components/bosch/pointtapi_entities.py

@@ -2660,6 +2660,20 @@ def _resolve_on_off(raw: Any) -> bool | None:
     return None
 
 
+def _burner_flame_state(data: dict[str, Any]) -> bool | None:
+    """Resolve burner flame state from current burner modulation.
+
+    POINTTAPI flameIndication can return multiple string dialects (off/ch/dhw),
+    which is not stable enough for a strict on/off parser. actualModulation is
+    a numeric signal and better reflects whether the burner is actively firing.
+    """
+    raw = _val(data, "/heatSources/actualModulation")
+    try:
+        return float(raw) > 0.0
+    except (TypeError, ValueError):
+        return None
+
+
 # ── Boost session: in-memory tracking of HA-triggered boost (v0.33.0) ──────
 
 
@@ -2780,6 +2794,7 @@ def available(self) -> bool:
         key="/heatSources/flameIndication",
         translation_key="burner_flame",
         device_class=BinarySensorDeviceClass.RUNNING,
+        value_fn=_burner_flame_state,
     ),
     BoschPoinTTAPIBinarySensorEntityDescription(
         key="/heatSources/refillNeeded",

unittests/test_pointtapi_new_entities.py

@@ -812,6 +812,34 @@ def test_dhw_binary_sensor_off_maps_to_false(self):
         ent._handle_coordinator_update()
         assert ent.is_on is False
 
+    def test_burner_flame_uses_actual_modulation_positive_as_on(self):
+        coord = _mock_coordinator(
+            {
+                "/heatSources/flameIndication": {"value": "dhw"},
+                "/heatSources/actualModulation": {"value": 12.0},
+            }
+        )
+        ent = _binary_sensor(coord, "/heatSources/flameIndication")
+        ent._handle_coordinator_update()
+        assert ent.is_on is True
+
+    def test_burner_flame_uses_actual_modulation_zero_as_off(self):
+        coord = _mock_coordinator(
+            {
+                "/heatSources/flameIndication": {"value": "ch"},
+                "/heatSources/actualModulation": {"value": 0.0},
+            }
+        )
+        ent = _binary_sensor(coord, "/heatSources/flameIndication")
+        ent._handle_coordinator_update()
+        assert ent.is_on is False
+
+    def test_burner_flame_unknown_when_actual_modulation_missing(self):
+        coord = _mock_coordinator({"/heatSources/flameIndication": {"value": "dhw"}})
+        ent = _binary_sensor(coord, "/heatSources/flameIndication")
+        ent._handle_coordinator_update()
+        assert ent.is_on is None
+
     def test_switch_unavailable_when_path_absent(self):
         coord = _mock_coordinator({})
         ent = _switch(coord, "/dhwCircuits/dhw1/extraDhw")