Skip to content

Conversation

@guy-starkware
Copy link
Contributor

@guy-starkware guy-starkware commented Feb 10, 2026

Note

Low Risk
Pure refactor that only changes how peer-reporting is invoked on parse errors; no consensus logic or data flow changes expected.

Overview
Refactors consensus vote handling to centralize the network report_peer call into a new MultiHeightManager::report_peer helper.

On vote parse failures in handle_vote, the duplicated inline report_peer logic is replaced with the helper, keeping behavior the same while standardizing error logging when the report cannot be sent.

Written by Cursor Bugbot for commit 7c9225f. This will update automatically on new commits. Configure here.

@reviewable-StarkWare
Copy link

This change is Reviewable

@guy-starkware guy-starkware changed the base branch from guyn/network/report_peer_struct to graphite-base/12464 February 10, 2026 14:39
@guy-starkware guy-starkware force-pushed the guyn/consensus/helper_report_peer branch from da70d20 to 7c9225f Compare February 10, 2026 14:40
@guy-starkware guy-starkware changed the base branch from graphite-base/12464 to main-v0.14.1-committer February 10, 2026 14:40
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.

2 participants