chore: Remove remote prover client crate

[sergerad]

Summary

Removes the miden-remote-prover-client crate and adds thin wrapper clients where necessary:

1. RemoteBlockProver — crates/block-producer/src/block_prover.rs

2. RemoteBatchProver — crates/block-producer/src/batch_builder/remote_prover.rs

3. RemoteTransactionProver — bin/ntx-builder/src/clients/prover.rs

4. RemoteTransactionProver — bin/benchmark/src/prover.rs

Closes #995.

Changelog

[[entry]]
scope       = "prover"
impact      = "removed"
description = "Removed the miden-remote-prover-client crate"

[kkovaacs]

Nit: now that the prover client is gone can we maybe remove the following line from rust-toolchain.toml?

targets    = ["wasm32-unknown-unknown"]

Other than that this looks good to me, thanks!

[SantiagoPittella]

Looks good. We need to coordinate with the client since we are switching to node's next branch and this will break it. cc @igamigo @juan518munoz

[igamigo]

Looks good. We need to coordinate with the client since we are switching to node's next branch and this will break it. cc @igamigo @juan518munoz

Yeah, was keeping this in mind (similarly in the faucet BTW). I think it will be simple to replace as the protobuf definitions are already shipped through the same crate as the others, so it's just generating a client in the same way we generate one for the RPC API. But to be clear, the version we are migrating to will not contain these changes yet.