Fix PAT Scope middleware for unhandled requests & add GraphQL-Features header constant

Author: omgitsads

pkg/http/middleware/pat_scope.go

@@ -41,6 +41,8 @@ func WithPATScopes(logger *slog.Logger, scopeFetcher scopes.FetcherInterface) fu
 
 				next.ServeHTTP(w, r.WithContext(ctx))
 			}
+
+			next.ServeHTTP(w, r)
 		}
 		return http.HandlerFunc(fn)
 	}

pkg/http/transport/graphql_features.go

@@ -5,6 +5,7 @@ import (
 	"strings"
 
 	ghcontext "github.com/github/github-mcp-server/pkg/context"
+	"github.com/github/github-mcp-server/pkg/http/headers"
 )
 
 // GraphQLFeaturesTransport is an http.RoundTripper that adds GraphQL-Features
@@ -44,7 +45,7 @@ func (t *GraphQLFeaturesTransport) RoundTrip(req *http.Request) (*http.Response,
 
 	// Check for GraphQL-Features in context and add header if present
 	if features := ghcontext.GetGraphQLFeatures(req.Context()); len(features) > 0 {
-		req.Header.Set("GraphQL-Features", strings.Join(features, ", "))
+		req.Header.Set(headers.GraphQLFeaturesHeader, strings.Join(features, ", "))
 	}
 
 	return transport.RoundTrip(req)