Commit8658c22

and

committed

Fix linting errors in docs.go and docs_test.go

Co-authored-by: colindembovsky <1932561+colindembovsky@users.noreply.github.com>

1 parent93a13ee commit8658c22Copy full SHA for 8658c22

File tree

-5

lines changed

-5

lines changed

Lines changed: 2 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ func SearchGitHubDocs(t translations.TranslationHelperFunc) (tool mcp.Tool, hand`
`56`	`56`	`mcp.Description("Maximum number of results to return (default: 10, max: 100)"),`
`57`	`57`	`),`
`58`	`58`	`),`
`59`		`-func(ctx context.Context,request mcp.CallToolRequest) (*mcp.CallToolResult,error) {`
	`59`	`+func(_ context.Context,request mcp.CallToolRequest) (*mcp.CallToolResult,error) {`
`60`	`60`	`query,err:=RequiredParam[string](request,"query")`
`61`	`61`	`iferr!=nil {`
`62`	`62`	`returnmcp.NewToolResultError(err.Error()),nil`
`@@ -104,6 +104,7 @@ func SearchGitHubDocs(t translations.TranslationHelperFunc) (tool mcp.Tool, hand`
`104`	`104`	`)`
`105`	`105`
`106`	`106`	`// Make the HTTP request`
	`107`	`+// #nosec G107 - URL is constructed from validated parameters with proper escaping`
`107`	`108`	`resp,err:=http.Get(searchURL)`
`108`	`109`	`iferr!=nil {`
`109`	`110`	`returnmcp.NewToolResultError(fmt.Sprintf("failed to search GitHub Docs: %v",err)),nil`

Lines changed: 3 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ func TestSearchGitHubDocs(t *testing.T) {`
`90`	`90`	`expectError:false,`
`91`	`91`	`},`
`92`	`92`	`{`
`93`		`-name:"missing required query parameter",`
	`93`	`+name:"missing required query parameter",`
`94`	`94`	`requestArgs:map[string]interface{}{`
`95`	`95`	`// no query`
`96`	`96`	`},`
`@@ -124,10 +124,10 @@ func TestSearchGitHubDocs(t *testing.T) {`
`124`	`124`	`varhandlerfunc(context.Context, mcp.CallToolRequest) (*mcp.CallToolResult,error)`
`125`	`125`
`126`	`126`	`if!tc.expectError\|\|tc.serverStatus!=0 {`
`127`		`-mockServer=httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter,r*http.Request) {`
	`127`	`+mockServer=httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter,_*http.Request) {`
`128`	`128`	`w.Header().Set("Content-Type","application/json")`
`129`	`129`	`w.WriteHeader(tc.serverStatus)`
`130`		`-json.NewEncoder(w).Encode(tc.serverResponse)`
	`130`	`+_=json.NewEncoder(w).Encode(tc.serverResponse)`
`131`	`131`	`}))`
`132`	`132`	`defermockServer.Close()`
`133`	`133`
`@@ -189,4 +189,3 @@ func TestDocsSearchResponse(t *testing.T) {`
`189`	`189`	`assert.Equal(t,"Test > Article",response.Hits[0].Breadcrumbs)`
`190`	`190`	`assert.Equal(t,"Test content",response.Hits[0].Content)`
`191`	`191`	`}`
`192`		`-`

Comments

(0)