starknet_api: add fee_proposal_fri to BlockHeaderWithoutHash

Author: sirandreww-starkware

crates/apollo_p2p_sync/src/client/test.rs

@@ -345,6 +345,7 @@ fn create_random_sync_block(
         sequencer,
         l1_da_mode,
         starknet_version,
+        fee_proposal_fri,
     } = BlockHeaderWithoutHash::get_test_instance(&mut rng);
     let block_header_without_hash = BlockHeaderWithoutHash {
         block_number,
@@ -359,6 +360,7 @@ fn create_random_sync_block(
         sequencer,
         l1_da_mode,
         starknet_version,
+        fee_proposal_fri,
     };
     let block_header_commitments = BlockHeaderCommitments::get_test_instance(&mut rng);
     SyncBlock {

crates/apollo_protobuf/src/converters/header.rs

@@ -205,6 +205,7 @@ impl TryFrom<protobuf::SignedBlockHeader> for SignedBlockHeader {
                     timestamp,
                     l1_da_mode,
                     starknet_version,
+                    fee_proposal_fri: None,
                 },
                 state_diff_commitment,
                 state_diff_length,

crates/apollo_protobuf/src/converters/header_test.rs

@@ -5,7 +5,9 @@ use crate::sync::{DataOrFin, HeaderQuery, SignedBlockHeader};
 #[test]
 fn block_header_to_bytes_and_back() {
     let mut rng = get_rng();
-    let signed_block_header = SignedBlockHeader::get_test_instance(&mut rng);
+    let mut signed_block_header = SignedBlockHeader::get_test_instance(&mut rng);
+    // The sync proto does not carry `fee_proposal_fri`; drop it to match the roundtrip.
+    signed_block_header.block_header.block_header_without_hash.fee_proposal_fri = None;
     let data = DataOrFin(Some(signed_block_header.clone()));
     let bytes_data = Vec::<u8>::from(data.clone());
     let res_data = DataOrFin::try_from(bytes_data).unwrap();
@@ -20,6 +22,8 @@ fn block_header_without_commitments_to_bytes_and_back() {
     signed_block_header.block_header.transaction_commitment = None;
     signed_block_header.block_header.event_commitment = None;
     signed_block_header.block_header.receipt_commitment = None;
+    // The sync proto does not carry `fee_proposal_fri`; drop it to match the roundtrip.
+    signed_block_header.block_header.block_header_without_hash.fee_proposal_fri = None;
 
     let data = DataOrFin(Some(signed_block_header.clone()));
     let bytes_data = Vec::<u8>::from(data.clone());

crates/apollo_starknet_client/src/reader/objects/block.rs

@@ -342,6 +342,7 @@ impl Block {
                 l1_data_gas_price: self.l1_data_gas_price(),
                 l1_da_mode: self.l1_da_mode(),
                 starknet_version: self.starknet_version(),
+                fee_proposal_fri: None,
             },
             state_diff_commitment: self.state_diff_commitment(),
             transaction_commitment,

crates/apollo_storage/src/header.rs

@@ -109,6 +109,8 @@ pub struct StorageBlockHeader {
     pub n_transactions: usize,
     /// The number of events in this block.
     pub n_events: usize,
+    /// SNIP-35: proposer's oracle-derived recommended fee. `None` for pre-V0_14_3 blocks.
+    pub fee_proposal_fri: Option<GasPrice>,
 }
 
 type BlockHashToNumberTable<'env> =
@@ -225,6 +227,7 @@ impl<Mode: TransactionKind> HeaderStorageReader for StorageTxn<'_, Mode> {
                 timestamp: block_header.timestamp,
                 l1_da_mode: block_header.l1_da_mode,
                 starknet_version,
+                fee_proposal_fri: block_header.fee_proposal_fri,
             },
             state_diff_commitment: block_header.state_diff_commitment,
             transaction_commitment: block_header.transaction_commitment,
@@ -322,6 +325,7 @@ impl HeaderStorageWriter for StorageTxn<'_, RW> {
             state_diff_length: block_header.state_diff_length,
             n_transactions: block_header.n_transactions,
             n_events: block_header.n_events,
+            fee_proposal_fri: block_header.block_header_without_hash.fee_proposal_fri,
         };
 
         headers_table.append(&self.txn, &block_number, &storage_block_header)?;
@@ -429,6 +433,7 @@ impl HeaderStorageWriter for StorageTxn<'_, RW> {
                     timestamp: reverted_header.timestamp,
                     l1_da_mode: reverted_header.l1_da_mode,
                     starknet_version,
+                    fee_proposal_fri: reverted_header.fee_proposal_fri,
                 },
                 state_diff_commitment: reverted_header.state_diff_commitment,
                 transaction_commitment: reverted_header.transaction_commitment,

crates/apollo_storage/src/serialization/serializers.rs

@@ -183,6 +183,7 @@ auto_storage_serde! {
         pub state_diff_length: Option<usize>,
         pub n_transactions: usize,
         pub n_events: usize,
+        pub fee_proposal_fri: Option<GasPrice>,
     }
     pub struct BlockHeaderCommitments {
         pub transaction_commitment: TransactionCommitment,

crates/apollo_storage/src/test_instances.rs

@@ -46,6 +46,7 @@ auto_impl_get_test_instance! {
         pub state_diff_length: Option<usize>,
         pub n_transactions: usize,
         pub n_events: usize,
+        pub fee_proposal_fri: Option<GasPrice>,
     }
 
     struct EventIndex(pub TransactionIndex, pub EventIndexInTransactionOutput);

crates/apollo_test_utils/src/lib.rs

@@ -450,6 +450,7 @@ auto_impl_get_test_instance! {
         pub timestamp: BlockTimestamp,
         pub l1_da_mode: L1DataAvailabilityMode,
         pub starknet_version: StarknetVersion,
+        pub fee_proposal_fri: Option<GasPrice>,
     }
     pub struct BlockHeaderCommitments {
         pub transaction_commitment: TransactionCommitment,

crates/starknet_api/src/block.rs

@@ -241,6 +241,8 @@ pub struct BlockHeaderWithoutHash {
     pub timestamp: BlockTimestamp,
     pub l1_da_mode: L1DataAvailabilityMode,
     pub starknet_version: StarknetVersion,
+    /// SNIP-35: proposer's oracle-derived recommended fee. `None` for pre-V0_14_3 blocks.
+    pub fee_proposal_fri: Option<GasPrice>,
 }
 
 /// The [transactions](`crate::transaction::Transaction`) and their