feat(service|btc)!: adapt org mempool updates in btc-assets-api

[ShookLyngs]

Changes

• Add BtcAssetsApi.getBtcRecommendedFeeRates() as the btc-assets-api service provides the API directly
• Add after_txid filter in BtcAssetsApi. getBtcTransactions() API to allow query starts after a btc txid
• Remove DataSource.getRecommendedFeeRates() and DataSource.getAverageFeeRate() API
• Set fastestFee as the default fee rate in TxBuilder (it was halfHourFee)
• Remove mempool.js from the service lib's dependencies
• Replace deprecated test info in the service tests
• Rename service tests to improve readability

Parent PR

Most of the updates in this pull request are based on the org mempool api updates in btc-assets-api:

• feat(bitcoin)!: use mempool.space REST API btc-assets-api#96

Related issues

• Resolves Re-enable btc/service tests in test.yaml #143
• Resolves Refactor the default feeRate of rgbpp-sdk/btc #92

TODOs

• Add tests of the after_txid filter
• Test the mempool api 503 error (see if still occurs)
• Re-enable btc/service tests once everything's normal again Re-enable btc/service tests in test.yaml #143

[ahonn]

Maybe we just need getRecommendedFeeRates, getAverageFeeRate looks like it just goes to one of the fields. Are there compatibility considerations here?

[ShookLyngs]

Yes, personally I would like to remove both DataSource.getRecommendedFeeRates and DataSource.getAverageFeeRate, but I didn't do it due to compatibility concerns. The PR #98 was stuck for the same reason. I think if JoyID is not using them directly, we can remove them with a BREAKING CHANGE mark in the PR.

Update: after discussion with ahonn and yuche, I think it's okay to remove these APIs with a BREAKING CHANGE mark. The reason for removing them: the BtcAssetsApi provides the fees API directly, and the DataSource does nothing but re-route it.