Follow-up from the uv migration PR review: #30
Copilot noted that tools/send_control_telemetry.py duplicates control telemetry protocol details that already live in the runtime code:
- local
AXES definition instead of lib.control_telemetry.AXES
- local
PORT = 5005 instead of CONTROL_TELEM_PORT
binascii.crc32 instead of the shared lib.crc.crc32_ieee- hardcoded expected packet size instead of
PACKET_SIZE
This predates the uv migration because the script was moved from tests/test_control_telemetry.py, so it should be fixed separately from the dependency-management PR.
Suggested fix: update the standalone sender to import the shared protocol constants and CRC helper from the app code, then verify it still emits the expected 80-byte packet.
Follow-up from the uv migration PR review: #30
Copilot noted that
tools/send_control_telemetry.pyduplicates control telemetry protocol details that already live in the runtime code:AXESdefinition instead oflib.control_telemetry.AXESPORT = 5005instead ofCONTROL_TELEM_PORTbinascii.crc32instead of the sharedlib.crc.crc32_ieeePACKET_SIZEThis predates the uv migration because the script was moved from
tests/test_control_telemetry.py, so it should be fixed separately from the dependency-management PR.Suggested fix: update the standalone sender to import the shared protocol constants and CRC helper from the app code, then verify it still emits the expected 80-byte packet.