W49 Lib Team Call

[nondiremanuel]

📆 Tue December 5, 2023 12:00 UTC
📍 Discord General Voice Channel

Please add comments to this issue to suggest issues and PRs to discuss.

General Discussion Items

• BDK 1.0 milestones and timeline: added an alpha.4 milestone to separate the work -> we will try with deadlines every two week
• BDK Review Club: no review scheduled
• Any review begs? 🙏
• Blockers? 🪨

Top BDK 1.0 PRs and issues

alpha.3

Recently merged:

• Add Wallet::list_output method bdk#1190
• chore: remove bdk dependency on log and dev dependency on env_logger bdk#1204
• esplora: fix incorrect gap limit check in blocking client bdk#1225

Ready to be merged:

• build(esplora): Add async-https-rustls flag to esplora client bdk#1179: useful for android binding, possibly to be included in the next bdk_esplora release

Need Review:

• ref(hwi): Move hwi out of bdk bdk#1161: not very high priority, but there are some comments that could need further discussion. If it takes longer, we can move it to alpha.4 -> needs comment by evan / more review
• Include the descriptor in keychain::Changeset bdk#1203: needs more discussion, but being pretty important we can leave it to alpha.3 -> discussion ongoing between daniela and evan
• Introduce block-by-block API to bdk::Wallet and add RPC wallet example bdk#1172: review ongoing, some changes were requested -> are all the review comments addressed? Needs a check before merging (and eventually some rebase)
• doc(bdk): Clarify the absolute_fee docs bdk#1158: needs review (it should be a quick one) -> it will be merged soon
• doc: Improve TxGraph & co docs bdk#1188: improvement on docs -> needs more review
• Use a universal lookahead value for KeychainTxOutIndex and have a reasonable default bdk#1229: new PR -> needs a final review before merging (look if comments have been all addressed)

alpha.4

Need Review:

• Migrate to bitcoin::FeeRate bdk#1216: some new commits
• Clean up clippy allows bdk#1186: no update
• Bump MSRV to 1.63.0 bdk#1183: needs to wait LDK's Bump MSRV to rustc 1.63.0 lightningdevkit/rust-lightning#2681 before merging, it will be moved back to draft. No update
• Get keychain for script bdk#1201: no update

In Progress

• Introduce new bdk_coin_select implementation bdk#1072: separate repo. Discussion is ongoing
• rename scan to scan_with_keychain bdk#1117: still not included in any other PR, we keep it open for the moment, moved to alpha.4
• Implement default index_tx method bdk#1176: seems to impact the API. No update
• The Wallet.is_mine() function should return K bdk#1042: WIP; code already implemented. For now using KeychainKind. No update
• fix: remove deprecated max_satisfaction_weight bdk#1115: No update. Still on hold, needs some comment from evan and/or Lloyd. The old method has some problems but it's not clear the impact of it. Degree of complexity (and consequently milestone) needs to be discussed
• feat: add bdk_sqlite crate implementing PersistBackend bdk#1128: No update. Another approach has to be tried
• Add wallet sync_request and full_scan_request functions bdk#1194: super issue on sync/scan. No update
• Remove TxGraph::missing_heights as ChangeSet::missing_heights_from should have replaced it bdk#1126: discussion ongoing. No update
• Move Balance out of keychain bdk#1175: discussion ongoing. No update
• chore: extract TestEnv into separate crate bdk#1171: no update

To Do:

• Upgrade to rust-bitcoin 0.31.0 bdk#1196: on the upgrade of rust-bitcoin. Unlocked after rust-miniscript release
• bdk_esplora redundantly stores height in its anchor bdk_wallet#119: new issue
• Rename electrum's sync/scan bdk#1154: should we close this since there is already Syncing/Rescanning super issue bdk#1195?
• Reintroduce a way verify a transaction bdk_wallet#127: it shouldn't be very hard and lower priority for alpha.4
• Rename is_coin_base to is_coinbase bdk#1191
• bdk + esplora update_local_chain logic might be off  bdk#1199: bug on esplora update local chain
• Do not include satisfaction in policy id calculation bdk_wallet#123
• refactor(bdk): use idiomatic naming for all getter methods bdk#1221
• [chain] Consider Merging IndexedTxGraph and TxGraph into one thing bdk_wallet#130
• misleading spendable balance bdk#797: API breaking
• Introduce bdk_core crate bdk#1155
• Remove ComputeSighash bdk#1038
• Calling example_electrum sync on an unconfirmed tx may result in the tx being stuck as unconfirmed eternally (according to the cli)) bdk#1125
• Wallet's persistence should have a fixed "header" section bdk_wallet#125
• Backwards compatibility of bdk::ChangeSet data structure bdk#1103: for now alpha.4, to be discussed -> needs discussion to understand if it's a good idea and how breaking it is going to be

###Milestone TBD

• Add absolute fee and fee rate sanity check in the Wallet API bdk_wallet#120
• WIP: add test for #1229 bdk#1230: tests for 1229 (in alpha.3)
• Clearly define and document stop_gap bdk#1227
• chore: fix typos bdk#1228: no description to this PR
• BIP 326 Anti Fee Sniping bdk_wallet#131: for now alpha.4, to be discussed -> needs to be analyzed and see if it breaks the API or not (if not we can defer it)
• Add sign_psbt_input bdk_wallet#122: in 1.1? -> same as 1134

Other updates for future releases

• fix: more typos bdk#1223: ready to merge and blocks ci: move to Nix bdk#1165
• ci: move to Nix bdk#1165: ci to Nix; possibly in the future also for the ffi repo. Put in beta, if it comes earlier we can include it in alpha
• wallet_esplora_async returns error 429 while scanning bdk#1120: discussion ongoing
• Implement utxo reservation bdk#913: brought up by DanGould after long standby
• new awesome-bdk list

New Projects and Support

Upcoming Events

• Africa Bitcoin Conference (Accra) Dec 1-3 2023

🌅 Future Call Notes

📜 Past Call Notes

[thunderbiscuit]

I'd like to pitch my new awesome-bdk list for addition to the bitcoindevkit org.