Skip to content

Fix: stop fetching docs at Cloudflare build time#15

Open
initializ-mk wants to merge 1 commit intomainfrom
fix/build-rate-limit
Open

Fix: stop fetching docs at Cloudflare build time#15
initializ-mk wants to merge 1 commit intomainfrom
fix/build-rate-limit

Conversation

@initializ-mk
Copy link
Copy Markdown
Contributor

Summary

The deployed site was missing skills on /hub (only 4 fallback skills shown instead of 14) and the GitHub star count in the nav was .

Root cause: Cloudflare Pages builds run on shared IPs without GITHUB_TOKEN. The newly-added sync:docs step makes 2 extra api.github.com calls before fetch:skills and fetch:contributors. That's enough to push past the 60/hour unauthenticated quota, after which both scripts silently fall back to empty/default data.

Fix:

  • Remove sync:docs from the build chain — the GitHub Actions workflow (sync-docs.yaml) already handles syncing docs on forge repo updates.
  • Un-gitignore the synced docs paths so the workflow's commits actually land in the repo and Cloudflare ships them as static content.
  • Commit the currently-synced docs as the baseline.

Test plan

  • After merge, Cloudflare rebuild completes and /hub shows all 14 skills
  • Nav shows actual star count, not
  • Docs pages still render (now from committed files instead of build-time fetch)
  • Future doc updates trigger the sync-docs GitHub Actions workflow which commits and triggers a Cloudflare rebuild

Cloudflare Pages builds run on shared IPs without GITHUB_TOKEN, hitting
the 60/hr unauthenticated rate limit. The added sync:docs step made
two extra api.github.com calls before fetch:skills and fetch:contributors,
tipping the quota and causing both to silently fall back to defaults
(only 4 skills shown on /hub, "—" star count in nav).

Move the docs sync out of the build chain — the GitHub Actions workflow
already runs it on docs/** changes. Un-gitignore the synced docs so the
workflow's commits land in the repo and Cloudflare ships them as-is.
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