feat(starknet_os): add more missing fields in OS input

[dorimedini-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[dorimedini-starkware]

• feat(starknet_os): add contract class component hashes in OS input #3994
• feat(starknet_os): add transaction related fields in OS input #3993
• feat(starknet_os): add more missing fields in OS input #3992 👈 (View in Graphite)
• feat(starknet_os): compiled classes in OS input #3991
• feat(starknet_os,starknet_patricia,starknet_committer): deserializable commitment info #3990
• main

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

[dorimedini-starkware]

Reviewable status: 0 of 1 files reviewed, all discussions resolved (waiting on @aner-starkware and @Yoni-Starkware)

---

crates/starknet_os/src/io/os_input.rs line 34 at r1 (raw file):

    _deprecated_compiled_classes: HashMap<ClassHash, ContractClass>,
    _compiled_classes: HashMap<ClassHash, CasmContractClass>,
    _chain_info: ChainInfo,

instead of general config. Seems to contain exactly the fields of the general config required in the OS

Code quote:

_chain_info: ChainInfo,

[aner-starkware]

crates/starknet_os/src/io/os_input.rs line 34 at r1 (raw file):

Previously, dorimedini-starkware wrote…

instead of general config. Seems to contain exactly the fields of the general config required in the OS

Really? Maybe worth refactoring on the python side as well?

[Yoni-Starkware]

Reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @aner-starkware)

[dorimedini-starkware]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @aner-starkware)

---

crates/starknet_os/src/io/os_input.rs line 34 at r1 (raw file):

Previously, aner-starkware wrote…

Really? Maybe worth refactoring on the python side as well?

not urgent, the chain info is contained in the general config; we can convert as we go.
the ambassador service itself still requires the general config anyway IIUC, so I don't see the major benefit

[Yoni-Starkware]

Reviewed 1 of 1 files at r3, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @aner-starkware)

[dorimedini-starkware]

Reviewed all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @aner-starkware)

[dorimedini-starkware]

Reviewed 1 of 1 files at r3.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @aner-starkware)

[graphite-app]

Merge activity

• Feb 6, 12:01 PM EST: Graphite rebased this pull request after merging its parent, because this pull request is set to merge when ready.