Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

[DRAFT] feat(node/engine): add reset event#1771

Closed
theochap wants to merge 3 commits intomainfrom
theo/reset-event
Closed

[DRAFT] feat(node/engine): add reset event#1771
theochap wants to merge 3 commits intomainfrom
theo/reset-event

Conversation

@theochap
Copy link
Member

No description provided.

// If the local safe head is not set, set it to the safe head.
self.state.set_local_safe_head(self.state.safe_head);

self.state.forkchoice_update_needed = true;
Copy link
Contributor

@refcell refcell May 14, 2025

Choose a reason for hiding this comment

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

This shouldn't be needed - setting the safe head via the set_safe_head method above will also set the forkchoice_update_needed = true

@clabby
Copy link
Contributor

clabby commented May 16, 2025

closing in favor of #1789, which extends this.

@clabby clabby closed this May 16, 2025
github-merge-queue bot pushed a commit that referenced this pull request May 16, 2025
## Overview

Builds on #1771, adding support for propagating the engine reset back to
the derivation pipeline.

Co-authored-by: theo <80177219+theochap@users.noreply.github.com>
@theochap theochap deleted the theo/reset-event branch June 13, 2025 15:32
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
## Overview

Builds on op-rs/kona#1771, adding support for propagating the engine reset back to
the derivation pipeline.

Co-authored-by: theo <80177219+theochap@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants