Skip to content

fix(ci): add HF_TOKEN and model caching, remove uv.lock#275

Merged
HumanBean17 merged 1 commit into
masterfrom
fix/ci-huggingface-rate-limit
Jun 7, 2026
Merged

fix(ci): add HF_TOKEN and model caching, remove uv.lock#275
HumanBean17 merged 1 commit into
masterfrom
fix/ci-huggingface-rate-limit

Conversation

@HumanBean17

Copy link
Copy Markdown
Owner

Summary

  • Add HF_TOKEN environment variable to CI workflow for authenticated HuggingFace downloads
  • Add HuggingFace model caching to avoid repeated downloads and HTTP 429 errors
  • Set TRANSFORMERS_CACHE and HF_HOME environment variables for consistent model paths
  • Remove uv.lock that was accidentally committed in rollback commit
  • Add uv.lock to .gitignore to prevent future commits

Problem

CI was failing with HTTP 429 rate limiting errors from HuggingFace when loading the sentence-transformers/all-MiniLM-L6-v2 model during tests. The model was being downloaded repeatedly without authentication or caching.

Solution

  1. Added HF_TOKEN from GitHub secrets for authenticated requests (higher rate limits)
  2. Added HuggingFace model caching using GitHub Actions cache
  3. Set proper cache environment variables (TRANSFORMERS_CACHE, HF_HOME)
  4. Removed accidentally committed uv.lock file
  5. Added uv.lock to .gitignore

Test plan

  • CI passes with the new HF_TOKEN and caching configuration
  • No more HTTP 429 errors from HuggingFace

Generated with Claude Code

- Add HF_TOKEN environment variable to CI workflow for authenticated HuggingFace downloads
- Add HuggingFace model caching to avoid repeated downloads and HTTP 429 errors
- Set TRANSFORMERS_CACHE and HF_HOME environment variables for consistent model paths
- Remove uv.lock that was accidentally committed in rollback commit
- Add uv.lock to .gitignore to prevent future commits

This fixes the CI failure caused by HuggingFace rate limiting when multiple tests load the sentence-transformers model.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@HumanBean17 HumanBean17 merged commit d1d7cf2 into master Jun 7, 2026
6 checks passed
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