Skip to content

Add cursor pagination for list timelines#56

Open
RuochenLyu wants to merge 1 commit intopublic-clis:mainfrom
RuochenLyu:codex/list-cursor-pagination
Open

Add cursor pagination for list timelines#56
RuochenLyu wants to merge 1 commit intopublic-clis:mainfrom
RuochenLyu:codex/list-cursor-pagination

Conversation

@RuochenLyu
Copy link
Copy Markdown

Summary

  • add --cursor support to twitter list
  • expose pagination.nextCursor for structured list timeline output
  • thread cursor/return-cursor support through fetch_list_timeline
  • document the list pagination workflow in README and SKILL.md

This follows the same shape as #49 for feed cursor pagination, reusing the existing timeline cursor extraction and structured pagination envelope.

Validation

  • uv run pytest -q
  • uv run mypy twitter_cli
  • uv run ruff check .
  • real list pagination smoke test with list 2049103846925029525: page 1 returned 5 tweets and a cursor; page 2 with that cursor returned 5 different tweets and a new cursor.

@RuochenLyu RuochenLyu marked this pull request as ready for review April 29, 2026 11:43
Copy link
Copy Markdown
Collaborator

@jackwener jackwener left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the list cursor pagination changes. No blocking findings. The implementation threads cursor/return_cursor through fetch_list_timeline, emits pagination.nextCursor for structured output, and has focused CLI/client tests. Local checks: ruff passed, mypy passed, new focused tests passed. Full pytest currently hits two pre-existing Rich table truncation assertions that also fail on main in this local env, so they are not introduced by this PR.

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