You can not select more than 25 topics Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.

Strycher 73d0fc9888 fix(#309): decode and encode path_len per firmware, not as a byte count ... The firmware path-len byte is packed and self-describing (src/Packet.h:79-84): high 2 bits = hash size - 1, low 6 bits = hash COUNT (hops), byte length = count * size. The app treated the low 6 bits as a byte count and ignored the width, on the strength of a comment asserting the high bits were "not reliably populated". Both halves of that comment were wrong. Receive: Contact.fromFrame read `count` bytes where firmware meant `count` hops, so at 2-byte width it kept HALF of every path and discarded the rest. DAYTON VA's 2-hop route became a 2-byte fragment, which is why every repeater login in Bandit's capture timed out. Send: buildUpdateContactPathFrame wrote the count raw, leaving mode bits 00. That tells the radio "1-byte hashes" while handing it 2-byte hash data, so it routed to nodes never on the route. This is the likelier cause of the observed "routes via c6 and 5c separately" behaviour. Also fixes a third unit inconsistency: setContactPath wrote customPath.length (bytes) into pathLength (hops) after every path set. Contact now carries pathHashWidth per path, so a stored path can never be re-sliced at a width it was not captured at. realHopCount() is removed rather than adjusted: it divided an already-correct hop count, halving it at 2-byte width, and every call site was compensating for a bug that no longer exists. Migration (Ben's call, option A): records written before this change have no pathHashWidth and hold truncated bytes that cannot be recovered by reinterpretation, so their path is dropped and the contact reverts to flood until the radio re-supplies it. Logged, not silent. Tests asserted the old semantics (expect(realHopCount(2, 2), 1)) and were rewritten against firmware truth rather than adjusted to keep passing. Added send-side encoding coverage, which did not exist. 456 tests pass. Refs #240, #279, #299 Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>		2 days ago
..
build_update_contact_path_frame_test.dart	fix(#309): decode and encode path_len per firmware, not as a byte count	2 days ago
direct_repeater_ranking_test.dart	feat(#186): topology service + readout + RX ingestion + route-builder UX	1 week ago
gif_url_outbound_guard_test.dart	feat(#282): send a Giphy URL instead of g:<code> for cross-client GIFs	3 days ago
gps_self_telemetry_test.dart	feat(#123): GPS refresh — self-telemetry stock-fw fallback (firmware-aware poll)	1 week ago
meshcore_connector_app_start_retry_test.dart	fix(#88): cap the BLE APP_START handshake retry with backoff + keep-alive	4 weeks ago
meshcore_connector_channel_sync_test.dart	fix(#82): skip empty channel slots on RESP_CODE_ERR instead of timing out	4 weeks ago
meshcore_connector_tcp_error_filter_test.dart	fix(tcp): reset state on aborted pre-handshake connect	4 months ago
observer_caps_test.dart	fix(#71): testable offband_caps read + byte-82 offset verification (N1)	4 weeks ago
observer_config_client_test.dart	test(#68): adversarial codec test suite (A1, TDD-RED)	4 weeks ago
path_hash_test.dart	fix(#309): decode and encode path_len per firmware, not as a byte count	2 days ago