Get rid of the `OnchainWallet` and `BoardingWallet` traits

Life will be simpler if we just use `KeyProvider` for everything related to signing. All the other features provided by these traits are not actually useful or used in a meaningful way in production code.