Do timeout handling at seq reset

Author: brefra

CHANGELOG.md

@@ -7,6 +7,7 @@
 - Improvement: Do a single ping request to validate if node is on-line
 - Improvement: Guard Scan sensitivity setting to medium
 - Improvement: Move general module code of messages, nodes, connection to the  __init__.py files.
+- Improvement: Do proper timeout handling while sequence counter resets (once every 65532 messages)
 - Improvement: Better code separation. All logic is in their designated files:
   1. Connection (connection/*.py)
   2. Data parsing (parser.py)

plugwise/controller.py

@@ -217,7 +217,7 @@ def message_handler(self, message):
         if self.last_seq_id:
             if int(self.last_seq_id, 16) < int(message.seq_id, 16) < 65533:
                 self.last_seq_id = message.seq_id
-            elif message.seq_id == b"0000":
+            elif message.seq_id == b"0000" and self.last_seq_id == b"FFFB":
                 self.last_seq_id = b"0000"
 
         if isinstance(message, NodeAckSmallResponse):

plugwise/util.py

@@ -55,7 +55,8 @@ def inc_seq_id(seq_id, value=1) -> bytearray:
     if seq_id is None:
         return b"0000"
     temp_int = int(seq_id, 16) + value
-    # Max seq_id = b'FFFC'
+    # Max seq_id = b'FFFB'
+    # b'FFFC' reserved for <unknown> message
     # b'FFFD' reserved for 'NodeJoinAckResponse' message
     # b'FFFE' reserved for 'NodeSwitchGroupResponse' message
     # b'FFFF' reserved for 'NodeAwakeResponse' message