Refactor factory

src/core/EulerAggregationVaultFactory.sol

@@ -10,8 +10,6 @@ import {Clones} from "@openzeppelin/contracts/proxy/Clones.sol";
 /// @custom:security-contact [email protected]
 /// @author Euler Labs (https://www.eulerlabs.com/)
 contract EulerAggregationVaultFactory {
-    error WithdrawalQueueAlreadyWhitelisted();
-    error NotWhitelistedWithdrawalQueueImpl();
     error InvalidQuery();
 
     /// core dependencies
@@ -23,7 +21,7 @@ contract EulerAggregationVaultFactory {
     address public immutable strategyModule;
     address public immutable rebalanceModule;
     /// aggregation vault implementation address
-    // address public immutable aggregationVaultImpl;
+    address public immutable aggregationVaultImpl;
 
     address[] public aggregationVaults;
 
@@ -52,15 +50,15 @@ contract EulerAggregationVaultFactory {
         strategyModule = Clones.clone(_factoryParams.strategyModuleImpl);
         rebalanceModule = Clones.clone(_factoryParams.rebalanceModuleImpl);
 
-        // IEulerAggregationVault.ConstructorParams memory aggregationVaultConstructorParams = IEulerAggregationVault
-        //     .ConstructorParams({
-        //     rewardsModule: rewardsModule,
-        //     hooksModule: hooksModule,
-        //     feeModule: feeModule,
-        //     strategyModule: strategyModule,
-        //     rebalanceModule: rebalanceModule
-        // });
-        // aggregationVaultImpl = address(new EulerAggregationVault(aggregationVaultConstructorParams));
+        IEulerAggregationVault.ConstructorParams memory aggregationVaultConstructorParams = IEulerAggregationVault
+            .ConstructorParams({
+            rewardsModule: rewardsModule,
+            hooksModule: hooksModule,
+            feeModule: feeModule,
+            strategyModule: strategyModule,
+            rebalanceModule: rebalanceModule
+        });
+        aggregationVaultImpl = address(new EulerAggregationVault(aggregationVaultConstructorParams));
     }
 
     /// @notice Deploy a new aggregation vault.
@@ -79,16 +77,7 @@ contract EulerAggregationVaultFactory {
         string memory _symbol,
         uint256 _initialCashAllocationPoints
     ) external returns (address) {
-        // address eulerAggregationVault = Clones.clone(aggregationVaultImpl);
-        IEulerAggregationVault.ConstructorParams memory aggregationVaultConstructorParams = IEulerAggregationVault
-            .ConstructorParams({
-            rewardsModule: rewardsModule,
-            hooksModule: hooksModule,
-            feeModule: feeModule,
-            strategyModule: strategyModule,
-            rebalanceModule: rebalanceModule
-        });
-        address eulerAggregationVault = address(new EulerAggregationVault(aggregationVaultConstructorParams));
+        address eulerAggregationVault = Clones.clone(aggregationVaultImpl);
 
         IEulerAggregationVault.InitParams memory aggregationVaultInitParams = IEulerAggregationVault.InitParams({
             aggregationVaultOwner: msg.sender,