fix(kubernetes): remove unneeded CacheInvalidate() method

Author: manusa

pkg/kubernetes/kubernetes.go

@@ -164,16 +164,6 @@ func (m *Manager) Derived(ctx context.Context) *Kubernetes {
 	return derived
 }
 
-// TODO: check test to see why cache isn't getting invalidated automatically https://github.com/manusa/kubernetes-mcp-server/pull/125#discussion_r2152194784
-func (k *Kubernetes) CacheInvalidate() {
-	if k.manager.discoveryClient != nil {
-		k.manager.discoveryClient.Invalidate()
-	}
-	if k.manager.deferredDiscoveryRESTMapper != nil {
-		k.manager.deferredDiscoveryRESTMapper.Reset()
-	}
-}
-
 func (k *Kubernetes) NewHelm() *helm.Helm {
 	// This is a derived Kubernetes, so it already has the Helm initialized
 	return helm.NewHelm(k.manager)

pkg/mcp/pods_top_test.go

@@ -19,7 +19,6 @@ func TestPodsTop(t *testing.T) {
 			// Request Performed by DiscoveryClient to Kube API (Get API Groups legacy -core-)
 			if req.URL.Path == "/api" {
 				if !metricsApiAvailable {
-
 					_, _ = w.Write([]byte(`{"kind":"APIVersions","versions":[],"serverAddressByClientCIDRs":[{"clientCIDR":"0.0.0.0/0"}]}`))
 				} else {
 					_, _ = w.Write([]byte(`{"kind":"APIVersions","versions":["metrics.k8s.io/v1beta1"],"serverAddressByClientCIDRs":[{"clientCIDR":"0.0.0.0/0"}]}`))
@@ -87,7 +86,6 @@ func TestPodsTop(t *testing.T) {
 		})
 		// Enable metrics API addon
 		metricsApiAvailable = true
-		c.mcpServer.k.Derived(t.Context()).CacheInvalidate() // Force discovery client to refresh
 		podsTopDefaults, err := c.callTool("pods_top", map[string]interface{}{})
 		t.Run("pods_top defaults returns pod metrics from all namespaces", func(t *testing.T) {
 			if err != nil {