fix: bring back PrintText so that parallel work doesn't fail

Author: Davidonium

internal/cmd/base/list.go

@@ -12,15 +12,17 @@ import (
 // ListCmd defines a command for fetching and displaying a list response.
 // T is the full response proto message (e.g. *clusterv1.ListClustersResponse).
 //
-// OutputTable must be set. The base automatically registers --no-headers and
-// handles header suppression.
+// At least one of OutputTable or PrintText must be set. OutputTable is
+// preferred; when set, --no-headers is automatically registered and handled.
+// PrintText is the legacy fallback for commands that have not yet migrated.
 type ListCmd[T any] struct {
 	Use               string
 	Short             string
 	Long              string
 	Example           string
 	Fetch             func(s *state.State, cmd *cobra.Command) (T, error)
 	OutputTable       func(cmd *cobra.Command, out io.Writer, resp T) (output.TableRenderer, error)
+	PrintText         func(cmd *cobra.Command, out io.Writer, resp T) error
 	ValidArgsFunction func(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective)
 }
 
@@ -40,20 +42,25 @@ func (lc ListCmd[T]) CobraCommand(s *state.State) *cobra.Command {
 			if s.Config.JSONOutput() {
 				return output.PrintJSON(cmd.OutOrStdout(), resp)
 			}
-			if lc.OutputTable == nil {
-				panic("ListCmd: OutputTable must be set")
+			if lc.OutputTable != nil {
+				r, err := lc.OutputTable(cmd, cmd.OutOrStdout(), resp)
+				if err != nil {
+					return err
+				}
+				noHeaders, _ := cmd.Flags().GetBool("no-headers")
+				r.SetNoHeaders(noHeaders)
+				r.Render()
+				return nil
 			}
-			r, err := lc.OutputTable(cmd, cmd.OutOrStdout(), resp)
-			if err != nil {
-				return err
+			if lc.PrintText != nil {
+				return lc.PrintText(cmd, cmd.OutOrStdout(), resp)
 			}
-			noHeaders, _ := cmd.Flags().GetBool("no-headers")
-			r.SetNoHeaders(noHeaders)
-			r.Render()
-			return nil
+			panic("ListCmd: OutputTable or PrintText must be set")
 		},
 	}
-	cmd.Flags().Bool("no-headers", false, "Do not print column headers")
+	if lc.OutputTable != nil {
+		cmd.Flags().Bool("no-headers", false, "Do not print column headers")
+	}
 	if lc.ValidArgsFunction != nil {
 		cmd.ValidArgsFunction = lc.ValidArgsFunction
 	}

internal/cmd/base/list_test.go

@@ -2,6 +2,7 @@ package base_test
 
 import (
 	"bytes"
+	"fmt"
 	"io"
 	"testing"
 
@@ -63,7 +64,22 @@ func TestListCmd_OutputTable_NoHeaders(t *testing.T) {
 	assert.Contains(t, stdout, "hello")
 }
 
-func TestListCmd_NoOutputTable_Panics(t *testing.T) {
+func TestListCmd_PrintText(t *testing.T) {
+	lc := base.ListCmd[string]{
+		Use:   "test",
+		Fetch: fetchHello,
+		PrintText: func(_ *cobra.Command, out io.Writer, resp string) error {
+			_, err := fmt.Fprint(out, resp)
+			return err
+		},
+	}
+
+	stdout, err := execListCmd(t, lc)
+	require.NoError(t, err)
+	assert.Equal(t, "hello", stdout)
+}
+
+func TestListCmd_NeitherOutputTableNorPrintText_Panics(t *testing.T) {
 	lc := base.ListCmd[string]{
 		Use:   "test",
 		Fetch: fetchHello,