Skip to content

Conversation

@OttoAllmendinger
Copy link
Contributor

This PR enhances the explainTransaction functionality across two packages:

  1. sdk-core: Adds ability to pass wallet instance to explainTransaction for
    additional context during transaction validation. This enables more accurate
    verification of staking transactions by providing wallet-specific
    information.

  2. abstract-utxo: Adds automatic keychain fetching in explainTransaction when
    a wallet is available but no keychains are explicitly provided.

These changes improve transaction validation by leveraging available wallet
context.

Issue: BTC-2963

OttoAllmendinger and others added 2 commits January 19, 2026 14:36
Allow passing wallet instance to explainTransaction for additional context
during transaction validation. This enables more accurate verification of
staking transactions by providing wallet-specific information.

Issue: BTC-2963

Co-authored-by: llm-git <llm-git@ttll.de>
Allow explainTransaction to automatically fetch keychains when none are
provided but a wallet is available.

Issue: BTC-2963

Co-authored-by: llm-git <llm-git@ttll.de>
@OttoAllmendinger OttoAllmendinger marked this pull request as ready for review January 20, 2026 09:52
@OttoAllmendinger OttoAllmendinger requested review from a team as code owners January 20, 2026 09:52
@OttoAllmendinger
Copy link
Contributor Author

note: audit failure is unrelated to this PR

@ivop624
Copy link

ivop624 commented Jan 20, 2026

note: audit failure is unrelated to this PR

Is this being handled separately?

Copy link
Contributor

@zhongxishen zhongxishen left a comment

Choose a reason for hiding this comment

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

lgtm

@OttoAllmendinger OttoAllmendinger merged commit a52ac2e into master Jan 20, 2026
29 of 32 checks passed
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.

6 participants