fix: remove "get-service-entries"

Author: krutsko

README.md

@@ -153,9 +153,8 @@ extensions:
 ## 🛠️ Available Tools
 
 ### 🌐 Networking Resources
-- `get-virtual-services` - List Virtual Services in a namespace
-- `get-destination-rules` - List Destination Rules in a namespace  
-- `get-service-entries` - List Service Entries in a namespace
+- `get-virtual-service` - Get a specific Virtual Service configuration by name
+- `get-destination-rule` - Get a specific Destination Rule configuration by name
 
 
 ### ⚙️ Configuration Resources

pkg/istio/istio.go

@@ -171,25 +171,6 @@ func (i *Istio) GetDestinationRule(ctx context.Context, namespace string, drName
 	return result, nil
 }
 
-func (i *Istio) GetServiceEntries(ctx context.Context, namespace string) (string, error) {
-	seList, err := i.istioClient.NetworkingV1alpha3().ServiceEntries(namespace).List(ctx, metav1.ListOptions{})
-	if err != nil {
-		return "", fmt.Errorf("failed to list service entries: %w", err)
-	}
-
-	result := fmt.Sprintf("Found %d Service Entries in namespace '%s':\n", len(seList.Items), namespace)
-	for _, se := range seList.Items {
-		result += fmt.Sprintf("- %s\n", se.Name)
-		if se.Spec.Hosts != nil {
-			result += fmt.Sprintf("  Hosts: %v\n", se.Spec.Hosts)
-		}
-		if se.Spec.Location.String() != "" {
-			result += fmt.Sprintf("  Location: %s\n", se.Spec.Location.String())
-		}
-	}
-	return result, nil
-}
-
 // Configuration resources
 
 // CheckExternalDependencyAvailability checks if an external dependency is properly configured and accessible for a service

pkg/mcp/profile.go

@@ -92,18 +92,6 @@ func (s *Server) initNetworkingTools() []server.ServerTool {
 			),
 			Handler: s.getDestinationRule,
 		},
-		{
-			Tool: mcp.NewTool("get-service-entries",
-				mcp.WithDescription("Get Istio Service Entries from any namespace. Service Entries allow adding external services to the service mesh registry. Use this to inspect external service configurations and mesh expansion settings."),
-				mcp.WithString("namespace",
-					mcp.Description("Namespace to query (defaults to 'default'). External service configurations may be centralized in specific namespaces."),
-				),
-				mcp.WithTitleAnnotation("Istio: Service Entries"),
-				mcp.WithReadOnlyHintAnnotation(true),
-				mcp.WithDestructiveHintAnnotation(false),
-			),
-			Handler: s.getServiceEntries,
-		},
 		{
 			Tool: mcp.NewTool("get-service-mesh-hosts",
 				mcp.WithDescription("List all services and hosts in the service mesh for a specific namespace. Provides a comprehensive overview of ServiceEntries (external services), Kubernetes Services (internal services), and VirtualService configurations. Use this to understand the complete service topology and routing setup in a namespace."),
@@ -343,15 +331,6 @@ func (s *Server) getDestinationRule(ctx context.Context, ctr mcp.CallToolRequest
 	return NewTextResult(content, err), nil
 }
 
-func (s *Server) getServiceEntries(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.GetServiceEntries(ctx, namespace)
-	return NewTextResult(content, err), nil
-}
-
 func (s *Server) listServiceMeshHosts(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 	namespace := "default"
 	if ns := ctr.GetArguments()["namespace"]; ns != nil {

pkg/mcp/profile_test.go

@@ -135,7 +135,6 @@ func TestToolAnnotations(t *testing.T) {
 			readOnlyTools := []string{
 				"get-virtual-service",
 				"get-destination-rule",
-				"get-service-entries",
 				"get-proxy-clusters",
 				"get-proxy-bootstrap",
 			}
@@ -251,7 +250,6 @@ func TestToolParameters(t *testing.T) {
 			namespaceTools := []string{
 				"get-virtual-service",
 				"get-destination-rule",
-				"get-service-entries",
 			}
 
 			for _, toolName := range namespaceTools {