Use original function with manual scaling (#96)

* Use original function with manual scaling

* Added comment

Author: Solar Mithril

crates/op-rbuilder/src/builders/context.rs

@@ -571,9 +571,12 @@ impl OpPayloadBuilderCtx {
                 // Create and sign the transaction
                 let builder_tx =
                     signed_builder_tx(db, builder_tx_gas, message, signer, base_fee, chain_id)?;
-                Ok(op_alloy_flz::data_gas_fjord(
-                    builder_tx.encoded_2718().as_slice(),
-                ))
+                Ok(
+                    op_alloy_flz::tx_estimated_size_fjord(builder_tx.encoded_2718().as_slice())
+                        // Downscaled by 1e6 to be compliant with op-geth estimate size function
+                        // https://github.com/ethereum-optimism/op-geth/blob/optimism/core/types/rollup_cost.go#L563
+                        .wrapping_div(1_000_000),
+                )
             })
             .transpose()
             .unwrap_or_else(|err: PayloadBuilderError| {

crates/op-rbuilder/src/tests/vanilla/data_availability.rs

@@ -79,7 +79,7 @@ async fn data_availability_block_fill() -> eyre::Result<()> {
     // Set block big enough so it could fit 3 transactions without tx size limit
     let call = harness
         .provider()?
-        .raw_request::<(i32, i32), bool>("miner_setMaxDASize".into(), (0, 1600 * 3))
+        .raw_request::<(i32, i32), bool>("miner_setMaxDASize".into(), (0, 100 * 3))
         .await?;
     assert!(call, "miner_setMaxDASize should be executed successfully");
 

crates/op-rbuilder/src/tx.rs

@@ -221,7 +221,9 @@ impl MaybeInteropTransaction for FBPooledTransaction {
 
 impl DataAvailabilitySized for FBPooledTransaction {
     fn estimated_da_size(&self) -> u64 {
-        op_alloy_flz::data_gas_fjord(self.inner.encoded_2718())
+        // Downscaled by 1e6 to be compliant with op-geth estimate size function
+        // https://github.com/ethereum-optimism/op-geth/blob/optimism/core/types/rollup_cost.go#L563
+        op_alloy_flz::tx_estimated_size_fjord(self.inner.encoded_2718()).wrapping_div(1_000_000)
     }
 }