descriptor: Expose key expression parsing #35918

pull pseudoramdom wants to merge 3 commits into bitcoin:master from pseudoramdom:parse-key-expression-v2 changing 3 files +292 −78

pseudoramdom commented at 6:44 PM on August 6, 2026: contributor
This PR exposes a utility function to parse and validate a standalone descriptor key expression. This is particularly useful in GUI multisig setup flows where the UI accepts a pasted key expression. The GUI can use this parser to validate input, inspect the parsed key, and apply setup-specific policy i.e. reject private keys, uncompressed keys etc. (Multisig Wizard tracking issue: #35645)

PubkeyProvider is the existing internal representation of a parsed key expression. This PR exposes its interface in descriptor.h and adds a public ParsePubkey() overload that returns the providers produced by the existing parser.

Key changes
- public overload uses P2TR key rules, permitting compressed, x-only, extended, and MuSig key expressions while rejecting uncompressed keys.
- Add GetOriginInfo() for retrieving explicitly provided key origin information.
Also, while I'm here, I noticed that musig parser accepted trailing characters apart from /.
- Add a fix & test to reject unexpected trailing characters after a musig() expression

descriptor: Reject trailing text after musig()

The musig() parser split the input at the closing parenthesis without
validating the remaining text. This allowed unexpected characters after
the expression to be ignored.
Add a requirement for nonempty char after musig() to begin with '/'.

56b7147d05

descriptor: move PubkeyProvider to descriptor.h

Expose the abstract interface by moving to descriptor.h to allow other
modules to consume parsed key expressions.
Mostly a move-only commit. Implementation of operator< is retained in
descriptor.cpp

20d3ddc629

descriptor: Add public ParsePubKey overload

Add an overload for parsing standalone key expressions using P2TR key
rules.
Also, expose explicitly provided origin information through GetOriginInfo(),
allowing callers to inspect the fingerprint and derivation path without
re-parsing the expression.

b5eb6a83cf

DrahtBot added the label Descriptors on Aug 6, 2026
DrahtBot commented at 6:44 PM on August 6, 2026: contributor


The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35918.



Reviews

See the guideline and AI policy for information on the review process. A summary of reviews will appear here.



Conflicts

Reviewers, this pull request conflicts with the following ones:
- #35742 (descriptors: check duplicate keys in all multipath Miniscript branches by yashbhutwala)
- #35493 (wallet, descriptor: Fix MuSig private key completeness checks on importdescriptors by w0xlt)
- #34697 (descriptor: fix musig() duplicate key checks and doubled PSBT origin paths by shuv-amp)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
DrahtBot closed this on Aug 7, 2026
DrahtBot reopened this on Aug 7, 2026

Contributors

pseudoramdom

DrahtBot

Labels

Descriptors

Linked (view graph)

#35645 Multisig Wizard tracking issue

descriptor: Expose key expression parsing #35918

Code Coverage & Benchmarks

Reviews

Conflicts