Merge commit 'refs/pull/188/head' of github.com:bitcoindevkit/bdk

Switch to `miniscript` from `crates.io`

Update `tiny-bip39` to v0.8

Fixes #185

Remove trait bounds on Wallet struct

see: https://github.com/rust-lang/api-guidelines/issues/6

[wallet] Eagerly finalize inputs

If we know the final witness/scriptsig for an input we should add it
right away to the PSBT. Before, if we couldn't finalize any of them we
finalized none of them.

Fix to at least bitcoin ^0.25.2

And fix the fallout.

Merge commit 'refs/pull/172/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/184/head' of github.com:bitcoindevkit/bdk

Use dirs-next instead of dirs since the latter is unmantained

More consistent references with 'signers' variables

[wallet] Don't wrap SignersContainer arguments in Arc

Merge commit 'refs/pull/169/head' of github.com:bitcoindevkit/bdk

Add scheduled audit check in CI

[ci] Generate a different cache key for every job

pin cc version because last breaks rocksdb build

[wallet] Add witness and redeem scripts to PSBT outputs

Allow to set concurrency in Esplora config and optionally pass it in repl

Make esplora call in parallel

Merge commit 'refs/pull/167/head' of github.com:bitcoindevkit/bdk

Merge commit 'refs/pull/166/head' of github.com:bitcoindevkit/bdk

fix typo

[descriptor] Add support for `sortedmulti` in `descriptor!`

Switch to "mainline" rust-miniscript

Remove unused varaint HeaderParseFail

Require esplora feature for repl example

Bring less data around

eagerly unwrap height option, save one collect

use flatten instead of unwrap_or

avoid a max() call by checking minus or equal

Use filter_map instead of filter and map

use proper type for EsploraHeader, make conversion to BlockHeader infallible

conditionally remove cli args according to enabled feature

Use our Instant struct to be compatible with wasm

[examples] support esplora blockchain source in repl

[sync] Improve sync

Make every request in batch, to save round trip times
Fetch timestamp of blockheader to populate timestamp field in transaction
Remove listunspent requests because we can compute it from our history

[wallet] Build output lookup inside complete transaction

To avoid the caller having to do it.

[wallet] Add test that shwpkh populates witness_utxo

[wallet] Make coin_select return UTXOs instead of TxIns

- We want to keep the metadata in the UTXO around for things later
- It is easier to turn a UTXO into a TxIn outside

[ci] Remove actions-rs, cleanup names

[wallet] Use the branch-and-bound cs by default

Keep the `LargestFirst` coin selection for the tests, to make them more
predictable.

remove unneeded pub modifier

ignore .idea

[sync] check last derivation in cache to avoid recomputation

[keys] Add a shortcut to generate keys with the default options

[keys] impl `ToDescriptorKey` for `GeneratedKey`

[keys] impl `GeneratableKey` for `bitcoin::PrivateKey`

[cli] Split the internal and external policy paths

[wallet] Take both spending policies into account in create_tx

This allows specifying different "policy paths" for the internal and external
descriptors, and adds additional checks to make sure they are compatibile (i.e.
the timelocks are expressed in the same unit).

It's still suboptimal, since the `n_sequence`s are per-input and not per-transaction,
so it should be possibile to spend different inputs with different, otherwise
incompatible, `CSV` timelocks, but that requires a larger refactor that
can be done in a future patch.

This commit also tries to clarify how the "policy path" should be used by adding
a fairly detailed example to the docs.

[wallet] Replace `must_use` with `required` in coin selection

[wallet] Add tests for BranchAndBoundCoinSelection::single_random_draw

[wallet] Add tests for BranchAndBoundCoinSelection::bnb

[wallet] Add tests for BranchAndBoundCoinSelection::coin_select

[wallet] Add Branch and Bound coin selection

[wallet] Use TXIN_DEFAULT_WEIGHT constant in coin selection

Replace all the occurences of `serialize(&txin)`
with TXIN_DEFAULT_WEIGHT.

[wallet] Split `send_all` into `set_single_recipient` and `drain_wallet`

Previously `send_all` was particularly confusing, because when used on a
`create_tx` it implied two things:
- spend everything that's in the wallet (if no utxos are specified)
- don't create a change output

But when used on a `bump_fee` it only meant to not add a change output
and instead reduce the only existing output to increase the fee.

This has now been split into two separate options that should hopefully
make it more clear to use, as described in #142.

Additionally, `TxBuilder` now has a "context", that basically allows to
make some flags available only when they are actually meaningful, either
for `create_tx` or `bump_fee`.

Closes #142.

Merge commit 'refs/pull/156/head' of github.com:bitcoindevkit/bdk

Make Signer and AddressValidator Send and Sync

Remove redundant Box around signers

Remove redundant Box around address validators

[ci] test with all features enabled in single run

[ci] Add code coverage github actions workflow

[repl] add max_addresses param in sync

Merge commit 'refs/pull/146/head' of github.com:bitcoindevkit/bdk

[ci] Enable clippy for stable and tests by default

[ci] Fix all-keys and cli-utils tests

[ci] Enable clippy and test for optional features

[ci] Fix or ignore clippy warnings for all optional features except compact_filters

Rename get_must_may_use_utxos to preselect_utxos

Rename may_use_utxos to optional_uxtos

Merge commit 'refs/pull/115/head' of github.com:bitcoindevkit/bdk

[descriptor] Comment out incomplete ExtractPolicy trait tests

[descriptor] Fix compile errors after rebase

[descriptor] Add descriptor macro tests

[descriptor] Add get_checksum tests, cleanup tests

[descriptor] Add ExtractPolicy trait tests

Rename must_use_utxos to required_utxos

Rename DumbCS to LargestFirstCoinSelection

Incorporate RBF rules into utxo selection function

[wallet] Stop implicitly enforcing manaul selection by .add_utxo

This makes it possible to choose a UTXO manually without having to
choose them *all* manually. I introduced the `manually_selected_only`
option to enforce that only manually selected utxos can be used.

To stop the cli semantics changing I made the `utxos` keep the old
behaviour by calling `manually_selected_only`.

[wallet] Make 'unspendable' into a HashSet

to avoid awkwardly later on.

[wallet] Replace ChangeSpendPolicy::filter_utxos with a predicate

To make composing it with other filtering conditions easier.

Merging two match expressions for fee calculation

Implementing review suggestions from afilini

complying with clippy from the github CI

allow to definie static fees for transactions Fixes #137

[wallet] Default to SIGHASH_ALL if not specified

Closes #133

[wallet] Refactor `Wallet::bump_fee()`

[wallet] Improve `CoinSelectionAlgorithm`

Implement the improvements described in issue #121.

Closes #121, closes #131.

[wallet] Make coin_select take may/must use utxo lists

so that in the future you can add a UTXO that you *must* spend and let
the coin selection fill in the rest.

This partially addresses #121

Use collect to avoid iter unwrapping Options

[blockchain] Fix receiving a coinbase using Electrum/Esplora

Closes #107

[keys]: Re-export tiny-bip39

Closes #104

Bump rust-bitcoin to 0.25, fix Cargo dependencies

Closes #112, closes #113, closes #124

[ci] Remove travis.yml

[ci] Fix clippy warnings for 1.47.0

[ci] Use bitcoindevkit/electrs base image for electrum tests

[ci] Fix clippy warnings, enable clippy checks

[ci] Consolidate build, test, clippy jobs

Add a Discord badge to the README

[ci] add CI github actions