Brice comments

Results 277 comments of


                                            Brice

feat(clarity): Improve emitted events for Clarity post-conditions

https://github.com/stacks-network/stacks-core/pull/6025 added the `vm_error` to the transaction event payload and that includes the post-condition errors. I think this alternative can be closed.

Revisit the `clarity_vm` code paths once the chainstate is fully MARF'ed

For example, `get_first_block_in_tenure` can be simplified once that work is complete: > Broadly speaking, this is just a placeholder. When the chainstate is fully MARF'ed, the code here will instead...

test: try re-enabling the flap tests

Still doesn't work ☹️

[Nakamoto] add a flow chart for the nakamoto miner

Would be nice to have the same for the signer as well.

Signer: Tracking unfortunately timed first block in tenure vs. bitcoin blocks

I believe this can be closed. Can you confirm @kantai?

Stacks signer: Track bitcoin block timing to allow for a miner to extend

This has been solved with time-based tenure extends.

Integration test from snapshot

> the caveat is that a lot of tests will need to be rewritten due to expected balances etc. Agreed, some re-writing will be necessary regardless of what we do....

Make use of `vm_error` in transaction receipts

Note that these changes are not live in the released stacks-node yet, so this data will not be showing up yet. I just wanted to let you know it was...

corner case in `is_pox_active` in /v2/pox endpoint

This field is also not updated based on PoX rejection votes. This can be retrieved via the `is-pox-active` read-only function.

Option to show STX price in Sats

> We could add this as another option besides USD instead of replacing it. Yes, definitely.