Skip to content

test(hf-hub): batch-3 private repo and auth correctness - #27

Merged
DiTo97 merged 4 commits into
masterfrom
issue/26-hf-hub-batch3-auth-visibility
Mar 12, 2026
Merged

test(hf-hub): batch-3 private repo and auth correctness#27
DiTo97 merged 4 commits into
masterfrom
issue/26-hf-hub-batch3-auth-visibility

Conversation

@DiTo97

@DiTo97 DiTo97 commented Mar 12, 2026

Copy link
Copy Markdown
Contributor

Summary

  • start batch 3 of the huggingface_hub compatibility rollout for private repo visibility and auth correctness
  • keep the scope limited to protected repo metadata, protected file reads, token=False behavior, and one narrow non-owner mutation boundary
  • update the roadmap/checklist in the same PR as the executable work

Test plan

  • HF_ENDPOINT=http://localhost:8080 uv run --with pytest --with huggingface_hub --with requests pytest tests/integration/hf_hub/test_auth_batch3.py -q
  • HF_ENDPOINT=http://localhost:8080 uv run --with pytest --with huggingface_hub --with requests pytest tests/integration/hf_hub/test_repo_visibility_batch3.py -q
  • HF_ENDPOINT=http://localhost:8080 uv run --with pytest --with huggingface_hub --with requests pytest tests/integration/hf_hub -q
  • HF_ENDPOINT=http://localhost:8080 uv run tests/integration/test_downloads.py

Closes #26

馃 Generated with Claude Code

DiTo97 and others added 4 commits March 12, 2026 04:07
Mark batch 2 as complete in the roadmap and checklist, move the active
pointer to batch 3, and link the new auth/visibility tracking issue so
the next slice starts from the merged mainline state.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Align batch-3 Hub behavior with huggingface_hub expectations by hiding private repos from anonymous reads, forbidding authenticated non-owner access, and rejecting non-owner writes to protected repos.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Keep the roadmap and checklist aligned with the active compatibility workflow by recording the live batch-3 pull request alongside the existing issue link.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Apply the CI-required rustfmt wrapping in the batch-3 auth helper module so the unit workflow can validate the branch cleanly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@DiTo97
DiTo97 marked this pull request as ready for review March 12, 2026 03:45
@DiTo97
DiTo97 merged commit 0bc8206 into master Mar 12, 2026
2 checks passed
@DiTo97
DiTo97 deleted the issue/26-hf-hub-batch3-auth-visibility branch March 12, 2026 03:45
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.

test(hf-hub): batch-3 private repo and auth correctness

1 participant