Skip to content

feat(dash-spv-ffi): add dash_spv_ffi_client_broadcast_transaction FFI function#447

Draft
ZocoLini wants to merge 3 commits intov0.42-devfrom
feat/transaction-broadcasting-ffi
Draft

feat(dash-spv-ffi): add dash_spv_ffi_client_broadcast_transaction FFI function#447
ZocoLini wants to merge 3 commits intov0.42-devfrom
feat/transaction-broadcasting-ffi

Conversation

@ZocoLini
Copy link
Collaborator

No description provided.

@ZocoLini ZocoLini marked this pull request as draft February 17, 2026 19:22
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 17, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/transaction-broadcasting-ffi

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ZocoLini ZocoLini force-pushed the feat/transaction-broadcasting-ffi branch 2 times, most recently from a1d2ed4 to beee4e8 Compare February 17, 2026 21:19
xdustinface and others added 3 commits February 18, 2026 16:09
All fields wrapped in `Arc` with locks. Callers hold a handle while the monitoring loop runs in a
spawned task via `run()` getting rid of the command channel pattern.
…441)

* fix: reset `FilterManager` when all peers disconnect

- Override `stop_sync` in `FiltersManager` to clear the in-flight state when all peers disconnect.
- Re-initialize the filter pipeline position in `start_sync` when filters are already at the stored tip preventing `extend_target` from re-queuing from height 1 after reconnecting.

* Fix typo

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

---------

Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@ZocoLini ZocoLini force-pushed the feat/transaction-broadcasting-ffi branch from beee4e8 to 0e2b884 Compare February 18, 2026 22:01
@github-actions
Copy link

This PR has merge conflicts with the base branch. Please rebase or merge the base branch into your branch to resolve them.

@github-actions github-actions bot added the merge-conflict The PR conflicts with the target branch. label Feb 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge-conflict The PR conflicts with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments