Skip to content
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.

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

chore: Decrement storage buffer, add EigenDA delay proof entrypoint, & refactor EigenDA tests #43

Conversation

ethenotethan
Copy link
Collaborator

@ethenotethan ethenotethan commented Jan 17, 2025

  • Added delay proof entry for eigenda
  • Updated SequencerInbox to validate unhappy case
  • Added a TODO describing that we'll need to test the delay proof via extending the existing hardhat tests
  • Decremented storage buffer size by 2. The new delayBuffer type occupies 64 bytes or two slots. moving the __gap array from slot 12 to 14. See historical mapping here for reference.

…& refactor EigenDA tests - disable failing test
…& refactor EigenDA tests - update storage layout for sequencer inbox
…& refactor EigenDA tests - update gap__ to ensure rollup manager still lives at slot 50
…& refactor EigenDA tests - update function sigs
@ethenotethan ethenotethan requested review from litt3 and 0x0aa0 January 21, 2025 12:56
Copy link
Collaborator

@0x0aa0 0x0aa0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rollup manager back in correct slot, lgtm

@ethenotethan ethenotethan merged commit 9d02c43 into eigenda Jan 21, 2025
5 checks passed
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