Skip to content

docs(spec): remove prev_block_hash from PublicInputs#685

Merged
rkrasiuk merged 1 commit into
mainfrom
rkrasiuk/remove-prev-block-hash
Jul 17, 2026
Merged

docs(spec): remove prev_block_hash from PublicInputs#685
rkrasiuk merged 1 commit into
mainfrom
rkrasiuk/remove-prev-block-hash

Conversation

@rkrasiuk

Copy link
Copy Markdown
Member

Summary

Remove prev_block_hash from PublicInputs since it's derivable from prev_block_header. Avoids unnecessary extra validation of the input.

@rkrasiuk
rkrasiuk enabled auto-merge July 16, 2026 15:44
Comment thread specs/spec.md
The witness contains everything needed to re-execute the batch:

- **PublicInputs**: `zone_id`, `prev_block_hash`, `tempo_block_number`, `anchor_block_number`, `anchor_block_hash`, `expected_withdrawal_batch_index`, `sequencer`. These are the values the portal passes to the verifier and the proof must be consistent with.
- **PublicInputs**: `zone_id`, `tempo_block_number`, `anchor_block_number`, `anchor_block_hash`, `expected_withdrawal_batch_index`, `sequencer`. These are the values the portal passes to the verifier and the proof must be consistent with. `prevBlockHash` is instead derived from `prev_block_header` and bound through the public `block_transition` output.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Did we define BatchOutput somewhere? I can't find it right now

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@rkrasiuk
rkrasiuk added this pull request to the merge queue Jul 16, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 16, 2026
@rkrasiuk
rkrasiuk added this pull request to the merge queue Jul 17, 2026
Merged via the queue into main with commit 1ff0aa5 Jul 17, 2026
13 checks passed
@rkrasiuk
rkrasiuk deleted the rkrasiuk/remove-prev-block-hash branch July 17, 2026 12:32
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