Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

Migrate transaction building from `TxBuilder` to `Wallet::create_psbt`

Đang mở
  • Có gì không ổn?
#338 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
48/100
Loại issue
Tái cấu trúc
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust
Lĩnh vực
cli

Hướng nghiên cứu

Read the five listed call sites in src/handlers/offline.rs, src/handlers/dns/mod.rs, and src/handlers/payjoin/mod.rs, then review the bdk_wallet 3.2.0 Wallet::create_psbt and replace_by_fee APIs. Run just pre-push to identify signing, deprecation, and feature-configuration failures; done means all transaction paths use the new APIs and the checks pass.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

The plan is to transition bdk_wallet to depreciate TxBuilder and use bdk_tx via Wallet::create_psbt (see https://github.com/bitcoindevkit/bdk_wallet/pull/516). So I'd suggest adding the check to bdk-tx and updating bdk-cli to enable the Wallet::create_psbt experimental feature and using it. But that all will need to be done in new PRs.

Originally posted by @notmandatory in https://github.com/bitcoindevkit/bdk-cli/issues/326#issuecomment-5799371337

This issue is about moving bdk-cli to Wallet::create_psbt from bdk_wallet 3.2.0. The tx size check discussed in the same thread is out of scope here.

bdk-cli is on bdk_wallet 3.1.0 and builds every transaction with TxBuilder. Wallet::create_psbt is available in 3.2.0, behind the bdk-tx feature and the --cfg bdk_wallet_unstable rustflag.

Call sites to migrate:

  • CreateTxCommand in src/handlers/offline.rs
  • CreateSpTxCommand in src/handlers/offline.rs
  • BumpFeeCommand in src/handlers/offline.rs, the only one using build_fee_bump
  • CreateDnsTxCommand in src/handlers/dns/mod.rs
  • PayjoinManager::send_payjoin in src/handlers/payjoin/mod.rs

Notes:

  • Wallet::replace_by_fee is the counterpart for bump_fee.
  • The bump also hits deprecations outside tx building, which -D warnings in CI turns into blockers: Wallet::sign, Wallet::get_signers, Wallet::policies, TxBuilder::policy_path and LoadParams::extract_keys.
  • The bump alone already breaks just pre-push: every test that signs a PSBT fails with Signer error: Missing non-witness UTXO, to be sorted out as part of this work.
Ngôn ngữ chính
Rust
Star
142
Fork
99
Merge trung bình
4 ngày 1 giờ
Pull request đã merge (30 ngày)
3

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của bitcoindevkit/bdk-cli

Tất cả issue của bitcoindevkit/bdk-cli

Issue tương tự

Thêm issue về Rust

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.