Skip to content

fix: Prove block that consumes an L1toL2 msg added on the same L1 block#16375

Merged
spalladino merged 1 commit intonextfrom
palla/l1-to-l2-message-proving-v2
Sep 3, 2025
Merged

fix: Prove block that consumes an L1toL2 msg added on the same L1 block#16375
spalladino merged 1 commit intonextfrom
palla/l1-to-l2-message-proving-v2

Conversation

@spalladino
Copy link
Copy Markdown
Contributor

@spalladino spalladino commented Aug 14, 2025

Fixes issue where proving fails for a tx that consumes an l1 to l2 message that was bridged on the same block as the tx is mined. The prover was failing to add the messages to the tree before running tx public function simulation, so txs didnt have access to the messages from that block, so their execution reverted, yielding a different end state root.

See also #16704

Fixes #16675

Co-authored-by: iLeila Wang <leizciw@gmail.com>
@spalladino spalladino force-pushed the palla/l1-to-l2-message-proving-v2 branch from d903029 to 594f977 Compare September 2, 2025 12:32
Copy link
Copy Markdown
Contributor

@LeilaWang LeilaWang left a comment

Choose a reason for hiding this comment

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

🚀

@spalladino spalladino added this pull request to the merge queue Sep 3, 2025
Merged via the queue into next with commit 92ea203 Sep 3, 2025
15 checks passed
@spalladino spalladino deleted the palla/l1-to-l2-message-proving-v2 branch September 3, 2025 20:37
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.

Insert L1 to L2 messages in the tree before public execution in public rollup circuits

2 participants