apollo_network: struct that holds the reason the peer is considered bad

[guy-starkware]

Note

Low Risk
Small, mostly type-level API change; risk is limited to potential downstream compile fixes where BadPeerReport.reason was previously treated as a String.

Overview
Introduces a typed BadPeerReason enum for BadPeerReport.reason (replacing the raw String), allowing callers to distinguish between conversion errors and duplicate-vote misconduct.

Removes the #[allow(dead_code)] attribute on consensus’s DuplicateVoteReport, making unused-report detection stricter as duplicate-vote reporting gets wired up.

^{Written by Cursor Bugbot for commit 826ed3e. This will update automatically on new commits. Configure here.}

[reviewable-StarkWare]

This change is 

[guy-starkware]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• apollo_network: pass the reBadPeerReport when reporting #12474
• apollo_network: struct that holds the reason the peer is considered bad #12470 👈 (View in Graphite)
• apollo_network: add BadPeerReport and PenaltyCard #12463
• apollo_consensus: check for duplicate votes before caching #12465
• apollo_consensus: add helper function to report peer #12464
• main-v0.14.1-committer

This stack of pull requests is managed by Graphite. Learn more about stacking.