Skip to content

fix: branch resolution#25

Open
SionoiS wants to merge 1 commit into
cryptidtech:mainfrom
SionoiS:fix--branch-resolution
Open

fix: branch resolution#25
SionoiS wants to merge 1 commit into
cryptidtech:mainfrom
SionoiS:fix--branch-resolution

Conversation

@SionoiS

@SionoiS SionoiS commented Jul 10, 2026

Copy link
Copy Markdown

My AI found this. Let me know what ya think.

Root cause: Log::verify never set Comrade’s domain, so branch("pubkey") always resolved under "/".

Change in crates/provenance-log/src/log.rs: before unlock/lock, set domain from the entry ops context (longest common branch):

let domain = entry.context().to_string();
let mut unlocked = match Comrade::new(&kvp_lock, &entry)
    .with_domain(&domain)
    .try_unlock(unlock)

That matches the README’s Mike/Walker model: lock at /delegated/, ops under /delegated/mike/ → branch("pubkey") → /delegated/mike/pubkey. Unlock scripts still use absolute paths, so setting domain before unlock is safe.

@SionoiS SionoiS changed the title fix branch resolution fix: branch resolution Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant