Checkpoints v2: Push v2 refs in parallel and clean up output#1094
Merged
computermode merged 5 commits intomainfrom May 2, 2026
Merged
Checkpoints v2: Push v2 refs in parallel and clean up output#1094computermode merged 5 commits intomainfrom
computermode merged 5 commits intomainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the v2 checkpoint pre-push behavior to reduce noisy per-ref progress output and speed up pushing by pushing multiple v2 refs concurrently.
Changes:
- Push v2 refs (
v2/main,v2/full/current, latestv2/full/<generation>) in parallel and emit aggregated, less-verbose stderr output. - Add
--no-write-fetch-headto v2 fetch operations used during conflict recovery to avoid writingFETCH_HEAD. - Expand test coverage to assert the new aggregated output format and to ensure failures still name the ref that failed.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| cmd/entire/cli/strategy/push_v2.go | Parallelizes v2 ref pushing and consolidates user-facing output; adjusts fetch options to avoid writing FETCH_HEAD. |
| cmd/entire/cli/strategy/push_v2_test.go | Adds assertions for aggregated output and a new test ensuring failures still identify the failed ref. |
pfleidi
reviewed
May 1, 2026
Entire-Checkpoint: 433d262793b3
Entire-Checkpoint: 49c7108d9f93
265592a to
ad2b9f7
Compare
Entire-Checkpoint: c6546764d9f0 Co-authored-by: Sven Pfleiderer <sven@entire.io>
ad2b9f7 to
36a727a
Compare
pfleidi
previously approved these changes
May 1, 2026
Entire-Checkpoint: f10bf2cbdfc7
pfleidi
approved these changes
May 1, 2026
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.
https://entire.io/gh/entireio/cli/trails/277
Push v2 checkpoints in parallel + reduce verbosity in output messaging.
Note
Medium Risk
Moderate risk because it changes the checkpoint push flow to run ref pushes concurrently and alters recovery/error reporting; concurrency could surface racey git/working-dir assumptions or make failures harder to reproduce.
Overview
Updates v2 checkpoint pushing to sync/push
v2/main,v2/full/current, and the latest archived generation concurrently, replacing per-ref progress spam with a single aggregated header, per-ref warnings, and a final success message.Refactors the v2 push path to return structured results (
v2RefPushResult) and perform fetch+merge recovery per ref without emitting interleaved output, and adds--no-write-fetch-headto checkpointgit fetchcalls to avoid updatingFETCH_HEAD.Tests are updated to capture stderr and assert the new aggregated messaging, and add coverage that failed pushes still name the specific ref that couldn’t be synced/pushed.
Reviewed by Cursor Bugbot for commit fb95c19. Configure here.