Skip to content

Make the deploy resilient to a bad API token#7

Merged
kwsantiago merged 1 commit into
mainfrom
fix-deploy-resilience
Jul 13, 2026
Merged

Make the deploy resilient to a bad API token#7
kwsantiago merged 1 commit into
mainfrom
fix-deploy-resilience

Conversation

@kwsantiago

Copy link
Copy Markdown
Contributor

The deploy on #6's merge failed with HTTP 401 because the API token was invalid, and list_open_prs was uncaught, so one bad data source took down the entire site build.

  • Build the open-Knots-PRs view first, so its ~150 calls get a fresh rate-limit budget before the large spec ingest (which can exhaust a token's hourly limit).
  • Wrap it so any failure (bad token, rate limit, network) logs a warning and skips just that view; the spec snapshot still deploys.

Verified locally: a bogus token now logs skipping open-Knots-PRs view and still writes a working index.html; a good token builds with the Knots view first.

@kwsantiago kwsantiago merged commit 316996e into main Jul 13, 2026
1 check passed
@kwsantiago kwsantiago deleted the fix-deploy-resilience branch July 13, 2026 22:32
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