Skip to content

remove outdated checks#969

Open
mikhailUshakoff wants to merge 3 commits into
masterfrom
refactor/operator
Open

remove outdated checks#969
mikhailUshakoff wants to merge 3 commits into
masterfrom
refactor/operator

Conversation

@mikhailUshakoff

Copy link
Copy Markdown
Member
  • Remove the check that PreconfRouter is configured in TaikoWrapper contract.
  • Remove reading HANDOVER_WINDOW from PreconfRouter contract.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Removes now-obsolete PreconfRouter-related checks from the pacaya operator flow and simplifies L1 execution layer implementations accordingly, aligning behavior with the updated contract/config expectations.

Changes:

  • Removes PreconfOperator trait APIs related to checking TaikoWrapper’s PreconfRouter configuration and reading handover window slots.
  • Simplifies Operator::get_status by removing the TaikoWrapper PreconfRouter check and per-epoch handover window slot reload logic.
  • Updates Shasta/RealTime execution layers and pacaya tests to match the streamlined PreconfOperator interface; bumps workspace version to 1.39.3.

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
shasta/src/l1/execution_layer.rs Drops removed PreconfOperator methods from Shasta execution layer implementation.
realtime/src/l1/execution_layer.rs Drops removed PreconfOperator methods from RealTime execution layer implementation.
pacaya/src/node/operator/tests.rs Updates mocks/tests to match the trimmed PreconfOperator interface and removed operator behaviors.
pacaya/src/node/operator/status.rs Removes duration fields tied to deleted checks.
pacaya/src/node/operator/mod.rs Removes PreconfRouter configuration check and handover window slot reload logic from get_status.
pacaya/src/l1/traits.rs Removes deprecated PreconfOperator trait methods.
Cargo.toml Bumps workspace package version to 1.39.3.
Cargo.lock Updates lockfile versions to 1.39.3.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread pacaya/src/node/operator/tests.rs
Comment thread pacaya/src/node/operator/tests.rs
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