Use partial fetch (blob:none) for checkpoints metadata#712
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR optimizes the push recovery sync flow for Entire’s metadata/session branches by switching the recovery git fetch to a partial fetch that avoids downloading blobs, since the merge logic only needs commit/tree structure.
Changes:
- Use
git fetch --filter=blob:nonewhen fetching the remote metadata branch during non-fast-forward push recovery. - Expand inline comments to explain why blobs are not required for the merge.
pjbgf
force-pushed
the
pjbgf/partial-fetch-sync
branch
from
d6ab7ca to
226de0e
Compare
pjbgf
changed the title
pjbgf
force-pushed
the
pjbgf/partial-fetch-sync
branch
from
842f8aa to
1f48a50
Compare
Soph
approved these changes
Apr 9, 2026
Skip downloading blobs when fetching the remote metadata branch during push recovery. The merge only needs the tree structure to combine entries — blobs are already local or fetched on demand by git. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 56094f38982e
Signed-off-by: Paulo Gomes <paulo@entire.io> Entire-Checkpoint: d8c9d9506694
Signed-off-by: Paulo Gomes <paulo@entire.io>
pjbgf
force-pushed
the
pjbgf/partial-fetch-sync
branch
from
1f48a50 to
73bea7b
Compare
Soph
added a commit
that referenced
this pull request
Apr 13, 2026
git fetch --filter=blob:none <remote-name> persists remote.<name>.promisor=true and remote.<name>.partialclonefilter=blob:none into .git/config. Since #712, the four fetch helpers in git_operations.go passed "origin" as a named remote, which turned users' origin into a promisor remote repo-wide — affecting future `git fetch origin` / `git pull` and not just checkpoint reads. Resolve origin's URL first and pass the URL as both CheckpointGitCommand target and the remote argument. Anonymous URL fetches do not touch remote.*.promisor / remote.*.partialclonefilter in local config. Adds a regression test that fails against the pre-fix code and passes after. Does not yet unstick the config on users who already ran the affected CLI — that cleanup is a follow-up. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 4e0f55d5b407
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.
Skip downloading blobs when fetching the remote metadata branch during push recovery. Consolidate that practice across the different use cases of
git fetchfor checkpoints.Blobs are already local or fetched on demand by git during
entire resume.Depends on #680.
Note
Medium Risk
Changes the
git fetchbehavior used in push-recovery sync, which could behave differently on older git/server setups or if partial clone filters are unsupported. Otherwise the change is localized to metadata-session syncing and should reduce network/IO by avoiding blob downloads.Overview
During push recovery, the metadata-branch sync now performs a partial fetch by adding
--filter=blob:noneto thegit fetchinvocation infetchAndMergeSessionsCommon.This aims to speed up/trim network usage by fetching only commits/trees needed for the tree-merge step, skipping blob objects.
Reviewed by Cursor Bugbot for commit 226de0e. Configure here.