fix: remove unused arguments variable in HandleToolsCallAsync

gkaleta · gkaleta · commit 39e877b18814 · 2026-02-13T08:51:05.000+01:00
The switch only has a default case (stub for future tools), so arguments was never read. Re-add when tool implementations are added. Resolves github-code-quality warning on PR #22.
diff --git a/Mcp/McpServer.cs b/Mcp/McpServer.cs
@@ -412,7 +412,6 @@ private async Task HandleToolsCallAsync(JsonRpcRequest request)
         }
 
         var name = request.Params.Value.GetProperty("name").GetString();
-        var arguments = request.Params.Value.GetProperty("arguments");
 
         switch (name)
         {

Original file line number	Diff line number	Diff line change
`@@ -412,7 +412,6 @@ private async Task HandleToolsCallAsync(JsonRpcRequest request)`
`412`	`412`	`}`
`413`	`413`
`414`	`414`	`var name = request.Params.Value.GetProperty("name").GetString();`
`415`		`- var arguments = request.Params.Value.GetProperty("arguments");`
`416`	`415`
`417`	`416`	`switch (name)`
`418`	`417`	`{`