feat: update pool collateral configuration logic

Synthetixio · Aug 15, 2023 · 7f36cf0 · 7f36cf0
1 parent 14f0ca8
commit 7f36cf0
Show file tree

Hide file tree

Showing 9 changed files with 44 additions and 67 deletions.
diff --git a/protocol/synthetix/contracts/interfaces/IPoolModule.sol b/protocol/synthetix/contracts/interfaces/IPoolModule.sol
@@ -118,6 +118,18 @@ interface IPoolModule {
         PoolCollateralConfiguration.Data memory newConfig
     ) external;
 
+    /**
+     * @notice Allows collaterals accepeted by the system will be accpeted by the pool by default
+     * @param poolId The id of the pool.
+     */
+    function enablePoolCollateralByDefault(uint128 poolId) external;
+
+    /**
+     * @notice Prevents collaterals accepeted by the system will not be accpeted by the pool by default
+     * @param poolId The id of the pool.
+     */
+    function disablePoolCollateralByDefault(uint128 poolId) external;
+
     /**
      * @notice Retrieves the MarketConfiguration of the specified pool.
      * @param poolId The id of the pool whose configuration is being queried.
@@ -186,16 +198,6 @@ interface IPoolModule {
      */
     function setMinLiquidityRatio(uint256 minLiquidityRatio) external;
 
-    /**
-     @notice Shows if a given collateral type is enabled for deposits and delegation in a given pool.
-     * @param poolId The id of the pool for to check the collateral for.
-     * @param collateral The address of the collateral.
-     */
-    function isDelegationEnabledByPool(
-        uint128 poolId,
-        address collateral
-    ) external view returns (bool enabled);
-
     /**
      @notice returns a pool minimum issuance ratio
      * @param poolId The id of the pool for to check the collateral for.

diff --git a/protocol/synthetix/contracts/modules/core/PoolModule.sol b/protocol/synthetix/contracts/modules/core/PoolModule.sol
@@ -197,6 +197,24 @@ contract PoolModule is IPoolModule {
         emit PoolConfigurationSet(poolId, newMarketConfigurations, msg.sender);
     }
 
+    /**
+     * @inheritdoc IPoolModule
+     */
+    function disablePoolCollateralByDefault(uint128 poolId) external override {
+        Pool.Data storage pool = Pool.loadExisting(poolId);
+
+        pool.collateralDisabledByDefault = true;
+    }
+
+    /**
+     * @inheritdoc IPoolModule
+     */
+    function enablePoolCollateralByDefault(uint128 poolId) external override {
+        Pool.Data storage pool = Pool.loadExisting(poolId);
+
+        pool.collateralDisabledByDefault = false;
+    }
+
     /**
      * @inheritdoc IPoolModule
      */
@@ -228,17 +246,6 @@ contract PoolModule is IPoolModule {
         emit PoolNameUpdated(poolId, name, msg.sender);
     }
 
-    /**
-     * @inheritdoc IPoolModule
-     */
-    function isDelegationEnabledByPool(
-        uint128 poolId,
-        address collateral
-    ) external view override returns (bool) {
-        return
-            !Pool.loadExisting(poolId).collateralConfigurations[collateral].collateralTypeDisabled;
-    }
-
     /**
      * @inheritdoc IPoolModule
      */

