Lint the code

Author: JoannaaKL

internal/ghmcp/server.go

@@ -118,8 +118,8 @@ func NewMCPServer(cfg MCPServerConfig) (*server.MCPServer, error) {
 
 	// Generate instructions based on enabled toolsets
 	instructions := github.GenerateInstructions(enabledToolsets)
-	
-	ghServer := github.NewServer(cfg.Version, 
+
+	ghServer := github.NewServer(cfg.Version,
 		server.WithInstructions(instructions),
 		server.WithHooks(hooks),
 	)

pkg/github/instructions.go

@@ -12,21 +12,21 @@ func GenerateInstructions(enabledToolsets []string) string {
 	if os.Getenv("DISABLE_INSTRUCTIONS") == "true" {
 		return "" // Baseline mode
 	}
-	
+
 	var instructions []string
-	
+
 	// Core instruction - always included if context toolset enabled
 	if slices.Contains(enabledToolsets, "context") {
 		instructions = append(instructions, "Always call 'get_me' first to understand current user permissions and context.")
 	}
-	
+
 	// Individual toolset instructions
 	for _, toolset := range enabledToolsets {
 		if inst := getToolsetInstructions(toolset); inst != "" {
 			instructions = append(instructions, inst)
 		}
 	}
-	
+
 	// Base instruction with context management
 	baseInstruction := `The GitHub MCP Server provides tools to interact with GitHub platform.
 
@@ -40,7 +40,7 @@ Context management:
 
 	allInstructions := []string{baseInstruction}
 	allInstructions = append(allInstructions, instructions...)
-	
+
 	return strings.Join(allInstructions, " ")
 }
 
@@ -57,4 +57,3 @@ func getToolsetInstructions(toolset string) string {
 		return ""
 	}
 }
-

pkg/github/instructions_test.go

@@ -163,4 +163,4 @@ func TestGetToolsetInstructions(t *testing.T) {
 			}
 		})
 	}
-}
+}

pkg/github/projects_test.go

@@ -1188,7 +1188,7 @@ func Test_DeleteProjectItem(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(
 				mock.WithRequestMatchHandler(
 					mock.EndpointPattern{Pattern: "/orgs/{org}/projectsV2/{project}/items/{item_id}", Method: http.MethodDelete},
-					http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+					http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 						w.WriteHeader(http.StatusNoContent)
 					}),
 				),
@@ -1206,7 +1206,7 @@ func Test_DeleteProjectItem(t *testing.T) {
 			mockedClient: mock.NewMockedHTTPClient(
 				mock.WithRequestMatchHandler(
 					mock.EndpointPattern{Pattern: "/users/{user}/projectsV2/{project}/items/{item_id}", Method: http.MethodDelete},
-					http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+					http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
 						w.WriteHeader(http.StatusNoContent)
 					}),
 				),