From c0bb50dc533f27610be58b1e682590fc22498f4c Mon Sep 17 00:00:00 2001 From: Marc Nuri Date: Thu, 12 Jun 2025 15:30:48 +0200 Subject: [PATCH] feat(output)!: default list-output changed from yaml to table --- pkg/kubernetes-mcp-server/cmd/root.go | 2 +- pkg/kubernetes-mcp-server/cmd/root_test.go | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/kubernetes-mcp-server/cmd/root.go b/pkg/kubernetes-mcp-server/cmd/root.go index 14d8cebc..f3dcce6c 100644 --- a/pkg/kubernetes-mcp-server/cmd/root.go +++ b/pkg/kubernetes-mcp-server/cmd/root.go @@ -118,7 +118,7 @@ func flagInit() { rootCmd.Flags().StringP("sse-base-url", "", "", "SSE public base URL to use when sending the endpoint message (e.g. https://example.com)") rootCmd.Flags().StringP("kubeconfig", "", "", "Path to the kubeconfig file to use for authentication") rootCmd.Flags().String("profile", "full", "MCP profile to use (one of: "+strings.Join(mcp.ProfileNames, ", ")+")") - rootCmd.Flags().String("list-output", "yaml", "Output format for resource lists (one of: "+strings.Join(output.Names, ", ")+")") + rootCmd.Flags().String("list-output", "table", "Output format for resource lists (one of: "+strings.Join(output.Names, ", ")+")") rootCmd.Flags().Bool("read-only", false, "If true, only tools annotated with readOnlyHint=true are exposed") rootCmd.Flags().Bool("disable-destructive", false, "If true, tools annotated with destructiveHint=true are disabled") _ = viper.BindPFlags(rootCmd.Flags()) diff --git a/pkg/kubernetes-mcp-server/cmd/root_test.go b/pkg/kubernetes-mcp-server/cmd/root_test.go index b9ff00ca..894d73f4 100644 --- a/pkg/kubernetes-mcp-server/cmd/root_test.go +++ b/pkg/kubernetes-mcp-server/cmd/root_test.go @@ -52,13 +52,13 @@ func TestListOutput(t *testing.T) { t.Fatalf("Expected all available outputs, got %s %v", out, err) } }) - t.Run("default", func(t *testing.T) { + t.Run("defaults to table", func(t *testing.T) { rootCmd.SetArgs([]string{"--version", "--log-level=1"}) rootCmd.ResetFlags() flagInit() out, err := captureOutput(rootCmd.Execute) - if !strings.Contains(out, "- ListOutput: yaml") { - t.Fatalf("Expected list-output 'yaml', got %s %v", out, err) + if !strings.Contains(out, "- ListOutput: table") { + t.Fatalf("Expected list-output 'table', got %s %v", out, err) } }) }