fix: remove analyze tool

Author: krutsko

PROXY_CONFIG.md

@@ -17,11 +17,11 @@ The Istio MCP Server supports these proxy configuration tools:
 - **get-proxy-bootstrap**: Get Envoy bootstrap configuration from a pod
 - **get-proxy-config-dump**: Get full Envoy configuration dump from a pod
 - **get-proxy-status**: Get proxy status information for all pods or a specific pod
-- **get-istio-analyze**: Analyze Istio configuration and report potential issues
+
 
 ## Implementation Details
 
-- Uses `istioctl proxy-config` commands under the hood (and `istioctl analyze` for configuration analysis)
+- Uses `istioctl proxy-config` commands under the hood
 - Requires `istioctl` to be installed on the system
 - Returns JSON formatted output for easy parsing
 - Includes proper error handling and timeouts
@@ -31,7 +31,7 @@ The Istio MCP Server supports these proxy configuration tools:
 
 Each tool requires:
 - `namespace` (optional, defaults to 'default')
-- `pod` (required for most tools, except `get-proxy-status` and `get-istio-analyze`)
+- `pod` (required for most tools, except `get-proxy-status`)
 
 ### Examples
 
@@ -51,11 +51,7 @@ get-proxy-status --namespace default
 # Get proxy status for a specific pod
 get-proxy-status --namespace default --pod my-app-pod
 
-# Analyze Istio configuration for a specific namespace
-get-istio-analyze --namespace default
 
-# Analyze Istio configuration for the entire cluster
-get-istio-analyze
 ```
 
 ## Prerequisites

go.mod

@@ -9,6 +9,7 @@ require (
 	github.com/spf13/viper v1.20.1
 	golang.org/x/net v0.41.0
 	istio.io/client-go v1.25.1
+	k8s.io/api v0.33.1
 	k8s.io/apimachinery v0.33.1
 	k8s.io/client-go v0.33.1
 	k8s.io/klog/v2 v2.130.1
@@ -59,7 +60,6 @@ require (
 	gopkg.in/inf.v0 v0.9.1 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	istio.io/api v1.25.1 // indirect
-	k8s.io/api v0.33.1 // indirect
 	k8s.io/kube-openapi v0.0.0-20250318190949-c8a335a9a2ff // indirect
 	k8s.io/utils v0.0.0-20241104100929-3ea5e8cea738 // indirect
 	sigs.k8s.io/json v0.0.0-20241010143419-9aa6b5e7a4b3 // indirect

pkg/istio/proxy_config.go

@@ -68,14 +68,6 @@ func (p *ProxyConfigClient) GetProxyStatusForPod(ctx context.Context, namespace,
 	return p.execIstioctl(ctx, "proxy-status", fmt.Sprintf("%s.%s", podName, namespace))
 }
 
-// GetAnalyze performs Istio configuration analysis and reports potential issues
-func (p *ProxyConfigClient) GetAnalyze(ctx context.Context, namespace string) (string, error) {
-	if namespace != "" {
-		return p.execIstioctl(ctx, "analyze", "-n", namespace)
-	}
-	return p.execIstioctl(ctx, "analyze")
-}
-
 // execIstioctl executes istioctl commands with proper error handling and timeout
 func (p *ProxyConfigClient) execIstioctl(ctx context.Context, args ...string) (string, error) {
 	// Create context with timeout

pkg/mcp/profile.go

@@ -354,18 +354,6 @@ func (s *Server) initProxyConfigTools() []server.ServerTool {
 			),
 			Handler: s.getProxyStatus,
 		},
-		{
-			Tool: mcp.NewTool("get-istio-analyze",
-				mcp.WithDescription("Analyze Istio configuration and report potential issues, misconfigurations, and best practice violations. This tool runs 'istioctl analyze' to provide comprehensive analysis of your Istio service mesh configuration."),
-				mcp.WithString("namespace",
-					mcp.Description("Namespace to analyze (optional). If specified, analyzes only the specified namespace. If not provided, analyzes the entire cluster."),
-				),
-				mcp.WithTitleAnnotation("Istio: Configuration Analysis"),
-				mcp.WithReadOnlyHintAnnotation(true),
-				mcp.WithDestructiveHintAnnotation(false),
-			),
-			Handler: s.getIstioAnalyze,
-		},
 	}
 }
 
@@ -607,17 +595,6 @@ func (s *Server) getProxyStatus(ctx context.Context, ctr mcp.CallToolRequest) (*
 	return NewTextResult(content, err), nil
 }
 
-// getIstioAnalyze performs Istio configuration analysis
-func (s *Server) getIstioAnalyze(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {
-	namespace := ""
-	if ns := ctr.GetArguments()["namespace"]; ns != nil {
-		namespace = ns.(string)
-	}
-
-	content, err := s.i.ProxyConfig.GetAnalyze(ctx, namespace)
-	return NewTextResult(content, err), nil
-}
-
 // Handler implementations (add to profile.go)
 func (s *Server) getServices(ctx context.Context, ctr mcp.CallToolRequest) (*mcp.CallToolResult, error) {
 	namespace := "default"