fix: load account and input notes advice maps into the advice provider before executing them

[greenhat]

Close #1452

This PR introduces the following changes:

• Load account and input notes advice maps into the advice provider before executing them;
• TransactionHost::new now expects &Account instead AccountHeader.

Testing

I tried to test these changes by pulling them through the miden-client (next branch, v0.10) and patching numerous deps along the way to try it in the test reproduced in #1452 but stumbled upon testnet node rejecting my request because my client is not v0.9. So, no luck.

Next, I looked into miden-testing to introduce a test there, but found that it'd take quite a lot of efforts to construct a proper test case (a note script, loading the data from the advice provider and calling an account that is also loads another data from the advice provider). OTOH, the code changes are quite straightforward, so I'd assume they should work. Please let me know if you want me to build one.

[bobbinth]

Looks good! Thank you! I left a couple of small comments inline.

[greenhat]

@bobbinth Done. Please do another round.

[bobbinth]

All looks good! Thank you!

[igamigo]

LGTM. I'll create an issue on the client for loading the advice maps accordingly for account codes .

[bobbinth]

I'll create an issue on the client for loading the advice maps accordingly for native and foreign account codes.

I'm not sure we need to do anything about this on the client side. In the future, foreign accounts would get processed correctly in the Host::get_mast_forest() method.