fix: provider auto-refetch (#3869)

* Removing duplicated operation

* Fixing `dryRun` type (it's not a mutation anymore)

* Adding `latestBlock` props to `chainInfo` fragment

* Considering `latestBlock` info when serializing/deserializing `chainInfo`

* Using correct `consensusParametersVersion`

* Fixing (false positive) test

* Adding changeset

* Lintfix (updating fixture)

* Trigger CI

Author: arboleya

.changeset/khaki-zoos-cough.md

@@ -0,0 +1,5 @@
+---
+"@fuel-ts/account": patch
+---
+
+fix: provider auto-refetch

packages/account/src/providers/operations.graphql

@@ -591,6 +591,11 @@ fragment chainInfoFragment on ChainInfo {
   consensusParameters {
     ...consensusParametersFragment
   }
+  latestBlock {
+    header {
+      consensusParametersVersion
+    }
+  }
 }
 
 fragment contractBalanceFragment on ContractBalance {
@@ -995,12 +1000,6 @@ query getAssetDetails($assetId: AssetId!) {
   }
 }
 
-query daCompressedBlock($height: U32!) {
-  daCompressedBlock(height: $height) {
-    bytes
-  }
-}
-
 query assembleTx(
   $tx: HexString!
   $blockHorizon: U32!
@@ -1071,7 +1070,7 @@ query assembleTx(
   }
 }
 
-mutation dryRun(
+query dryRun(
   $encodedTransactions: [HexString!]!
   $utxoValidation: Boolean
   $gasPrice: U64

packages/account/src/providers/provider.test.ts

@@ -2312,10 +2312,23 @@ describe('Provider', () => {
     const provider = new Provider(launched.provider.url);
     const fetchChainAndNodeInfo = vi.spyOn(provider, 'fetchChainAndNodeInfo');
 
-    // calling twice
+    // calling once to perform the first fetch
     await provider.autoRefetchConfigs();
+
+    // reset timestamp to force a refetch
     provider.consensusParametersTimestamp = 0;
 
+    // mock the `getConsensusParametersVersion` to return a different value,
+    // thus forcing a refetch (the first value will be zero)
+    vi.spyOn(provider.operations, 'getConsensusParametersVersion').mockResolvedValue({
+      chain: {
+        latestBlock: {
+          header: { consensusParametersVersion: '1' },
+        },
+      },
+    });
+
+    // calling again to perform the second fetch
     await provider.autoRefetchConfigs();
 
     expect(fetchChainAndNodeInfo).toHaveBeenCalledTimes(2);

packages/account/src/providers/provider.ts

@@ -249,6 +249,11 @@ export type ChainInfo = {
   name: string;
   baseChainHeight: BN;
   consensusParameters: ConsensusParameters;
+  latestBlock: {
+    header: {
+      consensusParametersVersion: string;
+    };
+  };
 };
 
 /**
@@ -1465,7 +1470,9 @@ export default class Provider {
     const chainInfo = Provider.chainInfoCache[this.urlWithoutAuth];
 
     const {
-      consensusParameters: { version: previous },
+      latestBlock: {
+        header: { consensusParametersVersion: previous },
+      },
     } = chainInfo;
 
     const {

packages/account/src/providers/utils/serialization.ts

@@ -80,7 +80,7 @@ export type TransactionSummaryJsonPartial = Omit<TransactionSummaryJson, 'id' |
 
 /** @hidden */
 export const deserializeChain = (chain: ChainInfoJson): ChainInfo => {
-  const { name, daHeight, consensusParameters } = chain;
+  const { name, daHeight, consensusParameters, latestBlock } = chain;
 
   const {
     contractParams,
@@ -134,12 +134,13 @@ export const deserializeChain = (chain: ChainInfoJson): ChainInfo => {
       },
       gasCosts,
     },
+    latestBlock,
   };
 };
 
 /** @hidden */
 export const serializeChain = (chain: ChainInfo): ChainInfoJson => {
-  const { name, baseChainHeight, consensusParameters } = chain;
+  const { name, baseChainHeight, consensusParameters, latestBlock } = chain;
 
   const {
     contractParameters,
@@ -193,6 +194,7 @@ export const serializeChain = (chain: ChainInfo): ChainInfoJson => {
       },
       gasCosts,
     },
+    latestBlock,
   };
 };
 

packages/account/test/fixtures/chain.ts

@@ -72,4 +72,9 @@ export const MOCK_CHAIN: GqlChainInfoFragment = {
       newStoragePerByte: '1',
     },
   },
+  latestBlock: {
+    header: {
+      consensusParametersVersion: '0',
+    },
+  },
 };