dgb(#82): won-block dispatch handler (m_on_block_found) + wiring [re-PR superseded #167] - #196
Closed
frstrtr wants to merge 2 commits into
Closed
dgb(#82): won-block dispatch handler (m_on_block_found) + wiring [re-PR superseded #167]#196frstrtr wants to merge 2 commits into
frstrtr wants to merge 2 commits into
Conversation
dgb(#82): dual-path won-block broadcaster + dispatch contract test
make_on_block_found turns a winning share hash into a broadcast: reconstruct share->block via an injected WonBlockReconstructor, then fire broadcast_won_block down both paths (P2P primary + external digibyted submitblock fallback). The reconstructor is injected so the run-loop dispatch contract is build-verified and run-tested now, before the faithful as_block reassembly is ported. won_block_dispatch_test drives the handler end-to-end: known share fires both paths, unknown/unassemblable share broadcasts nothing, RPC-only still reaches the network, missing reconstructor is a safe no-op. 4/4 PASS. Registered in test/CMakeLists.txt and both build.yml --target allowlists (avoids the #143 NOT_BUILT trap). src/impl/dgb/ only; no p2pool-merged-v36 surface.
frstrtr
force-pushed
the
dgb/won-block-dispatch-wiring
branch
from
June 19, 2026 09:04
5845422 to
67512ef
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Re-PR of the genuinely-missing won-block dispatch slice (old #167 was merged into the broadcaster intermediate branch, never reached master).
Adds src/impl/dgb/coin/won_block_dispatch.hpp (make_on_block_found -> installs as m_on_block_found, reconstruct share->block via injected WonBlockReconstructor, then broadcast_won_block dual-path) + won_block_dispatch_test (4/4).
Stack order on origin/master tip 0c28591:
#163 coin-node-submit-seam (e83f872) -> #166 broadcaster (984eb3c) -> THIS dispatch (5845422).
block_broadcast.hpp (#166) and won_block_dispatch.hpp (this) are the only two won-block-family headers ABSENT from master; reconstruct_won_block/other_tx_resolver/other_tx_assembler/block_assembly/gentx_unpack already landed. HOLD merge — integrator merges after #163, #166.