Skip to content

feat: surface what a GroupV2 conversation reports about itself - #219

Draft
osmaczko wants to merge 1 commit into
repro/199from
feat/surface-group-progress-events
Draft

feat: surface what a GroupV2 conversation reports about itself#219
osmaczko wants to merge 1 commit into
repro/199from
feat/surface-group-progress-events

Conversation

@osmaczko

Copy link
Copy Markdown
Contributor

de-mls narrates its own commit-and-recovery cycle, and libchat consumes three of the eleven events it emits and drops the rest. A group that has stopped moving therefore looks like one that is idle: the roster keeps reporting whatever it last committed, and the account of why sits in a log the application cannot reach.

Buffer the phase changes, the commit-round counts and the steps that did not go through the way causal-history observations are buffered, drain them after each drive of the core, and give each a client event. The count is the load-bearing one: a round that ends short of the candidates it expected is a round where members chose from different sets, which is how a group splits.


Note for the review, not for history: this is based on repro/199 (#216) rather than main, so the diff is the one commit. CI does not run here, because the workflow triggers only on pull requests whose base is main. Retarget once #216 merges.

de-mls narrates its own commit-and-recovery cycle, and libchat consumes three of the fourteen events it emits and drops the rest. A group that has stopped moving therefore looks like one that is idle: the roster keeps reporting whatever it last committed, and the account of why sits in a log the application cannot reach.

Buffer the phase changes, the commit-round counts and the steps that did not go through the way causal-history observations are buffered, drain them after each drive of the core, and give each a client event. The count is the load-bearing one: a round that ends short of the candidates it expected is a round where members chose from different sets, which is how a group splits.
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