fix linter issues

Author: omgitsads

internal/ghmcp/server.go

@@ -143,7 +143,7 @@ func NewMCPServer(cfg MCPServerConfig) (*mcp.Server, error) {
 	ghServer := github.NewServer(cfg.Version, &mcp.ServerOptions{
 		Instructions:      instructions,
 		Logger:            cfg.Logger,
-		CompletionHandler: github.GitHubCompletionsHandler(getClient),
+		CompletionHandler: github.CompletionsHandler(getClient),
 	})
 
 	// Add middlewares

pkg/github/repository_resource_completions_test.go

@@ -82,7 +82,7 @@ func TestRepositoryResourceCompletionHandler(t *testing.T) {
 }
 
 func TestRepositoryResourceCompletionHandler_GetClientError(t *testing.T) {
-	getClient := func(ctx context.Context) (*github.Client, error) {
+	getClient := func(_ context.Context) (*github.Client, error) {
 		return nil, errors.New("client error")
 	}
 
@@ -246,7 +246,7 @@ func TestRepositoryResourceArgumentResolvers_Existence(t *testing.T) {
 
 func TestRepositoryResourceCompletionHandler_MaxResults(t *testing.T) {
 	// Test that results are limited to 100 items
-	getClient := func(ctx context.Context) (*github.Client, error) {
+	getClient := func(_ context.Context) (*github.Client, error) {
 		return &github.Client{}, nil
 	}
 

pkg/github/server.go

@@ -27,15 +27,14 @@ func NewServer(version string, opts *mcp.ServerOptions) *mcp.Server {
 	return s
 }
 
-func GitHubCompletionsHandler(getClient GetClientFn) func(ctx context.Context, req *mcp.CompleteRequest) (*mcp.CompleteResult, error) {
+func CompletionsHandler(getClient GetClientFn) func(ctx context.Context, req *mcp.CompleteRequest) (*mcp.CompleteResult, error) {
 	return func(ctx context.Context, req *mcp.CompleteRequest) (*mcp.CompleteResult, error) {
 		switch req.Params.Ref.Type {
 		case "ref/resource":
 			if strings.HasPrefix(req.Params.Ref.URI, "repo://") {
 				return RepositoryResourceCompletionHandler(getClient)(ctx, req)
-			} else {
-				return nil, fmt.Errorf("unsupported resource URI: %s", req.Params.Ref.URI)
 			}
+			return nil, fmt.Errorf("unsupported resource URI: %s", req.Params.Ref.URI)
 		case "ref/prompt":
 			return nil, nil
 		default: