fix: forward non-matching HID++ reports during command exchanges

_request() reads all incoming HID++ reports while waiting for a specific
response, but previously discarded non-matching ones (e.g. diverted
button state reports). This caused the held-state tracker for extra
CIDs like 0x00C4 (SmartShift button) to get out of sync -- a button
release during a set_smart_shift write would be swallowed, leaving
held=True and requiring an extra press to re-sync.

Forward non-matching reports to _on_report() so button DOWN/UP tracking
stays correct during command exchanges. Also reset extra_diverts held
states on disconnect so reconnects start clean.

Author: TomBadash

core/hid_gesture.py

@@ -738,6 +738,9 @@ def _request(self, feat, func, params, timeout_ms=2000):
             expected_funcs = {func, (func + 1) & 0x0F}
             if r_feat == feat and r_sw == MY_SW and r_func in expected_funcs:
                 return msg
+            # Forward non-matching reports (e.g. diverted button events) so
+            # button held-state tracking stays in sync during command exchanges.
+            self._on_report(raw)
         print(f"[HidGesture] request timeout feat=0x{feat:02X} func=0x{func:X} "
               f"devIdx=0x{self._dev_idx:02X} params=[{_hex_bytes(req_params)}]")
         return None
@@ -1474,6 +1477,8 @@ def _main_loop(self):
             self._pending_battery = None
             self._last_logged_battery = None
             self._held = False
+            for info in self._extra_diverts.values():
+                info["held"] = False
             self._gesture_cid = DEFAULT_GESTURE_CID
             self._gesture_candidates = list(DEFAULT_GESTURE_CIDS)
             self._rawxy_enabled = False