feat(providers): add indivudal GitHub contributions #18
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #18 +/- ##
============================
============================
|
21 tasks
ReenigneArcher
commented
Nov 18, 2025
Member
Author
ReenigneArcher
left a comment
ReenigneArcher
left a comment
There was a problem hiding this comment.
Should also add an option to set an upper limit to contributions or view them logarithmic, so that one org doesn't end up overtaking everything. radiusMax does not seem to handle this correctly.
| - [**CrowdIn**](https://crowdin.com) | ||
| - [**GitHub**](https://github.com) | ||
| - [**GitHub Contributors**](https://github.com) (contributors to a specific repository) | ||
| - [**GitHub Contributions**](https://github.com) (commit-only contributions aggregated by repository owner across all repos for a single user) |
Member
Author
There was a problem hiding this comment.
Suggested change
| - [**GitHub Contributions**](https://github.com) (commit-only contributions aggregated by repository owner across all repos for a single user) | |
| - [**GitHub Contributions**](https://github.com) (merged PRs aggregated by repository owner across all repos for a single user) |
Comment on lines
+55
to
+59
| ; Optional tuning: | ||
| ; Retry attempts for transient errors (5xx), default 3 | ||
| CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRIES=3 | ||
| ; Initial retry delay in ms, default 500 (exponential backoff with jitter) | ||
| CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRY_DELAY_MS=500 |
Member
Author
There was a problem hiding this comment.
Suggested change
| ; Optional tuning: | |
| ; Retry attempts for transient errors (5xx), default 3 | |
| CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRIES=3 | |
| ; Initial retry delay in ms, default 500 (exponential backoff with jitter) | |
| CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRY_DELAY_MS=500 |
Not necessary.
| 1. **contributionsCollection** - Yearly commit timeline (full history) for discovering repositories you have committed to | ||
| 2. **Search API** - Repositories where you have merged PRs (`is:pr is:merged author:login`) | ||
| - When an owner has only one repo, the link points to that repo; otherwise to the owner profile | ||
| - Better metric than commits as it represents completed, reviewed contributions |
Member
Author
There was a problem hiding this comment.
Suggested change
| - Better metric than commits as it represents completed, reviewed contributions |
Comment on lines
+63
to
+64
| retries: Number(process.env.CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRIES) || 3, | ||
| retryDelayMs: Number(process.env.CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRY_DELAY_MS) || 500, |
Member
Author
There was a problem hiding this comment.
Suggested change
| retries: Number(process.env.CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRIES) || 3, | |
| retryDelayMs: Number(process.env.CONTRIBKIT_GITHUB_CONTRIBUTIONS_RETRY_DELAY_MS) || 500, |
Comment on lines
+13
to
+15
| // Ensure unique clipPath id per element. Previously, we only hashed the image bytes, | ||
| // which caused duplicates when many items shared the same avatar. Include geometry too. | ||
| const cropId = `c${crypto.createHash('md5').update(`${x}:${y}:${size}:${radius}:${base64Image}`).digest('hex').slice(0, 6)}` |
Member
Author
There was a problem hiding this comment.
Suggested change
| // Ensure unique clipPath id per element. Previously, we only hashed the image bytes, | |
| // which caused duplicates when many items shared the same avatar. Include geometry too. | |
| const cropId = `c${crypto.createHash('md5').update(`${x}:${y}:${size}:${radius}:${base64Image}`).digest('hex').slice(0, 6)}` | |
| // Unique clipPath id per element, ensuring duplicated images are properly rendered. | |
| const cropId = `c${crypto.createHash('md5').update(`${x}:${y}:${size}:${radius}:${base64Image}`).digest('hex').slice(0, 6)}` |
Comment on lines
+13
to
+16
| { | ||
| retries: config.githubContributions?.retries ?? 3, | ||
| retryDelayMs: config.githubContributions?.retryDelayMs ?? 500, | ||
| }, |
Member
Author
There was a problem hiding this comment.
Suggested change
| { | |
| retries: config.githubContributions?.retries ?? 3, | |
| retryDelayMs: config.githubContributions?.retryDelayMs ?? 500, | |
| }, |
Comment on lines
+35
to
+44
| interface FetchOptions { | ||
| retries?: number | ||
| retryDelayMs?: number | ||
| } | ||
|
|
||
| export async function fetchGitHubContributions( | ||
| token: string, | ||
| login: string, | ||
| options: FetchOptions = {}, | ||
| ): Promise<Sponsorship[]> { |
Member
Author
There was a problem hiding this comment.
Suggested change
| interface FetchOptions { | |
| retries?: number | |
| retryDelayMs?: number | |
| } | |
| export async function fetchGitHubContributions( | |
| token: string, | |
| login: string, | |
| options: FetchOptions = {}, | |
| ): Promise<Sponsorship[]> { | |
| export async function fetchGitHubContributions( | |
| token: string, | |
| login: string, | |
| ): Promise<Sponsorship[]> { |
Comment on lines
+51
to
+53
| const retries = Math.max(0, options.retries ?? 3) | ||
| const retryDelayMs = Math.max(0, options.retryDelayMs ?? 500) | ||
|
|
Member
Author
There was a problem hiding this comment.
Suggested change
| const retries = Math.max(0, options.retries ?? 3) | |
| const retryDelayMs = Math.max(0, options.retryDelayMs ?? 500) |
| const circles = p(root as any).descendants().slice(1) | ||
|
|
||
| // Draw smaller circles last so they appear on top and are not hidden by larger ones | ||
| circles.sort((a, b) => a.r - b.r) |
Member
Author
There was a problem hiding this comment.
Test without this change.
Comment on lines
+301
to
+310
| /** | ||
| * Number of retries for GitHub GraphQL requests on transient failures (e.g., 5xx). | ||
| * @default 3 | ||
| */ | ||
| retries?: number | ||
| /** | ||
| * Initial retry delay in milliseconds for backoff. | ||
| * @default 500 | ||
| */ | ||
| retryDelayMs?: number |
Member
Author
There was a problem hiding this comment.
Suggested change
| /** | |
| * Number of retries for GitHub GraphQL requests on transient failures (e.g., 5xx). | |
| * @default 3 | |
| */ | |
| retries?: number | |
| /** | |
| * Initial retry delay in milliseconds for backoff. | |
| * @default 500 | |
| */ | |
| retryDelayMs?: number |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Add an individual contributions provider for GitHub.
Screenshot
Issues Fixed or Closed
Roadmap Issues
Type of Change
Checklist
AI Usage