Concerns: This does result in a breaking change for anything that requires bclient/lib/wallet. Currently the majority of cases I looked into simply require bclient, which I have intentionally left unchanged.
If this is merged and released, it should be made a major version bump to prevent breakage with any downstream packages that do a full require of bclient/lib/wallet.
This allows wallet plugins to extend existing Wallet client more easily, without duplicate code.
Example: https://github.com/bcoin-org/bmultisig-client
Concerns: This does result in a breaking change for anything that requires
bclient/lib/wallet
. Currently the majority of cases I looked into simply requirebclient
, which I have intentionally left unchanged.