dangernoodle-io / breadboard / 28484988338

100%

push

github

fix(build): silence all compiler warnings; enable -Werror scoped to bb sources (B1-440) (#617)

- bb_telemetry_routes.c:359 (-Wformat-truncation, real bug): resize line[256]→[320]; worst case name(127)+{host="(7)+host_escaped(127)+"} (3)+%.6g(20)+\n(1)=285 exceeds old size
- bb_telemetry_routes.c:370 (-Wformat-truncation, real bug): resize label_frag[320]→[384]; worst case ,(1)+label_key(63)+="(2)+escaped_val(255)+"(1)=322 overruns old size
- bb_button_events.c:249 (-Wliteral-suffix): add required space before PRIu16 macro in format string
- bb_led_anim.c:341 (-Wliteral-suffix): add required space before PRIu32 macro in format string
- bb_button_gpio.cpp and three sibling arduino drivers (-Wodr): rename local state_t typedef to file-unique name (bb_button_gpio_state_t, bb_led_gpio_state_t, bb_led_pwm_state_t, bb_led_apa102_state_t) to fix c++ ODR violation across translation units linked in the same arduino build

-Werror scoping:
- native (host tests): build_src_flags = -Werror in [env:native] (platformio.ini); applies to bb sources added via SRC_FILTER, not to cJSON lib_dep
- arduino smoke envs: build_src_flags = -Werror in [arduino_base] (examples/smoke/platformio.ini); applies to bb project sources, not to lib_deps (ArduinoJson, Adafruit_CC3000_Library)
- esp-idf smoke envs: idf_component_get_property loop in examples/smoke/CMakeLists.txt applies target_compile_options(PRIVATE -Werror) to bb_* components only, excluding esp-idf system components and managed_components (espressif__mdns)

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

1850 of 1850 branches covered (100.0%)

Branch coverage included in aggregate %.

3190 of 3190 relevant lines covered (100.0%)

1225.03 hits per line