dangernoodle-io / breadboard / 29382284084

96%

push

github

refactor: make bb_udp_client's NVS namespace caller-supplied (B1-951) (#874)

bb_udp_client_init() hardcoded NVS namespace "bb_udp" internally. Per
user decision (epic B1-949), a client component declares WHAT it
stores; the composition decides WHERE. ns is now a required, borrowed
const char* parameter (NULL/empty -> BB_ERR_INVALID_ARG before any
storage touch), mirroring bb_tcp_client's precedent (PR #872) and
bb_tls_creds's resolve_one per-call field descriptor pattern. bb_nv
dropped from PRIV_REQUIRES (no remaining bb_nv symbol use once
BB_UDP_NVS_NS is caller-supplied instead of component-internal).


Claude-Session: https://claude.ai/code/session_01V5JXJNaYWPwjrcS8nk183m

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

6962 of 7597 branches covered (91.64%)

Branch coverage included in aggregate %.

18 of 18 new or added lines in 2 files covered. (100.0%)

11674 of 11929 relevant lines covered (97.86%)

2416.44 hits per line