Skip to content

fix(downloader): remove stale symlink targets#1140

Open
stamsam wants to merge 1 commit into
jundot:mainfrom
stamsam:codex/fix-download-existing-dir
Open

fix(downloader): remove stale symlink targets#1140
stamsam wants to merge 1 commit into
jundot:mainfrom
stamsam:codex/fix-download-existing-dir

Conversation

@stamsam
Copy link
Copy Markdown

@stamsam stamsam commented May 9, 2026

Summary

  • remove broken symlink placeholders before starting a Hugging Face download
  • preserve valid directories/symlinks so partial downloads can still resume
  • add regression coverage for the dashboard failure shape: [Errno 17] File exists: .../.omlx/models/<model>

Tests

  • .venv/bin/python -m pytest tests/test_hf_downloader.py (80 passed)
  • python3 -m py_compile omlx/admin/hf_downloader.py tests/test_hf_downloader.py
  • git diff --check

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