Remove extra config fields from HTTP server config

Author: omgitsads

pkg/http/handler.go

@@ -60,11 +60,6 @@ func (s *HttpMcpHandler) ServeHTTP(w http.ResponseWriter, r *http.Request) {
 	ghServer, err := github.NewMCPServer(&github.MCPServerConfig{
 		Version:           s.config.Version,
 		Host:              s.config.Host,
-		EnabledToolsets:   s.config.EnabledToolsets,
-		EnabledTools:      s.config.EnabledTools,
-		EnabledFeatures:   s.config.EnabledFeatures,
-		DynamicToolsets:   s.config.DynamicToolsets,
-		ReadOnly:          s.config.ReadOnly,
 		Translator:        s.t,
 		ContentWindowSize: s.config.ContentWindowSize,
 		Logger:            s.logger,

pkg/http/server.go

@@ -24,25 +24,6 @@ type HTTPServerConfig struct {
 	// GitHub Host to target for API requests (e.g. github.com or github.enterprise.com)
 	Host string
 
-	// EnabledToolsets is a list of toolsets to enable
-	// See: https://github.com/github/github-mcp-server?tab=readme-ov-file#tool-configuration
-	EnabledToolsets []string
-
-	// EnabledTools is a list of specific tools to enable (additive to toolsets)
-	// When specified, these tools are registered in addition to any specified toolset tools
-	EnabledTools []string
-
-	// EnabledFeatures is a list of feature flags that are enabled
-	// Items with FeatureFlagEnable matching an entry in this list will be available
-	EnabledFeatures []string
-
-	// Whether to enable dynamic toolsets
-	// See: https://github.com/github/github-mcp-server?tab=readme-ov-file#dynamic-tool-discovery
-	DynamicToolsets bool
-
-	// ReadOnly indicates if we should only register read-only tools
-	ReadOnly bool
-
 	// ExportTranslations indicates if we should export translations
 	// See: https://github.com/github/github-mcp-server?tab=readme-ov-file#i18n--overriding-descriptions
 	ExportTranslations bool
@@ -84,7 +65,7 @@ func RunHTTPServer(cfg HTTPServerConfig) error {
 		slogHandler = slog.NewTextHandler(logOutput, &slog.HandlerOptions{Level: slog.LevelInfo})
 	}
 	logger := slog.New(slogHandler)
-	logger.Info("starting server", "version", cfg.Version, "host", cfg.Host, "dynamicToolsets", cfg.DynamicToolsets, "readOnly", cfg.ReadOnly, "lockdownEnabled", cfg.LockdownMode)
+	logger.Info("starting server", "version", cfg.Version, "host", cfg.Host, "lockdownEnabled", cfg.LockdownMode)
 
 	apiHost, err := utils.ParseAPIHost(cfg.Host)
 	if err != nil {