fix: small changes from local testing

Signed-off-by: Calum Murray <[email protected]>

Author: Cali0707

pkg/mcp/mcp.go

@@ -132,7 +132,7 @@ func (s *Server) reloadKubernetesClusterProvider() error {
 		p.GetDefaultTarget(),
 		p.GetTargetParameterName(),
 		targets,
-		[]string{"configuration_view"}, // TODO: see which tools (if any) do not need the cluster parameter
+		[]string{"configuration_view", "contexts_list"}, // TODO: see which tools (if any) do not need the cluster parameter
 	)
 
 	applicableTools := make([]api.ServerTool, 0)
@@ -153,6 +153,9 @@ func (s *Server) reloadKubernetesClusterProvider() error {
 	}
 
 	s.server.SetTools(m3labsServerTools...)
+
+	// start new watch
+	s.p.WatchTargets(s.reloadKubernetesClusterProvider)
 	return nil
 }
 

pkg/mcp/tool_mutator.go

@@ -10,7 +10,7 @@ import (
 
 type ToolMutator func(tool api.ServerTool) api.ServerTool
 
-const maxTargetsInEnum = 15 // TODO: test and validate that this is a reasonable cutoff
+const maxTargetsInEnum = 5 // TODO: test and validate that this is a reasonable cutoff
 
 func WithTargetParameter(defaultCluster, targetParameterName string, targets, skipToolNames []string) ToolMutator {
 	skipNames := make(map[string]struct{}, len(skipToolNames))