psbt: classify missing Taproot script-path signatures as signer #36039

pull btcpavao wants to merge 1 commits into bitcoin:master from btcpavao:fix-taproot-psbt-analyze-role changing 3 files +23 −3

btcpavao commented at 2:30 PM on August 20, 2026: none

This is my first Bitcoin Core PR.

I originally reported #36035 after running into this while experimenting with Taproot 2-of-3 multisig on Signet. After reporting it, I wanted to see whether I could actually trace the problem in the source code and try to fix it myself.

I used ChatGPT Pro as a tool to help me navigate the codebase, understand the PSBT analysis/signing flow, and test different hypotheses. I followed the changes myself, built Bitcoin Core locally, and ran the tests rather than just submitting generated code.

The issue appears to come from the Taproot script-path signing path not recording a missing signature in missing_sigs when a Schnorr signature is not available. AnalyzePSBT() uses missing_sigs to decide whether the next role should be signer. Without it, the PSBT was classified as updater, which is also why the GUI displayed “Transaction is missing some information about inputs.”

My first attempt at the fix did not work. The regression test still returned updater, which led me to notice that SignMuSig2() can return successfully without actually producing a Taproot script-path signature. I adjusted the change so that, after the Schnorr/MuSig2 attempts, a missing signature is recorded when no script-path signature was produced.

I also added a regression test that checks that this Taproot script-path case is classified as signer.

I tested the change locally with:

test/functional/rpc_psbt.py test/functional/wallet_musig.py test_bitcoin --run_test=psbt_tests test_bitcoin --run_test=miniscript_tests git diff --check

All of these passed with the current patch.

This appears to fix the behavior I reported in #36035. Since this is my first contribution, feedback on both the approach and the implementation is very welcome.

Fixes #36035
DrahtBot added the label PSBT on Aug 20, 2026
DrahtBot commented at 2:30 PM on August 20, 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/36039.



Reviews

See the guideline and AI policy for information on the review process. A summary of reviews will appear here.
psbt: classify missing Taproot script-path signatures as signer 641b52828a
in src/script/sign.cpp:398 in 775e2040d8
```
 392 | @@ -393,8 +393,13 @@ static bool CreateTaprootScriptSig(const BaseSignatureCreator& creator, Signatur
 393 |  
 394 |      if (creator.CreateSchnorrSig(provider, sig_out, pubkey, &leaf_hash, nullptr, sigversion)) {
 395 |          sigdata.taproot_script_sigs[lookup_key] = sig_out;
 396 | -    } else if (!SignMuSig2(creator, sigdata, provider, sig_out, pubkey, /*merkle_root=*/nullptr, &leaf_hash, sigversion)) {
 397 | -        return false;
 398 | +    } else {
 399 | +        if (!SignMuSig2(creator, sigdata, provider, sig_out, pubkey, /*merkle_root=*/nullptr, &leaf_hash, sigversion)) {
 400 | +            return false;
```
achow101 commented at 6:16 PM on August 20, 2026:

This results in the analysis still being incorrect for transactions involving a musig.

btcpavao commented at 7:32 PM on August 20, 2026:

Thanks, I reproduced this with a Taproot script-path MuSig2 case. My previous change was incorrectly treating the aggregate key as an ordinary missing Schnorr signature.

I updated the patch to distinguish when SignMuSig2() actually matched a MuSig2 aggregate, and only add missing_sigs for the non-MuSig case. I also added a regression check for the MuSig2 script-path case. wallet_musig.py, rpc_psbt.py, psbt_tests, and miniscript_tests pass locally.
btcpavao force-pushed on Aug 20, 2026

Contributors

Labels

Linked (view graph)

#36035 Taproot multisig PSBT shows “missing information about inputs” even though it can be signed

psbt: classify missing Taproot script-path signatures as signer #36039

Code Coverage & Benchmarks

Reviews