From c462f074dcd5cf7c14ac77fd49cd59e8a61c7441 Mon Sep 17 00:00:00 2001
From: seolaoh <osa8361@gmail.com>
Date: Fri, 2 Aug 2024 17:53:41 +0900
Subject: [PATCH] fix(contracts): not upgrade already deployed contract

---
 packages/contracts/src/deploy-utils.ts | 17 ++++++++++++-----
 1 file changed, 12 insertions(+), 5 deletions(-)

diff --git a/packages/contracts/src/deploy-utils.ts b/packages/contracts/src/deploy-utils.ts
index 4e047ad47..be2cd3642 100644
--- a/packages/contracts/src/deploy-utils.ts
+++ b/packages/contracts/src/deploy-utils.ts
@@ -43,7 +43,10 @@ export const deploy = async (
   name: string,
   opts: DeployOptions = {}
 ): Promise<Contract | null> => {
-  const created = await deployImpl(hre, name, opts)
+  const [created, newlyDeployed] = await deployImpl(hre, name, opts)
+  if (!newlyDeployed) {
+    return created
+  }
 
   if (opts.isProxyImpl) {
     const { deployer } = await hre.getNamedAccounts()
@@ -107,7 +110,10 @@ export const deployAndUpgradeByDeployer = async (
   name: string,
   opts: DeployOptions = {}
 ): Promise<Contract | null> => {
-  const created = await deployImpl(hre, name, opts)
+  const [created, newlyDeployed] = await deployImpl(hre, name, opts)
+  if (!newlyDeployed) {
+    return created
+  }
 
   const { deployer } = await hre.getNamedAccounts()
   const proxyName = name + 'Proxy'
@@ -158,12 +164,13 @@ export const deployAndUpgradeByDeployer = async (
  * @param opts.args Arguments to pass to the contract constructor.
  * @param opts.postDeployAction Action to perform after the contract is deployed.
  * @returns A deployed contract object.
+ * @returns If the contract is newly deployed or not.
  */
 const deployImpl = async (
   hre: HardhatRuntimeEnvironment,
   name: string,
   opts: DeployOptions = {}
-): Promise<Contract | null> => {
+): Promise<[Contract | null, boolean]> => {
   const { deployer } = await hre.getNamedAccounts()
 
   // Wrap in a try/catch in case there is not a deployConfig for the current network.
@@ -194,7 +201,7 @@ const deployImpl = async (
 
   // If the contract is not newly deployed, do not proceed further.
   if (!result.newlyDeployed) {
-    return created
+    return [created, false]
   }
 
   // Always wait for the transaction to be mined, just in case.
@@ -211,7 +218,7 @@ const deployImpl = async (
     await opts.postDeployAction(created)
   }
 
-  return created
+  return [created, true]
 }
 
 /**