starknet-devnet-rs icon indicating copy to clipboard operation
starknet-devnet-rs copied to clipboard

Published 20 hours ago • verified publisher icon 0xSpaceShard

Make `self.blocks.pending_block` optional and refactor of restart_pending_block() logic

Open mikiw opened this issue 1 year ago • 9 comments

related to https://github.com/0xSpaceShard/starknet-devnet-rs/pull/430

Currently if you run devnet with --blocks-on-demand and do starknet_call with block_id=pending and there is no pending block, it will not default to latest

mikiw avatar Apr 25 '24 12:04 mikiw

I'll like to hop on this

muheebyusufbaba1 avatar Apr 25 '24 12:04 muheebyusufbaba1

I recommend to find something else

mikiw avatar Apr 25 '24 12:04 mikiw

@muheebyusufbaba1 please just consider tasks with the "ODHack" label :)

ivpavici avatar Apr 25 '24 14:04 ivpavici

I don't think there's any ODHack issue left for me to work on :/ But if you will add new ODHack issues, I am up for it @ivpavici

muheebyusufbaba1 avatar Apr 25 '24 16:04 muheebyusufbaba1

since this task is related to https://github.com/0xSpaceShard/starknet-devnet-rs/pull/430 after my vacation I'll also check:

  • ~~abort of pending block, this should not be possible~~
  • defaulting to the latest state in case calls are done on pending (and there is no pending)
  • check if blocks_on_demand can be removed from StarknetBlocks during this refactor of restart_pending_block()
  • ~~clarify if declaring, deploying, and invoking the same contract can be done in one block in Starknet? (Also this problem is related to nonce implementation)~~

mikiw avatar Apr 26 '24 13:04 mikiw

also to note: In devnet-py we always had a pending state which, on block creation, would get converted to latest state. Block creation was done after each tx in normal mode and on demand in blocks-on-demand mode.

Currently we are treating this pending state differently (maybe due to some language or blockifier limitations, not sure) - there are several checks if mode is on-demand or not

FabijanC avatar Apr 26 '24 14:04 FabijanC

  • clarify if declaring, deploying, and invoking the same contract can be done in one block in Starknet? (Also this problem is related to nonce implementation)

regarding that it should be possible

mikiw avatar May 08 '24 07:05 mikiw

  • clarify if declaring, deploying, and invoking the same contract can be done in one block in Starknet? (Also this problem is related to nonce implementation)

regarding that it should be possible

addressed in https://github.com/0xSpaceShard/starknet-devnet-rs/commit/bcd68e71afdee71cb19594fb88e1ec2f232f362c

mikiw avatar May 08 '24 10:05 mikiw

I planned to do the same refactoring to pending_state in StarknetState but now it seems it's necessary

mikiw avatar May 23 '24 08:05 mikiw

Actually, after consultation with starkware, we realized that this should work differently. This PR will address that https://github.com/0xSpaceShard/starknet-devnet-rs/pull/480 (Please check Development related changes).

mikiw avatar Jun 04 '24 14:06 mikiw