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

feat(supervisor/core): l2 finalized head update#2253

Merged
dhyaniarun1993 merged 8 commits intomainfrom
feat/l2-update-finalized
Jun 25, 2025
Merged

feat(supervisor/core): l2 finalized head update#2253
dhyaniarun1993 merged 8 commits intomainfrom
feat/l2-update-finalized

Conversation

@dhyaniarun1993
Copy link
Collaborator

Closes #2213

@dhyaniarun1993 dhyaniarun1993 self-assigned this Jun 24, 2025
@dhyaniarun1993 dhyaniarun1993 added K-feature Kind: feature W-supervisor Workstream: supervisor labels Jun 24, 2025
@codecov
Copy link

codecov bot commented Jun 25, 2025

Codecov Report

Attention: Patch coverage is 83.46883% with 61 lines in your changes missing coverage. Please review.

Project coverage is 81.3%. Comparing base (3ea31f7) to head (36d28ba).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
crates/supervisor/storage/src/chaindb.rs 0.0% 20 Missing ⚠️
crates/supervisor/core/src/supervisor.rs 0.0% 18 Missing ⚠️
crates/supervisor/core/src/l1_watcher/watcher.rs 94.9% 8 Missing ⚠️
...rates/supervisor/core/src/chain_processor/chain.rs 75.0% 6 Missing ⚠️
crates/supervisor/core/src/syncnode/node.rs 0.0% 5 Missing ⚠️
crates/supervisor/core/src/chain_processor/task.rs 97.2% 4 Missing ⚠️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@emhane emhane left a comment

Choose a reason for hiding this comment

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

lgtm

@dhyaniarun1993 dhyaniarun1993 marked this pull request as ready for review June 25, 2025 07:06
Copy link
Collaborator

@itschaindev itschaindev left a comment

Choose a reason for hiding this comment

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

lgtm

*last_finalized_number = block_number;
}

fn broadcast_finalized_source_update(&self, finalized_source_block: BlockInfo) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice!!

@dhyaniarun1993 dhyaniarun1993 enabled auto-merge June 25, 2025 11:21
@dhyaniarun1993 dhyaniarun1993 added this pull request to the merge queue Jun 25, 2025
Merged via the queue into main with commit b16d7d1 Jun 25, 2025
25 checks passed
@dhyaniarun1993 dhyaniarun1993 deleted the feat/l2-update-finalized branch June 25, 2025 11:43
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

K-feature Kind: feature W-supervisor Workstream: supervisor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(supervisor/core): Implement missing Finalized updates to L2

3 participants