[Client] Add --addressType to wallet creation; Squish all the bugs! :bug:

[kajoseph]

Currently, bitcore-client wallets are pubkeyhash wallets only. This adds the ability to create wallets using other address types.

Note, taproot addresses are not fully implemented in bitcore-lib yet and don't work (hence the bypass in the tests)

Also, this fixes several bugs:

• bitcore-lib-cash and bitcore-lib-doge disregarded the specified address type when computing an address from a public key
• wallet-send was all kinds of messed up due to the fee calculation.
  • There was a fallback of "20" which didn't make any sense so I added a --feeRate optional param.
  • The selectCoins method required a fee param, but none was given, so it produced a NaN, causing the tx to not have any inputs. So, I added a || 0 in the reduce.
  • The feeRate in wallet-send did a multiplication by scale, but the feerate value returned from bitcore-node was in a different scale for EVM and UTXO chains (i.e. EVM returned Wei, UTXO returned BTC). In both cases, this resulted in the tx having an exorbitant feeRate. I fixed this to calculate sats/KB for UTXO and Wei for EVM, considering if the user specified the --feeRate param or not.
• The change was also somewhat messed up in that there was no way to not have a change output. However, the wallet.deriveAddress fallback when constructing a tx does NOT save the address anywhere. Thus, the wallet would essentially lose those funds since it would have no awareness of the generated change address.
  • This PR removes the fallback which properly puts the onus of keeping track of addresses back on the upstream logic, and adds change generation logic to `Wallet.newTx
• wallet storage --import was broken because it would detect --name as a function, so the imported wallet would have a name of undefined, but in order to use the wallet you needed to pass in --name "function(str) { if (arguments.length === 0) return this._name; this._name = str; return this; }" :grimacing:
• Parameter validation was broken. I upgraded commander, removed program.ts (which unsuccessfully tried to do some param validation), and changed the param formatting to leverage commander's built-in param validation.

[kajoseph]

@escottalexander phew! pipeline is passing again. NPM really didn't like me manually adding those dev dependencies.

[escottalexander]

@kajoseph Still need to address this: https://github.com/bitpay/bitcore/pull/3638#discussion_r1346338177 I think it got lost when you pushed changes to fix the pipeline.