Skip to content

Refactor GitHub logic#13

Merged
awanlin merged 5 commits into
mainfrom
topic/refactor-github-logic
Feb 14, 2026
Merged

Refactor GitHub logic#13
awanlin merged 5 commits into
mainfrom
topic/refactor-github-logic

Conversation

@awanlin

@awanlin awanlin commented Feb 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

awanlin and others added 5 commits February 14, 2026 13:41
Add design document outlining approach to refactor GitHub issue
fetching from 87 individual API calls to 2 bulk fetch calls using
the GitHub Issues List API. This will reduce API calls by 43x and
eliminate rate limit errors.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add detailed step-by-step implementation plan for refactoring
GitHub issue fetching from 87 individual API calls to 2 bulk
fetch calls. Includes exact code, verification steps, and
success criteria.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add fetchContribfestIssues function to fetch all issues with
contribfest label from a repo using GitHub Issues List API.
Handles pagination automatically.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Add buildIssueLookupMap function to create a Map for O(1) lookup
of GitHub issues by repo and issue number.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Replace enrichIssuesWithGitHubData to use bulk fetching approach:
- Fetch all contribfest issues from both repos (2 API calls)
- Build lookup map for O(1) matching
- Match CSV entries with GitHub data
- Remove old fetchIssueMetadata and delay helper

This reduces API calls from 87 to 2, eliminating rate limit issues.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@awanlin
awanlin merged commit 2c3e5c9 into main Feb 14, 2026
1 check passed
@awanlin
awanlin deleted the topic/refactor-github-logic branch February 14, 2026 20:27
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