Address remaining PR review comments

Author: robzolkos

internal/commands/activity.go

@@ -24,6 +24,7 @@ var activityListCmd = &cobra.Command{
 	Use:   "list",
 	Short: "List activities",
 	Long:  "Lists activities with optional board and creator filters.",
+	Args:  cobra.NoArgs,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := requireAuthAndAccount(); err != nil {
 			return err

internal/commands/activity_test.go

@@ -160,6 +160,12 @@ func TestActivityList(t *testing.T) {
 		}
 	})
 
+	t.Run("rejects positional args", func(t *testing.T) {
+		if err := activityListCmd.Args(activityListCmd, []string{"unexpected"}); err == nil {
+			t.Fatal("expected positional args to be rejected")
+		}
+	})
+
 	t.Run("requires authentication", func(t *testing.T) {
 		mock := NewMockClient()
 		SetTestModeWithSDK(mock)

internal/commands/board.go

@@ -423,6 +423,7 @@ var boardAccessesCmd = &cobra.Command{
 	Use:   "accesses",
 	Short: "Show board accesses",
 	Long:  "Shows access settings and users for a board.",
+	Args:  cobra.NoArgs,
 	RunE: func(cmd *cobra.Command, args []string) error {
 		if err := requireAuthAndAccount(); err != nil {
 			return err

internal/commands/board_test.go

@@ -761,6 +761,12 @@ func TestBoardAccesses(t *testing.T) {
 		}
 	})
 
+	t.Run("rejects positional args", func(t *testing.T) {
+		if err := boardAccessesCmd.Args(boardAccessesCmd, []string{"unexpected"}); err == nil {
+			t.Fatal("expected positional args to be rejected")
+		}
+	})
+
 	t.Run("requires board", func(t *testing.T) {
 		mock := NewMockClient()
 		SetTestModeWithSDK(mock)

internal/commands/card.go

@@ -76,10 +76,7 @@ var cardListCmd = &cobra.Command{
 					}
 					effectiveIndexedBy = "maybe"
 				default:
-					if effectiveIndexedBy != "" {
-						return errors.NewInvalidArgsError("cannot combine --indexed-by with --column")
-					}
-					params = append(params, "column_ids[]="+columnFilter)
+					return errors.NewInvalidArgsError("invalid pseudo-column kind: " + pseudo.Kind)
 				}
 			} else {
 				if effectiveIndexedBy != "" {

internal/commands/pseudocolumns.go

@@ -11,7 +11,8 @@ type pseudoColumn struct {
 var (
 	// "Not Now" contains postponed cards (indexed_by=not_now)
 	pseudoColumnNotNow = pseudoColumn{ID: "not-now", Name: "Not Now", Kind: "not_now"}
-	// "Maybe?" contains triage/backlog cards (indexed_by=maybe)
+	// "Maybe?" contains triage/backlog cards. Kind remains "triage" for triage endpoints/aliases;
+	// card listing maps this pseudo-column to indexed_by=maybe server-side.
 	pseudoColumnMaybe = pseudoColumn{ID: "maybe", Name: "Maybe?", Kind: "triage"}
 	// "Done" contains closed cards (indexed_by=closed)
 	pseudoColumnDone = pseudoColumn{ID: "done", Name: "Done", Kind: "closed"}

skills/fizzy/SKILL.md

@@ -109,7 +109,7 @@ Want to change something?
 | step | `step list --card NUMBER` | `step show ID --card NUMBER` | `step create` | `step update ID` | `step delete ID` | - |
 | reaction | `reaction list` | - | `reaction create` | - | `reaction delete ID` | - |
 | tag | `tag list` | - | - | - | - | - |
-| user | `user list` | `user show ID`, `user export-show USER_ID EXPORT_ID` | `user export-create USER_ID`, `user email-change-request USER_ID --email user@example.com`, `user email-change-confirm USER_ID TOKEN` | `user update ID` | - | `user deactivate ID`, `user role ID`, `user avatar-remove ID`, `user push-subscription-create`, `user push-subscription-delete ID` |
+| user | `user list` | `user show ID` | - | `user update ID` | - | `user deactivate ID`, `user role ID`, `user avatar-remove ID`, `user export-create USER_ID`, `user export-show USER_ID EXPORT_ID`, `user email-change-request USER_ID --email user@example.com`, `user email-change-confirm USER_ID TOKEN`, `user push-subscription-create`, `user push-subscription-delete ID` |
 | notification | `notification list` | - | - | - | - | `notification tray`, `notification read-all`, `notification settings-show`, `notification settings-update` |
 | pin | `pin list` | - | - | - | - | `card pin NUMBER`, `card unpin NUMBER` |
 | webhook | `webhook list --board ID`, `webhook deliveries --board ID WEBHOOK_ID` | `webhook show ID --board ID` | `webhook create` | `webhook update ID` | `webhook delete ID` | `webhook reactivate ID` |