Fix activity discovery and board accesses pagination

Author: robzolkos

internal/commands/board.go

@@ -439,10 +439,11 @@ var boardAccessesCmd = &cobra.Command{
 			page = &pageVal
 		}
 
-		data, _, err := getSDK().Boards().ListBoardAccesses(cmd.Context(), boardID, page)
+		data, resp, err := getSDK().Boards().ListBoardAccesses(cmd.Context(), boardID, page)
 		if err != nil {
 			return convertSDKError(err)
 		}
+		linkNext := parseSDKLinkNext(resp)
 
 		summary := "Board accesses"
 		if boardAccessesPage > 0 {
@@ -454,7 +455,16 @@ var boardAccessesCmd = &cobra.Command{
 			breadcrumb("cards", fmt.Sprintf("fizzy card list --board %s", boardID), "List cards"),
 		}
 
-		printDetail(normalizeAny(data), summary, breadcrumbs)
+		hasNext := linkNext != ""
+		if hasNext {
+			nextPage := boardAccessesPage + 1
+			if boardAccessesPage == 0 {
+				nextPage = 2
+			}
+			breadcrumbs = append(breadcrumbs, breadcrumb("next", fmt.Sprintf("fizzy board accesses --board %s --page %d", boardID, nextPage), "Next page"))
+		}
+
+		printDetailPaginated(normalizeAny(data), summary, breadcrumbs, hasNext, linkNext)
 		return nil
 	},
 }

internal/commands/board_test.go

@@ -722,6 +722,45 @@ func TestBoardAccesses(t *testing.T) {
 		}
 	})
 
