Skip to content

[WIP] Update comment for P2TR support in PSBT - #6

Draft
PandaKitten96 with Codex wants to merge 2 commits into
masterfrom
codex/update-p2tr-comment
Draft

[WIP] Update comment for P2TR support in PSBT#6
PandaKitten96 with Codex wants to merge 2 commits into
masterfrom
codex/update-p2tr-comment

Conversation

@Codex

@Codex Codex AI commented Jun 22, 2026

Copy link
Copy Markdown

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>This comment says “P2TR aren't supported yet”, but PSBT parsing/building currently has a P2TR branch (key-path) in UtxoPsbt and the tx builder supports P2TR key-path spending. Please update the comment to reflect the actual limitation (e.g., P2WSH is unsupported; P2TR script-path is unsupported) so the test doesn’t mislead future readers.</issue_title>
<issue_description>This comment says “P2TR aren't supported yet”, but PSBT parsing/building currently has a P2TR branch (key-path) in UtxoPsbt and the tx builder supports P2TR key-path spending. Please update the comment to reflect the actual limitation (e.g., P2WSH is unsupported; P2TR script-path is unsupported) so the test doesn’t mislead future readers.

    // because `witness_utxo` must contain a supported witness UTXO type (currently P2WPKH in this test path). Please note that `P2WSH` and `P2TR` script-path are not supported yet.

Originally posted by @copilot-pull-request-reviewer in trustwallet#4749 (comment)</issue_description>

Comments on the Issue (you are @codex[agent] in this section)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants