Skip to content

docs: name the page cursor property as Python spells it - #607

Merged
razor-x merged 1 commit into
betafrom
claude/readme-page-cursor-heading
Aug 13, 2026
Merged

docs: name the page cursor property as Python spells it#607
razor-x merged 1 commit into
betafrom
claude/readme-page-cursor-heading

Conversation

@razor-x

@razor-x razor-x commented Aug 13, 2026

Copy link
Copy Markdown
Member

The pagination heading in the README called the cursor nextPageCursor, which is how the JavaScript SDK spells it. This SDK exposes it as pagination.next_page_cursor (seam/resources/pagination.py), and the example directly under that heading already uses the snake_case name:

if pagination.has_next_page:
    more_devices, _ = paginator.next_page(pagination.next_page_cursor)

So the heading named a property that does not exist in this SDK. This renames it to match, and updates the generated table of contents entry that references it.

Turned up while looking at camelCase identifiers in the SDK. Worth noting the API surface itself is consistent — every method and parameter is snake_case, since codegen converts the blueprint names. This was doc text that escaped that conversion.

uv run rstcheck README.rst passes, and the table of contents was regenerated with node generate-readme-toc.js rather than edited by hand.


Generated by Claude Code

The pagination heading called it nextPageCursor, which is how the JavaScript
SDK spells it. In this SDK it is pagination.next_page_cursor, as the example
under that very heading already shows.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PpqwDhZmyikGbjmCPqFW2A
@razor-x
razor-x requested a review from a team as a code owner August 13, 2026 03:32
@razor-x
razor-x merged commit 35b243b into beta Aug 13, 2026
15 checks passed
@razor-x
razor-x deleted the claude/readme-page-cursor-heading branch August 13, 2026 03:33
@razor-x razor-x mentioned this pull request Aug 13, 2026
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants