Skip to content

Add lookahead to cbf chain source#17

Closed
randomlogin wants to merge 0 commit into
febyeji:add-cbf-chain-sourcefrom
randomlogin:fix-cbf-bdk-spk-lookahead
Closed

Add lookahead to cbf chain source#17
randomlogin wants to merge 0 commit into
febyeji:add-cbf-chain-sourcefrom
randomlogin:fix-cbf-bdk-spk-lookahead

Conversation

@randomlogin
Copy link
Copy Markdown

BDK's reveal cursor won't advance based on matched scripts, so after wallet-funded activity the next register_or_subscribe_pending_sync will keep re-sending the same SPK set and may not reveal the next external/internal index.

Changed to be in style of https://github.com/bitcoindevkit/bdk-kyoto

@randomlogin randomlogin marked this pull request as draft May 5, 2026 02:41
@febyeji febyeji marked this pull request as ready for review May 6, 2026 10:06
@febyeji febyeji marked this pull request as draft May 6, 2026 10:06
@febyeji febyeji self-requested a review May 6, 2026 10:07
@randomlogin randomlogin marked this pull request as ready for review May 7, 2026 14:36
@febyeji febyeji force-pushed the add-cbf-chain-source branch from 43d908f to d1030d8 Compare May 12, 2026 11:47
Copy link
Copy Markdown
Owner

@febyeji febyeji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I will merge this after it is rebased.

@randomlogin randomlogin force-pushed the fix-cbf-bdk-spk-lookahead branch from 556545b to cb1d1e9 Compare May 12, 2026 18:16
@randomlogin randomlogin force-pushed the fix-cbf-bdk-spk-lookahead branch from cb1d1e9 to d1030d8 Compare May 12, 2026 18:22
@randomlogin
Copy link
Copy Markdown
Author

This change was already absorbed by another PR, sorry for miscommunication.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants