W43 Lib Team Call

[nondiremanuel]

📆 Tues October 24, 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 (add roadmap for 2023-24 bitcoindevkit.org#162: Roadmap blog post) -> high level functional roadmap description
• BDK Review Club -> next Thursday about ci migration to Nix (ci: move to Nix bdk#1165)
• Any review begs? 🙏
• Blockers? 🪨 -> some builds are broken for some problems with the dependencies. Working on it today

Top BDK 1.0 PRs and issues

alpha.2 -> released 🎉:

alpha.3

Recently merged:

Ready to be merged:

Changes requested:

Need Review:

• rename scan to scan_with_keychain bdk#1117: waiting to be reviewed
• Further improve unconfirmed tx conflict resolution bdk#1109: Lloyd is having some concerns on the solution proposed -> need some comments on the concerns
• ref(hwi): Move hwi out of bdk bdk#1161: new to alpha.3 -> good candidate for nix ci as well
• ci: replace toolchain bdk#1156: new to alpha.3 on ci -> it will need to be closed if we land to Nix ci

In progress:

• Add CreateTxError and use as error type for TxBuilder::finish() bdk#1028: rebased, needs some fixes. Does this fix bdk::Wallet should have context-specific error types bdk#994? -> probably yes, but needs to be double checked. Question about the usage of a new crate (everyone ok with it?). Discussion will go on in the PR.
• LocalChain with hardwired genesis block bdk#1178: WIP (evan)
• WIP: persist descriptor in wallet bdk#1167: WIP (daniela)
• The Wallet.is_mine() function should return K bdk#1042: WIP (vlad)
• Introduce block-by-block API to bdk::Wallet and add RPC wallet example bdk#1172: WIP (vlad), replaces 1106.
• fix(electrum): fixed chain sync issue bdk#1145: needs to be continued after discarding fix(electrum): fixed chain sync issue bdk#1163 solution
• 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
• Refactor: Migrate bdk::FeeRate to bitcoin::FeeRate bdk#1141: No update. Important to keep it since it affects the Wallet API. Not clear if it is close to end or not
• Test electrum ext bdk#979: No update. Related to #1145, so included in alpha.3

To Do:

• Rename electrum's sync/scan bdk#1154: discord discussion on scan vs rescan -> since it is related to elecrum, we might take into consideration the electrum naming. Discussion still ongoing, comments on that will be added
• [wallet] Consider having a higher-level method for syncing bdk#1153: after discussion on #1139, a new PR needs to be done -> experiment something in the bindings first
• Coinbase transaction that is not anchored in the best chain should always be filtered out bdk#1144: No update
• Taproot PSBT finalization fails often when signing with internal key bdk#1142: No update. Bug to be investigated before alpha.3 and then evaluate the priority
• Way too many clippy::allow bdk#1127: No update

Milestone TBD:

• chore: extract TestEnv into separate crate bdk#1171: WIP (LagginTimes): shall we assign to beta being it test related? -> put in beta, if it comes earlier we can include it in alpha.3
• Introduce new bdk_coin_select implementation bdk#1072: WIP (evan): is it correct to keep it high priority being it labeled for 2.0 release?
• doc(bdk): Clarify the absolute_fee docs bdk#1158: should we assign it to beta and then include in alpha.3 if it comes earlier? -> put in beta, if it comes earlier we can include it in alpha.3
• Implement default index_tx method bdk#1176: fixes #1097 -> seems to impact the API, so we assign it to alpha.3 for the moment

Top Maintenance PRs and issues

New Projects and Support

• ci: move to Nix bdk#1165: WIP (einherjar) on 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.3
• Upgrade bitcoin/miniscript dependencies bdk#1177: adjustments needed for the release of rust-bitcoin v0.31.0-rc1: not necessary for the 1.0 beta -> put in 2.0 but if we can get it sooner it could be useful
• build(esplora): Add async-https-rustls flag to esplora client bdk#1179: discussion ongoing -> useful for android binding, possibly to be included in the next bdk_esplora release. Since it should be quick adding a flag, we added to alpha.3
• feature: add tor support by using arti-hyper with new feature async-arti-hyper rust-esplora-client#61 and add http header support rust-esplora-client#66 -> brought by leonardolima, for the moment to 1.1

Upcoming Events

• BTC Azores Sept 23-24
• PlanB Lugano Oct 20-21

🌅 Future Call Notes

📜 Past Call Notes