fix: deprecate get-telemetry , envoy-filters

Author: krutsko

README.md

@@ -159,8 +159,6 @@ extensions:
 
 
 ### ⚙️ Configuration Resources
-- `get-envoy-filters` - List Envoy Filters in a namespace
-- `get-telemetry` - List Telemetry configurations in a namespace
 - `get-istio-config` - Get comprehensive Istio configuration summary
 
 ### 🔍 Proxy Configuration

pkg/istio/istio.go

@@ -157,37 +157,6 @@ func (i *Istio) GetServiceEntries(ctx context.Context, namespace string) (string
 }
 
 // Configuration resources
-func (i *Istio) GetEnvoyFilters(ctx context.Context, namespace string) (string, error) {
-	efList, err := i.istioClient.NetworkingV1alpha3().EnvoyFilters(namespace).List(ctx, metav1.ListOptions{})
-	if err != nil {
-		return "", fmt.Errorf("failed to list envoy filters: %w", err)
-	}
-
-	result := fmt.Sprintf("Found %d Envoy Filters in namespace '%s':\n", len(efList.Items), namespace)
-	for _, ef := range efList.Items {
-		result += fmt.Sprintf("- %s\n", ef.Name)
-		if ef.Spec.WorkloadSelector != nil && ef.Spec.WorkloadSelector.Labels != nil {
-			result += fmt.Sprintf("  Workload Selector: %v\n", ef.Spec.WorkloadSelector.Labels)
-		}
-	}
-	return result, nil
-}
-
-func (i *Istio) GetTelemetries(ctx context.Context, namespace string) (string, error) {
-	telList, err := i.istioClient.TelemetryV1alpha1().Telemetries(namespace).List(ctx, metav1.ListOptions{})
-	if err != nil {
-		return "", fmt.Errorf("failed to list telemetries: %w", err)
-	}
-
-	result := fmt.Sprintf("Found %d Telemetry configurations in namespace '%s':\n", len(telList.Items), namespace)
-	for _, tel := range telList.Items {
-		result += fmt.Sprintf("- %s\n", tel.Name)
-		if tel.Spec.Selector != nil && tel.Spec.Selector.MatchLabels != nil {
-			result += fmt.Sprintf("  Selector: %v\n", tel.Spec.Selector.MatchLabels)
-		}
-	}
-	return result, nil
-}
 
 // CheckExternalDependencyAvailability checks if an external dependency is properly configured and accessible for a service
 func (i *Istio) CheckExternalDependencyAvailability(ctx context.Context, serviceName, externalHost, namespace string) (string, error) {

pkg/mcp/profile.go

@@ -132,30 +132,6 @@ func (s *Server) initConfigurationTools() []server.ServerTool {
 			),
 			Handler: s.discoverIstioNamespaces,
 		},
-		{
-			Tool: mcp.NewTool("get-envoy-filters",
-				mcp.WithDescription("Get Istio Envoy Filters from any namespace. Envoy Filters allow custom configuration of Envoy proxy behavior, including custom filters, listeners, and clusters. Use this to inspect advanced Istio service mesh configurations."),
-				mcp.WithString("namespace",
-					mcp.Description("Namespace to query (defaults to 'default'). Custom Envoy configurations may be applied to specific namespaces or workloads."),
-				),
-				mcp.WithTitleAnnotation("Istio: Envoy Filters"),
-				mcp.WithReadOnlyHintAnnotation(true),
-				mcp.WithDestructiveHintAnnotation(false),
-			),
-			Handler: s.getEnvoyFilters,
-		},
-		{
-			Tool: mcp.NewTool("get-telemetry",
-				mcp.WithDescription("Get Istio Telemetry configurations from any namespace. Telemetry policies define observability settings including metrics, tracing, and logging for the service mesh. Use this to inspect monitoring and observability configurations."),
-				mcp.WithString("namespace",
-					mcp.Description("Namespace to query (defaults to 'default'). Telemetry policies may be namespace-specific or inherited from mesh-wide settings."),
-				),
-				mcp.WithTitleAnnotation("Istio: Telemetry"),
-				mcp.WithReadOnlyHintAnnotation(true),
-				mcp.WithDestructiveHintAnnotation(false),
-			),
-			Handler: s.getTelemetries,
-		},
 
 		{
 			Tool: mcp.NewTool("check-external-dependency-availability",
@@ -375,23 +351,6 @@ func (s *Server) listServiceMeshHosts(ctx context.Context, ctr mcp.CallToolReque
 // Handler methods for security tools
 
 // Handler methods for configuration tools
-func (s *Server) getEnvoyFilters(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-	namespace := "default"
-	if ns := ctr.GetArguments()["namespace"]; ns != nil {
-		namespace = ns.(string)
-	}
-	content, err := s.i.GetEnvoyFilters(ctx, namespace)
-	return NewTextResult(content, err), nil
-}
-
-func (s *Server) getTelemetries(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-	namespace := "default"
-	if ns := ctr.GetArguments()["namespace"]; ns != nil {
-		namespace = ns.(string)
-	}
-	content, err := s.i.GetTelemetries(ctx, namespace)
-	return NewTextResult(content, err), nil
-}
 
 // Handler method for external dependency availability check
 func (s *Server) checkExternalDependencyAvailability(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {

pkg/mcp/profile_test.go

@@ -85,18 +85,16 @@ func TestFullProfile(t *testing.T) {
 
 			// Check that we have tools from different categories
 			categories := map[string]bool{
-				"networking":    false,
-				"configuration": false,
-				"proxy-config":  false,
+				"networking":   false,
+				"proxy-config": false,
 			}
 
 			for _, tool := range tools {
 				toolName := tool.Tool.Name
 				switch {
 				case toolName == "get-virtual-service" || toolName == "get-destination-rules":
 					categories["networking"] = true
-				case toolName == "get-envoy-filters" || toolName == "get-telemetry":
-					categories["configuration"] = true
+				// No configuration tools currently available
 				case toolName == "get-proxy-clusters" || toolName == "get-proxy-listeners" || toolName == "get-proxy-routes":
 					categories["proxy-config"] = true
 				}