Deploy v2

.vscode/settings.json

@@ -7,7 +7,7 @@
     "editor.defaultFormatter": "JuanBlanco.solidity"
   },
   "editor.codeActionsOnSave": {
-    "source.fixAll": true
+    "source.fixAll": "explicit"
   },
   "solidity.compileUsingRemoteVersion": "v0.8.17+commit.8df45f5f"
 }

contracts/Distributor.sol

@@ -39,7 +39,7 @@
 import { SafeERC20 } from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
 import { SafeCast } from "@openzeppelin/contracts/utils/math/SafeCast.sol";
 
 import "./utils/UUPSHelper.sol";
 
 struct MerkleTree {
     // Root of a Merkle tree which leaves are `(address user, address token, uint amount)`
@@ -58,7 +58,7 @@
 }
 
 /// @title Distributor
-/// @notice Allows LPs on AMMs with concentrated liquidity to claim the rewards that were distributed to them
+/// @notice Allows to claim rewards distributed to them through Merkl
 /// @author Angle Labs. Inc
 contract Distributor is UUPSHelper {
     using SafeERC20 for IERC20;
@@ -130,6 +130,12 @@
         _;
     }
 
+    /// @notice Checks whether the `msg.sender` has the governor role or the guardian role
+    modifier onlyGovernor() {
+        if (!core.isGovernor(msg.sender)) revert NotGovernor();
+        _;
+    }
+
     /// @notice Checks whether the `msg.sender` is the `user` address or is a trusted address
     modifier onlyTrustedOrUser(address user) {
         if (user != msg.sender && canUpdateMerkleRoot[msg.sender] != 1 && !core.isGovernorOrGuardian(msg.sender))
@@ -147,7 +153,7 @@
     }
 
     /// @inheritdoc UUPSUpgradeable
-    function _authorizeUpgrade(address) internal view override onlyGuardianUpgrader(core) {}
+    function _authorizeUpgrade(address) internal view override onlyGovernorUpgrader(core) {}
 
     // =============================== MAIN FUNCTION ===============================
 
@@ -205,7 +211,7 @@
     // ============================ GOVERNANCE FUNCTIONS ===========================
 
     /// @notice Adds or removes EOAs which are trusted to update the Merkle root
-    function toggleTrusted(address eoa) external onlyGovernorOrGuardian {
+    function toggleTrusted(address eoa) external onlyGovernor {
         uint256 trustedStatus = 1 - canUpdateMerkleRoot[eoa];
         canUpdateMerkleRoot[eoa] = trustedStatus;
         emit TrustedToggled(eoa, trustedStatus == 1);
@@ -218,7 +224,7 @@
             // A trusted address cannot update a tree right after a precedent tree update otherwise it can de facto
             // validate a tree which has not passed the dispute period
             ((canUpdateMerkleRoot[msg.sender] != 1 || block.timestamp < endOfDisputePeriod) &&
-                !core.isGovernorOrGuardian(msg.sender))
+                !core.isGovernor(msg.sender))
         ) revert NotTrusted();
         MerkleTree memory _lastTree = tree;
         tree = _tree;
@@ -278,26 +284,26 @@
     }
 
     /// @notice Recovers any ERC20 token
