docs(mcp): update tool descriptions for ping, hints, and disconnect

Ping description no longer advertises unimplemented peer ping. Hint
descriptions explain the auto-negotiation model. Disconnect description
clarifies that peers may auto-reconnect after being dropped.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: maxholman

crates/cli/src/repl.rs

@@ -255,7 +255,7 @@ fn print_help() {
 
     let mut tw = TabWriter::new(std::io::stdout());
     let _ = writeln!(tw, "Commands:");
-    let _ = writeln!(tw, "  ping [<peer>]\tPing a peer");
+    let _ = writeln!(tw, "  ping\tPing the daemon (peer ping not yet supported)");
     let _ = writeln!(tw, "  info\tShow daemon info");
     let _ = writeln!(tw, "  version\tShow version");
     let _ = writeln!(tw, "  stats\tShow traffic statistics");
@@ -265,14 +265,20 @@ fn print_help() {
     let _ = writeln!(tw, "  route del <cidr>\tRemove a route");
     let _ = writeln!(tw, "  connect <addr>\tConnect to a peer");
     let _ = writeln!(tw, "  listen <addr>\tStart listening for connections");
-    let _ = writeln!(tw, "  disconnect [peer]\tDisconnect peer");
+    let _ = writeln!(
+        tw,
+        "  disconnect [peer]\tDrop peer session (peer may reconnect)"
+    );
     let _ = writeln!(tw, "  role\tShow current role");
     let _ = writeln!(tw, "  role <entry|exit|relay>\tSet role hint");
     let _ = writeln!(
         tw,
-        "  hint <prefer|exclude|fixed> <role>\tApply a role hint"
+        "  hint <prefer|exclude|fixed> <role>\tSet hint to influence auto-negotiation"
+    );
+    let _ = writeln!(
+        tw,
+        "  hint auto\tRemove all hints, return to automatic negotiation"
     );
-    let _ = writeln!(tw, "  hint auto\tReturn to capability-based negotiation");
     let _ = writeln!(tw, "  shutdown\tShut down the daemon");
     let _ = writeln!(tw, "  help / ?\tShow this help");
     let _ = writeln!(tw, "  quit \tQuit the REPL");

crates/mcp/src/tools.rs

@@ -11,7 +11,7 @@ use crate::convert;
 
 #[derive(Debug, serde::Deserialize, schemars::JsonSchema)]
 pub struct PingParams {
-    /// Peer name prefix to ping. Omit to ping the daemon itself.
+    /// Reserved — peer-specific ping is not yet supported. Leave empty.
     pub peer: Option<String>,
 }
 
@@ -43,7 +43,7 @@ pub struct AddrParams {
 
 #[derive(Debug, serde::Deserialize, schemars::JsonSchema)]
 pub struct HintSetParams {
-    /// Hint level: "prefer", "exclude", or "fixed"
+    /// How strongly to apply the hint: "prefer" (soft), "exclude" (avoid), or "fixed" (force)
     pub level: String,
     /// Target role: "entry", "exit", or "relay"
     pub role: String,
@@ -75,7 +75,9 @@ impl WallhackServer {
         ipc_call(management_request::Request::Info(InfoRequest {})).await
     }
 
-    #[tool(description = "Ping the daemon (or a specific peer by name prefix)")]
+    #[tool(
+        description = "Ping the daemon to check liveness. Returns role, version, and uptime. (Peer-specific ping is not yet supported.)"
+    )]
     async fn ping(
         &self,
         Parameters(params): Parameters<PingParams>,
@@ -151,7 +153,7 @@ impl WallhackServer {
     }
 
     #[tool(
-        description = "Disconnect a peer by name, or disconnect the transport if no peer specified"
+        description = "Disconnect a peer by name (drops the current session; the peer may reconnect automatically). If no peer specified, disconnect the local transport."
     )]
     async fn disconnect(
         &self,
@@ -179,7 +181,7 @@ impl WallhackServer {
     }
 
     #[tool(
-        description = "Set a role hint to influence auto-negotiation (prefer/exclude/fixed + entry/exit/relay)"
+        description = "Set a role hint to influence auto-negotiation: \"prefer\" makes the role more likely, \"exclude\" prevents it, \"fixed\" forces it"
     )]
     async fn hint_set(
         &self,
@@ -214,7 +216,9 @@ impl WallhackServer {
         .await
     }
 
-    #[tool(description = "Return to capability-based negotiation by removing all role hints")]
+    #[tool(
+        description = "Remove all role hints and return to automatic role negotiation based on peer capabilities. Use to undo a previous hint_set."
+    )]
     async fn hint_set_auto(&self) -> Result<String, rmcp::ErrorData> {
         ipc_call(management_request::Request::HintSetAuto(
             HintSetAutoRequest {},