Skip to content

RoB place holder#67

Merged
bianyuanop merged 13 commits intomainfrom
cert-fix
Feb 25, 2025
Merged

RoB place holder#67
bianyuanop merged 13 commits intomainfrom
cert-fix

Conversation

@bianyuanop
Copy link
Contributor

@bianyuanop bianyuanop commented Feb 20, 2025

📝 Summary

This PR is a complementary PR for #65, in case when getPayload call is received but no RoB is accepted at the height for that rollup, we need to insert one place holder rob to chunk manager to allow chunk manager submit its correspond DA cert to SEQ for reference on derivation pipeline on opstack.

Other fixes & changes including:

  • initial ToBNonce is set to 1 to guartanee the correctness that each layer is only settled or preconfirmed after settling on SEQ or preconfirmed by SEQ.
  • an additional layer will be added after replaying the chunks, this guarantees that no previous settled chunks will be override.
  • minor refactors & unit tests fixes

Loading
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.

3 participants