feat(protocol-kit): Migrate Safe Proxy Factory contract to Abitype

packages/protocol-kit/scripts/generateTypechainFiles.ts

@@ -17,25 +17,21 @@ const safeContractsPath = '../../node_modules/@safe-global/safe-deployments/dist
 
 const safeContracts_V1_4_1 = [
   // `${safeContractsPath}/v1.4.1/safe.json`, // Remove contract 1.4.1 from typechain as it's migrated to Abitype
-  `${safeContractsPath}/v1.4.1/safe_proxy_factory.json`,
+  // `${safeContractsPath}/v1.4.1/safe_proxy_factory.json`, // Remove contract 1.4.1 from typechain as it's migrated to Abitype
   `${safeContractsPath}/v1.4.1/compatibility_fallback_handler.json`,
   `${safeContractsPath}/v1.4.1/create_call.json`,
   `${safeContractsPath}/v1.4.1/simulate_tx_accessor.json`
 ].join(' ')
 const safeContracts_V1_3_0 = [
   // `${safeContractsPath}/v1.3.0/gnosis_safe.json`, // Remove contract 1.3.0 from typechain as it's migrated to Abitype
-  `${safeContractsPath}/v1.3.0/proxy_factory.json`,
+  // `${safeContractsPath}/v1.3.0/proxy_factory.json`, // Remove contract 1.3.0 from typechain as it's migrated to Abitype
   `${safeContractsPath}/v1.3.0/compatibility_fallback_handler.json`,
   `${safeContractsPath}/v1.3.0/create_call.json`,
   `${safeContractsPath}/v1.3.0/simulate_tx_accessor.json`
 ].join(' ')
-const safeContracts_V1_1_1 = [
-  // `${safeContractsPath}/v1.1.1/gnosis_safe.json`, // Remove contract 1.1.1 from typechain as it's migrated to Abitype,
-  `${safeContractsPath}/v1.1.1/proxy_factory.json`
-].join(' ')
 const safeContracts_V1_0_0 = [
-  `${safeContractsPath}/v1.0.0/gnosis_safe.json`,
-  `${safeContractsPath}/v1.0.0/proxy_factory.json`
+  `${safeContractsPath}/v1.0.0/gnosis_safe.json`
+  // `${safeContractsPath}/v1.0.0/proxy_factory.json` // Remove contract 1.0.0 from typechain as it's migrated to Abitype
 ].join(' ')
 
 // Won't be included in dist/ folder
@@ -90,7 +86,7 @@ function moveTypechainFiles(inDir: string, outDir: string): void {
   })
 }
 
