1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
//! Client-side stale-daemon version check for `tm doctor` (issue #2332).
//!
//! Why: split out of `commands::misc` to keep that file under the 500-SLOC
//! production cap (`misc.rs` was already near the ceiling before this
//! addition). This check must live in the `tm` CLI binary — not the daemon's
//! server-side `run_doctor` — because it needs THIS process's own
//! `CARGO_PKG_VERSION`: `tm doctor` always executes as the just-installed
//! binary, so its compiled-in version is the "installed" side of the
//! comparison; the daemon can only ever report on itself.
//! What: [`stale_daemon_check`] fetches the running daemon's `/health`
//! version via [`trusty_mpm::client::DaemonClient::health_snapshot`] and
//! folds it into a [`trusty_mpm::core::doctor::DoctorCheck`] via
//! [`trusty_mpm::core::version_staleness::check_daemon_version_staleness`].
//! Test: the pure comparison logic is covered by
//! `core::version_staleness`'s own unit tests; the network fetch itself is
//! exercised indirectly by the executor's live-daemon doctor test.
use trusty_mpm::client::DaemonClient;
use trusty_mpm::core::doctor::{CheckStatus, DoctorCheck};
use trusty_mpm::core::version_staleness::{CHECK_NAME, check_daemon_version_staleness};
/// Fetch the daemon's `/health` version and fold it into the #2332
/// stale-daemon [`DoctorCheck`].
///
/// Why: separates the network fetch (untestable without a live daemon) from
/// the pure comparison in [`trusty_mpm::core::version_staleness`], and keeps
/// `commands::misc::doctor` itself free of the "daemon unreachable" branch.
/// What: calls `daemon.health_snapshot()`; on success, compares its `version`
/// against this process's own `env!("CARGO_PKG_VERSION")`. On a transport
/// failure, returns `Warn` — the daemon is presumably unreachable, which the
/// caller's own `report.checks` output already explains in detail; this just
/// keeps the stale-daemon line from silently vanishing.
/// Test: the comparison logic is covered by `core::version_staleness`'s unit
/// tests; the network fetch itself is exercised indirectly by the executor's
/// live-daemon doctor test.
pub(crate) async fn stale_daemon_check(daemon: &DaemonClient) -> DoctorCheck {
match daemon.health_snapshot().await {
Ok(snapshot) => {
check_daemon_version_staleness(env!("CARGO_PKG_VERSION"), &snapshot.version)
}
Err(e) => DoctorCheck::new(
CHECK_NAME,
CheckStatus::Warn,
format!("could not fetch daemon version from /health: {e}"),
),
}
}