Fix linting errors in docs.go and docs_test.go

Co-authored-by: colindembovsky <[email protected]>

Author: Copilot

pkg/github/docs.go

@@ -56,7 +56,7 @@ func SearchGitHubDocs(t translations.TranslationHelperFunc) (tool mcp.Tool, hand
 				mcp.Description("Maximum number of results to return (default: 10, max: 100)"),
 			),
 		),
-		func(ctx context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
+		func(_ context.Context, request mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 			query, err := RequiredParam[string](request, "query")
 			if err != nil {
 				return mcp.NewToolResultError(err.Error()), nil
@@ -104,6 +104,7 @@ func SearchGitHubDocs(t translations.TranslationHelperFunc) (tool mcp.Tool, hand
 			)
 
 			// Make the HTTP request
+			// #nosec G107 - URL is constructed from validated parameters with proper escaping
 			resp, err := http.Get(searchURL)
 			if err != nil {
 				return mcp.NewToolResultError(fmt.Sprintf("failed to search GitHub Docs: %v", err)), nil

pkg/github/docs_test.go

@@ -90,7 +90,7 @@ func TestSearchGitHubDocs(t *testing.T) {
 			expectError:    false,
 		},
 		{
-			name: "missing required query parameter",
+			name:        "missing required query parameter",
 			requestArgs: map[string]interface{}{
 				// no query
 			},
@@ -124,10 +124,10 @@ func TestSearchGitHubDocs(t *testing.T) {
 			var handler func(context.Context, mcp.CallToolRequest) (*mcp.CallToolResult, error)
 
 			if !tc.expectError || tc.serverStatus != 0 {
-				mockServer = httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+				mockServer = httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 					w.Header().Set("Content-Type", "application/json")
 					w.WriteHeader(tc.serverStatus)
-					json.NewEncoder(w).Encode(tc.serverResponse)
+					_ = json.NewEncoder(w).Encode(tc.serverResponse)
 				}))
 				defer mockServer.Close()
 
@@ -189,4 +189,3 @@ func TestDocsSearchResponse(t *testing.T) {
 	assert.Equal(t, "Test > Article", response.Hits[0].Breadcrumbs)
 	assert.Equal(t, "Test content", response.Hits[0].Content)
 }
-