fixing name

Author: tonytrg

cmd/github-mcp-server/main.go

@@ -83,7 +83,7 @@ var (
 				LogFilePath:          viper.GetString("log-file"),
 				ContentWindowSize:    viper.GetInt("content-window-size"),
 				LockdownMode:         viper.GetBool("lockdown-mode"),
-				Experimental:         viper.GetBool("experimental"),
+				InsiderMode:          viper.GetBool("insider-mode"),
 				RepoAccessCacheTTL:   &ttl,
 			}
 			return ghmcp.RunStdioServer(stdioServerConfig)
@@ -109,7 +109,7 @@ func init() {
 	rootCmd.PersistentFlags().String("gh-host", "", "Specify the GitHub hostname (for GitHub Enterprise etc.)")
 	rootCmd.PersistentFlags().Int("content-window-size", 5000, "Specify the content window size")
 	rootCmd.PersistentFlags().Bool("lockdown-mode", false, "Enable lockdown mode")
-	rootCmd.PersistentFlags().Bool("experimental", false, "Enable experimental features")
+	rootCmd.PersistentFlags().Bool("insider-mode", false, "Enable insider features")
 	rootCmd.PersistentFlags().Duration("repo-access-cache-ttl", 5*time.Minute, "Override the repo access cache TTL (e.g. 1m, 0s to disable)")
 
 	// Bind flag to viper
@@ -124,7 +124,7 @@ func init() {
 	_ = viper.BindPFlag("host", rootCmd.PersistentFlags().Lookup("gh-host"))
 	_ = viper.BindPFlag("content-window-size", rootCmd.PersistentFlags().Lookup("content-window-size"))
 	_ = viper.BindPFlag("lockdown-mode", rootCmd.PersistentFlags().Lookup("lockdown-mode"))
-	_ = viper.BindPFlag("experimental", rootCmd.PersistentFlags().Lookup("experimental"))
+	_ = viper.BindPFlag("insider-mode", rootCmd.PersistentFlags().Lookup("insider-mode"))
 	_ = viper.BindPFlag("repo-access-cache-ttl", rootCmd.PersistentFlags().Lookup("repo-access-cache-ttl"))
 
 	// Add subcommands

internal/ghmcp/server.go

@@ -64,8 +64,8 @@ type MCPServerConfig struct {
 	// LockdownMode indicates if we should enable lockdown mode
 	LockdownMode bool
 
-	// Experimental indicates if we should enable experimental features
-	Experimental bool
+	// Insider indicates if we should enable experimental features
+	InsiderMode bool
 
 	// Logger is used for logging within the server
 	Logger *slog.Logger
@@ -213,7 +213,7 @@ func NewMCPServer(cfg MCPServerConfig) (*mcp.Server, error) {
 		cfg.Translator,
 		github.FeatureFlags{
 			LockdownMode: cfg.LockdownMode,
-			Experimental: cfg.Experimental,
+			InsiderMode: cfg.InsiderMode,
 		},
 		cfg.ContentWindowSize,
 		featureChecker,
@@ -332,8 +332,8 @@ type StdioServerConfig struct {
 	// LockdownMode indicates if we should enable lockdown mode
 	LockdownMode bool
 
-	// Experimental indicates if we should enable experimental features
-	Experimental bool
+	// InsiderMode indicates if we should enable experimental features
+	InsiderMode bool
 
 	// RepoAccessCacheTTL overrides the default TTL for repository access cache entries.
 	RepoAccessCacheTTL *time.Duration
@@ -391,7 +391,7 @@ func RunStdioServer(cfg StdioServerConfig) error {
 		Translator:        t,
 		ContentWindowSize: cfg.ContentWindowSize,
 		LockdownMode:      cfg.LockdownMode,
-		Experimental:      cfg.Experimental,
+		InsiderMode:       cfg.InsiderMode,
 		Logger:            logger,
 		RepoAccessTTL:     cfg.RepoAccessCacheTTL,
 		TokenScopes:       tokenScopes,

internal/ghmcp/server_test.go

@@ -23,7 +23,7 @@ func TestNewMCPServer_CreatesSuccessfully(t *testing.T) {
 		Translator:        translations.NullTranslationHelper,
 		ContentWindowSize: 5000,
 		LockdownMode:      false,
-		Experimental:      false,
+		InsiderMode:       false,
 	}
 
 	// Create the server

pkg/github/feature_flags.go

@@ -3,5 +3,5 @@ package github
 // FeatureFlags defines runtime feature toggles that adjust tool behavior.
 type FeatureFlags struct {
 	LockdownMode bool
-	Experimental bool
+	InsiderMode  bool
 }

pkg/github/feature_flags_test.go

@@ -15,8 +15,7 @@ import (
 	"github.com/github/github-mcp-server/pkg/utils"
 )
 
-// RemoteMCPExperimental is a long-lived feature flag for experimental remote MCP features.
-// This flag enables experimental behaviors in tools that are being tested for remote server deployment.
+// RemoteMCPEnthusiasticGreeting is a dummy test feature flag .
 const RemoteMCPEnthusiasticGreeting = "remote_mcp_enthusiastic_greeting"
 
 // FeatureChecker is an interface for checking if a feature flag is enabled.
@@ -46,7 +45,7 @@ func HelloWorldTool(t translations.TranslationHelperFunc) inventory.ServerTool {
 			if deps.IsFeatureEnabled(ctx, RemoteMCPEnthusiasticGreeting) {
 				greeting += " Welcome to the future of MCP! 🎉"
 			}
-			if deps.GetFlags().Experimental {
+			if deps.GetFlags().InsiderMode {
 				greeting += " Experimental features are enabled! 🚀"
 			}
 
@@ -114,10 +113,10 @@ func TestHelloWorld_ConditionalBehavior_Featureflag(t *testing.T) {
 			// Call the handler with deps in context
 			ctx := ContextWithDeps(context.Background(), deps)
 			result, err := handler(ctx, &mcp.CallToolRequest{
-			Params: &mcp.CallToolParamsRaw{
-				Arguments: json.RawMessage(`{}`),
-			},
-		})
+				Params: &mcp.CallToolParamsRaw{
+					Arguments: json.RawMessage(`{}`),
+				},
+			})
 			require.NoError(t, err)
 			require.NotNil(t, result)
 			require.Len(t, result.Content, 1)
@@ -141,17 +140,17 @@ func TestHelloWorld_ConditionalBehavior_Config(t *testing.T) {
 
 	tests := []struct {
 		name             string
-		experimental     bool
+		insiderMode      bool
 		expectedGreeting string
 	}{
 		{
 			name:             "Experimental disabled - default greeting",
-			experimental:     false,
+			insiderMode:      false,
 			expectedGreeting: "Hello, world!",
 		},
 		{
 			name:             "Experimental enabled - experimental greeting",
-			experimental:     true,
+			insiderMode:      true,
 			expectedGreeting: "Hello, world! Experimental features are enabled! 🚀",
 		},
 	}
@@ -164,7 +163,7 @@ func TestHelloWorld_ConditionalBehavior_Config(t *testing.T) {
 			deps := NewBaseDeps(
 				nil, nil, nil, nil,
 				translations.NullTranslationHelper,
-				FeatureFlags{Experimental: tt.experimental},
+				FeatureFlags{InsiderMode: tt.insiderMode},
 				0,
 				nil,
 			)
@@ -176,10 +175,10 @@ func TestHelloWorld_ConditionalBehavior_Config(t *testing.T) {
 			// Call the handler with deps in context
 			ctx := ContextWithDeps(context.Background(), deps)
 			result, err := handler(ctx, &mcp.CallToolRequest{
-			Params: &mcp.CallToolParamsRaw{
-				Arguments: json.RawMessage(`{}`),
-			},
-		})
+				Params: &mcp.CallToolParamsRaw{
+					Arguments: json.RawMessage(`{}`),
+				},
+			})
 			require.NoError(t, err)
 			require.NotNil(t, result)
 			require.Len(t, result.Content, 1)

pkg/github/server_test.go

@@ -84,7 +84,7 @@ func stubRepoAccessCache(client *githubv4.Client, ttl time.Duration) *lockdown.R
 func stubFeatureFlags(enabledFlags map[string]bool) FeatureFlags {
 	return FeatureFlags{
 		LockdownMode: enabledFlags["lockdown-mode"],
-		Experimental: enabledFlags["experimental"],
+		InsiderMode:  enabledFlags["insider-mode"],
 	}
 }