From f996863e84b4557bb48db9367ed17bd85e5c297b Mon Sep 17 00:00:00 2001
From: MujkicA <32431923+MujkicA@users.noreply.github.com>
Date: Wed, 27 Sep 2023 23:41:21 +0200
Subject: [PATCH] feat: pull tx param defaults from network (#1121)

Closes #1113

Bundles relevant NodeInfo and ChainInfo field to NetworkInfo so that it can be provided to a tx builder. The builder is then able to use default values for unset tx parameters.


---------

Co-authored-by: hal3e <git@hal3e.io>
---
 docs/src/calling-contracts/tx-params.md       |   6 +-
 docs/src/connecting/external-node.md          |   2 +-
 examples/cookbook/src/lib.rs                  |   9 +-
 packages/fuels-accounts/src/lib.rs            |  31 ++-
 packages/fuels-accounts/src/predicate.rs      |   2 -
 packages/fuels-accounts/src/provider.rs       |   8 +
 packages/fuels-accounts/src/wallet.rs         |   2 -
 .../src/types/transaction_builders.rs         | 206 +++++++++++-------
 .../fuels-core/src/types/wrappers/block.rs    |   4 +-
 .../src/types/wrappers/chain_info.rs          |   3 +
 .../src/types/wrappers/node_info.rs           |   2 +-
 .../src/types/wrappers/transaction.rs         |  78 +++----
 packages/fuels-core/src/utils/constants.rs    |   6 -
 packages/fuels-programs/src/call_utils.rs     |   8 +-
 packages/fuels-programs/src/contract.rs       |   3 +
 packages/fuels-programs/src/script_calls.rs   |  12 +-
 packages/fuels/tests/predicates.rs            |  23 +-
 packages/fuels/tests/providers.rs             |  32 ++-
 packages/fuels/tests/wallets.rs               |  27 ++-
 19 files changed, 289 insertions(+), 175 deletions(-)

diff --git a/docs/src/calling-contracts/tx-params.md b/docs/src/calling-contracts/tx-params.md
index 2b8fcfaf4e..3b7bdb64a8 100644
--- a/docs/src/calling-contracts/tx-params.md
+++ b/docs/src/calling-contracts/tx-params.md
@@ -17,13 +17,9 @@ You can configure these parameters by creating an instance of `TxParameters` and
 
 <!-- This section should explain how to use the defauly tx params -->
 <!-- tx_params_default:example:start -->
-You can also use `TxParameters::default()` to use the default values:
+You can also use `TxParameters::default()` to use the default values. If `gas_price` or `gas_limit` is set to `None`, the SDK will use the network's default value:
 <!-- tx_params_default:example:end -->
 
-```rust,ignore
-{{#include ../../../packages/fuels-core/src/utils/constants.rs:default_tx_parameters}}
-```
-
 This way:
 
 ```rust,ignore
diff --git a/docs/src/connecting/external-node.md b/docs/src/connecting/external-node.md
index b00cae3f82..08e3b47fa9 100644
--- a/docs/src/connecting/external-node.md
+++ b/docs/src/connecting/external-node.md
@@ -14,7 +14,7 @@ In the code example, we connected a new provider to the Testnet node and created
 
 > **Note:** New wallets on the Testnet will not have any assets! They can be obtained by providing the wallet address to the faucet at
 >
->[faucet-beta-3.fuel.network](https://faucet-beta-3.fuel.network)
+>[faucet-beta-4.fuel.network](https://faucet-beta-4.fuel.network)
 >
 > Once the assets have been transferred to the wallet, you can reuse it in other tests by providing the private key!
 >
diff --git a/examples/cookbook/src/lib.rs b/examples/cookbook/src/lib.rs
index 6113a8a765..1d7fccd4fb 100644
--- a/examples/cookbook/src/lib.rs
+++ b/examples/cookbook/src/lib.rs
@@ -166,8 +166,13 @@ mod tests {
         // ANCHOR_END: transfer_multiple_inout
 
         // ANCHOR: transfer_multiple_transaction
-        let mut tb =
-            ScriptTransactionBuilder::prepare_transfer(inputs, outputs, TxParameters::default());
+        let network_info = provider.network_info().await?;
+        let mut tb = ScriptTransactionBuilder::prepare_transfer(
+            inputs,
+            outputs,
+            TxParameters::default(),
+            network_info,
+        );
         wallet_1.sign_transaction(&mut tb);
         let tx = tb.build()?;
 
diff --git a/packages/fuels-accounts/src/lib.rs b/packages/fuels-accounts/src/lib.rs
index 8ea265a590..98245ca2aa 100644
--- a/packages/fuels-accounts/src/lib.rs
+++ b/packages/fuels-accounts/src/lib.rs
@@ -186,15 +186,18 @@ pub trait Account: ViewOnlyAccount {
         tx_parameters: TxParameters,
     ) -> Result<(TxId, Vec<Receipt>)> {
         let provider = self.try_provider()?;
+        let network_info = provider.network_info().await?;
 
         let inputs = self.get_asset_inputs_for_amount(asset_id, amount).await?;
 
         let outputs = self.get_asset_outputs_for_amount(to, asset_id, amount);
 
-        let consensus_parameters = provider.consensus_parameters();
-
-        let tx_builder = ScriptTransactionBuilder::prepare_transfer(inputs, outputs, tx_parameters)
-            .with_consensus_parameters(consensus_parameters);
+        let tx_builder = ScriptTransactionBuilder::prepare_transfer(
+            inputs,
+            outputs,
+            tx_parameters,
+            network_info,
+        );
 
         // if we are not transferring the base asset, previous base amount is 0
         let previous_base_amount = if asset_id == AssetId::default() {
@@ -233,6 +236,7 @@ pub trait Account: ViewOnlyAccount {
         tx_parameters: TxParameters,
     ) -> std::result::Result<(String, Vec<Receipt>), Error> {
         let provider = self.try_provider()?;
+        let network_info = provider.network_info().await?;
 
         let zeroes = Bytes32::zeroed();
         let plain_contract_id: ContractId = to.into();
@@ -253,8 +257,6 @@ pub trait Account: ViewOnlyAccount {
         ];
 
         // Build transaction and sign it
-        let params = provider.consensus_parameters();
-
         let tb = ScriptTransactionBuilder::prepare_contract_transfer(
             plain_contract_id,
             balance,
@@ -262,8 +264,8 @@ pub trait Account: ViewOnlyAccount {
             inputs,
             outputs,
             tx_parameters,
-        )
-        .with_consensus_parameters(params);
+            network_info,
+        );
 
         // if we are not transferring the base asset, previous base amount is 0
         let base_amount = if asset_id == AssetId::default() {
@@ -294,6 +296,7 @@ pub trait Account: ViewOnlyAccount {
         tx_parameters: TxParameters,
     ) -> std::result::Result<(TxId, MessageId, Vec<Receipt>), Error> {
         let provider = self.try_provider()?;
+        let network_info = provider.network_info().await?;
 
         let inputs = self
             .get_asset_inputs_for_amount(BASE_ASSET_ID, amount)
@@ -304,6 +307,7 @@ pub trait Account: ViewOnlyAccount {
             amount,
             inputs,
             tx_parameters,
+            network_info,
         );
 
         let tx = self.add_fee_resources(tb, amount).await?;
@@ -327,7 +331,7 @@ mod tests {
 
     use fuel_crypto::{Message, SecretKey};
     use fuel_tx::{Address, Output};
-    use fuels_core::types::transaction::Transaction;
+    use fuels_core::types::{transaction::Transaction, transaction_builders::NetworkInfo};
     use rand::{rngs::StdRng, RngCore, SeedableRng};
 
     use super::*;
@@ -376,6 +380,12 @@ mod tests {
         )?;
         let wallet = WalletUnlocked::new_from_private_key(secret, None);
 
+        let network_info = NetworkInfo {
+            consensus_parameters: Default::default(),
+            max_gas_per_tx: 0,
+            min_gas_price: 0,
+            gas_costs: Default::default(),
+        };
         // Set up a transaction
         let mut tb = {
             let input_coin = Input::ResourceSigned {
@@ -398,6 +408,7 @@ mod tests {
                 vec![input_coin],
                 vec![output_coin],
                 Default::default(),
+                network_info,
             )
         };
 
@@ -417,7 +428,7 @@ mod tests {
         assert_eq!(signature, tx_signature);
 
         // Check if the signature is what we expect it to be
-        assert_eq!(signature, Signature::from_str("d7027be16db0aada625ac8cd438f9b6187bd74465495ba39511c1ad72b7bb10af4ef582c94cc33433f7a1eb4f2ad21c471473947f5f645e90924ba273e2cee7f")?);
+        assert_eq!(signature, Signature::from_str("51198e39c541cd3197785fd8add8cdbec3dc5aba7f8fbb23eb09455dd1003a8b78d94f247df8e1577805ea7eebd6d58336393942fd98484609e9e7d6d7a55f28")?);
 
         // Recover the address that signed the transaction
         let recovered_address = signature.recover(&message)?;
diff --git a/packages/fuels-accounts/src/predicate.rs b/packages/fuels-accounts/src/predicate.rs
index 510d00db58..6e2e7a2097 100644
--- a/packages/fuels-accounts/src/predicate.rs
+++ b/packages/fuels-accounts/src/predicate.rs
@@ -146,8 +146,6 @@ impl Account for Predicate {
         previous_base_amount: u64,
     ) -> Result<Tb::TxType> {
         let consensus_parameters = self.try_provider()?.consensus_parameters();
-        tb = tb.with_consensus_parameters(consensus_parameters);
-
         let new_base_amount =
             calculate_base_amount_with_fee(&tb, &consensus_parameters, previous_base_amount)?;
 
diff --git a/packages/fuels-accounts/src/provider.rs b/packages/fuels-accounts/src/provider.rs
index aa3da6038f..438dd4395b 100644
--- a/packages/fuels-accounts/src/provider.rs
+++ b/packages/fuels-accounts/src/provider.rs
@@ -26,6 +26,7 @@ use fuels_core::{
         message_proof::MessageProof,
         node_info::NodeInfo,
         transaction::Transaction,
+        transaction_builders::NetworkInfo,
         transaction_response::TransactionResponse,
         tx_status::TxStatus,
     },
@@ -274,6 +275,13 @@ impl Provider {
         self.consensus_parameters
     }
 
+    pub async fn network_info(&self) -> ProviderResult<NetworkInfo> {
+        let node_info = self.node_info().await?;
+        let chain_info = self.chain_info().await?;
+
+        Ok(NetworkInfo::new(node_info, chain_info))
+    }
+
     pub fn chain_id(&self) -> ChainId {
         self.consensus_parameters.chain_id
     }
diff --git a/packages/fuels-accounts/src/wallet.rs b/packages/fuels-accounts/src/wallet.rs
index 4aaa9f51c0..907abc63f1 100644
--- a/packages/fuels-accounts/src/wallet.rs
+++ b/packages/fuels-accounts/src/wallet.rs
@@ -260,8 +260,6 @@ impl Account for WalletUnlocked {
         previous_base_amount: u64,
     ) -> Result<Tb::TxType> {
         let consensus_parameters = self.try_provider()?.consensus_parameters();
-        tb = tb.with_consensus_parameters(consensus_parameters);
-
         self.sign_transaction(&mut tb);
 
         let new_base_amount =
diff --git a/packages/fuels-core/src/types/transaction_builders.rs b/packages/fuels-core/src/types/transaction_builders.rs
index 5e87f65211..076c076c8c 100644
--- a/packages/fuels-core/src/types/transaction_builders.rs
+++ b/packages/fuels-core/src/types/transaction_builders.rs
@@ -9,7 +9,7 @@ use fuel_tx::{
     Cacheable, ConsensusParameters, Create, Input as FuelInput, Output, Script, StorageSlot,
     Transaction as FuelTransaction, TransactionFee, TxPointer, UniqueIdentifier, Witness,
 };
-use fuel_types::{bytes::padded_len_usize, Bytes32, MemLayout, Salt};
+use fuel_types::{bytes::padded_len_usize, Bytes32, ChainId, MemLayout, Salt};
 use fuel_vm::{checked_transaction::EstimatePredicates, gas::GasCosts};
 
 use crate::{
@@ -28,6 +28,31 @@ use crate::{
     },
 };
 
+use super::{chain_info::ChainInfo, node_info::NodeInfo};
+
+#[derive(Debug, Clone)]
+pub struct NetworkInfo {
+    pub consensus_parameters: ConsensusParameters,
+    pub max_gas_per_tx: u64,
+    pub min_gas_price: u64,
+    pub gas_costs: GasCosts,
+}
+
+impl NetworkInfo {
+    pub fn new(node_info: NodeInfo, chain_info: ChainInfo) -> Self {
+        Self {
+            max_gas_per_tx: chain_info.consensus_parameters.max_gas_per_tx,
+            consensus_parameters: chain_info.consensus_parameters.into(),
+            min_gas_price: node_info.min_gas_price,
+            gas_costs: chain_info.gas_costs,
+        }
+    }
+
+    pub fn chain_id(&self) -> ChainId {
+        self.consensus_parameters.chain_id
+    }
+}
+
 #[derive(Debug, Clone, Default)]
 struct UnresolvedSignatures {
     addr_idx_offset_map: HashMap<Bech32Address, u8>,
@@ -47,7 +72,6 @@ pub trait TransactionBuilder: Send {
     fn with_inputs(self, inputs: Vec<Input>) -> Self;
     fn with_outputs(self, outputs: Vec<Output>) -> Self;
     fn with_witnesses(self, witnesses: Vec<Witness>) -> Self;
-    fn with_consensus_parameters(self, consensus_parameters: ConsensusParameters) -> Self;
     fn inputs(&self) -> &Vec<Input>;
     fn inputs_mut(&mut self) -> &mut Vec<Input>;
     fn outputs(&self) -> &Vec<Output>;
@@ -62,27 +86,21 @@ macro_rules! impl_tx_trait {
             type TxType = $tx_ty;
             fn build(self) -> Result<$tx_ty> {
                 let uses_predicates = self.is_using_predicates();
-                let (base_offset, consensus_parameters) = if uses_predicates {
-                    let consensus_params = self
-                        .consensus_parameters
-                        .ok_or(error!(
-                                TransactionBuildError,
-                                "predicate inputs require consensus parameters. Use `.set_consensus_parameters()`."))?;
-                    (self.base_offset(&consensus_params), consensus_params)
+                let base_offset = if uses_predicates {
+                    self.base_offset()
                 } else {
-                    // If no ConsensusParameters have been set, we can use the default instead of
-                    // erroring out since the tx doesn't use predicates
-                    (0, self.consensus_parameters.unwrap_or_default())
+                    0
                 };
 
+                let network_info = self.network_info.clone();
+
                 let num_witnesses = self.num_witnesses()?;
-                let mut tx =
-                    self.resolve_fuel_tx(base_offset, num_witnesses, &consensus_parameters)?;
+                let mut tx = self.resolve_fuel_tx(base_offset, num_witnesses)?;
 
-                tx.precompute(&consensus_parameters.chain_id)?;
+                tx.precompute(&network_info.chain_id())?;
 
                 if uses_predicates {
-                    estimate_predicates(&mut tx, &consensus_parameters)?;
+                    estimate_predicates(&mut tx, &network_info)?;
                 };
 
                 Ok($tx_ty { tx })
@@ -91,10 +109,15 @@ macro_rules! impl_tx_trait {
             fn add_unresolved_signature(&mut self, owner: Bech32Address, secret_key: SecretKey) {
                 let index_offset = self.unresolved_signatures.secret_keys.len() as u8;
                 self.unresolved_signatures.secret_keys.push(secret_key);
-                self.unresolved_signatures.addr_idx_offset_map.insert(owner, index_offset);
+                self.unresolved_signatures
+                    .addr_idx_offset_map
+                    .insert(owner, index_offset);
             }
 
-            fn fee_checked_from_tx(&self, params: &ConsensusParameters) -> Result<Option<TransactionFee>>{
+            fn fee_checked_from_tx(
+                &self,
+                params: &ConsensusParameters,
+            ) -> Result<Option<TransactionFee>> {
                 let tx = self.clone().build()?.tx;
                 Ok(TransactionFee::checked_from_tx(params, &tx))
             }
@@ -105,19 +128,20 @@ macro_rules! impl_tx_trait {
             }
 
             fn with_gas_price(mut self, gas_price: u64) -> Self {
-                self.gas_price = gas_price;
+                self.gas_price = Some(gas_price);
                 self
             }
 
             fn with_gas_limit(mut self, gas_limit: u64) -> Self {
-                self.gas_limit = gas_limit;
+                self.gas_limit = Some(gas_limit);
                 self
             }
 
-            fn with_tx_params(self, tx_params: TxParameters) -> Self {
-                self.with_gas_limit(tx_params.gas_limit())
-                    .with_gas_price(tx_params.gas_price())
-                    .with_maturity(tx_params.maturity().into())
+            fn with_tx_params(mut self, tx_params: TxParameters) -> Self {
+                self.gas_limit = tx_params.gas_limit();
+                self.gas_price = tx_params.gas_price();
+
+                self.with_maturity(tx_params.maturity().into())
             }
 
             fn with_inputs(mut self, inputs: Vec<Input>) -> Self {
@@ -135,14 +159,6 @@ macro_rules! impl_tx_trait {
                 self
             }
 
-            fn with_consensus_parameters(
-                mut self,
-                consensus_parameters: ConsensusParameters,
-            ) -> Self {
-                self.consensus_parameters = Some(consensus_parameters);
-                self
-            }
-
             fn inputs(&self) -> &Vec<Input> {
                 self.inputs.as_ref()
             }
@@ -176,12 +192,13 @@ macro_rules! impl_tx_trait {
             }
 
             fn num_witnesses(&self) -> Result<u8> {
-                let num_witnesses = self
-                    .witnesses()
-                    .len();
+                let num_witnesses = self.witnesses().len();
 
                 if num_witnesses + self.unresolved_signatures.secret_keys.len() > 256 {
-                    return Err(error!(InvalidData, "tx can not have more than 256 witnesses"));
+                    return Err(error!(
+                        InvalidData,
+                        "tx can not have more than 256 witnesses"
+                    ));
                 }
 
                 Ok(num_witnesses as u8)
@@ -190,24 +207,24 @@ macro_rules! impl_tx_trait {
     };
 }
 
-#[derive(Debug, Clone, Default)]
+#[derive(Debug, Clone)]
 pub struct ScriptTransactionBuilder {
-    pub gas_price: u64,
-    pub gas_limit: u64,
+    pub gas_price: Option<u64>,
+    pub gas_limit: Option<u64>,
     pub maturity: u32,
     pub script: Vec<u8>,
     pub script_data: Vec<u8>,
     pub inputs: Vec<Input>,
     pub outputs: Vec<Output>,
     pub witnesses: Vec<Witness>,
-    pub(crate) consensus_parameters: Option<ConsensusParameters>,
+    pub(crate) network_info: NetworkInfo,
     unresolved_signatures: UnresolvedSignatures,
 }
 
-#[derive(Debug, Clone, Default)]
+#[derive(Debug, Clone)]
 pub struct CreateTransactionBuilder {
-    pub gas_price: u64,
-    pub gas_limit: u64,
+    pub gas_price: Option<u64>,
+    pub gas_limit: Option<u64>,
     pub maturity: u32,
     pub bytecode_length: u64,
     pub bytecode_witness_index: u8,
@@ -216,7 +233,7 @@ pub struct CreateTransactionBuilder {
     pub outputs: Vec<Output>,
     pub witnesses: Vec<Witness>,
     pub salt: Salt,
-    pub(crate) consensus_parameters: Option<ConsensusParameters>,
+    pub(crate) network_info: NetworkInfo,
     unresolved_signatures: UnresolvedSignatures,
 }
 
@@ -224,15 +241,28 @@ impl_tx_trait!(ScriptTransactionBuilder, ScriptTransaction);
 impl_tx_trait!(CreateTransactionBuilder, CreateTransaction);
 
 impl ScriptTransactionBuilder {
-    fn resolve_fuel_tx(
-        self,
-        base_offset: usize,
-        num_witnesses: u8,
-        consensus_parameters: &ConsensusParameters,
-    ) -> Result<Script> {
+    pub fn new(network_info: NetworkInfo) -> ScriptTransactionBuilder {
+        ScriptTransactionBuilder {
+            gas_price: None,
+            gas_limit: None,
+            maturity: 0,
+            script: vec![],
+            script_data: vec![],
+            inputs: vec![],
+            outputs: vec![],
+            witnesses: vec![],
+            network_info,
+            unresolved_signatures: Default::default(),
+        }
+    }
+
+    fn resolve_fuel_tx(self, base_offset: usize, num_witnesses: u8) -> Result<Script> {
+        let gas_price = self.gas_price.unwrap_or(self.network_info.min_gas_price);
+        let gas_limit = self.gas_limit.unwrap_or(self.network_info.max_gas_per_tx);
+
         let mut tx = FuelTransaction::script(
-            self.gas_price,
-            self.gas_limit,
+            gas_price,
+            gas_limit,
             self.maturity.into(),
             self.script,
             self.script_data,
@@ -247,7 +277,7 @@ impl ScriptTransactionBuilder {
         );
 
         let missing_witnesses = generate_missing_witnesses(
-            tx.id(&consensus_parameters.chain_id),
+            tx.id(&self.network_info.chain_id()),
             &self.unresolved_signatures,
         );
         tx.witnesses_mut().extend(missing_witnesses);
@@ -255,8 +285,8 @@ impl ScriptTransactionBuilder {
         Ok(tx)
     }
 
-    fn base_offset(&self, consensus_parameters: &ConsensusParameters) -> usize {
-        offsets::base_offset_script(consensus_parameters)
+    fn base_offset(&self) -> usize {
+        offsets::base_offset_script(&self.network_info.consensus_parameters)
             + padded_len_usize(self.script_data.len())
             + padded_len_usize(self.script.len())
     }
@@ -275,8 +305,9 @@ impl ScriptTransactionBuilder {
         inputs: Vec<Input>,
         outputs: Vec<Output>,
         params: TxParameters,
+        network_info: NetworkInfo,
     ) -> Self {
-        ScriptTransactionBuilder::default()
+        ScriptTransactionBuilder::new(network_info)
             .with_inputs(inputs)
             .with_outputs(outputs)
             .with_tx_params(params)
@@ -290,6 +321,7 @@ impl ScriptTransactionBuilder {
         inputs: Vec<Input>,
         outputs: Vec<Output>,
         params: TxParameters,
+        network_info: NetworkInfo,
     ) -> Self {
         let script_data: Vec<u8> = [
             to.to_vec(),
@@ -317,7 +349,7 @@ impl ScriptTransactionBuilder {
         .into_iter()
         .collect();
 
-        ScriptTransactionBuilder::default()
+        ScriptTransactionBuilder::new(network_info)
             .with_tx_params(params)
             .with_script(script)
             .with_script_data(script_data)
@@ -331,6 +363,7 @@ impl ScriptTransactionBuilder {
         amount: u64,
         inputs: Vec<Input>,
         params: TxParameters,
+        network_info: NetworkInfo,
     ) -> Self {
         let script_data: Vec<u8> = [to.to_vec(), amount.to_be_bytes().to_vec()]
             .into_iter()
@@ -354,7 +387,7 @@ impl ScriptTransactionBuilder {
 
         let outputs = vec![Output::change(to, 0, BASE_ASSET_ID)];
 
-        ScriptTransactionBuilder::default()
+        ScriptTransactionBuilder::new(network_info)
             .with_tx_params(params)
             .with_script(script)
             .with_script_data(script_data)
@@ -364,16 +397,32 @@ impl ScriptTransactionBuilder {
 }
 
 impl CreateTransactionBuilder {
-    fn resolve_fuel_tx(
-        self,
-        base_offset: usize,
-        num_witnesses: u8,
-        consensus_parameters: &ConsensusParameters,
-    ) -> Result<Create> {
+    fn new(network_info: NetworkInfo) -> CreateTransactionBuilder {
+        CreateTransactionBuilder {
+            gas_price: None,
+            gas_limit: None,
+            maturity: 0,
+            bytecode_length: 0,
+            bytecode_witness_index: 0,
+            storage_slots: vec![],
+            salt: Default::default(),
+            inputs: vec![],
+            outputs: vec![],
+            witnesses: vec![],
+            network_info,
+            unresolved_signatures: Default::default(),
+        }
+    }
+
+    fn resolve_fuel_tx(self, base_offset: usize, num_witnesses: u8) -> Result<Create> {
         let num_of_storage_slots = self.storage_slots.len();
+
+        let gas_price = self.gas_price.unwrap_or(self.network_info.min_gas_price);
+        let gas_limit = self.gas_limit.unwrap_or(self.network_info.max_gas_per_tx);
+
         let mut tx = FuelTransaction::create(
-            self.gas_price,
-            self.gas_limit,
+            gas_price,
+            gas_limit,
             self.maturity.into(),
             self.bytecode_witness_index,
             self.salt,
@@ -389,7 +438,7 @@ impl CreateTransactionBuilder {
         );
 
         let missing_witnesses = generate_missing_witnesses(
-            tx.id(&consensus_parameters.chain_id),
+            tx.id(&self.network_info.chain_id()),
             &self.unresolved_signatures,
         );
 
@@ -398,8 +447,8 @@ impl CreateTransactionBuilder {
         Ok(tx)
     }
 
-    fn base_offset(&self, consensus_parameters: &ConsensusParameters) -> usize {
-        offsets::base_offset_create(consensus_parameters)
+    fn base_offset(&self) -> usize {
+        offsets::base_offset_create(&self.network_info.consensus_parameters)
     }
 
     pub fn with_bytecode_length(mut self, bytecode_length: u64) -> Self {
@@ -432,12 +481,13 @@ impl CreateTransactionBuilder {
         salt: Salt,
         storage_slots: Vec<StorageSlot>,
         params: TxParameters,
+        network_info: NetworkInfo,
     ) -> Self {
         let bytecode_witness_index = 0;
         let outputs = vec![Output::contract_created(contract_id, state_root)];
         let witnesses = vec![binary.into()];
 
-        CreateTransactionBuilder::default()
+        CreateTransactionBuilder::new(network_info)
             .with_tx_params(params)
             .with_bytecode_witness_index(bytecode_witness_index)
             .with_salt(salt)
@@ -653,18 +703,18 @@ fn generate_missing_witnesses(
         .collect()
 }
 
-fn estimate_predicates<T>(tx: &mut T, consensus_parameters: &ConsensusParameters) -> Result<()>
+fn estimate_predicates<T>(tx: &mut T, network_info: &NetworkInfo) -> Result<()>
 where
     T: GasLimit + GasPrice + EstimatePredicates,
 {
+    let consensus_parameters = &network_info.consensus_parameters;
+
     let gas_price = *tx.gas_price();
     let gas_limit = *tx.gas_limit();
     *tx.gas_price_mut() = 0;
     *tx.gas_limit_mut() = consensus_parameters.max_gas_per_tx;
 
-    // TODO: Fetch `GasCosts` from the `fuel-core`:
-    //  https://github.com/FuelLabs/fuel-core/issues/1221
-    tx.estimate_predicates(consensus_parameters, &GasCosts::default())?;
+    tx.estimate_predicates(consensus_parameters, &network_info.gas_costs)?;
     *tx.gas_price_mut() = gas_price;
     *tx.gas_limit_mut() = gas_limit;
 
@@ -681,8 +731,14 @@ mod tests {
         let unsorted_storage_slots = [2, 1].map(given_a_storage_slot).to_vec();
         let sorted_storage_slots = [1, 2].map(given_a_storage_slot).to_vec();
 
+        let network_info = NetworkInfo {
+            max_gas_per_tx: 0,
+            min_gas_price: 0,
+            gas_costs: Default::default(),
+            consensus_parameters: Default::default(),
+        };
         let builder =
-            CreateTransactionBuilder::default().with_storage_slots(unsorted_storage_slots);
+            CreateTransactionBuilder::new(network_info).with_storage_slots(unsorted_storage_slots);
 
         assert_eq!(builder.storage_slots, sorted_storage_slots);
     }
diff --git a/packages/fuels-core/src/types/wrappers/block.rs b/packages/fuels-core/src/types/wrappers/block.rs
index 080e0e1d68..b5b4d36c90 100644
--- a/packages/fuels-core/src/types/wrappers/block.rs
+++ b/packages/fuels-core/src/types/wrappers/block.rs
@@ -6,7 +6,7 @@ use fuel_core_client::client::types::{
     primitives::Bytes32,
 };
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct Header {
     pub id: Bytes32,
     pub da_height: u64,
@@ -40,7 +40,7 @@ impl From<ClientHeader> for Header {
     }
 }
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct Block {
     pub id: Bytes32,
     pub header: Header,
diff --git a/packages/fuels-core/src/types/wrappers/chain_info.rs b/packages/fuels-core/src/types/wrappers/chain_info.rs
index 5e9d04e400..d8c0525bc6 100644
--- a/packages/fuels-core/src/types/wrappers/chain_info.rs
+++ b/packages/fuels-core/src/types/wrappers/chain_info.rs
@@ -3,6 +3,7 @@
 use fuel_core_client::client::types::{
     chain_info::ChainInfo as ClientChainInfo, ConsensusParameters,
 };
+use fuel_vm::prelude::GasCosts;
 
 use crate::types::block::Block;
 
@@ -13,6 +14,7 @@ pub struct ChainInfo {
     pub peer_count: i32,
     pub latest_block: Block,
     pub consensus_parameters: ConsensusParameters,
+    pub gas_costs: GasCosts,
 }
 
 impl From<ClientChainInfo> for ChainInfo {
@@ -23,6 +25,7 @@ impl From<ClientChainInfo> for ChainInfo {
             peer_count: client_chain_info.peer_count,
             latest_block: client_chain_info.latest_block.into(),
             consensus_parameters: client_chain_info.consensus_parameters,
+            gas_costs: client_chain_info.gas_costs.into(),
         }
     }
 }
diff --git a/packages/fuels-core/src/types/wrappers/node_info.rs b/packages/fuels-core/src/types/wrappers/node_info.rs
index d5a8d9f27b..9a4bb8613c 100644
--- a/packages/fuels-core/src/types/wrappers/node_info.rs
+++ b/packages/fuels-core/src/types/wrappers/node_info.rs
@@ -2,7 +2,7 @@
 
 use fuel_core_client::client::types::node_info::NodeInfo as ClientNodeInfo;
 
-#[derive(Debug)]
+#[derive(Debug, Clone)]
 pub struct NodeInfo {
     pub utxo_validation: bool,
     pub vm_backtrace: bool,
diff --git a/packages/fuels-core/src/types/wrappers/transaction.rs b/packages/fuels-core/src/types/wrappers/transaction.rs
index 13707ed661..00f3fa907a 100644
--- a/packages/fuels-core/src/types/wrappers/transaction.rs
+++ b/packages/fuels-core/src/types/wrappers/transaction.rs
@@ -10,33 +10,17 @@ use fuel_tx::{
 };
 use fuel_types::ChainId;
 
-use crate::{
-    constants::{DEFAULT_GAS_LIMIT, DEFAULT_GAS_PRICE, DEFAULT_MATURITY},
-    types::Result,
-};
+use crate::types::Result;
 
-#[derive(Debug, Copy, Clone)]
+#[derive(Default, Debug, Copy, Clone)]
 pub struct TxParameters {
-    gas_price: u64,
-    gas_limit: u64,
+    gas_price: Option<u64>,
+    gas_limit: Option<u64>,
     maturity: u32,
 }
 
-macro_rules! impl_setter_getter {
-    ($name: ident, $field: ident, $ty: ty) => {
-        pub fn $name(mut self, $field: $ty) -> Self {
-            self.$field = $field;
-            self
-        }
-
-        pub fn $field(&self) -> $ty {
-            self.$field
-        }
-    };
-}
-
 impl TxParameters {
-    pub fn new(gas_price: u64, gas_limit: u64, maturity: u32) -> Self {
+    pub fn new(gas_price: Option<u64>, gas_limit: Option<u64>, maturity: u32) -> Self {
         Self {
             gas_price,
             gas_limit,
@@ -44,21 +28,34 @@ impl TxParameters {
         }
     }
 
-    impl_setter_getter!(with_gas_price, gas_price, u64);
-    impl_setter_getter!(with_gas_limit, gas_limit, u64);
-    impl_setter_getter!(with_maturity, maturity, u32);
-}
+    pub fn with_gas_price(mut self, gas_price: u64) -> Self {
+        self.gas_price = Some(gas_price);
+        self
+    }
 
-impl Default for TxParameters {
-    fn default() -> Self {
-        Self {
-            gas_price: DEFAULT_GAS_PRICE,
-            gas_limit: DEFAULT_GAS_LIMIT,
-            // By default, transaction is immediately valid
-            maturity: DEFAULT_MATURITY,
-        }
+    pub fn gas_price(&self) -> Option<u64> {
+        self.gas_price
+    }
+
+    pub fn with_gas_limit(mut self, gas_limit: u64) -> Self {
+        self.gas_limit = Some(gas_limit);
+        self
+    }
+
+    pub fn gas_limit(&self) -> Option<u64> {
+        self.gas_limit
+    }
+
+    pub fn with_maturity(mut self, maturity: u32) -> Self {
+        self.maturity = maturity;
+        self
+    }
+
+    pub fn maturity(&self) -> u32 {
+        self.maturity
     }
 }
+
 use fuel_tx::field::{BytecodeLength, BytecodeWitnessIndex, Salt, StorageSlots};
 
 #[derive(Debug, Clone)]
@@ -93,8 +90,6 @@ pub trait Transaction: Into<FuelTransaction> + Clone {
 
     fn with_gas_limit(self, gas_limit: u64) -> Self;
 
-    fn with_tx_params(self, tx_params: TxParameters) -> Self;
-
     fn metered_bytes_size(&self) -> usize;
 
     fn inputs(&self) -> &Vec<Input>;
@@ -191,13 +186,6 @@ impl Transaction for TransactionType {
         }
     }
 
-    fn with_tx_params(self, tx_params: TxParameters) -> Self {
-        match self {
-            TransactionType::Script(tx) => TransactionType::Script(tx.with_tx_params(tx_params)),
-            TransactionType::Create(tx) => TransactionType::Create(tx.with_tx_params(tx_params)),
-        }
-    }
-
     fn metered_bytes_size(&self) -> usize {
         match self {
             TransactionType::Script(tx) => tx.metered_bytes_size(),
@@ -308,12 +296,6 @@ macro_rules! impl_tx_wrapper {
                 self
             }
 
-            fn with_tx_params(self, tx_params: TxParameters) -> Self {
-                self.with_gas_limit(tx_params.gas_limit)
-                    .with_gas_price(tx_params.gas_price)
-                    .with_maturity(tx_params.maturity)
-            }
-
             fn metered_bytes_size(&self) -> usize {
                 self.tx.metered_bytes_size()
             }
diff --git a/packages/fuels-core/src/utils/constants.rs b/packages/fuels-core/src/utils/constants.rs
index 23673d4287..76228dbc9c 100644
--- a/packages/fuels-core/src/utils/constants.rs
+++ b/packages/fuels-core/src/utils/constants.rs
@@ -4,12 +4,6 @@ use fuel_types::AssetId;
 pub const ENUM_DISCRIMINANT_WORD_WIDTH: usize = 1;
 pub const WORD_SIZE: usize = core::mem::size_of::<Word>();
 
-// ANCHOR: default_tx_parameters
-pub const DEFAULT_GAS_PRICE: u64 = 0;
-pub const DEFAULT_GAS_LIMIT: u64 = 10_000_000;
-pub const DEFAULT_MATURITY: u32 = 0;
-// ANCHOR_END: default_tx_parameters
-
 // ANCHOR: default_call_parameters
 pub const DEFAULT_CALL_PARAMS_AMOUNT: u64 = 0;
 // Bytes representation of the asset ID of the "base" asset used for gas fees.
diff --git a/packages/fuels-programs/src/call_utils.rs b/packages/fuels-programs/src/call_utils.rs
index 6788be56f4..d8fdf9647d 100644
--- a/packages/fuels-programs/src/call_utils.rs
+++ b/packages/fuels-programs/src/call_utils.rs
@@ -128,9 +128,11 @@ pub(crate) async fn build_tx_from_contract_calls(
 
     let (inputs, outputs) = get_transaction_inputs_outputs(calls, asset_inputs, account);
 
-    let tb = ScriptTransactionBuilder::prepare_transfer(inputs, outputs, tx_parameters)
-        .with_script(script)
-        .with_script_data(script_data.clone());
+    let network_info = account.try_provider()?.network_info().await?;
+    let tb =
+        ScriptTransactionBuilder::prepare_transfer(inputs, outputs, tx_parameters, network_info)
+            .with_script(script)
+            .with_script_data(script_data.clone());
 
     let base_asset_amount = required_asset_amounts
         .iter()
diff --git a/packages/fuels-programs/src/contract.rs b/packages/fuels-programs/src/contract.rs
index 2f413dcaf9..c60f74dc58 100644
--- a/packages/fuels-programs/src/contract.rs
+++ b/packages/fuels-programs/src/contract.rs
@@ -298,6 +298,8 @@ impl Contract {
         account: &impl Account,
         tx_parameters: TxParameters,
     ) -> Result<Bech32ContractId> {
+        let network_info = account.try_provider()?.network_info().await?;
+
         let tb = CreateTransactionBuilder::prepare_contract_deployment(
             self.binary,
             self.contract_id,
@@ -305,6 +307,7 @@ impl Contract {
             self.salt,
             self.storage_slots,
             tx_parameters,
+            network_info,
         );
 
         let tx = account
diff --git a/packages/fuels-programs/src/script_calls.rs b/packages/fuels-programs/src/script_calls.rs
index df5026a8cc..e521de9e09 100644
--- a/packages/fuels-programs/src/script_calls.rs
+++ b/packages/fuels-programs/src/script_calls.rs
@@ -194,9 +194,15 @@ where
         )
         .collect();
 
-        let tb = ScriptTransactionBuilder::prepare_transfer(inputs, outputs, self.tx_parameters)
-            .with_script(self.script_call.script_binary.clone())
-            .with_script_data(self.compute_script_data().await?);
+        let network_info = self.account.try_provider()?.network_info().await?;
+        let tb = ScriptTransactionBuilder::prepare_transfer(
+            inputs,
+            outputs,
+            self.tx_parameters,
+            network_info,
+        )
+        .with_script(self.script_call.script_binary.clone())
+        .with_script_data(self.compute_script_data().await?);
 
         Ok(tb)
     }
diff --git a/packages/fuels/tests/predicates.rs b/packages/fuels/tests/predicates.rs
index a90b8ba356..bfe1fe2fa9 100644
--- a/packages/fuels/tests/predicates.rs
+++ b/packages/fuels/tests/predicates.rs
@@ -219,7 +219,7 @@ async fn pay_with_predicate() -> Result<()> {
     .await?;
 
     let contract_methods = MyContract::new(contract_id.clone(), predicate.clone()).methods();
-    let tx_params = TxParameters::new(1, 1000000, 0);
+    let tx_params = TxParameters::new(Some(1), Some(1000000), 0);
 
     assert_eq!(predicate.get_asset_balance(&BASE_ASSET_ID).await?, 192);
 
@@ -457,9 +457,13 @@ async fn predicate_transfer_with_signed_resources() -> Result<()> {
 
     let outputs = vec![Output::change(predicate.address().into(), 0, asset_id)];
 
-    let params = provider.consensus_parameters();
-    let mut tb = ScriptTransactionBuilder::prepare_transfer(inputs, outputs, Default::default())
-        .with_consensus_parameters(params);
+    let network_info = provider.network_info().await?;
+    let mut tb = ScriptTransactionBuilder::prepare_transfer(
+        inputs,
+        outputs,
+        Default::default(),
+        network_info,
+    );
     wallet.sign_transaction(&mut tb);
     let tx = tb.build()?;
 
@@ -691,13 +695,13 @@ async fn predicate_add_fee_persists_message_w_data() -> Result<()> {
     let provider = setup_test_provider(coins, vec![message.clone()], None, None).await;
     predicate.set_provider(provider.clone());
 
-    let params = provider.consensus_parameters();
+    let network_info = provider.network_info().await?;
     let tb = ScriptTransactionBuilder::prepare_transfer(
         vec![message_input.clone()],
         vec![],
         Default::default(),
-    )
-    .with_consensus_parameters(params);
+        network_info,
+    );
     let tx = predicate.add_fee_resources(tb, 1000).await?;
 
     assert_eq!(tx.inputs().len(), 2);
@@ -745,12 +749,13 @@ async fn predicate_transfer_non_base_asset() -> Result<()> {
         Output::change(wallet.address().into(), 0, BASE_ASSET_ID),
     ];
 
+    let network_info = provider.network_info().await?;
     let tb = ScriptTransactionBuilder::prepare_transfer(
         inputs,
         outputs,
         TxParameters::default().with_gas_price(1),
-    )
-    .with_consensus_parameters(provider.consensus_parameters());
+        network_info,
+    );
 
     let tx = wallet.add_fee_resources(tb, 0).await?;
 
diff --git a/packages/fuels/tests/providers.rs b/packages/fuels/tests/providers.rs
index 0604f994e6..b1eccb27a1 100644
--- a/packages/fuels/tests/providers.rs
+++ b/packages/fuels/tests/providers.rs
@@ -14,7 +14,10 @@ use fuels::{
     tx::Receipt,
     types::{block::Block, coin_type::CoinType, errors::error, message::Message},
 };
-use fuels_core::types::Bits256;
+use fuels_core::types::{
+    transaction_builders::{ScriptTransactionBuilder, TransactionBuilder},
+    Bits256,
+};
 
 #[tokio::test]
 async fn test_provider_launch_and_connect() -> Result<()> {
@@ -322,6 +325,33 @@ async fn contract_deployment_respects_maturity() -> Result<()> {
     Ok(())
 }
 
+#[tokio::test]
+async fn test_default_tx_params_match_network() -> Result<()> {
+    let wallet = launch_provider_and_get_wallet().await;
+    let provider = wallet.try_provider()?;
+    let consensus_params = provider.consensus_parameters();
+
+    let inputs = wallet
+        .get_asset_inputs_for_amount(BASE_ASSET_ID, 100)
+        .await?;
+    let outputs =
+        wallet.get_asset_outputs_for_amount(&Bech32Address::default(), BASE_ASSET_ID, 100);
+
+    let network_info = provider.network_info().await?;
+    let mut tb = ScriptTransactionBuilder::prepare_transfer(
+        inputs,
+        outputs,
+        TxParameters::default(),
+        network_info,
+    );
+    wallet.sign_transaction(&mut tb);
+    let tx = tb.build()?;
+
+    assert_eq!(tx.gas_limit(), consensus_params.max_gas_per_tx);
+
+    Ok(())
+}
+
 #[tokio::test]
 async fn test_gas_forwarded_defaults_to_tx_limit() -> Result<()> {
     setup_program_test!(
diff --git a/packages/fuels/tests/wallets.rs b/packages/fuels/tests/wallets.rs
index ee9f544af3..897a23775d 100644
--- a/packages/fuels/tests/wallets.rs
+++ b/packages/fuels/tests/wallets.rs
@@ -137,7 +137,13 @@ async fn add_fee_resources_empty_transaction() -> Result<()> {
         .pop()
         .unwrap();
 
-    let tb = ScriptTransactionBuilder::prepare_transfer(vec![], vec![], TxParameters::default());
+    let network_info = wallet.try_provider()?.network_info().await?;
+    let tb = ScriptTransactionBuilder::prepare_transfer(
+        vec![],
+        vec![],
+        TxParameters::default(),
+        network_info,
+    );
     let tx = wallet.add_fee_resources(tb, 0).await?;
 
     let zero_utxo_id = UtxoId::new(Bytes32::zeroed(), 0);
@@ -173,7 +179,13 @@ async fn add_fee_resources_to_transfer_with_base_asset() -> Result<()> {
     let outputs =
         wallet.get_asset_outputs_for_amount(&Address::zeroed().into(), BASE_ASSET_ID, base_amount);
 
-    let tb = ScriptTransactionBuilder::prepare_transfer(inputs, outputs, TxParameters::default());
+    let network_info = wallet.try_provider()?.network_info().await?;
+    let tb = ScriptTransactionBuilder::prepare_transfer(
+        inputs,
+        outputs,
+        TxParameters::default(),
+        network_info,
+    );
     let tx = wallet.add_fee_resources(tb, base_amount).await?;
 
     let zero_utxo_id = UtxoId::new(Bytes32::zeroed(), 0);
@@ -245,7 +257,7 @@ async fn send_transfer_transactions() -> Result<()> {
     let gas_limit = 500_000;
     let maturity = 0u32;
 
-    let tx_params = TxParameters::new(gas_price, gas_limit, maturity);
+    let tx_params = TxParameters::new(Some(gas_price), Some(gas_limit), maturity);
 
     // Transfer 1 from wallet 1 to wallet 2.
     const SEND_AMOUNT: u64 = 1;
@@ -434,8 +446,13 @@ async fn test_transfer_with_multiple_signatures() -> Result<()> {
         amount_to_receive,
     );
 
-    let mut tb =
-        ScriptTransactionBuilder::prepare_transfer(inputs, outputs, TxParameters::default());
+    let network_info = provider.network_info().await?;
+    let mut tb = ScriptTransactionBuilder::prepare_transfer(
+        inputs,
+        outputs,
+        TxParameters::default(),
+        network_info,
+    );
 
     for wallet in wallets.iter() {
         wallet.sign_transaction(&mut tb);