Skip to content

Comments

CLI: Update stake history#247

Open
aoikurokawa wants to merge 3 commits intomasterfrom
ak/update_stake_history
Open

CLI: Update stake history#247
aoikurokawa wants to merge 3 commits intomasterfrom
ak/update_stake_history

Conversation

@aoikurokawa
Copy link
Collaborator

No description provided.

Copy link

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 adds a new CLI command to update stake history for a specific vote account. The implementation introduces an "UpdateStakeHistory" action that calculates stake rank and superminority status for a given validator and submits an on-chain transaction to update the validator history.

  • Adds new CLI command UpdateStakeHistory to update stake history for a single vote account
  • Exposes get_stake_rank_map_and_superminority_count function as public API
  • Creates new actions module to organize action-based commands separately from cranks

Reviewed changes

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

Show a summary per file
File Description
utils/validator-history-cli/src/main.rs Imports and registers the new UpdateStakeHistory CLI command
utils/validator-history-cli/src/commands/mod.rs Adds public module declaration for actions
utils/validator-history-cli/src/commands/actions/mod.rs Module declaration for update_stake_history
utils/validator-history-cli/src/commands/actions/update_stake_history.rs Implements the core logic to update stake history for a specific vote account
keepers/stakenet-keeper/src/operations/stake_upload.rs Changes function visibility from private to public to enable reuse

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

@aoikurokawa aoikurokawa marked this pull request as ready for review December 17, 2025 03:32
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.

1 participant