fix(contract): remove reinitializers from AlignedLayerServiceManager (#1770)

Co-authored-by: MauroFab <[email protected]>

Author: uri-99

contracts/script/upgrade/AlignedLayerPauserUpgrader.s.sol

@@ -84,23 +84,26 @@ contract AlignedLayerServiceManager is
         _initializePauser(_pauserRegistry, _initialPausedStatus);
     }
 
-    // This function is to be run only on upgrade
-    // If a new contract is deployed, this function should be removed
-    // Because this new value is also added in the initializer
-    function initializeAggregator(
-        address _alignedAggregator
-    ) public reinitializer(2) {
-        setAggregator(_alignedAggregator);
-    }
-
-    // Just to be used to upgrade contracts without the pausable functionality
-    // Once the contract is pausable this method is not needed anymore
-    function initializePauser(
-        IPauserRegistry _pauserRegistry,
-        uint256 _initialPausedStatus
-    ) public reinitializer(3) {
-        _initializePauser(_pauserRegistry, _initialPausedStatus);
-    }
+    // Reinitializers:
+    // Notice Testnet had more upgrades than Mainnet. 
+    // In Testnet, we executed the reinitializer(2) and reinitializer(3)
+
+    // These are not needed in Mainnet. 
+    // In the future, in case of needing to add a reinitializer, 
+    // either add it as reinitializer(4) or redeploy Testnet from scratch
+    
+    // function initializeAggregator( // applied on Testnet
+    //     address _alignedAggregator
+    // ) public reinitializer(2) {
+    //     setAggregator(_alignedAggregator);
+    // }
+
+    // function initializePauser( // applied on Testnet
+    //     IPauserRegistry _pauserRegistry,
+    //     uint256 _initialPausedStatus
+    // ) public reinitializer(3) {
+    //     _initializePauser(_pauserRegistry, _initialPausedStatus);
+    // }
 
     function createNewTask(
         bytes32 batchMerkleRoot,