fix(common): allow overriding fees in writeContract and sendTransacti…

…on (#3288)

.changeset/spotty-crabs-prove.md

@@ -0,0 +1,7 @@
+---
+"@latticexyz/common": patch
+---
+
+The `transactionQueue` decorator internally keeps an updated reference for the recommended `baseFeePerGas` and `maxPriorityFeePerGas` from the connected chain to avoid having to fetch it right before sending a transaction.
+However, due to the way the fee values were overridden, it wasn't possible for users to explicitly pass in custom fee values.
+Now explicitly provided fee values have precedence over the internally estimated fee values.

packages/common/src/sendTransaction.ts

@@ -74,9 +74,9 @@ export async function sendTransaction<
           const params = {
             // viem_sendTransaction internally estimates gas, which we want to happen on the pending block
             blockTag: "pending",
+            ...feeRef.fees,
             ...request,
             nonce,
-            ...feeRef.fees,
           } as const satisfies SendTransactionParameters<chain, account, chainOverride, request>;
           debug("sending tx to", request.to, "with nonce", nonce);
           return await getAction(client, viem_sendTransaction, "sendTransaction")(params as never);

packages/common/src/writeContract.ts

@@ -78,9 +78,9 @@ export async function writeContract<
           const params = {
             // viem_writeContract internally estimates gas, which we want to happen on the pending block
             blockTag: "pending",
+            ...feeRef.fees,
             ...request,
             nonce,
-            ...feeRef.fees,
           } as const satisfies WriteContractParameters<abi, functionName, args, chain, account, chainOverride>;
           debug("calling", params.functionName, "at", params.address, "with nonce", nonce);
           return await getAction(client, viem_writeContract, "writeContract")(params as never);