On demand blob downloads for patches #17
Merged
Conversation
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
…ollback This change significantly reduces disk space usage by: - Only downloading afterHash blobs during download/apply/repair - Downloading beforeHash blobs on-demand during rollback operations - Removing beforeHash blobs during cleanup since they're fetched on-demand Changes: - Add getAfterHashBlobs() and getBeforeHashBlobs() to manifest/operations.ts - Update getMissingBlobs() to only check for afterHash blobs - Update savePatch() to skip saving beforeBlobContent - Update cleanupUnusedBlobs() to use getAfterHashBlobs() - Rename gc command to repair (with gc alias for backwards compatibility) - Add --offline flag to apply, rollback, and repair commands - Add blob-fetcher utility for on-demand blob downloading - Add fetchBlob() method to API client for downloading blobs by SHA256 hash - Add comprehensive tests for new functionality 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
mikolalysenko
changed the title
- Add orgSlug option to APIClient for authenticated blob downloads
- Use authenticated /v0/orgs/{org}/patches/blob/{hash} endpoint when token
and orgSlug are available, fall back to public proxy otherwise
- Add SOCKET_ORG_SLUG environment variable support
This enables blob downloads during rollback to work in CI environments where
the public proxy is not accessible.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
The rollback command now correctly uses fetchBlobsByHash to download the specific beforeHash blobs it needs, rather than fetchMissingBlobs which only downloads afterHash blobs. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
This change significantly reduces disk space usage by:
Changes: