Skip to content

feat(cursor-origin): Write pull request comments and reviews - #145

Merged
wedamija merged 1 commit into
mainfrom
danf/origin-provider-comment-writes
Sep 25, 2026
Merged

wedamija merged 1 commit into
mainfrom
danf/origin-provider-comment-writes

Conversation

@wedamija

@wedamija wedamija commented Sep 24, 2026 •

Copy link
Copy Markdown
Member

Add support for posting general and inline review comments, editing them, and collapsing them. Origin cannot hide a comment, so collapsing resolves its thread.

Add support for listing a pull request's reviews and submitting a review with its comments in one request. Origin submits the review and comments atomically, so if any comment cannot be anchored to a line in the diff, nothing is posted.

@wedamija
wedamija requested a review from a team as a code owner September 24, 2026 23:16
@wedamija
wedamija force-pushed the danf/origin-provider-comment-reads branch from dd0dd9f to 5538d55 Compare September 25, 2026 16:57
Comment thread src/scm/providers/cursor_origin/provider.py
Add support for posting general and inline review comments, editing them, and collapsing them. Origin cannot hide a comment, so collapsing resolves its thread.

Add support for listing a pull request's reviews and submitting a review with its comments in one request. Origin submits the review and comments atomically, so if any comment cannot be anchored to a line in the diff, nothing is posted.
@wedamija
wedamija force-pushed the danf/origin-provider-comment-writes branch from f5a7399 to 1536c2e Compare September 25, 2026 17:00
@wedamija wedamija changed the title feat(cursor-origin): Post, edit and resolve pull request comments feat(cursor-origin): Write pull request comments and reviews Sep 25, 2026
@wedamija
wedamija changed the base branch from danf/origin-provider-comment-reads to main September 25, 2026 17:00
@wedamija
wedamija merged commit d82e205 into main Sep 25, 2026
17 of 19 checks passed
@wedamija
wedamija deleted the danf/origin-provider-comment-writes branch September 25, 2026 17:05
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