feat: Expose miniscript has_wildcard and sanity_check

[ItoroD]

Description

Adding these based off of #687

Notes to the reviewers

Documentation

• https://docs.rs/miniscript/latest/miniscript/descriptor/enum.Descriptor.html#method.has_wildcard

• https://docs.rs/miniscript/latest/miniscript/descriptor/enum.Descriptor.html#method.sanity_check

• bdk_wallet

• bitcoin

• uniffi

• Other:

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 a changelog in the next release tracking issue (see example)
• 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

[ItoroD]

I think the sanity method check is great! I wonder what other kinds of things it could catch.

[reez]

nice addition. are you thinking of adding test coverage for has_wildcard too or are you thinking its not needed? just wondering is all, not a huge blocker or anything

[ItoroD]

nice addition. are you thinking of adding test coverage for has_wildcard too or are you thinking its not needed? just wondering is all, not a huge blocker or anything

Yes, I thought it wasn't worthy enough to have its own method 😁

I do not mind adding it.

[reez]

Yes, I thought it wasn't worthy enough to have its own method 😁

that's what I assumed just wanted to check 👍 all good and already approved regardless

[thunderbiscuit]

Tested ACK 16a6da8. @ItoroD can you add a changelog entry as well into the changelog file?

Also thanks for the little tests! Makes it easy to undersand the shape of the new APIs when you see them in action.

[thunderbiscuit]

Mmm I'm sending mixed messages 😆. I ACK'ed but also requested changes sorry. I'll merge once you add the changelog entry!

[ItoroD]

Tested ACK 16a6da8. @ItoroD can you add a changelog entry as well into the changelog file?

Will do.