chore: add the gateway compiler to appstate

[ArniStarkware]

This change is 

[ArniStarkware]

• feat: post compilation size limit validation #298 : 2 dependent PRs (#348 , #384 )
• chore: add declare_tx fixture to compilation tests #515
• refactor: gateway compiler handle declare tx #439
• chore: add the gateway compiler to appstate #391 👈
• chore: create struct for gateway compiler #383 : 1 other dependent PR (#392 )
• main

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

Join @ArniStarkware and the rest of your teammates on Graphite

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.

Project coverage is 83.21%. Comparing base (ffb8b05) to head (33dbd4d).

Files	Patch %	Lines
crates/gateway/src/config.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #391      +/-   ##
==========================================
+ Coverage   83.18%   83.21%   +0.03%     
==========================================
  Files          37       37              
  Lines        1748     1752       +4     
  Branches     1748     1752       +4     
==========================================
+ Hits         1454     1458       +4     
  Misses        215      215              
  Partials       79       79              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yair-starkware]

Reviewed 4 of 4 files at r1, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dafnamatsry and @Yael-Starkware)

[yair-starkware]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dafnamatsry and @Yael-Starkware)

[dafnamatsry]

Reviewed 1 of 4 files at r3, 3 of 3 files at r4, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @ArniStarkware and @Yael-Starkware)

---

crates/gateway/src/gateway.rs line 57 at r4 (raw file):

            }),
            state_reader_factory,
            gateway_compiler: GatewayCompiler { config: config.compiler_config.clone() },

Please accept the compiler as an argument to new.
(as discussed before, we also want to do this for the stateless/stateful validators).

Code quote:

gateway_compiler: GatewayCompiler { config: config.compiler_config.clone() },

---

crates/gateway/src/gateway_test.rs line 117 at r4 (raw file):

    let optional_class_info = match &external_tx {
        RPCTransaction::Declare(declare_tx) => Some(
            GatewayCompiler { config: GatewayCompilerConfig {} }

Use the AppState's compiler.

Code quote:

GatewayCompiler { config: GatewayCompilerConfig {} }

[ArniStarkware]

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

---

crates/gateway/src/gateway.rs line 57 at r4 (raw file):

Previously, dafnamatsry wrote…

Please accept the compiler as an argument to new.
(as discussed before, we also want to do this for the stateless/stateful validators).

Done.

---

crates/gateway/src/gateway_test.rs line 117 at r4 (raw file):

Previously, dafnamatsry wrote…

Use the AppState's compiler.

Done.

[dafnamatsry]

Reviewed 4 of 4 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Yael-Starkware)

[ArniStarkware]

Merge activity

• Jul 14, 7:12 AM EDT: @ArniStarkware started a stack merge that includes this pull request via Graphite.
• Jul 14, 7:14 AM EDT: Graphite rebased this pull request as part of a merge.
• Jul 14, 7:16 AM EDT: @ArniStarkware merged this pull request with Graphite.

[ArniStarkware]

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