Skip to content

Conversation

@qj0r9j0vc2
Copy link
Member

Summary

  • Pass Cut to Primary when consensus decides, enabling position synchronization
  • Add sync_positions_from_cut() to update last_seen_positions from decided CARs
  • Prevents PositionGap errors that caused CAR attestation failures at height 3+

Root Cause

When consensus decided on a Cut, only height was passed to Primary. Validators that missed CARs during collection had stale position tracking, causing subsequent CARs to be rejected with PositionGap errors.

Test plan

  • cargo check passes
  • All 66 tests pass
  • Devnet cluster test with 4 validators

@qj0r9j0vc2 qj0r9j0vc2 self-assigned this Jan 25, 2026
@qj0r9j0vc2 qj0r9j0vc2 merged commit 09ac2fd into main Jan 25, 2026
9 checks passed
@qj0r9j0vc2 qj0r9j0vc2 deleted the qj0r9j0vc2/providence-v1 branch January 25, 2026 12:46
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.

2 participants