Skip to content

Update bdk_chain to latest (b45ecb9b)#360

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

Update bdk_chain to latest (b45ecb9b)#360
ValuedMammal wants to merge 2 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 69.39891% with 56 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.01%. Comparing base (c422104) to head (7c6a87e).

Files with missing lines Patch % Lines
src/wallet/mod.rs 68.14% 36 Missing ⚠️
src/descriptor/error.rs 29.41% 12 Missing ⚠️
src/wallet/export.rs 63.63% 4 Missing ⚠️
src/descriptor/mod.rs 82.35% 3 Missing ⚠️
src/wallet/error.rs 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #360      +/-   ##
==========================================
- Coverage   79.17%   79.01%   -0.17%     
==========================================
  Files          24       24              
  Lines        5311     5298      -13     
  Branches      242      244       +2     
==========================================
- Hits         4205     4186      -19     
- Misses       1029     1035       +6     
  Partials       77       77              
Flag Coverage Δ
rust 79.01% <69.39%> (-0.17%) ⬇️

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
@ValuedMammal ValuedMammal force-pushed the deps/chain_0_24_0 branch 3 times, most recently from 045ac47 to 43a7ffb Compare February 19, 2026 22:21
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 `Wallet::balance` with re-defined trust predicate.
- Updated docs for `apply_update` to include an Errors section
  and a note about the canonical view.
- Changed `insert_txout`, `apply_evicted_txs` to internally use
  `apply_update` as a way to centralize the update logic when it
  comes to adding transaction data to the wallet.
- `apply_unconfirmed_txs` now calls `self.update_canonical_view`
  internally.
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