add refresh status button for spino, request report status on setup, change suggested area

Author: mikey0000

custom_components/mammotion/button.py

@@ -59,6 +59,11 @@ class MammotionSpinoButtonEntityDescription(ButtonEntityDescription):
         entity_category=EntityCategory.CONFIG,
         press_fn=lambda coordinator: coordinator.async_fetch_pool_line(),
     ),
+    MammotionSpinoButtonEntityDescription(
+        key="spino_refresh_status",
+        entity_category=EntityCategory.DIAGNOSTIC,
+        press_fn=lambda coordinator: coordinator.async_request_status(),
+    ),
 )
 
 

custom_components/mammotion/coordinator.py

@@ -2237,19 +2237,31 @@ async def _async_setup(self) -> None:
         pushed to entities via the inherited ``_on_state_changed`` callback.
         """
         await super()._async_setup()
-        for toggle in SpinoToggle:
+        # Start the status report stream so the device pushes dev_statue_t
+        # (sys_status / work_mode / battery) — the pool cleaner doesn't report
+        # unsolicited otherwise. See get_report_cfg_spino / async_subscribe_status.
+
+        try:
             with contextlib.suppress(
-                DeviceOfflineException,
                 GatewayTimeoutException,
                 NoTransportAvailableError,
+                HomeAssistantError,
             ):
-                await self.async_send_and_wait(
-                    "read_write_device",
-                    "bidire_comm_cmd",
-                    rw_id=int(toggle),
-                    context=0,
-                    rw=0,
-                )
+                await self.async_subscribe_status()
+            for toggle in SpinoToggle:
+                with contextlib.suppress(
+                    GatewayTimeoutException,
+                    NoTransportAvailableError,
+                ):
+                    await self.async_send_and_wait(
+                        "read_write_device",
+                        "bidire_comm_cmd",
+                        rw_id=int(toggle),
+                        context=0,
+                        rw=0,
+                    )
+        except DeviceOfflineException:
+            self.device.online = False
 
     async def get_coordinator_data(
         self, device: PoolCleanerDevice
@@ -2327,6 +2339,18 @@ async def update_firmware(self, version: str) -> None:
 
     # === Pool cleaner control helpers (called by control entities) ===
 
+    async def async_subscribe_status(self) -> None:
+        """Start the Spino status report stream (called once at setup).
+
+        Subscribes to RIT_CONNECT + RIT_DEV_STA with count=0 (continuous) so the
+        device pushes dev_statue_t frames; PoolStateReducer applies them.
+        """
+        await self.async_send_command("get_report_cfg_spino", count=0)
+
+    async def async_request_status(self) -> None:
+        """One-shot Spino status poll, backing the refresh-status button."""
+        await self.async_send_command("get_report_cfg_spino", count=1)
+
     async def async_set_work_mode(self, work_mode: int) -> None:
         """Set the Spino cleaning work mode."""
         await self.async_send_command("set_swimming_work_mode", work_mode=work_mode)

custom_components/mammotion/entity.py

@@ -231,7 +231,7 @@ def device_info(self) -> DeviceInfo:
             model=spino_device.name,
             model_id=self.coordinator.device.product_key,
             sw_version=spino_device.device_firmwares.device_version,
-            suggested_area="Garden",
+            suggested_area="Pool",
         )
 
     @property

custom_components/mammotion/manifest.json

@@ -1,7 +1,7 @@
 {
   "domain": "mammotion",
   "name": "Mammotion",
-  "version": "0.5.48-beta15",
+  "version": "0.5.48",
   "issue_tracker": "https://github.com/mikey0000/Mammotion-HA/issues",
   "integration_type": "device",
   "bluetooth": [
@@ -16,19 +16,11 @@
       "connectable": true
     }
   ],
-  "codeowners": [
-    "@mikey0000"
-  ],
+  "codeowners": ["@mikey0000"],
   "config_flow": true,
-  "dependencies": [
-    "bluetooth_adapters"
-  ],
+  "dependencies": ["bluetooth_adapters"],
   "documentation": "https://github.com/mikey0000/Mammotion-HA/wiki",
-  "loggers": [
-    "pymammotion"
-  ],
+  "loggers": ["pymammotion"],
   "iot_class": "local_push",
-  "requirements": [
-    "pymammotion==0.7.129"
-  ]
+  "requirements": ["pymammotion==0.7.129"]
 }

custom_components/mammotion/strings.json

@@ -243,6 +243,9 @@
       },
       "spino_fetch_line": {
         "name": "Fetch cleaning route"
+      },
+      "spino_refresh_status": {
+        "name": "Refresh status"
       }
     },
     "switch": {

custom_components/mammotion/translations/cs.json

@@ -313,6 +313,9 @@
       },
       "restart_mower": {
         "name": "Restartovat sekačku"
+      },
+      "spino_refresh_status": {
+        "name": "Obnovit stav"
       }
     },
     "switch": {

custom_components/mammotion/translations/da.json

@@ -313,6 +313,9 @@
       },
       "restart_mower": {
         "name": "Genstart plæneklipperen"
+      },
+      "spino_refresh_status": {
+        "name": "Opdater status"
       }
     },
     "switch": {

custom_components/mammotion/translations/de.json

@@ -313,6 +313,9 @@
       },
       "restart_mower": {
         "name": "Mähroboter neustarten"
+      },
+      "spino_refresh_status": {
+        "name": "Status aktualisieren"
       }
     },
     "switch": {

custom_components/mammotion/translations/en.json

@@ -322,6 +322,9 @@
       },
       "spino_fetch_line": {
         "name": "Fetch cleaning route"
+      },
+      "spino_refresh_status": {
+        "name": "Refresh status"
       }
     },
     "switch": {

custom_components/mammotion/translations/fr.json

@@ -313,6 +313,9 @@
       },
       "restart_mower": {
         "name": "Redémarrer la tondeuse"
+      },
+      "spino_refresh_status": {
+        "name": "Actualiser le statut"
       }
     },
     "switch": {