caravan-bitcoin / caravan

Caravan monorepo
https://caravanmultisig.com
MIT License
24 stars 19 forks source link

upgrade more legacy psbt functions for s2tr #104

Closed bucko13 closed 3 weeks ago

bucko13 commented 4 weeks ago

a couple other functions that needed to be using bitcoinjs-lib v6 for taproot output support. previously this would cause an error in trezor signing if the interaction had a psbt and key details, then the old translatePSBT would result in an output without an address.

In caravan coordinator there should be no change in functionality (i added an upgrade so that it would deploy and reflect the other s2tr changes in the prior PR on this subject), but it can be verified by changing the DirectSignatureImporter to pass a PSBT and keyDetails to the signing interaction. This will cause the TrezorSignMultisigTransaction to hit the code path that calls translatePSBT. With the old translatePSBT from @caravan/bitcoin, trezor-connect throws an error.

changeset-bot[bot] commented 4 weeks ago

🦋 Changeset detected

Latest commit: 56ebc1572e677bb38a98998d85384bba44efd6df

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages | Name | Type | | ------------------- | ----- | | @caravan/psbt | Minor | | @caravan/wallets | Patch | | caravan-coordinator | Minor |

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

vercel[bot] commented 4 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
caravan-coordinator ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 13, 2024 2:25am