Design: Full API parity for CLI and MCP

cli/calendar_test.go

@@ -0,0 +1,83 @@
+package cli
+
+import (
+	"bytes"
+	"strings"
+	"testing"
+)
+
+func TestCalendarHelp_ShowsSubcommands(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"calendar", "--help"})
+
+	err := cmd.Execute()
+	if err != nil {
+		t.Fatalf("calendar --help should not error: %v", err)
+	}
+
+	output := buf.String()
+	subcommands := []string{"list", "show", "create", "update", "delete", "calendars"}
+	for _, sub := range subcommands {
+		if !strings.Contains(output, sub) {
+			t.Errorf("expected %q subcommand in help, got: %q", sub, output)
+		}
+	}
+}
+
+func TestCalendarCreate_RequiresSummary(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"calendar", "create", "--start", "2026-01-01T10:00", "--end", "2026-01-01T11:00"})
+
+	err := cmd.Execute()
+	if err == nil {
+		t.Fatal("calendar create without --summary should error")
+	}
+}
+
+func TestCalendarShow_RequiresIDArg(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"calendar", "show"})
+
+	err := cmd.Execute()
+	if err == nil {
+		t.Fatal("calendar show without ID should error")
+	}
+}
+
+func TestCalendarDelete_RequiresForce(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"calendar", "delete", "event-123"})
+
+	err := cmd.Execute()
+	if err == nil {
+		t.Fatal("calendar delete without --force should error")
+	}
+	if !strings.Contains(err.Error(), "force") {
+		t.Errorf("expected 'force' in error, got: %v", err)
+	}
+}
+
+func TestCalendarUpdate_RequiresIDArg(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"calendar", "update"})
+
+	err := cmd.Execute()
+	if err == nil {
+		t.Fatal("calendar update without ID should error")
+	}
+}

cli/identity.go

@@ -0,0 +1,69 @@
+package cli
+
+import (
+	"context"
+	"encoding/json"
+	"fmt"
+
+	"github.com/spf13/cobra"
+
+	"github.com/seanb4t/fastmail-cli/pkg/fastmail"
+)
+
+func newIdentityCommand() *cobra.Command {
+	cmd := &cobra.Command{
+		Use:   "identity",
+		Short: "Identity operations",
+		Long:  "Commands for managing sender identities.",
+	}
+
+	cmd.AddCommand(newIdentityListCommand())
+	return cmd
+}
+
+func newIdentityListCommand() *cobra.Command {
+	return &cobra.Command{
+		Use:   "list",
+		Short: "List sender identities",
+		Long:  "List all configured sender identities.",
+		RunE: func(cmd *cobra.Command, _ []string) error {
+			client, err := createClient()
+			if err != nil {
+				return err
+			}
+
+			ctx := context.Background()
+			if err := client.Connect(ctx); err != nil {
+				return fmt.Errorf("connecting: %w", err)
+			}
+
+			identities, err := client.Identity().List(ctx)
+			if err != nil {
+				return fmt.Errorf("listing identities: %w", err)
+			}
+
+			return outputIdentities(cmd, identities)
+		},
+	}
+}
+
+func outputIdentities(cmd *cobra.Command, identities []fastmail.Identity) error {
+	if IsQuiet() {
+		return nil
+	}
+
+	if IsJSONOutput() {
+		enc := json.NewEncoder(cmd.OutOrStdout())
+		enc.SetIndent("", "  ")
+		return enc.Encode(identities)
+	}
+
+	for _, id := range identities {
+		deletable := ""
+		if !id.MayDelete {
+			deletable = " [locked]"
+		}
+		_, _ = fmt.Fprintf(cmd.OutOrStdout(), "%s  %s <%s>%s\n", id.ID, id.Name, id.Email, deletable)
+	}
+	return nil
+}

cli/identity_test.go

@@ -0,0 +1,25 @@
+package cli
+
+import (
+	"bytes"
+	"strings"
+	"testing"
+)
+
+func TestIdentityHelp_ShowsSubcommands(t *testing.T) {
+	cmd := NewRootCommand()
+	buf := new(bytes.Buffer)
+	cmd.SetOut(buf)
+	cmd.SetErr(buf)
+	cmd.SetArgs([]string{"identity", "--help"})
+
+	err := cmd.Execute()
+	if err != nil {
+		t.Fatalf("identity --help should not error: %v", err)
+	}
+
+	output := buf.String()
+	if !strings.Contains(output, "list") {
+		t.Errorf("expected 'list' subcommand in help, got: %q", output)
+	}
+}