Abstract Repair Service Actions

[bw-solana]

Problem

fn run in RepairService is over 200 lines and super hard to understand the first time. It would be a little easier to understand things by abstracting away the large chunks of actions that are happening.

Given we have plans to improve the repair algorithm, making it easier for more people to understand seems worthwhile.

Summary of Changes

This list maps to the commits to help make it easier to digest

1. Add repair iteration function to abstract everything happening inside the run loop (except reporting stats/sleeping)
2. cleanup a few things and un-nest the stuff that was under let repair = {. Not sure why we ever needed that...
3. Sub-function for update weighting heuristic
4. Sub-function for handle popular pruned forks
5. Sub-function for build and send repair
6. Sub-function for identify repairs
7. Cleanup some places where we are using an intermediate var when it's not necessary