chore: Add comments everywhere on implicit refreshes

Author: allenporter

roborock/devices/traits/v1/child_lock.py

@@ -19,9 +19,11 @@ def is_on(self) -> bool:
     async def enable(self) -> None:
         """Enable the child lock."""
         await self.rpc_channel.send_command(RoborockCommand.SET_CHILD_LOCK_STATUS, params={_STATUS_PARAM: 1})
+        # Optimistcally update state to avoid an extra refresh
         self.lock_status = 1
 
     async def disable(self) -> None:
         """Disable the child lock."""
         await self.rpc_channel.send_command(RoborockCommand.SET_CHILD_LOCK_STATUS, params={_STATUS_PARAM: 0})
+        # Optimistcally update state to avoid an extra refresh
         self.lock_status = 0

roborock/devices/traits/v1/do_not_disturb.py

@@ -31,9 +31,11 @@ async def enable(self) -> None:
             RoborockCommand.SET_DND_TIMER,
             params=self.as_list(),
         )
+        # Optimistcally update state to avoid an extra refresh
         self.enabled = 1
 
     async def disable(self) -> None:
         """Disable the Do Not Disturb (DND) timer settings of the device."""
         await self.rpc_channel.send_command(RoborockCommand.CLOSE_DND_TIMER)
+        # Optimistcally update state to avoid an extra refresh
         self.enabled = 0

roborock/devices/traits/v1/flow_led_status.py

@@ -19,9 +19,11 @@ def is_on(self) -> bool:
     async def enable(self) -> None:
         """Enable the Flow LED status."""
         await self.rpc_channel.send_command(RoborockCommand.SET_FLOW_LED_STATUS, params={_STATUS_PARAM: 1})
+        # Optimistcally update state to avoid an extra refresh
         self.status = 1
 
     async def disable(self) -> None:
         """Disable the Flow LED status."""
         await self.rpc_channel.send_command(RoborockCommand.SET_FLOW_LED_STATUS, params={_STATUS_PARAM: 0})
+        # Optimistcally update state to avoid an extra refresh
         self.status = 0

roborock/devices/traits/v1/led_status.py

@@ -19,11 +19,13 @@ def is_on(self) -> bool:
     async def enable(self) -> None:
         """Enable the LED status."""
         await self.rpc_channel.send_command(RoborockCommand.SET_LED_STATUS, params=[1])
+        # Optimistcally update state to avoid an extra refresh
         self.status = 1
 
     async def disable(self) -> None:
         """Disable the LED status."""
         await self.rpc_channel.send_command(RoborockCommand.SET_LED_STATUS, params=[0])
+        # Optimistcally update state to avoid an extra refresh
         self.status = 0
 
     @classmethod

roborock/devices/traits/v1/valley_electricity_timer.py

@@ -19,6 +19,7 @@ def is_on(self) -> bool:
     async def set_timer(self, timer: ValleyElectricityTimer) -> None:
         """Set the Valley Electricity Timer settings of the device."""
         await self.rpc_channel.send_command(RoborockCommand.SET_VALLEY_ELECTRICITY_TIMER, params=timer.as_list())
+        await self.refresh()
 
     async def clear_timer(self) -> None:
         """Clear the Valley Electricity Timer settings of the device."""
@@ -31,11 +32,13 @@ async def enable(self) -> None:
             RoborockCommand.SET_VALLEY_ELECTRICITY_TIMER,
             params=self.as_list(),
         )
+        # Optimistcally update state to avoid an extra refresh
         self.enabled = 1
 
     async def disable(self) -> None:
         """Disable the Valley Electricity Timer settings of the device."""
         await self.rpc_channel.send_command(
             RoborockCommand.CLOSE_VALLEY_ELECTRICITY_TIMER,
         )
+        # Optimistcally update state to avoid an extra refresh
         self.enabled = 0