refactor: define sierra to casm compiler trait

[ArniStarkware]

This change is 

[ArniStarkware]

• feat: use command line compiler in gateway #466
• feat: use the installed starknet-sierra-compile crate to compile #331
• chore(gateway): rename cairo_lang_compiler file #330
• refactor: define sierra to casm compiler trait #319 👈
• refactor: move the panic catch into the compile sierra to casm method #318
• chore: expose max bytecode size arg for the compile sierra to casm util #283
• chore: add declare_tx fixture to compilation tests #250
• main

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

Join @ArniStarkware and the rest of your teammates on Graphite

[dafnamatsry]

Reviewed 8 of 8 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @ArniStarkware, @Yael-Starkware, and @yair-starkware)

---

crates/starknet_sierra_compile/src/compile.rs line 18 at r1 (raw file):

/// `cairo_lang_starknet_classes` crate.
#[derive(Clone)]
pub struct CairoLangCompiler {

Suggestion:

CairoLangSierraToCasmCompiler

---

crates/starknet_sierra_compile/src/lib.rs line 16 at r1 (raw file):

pub trait SierraToCasmCompiler: Send + Sync {
    fn compile_sierra_to_casm(

Suggestion:

compile

[ArniStarkware]

Reviewable status: 4 of 8 files reviewed, 2 unresolved discussions (waiting on @dafnamatsry, @Yael-Starkware, and @yair-starkware)

---

crates/starknet_sierra_compile/src/compile.rs line 18 at r1 (raw file):

/// `cairo_lang_starknet_classes` crate.
#[derive(Clone)]
pub struct CairoLangCompiler {

Done.

---

crates/starknet_sierra_compile/src/lib.rs line 16 at r1 (raw file):

pub trait SierraToCasmCompiler: Send + Sync {
    fn compile_sierra_to_casm(

Done.

[dafnamatsry]

Reviewed 4 of 4 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Yael-Starkware and @yair-starkware)

[dafnamatsry]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Yael-Starkware and @yair-starkware)

[ArniStarkware]

Merge activity

• Aug 8, 5:47 AM EDT: @ArniStarkware started a stack merge that includes this pull request via Graphite.
• Aug 8, 5:57 AM EDT: Graphite rebased this pull request as part of a merge.
• Aug 8, 6:01 AM EDT: @ArniStarkware merged this pull request with Graphite.