diff --git a/protocol/synthetix/contracts/modules/core/VaultModule.sol b/protocol/synthetix/contracts/modules/core/VaultModule.sol
@@ -79,9 +79,6 @@ contract VaultModule is IVaultModule {
             // Check if the collateral is enabled here because we still want to allow reducing delegation for disabled collaterals.
             CollateralConfiguration.collateralEnabled(collateralType);
 
-            // Check if delegation is enabled for the pool
-            Pool.load(poolId).checkDelegationEnabled(collateralType);
-
             Account.requireSufficientCollateral(
                 accountId,
                 collateralType,

diff --git a/protocol/synthetix/contracts/storage/Pool.sol b/protocol/synthetix/contracts/storage/Pool.sol
@@ -60,11 +60,6 @@ library Pool {
         uint256 maxCollateral
     );
 
-    /**
-     * @notice Thrown when attempting delegate a pool disabled collateral
-     */
-    error PoolCollateralIsDisabled(address collateral, uint128 poolId);
-
     bytes32 private constant _CONFIG_SET_MARKET_MIN_DELEGATE_MAX = "setMarketMinDelegateTime_max";
 
     struct Data {
@@ -131,6 +126,14 @@ library Pool {
         uint64 __reserved2;
         uint64 __reserved3;
         mapping(address => PoolCollateralConfiguration.Data) collateralConfigurations;
+        /**
+         * @dev A switch to make the pool opt-in for new collateral
+         *
+         * By default it's set to false, which means any new collateral accepeted by the system will be accpeted by the pool.
+         *
+         * If the pool owner sets this value to true, then new collaterals will be disabled for the pool unless a maxDeposit is set for a that collateral.
+         */
+        bool collateralDisabledByDefault;
     }
 
     /**
@@ -534,8 +537,9 @@ library Pool {
         uint256 maxDeposit = self.collateralConfigurations[collateralType].maxDepositD18;
 
         if (
-            maxDeposit > 0 &&
-            self.vaults[collateralType].currentCollateral() + collateralAmountD18 > maxDeposit
+            (self.collateralDisabledByDefault && maxDeposit == 0) ||
+            (maxDeposit > 0 &&
+                self.vaults[collateralType].currentCollateral() + collateralAmountD18 > maxDeposit)
         ) {
             revert PoolCollateralLimitExceeded(
                 self.id,
@@ -545,14 +549,4 @@ library Pool {
             );
         }
     }
-
-    /**
-     * @notice Shows if a given collateral type is enabled for deposits and delegation in given pool.
-     * @param collateralType The address of the collateral.
-     */
-    function checkDelegationEnabled(Data storage self, address collateralType) internal view {
-        if (self.collateralConfigurations[collateralType].collateralTypeDisabled) {
-            revert PoolCollateralIsDisabled(collateralType, self.id);
-        }
-    }
 }
diff --git a/protocol/synthetix/contracts/storage/PoolCollateralConfiguration.sol b/protocol/synthetix/contracts/storage/PoolCollateralConfiguration.sol
@@ -7,7 +7,6 @@ library PoolCollateralConfiguration {
 
     struct Data {
         uint256 maxDepositD18;
-        bool collateralTypeDisabled;
         uint256 issuanceRatioD18;
     }
 }
diff --git a/protocol/synthetix/test/integration/bootstrap.ts b/protocol/synthetix/test/integration/bootstrap.ts
@@ -113,7 +113,6 @@ export function bootstrapWithMockMarketAndPool() {
       .Core.connect(owner)
       .setPoolCollateralConfiguration(r.poolId, r.collateralAddress(), {
         maxDepositD18: bn(1000000000),
-        collateralTypeDisabled: false,
         issuanceRatioD18: bn(1),
       });
   });

diff --git a/protocol/synthetix/test/integration/modules/core/IssueUSDModule.test.ts b/protocol/synthetix/test/integration/modules/core/IssueUSDModule.test.ts
@@ -467,7 +467,6 @@ describe('IssueUSDModule', function () {
       await systems()
         .Core.connect(owner)
         .setPoolCollateralConfiguration(poolId, collateralAddress(), {
-          collateralTypeDisabled: false,
           maxDepositD18: bn(10),
           issuanceRatioD18: bn(6),
         });

diff --git a/protocol/synthetix/test/integration/modules/core/PoolModuleFundAdmin.test.ts b/protocol/synthetix/test/integration/modules/core/PoolModuleFundAdmin.test.ts
@@ -714,37 +714,21 @@ describe('PoolModule Admin', function () {
           .Core.connect(user2)
           .setPoolCollateralConfiguration(thirdPoolId, collateralAddress(), {
             maxDepositD18: bn(10),
-            collateralTypeDisabled: false,
             issuanceRatioD18: bn(0),
           }),
         `Unauthorized("${await user2.getAddress()}")`,
         systems().Core
       );
     });
 
-    it('collateral is enabled by default for the pool', async () => {
-      await assert.equal(
-        await systems().Core.isDelegationEnabledByPool(thirdPoolId, collateralAddress()),
-        true
-      );
-    });
-
     it('disable the collateral by pool owner', async () => {
       await systems()
         .Core.connect(user1)
         .setPoolCollateralConfiguration(thirdPoolId, collateralAddress(), {
           maxDepositD18: bn(10),
-          collateralTypeDisabled: true,
           issuanceRatioD18: bn(2),
         });
     });
-
-    it('collateral is disabled for the pool', async () => {
-      await assert.equal(
-        await systems().Core.isDelegationEnabledByPool(thirdPoolId, collateralAddress()),
-        false
-      );
-    });
   });
 
   describe('set pool collateral issuance ratio', async () => {
@@ -773,7 +757,6 @@ describe('PoolModule Admin', function () {
           .Core.connect(user2)
           .setPoolCollateralConfiguration(thirdPoolId, collateralAddress(), {
             maxDepositD18: bn(10),
-            collateralTypeDisabled: false,
             issuanceRatioD18: bn(2),
           }),
         `Unauthorized("${await user2.getAddress()}")`,
@@ -793,7 +776,6 @@ describe('PoolModule Admin', function () {
         .Core.connect(user1)
         .setPoolCollateralConfiguration(thirdPoolId, collateralAddress(), {
           maxDepositD18: bn(10),
-          collateralTypeDisabled: false,
           issuanceRatioD18: bn(2),
         });
 

diff --git a/protocol/synthetix/test/integration/modules/core/VaultModule.test.ts b/protocol/synthetix/test/integration/modules/core/VaultModule.test.ts
@@ -340,7 +340,6 @@ describe('VaultModule', function () {
           .Core.connect(user1)
           .setPoolCollateralConfiguration(fakeVaultId, collateralAddress(), {
             maxDepositD18: bn(10),
-            collateralTypeDisabled: true,
             issuanceRatioD18: bn(0),
           });
       });
@@ -357,7 +356,9 @@ describe('VaultModule', function () {
               depositAmount.div(50),
               ethers.utils.parseEther('1')
             ),
-          `PoolCollateralIsDisabled("${collateralAddress()}", "${fakeVaultId}")`,
+          `PoolCollateralLimitExceeded("${fakeVaultId}", "${collateralAddress()}", "${depositAmount
+            .div(50)
+            .toString()}", "${bn(10).toString()}")`,
           systems().Core
         );
       });
@@ -367,7 +368,6 @@ describe('VaultModule', function () {
           .Core.connect(user1)
           .setPoolCollateralConfiguration(fakeVaultId, collateralAddress(), {
             maxDepositD18: bn(1000000),
-            collateralTypeDisabled: false,
             issuanceRatioD18: bn(0),
           });
       });
@@ -391,7 +391,6 @@ describe('VaultModule', function () {
           .Core.connect(owner)
           .setPoolCollateralConfiguration(poolId, collateralAddress(), {
             maxDepositD18: depositAmount.div(2),
-            collateralTypeDisabled: false,
             issuanceRatioD18: bn(0),
           });
       });
@@ -454,7 +453,6 @@ describe('VaultModule', function () {
             .Core.connect(owner)
             .setPoolCollateralConfiguration(poolId, collateralAddress(), {
               maxDepositD18: depositAmount.mul(10),
-              collateralTypeDisabled: false,
               issuanceRatioD18: bn(0),
             });
         });