createAccessList pending default

Author: TateB

src/components/@molecules/TransactionDialogManager/stage/TransactionStageModal.test.tsx

@@ -14,7 +14,7 @@ import { useAddRecentTransaction } from '@app/hooks/transactions/useAddRecentTra
 import { useRecentTransactions } from '@app/hooks/transactions/useRecentTransactions'
 import { useIsSafeApp } from '@app/hooks/useIsSafeApp'
 import { GenericTransaction } from '@app/transaction-flow/types'
-import { getAccessList } from '@app/utils/query/getAccessList'
+import { createAccessList } from '@app/utils/query/createAccessList'
 import { checkIsSafeApp } from '@app/utils/safe'
 
 import { makeMockIntersectionObserver } from '../../../../../test/mock/makeMockIntersectionObserver'
@@ -29,7 +29,7 @@ vi.mock('@app/hooks/transactions/useAddRecentTransaction')
 vi.mock('@app/hooks/transactions/useRecentTransactions')
 vi.mock('@app/hooks/chain/useInvalidateOnBlock')
 vi.mock('@app/utils/safe')
-vi.mock('@app/utils/query/getAccessList')
+vi.mock('@app/utils/query/createAccessList')
 vi.mock('@wagmi/core', async () => {
   const actual = await vi.importActual('@wagmi/core')
   return {
@@ -81,7 +81,7 @@ const mockUseClient = mockFunction(useClient)
 const mockUseConnectorClient = mockFunction(useConnectorClient)
 
 const mockEstimateGas = mockFunction(estimateGas)
-const mockGetAccessList = getAccessList as MockedFunctionDeep<typeof getAccessList>
+const mockCreateAccessList = createAccessList as MockedFunctionDeep<typeof createAccessList>
 const mockPrepareTransactionRequest = prepareTransactionRequest as MockedFunctionDeep<
   typeof prepareTransactionRequest
 >
@@ -211,7 +211,7 @@ describe('TransactionStageModal', () => {
       })
 
       it('should disable confirm button and re-estimate gas if a unique identifier is changed', async () => {
-        mockGetAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
+        mockCreateAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
         mockEstimateGas.mockResolvedValue(1n)
         mockUseIsSafeApp.mockReturnValue({ data: false })
         mockUseSendTransaction.mockReturnValue({
@@ -240,7 +240,7 @@ describe('TransactionStageModal', () => {
       })
 
       it('should only show confirm button as enabled if gas is estimated and sendTransaction func is defined', async () => {
-        mockGetAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
+        mockCreateAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
         mockEstimateGas.mockResolvedValue(1n)
         mockUseSendTransaction.mockReturnValue({
           sendTransaction: () => Promise.resolve(),
@@ -251,7 +251,7 @@ describe('TransactionStageModal', () => {
         )
       })
       it('should run set sendTransaction on action click', async () => {
-        mockGetAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
+        mockCreateAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
         mockEstimateGas.mockResolvedValue(1n)
         const mockSendTransaction = vi.fn()
         mockUseSendTransaction.mockReturnValue({
@@ -292,7 +292,7 @@ describe('TransactionStageModal', () => {
       })
       it('should pass the request to send transaction', async () => {
         mockUseIsSafeApp.mockReturnValue({ data: false })
-        mockGetAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
+        mockCreateAccessList.mockResolvedValue({ accessList: [], gasUsed: '0x64' })
         mockEstimateGas.mockResolvedValue(1n)
         const mockSendTransaction = vi.fn()
         mockUseSendTransaction.mockReturnValue({
@@ -525,7 +525,7 @@ describe('calculateGasLimit', () => {
 
   it('should calculate gas limit', async () => {
     mockEstimateGas.mockResolvedValueOnce(100000n)
-    mockGetAccessList.mockResolvedValueOnce(mockAccessListResponse)
+    mockCreateAccessList.mockResolvedValueOnce(mockAccessListResponse)
     const result = await calculateGasLimit({
       txWithZeroGas: mockTxWithZeroGas,
       transactionName: mockTransactionName,

src/components/@molecules/TransactionDialogManager/stage/query.ts

@@ -24,7 +24,7 @@ import {
   CreateQueryKey,
 } from '@app/types'
 import { getReadableError } from '@app/utils/errors'
-import { getAccessList } from '@app/utils/query/getAccessList'
+import { createAccessList } from '@app/utils/query/createAccessList'
 import { wagmiConfig } from '@app/utils/query/wagmi'
 import { hasParaConnection } from '@app/utils/utils'
 
@@ -111,7 +111,7 @@ export const calculateGasLimit = async ({
   txWithZeroGas: BasicTransactionRequest
   transactionName: TransactionName
 }) => {
-  const accessListResponse = await getAccessList(client, {
+  const accessListResponse = await createAccessList(client, {
     to: txWithZeroGas.to,
     data: txWithZeroGas.data,
     from: connectorClient.account!.address,

src/hooks/chain/useEstimateGasWithStateOverride.ts

@@ -33,7 +33,7 @@ import {
 import { emptyAddress } from '@app/utils/constants'
 import { getIsCachedData } from '@app/utils/getIsCachedData'
 import { prepareQueryOptions } from '@app/utils/prepareQueryOptions'
-import { getAccessList } from '@app/utils/query/getAccessList'
+import { createAccessList } from '@app/utils/query/createAccessList'
 import { useQuery } from '@app/utils/query/useQuery'
 
 import { useGasPrice } from './useGasPrice'
@@ -176,7 +176,7 @@ const estimateIndividualGas = async <TName extends TransactionName>({
   // To get the access list, we're executing the bytecode of this Yul code: https://gist.github.com/TateB/777287c9a63d5f02fcd905232ce5748a
   // (note: 0xed3869F3020315C839b2f4E9a73bEbE9a9670534 is replaced with `connectorClient.account.address`)
   // It does a simple transfer, and accesses any storage slot that would be accessed by any other transfer.
-  const accessList = await getAccessList(client, {
+  const accessList = await createAccessList(client, {
     from: emptyAddress,
     data: concatHex(['0x5f808080600173', connectorClient.account.address, '0x5af100']),
     value: '0x1',

src/utils/query/getAccessList.ts src/utils/query/createAccessList.tssrc/utils/query/getAccessList.ts renamed to src/utils/query/createAccessList.ts

@@ -8,10 +8,13 @@ type AccessListResponse = {
   gasUsed: Hex
 }
 
-export const getAccessList = async (
+export const createAccessList = async (
   client: Client,
-  tx: TransactionRequest<Hex>,
+  tx: TransactionRequest<Hex> & {
+    blockTag?: BlockTag
+  },
 ): Promise<AccessListResponse> => {
+  const blockTag = tx.blockTag ?? 'pending'
   const accessListResponse = await client.request<{
     Method: 'eth_createAccessList'
     Parameters: [tx: TransactionRequest<Hex>, blockTag: BlockTag]
@@ -25,7 +28,7 @@ export const getAccessList = async (
         from: tx.from,
         value: tx.value,
       },
-      'latest',
+      blockTag,
     ],
   })