+	t.Run("includes next pagination context and breadcrumb", func(t *testing.T) {
+		mock := NewMockClient()
+		mock.GetResponse = &client.APIResponse{
+			StatusCode: 200,
+			Data:       map[string]any{"board_id": "123", "all_access": false, "users": []any{}},
+			LinkNext:   "/boards/123/accesses.json?page=2",
+		}
+
+		result := SetTestModeWithSDK(mock)
+		SetTestConfig("token", "account", "https://api.example.com")
+		defer resetTest()
+
+		boardAccessesBoard = "123"
+		err := boardAccessesCmd.RunE(boardAccessesCmd, []string{})
+		boardAccessesBoard = ""
+		boardAccessesPage = 0
+
+		assertExitCode(t, err, 0)
+
+		var nextCmd string
+		for _, bc := range result.Response.Breadcrumbs {
+			if bc.Action == "next" {
+				nextCmd = bc.Cmd
+				break
+			}
+		}
+		if nextCmd != "fizzy board accesses --board 123 --page 2" {
+			t.Fatalf("expected next breadcrumb, got %q", nextCmd)
+		}
+
+		pagination, ok := result.Response.Context["pagination"].(map[string]any)
+		if !ok {
+			t.Fatalf("expected pagination context, got %#v", result.Response.Context)
+		}
+		if pagination["has_next"] != true || pagination["next_url"] != "/boards/123/accesses.json?page=2" {
+			t.Fatalf("unexpected pagination context: %#v", pagination)
+		}
+	})
+
 	t.Run("requires board", func(t *testing.T) {
 		mock := NewMockClient()
 		SetTestModeWithSDK(mock)

internal/commands/commands.go

@@ -31,7 +31,7 @@ var commandCatalogTitles = map[string]string{
 }
 
 var commandCatalogGroups = map[string][]string{
-	"core":          {"board", "card", "column", "comment", "search", "step"},
+	"core":          {"activity", "board", "card", "column", "comment", "search", "step"},
 	"collaboration": {"notification", "pin", "reaction", "tag", "user"},
 	"admin":         {"auth", "account", "identity", "webhook", "upload", "migrate"},
 	"utilities":     {"setup", "signup", "completion", "doctor", "config", "skill", "commands", "version"},

internal/commands/commands_test.go

@@ -21,7 +21,7 @@ func TestCommandsStyledOutputRendersHumanCatalog(t *testing.T) {
 	if !strings.Contains(raw, "CORE COMMANDS") {
 		t.Fatalf("expected styled catalog heading, got:\n%s", raw)
 	}
-	if !strings.Contains(raw, "auth") || !strings.Contains(raw, "board") {
+	if !strings.Contains(raw, "auth") || !strings.Contains(raw, "activity") || !strings.Contains(raw, "board") {
 		t.Fatalf("expected styled catalog to include commands, got:\n%s", raw)
 	}
 	if strings.Contains(raw, "list, show") {
@@ -51,6 +51,25 @@ func TestCommandsFilterRendersMatchingHumanCatalog(t *testing.T) {
 	}
 }
 
+func TestCommandsFilterFindsActivity(t *testing.T) {
+	mock := NewMockClient()
+	SetTestModeWithSDK(mock)
+	SetTestFormat(output.FormatStyled)
+	defer resetTest()
+
+	if err := commandsCmd.RunE(commandsCmd, []string{"activity"}); err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+
+	raw := TestOutput()
+	if !strings.Contains(raw, "activity") || !strings.Contains(raw, "list") {
+		t.Fatalf("expected filtered catalog to include activity list, got:\n%s", raw)
+	}
+	if strings.Contains(raw, "No commands match") {
+		t.Fatalf("expected activity to be discoverable, got:\n%s", raw)
+	}
+}
+
 func TestCommandsJSONOutputReturnsStructuredCatalog(t *testing.T) {
 	mock := NewMockClient()
 	result := SetTestModeWithSDK(mock)

internal/commands/help.go

@@ -379,7 +379,7 @@ var rootCommandGroupTitles = map[string]string{
 }
 
 var rootCommandGroups = map[string][]string{
-	"core":            {"auth", "board", "card", "search"},
+	"core":            {"auth", "activity", "board", "card", "search"},
 	"collaboration":   {"comment", "notification"},
 	"getting-started": {"setup", "signup"},
 	"discover":        {"doctor", "config", "commands", "version"},
@@ -389,6 +389,8 @@ var commandExamples = map[string]string{
 	"fizzy auth":              "$ fizzy auth status\n$ fizzy auth login TOKEN --profile acme",
 	"fizzy auth status":       "$ fizzy auth status",
 	"fizzy auth list":         "$ fizzy auth list\n$ fizzy auth switch acme",
+	"fizzy activity":          "$ fizzy activity list\n$ fizzy activity list --board <id>",
+	"fizzy activity list":     "$ fizzy activity list --board <id>\n$ fizzy activity list --creator <user_id>",
 	"fizzy board":             "$ fizzy board list\n$ fizzy board show <id>",
 	"fizzy board list":        "$ fizzy board list\n$ fizzy board list --page 2",
 	"fizzy board show":        "$ fizzy board show <id>",

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", "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", "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/root.go

@@ -808,6 +808,11 @@ func printListPaginated(data any, cols render.Columns, hasNext bool, nextURL str
 
 // printDetail renders a single object with format-aware dispatch.
 func printDetail(data any, summary string, breadcrumbs []Breadcrumb) {
+	printDetailPaginated(data, summary, breadcrumbs, false, "")
+}
+
+// printDetailPaginated renders a single object and includes pagination context when present.
+func printDetailPaginated(data any, summary string, breadcrumbs []Breadcrumb, hasNext bool, nextURL string) {
 	switch out.EffectiveFormat() {
 	case output.FormatStyled:
 		body := render.StyledDetail(toMap(data), summary)
@@ -818,7 +823,18 @@ func printDetail(data any, summary string, breadcrumbs []Breadcrumb) {
 		writeOutputString(appendHumanSections(body, "", "", breadcrumbs, true))
 		captureResponse()
 	default:
-		printSuccessWithBreadcrumbs(data, summary, breadcrumbs)
+		opts := []output.ResponseOption{output.WithBreadcrumbs(breadcrumbs...)}
+		if summary != "" {
+			opts = append(opts, output.WithSummary(summary))
+		}
+		if hasNext || nextURL != "" {
+			opts = append(opts, output.WithContext("pagination", map[string]any{
+				"has_next": hasNext,
+				"next_url": nextURL,
+			}))
+		}
+		recordOutputError(out.OK(data, opts...))
+		captureResponse()
 	}
 }