Skip to content

Move GitHub contributors fetching to scheduled data-layer task#17174

Draft
pettinarip wants to merge 11 commits intodevfrom
perf/github-contributors-data-layer
Draft

Move GitHub contributors fetching to scheduled data-layer task#17174
pettinarip wants to merge 11 commits intodevfrom
perf/github-contributors-data-layer

Conversation

@pettinarip
Copy link
Member

@pettinarip pettinarip commented Jan 27, 2026

Summary

  • Replace per-request GitHub API calls with pre-fetched data stored in Netlify Blobs
  • Add fetchGitHubContributors scheduled task running weekly to fetch all contributor data
  • Use git/trees API for single-request content file listing (replaces recursive directory walks)
  • Fetch commits in parallel batches (20 concurrent requests) for performance
  • Update contributors.ts to read from data-layer instead of making live API calls
  • Bump @trigger.dev/sdk to 4.3.3

Test plan

  • Verify the scheduled task runs successfully and stores data in Netlify Blobs
  • Verify contributor information displays correctly on content pages
  • Verify contributor information displays correctly on app pages
  • Verify build succeeds without errors

Replace per-request GitHub API calls with pre-fetched data stored in
Netlify Blobs. This eliminates hundreds of API calls during page renders.

- Add fetchGitHubContributors scheduled task (weekly)
- Use git/trees API for single-request content listing
- Fetch commits in parallel batches (20 concurrent)
- Store results in data-layer for fast retrieval
- Update contributors.ts to read from data-layer
- Bump @trigger.dev/sdk to 4.3.3
@github-actions github-actions bot added dependencies 📦 Changes related to project dependencies tooling 🔧 Changes related to tooling of the project labels Jan 27, 2026
@netlify
Copy link

netlify bot commented Jan 27, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 61d79bd
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69a69cad39946e000807870e
😎 Deploy Preview https://deploy-preview-17174.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 62 (🟢 up 2 from production)
Accessibility: 94 (🟢 up 1 from production)
Best Practices: 100 (no change from production)
SEO: 99 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@pettinarip pettinarip marked this pull request as draft January 27, 2026 13:46
The previous commit moved GitHub contributor fetching to the data-layer.
This removes the now-unused fetchAndCacheGitHubContributors and
fetchWithRateLimit functions from gh.ts, along with their imports.
Also consolidates CONTENT_DIR and OLD_CONTENT_DIR imports in the new
fetcher to use the shared constants.
@github-actions github-actions bot added the documentation 📖 Change or add documentation label Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies 📦 Changes related to project dependencies documentation 📖 Change or add documentation tooling 🔧 Changes related to tooling of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant