Skip to content

fix(ci): retry whisper model downloads#39

Merged
kentcdodds merged 4 commits into
mainfrom
cursor/fix-whisper-model-download-0ebd
May 13, 2026
Merged

fix(ci): retry whisper model downloads#39
kentcdodds merged 4 commits into
mainfrom
cursor/fix-whisper-model-download-0ebd

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented May 13, 2026

Summary

  • Add retry/backoff handling for whisper.cpp model downloads, including 429 responses from Hugging Face.
  • Retry interrupted response body reads as well as retryable HTTP/network failures.
  • Add unit coverage for rate-limit retries, interrupted downloads, and permanent failures.
  • Narrow the injectable fetch type so test doubles do not need Bun-specific fetch properties.

Testing

  • bun test src/whispercpp-transcribe.test.ts
  • bun run typecheck
  • bun run validate
  • GitHub PR checks: validate test/main jobs passed, playwright-smoke passed, e2e passed, Bugbot passed; release job skipped as expected.
Open in Web Open in Cursor 

cursoragent and others added 3 commits May 13, 2026 21:00
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds marked this pull request as ready for review May 13, 2026 21:02
Co-authored-by: Kent C. Dodds <me+github@kentcdodds.com>
@kentcdodds kentcdodds merged commit 034c6d5 into main May 13, 2026
6 checks passed
@kentcdodds kentcdodds deleted the cursor/fix-whisper-model-download-0ebd branch May 13, 2026 21:19
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.

2 participants