Skip to content

feat(consensus): implement Tendermint weighted round-robin proposer selection #193

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
qj0r9j0vc2 wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat(consensus): implement Tendermint weighted round-robin proposer selection #193

qj0r9j0vc2 wants to merge 8 commits into from

Conversation

@qj0r9j0vc2
Copy link
Member

@qj0r9j0vc2 qj0r9j0vc2 commented Jan 30, 2026

Summary

  • Implement Tendermint's weighted round-robin proposer selection algorithm
  • Create ProposerSelector struct with priority tracking using parking_lot::RwLock
  • Integrate with CipherBftContext for deterministic proposer selection
  • Add epoch transition handling and priority overflow prevention

Test Plan

  • All existing consensus tests pass (63 tests)
  • New unit tests for proposer selection algorithm
  • Clippy clean, rustfmt applied

@qj0r9j0vc2 qj0r9j0vc2 self-assigned this Jan 30, 2026
@qj0r9j0vc2 qj0r9j0vc2 force-pushed the feat/weighted-proposer branch from d88e2af to 404d244 Compare January 30, 2026 10:25
qj0r9j0vc2 and others added 6 commits January 30, 2026 20:14
@qj0r9j0vc2
fix(consensus): correct compute_advances formula for cross-height Nil…
a2df901 
… transitions
@qj0r9j0vc2
feat(consensus): add configurable wait_for_cut timeout with 50ms default
58a9fcf
@qj0r9j0vc2
fix(consensus): use struct init syntax for HostConfig to satisfy clippy
e4b75ba
@qj0r9j0vc2
Merge branch 'main' of github.com:decipherhub/cipherbft into feat/wei…
ade5603 
…ghted-proposer
@qj0r9j0vc2 @claude
fix(consensus): compute proposer advances from genesis for determinism
c5616d6 
Nodes starting at different heights (e.g., after syncing) were computing
different numbers of advances to reach the same (height, round), causing
proposer selection disagreement and consensus stalls.

The fix ensures all nodes compute advances from a fixed genesis point
(height 1, round Nil) on their first select_proposer call, regardless
of their initial_height. This guarantees deterministic proposer selection
across all nodes in the network.

Also fixes a test in execution_bridge.rs that wasn't destructuring the
tuple returned by create_default_bridge().

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@qj0r9j0vc2 @claude
style: fix rustfmt formatting in proposer_selector tests
2f77c13 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@qj0r9j0vc2 qj0r9j0vc2

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@qj0r9j0vc2