feat: Add descriptor miniscript constructor method

[ItoroD]

Description

• Add new_tr, new_sh_with_wsh, and new_sh_with_wpkh

Notes to the reviewers

Documentation

• new_tr - https://docs.rs/miniscript/12.3.5/miniscript/descriptor/enum.Descriptor.html#method.new_tr

• new_sh_with_wsh - https://docs.rs/miniscript/12.3.5/miniscript/descriptor/enum.Descriptor.html#method.new_sh_with_wsh

• new_sh_with_wpkh - https://docs.rs/miniscript/12.3.5/miniscript/descriptor/enum.Descriptor.html#method.new_sh_with_wpkh

• bdk_wallet

• bitcoin

• uniffi

• Other:

Changelog

Checklists

All Submissions:

• I've signed all my commits
• I followed the contribution guidelines
• I ran cargo fmt and cargo clippy before committing
• I've added exactly one changelog:* label
• I've linked the relevant upstream docs or specs above

New Features:

• I've added tests for the new feature
• I've added docs for the new feature

[reez]

Looks great overall, 2 comments

[reez]

Can we split these into separate assertFailsWith blocks? I think this new newShWithWpkh case never executes because Descriptor.newBare(...) throws first in the same block.

[ItoroD]

woaw. Good catch!

[reez]

Should we remove this debug println before merge?

[reez]

ACK c1a34cd

Thanks for addressing both comments. Looks good to me pending the Android CI finishing green.

[thunderbiscuit]

Looks good! One little fix we need is that for the 3.1 release we're trying a new thing where the changelog is now not updated directly and instead built from the GitHub auto generated notes. Sorry I know we keep flipfloping on the changelog mess. Hopefully this one works well and we can simply do it like this in the future!

You can remove the changelog entry and rebase, and this is ready to go.