Fix token command validation and help

Author: robzolkos

internal/commands/help.go

@@ -379,7 +379,7 @@ var rootCommandGroupTitles = map[string]string{
 }
 
 var rootCommandGroups = map[string][]string{
-	"core":            {"auth", "activity", "board", "card", "search"},
+	"core":            {"auth", "token", "activity", "board", "card", "search"},
 	"collaboration":   {"comment", "notification"},
 	"getting-started": {"setup", "signup"},
 	"discover":        {"doctor", "config", "commands", "version"},

internal/commands/help_test.go

@@ -15,7 +15,7 @@ func TestRenderRootHelp(t *testing.T) {
 	renderHelp(rootCmd, &buf)
 	out := buf.String()
 
-	for _, want := range []string{"CORE COMMANDS", "activity", "GETTING STARTED", "DISCOVER", "FLAGS", "--profile", "LEARN MORE", "Use `fizzy commands` to see the full command catalog.", "implies --json"} {
+	for _, want := range []string{"CORE COMMANDS", "activity", "token", "GETTING STARTED", "DISCOVER", "FLAGS", "--profile", "LEARN MORE", "Use `fizzy commands` to see the full command catalog.", "implies --json"} {
 		if !strings.Contains(out, want) {
 			t.Fatalf("expected root help to contain %q, got:\n%s", want, out)
 		}

internal/commands/token.go

@@ -17,6 +17,7 @@ var tokenListCmd = &cobra.Command{
 	Use:   "list",
 	Short: "List personal access tokens",
 	Long:  "Lists your personal access tokens.",
+	Args:  cobra.NoArgs,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := requireAuth(); err != nil {
 			return err
@@ -55,6 +56,7 @@ var tokenCreateCmd = &cobra.Command{
 	Use:   "create",
 	Short: "Create a personal access token",
 	Long:  "Creates a new personal access token. The token value is shown once at creation and cannot be retrieved later.",
+	Args:  cobra.NoArgs,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := requireAuth(); err != nil {
 			return err

internal/commands/token_test.go

@@ -0,0 +1,21 @@
+package commands
+
+import "testing"
+
+func TestTokenListRejectsUnexpectedArgs(t *testing.T) {
+	if err := tokenListCmd.Args(tokenListCmd, []string{"extra"}); err == nil {
+		t.Fatal("expected token list to reject unexpected positional args")
+	}
+	if err := tokenListCmd.Args(tokenListCmd, []string{}); err != nil {
+		t.Fatalf("expected token list to allow no positional args, got %v", err)
+	}
+}
+
+func TestTokenCreateRejectsUnexpectedArgs(t *testing.T) {
+	if err := tokenCreateCmd.Args(tokenCreateCmd, []string{"extra"}); err == nil {
+		t.Fatal("expected token create to reject unexpected positional args")
+	}
+	if err := tokenCreateCmd.Args(tokenCreateCmd, []string{}); err != nil {
+		t.Fatalf("expected token create to allow no positional args, got %v", err)
+	}
+}