checking

Signed-off-by: Rohit Patil <[email protected]>

Author: ropatil010

docs/PROMPTS.md

@@ -57,6 +57,43 @@ content = "I'll retrieve and analyze the logs for you."
 ### Argument Substitution
 Use `{{argument_name}}` placeholders in message content. The template engine replaces these with actual values when the prompt is called.
 
+## Built-in Prompts
+
+The Kubernetes MCP Server includes several built-in prompts that are always available:
+
+### `cluster-health-check`
+
+Performs a comprehensive health assessment of your Kubernetes or OpenShift cluster.
+
+**Arguments:**
+- `namespace` (optional): Limit the health check to a specific namespace. Default: all namespaces.
+- `verbose` (optional): Enable detailed resource-level information. Values: `true` or `false`. Default: `false`.
+- `check_events` (optional): Include recent warning/error events in the analysis. Values: `true` or `false`. Default: `true`.
+
+**What it checks:**
+- **Nodes**: Status and conditions (Ready, MemoryPressure, DiskPressure, etc.)
+- **Cluster Operators** (OpenShift only): Available and degraded status
+- **Pods**: Phase, container statuses, restart counts, and common issues (CrashLoopBackOff, ImagePullBackOff, etc.)
+- **Workload Controllers**: Deployments, StatefulSets, and DaemonSets replica status
+- **Persistent Volume Claims**: Binding status
+- **Events**: Recent warning and error events from the last hour
+
+**Example usage:**
+```
+Check the health of my cluster
+```
+
+Or with specific parameters:
+```
+Check the health of namespace production with verbose output
+```
+
+The prompt gathers comprehensive diagnostic data and presents it to the LLM for analysis, which will provide:
+1. Overall health status (Healthy, Warning, or Critical)
+2. Critical issues requiring immediate attention
+3. Warnings and recommendations
+4. Summary by component
+
 ## Configuration File Location
 
 Place your prompts in the `config.toml` file used by the MCP server. Specify the config file path using the `--config` flag when starting the server.

pkg/mcp/mcp.go

@@ -167,8 +167,16 @@ func (s *Server) reloadToolsets() error {
 	// Track previously enabled prompts
 	previousPrompts := s.enabledPrompts
 
+	// Save built-in prompts before clearing (prompts registered via init() in code)
+	// These need to be preserved across config reloads
+	builtInPrompts := prompts.ConfigPrompts()
+
 	// Load config prompts into registry
 	prompts.Clear()
+
+	// Re-register built-in prompts first
+	prompts.Register(builtInPrompts...)
+
 	if s.configuration.HasPrompts() {
 		ctx := context.Background()
 		md := s.configuration.GetPromptsMetadata()
@@ -177,7 +185,7 @@ func (s *Server) reloadToolsets() error {
 		}
 	}
 
-	// Get prompts from registry
+	// Get prompts from registry (includes both built-in and config prompts)
 	configPrompts := prompts.ConfigPrompts()
 
 	// Update enabled prompts list

pkg/mcp/mcp_prompts_test.go

@@ -150,6 +150,93 @@ content = "Content with {{required_arg}}"
 	})
 }
 
+func (s *McpPromptsSuite) TestClusterHealthCheckPrompt() {
+	// No need to load config - cluster-health-check is a built-in prompt
+	s.InitMcpClient()
+
+	s.Run("cluster-health-check prompt is listed", func() {
+		prompts, err := s.ListPrompts(s.T().Context(), mcp.ListPromptsRequest{})
+		s.Require().NoError(err, "ListPrompts should succeed")
+		s.Require().NotNil(prompts)
+
+		var healthCheckPrompt *mcp.Prompt
+		for _, prompt := range prompts.Prompts {
+			if prompt.Name == "cluster-health-check" {
+				healthCheckPrompt = &prompt
+				break
+			}
+		}
+
+		s.Require().NotNil(healthCheckPrompt, "cluster-health-check prompt should be found")
+		s.Equal("cluster-health-check", healthCheckPrompt.Name)
+		s.Contains(healthCheckPrompt.Description, "comprehensive health assessment")
+		s.Require().Len(healthCheckPrompt.Arguments, 3, "should have 3 arguments")
+
+		// Verify arguments
+		s.Equal("namespace", healthCheckPrompt.Arguments[0].Name)
+		s.Equal("verbose", healthCheckPrompt.Arguments[1].Name)
+		s.Equal("check_events", healthCheckPrompt.Arguments[2].Name)
+	})
+
+	s.Run("invoke cluster-health-check with no arguments", func() {
+		result, err := s.GetPrompt(s.T().Context(), mcp.GetPromptRequest{
+			Params: mcp.GetPromptParams{
+				Name:      "cluster-health-check",
+				Arguments: map[string]string{},
+			},
+		})
+
+		s.NoError(err, "GetPrompt should succeed")
+		s.Require().NotNil(result)
+		s.Contains(result.Description, "successfully")
+		s.Require().Len(result.Messages, 2, "should have 2 messages (user + assistant)")
+
+		// Verify message roles
+		s.Equal("user", string(result.Messages[0].Role))
+		s.Equal("assistant", string(result.Messages[1].Role))
+
+		// Verify user message contains diagnostic data
+		userContent, ok := result.Messages[0].Content.(mcp.TextContent)
+		s.Require().True(ok, "expected TextContent")
+		s.Contains(userContent.Text, "Cluster Health Check Diagnostic Data")
+		s.Contains(userContent.Text, "## Your Task")
+		s.Contains(userContent.Text, "## 1. Nodes")
+	})
+
+	s.Run("invoke cluster-health-check with namespace argument", func() {
+		result, err := s.GetPrompt(s.T().Context(), mcp.GetPromptRequest{
+			Params: mcp.GetPromptParams{
+				Name: "cluster-health-check",
+				Arguments: map[string]string{
+					"namespace": "default",
+				},
+			},
+		})
+
+		s.NoError(err, "GetPrompt should succeed with namespace argument")
+		s.Require().NotNil(result)
+
+		// Verify message contains namespace scope
+		userContent, ok := result.Messages[0].Content.(mcp.TextContent)
+		s.Require().True(ok, "expected TextContent")
+		s.Contains(userContent.Text, "Namespace `default`")
+	})
+
+	s.Run("invoke cluster-health-check with verbose flag", func() {
+		result, err := s.GetPrompt(s.T().Context(), mcp.GetPromptRequest{
+			Params: mcp.GetPromptParams{
+				Name: "cluster-health-check",
+				Arguments: map[string]string{
+					"verbose": "true",
+				},
+			},
+		})
+
+		s.NoError(err, "GetPrompt should succeed with verbose flag")
+		s.Require().NotNil(result)
+	})
+}
+
 func TestMcpPromptsSuite(t *testing.T) {
 	suite.Run(t, new(McpPromptsSuite))
 }