-// Contract 1.2.0 is migrated to Abitype already, so it's not included in here
+// Contracts v1.1.1 + v1.2.0 are migrated to Abitype already, so they're not included in here
 function generateTypes(typechainTarget: string) {
   // Src
   generateTypechainFiles(
@@ -103,11 +99,6 @@ function generateTypes(typechainTarget: string) {
     `${outDirSrc}${typechainTarget}/v1.3.0`,
     safeContracts_V1_3_0
   )
-  generateTypechainFiles(
-    typechainTarget,
-    `${outDirSrc}${typechainTarget}/v1.1.1`,
-    safeContracts_V1_1_1
-  )
   generateTypechainFiles(
     typechainTarget,
     `${outDirSrc}${typechainTarget}/v1.0.0`,
@@ -121,10 +112,6 @@ function generateTypes(typechainTarget: string) {
     `${typeChainDirectorySrcPath}${typechainTarget}/v1.3.0`,
     `${typeChainDirectoryBuildPath}${typechainTarget}/v1.3.0`
   )
-  moveTypechainFiles(
-    `${typeChainDirectorySrcPath}${typechainTarget}/v1.1.1`,
-    `${typeChainDirectoryBuildPath}${typechainTarget}/v1.1.1`
-  )
   moveTypechainFiles(
     `${typeChainDirectorySrcPath}${typechainTarget}/v1.0.0`,
     `${typeChainDirectoryBuildPath}${typechainTarget}/v1.0.0`

packages/protocol-kit/src/adapters/ethers/EthersAdapter.ts

@@ -11,7 +11,6 @@ import { ethers, TransactionResponse, AbstractSigner, Provider } from 'ethers'
 import CompatibilityFallbackHandlerContractEthers from './contracts/CompatibilityFallbackHandler/CompatibilityFallbackHandlerEthersContract'
 import CreateCallEthersContract from './contracts/CreateCall/CreateCallEthersContract'
 import SafeContractEthers from './contracts/Safe/SafeContractEthers'
-import SafeProxyFactoryEthersContract from './contracts/SafeProxyFactory/SafeProxyFactoryEthersContract'
 import SimulateTxAccessorEthersContract from './contracts/SimulateTxAccessor/SimulateTxAccessorEthersContract'
 import {
   getCompatibilityFallbackHandlerContractInstance,
@@ -122,16 +121,23 @@ class EthersAdapter implements EthAdapter {
   async getSafeProxyFactoryContract({
     safeVersion,
     singletonDeployment,
-    customContractAddress
-  }: GetContractProps): Promise<SafeProxyFactoryEthersContract> {
+    customContractAddress,
+    customContractAbi
+  }: GetContractProps) {
     const chainId = await this.getChainId()
     const contractAddress =
       customContractAddress ?? singletonDeployment?.networkAddresses[chainId.toString()]
     if (!contractAddress) {
       throw new Error('Invalid SafeProxyFactory contract address')
     }
     const signerOrProvider = this.#signer || this.#provider
-    return getSafeProxyFactoryContractInstance(safeVersion, contractAddress, signerOrProvider)
+    return getSafeProxyFactoryContractInstance(
+      safeVersion,
+      contractAddress,
+      signerOrProvider,
+      this,
+      customContractAbi
+    )
   }
 
   async getMultiSendContract({

packages/protocol-kit/src/adapters/ethers/contracts/SafeProxyFactory/safeProxyFactoryBaseContractEthers.ts

@@ -0,0 +1,62 @@
+import { Contract, ContractRunner, InterfaceAbi } from 'ethers'
+
+import EthersAdapter from '@safe-global/protocol-kit/adapters/ethers/EthersAdapter'
+import SafeProxyFactoryBaseContract from '@safe-global/protocol-kit/adapters/safeProxyFactoryBaseContract'
+import { SafeVersion } from '@safe-global/safe-core-sdk-types'
+
+/**
+ * Abstract class SafeProxyFactoryBaseContractEthers extends SafeProxyFactoryBaseContract to specifically integrate with the Ethers.js v6 library.
+ * It is designed to be instantiated for different versions of the Safe contract.
+ *
+ * This abstract class sets up the Ethers v6 Contract object that interacts with a Safe Proxy Factory contract version.
+ *
+ * Subclasses of SafeProxyFactoryBaseContractEthers are expected to represent specific versions of the contract.
+ *
+ * @template SafeProxyFactoryContractAbiType - The ABI type specific to the version of the Safe Proxy Factory contract, extending InterfaceAbi from Ethers.
+ * @extends SafeProxyFactoryBaseContract<SafeProxyFactoryContractAbiType> - Extends the generic SafeProxyFactoryBaseContract with Ethers-specific implementation.
+ *
+ * Example subclasses:
+ * - SafeProxyFactory_v1_4_1_Ethers extends SafeProxyFactoryBaseContractEthers<SafeProxyFactoryContract_v1_4_1_Abi>
+ * - SafeProxyFactory_v1_3_0_Ethers extends SafeProxyFactoryBaseContractEthers<SafeProxyFactoryContract_v1_3_0_Abi>
+ * - SafeProxyFactory_v1_2_0_Ethers extends SafeProxyFactoryBaseContractEthers<SafeProxyFactoryContract_v1_2_0_Abi>
+ * - SafeProxyFactory_v1_1_1_Ethers extends SafeProxyFactoryBaseContractEthers<SafeProxyFactoryContract_v1_1_1_Abi>
+ * - SafeProxyFactory_v1_0_0_Ethers extends SafeProxyFactoryBaseContractEthers<SafeProxyFactoryContract_v1_0_0_Abi>
+ */
+abstract class SafeProxyFactoryBaseContractEthers<
+  SafeProxyFactoryContractAbiType extends InterfaceAbi
+> extends SafeProxyFactoryBaseContract<SafeProxyFactoryContractAbiType> {
+  contract: Contract
+  adapter: EthersAdapter
+
+  /**
+   * @constructor
+   * Constructs an instance of SafeProxyFactoryBaseContractEthers.
+   *
+   * @param chainId - The chain ID of the contract.
+   * @param ethersAdapter - An instance of EthersAdapter.
+   * @param defaultAbi - The default ABI for the Safe contract. It should be compatible with the specific version of the contract.
+   * @param safeVersion - The version of the Safe contract.
+   * @param customContractAddress - Optional custom address for the contract. If not provided, the address is derived from the Safe deployments based on the chainId and safeVersion.
+   * @param customContractAbi - Optional custom ABI for the contract. If not provided, the ABI is derived from the Safe deployments or the defaultAbi is used.
+   */
+  constructor(
+    chainId: bigint,
+    ethersAdapter: EthersAdapter,
+    defaultAbi: SafeProxyFactoryContractAbiType,
+    safeVersion: SafeVersion,
+    customContractAddress?: string,
+    customContractAbi?: SafeProxyFactoryContractAbiType,
+    runner?: ContractRunner | null
+  ) {
+    super(chainId, defaultAbi, safeVersion, customContractAddress, customContractAbi)
+
+    this.adapter = ethersAdapter
+    this.contract = new Contract(
+      this.contractAddress,
+      this.contractAbi,
+      runner || this.adapter.getSigner()
+    )
+  }
+}
+
+export default SafeProxyFactoryBaseContractEthers