Alan-Jowett / sonde / 24151899330 / 1
83%
main: 83%

DEFAULT BRANCH: main
Ran
Files 85
Run time 3s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 81.607% (-0.06%) from 81.666%
24151899330.1

push

github

web-flow 
feat: centralize workspace version to 0.4.0 and add `firmware_version` to WAKE (#690)

* feat: centralize workspace version to 0.4.0 and add irmware_version to WAKE

Centralize version management across the sonde workspace:

- Add [workspace.package] version = "

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* style: apply rustfmt formatting

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address PR review feedback

- Modem: use CARGO_PKG_VERSION_{MAJOR,MINOR,PATCH} env vars instead of
  manually parsing CARGO_PKG_VERSION. Adds explicit range checks and
  handles pre-release suffixes safely.
- Registry: take firmware_version: String by value in update_telemetry
  to avoid redundant allocation on the WAKE hot path.
- State bundle: persist firmware_version through export/import round-trips
  via new CBOR key NODE_KEY_FW_VERSION (14). Optional for backward compat.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: validate firmware_version length and ASCII on decode

Add length cap (32 bytes) and ASCII-only check to get_string() in the
protocol message decoder. Prevents unbounded allocation from malicious
or buggy peers sending oversized CBOR text values for firmware_version.

Add tests for oversized and non-ASCII firmware_version rejection.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: address second round of PR review feedback

- protocol.md: document exact wire constraints for firmware_version
  (ASCII-only, max 32 bytes) instead of saying "semantic version".
- protocol.md: reword evolution rules to explicitly allow pre-1.0
  breaking changes, resolving inconsistency with the additive-only rule.
- engine.rs: move firmware_version into update_telemetry instead of
  cloning, read back from updated_node for the event payload.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Signed-off-by: Alan Jowett <alan.jowett@microsoft.... (continued)

21274 of 26069 relevant lines covered (81.61%)

140.72 hits per line

Source Files on job 24151899330.1