Skip to content

Expose additional signing options in wallet sign - #365

Open
emma31-dev wants to merge 4 commits into
bitcoindevkit:masterfrom
emma31-dev:feat/358-sign-options
Open

emma31-dev wants to merge 4 commits into
bitcoindevkit:masterfrom
emma31-dev:feat/358-sign-options

Conversation

@emma31-dev

Copy link
Copy Markdown

Description

Expose the remaining signing controls in wallet sign, including sighash handling, partial-signature removal, finalization, Taproot signing, and signature grinding. Closes #358 .

Notes to the reviewers

tap_leaves_options supports all and none, as discussed in the issue. Unspecified options retain their defaults.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature

Bugfixes:

  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

@emma31-dev
emma31-dev marked this pull request as draft October 21, 2026 17:30
@emma31-dev
emma31-dev force-pushed the feat/358-sign-options branch from 643b19f to 40ead92 Compare October 21, 2026 17:51
@emma31-dev
emma31-dev marked this pull request as ready for review October 21, 2026 17:57

This branch has not been deployed

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

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

Expose remaining SignOptions fields in the wallet sign command

1 participant