Skip to content

feat(cursor-origin): List and compare commits - #140

Merged
wedamija merged 6 commits into
mainfrom
danf/origin-provider-commits
Sep 24, 2026
Merged

wedamija merged 6 commits into
mainfrom
danf/origin-provider-commits

Conversation

@wedamija

Copy link
Copy Markdown
Member

Add get_commits and compare_commits. Origin's comparison only returns a commit count, so compare_commits cannot return the commits themselves. Changed files are returned separately and paginated.

Stacked on #135.

@wedamija
wedamija requested a review from a team as a code owner September 23, 2026 23:16
@wedamija
wedamija changed the base branch from danf/origin-provider-core to main September 24, 2026 18:13
@wedamija
wedamija force-pushed the danf/origin-provider-commits branch 2 times, most recently from e01ee47 to 86d58a5 Compare September 24, 2026 18:28
Comment thread src/scm/providers/cursor_origin/provider.py
@wedamija
wedamija force-pushed the danf/origin-provider-commits branch 3 times, most recently from f26d9d6 to 8667f58 Compare September 24, 2026 18:43
Add `get_commits` and `compare_commits`. Origin's comparison only returns a commit count, so `compare_commits` cannot return the commits themselves. Changed files are returned separately and paginated.
With no pagination, compare_commits returned Origin's default first page
of 30 files. Read pages of 100, up to the 300 files GitHub's compare
returns.
@wedamija
wedamija force-pushed the danf/origin-provider-commits branch from 6c72a60 to e1e9be4 Compare September 24, 2026 18:55
A caller's pagination reads one page; without it, up to three pages of
100 are read. The next cursor is returned in both cases.
@wedamija
wedamija merged commit fd0a9a4 into main Sep 24, 2026
14 of 15 checks passed
@wedamija
wedamija deleted the danf/origin-provider-commits branch September 24, 2026 19:00
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.

1 participant