All Stacks-on-Bitcoin operations need to be considered in Nakamoto
In Nakamoto, all Bitcoin operations since the last sortition need to be extracted an applied when processing the next sortition. This currently does not happen because we rely on StacksChainState::process_stacking_and_transfer_and_delegate_burn_ops(), which uses a window of the last 6 Bitcoin blocks.
A few things need to happen:
- The equivalent method needs to be ported to
NakamotoChainState - The window needs to be removed
- We really should clean up this method -- we're pulling in aggregate key votes now as well, and soon we'll be pulling in sBTC ops too. We need a struct for containing all the ops, and the method should be renamed to
get_stacks_on_burnchain_ops()or something to reflect its true nature.
Write an Integration test to verify
This is done and integration tests are written as well (unless I'm misunderstanding parts of the description)
This is done and integration tests are written as well (unless I'm misunderstanding parts of the description)
I was just about to write the same thing. I think this is tested via test_process_block_ops and confirm_unparsed_ongoing_ops and stack_stx_burn_op_test
This is NOT done. Nakamoto still uses BURNCHAIN_TX_SEARCH_WINDOW to find a limited number of prior burnchain blocks.
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.