feat(init): adopt existing remote stackr metadata instead of initializing fresh - #41
Open
amustafa wants to merge 1 commit into
Open
feat(init): adopt existing remote stackr metadata instead of initializing fresh#41amustafa wants to merge 1 commit into
amustafa wants to merge 1 commit into
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
amustafa
force-pushed
the
am/init-adopt-remote-meta
branch
from
August 18, 2026 18:34
34e9a12 to
64819b2
Compare
amustafa
force-pushed
the
am/init-adopt-remote-meta
branch
from
August 19, 2026 14:56
64819b2 to
0d57b0d
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.
What
sr initin a clone whose remote already holds stackr metadata now adopts that shared state instead of creating blank local state.Why
Creating blank state in a second clone silently shadows the shared graph: two clones then disagree about the stack, and the first push-meta from the blank one clobbers or forks the shared metadata. Init is the natural first command a new clone runs, so it should notice the metadata and do the right thing.
How
runInitprobes each remote for the stackr metadata ref (newRunner.RemoteHasRef, anls-remote— unlikeRemoteBranchExistsit needs no prior fetch, since custom refs are never fetched by default). On a hit it announces the adoption and delegates toengine.PullMeta.sr initstill works.--resetkeeps meaning "start over deliberately" and skips the adoption probe.Stacked on the
pull-metabootstrap PR, whose semantics this relies on.🤖 Generated with Claude Code