Skip to content

Cbf fix block fetch (rebased)#35

Merged
febyeji merged 4 commits into
cbf-chain-source-cleanupfrom
cbf-fix-block-fetch-rebased
Jul 16, 2026
Merged

Cbf fix block fetch (rebased)#35
febyeji merged 4 commits into
cbf-chain-source-cleanupfrom
cbf-fix-block-fetch-rebased

Conversation

@febyeji

@febyeji febyeji commented Jul 16, 2026

Copy link
Copy Markdown
Owner

This replaces #34 after the cbf-chain-source-cleanup stack was rebased onto the current main. The original contributor fork could not be updated with maintainer credentials, so the four head-only commits were replayed onto the cleaned-up base and published from this repository.

What changed

  • bump the Kyoto version
  • track whether CBF has actually synchronized to the chain tip
  • include lookahead addresses in the wallet scripts watched during sync
  • time out stalled block-fetch attempts

All four commits retain Alexander Shevtsov as their Git author. The committer records reflect the rebase operation.

Validation

  • cargo fmt --all
  • cargo check
  • cargo test --lib (78 passed)

AI tool disclosure

OpenAI Codex performed the rebase, formatting, validation, and pull-request preparation. The original implementation authorship is preserved.

Previously we did not track the `FiltersSynced` kyoto event, so we could
not tell when we had applied all blocks up to the tip. For example, when
we stop and restart the node, kyoto's tip is 0 at the instant of start
(it does not persist its chain), so our applied height trivially matches
kyoto's tip and we would falsely conclude we had reached it. That is only
actually true once we have received `FiltersSynced`.
Now the function is called `list_watched_scripts`.
Previously stalled fetch would hang indefinitely.

AI-assisted-by: OpenAI Codex
@febyeji
febyeji merged commit 539d7bd into cbf-chain-source-cleanup Jul 16, 2026
23 of 72 checks passed
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