• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Alan-Jowett / sonde / 23522479144
82%

Build:
DEFAULT BRANCH: main
Ran 25 Mar 2026 02:50AM UTC
Jobs 1
Files 76
Run time 1min
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

25 Mar 2026 02:47AM UTC coverage: 85.819% (-0.04%) from 85.859%
23522479144

push

github

web-flow
fix(gateway): share pending_commands when handler config is used (#488)

* fix(node): remove RngHealthCheckFailed match arm from firmware binary

The RngHealthCheckFailed enum variant was removed during the PR #418
merge resolution -- RNG health-check failure now returns
WakeCycleOutcome::Sleep (graceful degradation). The bin/node.rs match
arm referencing the removed variant was auto-merged from the PR branch
and breaks the ESP32 firmware build.

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

* fix(gateway): share pending_commands when handler config is used

Design (§9.1):
- Document that --handler-config must use new_with_pending +
  set_handler_router to share pending_commands with admin API (D-485).

Code:
- Add Gateway::set_handler_router() method.
- gateway.rs: use new_with_pending + set_handler_router instead of
  new_with_handler when --handler-config is provided.

Fixes #485.

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

* Address review findings: guard set_handler_router against double-set, strengthen D-485 design note

- Add assert in set_handler_router to prevent accidental double-init
- Strengthen gateway-design.md D-485 note to explicitly forbid constructors
  that create separate pending_commands maps

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

* fix: return Result from set_handler_router, clarify new_with_handler scope

- Replace assert! in set_handler_router with Result<(), &'static str>
  so a double-set returns an error instead of panicking in production.
- Add doc warning on new_with_handler noting it allocates its own
  pending_commands/SessionManager and is test-only (D-485).
- Update gateway-design.md D-485 note to accurately describe the
  current API contract and new_with_handler limitations.

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

---------

Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <... (continued)

0 of 15 new or added lines in 2 files covered. (0.0%)

22125 of 25781 relevant lines covered (85.82%)

162.99 hits per line

Uncovered Changes

Lines Coverage ∆ File
9
91.9
-1.23% crates/sonde-gateway/src/engine.rs
6
0.0
0.0% crates/sonde-gateway/src/bin/gateway.rs
Jobs
ID Job ID Ran Files Coverage
1 23522479144.1 25 Mar 2026 02:50AM UTC 76
85.82
GitHub Action Run
Source Files on build 23522479144
  • Tree
  • List 76
  • Changed 2
  • Source Changed 2
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • Github Actions Build #23522479144
  • 33b536ed on github
  • Prev Build on main (#23521283250)
  • Next Build on main (#23522541788)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc