rust-bitcoin / rust-bitcoin/rust-miniscript
Support for Non Standard Miniscripts
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 426
- Forks
- 200
- Avg merge
- 7d 17h
- Merged PRs (30d)
- 8
Description
Miniscript language is designed to be standardness complete, however in certain applications like block explorer may require the ability to parse non-standard Miniscripts.
However, at this point, the current solution to change Miniscript ScriptContext from Generic Type to a parameter in every function seems overly invasive and is not worth the change as lose the strong type guarantees of Miniscript. (For example, using uncompressed keys in segwit context).
Considering the tradeoff that we maybe gaining very little compared by supporting non-standard miniscripts, and that there would be scripts on the blockchain that are not miniscripts which we would not be able to parse anyways, it is probably not worth it to support for those.
Marking this as brainstorming and a low priority issue for now. If there are additional usecases for converting Miniscript<MiniscriptKey, ScriptContext> to Miniscript<MiniscriptKey> we can consider implementing the changes.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No source file, test, or entry point is named. First study how ScriptContext is represented across Miniscript parsing and conversion, then determine whether a concrete use case justifies the proposed generic-context changes; done would require an agreed design and tests for non-standard Miniscript handling.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- blockchain, cryptography
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100