Skip to content

feat(github): add GitHub connector for PR reviews and mentions#29

Merged
MaximeGaudin merged 3 commits into
mainfrom
feat/github-connector
Jun 21, 2026
Merged

feat(github): add GitHub connector for PR reviews and mentions#29
MaximeGaudin merged 3 commits into
mainfrom
feat/github-connector

Conversation

@MaximeGaudin

Copy link
Copy Markdown
Owner

Summary

  • Adds a read-only void-github connector that surfaces GitHub activity in the Void inbox:
    • Open PRs requesting your review (GitHub Search API)
    • Comments on pull requests you authored (Notifications API)
    • @mentions of your handle (Notifications API)
  • Each repository maps to a conversation, so repo muting works via void mute owner/repo or ignore_conversations
  • Includes setup wizard, config schema (token, username), and docs updates

Test plan

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace
  • cargo build --release
  • Run void setup, add GitHub connection with a PAT
  • Run void sync --daemon and verify PR review requests, PR comments, and mentions appear in void inbox
  • Run void mute owner/repo and confirm muted repo messages are excluded from inbox

Made with Cursor

MaximeGaudin and others added 3 commits June 21, 2026 17:50
Introduce void-github connector that syncs open PRs requesting review,
comments on authored PRs, and @mentions via GitHub REST APIs. Each
repository maps to a conversation so ignore_conversations supports
repo-level muting. Includes setup wizard, config schema, and docs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add wiremock-backed tests for the GitHub client error paths (401, 403
rate limit, 5xx, malformed JSON). Drop the unused GitHubError type (the
crate threads anyhow::Result throughout) and its now-unused thiserror
dependency. Remove the dead username argument from the sync path; it is
only needed for the authenticate() warning since review search uses
@me and mention notifications are already user-scoped. Advance the
notification cursor for every notification so a batch of only
filtered-out items still moves `since` forward instead of re-fetching
the same window each poll.

Co-authored-by: Cursor <cursoragent@cursor.com>
… cursor stalls

Co-authored-by: Cursor <cursoragent@cursor.com>
@MaximeGaudin MaximeGaudin merged commit d813614 into main Jun 21, 2026
7 checks passed
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