From 72c4bb18c9022656ac4d04125d1b7cdd5731810a Mon Sep 17 00:00:00 2001
From: damocles <damocles@hyperhive.local>
Date: Mon, 29 Jun 2026 00:18:35 +0200
Subject: [PATCH] refactor(#2077): drop impl-history from broker/dashboard/mcp
 comments

---
 hive-ag3nt/src/mcp.rs      |  5 ++---
 hive-c0re/src/broker.rs    |  4 ++--
 hive-c0re/src/dashboard.rs | 11 ++++-------
 3 files changed, 8 insertions(+), 12 deletions(-)

diff --git a/hive-ag3nt/src/mcp.rs b/hive-ag3nt/src/mcp.rs
index fb5a1b49..610e26c3 100644
--- a/hive-ag3nt/src/mcp.rs
+++ b/hive-ag3nt/src/mcp.rs
@@ -535,9 +535,8 @@ fn format_age_secs(secs: u64) -> String {
 }
 
 /// Common envelope around every MCP tool handler: pre-log → run →
-/// post-log. The inbox-status hint used to be appended to every tool
-/// result; that lives in the wake prompt + UI header now, so tool
-/// results stay clean.
+/// post-log. Tool results stay clean — the inbox-status hint lives in
+/// the wake prompt + UI header, not appended here.
 pub async fn run_tool_envelope<F>(tool: &'static str, args: String, body: F) -> String
 where
     F: Future<Output = String>,
diff --git a/hive-c0re/src/broker.rs b/hive-c0re/src/broker.rs
index 3f28787b..be3c9b42 100644
--- a/hive-c0re/src/broker.rs
+++ b/hive-c0re/src/broker.rs
@@ -1260,8 +1260,8 @@ mod tests {
     }
 
     /// Convenience wrapper for tests that want single-pop semantics
-    /// — the broker only exposes `recv_batch` publicly now, so
-    /// every test that used to call `recv` goes through here.
+    /// — the broker only exposes `recv_batch` publicly, so tests
+    /// funnel single pops through here.
     fn pop_one(broker: &Broker, recipient: &str) -> Option<Delivery> {
         let mut batch = broker.recv_batch(recipient, 1).unwrap();
         batch.pop()
diff --git a/hive-c0re/src/dashboard.rs b/hive-c0re/src/dashboard.rs
index c6da06a8..61d8dd49 100644
--- a/hive-c0re/src/dashboard.rs
+++ b/hive-c0re/src/dashboard.rs
@@ -563,13 +563,10 @@ async fn api_state(headers: HeaderMap, State(state): State<AppState>) -> axum::J
     let tombstones = build_tombstone_views(&state.coord, &containers, &transient_snapshot);
     let port_conflicts = build_port_conflicts(&containers);
 
-    // operator_inbox used to be served here as a 50-row array; the
-    // dashboard now derives it client-side from the message stream
-    // (terminal backfill + live SSE), so the snapshot stops shipping it.
-    // Both operator-targeted and peer threads now surface on the
-    // dashboard. Client filters by target client-side. Each row is
-    // wrapped in QuestionView so the snapshot carries the same
-    // file_refs the live event variants attach.
+    // Both operator-targeted and peer threads surface on the dashboard
+    // (the client filters by target). Each row is wrapped in QuestionView
+    // so the snapshot carries the same file_refs the live event variants
+    // attach.
     let questions: Vec<QuestionView> =
         log_default("questions.pending_all", state.coord.questions.pending_all())
             .into_iter()