starknet_os_runner: virtual block executor

[AvivYossef-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[AvivYossef-starkware]

• starknet_os_runner: aviv refactor the rpc state reader to compute the compiled class hash #11302
• starknet_os_runner: base block info #11295
• starknet_os_runner: fetch classes concurrent #11288
• starknet_os_runner: prev block hash #11188
• starknet_os_runner: virtual block input #11176
• starknet_os_runner: classes provider using class manager #11006
• blockifier: convert entry points #11005
• starknet_os_runner: run os #10941
• starknet_os_runner: build os input #10940
• starknet_os_runner: class provider #10939
• starknet_os_runner: share test utils #10934
• starknet_os_runner: storage proof provider trait #10891
• starknet_os_runner: rpc storage_proofs #10890
• starknet_patricia: from merkle node to preimage #10889
• starknet_os_runner: add executed_class_hashes to virtual block execution data #10932
• starknet_os_runner: virtual block executor #10930 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Yoni-Starkware]

@Yoni-Starkware made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware and @noaov1).

---

crates/starknet_os_runner/src/virtual_block_executor.rs line 179 at r1 (raw file):

/// without block preprocessing. Validation and fee charging are always skipped,
/// making it suitable for simulation and OS input generation.
impl VirtualBlockExecutor for RpcStateReader {

It's a bit weird. Can you please restore the dedicated struct for execution?

Code quote:

impl VirtualBlockExecutor for RpcStateReader {

[Yoni-Starkware]

@Yoni-Starkware reviewed 1 file.
Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware and @noaov1).

[AvivYossef-starkware]

@AvivYossef-starkware made 1 comment.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @noaov1 and @Yoni-Starkware).

---

crates/starknet_os_runner/src/virtual_block_executor.rs line 179 at r1 (raw file):

Previously, Yoni-Starkware (Yoni) wrote…

It's a bit weird. Can you please restore the dedicated struct for execution?

Done.

[Yoni-Starkware]

@Yoni-Starkware reviewed 2 files and all commit messages, made 2 comments, and resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @AvivYossef-starkware and @noaov1).

---

crates/starknet_os_runner/src/virtual_block_executor.rs line 55 at r2 (raw file):

///
/// ```text
/// let executor = RpcStateReader::new_with_config_from_url(

Update docs

Code quote:

 RpcStateReader::new_with_config_from_url(

[AvivYossef-starkware]

@AvivYossef-starkware reviewed 1 file and all commit messages, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @noaov1).