doc: Use multi-path descriptors in descriptors.md and linked tests. #34086

issue BenWestgate openend this issue on December 17, 2025
  1. BenWestgate commented at 4:16 am on December 17, 2025: contributor

    Motivation

    A single multipath descriptor is the most convenient pattern for multisig; our documentation should use it.

    These files should use multipath descriptors as per https://github.com/bitcoin/bitcoin/commit/2a46e94a1600a4f28e01db23a89f039acaa2c45e and #33286:

    Locations:

    • doc/descriptors.md:
      • “Basic Multisig Example”
      • “Examples” with this text:

        For brevity, the internal “change” descriptor accompanying the above external “receiving” descriptor is not included here, but it typically differs only in the xpub derivation steps, ending in /1/* for change addresses.

    • wallet_miniscript_decaying_multisig_descriptor_psbt.py functional test

    Possible solution

    This only requires familiarity with multi-path descriptor semantics to solve. See Specifying receiving and change descriptors in one descriptor for how to update desciptors.md and #33286’s functional test diff for an approach to solve this issue in the functional test.

    Useful Skills

    • Understanding of multi-path descriptor semantics
    • Running the Python functional tests
    • Basic Python

    Guidance for new contributors

    Want to work on this issue?

    For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

  2. BenWestgate added the label good first issue on Dec 17, 2025
  3. ritoban23 commented at 1:59 pm on December 17, 2025: none
    @BenWestgate Hi I would like to work on this! will submit a PR soon
  4. anuragchvn-blip referenced this in commit 912d837326 on Dec 18, 2025
  5. anuragchvn-blip referenced this in commit dddf2cbdf5 on Dec 19, 2025
  6. anuragchvn-blip referenced this in commit 552bc82b17 on Dec 19, 2025
  7. ShirshenduR commented at 10:29 pm on December 30, 2025: none
    @fanquake Is this issue solved or still available?
  8. BenWestgate commented at 7:47 am on January 1, 2026: contributor
    @ShirshenduR the solution in #34100 could use additional review.
  9. Heiwabit commented at 1:45 pm on January 28, 2026: none
    I added the word ‘Bitcoin’ in the White Paper of the blockchain I created (https://github.com/Heiwabit/BriXNetworkWhitepaper.git ) to show it’s based on Bitcoin’s libraries. The full system is ready to lunch and it will launch soon. Cheers
  10. achow101 closed this on Jan 31, 2026

  11. achow101 referenced this in commit 88f8029835 on Jan 31, 2026
  12. BenWestgate commented at 0:40 am on February 2, 2026: contributor

    I don’t think those descriptor checksums were fake.

    I used them as test vectors and they all passed.

    Grabbing the checksum of a multi-path descriptor is in a different field than for canonical form but they do have a checksum.

    And it should be used for backups.

    If you agree with this nit, I’ll open a PR to append the multi-path descriptor’s checksum in descriptors.md


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-02-04 06:13 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me