Skip to content

Comments

Update bdk_chain to latest (b45ecb9b)#360

Draft
ValuedMammal wants to merge 3 commits intomasterfrom
deps/chain_0_24_0
Draft

Update bdk_chain to latest (b45ecb9b)#360
ValuedMammal wants to merge 3 commits intomasterfrom
deps/chain_0_24_0

Conversation

@ValuedMammal
Copy link
Collaborator

@ValuedMammal ValuedMammal commented Dec 28, 2025

Description

Currently in draft, the purpose of this PR is to test the wallet crate using the latest version of bdk. This branch can be updated to depend on version 0.24.0 of bdk_chain when that is released.

Notes to the reviewers

Changelog notice

Checklists

All Submissions:

New Features:

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

Bugfixes:

  • This pull request breaks the existing API
  • I've added tests to reproduce the issue which are now passing
  • I'm linking the issue being fixed by this PR

@codecov
Copy link

codecov bot commented Dec 28, 2025

Codecov Report

❌ Patch coverage is 80.54299% with 43 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.98%. Comparing base (bb9b07f) to head (43a7ffb).

Files with missing lines Patch % Lines
src/wallet/mod.rs 79.83% 25 Missing ⚠️
src/descriptor/error.rs 29.41% 12 Missing ⚠️
src/wallet/export.rs 63.63% 4 Missing ⚠️
src/descriptor/mod.rs 94.44% 1 Missing ⚠️
src/wallet/error.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
- Coverage   86.99%   86.98%   -0.02%     
==========================================
  Files          25       25              
  Lines        8860     8853       -7     
==========================================
- Hits         7708     7701       -7     
  Misses       1152     1152              
Flag Coverage Δ
rust 86.98% <80.54%> (-0.02%) ⬇️

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.

@ValuedMammal ValuedMammal added the api A breaking API change label Dec 28, 2025
@ValuedMammal ValuedMammal self-assigned this Dec 28, 2025
@ValuedMammal ValuedMammal moved this to In Progress in BDK Wallet Dec 28, 2025
@ValuedMammal ValuedMammal added this to the Wallet 3.0.0 milestone Dec 28, 2025
@ValuedMammal ValuedMammal changed the title [DRAFT] Update bdk_chain to latest Update bdk_chain to latest (b4c8c6a1) Feb 9, 2026
@ValuedMammal ValuedMammal force-pushed the deps/chain_0_24_0 branch 2 times, most recently from feb8814 to d469e63 Compare February 12, 2026 21:00
@ValuedMammal ValuedMammal changed the title Update bdk_chain to latest (b4c8c6a1) Update bdk_chain to latest (b45ecb9b) Feb 12, 2026
Update dependencies:

chain
core
bitcoind_rpc -> 0.22.0
electrum
esplora
file_store -> 0.22.0

Changes:

Update usage of `LocalChain` and `CheckPoint` which are now generic.

Change structure of `WalletTx` (alias for `CanonicalTx`).

Introduce `Wallet::canonical_view`.

Update `balance` with re-defined trust predicate.

Fixup tests and documentation examples.

Currently *not all* `Wallet` methods that modify the transaction graph
will update the `canonical_view`, though they arguably should, e.g.
`apply_unconfirmed_txs`.

Docs for `apply_update` are updated to include an Errors section
and a note about the canonical view.
@ValuedMammal ValuedMammal force-pushed the deps/chain_0_24_0 branch 2 times, most recently from b80a98d to 045ac47 Compare February 19, 2026 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api A breaking API change

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant