apollo_protobuf: add fee_proposal_fri to sync proto SignedBlockHeader

[sirandreww-starkware]

No description provided.

[reviewable-StarkWare]

This change is 

[sirandreww-starkware]

• apollo_storage: add fee_proposal_fri migrator and bump blocks version to 6.1 #13936 : 3 dependent PRs (#13814 , #13854 , #13934 )
• apollo_protobuf: add fee_proposal_fri to sync proto SignedBlockHeader #13957 👈 (View in Graphite)
• starknet_api: add fee_proposal_fri to BlockHeaderWithoutHash #13822
• apollo_protobuf: add fee_proposal_fri to ProposalInit #13812 : 2 other dependent PRs (#13811 , #13935 )
• main

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

[cursor]

PR Summary

Medium Risk
Changes the P2P sync wire format by adding a new optional field to SignedBlockHeader and updating conversions, which can impact cross-version compatibility and header roundtrips.

Overview
Adds fee_proposal_fri (SNIP-35 recommended fee) to the sync SignedBlockHeader protobuf message and regenerates the prost output.

Updates the Rust converters to roundtrip this new optional field by mapping it to/from starknet_api::block::GasPrice, and removes the prior tests’ #[should_panic] expectations now that header serialization/deserialization preserves the field.

^{Reviewed by Cursor Bugbot for commit f425831. Bugbot is set up for automated code reviews on this repo. Configure here.}

[ShahakShama]

@ShahakShama reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on sirandreww-starkware).

---

crates/apollo_protobuf/src/proto/p2p/proto/sync/header.proto line 34 at r1 (raw file):

    // for now, we assume a small consensus, so this fits in 1M. Else, these will be repeated and extracted from this message.
    repeated ConsensusSignature signatures = 21;
    // SNIP-35: proposer's oracle-derived recommended fee. Absent for pre-V0_14_3 blocks.

Add a warning here that this is currently not part of the block hash and the value is trusted

[sirandreww-starkware]

@sirandreww-starkware made 1 comment.
Reviewable status: 1 of 4 files reviewed, 1 unresolved discussion (waiting on ShahakShama).

---

crates/apollo_protobuf/src/proto/p2p/proto/sync/header.proto line 34 at r1 (raw file):

Previously, ShahakShama wrote…

Add a warning here that this is currently not part of the block hash and the value is trusted

Done.

[ShahakShama]

@ShahakShama reviewed 3 files and all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on sirandreww-starkware).

[graphite-app]

Merge activity

• May 6, 2:58 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.