fix: handle git-lfs pointer files#438
Merged
Merged
Conversation
04f0435 to
9ec96fd
Compare
58576c8 to
5819d57
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
What
Handle git-lfs pointer files in tarball clones by extracting to a temp directory, detecting exact pointer files, and falling back to git clone when needed.
Why
Some repositories serve tarballs that contain Git LFS pointer files instead of real blob contents, so tarball-first cloning needs a fallback path for those repos.
resolved: #381
Changes
Testing
How you verified this (commands, scenarios, or N/A):
bun run test)Review notes
Breaking changes (or none)
None.
Risks / rollout (or none)
The tarball path now scans extracted files before copying them into place.
Focus areas for reviewers (optional)
Tar extraction flow, pointer detection, and the regression coverage for pointer detection and fallback.
Checklist