sync the nodes with sync_all

merged with D9742

merge D9762 and D9742

I think that should be merged in D9742 . it doesn't make much sense to separate the code and the tests.

address linter error

register the proof at tip update rather than init, like before. We just no longer to need to wait for IBD to finish. And we don't need a flag, as we can just call registerProof which returns immediateley if the proof exists.

• extension/src/components/App.js still has fbt imports and tags
• .gitignore still has an fbt related file section
• test plan doesn't test all changes

add the lock also when using the key in AVARESPONSE (fixes clang error)

move Mutex next to the data it is guarding, use lock in addavalanchenode (rebase on D9719), remove need for lock in processor.cpp by moving the buildRemoteSighash code to the AvaHello processing code (already locked) in net_processing.cpp (rebase on D9708).

Need to work on the error message and if possible figure out how to not duplicate all the stateless proof verification tests again (already done in Processor::MakeProcessor)

We have established that the failures I'm seeing are unrelated to this diff, as I can reproduce them on master.
It is most likely a port collision with some other daemon running on my machine.

Check the nodes are not removed with the peer in the test

New test produces error as expected, patch fixes it as expected.

Make removeNodeFromPeer valid when the peer no longer exist.
Add a test case for removeNode as well.

Reviewed and tested, with and without the patch.

I have no way to test