apunkt/vestige
1
0
Fork 0
mirror of https://github.com/samvallad33/vestige.git synced 2026-04-25 00:36:22 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

chore(server): fix stale tool-count comment (23 -> 24)

Browse source 
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.
This commit is contained in:
Sam Valladares 2026-04-19 16:53:10 -05:00
parent ff0324a0e5
commit 2da0a9a5c5
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
crates/vestige-mcp/src/server.rs
View file

@ -209,7 +209,9 @@ impl McpServer {
/// Handle tools/list request /// Handle tools/list request
async fn handle_tools_list(&self) -> Result<serde_json::Value, JsonRpcError> { async fn handle_tools_list(&self) -> Result<serde_json::Value, JsonRpcError> {
// 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![ let tools = vec![
// ================================================================ // ================================================================
// UNIFIED TOOLS (v1.1+) // UNIFIED TOOLS (v1.1+)