fix: detect and repair disconnected metadata branches at read/write time by dvydra · Pull Request #533 · entireio/cli

dvydra · 2026-02-26T22:50:14Z

Summary

Detects and repairs disconnected entire/checkpoints/v1 metadata branches — a state caused by the empty-orphan bug (fixed going forward by #511). Users who hit that bug and continued working have local checkpoint data on a completely disconnected history from remote (no common ancestor).

Key distinction: Diverged (shared ancestor, unique commits on each side) is normal multi-dev flow handled by the push path's tree merge. Disconnected (no common ancestor) is the bug state that this PR repairs.

What changed

New: cherry-pick reconciliation (`strategy/metadata_reconcile.go`)

When local and remote share no common ancestor, local commits are cherry-picked onto the remote tip:

Detect via git merge-base (exit 0 = shared ancestry, exit 1 = disconnected)
Walk local branch tip→root, collect commits oldest-first
Skip empty-tree commits (the orphan bug artifact)
For each commit: compute full delta from parent (adds + modifies + deletes), apply onto current tip
Update local branch ref

Checkpoint shards use unique paths (<id[:2]>/<id[2:]>/), so cherry-picks always apply cleanly. All data is preserved with linear history.

Reconciliation runs at read/write time, not `enable` time

Runs once per process (via sync.Once) when commands actually access the metadata branch — explain, rewind, resume, clean, condensation. Three call sites:

ManualCommitStrategy.getCheckpointStore() — covers strategy-mediated reads/writes
getBranchCheckpoints() in explain.go — bypasses strategy, creates own GitStore
ListCheckpoints() in strategy/common.go — opens own repo, reads branch directly

This is the right place because enable may run before the user has fetched remote data.

Simplified `EnsureMetadataBranch` (runs during `enable`)

Now only handles:

Create local from remote when local doesn't exist
Replace empty orphan with remote
Create empty orphan when neither exists

Removed syncMetadataBranch and isAncestorCLI — disconnected repair is now handled by EnsureMetadataReconciled.

User-facing output

[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Cherry-picking 3 local checkpoint(s) onto remote...
[entire] Done — all local and remote checkpoints preserved

Files

File	Change
`strategy/metadata_reconcile.go`	New — detection + cherry-pick repair
`strategy/metadata_reconcile_test.go`	New — 7 tests covering no-ops, disconnected repair, multi-commit, modifications
`strategy/common.go`	`EnsureMetadataReconciled()` gate; simplified `EnsureMetadataBranch`; removed `syncMetadataBranch`/`isAncestorCLI`
`strategy/manual_commit.go`	Reconciliation call in `getCheckpointStore()`
`strategy/common_test.go`	Updated existing tests for new behavior
`cli/explain.go`	Reconciliation call in `getBranchCheckpoints()`

Test plan

mise run fmt && mise run lint && mise run test:ci
go test ./cmd/entire/cli/strategy/ -run TestReconcileDisconnected -v — all 7 tests pass
Manual: clone repo with remote checkpoints, create disconnected local orphan, run entire explain, verify reconciliation and explain succeeds

🤖 Generated with Claude Code

When enable previously created an empty orphan entire/checkpoints/v1 and the remote later has real checkpoint data, detect the empty local branch and update it from the remote-tracking branch. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 15849a699702

When a user hit the empty-orphan bug and then worked normally, the local entire/checkpoints/v1 has real checkpoint data on a disconnected history from the remote. EnsureMetadataBranch now detects this using git merge-base and reconciles by merging both trees (checkpoint shards are unique, so no conflicts). Also handles the simpler case where local is behind remote (fast-forward). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: cee58fd2febc

…ite time Move detection and repair of disconnected local/remote metadata branches from EnsureMetadataBranch (enable) to a sync.Once-guarded reconciliation that runs when commands actually access the metadata branch (explain, rewind, resume, etc.). Uses cherry-pick strategy instead of merge to preserve linear commit history. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 639c85c9977a

cursor · 2026-02-26T22:50:20Z

PR Summary

Medium Risk
Changes git-ref handling and creates new commits to repair metadata history; mistakes could hide or overwrite checkpoint metadata despite added test coverage.

Overview
Fixes corrupted/disconnected checkpoint metadata histories. Adds ReconcileDisconnectedMetadataBranch to detect when local and origin/entire/checkpoints/v1 share no merge-base and repair by cherry-picking local metadata commits onto the remote tip (skipping empty-tree orphan commits) and updating the local ref.

Moves reconciliation to metadata access paths (once per process). EnsureMetadataReconciled (sync.Once gated) is invoked before listing/reading checkpoints in ListCheckpoints, ManualCommitStrategy.getCheckpointStore, and explain’s getBranchCheckpoints, emitting warnings on failure.

Tightens entire enable behavior. EnsureMetadataBranch now only seeds the local branch from origin when missing or when the local branch is an empty orphan, and explicitly avoids reconciling/fast-forwarding non-empty or diverged branches; tests were expanded and new reconciliation tests were added.

^{Written by Cursor Bugbot for commit 86d7433. Configure here.}

Copilot

Pull request overview

This PR moves “disconnected metadata branch” detection/repair out of entire enable and into a read/write-time reconciliation path, aiming to automatically fix repos affected by the historical empty-orphan metadata branch bug while preserving linear history.

Changes:

Adds ReconcileDisconnectedMetadataBranch (+ sync.Once-guarded EnsureMetadataReconciled) to detect disconnected entire/checkpoints/v1 local vs origin/* and repair via cherry-picking.
Updates several read/write entrypoints to call reconciliation before accessing checkpoint metadata.
Expands strategy tests to cover new metadata-branch behaviors and adds a dedicated reconciliation test suite.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
cmd/entire/cli/strategy/metadata_reconcile.go	Implements disconnected-branch detection and cherry-pick-based repair logic.
cmd/entire/cli/strategy/metadata_reconcile_test.go	Adds unit tests for reconciliation scenarios (no remote/local, shared ancestry, disconnected, multi-local commits).
cmd/entire/cli/strategy/common.go	Adds `EnsureMetadataReconciled` gate; adjusts `EnsureMetadataBranch`; calls reconcile before listing checkpoints.
cmd/entire/cli/strategy/common_test.go	Adds tests for updated `EnsureMetadataBranch` semantics and new helper `cloneWithConfig`.
cmd/entire/cli/strategy/manual_commit.go	Ensures reconciliation is invoked before initializing the checkpoint store.
cmd/entire/cli/explain.go	Ensures reconciliation is invoked before reading committed checkpoints for explain/rewind points.

Only treat plumbing.ErrReferenceNotFound as "no branch" when checking local and remote metadata refs. Other errors (e.g. corrupt packfiles) are now surfaced instead of silently swallowed. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 95aa08abfa40

git merge-base uses exit code 1 for "no common ancestor" and 128+ for actual errors (corrupt repo, invalid hash, timeout). Only treat exit code 1 as "disconnected"; surface other failures as errors to prevent spurious cherry-pick reconciliation. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: e73dd6177226

…tions The previous additions-only diff missed modifications (e.g., multi-session condensation updating metadata.json) and deletions. Now computes the full diff between each commit and its parent: entries with changed hashes are applied, and entries removed in the commit are deleted from the tip tree. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 562308d98f2b

Previously, if isEmptyMetadataBranch returned an error (e.g., corrupt object, missing commit), EnsureMetadataBranch silently skipped the empty orphan check, potentially leaving the repo in a broken state. Now surfaces the error to callers. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: fd02a6f99c6c

Hoist reconcileResult to package level so subsequent callers of EnsureMetadataReconciled see the same error, not nil. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: fc3459657b3d

dvydra · 2026-02-26T23:38:49Z

bugbot run

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.

…l-flow Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 5d5f08442ce7

cursor

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Add a test utility to reset the sync.Once reconciliation gate, matching the existing resetProtectedDirsForTest pattern. Prevents stale cached results from leaking across tests. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: c20cb099d45c

…e trees A corrupted tree object during data repair should abort reconciliation, not silently drop the commit and lose user data. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 49496929adf3

A partial FlattenTree read on the parent would produce an incorrect delta (treating unread entries as deleted). Propagate all three error paths: parent commit, parent tree, and FlattenTree. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: c2930cbb7765

Prevents silent data loss if the local metadata branch has more than 1000 commits — reconciliation aborts with an error instead of silently truncating the chain. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: e59e395232b3

The repo parameter was only used on the first call due to sync.Once, making the API misleading. Now opens its own repository internally, matching the ListCheckpoints pattern and simplifying all three callers. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 6242e73a4234

Log at debug level when EnsureMetadataBranch detects a non-empty local branch that differs from remote, making it visible in logs that reconciliation was deferred to read/write time. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: f6f13afacc0a

Move the debug log into an else clause so it only fires when the local metadata branch has real data that differs from remote. Previously it fell through after the empty-orphan update case, logging misleadingly. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 851f7418bdb5

The function was never called — no tests trigger EnsureMetadataReconciled through the sync.Once gate. Remove it and the comment referencing it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: f5b938b258c8

dvydra · 2026-02-27T01:39:03Z

Manual test

Tested against a fresh repo with real remote checkpoints and a disconnected local metadata branch.

Setup:

Created test repo, did work with Claude, pushed (pre-push hook pushed entire/checkpoints/v1)
Cloned into /tmp/frog, ran entire enable — this created a disconnected local orphan (empty, no checkpoint data) vs remote's real checkpoint data
Built the fix branch: go build -o /tmp/entire ./cmd/entire

Result:

$ /tmp/entire explain --no-pager
[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Done — local had no checkpoint data, reset to remote
Branch: main
Checkpoints: 1

[0c66dc95f5c0] "make a test file with a frog ascii in it."
  02-27 12:24 (a2d64e9) Add enterprise-ready frog ASCII art with legal disclaimer

Reconciliation detected the disconnected state, recognized the local branch had no real checkpoint data (just the empty orphan), reset to remote, and explain succeeded showing the remote checkpoint.

dvydra · 2026-02-27T03:29:11Z

Manual testing (round 2 — full cherry-pick path)

Test 1: Empty orphan reset (frog)

Cloned repo with remote checkpoints. entire enable created a disconnected empty orphan. Fix binary detected and reset to remote:

$ /tmp/entire explain --no-pager
[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Done — local had no checkpoint data, reset to remote
Branch: main
Checkpoints: 1

[0c66dc95f5c0] "make a test file with a frog ascii in it."
  02-27 12:24 (a2d64e9) Add enterprise-ready frog ASCII art with legal disclaimer

Test 2: Cherry-pick with real local checkpoints (frog4)

Created a properly disconnected empty orphan (simulating the pre-#511 bug), then used Claude + stock entire to create a real commit with a real checkpoint on the disconnected branch. Did NOT push.

Stock entire (before fix) — only sees local checkpoint, no remote data:

$ entire explain
Branch: multi-amphibian-architecture
Checkpoints: 1

[8b68bd0ab277] "double frogs lol"
  02-27 14:26 (c9a79b8) Migrate to multi-amphibian microswamp architecture

Fix binary — detects disconnected state, cherry-picks local onto remote:

$ /tmp/entire explain --no-pager
[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Cherry-picking 1 local checkpoint(s) onto remote...
[entire] Done — all local and remote checkpoints preserved
Branch: multi-amphibian-architecture
Checkpoints: 1

[8b68bd0ab277] "double frogs lol"
  02-27 14:26 (c9a79b8) Migrate to multi-amphibian microswamp architecture

After reconciliation, switching to main shows all checkpoints:

$ /tmp/entire explain --no-pager
Branch: main
Checkpoints: 2

[e5b34036e8da] "let's review our last pr, with the hate"
  02-27 13:05 (d603ca8) hat

[0c66dc95f5c0] "make a test file with a frog ascii in it."
  02-27 12:24 (a2d64e9) Add enterprise-ready frog ASCII art with legal disclaimer

Both local and remote checkpoint data preserved. Reconciliation runs once, subsequent explains are instant.

khaong

Overall this is a well-designed repair mechanism for the disconnected metadata branch bug state. The cherry-pick approach is correct, the distinction between disconnected vs diverged is clear, and test coverage is thorough (7 tests covering no-ops, single/multi-commit cherry-picks, and modifications).

A few items worth discussing inline — the most impactful is the permanent error caching via sync.Once.

Review generated by Claude, checked by Alex.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 2df5313f9ab9

dvydra · 2026-03-02T23:08:23Z

better approach, only use in two situations:

pre-push before we merge trees
in entire doctor, with detection and pointers to run doctor

Instead of eagerly reconciling disconnected metadata branches on every checkpoint read (ListCheckpoints, getCheckpointStore, getBranchCheckpoints), limit reconciliation to two explicit fix points: 1. Pre-push hook: reconcile in fetchAndMergeSessionsCommon() after fetch but before tree merge, so local branch is consistent before combining. 2. entire doctor: detect disconnected branches, explain the situation, and offer to fix (with --force support). All other read paths now use WarnIfMetadataDisconnected() which prints a one-time warning pointing users to 'entire doctor' without attempting repair. - Add IsMetadataDisconnected() for lightweight detection - Add WarnIfMetadataDisconnected() with sync.Once warning - Remove EnsureMetadataReconciled() and its global state - Broaden doctor command to "Diagnose and fix session issues" - Add 5 tests for IsMetadataDisconnected covering all edge cases - Fix pre-existing missing closing brace in common_test.go Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 10eb4a886bf0

…omment - Thread context.Context through IsMetadataDisconnected, ReconcileDisconnectedMetadataBranch, and isDisconnected for proper cancellation propagation - Rename authorName/authorEmail to committerName/committerEmail in createCherryPickCommit (they populate the Committer field, not Author) - Clarify isEmptyMetadataBranch doc comment: only checks tip commit, which is correct for the empty-orphan bug scenario Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 14ee21d50eff

dvydra · 2026-03-03T09:41:24Z

Manual Test Results: Disconnected Metadata Branch Reconciliation

Binary: Built from 6905d4d3 (latest on this branch)
Test repo: entirehq/terrible-temp-test-repo-that-sucks-to-test-pr-533

All 8 tests passed.

Test 1: Doctor reports OK when branches are connected

Created local metadata branch from remote (same hash), ran entire doctor --force.


Expected	`Metadata branches: OK`
Result	`Metadata branches: OK`

Test 2: Break it — create a disconnected local metadata branch

Created a disconnected orphan branch with a fake checkpoint (ff/ff00112233), pointed local entire/checkpoints/v1 at it. Verified with git merge-base.


Expected	`git merge-base` exit code 1 (no common ancestor)
Result	Exit code: 1

Test 3: Warning on read paths (`entire explain`)

Ran entire explain with disconnected branches.


Expected	One-time warning on stderr pointing to `entire doctor`
Result	Warning shown, no crash, explain continued normally

[entire] Warning: Local and remote session metadata branches are disconnected.
[entire] Some checkpoints from remote may not be visible. Run 'entire doctor' to fix.
Branch: main
Checkpoints: 0

Test 4: Doctor detects and fixes the problem

Ran entire doctor interactively, selected Yes.


Expected	Detect disconnection, explain, prompt, cherry-pick, confirm fix
Result	All steps executed correctly

Metadata branches: DISCONNECTED
  Local and remote entire/checkpoints/v1 branches share no common ancestor.
  Some remote checkpoints may not be visible locally.
  Fix: cherry-pick local checkpoints onto remote tip (preserves all data).

[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Cherry-picking 1 local checkpoint(s) onto remote...
[entire] Done — all local and remote checkpoints preserved
  -> Fixed: metadata branches reconciled

Test 5: Verify the fix preserved all data

Checked git log and tree contents after reconciliation.


Expected	Linear history, all checkpoint shards present
Result	Cherry-picked commit sits linearly on remote tip, all data preserved

* e601dc7 (entire/checkpoints/v1) Checkpoint: ffff00112233      <-- cherry-picked local
* b3e9833 (origin/entire/checkpoints/v1) Checkpoint: e863f7b5d8b2  <-- old remote tip
* 55b6295 Checkpoint: 8b68bd0ab277
*   91d8939 Merge remote session logs                               <-- pre-existing history

The merge commit at 91d8939 is pre-existing remote history (from a previous push), not from our reconciliation.

Test 6: Doctor reports OK after fix


Expected	`Metadata branches: OK`
Result	`Metadata branches: OK`

Test 7: Pre-push reconciliation

Broke the branches again (new disconnected orphan with aa/bb11223344), then pushed to trigger the pre-push hook.


Expected	Pre-push hook reconciles before tree merge, push succeeds
Result	Reconciliation ran during push, data pushed to remote

[entire] Pushing session logs to origin...
[entire] Syncing with remote session logs...
[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Cherry-picking 1 local checkpoint(s) onto remote...
[entire] Done — all local and remote checkpoints preserved

Test 8: Empty orphan case (exact bug artifact)

Created an empty orphan commit (no data, just the bug artifact Initialize metadata branch), ran entire doctor --force.


Expected	Detect empty orphan, reset to remote (no cherry-pick needed)
Result	Reset to remote, hashes match

[entire] Detected disconnected session metadata (local and remote share no common ancestor)
[entire] Done — local had no checkpoint data, reset to remote
  -> Fixed: metadata branches reconciled

$ git rev-parse entire/checkpoints/v1
32d64a5e58fe53b99d37f696f3721eb7e3d4aa21
$ git rev-parse origin/entire/checkpoints/v1
32d64a5e58fe53b99d37f696f3721eb7e3d4aa21

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 8c9f5138534c

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: aa35dcf5b7de

…-commit paths Three error-handling fixes from PR review: 1. Push path (push_common.go): reconciliation failure now returns an error instead of warning-and-continue. Proceeding to tree merge on disconnected branches would silently combine unrelated histories. 2. Doctor command (doctor.go): metadata check failure now propagates to the return value (non-zero exit). Previously, `entire doctor` could report success when the recommended repair path itself failed. 3. Post-commit hook (manual_commit_hooks.go): filesChangedInCommit now falls back to go-git tree walk when git diff-tree fails, instead of returning an empty map that breaks downstream condensation logic. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 7c5b9b10fb55

…etadata reconciliation - Add io.Writer parameter to ReconcileDisconnectedMetadataBranch for testable progress output - Add debug logging to WarnIfMetadataDisconnected error paths (sync.Once suppression) - Add tests: depth limit, all-empty-orphan reconciliation, cherry-pick deletion propagation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 58ca8a0a0cb5

…ected-metadata-branch

khaong

Reviewed: error propagation, observability, testability, and edge-case coverage all look good. All tests pass after merge with main.

dvydra and others added 3 commits February 26, 2026 16:35

Copilot AI review requested due to automatic review settings February 26, 2026 22:50

dvydra requested a review from a team as a code owner February 26, 2026 22:50

Copilot started reviewing on behalf of dvydra February 26, 2026 22:50 View session

This was referenced Feb 26, 2026

fix: update empty local metadata branch from remote on enable #516

Closed

fix: reconcile disconnected local/remote metadata branches on enable #518

Closed

Copilot AI reviewed Feb 26, 2026

View reviewed changes

cursor Bot reviewed Feb 26, 2026

View reviewed changes

Comment thread cmd/entire/cli/strategy/metadata_reconcile.go

Comment thread cmd/entire/cli/strategy/common.go Outdated

dvydra and others added 6 commits February 27, 2026 10:00

fix: cache reconciliation error for all callers

1cc51d7

Hoist reconcileResult to package level so subsequent callers of EnsureMetadataReconciled see the same error, not nil. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: fc3459657b3d

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

86d7433

dvydra requested a review from Copilot February 26, 2026 23:38

Copilot started reviewing on behalf of dvydra February 26, 2026 23:38 View session

Copilot AI reviewed Feb 26, 2026

View reviewed changes

Comment thread cmd/entire/cli/strategy/metadata_reconcile.go

Comment thread cmd/entire/cli/strategy/metadata_reconcile.go

Comment thread cmd/entire/cli/strategy/metadata_reconcile.go

Comment thread cmd/entire/cli/strategy/metadata_reconcile_test.go

fix: use MkdirAll before WriteFile in test instead of error-as-contro…

321a014

…l-flow Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 5d5f08442ce7

cursor Bot reviewed Feb 26, 2026

View reviewed changes

dvydra and others added 7 commits February 27, 2026 10:53

chore: remove unused resetReconcileOnceForTest

8e3df86

The function was never called — no tests trigger EnsureMetadataReconciled through the sync.Once gate. Remove it and the comment referencing it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: f5b938b258c8

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

3351de8

dvydra commented Feb 27, 2026

View reviewed changes

Comment thread cmd/entire/cli/strategy/common.go

khaong reviewed Feb 27, 2026

View reviewed changes

dvydra added a commit that referenced this pull request Mar 2, 2026

fix: remove metadata_reconcile.go (belongs to PR #533, not yet merged)

ba48780

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 2df5313f9ab9

dvydra mentioned this pull request Mar 2, 2026

refactor: unify tree modification to use ApplyTreeChanges #573

Closed

3 tasks

dvydra and others added 3 commits March 3, 2026 14:49

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

b7304ed

dvydra and others added 11 commits March 3, 2026 20:50

Use [entire] prefix instead of tick mark in metadata branch message

9832e9a

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: 8c9f5138534c

Add missing t.Parallel() to empty orphan subtest

9be77b4

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Entire-Checkpoint: aa35dcf5b7de

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

a942b79

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

ec4b703

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

a691b23

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

fcebf4b

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

de88806

Merge branch 'main' into fix/reconcile-disconnected-metadata-branch

9fe689d

Merge remote-tracking branch 'origin/main' into fix/reconcile-disconn…

7ebda07

…ected-metadata-branch

khaong approved these changes Mar 5, 2026

View reviewed changes

dvydra merged commit 329f05c into main Mar 6, 2026
3 checks passed

dvydra deleted the fix/reconcile-disconnected-metadata-branch branch March 6, 2026 01:13

Uh oh!

Conversation

dvydra commented Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

What changed

New: cherry-pick reconciliation (strategy/metadata_reconcile.go)

Reconciliation runs at read/write time, not enable time

Simplified EnsureMetadataBranch (runs during enable)

User-facing output

Files

Test plan

Uh oh!

cursor Bot commented Feb 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dvydra commented Feb 26, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

dvydra commented Feb 27, 2026

Manual test

Uh oh!

dvydra commented Feb 27, 2026

Manual testing (round 2 — full cherry-pick path)

Test 1: Empty orphan reset (frog)

Test 2: Cherry-pick with real local checkpoints (frog4)

Uh oh!

Uh oh!

khaong left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

dvydra commented Mar 2, 2026

Uh oh!

dvydra commented Mar 3, 2026

Manual Test Results: Disconnected Metadata Branch Reconciliation

Test 1: Doctor reports OK when branches are connected

Test 2: Break it — create a disconnected local metadata branch

Test 3: Warning on read paths (entire explain)

Test 4: Doctor detects and fixes the problem

Test 5: Verify the fix preserved all data

Test 6: Doctor reports OK after fix

Test 7: Pre-push reconciliation

Test 8: Empty orphan case (exact bug artifact)

Uh oh!

khaong left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

dvydra commented Feb 26, 2026 •

edited

Loading

New: cherry-pick reconciliation (`strategy/metadata_reconcile.go`)

Reconciliation runs at read/write time, not `enable` time

Simplified `EnsureMetadataBranch` (runs during `enable`)

cursor Bot commented Feb 26, 2026 •

edited

Loading

Test 3: Warning on read paths (`entire explain`)