Skip to content

Conversation

@apoelstra
Copy link
Contributor

This does not include documentation, tutorials or extended unit tests. I believe that this is not discoverable. But it is better that it exist than not, so I'm PR'ing it as-is.

In short:

  • adds a -s <state as 32-byte hex> flag to simplicity info which tweaks the generated addresses/scriptpubkeys accordingly
  • adds a -s <state as 32-byte hex> flag to simplicity pset update-input which creates a two-leaf taptree rather than a one-leaf one when computing control blocks

If you fail to provide these, in both cases the command will just silently produce output you didn't intend.

Copy link
Contributor Author

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On 7e16364 successfully ran local tests

@schoen
Copy link

schoen commented Dec 4, 2025

If you fail to provide these, in both cases the command will just silently produce output you didn't intend.

Does this mean that it actually breaks existing uses of simplicity pset update-input? Like my scripts that use it for transactions would no longer work?

@apoelstra
Copy link
Contributor Author

No, existing uses will continue to work. It's only if you're trying to use state, but do so incorrectly or incompletely, that the UX has some footguns.

I will file an issue with some suggestions.

@apoelstra apoelstra mentioned this pull request Dec 4, 2025
@apoelstra
Copy link
Contributor Author

#34

Copy link
Contributor

@delta1 delta1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 7e16364

Ran tests locally

@apoelstra apoelstra merged commit 01236a0 into BlockstreamResearch:master Dec 9, 2025
11 checks passed
@apoelstra apoelstra deleted the 2025-12/state branch December 9, 2025 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants