chore(sequencer_infra): add component client trait for local client

[Itay-Tsabary-Starkware]

Stack:

• chore(mempool_types): unify client implementations #2090
• chore(mempool_p2p_types): unify client implementations #2089
• chore(gateway): unify client implementations #2088
• chore(sequencer_infra): restructure code to avoid clone trait bound #2087
• chore(batcher): unify client implementations #2086
• chore(sequencer_infra): add component client trait for remote client #2085
• chore(sequencer_infra): add component client trait for local client #2084 ⬅

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

[reviewable-StarkWare]

This change is 

[codecov]

Codecov Report

Attention: Patch coverage is 7.40741% with 25 lines in your changes missing coverage. Please review.

Project coverage is 18.12%. Comparing base (e3165c4) to head (2debf78).
Report is 426 commits behind head on main.

Files with missing lines	Patch %	Lines
crates/starknet_batcher_types/src/communication.rs	0.00%	12 Missing ⚠️
crates/starknet_mempool_types/src/communication.rs	0.00%	6 Missing ⚠️
...es/starknet_mempool_p2p_types/src/communication.rs	0.00%	4 Missing ⚠️
crates/starknet_gateway_types/src/communication.rs	0.00%	2 Missing ⚠️
...tes/starknet_state_sync_types/src/communication.rs	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2084       +/-   ##
===========================================
- Coverage   40.10%   18.12%   -21.99%     
===========================================
  Files          26      243      +217     
  Lines        1895    28988    +27093     
  Branches     1895    28988    +27093     
===========================================
+ Hits          760     5254     +4494     
- Misses       1100    23375    +22275     
- Partials       35      359      +324     

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

[nadin-Starkware]

Reviewed 8 of 9 files at r1, all commit messages.
Reviewable status: 8 of 9 files reviewed, 1 unresolved discussion (waiting on @Itay-Tsabary-Starkware)

---

crates/starknet_sequencer_infra/src/component_server/remote_component_server.rs line 111 at r1 (raw file):

    Request: Serialize + DeserializeOwned + Send + Sync + 'static,
    Response: Serialize + DeserializeOwned + Send + Sync + 'static,
    LocalComponentClient<Request, Response>: ComponentClient<Request, Response>,

Why is this necessary? The concrete LocalComponentClient struct already implements the ComponentClient trait.

Code quote:

LocalComponentClient<Request, Response>: ComponentClient<Request, Response>,

[github-actions]

Artifacts upload triggered. View details here

[Itay-Tsabary-Starkware]

Reviewable status: 8 of 10 files reviewed, 1 unresolved discussion (waiting on @nadin-Starkware)

---

crates/starknet_sequencer_infra/src/component_server/remote_component_server.rs line 111 at r1 (raw file):

Previously, nadin-Starkware (Nadin Jbara) wrote…

Why is this necessary? The concrete LocalComponentClient struct already implements the ComponentClient trait.

Done.

[nadin-Starkware]

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

[Itay-Tsabary-Starkware]

Reviewed 8 of 9 files at r1, 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware)

[ShahakShama]

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

[ShahakShama]

Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware)