Audit-driven renames:
- DirectRadioControllerImpl -> RadioControllerImpl: the "Direct" prefix
was vestigial (it once contrasted with the AIDL-routed
AndroidRadioControllerImpl, now deleted); it is the sole impl and now
matches its parts (AdminControllerImpl, etc.).
- RadioController.getPacketId() -> generatePacketId(): a `get`-prefixed
function that generates a fresh id each call violates the
getter-is-idempotent convention; also aligns with
CommandSender.generatePacketId().
- RequestController -> QueryController (+ Impl, + the `requestController`
params/mocks): clearer intent for the pull/query surface; "request" was
generic.
- RequestTimer param `label` -> `logLabel`.
- AdminControllerImpl DEFAULT_REBOOT_DELAY -> DEFAULT_DELAY_SECONDS
(shared by reboot + shutdown; conveys the unit).
Interface-consumer-safe; docs/READMEs/architecture guide updated to match.
Koin graph verifies on Android + desktop; affected test suites green.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Update core/repository README and architecture docs to reflect the new
focused provider interfaces (ConnectionStateProvider, TracerouteResponseProvider,
NeighborInfoResponseProvider, ServiceStateWriter) and ViewModel narrowing.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Add a "Radio Control" section to the developer architecture guide
covering how features issue radio commands: the RadioController
composite, its four sub-interfaces (Admin/Messaging/Node/Request), and
DirectRadioControllerImpl as the in-process composition root that
assembles them via interface delegation. Notes the fire-and-forget admin
model and the deliberate alignment with the meshtastic-sdk API shape.
Additive only (no stale-reference fix); bumps last_updated.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
