Ms.peer required

[mariano54]

These three methods in wallet_node: fetch_puzzle_solution, get_coin_state, and fetch_children, are all called during sync, by the different wallets (and WSM). If any of them fail, the whole sync will stop. This is a problem, especially if we are connected to a peer that is not synced to the chain (for example our local node).

This fixes it, by forcing the sync target peer to be passed in. This peer must be synced up to the chain, so they will respond with the correct data. If something goes wrong, sync stops, and it can resume again with another peer.

This seems like a big change but it's not. It's just passing in an additional peer argument everywhere.

[github-actions[bot]]

This pull request has conflicts, please resolve those before we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved. A maintainer will review the pull request shortly.

[mariano54]

This one should be merged to main after ms.wlt_node_unsynced @wallentx