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

fix(supervisor/core): unsafe block processing#2357

Merged
dhyaniarun1993 merged 10 commits intomainfrom
sa/fix/log-indexing
Jul 9, 2025
Merged

fix(supervisor/core): unsafe block processing#2357
dhyaniarun1993 merged 10 commits intomainfrom
sa/fix/log-indexing

Conversation

@sadiq1971
Copy link
Collaborator

Closes #2347

@sadiq1971 sadiq1971 marked this pull request as draft July 7, 2025 14:58
@codecov
Copy link

codecov bot commented Jul 7, 2025

Codecov Report

Attention: Patch coverage is 94.61078% with 9 lines in your changes missing coverage. Please review.

Project coverage is 82.2%. Comparing base (38a79d2) to head (0673c10).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...rates/supervisor/core/src/chain_processor/chain.rs 70.0% 3 Missing ⚠️
crates/supervisor/core/src/logindexer/indexer.rs 97.9% 3 Missing ⚠️
crates/supervisor/core/src/syncnode/node.rs 0.0% 3 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.

@sadiq1971 sadiq1971 marked this pull request as ready for review July 7, 2025 22:55
Copy link
Collaborator

@dhyaniarun1993 dhyaniarun1993 left a comment

Choose a reason for hiding this comment

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

lgtm.
Might need to refactor code later.

@dhyaniarun1993 dhyaniarun1993 added this pull request to the merge queue Jul 9, 2025
Merged via the queue into main with commit 42839d1 Jul 9, 2025
25 checks passed
@dhyaniarun1993 dhyaniarun1993 deleted the sa/fix/log-indexing branch July 9, 2025 06:45
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
Closes op-rs/kona#2347

---------

Co-authored-by: Arun Dhyani <dhyaniarun7@gmail.com>
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
Closes #2347

---------

Co-authored-by: Arun Dhyani <dhyaniarun7@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(supervisor/core): handle unsafe block indexing

2 participants