Make sure we get a logged path when parsing fails. (#170)

- Drop compiled parsers, speed here is not that important, it's better
to know what things fails.
- Use a focusted sequence and switch to select the message type so the
parsing path is fixed

nibe/connection/nibegw.py

@@ -728,7 +728,7 @@ def _encode(self, obj, context, path):
         )
     ),
     "checksum" / Checksum(Int8ub, xor8, this.fields.data),
-).compile()
+)
 
 
 RequestData = Switch(
@@ -792,5 +792,5 @@ def _encode(self, obj, context, path):
         )
     ),
     "checksum" / Checksum(Int8ub, xor8, this.fields.data),
-).compile()
+)
 # fmt: on

nibe/console_scripts/cli.py

@@ -12,12 +12,14 @@
 from construct import (
     Const,
     ConstructError,
+    FocusedSeq,
     GreedyRange,
     Int8ul,
+    Peek,
     RawCopy,
-    Select,
     Struct,
-    Terminated,
+    Switch,
+    this,
 )
 
 from ..coil import CoilData
@@ -31,12 +33,10 @@
 
 Nak = Struct("fields" / RawCopy(Struct("Nak" / Const(0x15, Int8ul))))
 
-Block = Select(
-    Terminated,
-    Response,
-    Request,
-    Ack,
-    Nak,
+Block = FocusedSeq(
+    "data",
+    "start" / Peek(Int8ul),
+    "data" / Switch(this.start, {0x5C: Response, 0xC0: Request, 0x06: Ack, 0x15: Nak}),
 )
 
 Stream = GreedyRange(Block)