Skip to content

feat: add multisig transaction signature audit - #517

Open
cc-unchained wants to merge 2 commits into
caravan-bitcoin:mainfrom
cc-unchained:feat/signature-audit
Open

feat: add multisig transaction signature audit#517
cc-unchained wants to merge 2 commits into
caravan-bitcoin:mainfrom
cc-unchained:feat/signature-audit

Conversation

@cc-unchained

Copy link
Copy Markdown
Contributor

What kind of change does this PR introduce?

Adds a signature audit tool under Wallet → Transactions that determines which quorum key produced each
signature in a finalized multisig transaction.

The tool:

  • Accepts either a transaction ID or raw transaction hex
  • Supports P2SH, P2SH-P2WSH, and P2WSH multisig inputs
  • Cryptographically verifies signatures against the public keys in each input’s redeem or witness script
  • Uses the loaded wallet configuration to show the signer name, root fingerprint, derivation path, public
    key, and signature
  • Fetches previous-output amounts through the configured blockchain client when required for SegWit
    verification

Issue Number:

Fixes #

Snapshots/Videos:

image

If relevant, did you update the documentation?

Summary

Does this PR introduce a breaking change?

Checklist

  • I have tested my changes thoroughly.
  • I have added or updated tests to cover my changes (if applicable).
  • I have verified that test coverage meets or exceeds 95% (if applicable).
  • I have run the test suite locally, and all tests pass.
  • I have written tests for all new changes/features
  • I have followed the project's coding style and conventions.
  • I have created a changeset to document my changes (npm run changeset)

Other information

Have you read the contributing guide?

For information on creating and using changesets, please refer to our documentation on changesets.

@changeset-bot

changeset-bot Bot commented Jul 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 644d487

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

This PR includes changesets to release 2 packages
Name Type
@caravan/transactions Minor
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

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
caravan-coordinator Ready Ready Preview, Comment Jul 15, 2026 8:46pm

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

This pull request has been inactive for 30 days and has been marked as stale. It will be closed in 7 days if no further activity occurs. To keep this PR open, add the "long-lived" label or comment on it.

@github-actions github-actions Bot added the stale label Aug 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant