Alan-Jowett / sonde / 23564908922

81%

Build Type

push

github

Committed by web-flow

Commit Message

feat(gateway): wire up ExportState, FactoryReset, and health monitor (#505)

* feat(gateway): wire up ExportState, FactoryReset, and health monitor (#452)

F-001: ExportState now calls ncrypt_state_full() instead of
ncrypt_state(), including gateway identity, phone PSKs, and handler
routing configs in the encrypted bundle. Added with_handler_configs()
builder on AdminService and wired it in gateway.rs.

F-002: Added FactoryReset RPC to admin proto, AdminService, and
sonde-admin CLI (\
ode factory-reset\). Removes the node from the
registry and clears pending commands. A TODO notes that node-side
protocol-level erasure requires a future sonde-protocol extension.

F-003: spawn_health_monitor() (GW-1102) is now spawned inside the
modem reconnect loop in \gateway.rs\, polling modem status every 30 s
and logging tx-failure deltas and reboot detection.

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

* fix(gateway): invalidate sessions on node removal and factory reset

Add `session_manager.remove_session()` calls to both `remove_node` and`nfactory_reset` RPCs so that active in-memory sessions are immediately`ninvalidated when a node is removed from the registry.

Without this, a node with an active session could continue communicating`nuntil the session times out naturally, violating GW-0705 AC1.

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

* Address review findings

- Remove dead PendingCommand::FactoryReset variant that was never
  handled in select_command or queued by the RPC (the protocol-level
  factory-reset command is not yet defined).

- Add explanatory comment to actory_reset existence-check block
  matching the pattern in emove_node, documenting why the full
  record is loaded (no lightweight exists query in Storage trait).

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

---------

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

Run Details

9 of 52 new or added lines in 4 files covered. (17.31%)

5 existing lines in 2 files now uncovered.

22164 of 25864 relevant lines covered (85.69%)

163.14 hits per line