Update Wallet to 2.3.0 and other deps by tvpeter · Pull Request #248 · bitcoindevkit/bdk-cli

tvpeter · 2026-03-13T14:44:37Z

Description

This PR updates the Wallet api to v2.3.0 and other dependencies.

Fixes #243

Changelog notice

Update bdk_wallet to v2.3.0
Add WalletEvents to the project
Update bdk_bitcoind_rpc to v0.22.0
Update bdk_electrum to v0.23.2
Update bdk_kyoto to v0.15.4

Checklists

All Submissions:

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

tvpeter · 2026-03-13T14:56:29Z

@notmandatory, I’m wondering if we should update our signing approach since the current method is deprecated. If we choose to proceed with the update, we will need to add miniscript as a separate dependency, as the version exported from the Wallet (v2.3.0) is 12.3.1. We require at least version 13.0.0. Or should we postpone this update until a later version?

notmandatory · 2026-03-13T16:41:17Z

@notmandatory, I’m wondering if we should update our signing approach since the current method is deprecated. If we choose to proceed with the update, we will need to add miniscript as a separate dependency, as the version exported from the Wallet (v2.3.0) is 12.3.1. We require at least version 13.0.0. Or should we postpone this update until a later version?

Good question. I've created bitcoindevkit/bdk_wallet#405 to revert the deprecating of the signer mod. The new PSBT signer probably won't be available until bdk_wallet 3.1.

va-an · 2026-03-16T19:17:29Z

Related #243.

va-an

Blocked by #252.

src/handlers.rs

tvpeter · 2026-03-17T19:44:15Z

Blocked by #252.

Hi @va-an, could you please verify that you tagged the correct PR? Thank you!

va-an · 2026-03-17T20:03:14Z

Blocked by #252.

Hi @va-an, could you please verify that you tagged the correct PR? Thank you!

All correct. I've also reproduced error from PR's CI build in my local run just pre-push:

error[E0061]: this method takes 1 argument but 2 arguments were supplied
    --> src/payjoin/mod.rs:605:30
     |
 605 |   ...                   .check_payment(
     |                          ^^^^^^^^^^^^^
...
 618 | / ...                       |outpoint| {
 619 | | ...                           let utxo = self.wallet.get_utxo(outpoint);
 620 | | ...                           match utxo {
 621 | | ...                               Some(_) => Ok(false),
...    |
 624 | | ...                       }
     | |___________________________- unexpected argument #2 of type `{closure@src/payjoin/mod.rs:618:33: 618:43}`
     |
note: method defined here
    --> /home/vaan/.cargo/registry/src/index.crates.io-1949cf8c6b5b557f/payjoin-1.0.0-rc.2/src/core/receive/v2/mod.rs:1262:12
     |
1262 |     pub fn check_payment(
     |            ^^^^^^^^^^^^^
help: remove the extra argument
     |
 617 -                                 },
 618 -                                 |outpoint| {
 619 -                                     let utxo = self.wallet.get_utxo(outpoint);
 620 -                                     match utxo {
 621 -                                         Some(_) => Ok(false),
 622 -                                         None => Ok(true),
 623 -                                     }
     |

I've applied the fix from #252 to my local code to verify that it helps.

va-an · 2026-03-31T19:09:35Z

@tvpeter #252 is merged now, so this should be fine now.
Could you rebase on master? Other PRs will likely need to rebase on top of this once it's merged.

- Add wallet events to full_scan and sync subcommands

- update bdk_bitcoind_rpc to v0.22.0 - update bdk_electrum to v0.23.2 - update bdk_kyoto to v0.15.4

codecov · 2026-03-31T21:17:12Z

Codecov Report

❌ Patch coverage is 0% with 44 lines in your changes missing coverage. Please review.
✅ Project coverage is 10.97%. Comparing base (07fd32f) to head (4a41057).
⚠️ Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
src/utils.rs	0.00%	30 Missing ⚠️
src/handlers.rs	0.00%	14 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #248      +/-   ##
==========================================
- Coverage   11.13%   10.97%   -0.16%     
==========================================
  Files           8        8              
  Lines        2488     2524      +36     
==========================================
  Hits          277      277              
- Misses       2211     2247      +36

Flag	Coverage Δ
rust	`10.97% <0.00%> (-0.16%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

va-an · 2026-04-01T08:15:11Z

Thanks for the rebase, @tvpeter!

Unfortunately, CI is now failing because bdk_wallet 2.3.0 introduced a deprecation that causes warnings-as-errors to fail. The revert has already been merged but 2.4.0 hasn't been released yet.

Pinning back to 2.1.0 won't work either - this branch already uses apply_update_events which was introduced in 2.3.0, so downgrading breaks the build:

error[E0599]: no method named `apply_update_events` found for mutable reference `&mut Wallet`

So it looks like we're stuck waiting for 2.4.0 to be released. Until then, CI will fail on this PR (and on any PR that rebases after merging this one).
We could just keep this in mind during reviews and merges - the CI failure is a known issue, not a code problem.

va-an

utACK 4a41057

va-an · 2026-04-01T08:40:26Z

src/handlers.rs

+            let wallet_events = wallet
+                .apply_update_events(update)
+                .map_err(|e| Error::Generic(e.to_string()))?;
+            print_wallet_events(&wallet_events);
+            Ok(())


We could use tap for convenience:

Suggested change

let wallet_events = wallet

.apply_update_events(update)

.map_err(|e| Error::Generic(e.to_string()))?;

print_wallet_events(&wallet_events);

Ok(())

wallet

.apply_update_events(update)

.map_err(|e| Error::Generic(e.to_string()))?

.tap(print_wallet_events);

Ok(())

va-an · 2026-04-01T08:46:07Z

src/utils.rs

    Ok((wallet_opts, network))
 }
+#[cfg(any(feature = "electrum", feature = "esplora",))]
+pub fn print_wallet_events(events: &Vec<WalletEvent>) {


Suggested change

pub fn print_wallet_events(events: &Vec<WalletEvent>) {

pub fn print_wallet_events(events: &[WalletEvent]) {

tvpeter · 2026-04-01T09:32:58Z

Unfortunately, CI is now failing because bdk_wallet 2.3.0 introduced a deprecation that causes warnings-as-errors to fail.

Yes, I updated the warnings to be treated as errors in this commit 8d20cf2 because many Clippy warnings were being neglected and had to be merged into master. It's not coming from bdk_wallet

github-project-automation bot added this to BDK-CLI Mar 13, 2026

va-an suggested changes Mar 17, 2026

View reviewed changes

src/handlers.rs Outdated Show resolved Hide resolved

src/handlers.rs Outdated Show resolved Hide resolved

tvpeter added 2 commits March 31, 2026 21:32

chore: Update bdk_wallet to v2.3

b71dafc

- Add wallet events to full_scan and sync subcommands

chore: Update project dependencies

bfdabf7

- update bdk_bitcoind_rpc to v0.22.0 - update bdk_electrum to v0.23.2 - update bdk_kyoto to v0.15.4

tvpeter force-pushed the chore/update-wallet-to-2_3 branch from f7817a9 to bfdabf7 Compare March 31, 2026 20:43

chore(update-deps): Fix clippy warnings

4a41057

va-an approved these changes Apr 1, 2026

View reviewed changes

	pub fn print_wallet_events(events: &Vec<WalletEvent>) {
	pub fn print_wallet_events(events: &[WalletEvent]) {

Conversation

tvpeter commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Changelog notice

Checklists

All Submissions:

Uh oh!

tvpeter commented Mar 13, 2026

Uh oh!

notmandatory commented Mar 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

va-an commented Mar 16, 2026

Uh oh!

va-an left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

tvpeter commented Mar 17, 2026

Uh oh!

va-an commented Mar 17, 2026

Uh oh!

va-an commented Mar 31, 2026

Uh oh!

codecov bot commented Mar 31, 2026

Codecov Report

Uh oh!

va-an commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

va-an left a comment

Choose a reason for hiding this comment

Uh oh!

va-an Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

va-an Apr 1, 2026

Choose a reason for hiding this comment

Uh oh!

tvpeter commented Apr 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

tvpeter commented Mar 13, 2026 •

edited

Loading

notmandatory commented Mar 13, 2026 •

edited

Loading

va-an commented Apr 1, 2026 •

edited

Loading

tvpeter commented Apr 1, 2026 •

edited

Loading