remove old flags

Author: mattdholloway

pkg/github/actions.go

@@ -26,10 +26,6 @@ const (
 	DescriptionRepositoryName  = "Repository name"
 )
 
-// FeatureFlagConsolidatedActions is the legacy feature flag (deprecated, no longer used).
-// Kept for documentation purposes only.
-const FeatureFlagConsolidatedActions = "remote_mcp_consolidated_actions"
-
 // FeatureFlagHoldbackConsolidatedActions is the feature flag that, when enabled, reverts to
 // individual actions tools instead of the consolidated actions tools.
 const FeatureFlagHoldbackConsolidatedActions = "mcp_holdback_consolidated_actions"

pkg/github/projects.go

@@ -27,10 +27,6 @@ const (
 	MaxProjectsPerPage       = 50
 )
 
-// FeatureFlagConsolidatedProjects is the legacy feature flag (deprecated, no longer used).
-// Kept for documentation purposes only.
-const FeatureFlagConsolidatedProjects = "remote_mcp_consolidated_projects"
-
 // FeatureFlagHoldbackConsolidatedProjects is the feature flag that, when enabled, reverts to
 // individual project tools instead of the consolidated project tools.
 const FeatureFlagHoldbackConsolidatedProjects = "mcp_holdback_consolidated_projects"

pkg/http/features.go

@@ -0,0 +1,42 @@
+package http
+
+import (
+	"context"
+	"slices"
+
+	"github.com/github/github-mcp-server/pkg/github"
+	"github.com/github/github-mcp-server/pkg/inventory"
+)
+
+// KnownFeatureFlags are the feature flags that can be enabled via X-MCP-Features header.
+var KnownFeatureFlags = []string{
+	github.FeatureFlagHoldbackConsolidatedProjects,
+	github.FeatureFlagHoldbackConsolidatedActions,
+}
+
+// ComposeFeatureChecker combines header-based feature flags with a static checker.
+func ComposeFeatureChecker(headerFeatures []string, staticChecker inventory.FeatureFlagChecker) inventory.FeatureFlagChecker {
+	if len(headerFeatures) == 0 && staticChecker == nil {
+		return nil
+	}
+
+	// Only accept header features that are in the known list
+	headerSet := make(map[string]bool, len(headerFeatures))
+	for _, f := range headerFeatures {
+		if slices.Contains(KnownFeatureFlags, f) {
+			headerSet[f] = true
+		}
+	}
+
+	return func(ctx context.Context, flag string) (bool, error) {
+		// Header-based: static string matching
+		if headerSet[flag] {
+			return true, nil
+		}
+		// Static checker
+		if staticChecker != nil {
+			return staticChecker(ctx, flag)
+		}
+		return false, nil
+	}
+}