Skip to content

Make pagination discoverable on list commands#55

Merged
weppos merged 4 commits into
mainfrom
pagination-hint
May 25, 2026
Merged

Make pagination discoverable on list commands#55
weppos merged 4 commits into
mainfrom
pagination-hint

Conversation

@weppos
Copy link
Copy Markdown
Member

@weppos weppos commented May 25, 2026

Make pagination obvious on every list command, so a person (or an AI agent) can immediately see when results span multiple pages and how to fetch the rest. A customer reported that dnsimple records list silently returned only the first page, with no indication that more records existed.

billing charges is intentionally excluded: the SDK options type for that endpoint does not support paging.

Fixes #51

📋 Deployment Pre/Post tasks

N/A

:shipit: Deployment Verification

  • dnsimple records list <zone> on a zone with more than 30 records prints Showing 30 of N records (page 1 of M) to stderr, above the table.
  • The hint is absent when output is piped/redirected (stdout stays clean) and when a single page covers all results.
  • dnsimple records list <zone> --json includes a pagination object and prints no hint.
  • dnsimple records list <zone> --all fetches every page and prints no hint.
  • --page / --per-page work on every paginated list command (e.g. dnsimple services list --page 2).

@weppos weppos self-assigned this May 25, 2026
@weppos weppos added the enhancement New feature or request label May 25, 2026
@weppos weppos merged commit 24b5ba0 into main May 25, 2026
8 checks passed
@weppos weppos deleted the pagination-hint branch May 25, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pagination is not obvious

1 participant