cli: make list output readable

Author: changkun

internal/api/client.go

@@ -156,7 +156,7 @@ func (e *APIError) Error() string {
 	if e.Code == "policy_sidecar_required" {
 		return "cannot create cella: the selected policy requires Cella's credential sidecar, but the server has no complete sidecar configuration for this CLI token.\n" +
 			"This is not a local command syntax problem. Re-run `latere auth login` with the latest CLI, then retry.\n" +
-			"To choose another policy, run `latere cella policy list` and retry with `latere cella create --policy <name>` using a selectable policy where SIDECAR is `no`.\n" +
+			"To choose another policy, run `latere cella policy list` and retry with `latere cella create --policy <name>` using a selectable policy where sidecar is `no`.\n" +
 			"If no such policy is available, ask your Latere admin/support to configure the CLI sidecar client or assign a non-sidecar policy.\n" +
 			"server code: policy_sidecar_required"
 	}

internal/api/client_test.go

@@ -19,7 +19,7 @@ func TestAPIErrorPolicySidecarRequiredIsActionable(t *testing.T) {
 		"latere auth login",
 		"latere cella policy list",
 		"latere cella create --policy <name>",
-		"SIDECAR is `no`",
+		"sidecar is `no`",
 		"server code: policy_sidecar_required",
 	} {
 		if !strings.Contains(err, want) {

internal/commands/cella.go

@@ -16,7 +16,6 @@ import (
 	"path/filepath"
 	"strconv"
 	"strings"
-	"text/tabwriter"
 	"time"
 
 	"github.com/spf13/cobra"
@@ -303,14 +302,12 @@ cellas returned by the backend, including warm-pool cellas.`,
 			if jsonF {
 				return printJSON(sbs)
 			}
-			tw := tabwriter.NewWriter(os.Stdout, 0, 2, 2, ' ', 0)
-			_, _ = fmt.Fprintln(tw, "NAME\tID\tSTATE\tTIER\tDISK\tCREATED")
-			for _, s := range sbs {
-				_, _ = fmt.Fprintf(tw, "%s\t%s\t%s\t%s\t%dGi\t%s\n",
-					nameOrDash(s.Name), s.ID, s.State, defaultStr(s.Tier, "-"),
-					s.DiskGB, humanAge(s.CreatedAt))
+			if len(sbs) == 0 {
+				fmt.Fprintln(os.Stdout, "No cellas are visible to this token.")
+				return nil
 			}
-			return tw.Flush()
+			printSandboxList(sbs)
+			return nil
 		},
 	}
 	cmd.Flags().StringVar(&apiURL, "api-url", "", "override Cella API base URL")
@@ -338,7 +335,7 @@ Use a selectable policy with:
   latere cella create --policy <name>
 
 If create fails because the selected policy requires the sidecar, list
-policies and choose a selectable policy where SIDECAR is "no", or ask
+policies and choose a selectable policy where sidecar is "no", or ask
 an admin to configure the sidecar client for your token.`,
 		Example: `  latere cella policy
   latere cella policy list
@@ -391,20 +388,19 @@ func runPolicyList(ctx context.Context, apiURL string, jsonF bool) error {
 }
 
 func printPolicies(policies []policyDTO) {
-	tw := tabwriter.NewWriter(os.Stdout, 0, 2, 2, ' ', 0)
-	_, _ = fmt.Fprintln(tw, "NAME\tDEFAULT\tSELECTABLE\tSIDECAR\tCAPABILITY\tSOURCE\tDESCRIPTION")
-	for _, p := range policies {
-		_, _ = fmt.Fprintf(tw, "%s\t%s\t%s\t%s\t%s\t%s\t%s\n",
-			p.Name,
-			yesNo(p.IsDefault),
-			yesNo(p.Selectable),
-			yesNo(p.SidecarRequired),
-			defaultStr(p.CapabilityProfile, "-"),
-			defaultStr(p.AssignmentSource, "-"),
-			oneLine(defaultStr(p.Description, p.Label)),
-		)
-	}
-	_ = tw.Flush()
+	for i, p := range policies {
+		if i > 0 {
+			fmt.Fprintln(os.Stdout)
+		}
+		printWrappedField("policy", p.Name)
+		printWrappedField("label", p.Label)
+		printWrappedField("default", yesNo(p.IsDefault))
+		printWrappedField("selectable", yesNo(p.Selectable))
+		printWrappedField("sidecar", yesNo(p.SidecarRequired))
+		printWrappedField("capability", defaultStr(p.CapabilityProfile, "-"))
+		printWrappedField("source", defaultStr(p.AssignmentSource, "-"))
+		printWrappedField("description", defaultStr(p.Description, "-"))
+	}
 }
 
 func newCeGetCmd() *cobra.Command {
@@ -1515,12 +1511,35 @@ func printJSON(v any) error {
 	return enc.Encode(v)
 }
 
+func printSandboxList(sbs []sandboxDTO) {
+	for i, s := range sbs {
+		if i > 0 {
+			fmt.Fprintln(os.Stdout)
+		}
+		printSandbox(s)
+	}
+}
+
 func printSandbox(s sandboxDTO) {
+	printWrappedField("cella", nameOrDash(s.Name))
+	printWrappedField("id", s.ID)
+	printWrappedField("state", s.State)
+	printWrappedField("tier", defaultStr(s.Tier, "-"))
+	if s.DiskGB > 0 {
+		printWrappedField("disk", fmt.Sprintf("%dGi", s.DiskGB))
+	}
 	if size := sandboxResourceSummary(s); size != "" {
-		fmt.Printf("%s  %s  %s  %s\n", nameOrDash(s.Name), s.ID, s.State, size)
-		return
+		printWrappedField("resources", size)
+	}
+	if !s.CreatedAt.IsZero() {
+		printWrappedField("created", humanAge(s.CreatedAt)+" ago")
+	}
+	if !s.Deadline.IsZero() {
+		printWrappedField("deadline", s.Deadline.Format(time.RFC3339))
+	}
+	if s.Workdir != "" {
+		printWrappedField("workdir", s.Workdir)
 	}
-	fmt.Printf("%s  %s  %s\n", nameOrDash(s.Name), s.ID, s.State)
 }
 
 // sandboxResourceSummary renders the cpu_milli / memory_mb fields
@@ -1560,6 +1579,50 @@ func yesNo(v bool) string {
 	return "no"
 }
 
+func printWrappedField(label, value string) {
+	value = oneLine(value)
+	if value == "" {
+		return
+	}
+	const (
+		labelWidth = 12
+		maxWidth   = 88
+	)
+	prefix := fmt.Sprintf("%-*s", labelWidth, label+":")
+	lines := wrapText(value, maxWidth-labelWidth)
+	if len(lines) == 0 {
+		fmt.Fprintln(os.Stdout, prefix)
+		return
+	}
+	fmt.Fprintln(os.Stdout, prefix+lines[0])
+	indent := strings.Repeat(" ", labelWidth)
+	for _, line := range lines[1:] {
+		fmt.Fprintln(os.Stdout, indent+line)
+	}
+}
+
+func wrapText(s string, width int) []string {
+	words := strings.Fields(s)
+	if len(words) == 0 {
+		return nil
+	}
+	if width <= 0 {
+		width = 76
+	}
+	var lines []string
+	line := words[0]
+	for _, word := range words[1:] {
+		if len(line)+1+len(word) > width {
+			lines = append(lines, line)
+			line = word
+			continue
+		}
+		line += " " + word
+	}
+	lines = append(lines, line)
+	return lines
+}
+
 func oneLine(s string) string {
 	return strings.Join(strings.Fields(s), " ")
 }

internal/commands/policy_test.go

@@ -57,20 +57,22 @@ func TestPolicyListPrintsCreateGuidanceFields(t *testing.T) {
 		t.Fatalf("Authorization = %q, want bearer token", authz)
 	}
 	for _, want := range []string{
-		"NAME",
-		"DEFAULT",
-		"SELECTABLE",
-		"SIDECAR",
+		"policy:     agent-default",
+		"default:    yes",
+		"selectable: yes",
+		"sidecar:    yes",
 		"agent-default",
-		"yes",
 		"restricted-network",
 		"restricted-no-network",
-		"client",
+		"source:     client",
 	} {
 		if !strings.Contains(out, want) {
 			t.Fatalf("policy list output missing %q:\n%s", want, out)
 		}
 	}
+	if strings.Contains(out, "NAME") || strings.Contains(out, "\t") {
+		t.Fatalf("policy list should be pure text, not a wide table:\n%s", out)
+	}
 }
 
 func TestPolicyListEmptyExplainsNextStep(t *testing.T) {
@@ -96,6 +98,45 @@ func TestPolicyListEmptyExplainsNextStep(t *testing.T) {
 	}
 }
 
+func TestSandboxListPrintsReadableRecords(t *testing.T) {
+	out := capturePolicyStdout(t, func() {
+		printSandboxList([]sandboxDTO{
+			{
+				ID:       "sb-019dc976-2b28-7c55-8778-bf7d5ae6c58d",
+				Name:     "workspace-1",
+				State:    "stopped",
+				Tier:     "persistent",
+				DiskGB:   5,
+				CPUMilli: 1000,
+				MemoryMB: 2048,
+			},
+			{
+				ID:     "sb-019dc976-2b28-7c55-8778-warm-pool",
+				Name:   "warm-pool",
+				State:  "running",
+				Tier:   "ephemeral",
+				DiskGB: 5,
+			},
+		})
+	})
+
+	for _, want := range []string{
+		"cella:      workspace-1",
+		"id:         sb-019dc976-2b28-7c55-8778-bf7d5ae6c58d",
+		"state:      stopped",
+		"tier:       persistent",
+		"resources:  cpu=1000m memory=2048Mi",
+		"\n\ncella:      warm-pool",
+	} {
+		if !strings.Contains(out, want) {
+			t.Fatalf("sandbox list output missing %q:\n%s", want, out)
+		}
+	}
+	if strings.Contains(out, "NAME") || strings.Contains(out, "\t") {
+		t.Fatalf("sandbox list should be pure text, not a wide table:\n%s", out)
+	}
+}
+
 func writeTestToken(t *testing.T) {
 	t.Helper()
 	path := filepath.Join(t.TempDir(), "token.json")

internal/commands/root_test.go

@@ -91,7 +91,7 @@ func TestHelpIncludesUserExamples(t *testing.T) {
 			want: []string{
 				"List Cella policy profiles visible to the current token.",
 				"latere cella create --policy <name>",
-				"choose a selectable policy where SIDECAR is \"no\"",
+				"choose a selectable policy where sidecar is \"no\"",
 			},
 		},
 		{