dangernoodle-io / breadboard / 26201087071

100%

push

github

refactor: collapse bb_button_events espidf↔host duplicates via bb_timer (#289)

- delete platform/espidf/bb_button_events/bb_button_events.c (253 lines)
- promote platform/host/bb_button_events/bb_button_events.c as the canonical
  source for all platforms; CMakeLists.txt now points at the host path
- route the auto-start periodic timer through bb_timer (create/start/stop/delete)
  instead of esp_timer directly — same behavior, no #ifdef ESP_PLATFORM in hot paths
- guard bb_timer include and struct field with #ifndef ARDUINO; on Arduino,
  auto_start_timer=true returns BB_ERR_UNSUPPORTED so callers use tick() from loop()
- replace now_ms() implementation: uses bb_timer_now_us() on host/ESP-IDF,
  millis() on Arduino, settable mock on test builds
- add bb_timer to bb_button_events PRIV_REQUIRES; remove esp_timer (now indirect
  via bb_timer on ESP-IDF)
- add bb_timer to bb_button_events native_scaffold depends
- add host tests: auto_start_timer_fires (timer drives state machine) and
  no_auto_start_timer_does_not_fire
- 905 tests pass, 100% branch/line/function coverage maintained

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

1026 of 1026 branches covered (100.0%)

Branch coverage included in aggregate %.

1899 of 1899 relevant lines covered (100.0%)

958.66 hits per line