Skip to content

CombinerGroup.establish: found its commits pathless - #115

Merged
germ-mark merged 1 commit into
mainfrom
mark/ger-2596-founding-paths
Sep 28, 2026
Merged

germ-mark merged 1 commit into
mainfrom
mark/ger-2596-founding-paths

Conversation

@germ-mark

@germ-mark germ-mark commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

CombinerGroup.establish creates a one-member group and commits the peer add in the immediately-following commit — there is no gap for a founding UpdatePath to cover, so both halves are now founded pathless. No parameter is added: the convenience constructor simply omits the path, and the RFC9420 committing(includePath:) seam is unchanged for callers wanting a path elsewhere.

Relative to the released 0.1.6 this changes behavior only (no signature change), so the changeset is patch.

Part of GER-2596.

Tests

  • establishOmitsBothFoundingPaths — both halves' parent nodes are blank (node count pinned), and the pair still joins and verifies from the Welcome.
  • Full swift-mls suite green. Mutation-verified: forcing the internal committing call back to includePath: true fails exactly the two blank-parent expectations and nothing else.

🤖 Generated with Claude Code

@changeset-bot

changeset-bot Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 6987d17

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@germ-network/swift-mls Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@germ-mark
germ-mark force-pushed the mark/ger-2596-founding-paths branch from 08621fd to 9dc57ad Compare September 28, 2026 08:41
@germ-mark germ-mark changed the title CombinerGroup.establish: add a pqHalfIncludesPath setting CombinerGroup.establish: add an includePath setting Sep 28, 2026
establish creates a one-member group and commits the peer add in the
immediately-following commit, so there is no gap for a founding
UpdatePath to cover. Both halves are now founded pathless. The RFC9420
committing(includePath:) seam is unchanged for callers wanting a path
elsewhere.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@germ-mark
germ-mark force-pushed the mark/ger-2596-founding-paths branch from 9dc57ad to 6987d17 Compare September 28, 2026 09:03
@germ-mark germ-mark changed the title CombinerGroup.establish: add an includePath setting CombinerGroup.establish: found its commits pathless Sep 28, 2026
@germ-mark
germ-mark merged commit 7f0a13e into main Sep 28, 2026
4 checks passed
@github-actions github-actions Bot mentioned this pull request Sep 28, 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