Update StackStxOp to include signer key

[8marz8]

Description

Following the signer-key authorization in #4377, this PR updates stack-stx burn op (StackStxOp) to include signer-key.

Applicable issues

• closes [Nakamoto] stack-stx burn op requires a signing key and signature #4285

[codecov]

Codecov Report

Attention: Patch coverage is 95.51167% with 50 lines in your changes are missing coverage. Please review.

Project coverage is 83.39%. Comparing base (0e9cf5d) to head (cb136bf).
Report is 7 commits behind head on next.

Additional details and impacted files

@@            Coverage Diff             @@
##             next    #4412      +/-   ##
==========================================
+ Coverage   83.33%   83.39%   +0.06%     
==========================================
  Files         453      453              
  Lines      328494   329585    +1091     
  Branches      323      323              
==========================================
+ Hits       273744   274853    +1109     
+ Misses      54742    54724      -18     
  Partials        8        8              

Files	Coverage Δ
...c/chainstate/burn/operations/test/serialization.rs	100.00% <100.00%> (ø)
stackslib/src/chainstate/coordinator/tests.rs	90.80% <100.00%> (+0.01%)	⬆️
stackslib/src/chainstate/burn/operations/mod.rs	62.36% <75.00%> (+0.54%)	⬆️
stackslib/src/chainstate/burn/db/sortdb.rs	91.97% <91.66%> (+<0.01%)	⬆️
stackslib/src/chainstate/stacks/db/blocks.rs	90.04% <92.85%> (+<0.01%)	⬆️
...ckslib/src/chainstate/burn/operations/stack_stx.rs	86.67% <94.37%> (+3.61%)	⬆️
...net/stacks-node/src/tests/nakamoto_integrations.rs	95.56% <97.12%> (+0.29%)	⬆️
testnet/stacks-node/src/tests/neon_integrations.rs	74.12% <97.01%> (+0.82%)	⬆️
...-node/src/burnchains/bitcoin_regtest_controller.rs	87.30% <80.88%> (+0.12%)	⬆️

... and 22 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0e9cf5d...cb136bf. Read the comment docs.

[hstove]

Nice job! I think this is pretty close. You should mainly add an integration test (like in nakamoto_integrations)

[hstove]

Ah, I just realized that #4461 will require an update to this burn op (two new u128 fields)

[hstove]

Everything looks great, I just have some comments around backwards compatibility with the existing burn op structure. Nice work!

Dismissing because I committed

[kantai]

LGTM -- just one comment trying to spread the good news about the PoxVersions enum.

[blockstack-devops]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.