From 40403fc6d59c992ae08a4ba163dca4aa8b9714fc Mon Sep 17 00:00:00 2001 From: atlas Date: Thu, 13 Aug 2026 23:33:08 +0200 Subject: [PATCH] docs(#2916): drop two more doc-comments left dangling by the deletion argus caught these on review. `recent_transient`'s doc still pointed at NO_NODE_LABEL, and `running_transients`' still described destroy as having no queue node behind it and linked suppress_crash_watch -- all three deleted in the parent commit, so the first two were broken intra-doc links and the third was simply false. Migration is now the only operation in that sentence. --- hive-c0re/src/coordinator.rs | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/hive-c0re/src/coordinator.rs b/hive-c0re/src/coordinator.rs index 7be394d5..4f1cf362 100644 --- a/hive-c0re/src/coordinator.rs +++ b/hive-c0re/src/coordinator.rs @@ -137,9 +137,7 @@ pub struct Coordinator { /// live and both clear. Keyed by agent, the last clear *overwrites* the /// others: a `Prebuild` (`deliberate_stop = false`) landing after a /// `StopForUpdate` (`true`) leaves the tombstone reading `false`, and the - /// crash watcher then reports an intentional stop as a **crash**. The - /// out-of-band suppression guard, which has no node behind it, uses - /// [`NO_NODE_LABEL`]. + /// crash watcher then reports an intentional stop as a **crash**. recent_transient: Mutex>, /// Timestamps of recent unexpected container crashes, keyed by agent. /// Fed by `crash_watch` each time it classifies a stop as a crash (so @@ -1247,11 +1245,10 @@ impl Coordinator { /// there is no cached copy to go stale, leak, or disagree with what is /// actually running. /// - /// Work with no queue node behind it (destroy, migration) therefore shows - /// **no pill** — there is nothing in the graph to derive one from. Its - /// crash-watch suppression is a separate, narrower thing - /// ([`Coordinator::suppress_crash_watch`]); the pill comes back for free - /// once those become real nodes. + /// Migration is the last operation with no queue node behind it, so it + /// shows **no pill** — there is nothing in the graph to derive one from. + /// It gets one for free once it becomes real nodes, the way destroy did. + /// /// ⚠️ **A `Vec` per agent, not one entry.** `running_transients` tests /// status alone, so a lease-exempt `Prebuild` for `a` and a lease-holding /// `StopForUpdate` for `a` are both live pills. Collapsing them to one