From 2da0a9a5c5ce970cfd709ad0ffce1ebac974e25c Mon Sep 17 00:00:00 2001 From: Sam Valladares Date: Sun, 19 Apr 2026 16:53:10 -0500 Subject: [PATCH] chore(server): fix stale tool-count comment (23 -> 24) server.rs:212 was documenting v2.0.4's state (23 tools) but the handle_tools_list test two functions below asserts 24 tools since v2.0.5 (when `suppress` landed). Align the comment with the assertion and point at the assertion as the source of truth so future tool additions can't introduce comment drift without also tripping a test failure. --- crates/vestige-mcp/src/server.rs | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/crates/vestige-mcp/src/server.rs b/crates/vestige-mcp/src/server.rs index c6d87b1..711b37a 100644 --- a/crates/vestige-mcp/src/server.rs +++ b/crates/vestige-mcp/src/server.rs @@ -209,7 +209,9 @@ impl McpServer { /// Handle tools/list request async fn handle_tools_list(&self) -> Result { - // v2.0.4: 23 tools. Deprecated tools still work via redirects in handle_tools_call. + // v2.0.5+: 24 tools (verified by the `tools.len() == 24` assertion in the + // handle_tools_list test below — the `suppress` tool landed in v2.0.5). + // Deprecated tools still work via redirects in handle_tools_call. let tools = vec![ // ================================================================ // UNIFIED TOOLS (v1.1+)