-    function recoverERC20(address tokenAddress, address to, uint256 amountToRecover) external onlyGovernorOrGuardian {
+    function recoverERC20(address tokenAddress, address to, uint256 amountToRecover) external onlyGovernor {
         IERC20(tokenAddress).safeTransfer(to, amountToRecover);
         emit Recovered(tokenAddress, to, amountToRecover);
     }
 
     /// @notice Sets the dispute period after which a tree update becomes effective
-    function setDisputePeriod(uint48 _disputePeriod) external onlyGovernorOrGuardian {
+    function setDisputePeriod(uint48 _disputePeriod) external onlyGovernor {
         disputePeriod = uint48(_disputePeriod);
         emit DisputePeriodUpdated(_disputePeriod);
     }
 
     /// @notice Sets the token used as a caution during disputes
-    function setDisputeToken(IERC20 _disputeToken) external onlyGovernorOrGuardian {
+    function setDisputeToken(IERC20 _disputeToken) external onlyGovernor {
         if (disputer != address(0)) revert UnresolvedDispute();
         disputeToken = _disputeToken;
         emit DisputeTokenUpdated(address(_disputeToken));
     }
 
     /// @notice Sets the amount of `disputeToken` used as a caution during disputes
-    function setDisputeAmount(uint256 _disputeAmount) external onlyGovernorOrGuardian {
+    function setDisputeAmount(uint256 _disputeAmount) external onlyGovernor {
         if (disputer != address(0)) revert UnresolvedDispute();
         disputeAmount = _disputeAmount;
         emit DisputeAmountUpdated(_disputeAmount);

contracts/deprecated/OldDistributor.sol

@@ -35,9 +35,9 @@
 
 pragma solidity ^0.8.17;
 
-import "@openzeppelin/contracts/token/ERC20/IERC20.sol";
-import "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
-import "@openzeppelin/contracts/utils/math/SafeCast.sol";
+import { IERC20 } from "@openzeppelin/contracts/token/ERC20/IERC20.sol";
+import { SafeERC20 } from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol";
+import { SafeCast } from "@openzeppelin/contracts/utils/math/SafeCast.sol";
 
 import "../utils/UUPSHelper.sol";
 
@@ -54,14 +54,18 @@ struct MerkleTree {
 struct Claim {
     uint208 amount;
     uint48 timestamp;
+    bytes32 merkleRoot;
 }
 
-/// @title OldDistributor
+/// @title Distributor
 /// @notice Allows LPs on AMMs with concentrated liquidity to claim the rewards that were distributed to them
 /// @author Angle Labs. Inc
 contract OldDistributor is UUPSHelper {
     using SafeERC20 for IERC20;
 
+    /// @notice Epoch duration
+    uint32 internal constant _EPOCH_DURATION = 3600;
+
     // ================================= VARIABLES =================================
 
     /// @notice Tree of claimable tokens through this contract
@@ -80,10 +84,10 @@ contract OldDistributor is UUPSHelper {
     /// @dev Used to store if there is an ongoing dispute
     address public disputer;
 
-    /// @notice Last time the `tree` was updated
-    uint48 public lastTreeUpdate;
+    /// @notice When the current tree will become valid
+    uint48 public endOfDisputePeriod;
 
-    /// @notice Time before which a change in a tree becomes effective
+    /// @notice Time after which a change in a tree becomes effective, in EPOCH_DURATION
     uint48 public disputePeriod;
 
     /// @notice Amount to deposit to freeze the roots update
@@ -105,16 +109,17 @@ contract OldDistributor is UUPSHelper {
 
     // =================================== EVENTS ==================================
 
-    event Claimed(address user, address token, uint256 amount);
+    event Claimed(address indexed user, address indexed token, uint256 amount);
+    event DisputeAmountUpdated(uint256 _disputeAmount);
     event Disputed(string reason);
     event DisputePeriodUpdated(uint48 _disputePeriod);
-    event DisputeTokenUpdated(address indexed _disputeToken);
-    event DisputeAmountUpdated(uint256 _disputeAmount);
     event DisputeResolved(bool valid);
-    event OperatorClaimingToggled(address user, bool isEnabled);
-    event OperatorToggled(address user, address operator, bool isWhitelisted);
+    event DisputeTokenUpdated(address indexed _disputeToken);
+    event OperatorClaimingToggled(address indexed user, bool isEnabled);
+    event OperatorToggled(address indexed user, address indexed operator, bool isWhitelisted);
     event Recovered(address indexed token, address indexed to, uint256 amount);
-    event TreeUpdated(bytes32 merkleRoot, bytes32 ipfsHash);
+    event Revoked(); // With this event an indexer could maintain a table (timestamp, merkleRootUpdate)
+    event TreeUpdated(bytes32 merkleRoot, bytes32 ipfsHash, uint48 endOfDisputePeriod);
     event TrustedToggled(address indexed eoa, bool trust);
 
     // ================================= MODIFIERS =================================
@@ -181,7 +186,7 @@ contract OldDistributor is UUPSHelper {
 
             // Closing reentrancy gate here
             uint256 toSend = amount - claimed[user][token].amount;
-            claimed[user][token] = Claim(SafeCast.toUint208(amount), uint48(block.timestamp));
+            claimed[user][token] = Claim(SafeCast.toUint208(amount), uint48(block.timestamp), getMerkleRoot());
 
             IERC20(token).safeTransfer(user, toSend);
             emit Claimed(user, token, toSend);
@@ -193,7 +198,7 @@ contract OldDistributor is UUPSHelper {
 
     /// @notice Returns the MerkleRoot that is currently live for the contract
     function getMerkleRoot() public view returns (bytes32) {
-        if (block.timestamp - lastTreeUpdate >= disputePeriod) return tree.merkleRoot;
+        if (block.timestamp >= endOfDisputePeriod && disputer == address(0)) return tree.merkleRoot;
         else return lastTree.merkleRoot;
     }
 
@@ -212,21 +217,24 @@ contract OldDistributor is UUPSHelper {
             disputer != address(0) ||
             // A trusted address cannot update a tree right after a precedent tree update otherwise it can de facto
             // validate a tree which has not passed the dispute period
-            ((canUpdateMerkleRoot[msg.sender] != 1 || block.timestamp - lastTreeUpdate < disputePeriod) &&
+            ((canUpdateMerkleRoot[msg.sender] != 1 || block.timestamp < endOfDisputePeriod) &&
                 !core.isGovernorOrGuardian(msg.sender))
         ) revert NotTrusted();
         MerkleTree memory _lastTree = tree;
         tree = _tree;
         lastTree = _lastTree;
-        lastTreeUpdate = uint48(block.timestamp);
-        emit TreeUpdated(_tree.merkleRoot, _tree.ipfsHash);
+
+        uint48 _endOfPeriod = _endOfDisputePeriod(uint48(block.timestamp));
+        endOfDisputePeriod = _endOfPeriod;
+        emit TreeUpdated(_tree.merkleRoot, _tree.ipfsHash, _endOfPeriod);
     }
 
     /// @notice Freezes the Merkle tree update until the dispute is resolved
     /// @dev Requires a deposit of `disputeToken` that'll be slashed if the dispute is not accepted
-    /// @dev It is only possible to create a dispute for `disputePeriod` after each tree update
+    /// @dev It is only possible to create a dispute within `disputePeriod` after each tree update
     function disputeTree(string memory reason) external {
-        if (block.timestamp - lastTreeUpdate >= disputePeriod) revert InvalidDispute();
+        if (disputer != address(0)) revert UnresolvedDispute();
+        if (block.timestamp >= endOfDisputePeriod) revert InvalidDispute();
         IERC20(disputeToken).safeTransferFrom(msg.sender, address(this), disputeAmount);
         disputer = msg.sender;
         emit Disputed(reason);
@@ -242,7 +250,7 @@ contract OldDistributor is UUPSHelper {
             _revokeTree();
         } else {
             IERC20(disputeToken).safeTransfer(msg.sender, disputeAmount);
-            lastTreeUpdate = uint48(block.timestamp);
+            endOfDisputePeriod = _endOfDisputePeriod(uint48(block.timestamp));
         }
         disputer = address(0);
         emit DisputeResolved(valid);
@@ -275,9 +283,8 @@ contract OldDistributor is UUPSHelper {
         emit Recovered(tokenAddress, to, amountToRecover);
     }
 
-    /// @notice Sets the dispute period before which a tree update becomes effective
+    /// @notice Sets the dispute period after which a tree update becomes effective
     function setDisputePeriod(uint48 _disputePeriod) external onlyGovernorOrGuardian {
-        if (_disputePeriod > block.timestamp) revert InvalidParam();
         disputePeriod = uint48(_disputePeriod);
         emit DisputePeriodUpdated(_disputePeriod);
     }
@@ -301,9 +308,20 @@ contract OldDistributor is UUPSHelper {
     /// @notice Fallback to the last version of the tree
     function _revokeTree() internal {
         MerkleTree memory _tree = lastTree;
-        lastTreeUpdate = 0;
+        endOfDisputePeriod = 0;
         tree = _tree;
-        emit TreeUpdated(_tree.merkleRoot, _tree.ipfsHash);
+        emit Revoked();
+        emit TreeUpdated(
+            _tree.merkleRoot,
+            _tree.ipfsHash,
+            (uint48(block.timestamp) / _EPOCH_DURATION) * (_EPOCH_DURATION) // Last hour
+        );
+    }
+
+    /// @notice Returns the end of the dispute period
+    /// @dev treeUpdate is rounded up to next hour and then `disputePeriod` hours are added
+    function _endOfDisputePeriod(uint48 treeUpdate) internal view returns (uint48) {
+        return ((treeUpdate - 1) / _EPOCH_DURATION + 1 + disputePeriod) * (_EPOCH_DURATION);
     }
 
     /// @notice Checks the validity of a proof

contracts/struct/CampaignParameters.sol

@@ -0,0 +1,29 @@
+// SPDX-License-Identifier: BUSL-1.1
+
+pragma solidity ^0.8.17;
+
+struct CampaignParameters {
+    // POPULATED ONCE CREATED
+
+    // ID of the campaign. This can be left as a null bytes32 when creating campaigns
+    // on Merkl.
+    bytes32 campaignId;
+    // CHOSEN BY CAMPAIGN CREATOR
+
+    // Address of the campaign creator, if marked as address(0), it will be overriden with the
+    // address of the `msg.sender` creating the campaign
+    address creator;
+    // Address of the token used as a reward
+    address rewardToken;
+    // Amount of `rewardToken` to distribute across all the epochs
+    // Amount distributed per epoch is `amount/numEpoch`
+    uint256 amount;
+    // Type of campaign
+    uint32 campaignType;
+    // Timestamp at which the campaign should start
+    uint32 startTimestamp;
+    // Duration of the campaign in seconds. Has to be a multiple of EPOCH = 3600
+    uint32 duration;
+    // Extra data to pass to specify the campaign
+    bytes campaignData;
+}

contracts/utils/Errors.sol

@@ -2,6 +2,12 @@
 
 pragma solidity ^0.8.17;
 
+error CampaignDoesNotExist();
+error CampaignAlreadyExists();
+error CampaignDurationBelowHour();
+error CampaignRewardTokenNotWhitelisted();
+error CampaignRewardTooLow();
+error CampaignSouldStartInFuture();
 error InvalidDispute();
 error InvalidLengths();
 error InvalidParam();

deploy/0_distributor.ts

@@ -27,35 +27,35 @@ const func: DeployFunction = async ({ deployments, ethers, network }) => {
 
   console.log('Now deploying Distributor');
   console.log('Starting with the implementation');
-  /*
-  await deploy('Distributor_Implementation_2', {
+
+  await deploy('Distributor_Implementation_V2_0', {
     contract: 'Distributor',
     from: deployer.address,
     log: !argv.ci,
   });
-  */
 
-  const implementationAddress = (await ethers.getContract('Distributor_Implementation_2')).address;
+  const implementationAddress = (await ethers.getContract('Distributor_Implementation_V2_0')).address;
 
   console.log(`Successfully deployed the implementation for Distributor at ${implementationAddress}`);
   console.log('');
-
+  /*
   console.log('Now deploying the Proxy');
 
-  await deploy('Distributor', {
+  await deploy('TestDistributor', {
     contract: 'ERC1967Proxy',
     from: deployer.address,
     args: [implementationAddress, '0x'],
     log: !argv.ci,
   });
 
-  const distributor = (await deployments.get('Distributor')).address;
+  const distributor = (await deployments.get('TestDistributor')).address;
   console.log(`Successfully deployed contract at the address ${distributor}`);
   console.log('Initializing the contract');
   const contract = new ethers.Contract(distributor, Distributor__factory.createInterface(), deployer) as Distributor;
   await (await contract.connect(deployer).initialize(core)).wait();
   console.log('Contract successfully initialized');
   console.log('');
+  */
 };
 
 func.tags = ['distributor'];

deploy/1_distributionCreator.ts

@@ -26,17 +26,17 @@ const func: DeployFunction = async ({ deployments, ethers, network }) => {
   console.log('Now deploying DistributionCreator');
   console.log('Starting with the implementation');
   console.log('deployer ', await deployer.getBalance());
-  await deploy('DistributionCreator_Implementation_7', {
+  await deploy('DistributionCreator_Implementation_V2_0', {
     contract: 'DistributionCreator',
     from: deployer.address,
     log: !argv.ci,
   });
 
-  const implementationAddress = (await ethers.getContract('DistributionCreator_Implementation_7')).address;
+  const implementationAddress = (await ethers.getContract('DistributionCreator_Implementation_V2_0')).address;
 
   console.log(`Successfully deployed the implementation for DistributionCreator at ${implementationAddress}`);
   console.log('');
-
+  /*
   const distributor = (await deployments.get('Distributor')).address;
   console.log('Now deploying the Proxy');
 
@@ -60,6 +60,7 @@ const func: DeployFunction = async ({ deployments, ethers, network }) => {
   console.log('Contract successfully initialized');
   console.log('');
   console.log(await contract.core());
+  */
 
   /* Once good some functions need to be called to have everything setup.
 
@@ -78,5 +79,5 @@ const func: DeployFunction = async ({ deployments, ethers, network }) => {
 };
 
 func.tags = ['distributionCreator'];
-func.dependencies = [''];
+func.dependencies = ['distributor'];
 export default func;