Skip to content

Add startup checks for process-meta-vault#642

Merged
evgeny-stakewise merged 2 commits intomasterfrom
meta-vault-startup-checks
Jan 27, 2026
Merged

Add startup checks for process-meta-vault#642
evgeny-stakewise merged 2 commits intomasterfrom
meta-vault-startup-checks

Conversation

@evgeny-stakewise
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings January 26, 2026 18:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the process_meta_vault setting from the global settings and introduces dedicated startup checks for the process-meta-vaults command. The change decouples meta vault processing from the general operator settings and makes the command self-contained with its own validation logic.

Changes:

  • Removed process_meta_vault boolean setting from the global Settings class
  • Added dedicated startup checks function in the process-meta-vaults command
  • Refactored graph node sync check to use execution client directly instead of requiring consensus client

Reviewed changes

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

File Description
src/config/settings.py Removed process_meta_vault field and parameter from Settings class
src/common/startup_check.py Removed meta vault check from general startup, made check_execution_nodes_network public, added _get_finalized_block_number helper
src/commands/process_meta_vaults.py Added dedicated startup_checks function with meta vault validation logic

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

@evgeny-stakewise evgeny-stakewise merged commit be874fb into master Jan 27, 2026
8 checks passed
@evgeny-stakewise evgeny-stakewise deleted the meta-vault-startup-checks branch January 27, 2026 08:17
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