From 1471cb2365b84f97b8c3db58347bb91c746720c3 Mon Sep 17 00:00:00 2001 From: Alon Girmonsky <1990761+alongir@users.noreply.github.com> Date: Thu, 5 Feb 2026 21:04:32 -0800 Subject: [PATCH] Fix lint errors in mcpRunner.go MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Use type conversion instead of struct literals for hubMCPTool -> mcpTool and hubMCPPromptArg -> mcpPromptArg (S1016 gosimple) - Lowercase error string to follow Go conventions (ST1005 staticcheck) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- cmd/mcpRunner.go | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/cmd/mcpRunner.go b/cmd/mcpRunner.go index a32105754..0a826e100 100644 --- a/cmd/mcpRunner.go +++ b/cmd/mcpRunner.go @@ -212,7 +212,7 @@ func (s *mcpServer) validateDirectURL() error { // Verify it looks like a valid MCP response if mcpInfo.Name == "" && len(mcpInfo.Tools) == 0 { - return fmt.Errorf("Kubeshark at %s does not appear to have MCP enabled", urlStr) + return fmt.Errorf("kubeshark at %s does not appear to have MCP enabled", urlStr) } // Set the hub base URL @@ -497,11 +497,7 @@ func (s *mcpServer) handleListTools(req *jsonRPCRequest) { // Fetch tools from Hub and merge if hubMCP := s.fetchHubMCP(); hubMCP != nil { for _, hubTool := range hubMCP.Tools { - tools = append(tools, mcpTool{ - Name: hubTool.Name, - Description: hubTool.Description, - InputSchema: hubTool.InputSchema, - }) + tools = append(tools, mcpTool(hubTool)) } } @@ -522,11 +518,7 @@ func (s *mcpServer) handleListPrompts(req *jsonRPCRequest) { for _, hubPrompt := range hubMCP.Prompts { var args []mcpPromptArg for _, hubArg := range hubPrompt.Arguments { - args = append(args, mcpPromptArg{ - Name: hubArg.Name, - Description: hubArg.Description, - Required: hubArg.Required, - }) + args = append(args, mcpPromptArg(hubArg)) } prompts = append(prompts, mcpPrompt{ Name: hubPrompt.Name,