chore(rns-unified): merge from release/v0.2.0 (#85)

.github/workflows/create-PR-release.yml

@@ -13,15 +13,19 @@ env:
   HEAD_BRANCH: ${{ github.head_ref || github.ref_name }}
 
 jobs:
-  mergeRelease2FeatureRnsUnified:
+  mergeRelease2FeatureRepo:
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        branch_name: [feature/rns-unified, feature/controller, feature/domain-price, feature/auction, feature/public-resolver, feature/ci, feature/reverse-registrar]
     steps:
       - name: Set env
         run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/rns-unified" >> $GITHUB_ENV
+          echo "PR_BRANCH=merge/${HEAD_BRANCH}-${{matrix.branch_name}}" >> $GITHUB_ENV
+          echo "FEATURE_NAME=$(echo ${{matrix.branch_name}} | cut -d'/' -f2)" >> $GITHUB_ENV
       - uses: actions/checkout@v3
         with:
-          ref: feature/rns-unified
+          ref: ${{matrix.branch_name}}
       - name: Reset promotion branch
         run: |
           git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
@@ -34,205 +38,13 @@ jobs:
           template: .github/template/create-pull-request.md
           vars: |
             fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/rns-unified
+            toBranch: ${{matrix.branch_name}}
 
       - name: Create Pull Request
         uses: peter-evans/create-pull-request@v5
         with:
           labels: automated PR
           delete-branch: true
-          title: 'chore(rns-unified): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeatureController:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/controller" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/controller
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/controller
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(controller): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeatureDomainPrice:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/domain-price" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/domain-price
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/domain-price
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(domain-price): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeatureAuction:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/auction" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/auction
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/auction
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(auction): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeaturePublicResolver:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/public-resolver" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/public-resolver
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/public-resolver
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(public-resolver): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeatureCI:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/ci" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/ci
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/ci
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(ci): merge from `${{env.HEAD_BRANCH}}`'
-          body: ${{ steps.template.outputs.result }}
-          branch: ${{env.PR_BRANCH}}
-
-  mergeRelease2FeatureReverseRegistrar:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Set env
-        run: |
-          echo "PR_BRANCH=merge/${HEAD_BRANCH}-feature/reverse-registrar" >> $GITHUB_ENV
-      - uses: actions/checkout@v3
-        with:
-          ref: feature/reverse-registrar
-      - name: Reset promotion branch
-        run: |
-          git fetch origin ${HEAD_BRANCH}:${HEAD_BRANCH}
-          git reset --hard ${HEAD_BRANCH}
-
-      - name: Render template
-        id: template
-        uses: chuhlomin/[email protected]
-        with:
-          template: .github/template/create-pull-request.md
-          vars: |
-            fromBranch: ${{env.HEAD_BRANCH}}
-            toBranch: feature/reverse-registrar
-
-      - name: Create Pull Request
-        uses: peter-evans/create-pull-request@v5
-        with:
-          labels: automated PR
-          delete-branch: true
-          title: 'chore(reverse-registrar): merge from `${{env.HEAD_BRANCH}}`'
+          title: 'chore(`${{env.FEATURE_NAME}}`): merge from `${{env.HEAD_BRANCH}}`'
           body: ${{ steps.template.outputs.result }}
           branch: ${{env.PR_BRANCH}}

script/20231021-upgrade-domain-price-and-auction/20231021_UpgradeDomainPriceAndAuction.s.sol

@@ -0,0 +1,12 @@
+// SPDX-License-Identifier: MIT
+pragma solidity ^0.8.19;
+
+import { ContractKey } from "foundry-deployment-kit/configs/ContractConfig.sol";
+import { RNSDeploy } from "script/RNSDeploy.s.sol";
+
+contract Migration__20231021_UpgradeDomainPriceAndAuction is RNSDeploy {
+  function run() public trySetUp {
+    _upgradeProxy(ContractKey.RNSAuction, EMPTY_ARGS);
+    _upgradeProxy(ContractKey.RNSDomainPrice, EMPTY_ARGS);
+  }
+}

script/20231024-upgrade-auction/20231024_UpgradeAuction.s.sol

@@ -0,0 +1,11 @@
+// SPDX-License-Identifier: MIT
+pragma solidity ^0.8.19;
+
+import { ContractKey } from "foundry-deployment-kit/configs/ContractConfig.sol";
+import { RNSDeploy } from "script/RNSDeploy.s.sol";
+
+contract Migration__20231024_UpgradeAuction is RNSDeploy {
+  function run() public trySetUp {
+    _upgradeProxy(ContractKey.RNSAuction, EMPTY_ARGS);
+  }
+}

script/20231024-upgrade-domain-price/20231024_Config.s.sol

@@ -0,0 +1,17 @@
+// SPDX-License-Identifier: MIT
+pragma solidity ^0.8.19;
+
+import { Network, RNSDeploy } from "script/RNSDeploy.s.sol";
+
+abstract contract Config__20231024 is RNSDeploy {
+  function _buildMigrationConfig() internal view virtual override returns (Config memory config) {
+    config = super._buildMigrationConfig();
+    if (_network == Network.RoninTestnet) {
+      config.overrider = config.operator;
+    } else if (_network == Network.RoninMainnet) {
+      revert("Missing config");
+    } else {
+      revert("Missing config");
+    }
+  }
+}

script/20231024-upgrade-domain-price/20231024_UpgradeDomainPrice.s.sol

@@ -0,0 +1,23 @@
+// SPDX-License-Identifier: MIT
+pragma solidity ^0.8.19;
+
+import { console2 } from "forge-std/console2.sol";
+import { ContractKey } from "foundry-deployment-kit/configs/ContractConfig.sol";
+import { RNSDomainPrice } from "@rns-contracts/RNSDomainPrice.sol";
+import { Config__20231024 } from "./20231024_Config.s.sol";
+
+contract Migration__20231024_UpgradeDomainPrice is Config__20231024 {
+  function run() public trySetUp {
+    Config memory config = getConfig();
+    _upgradeProxy(ContractKey.RNSDomainPrice, EMPTY_ARGS);
+
+    console2.log("operator", config.operator);
+    console2.log("overrider", config.overrider);
+
+    RNSDomainPrice domainPrice = RNSDomainPrice(_config.getAddressFromCurrentNetwork(ContractKey.RNSDomainPrice));
+    address admin = domainPrice.getRoleMember(0x00, 0);
+    bytes32 overriderRole = domainPrice.OVERRIDER_ROLE();
+    vm.broadcast(admin);
+    domainPrice.grantRole(overriderRole, config.overrider);
+  }
+}

script/Debug.s.sol

@@ -9,7 +9,7 @@ contract Debug is RNSDeploy {
 
   function debug(uint256 forkBlock, address from, address to, uint256 value, bytes calldata callData) external {
     if (forkBlock != 0) {
-        vm.rollFork(forkBlock);
+      vm.rollFork(forkBlock);
     }
     vm.prank(from);
     (bool success, bytes memory returnOrRevertData) = to.call{ value: value }(callData);

script/RNSDeploy.s.sol

@@ -9,6 +9,7 @@ abstract contract RNSDeploy is BaseDeploy {
     IPyth pyth;
     address admin;
     address pauser;
+    address overrider;
     address controller;
     uint8 minWord;
     uint8 maxWord;

src/RNSAuction.sol

@@ -23,11 +23,12 @@ contract RNSAuction is Initializable, AccessControlEnumerable, INSAuction {
   /// @inheritdoc INSAuction
   uint64 public constant DOMAIN_EXPIRY_DURATION = 365 days;
   /// @inheritdoc INSAuction
+  uint64 public constant MAX_AUCTION_DOMAIN_EXPIRY = 365 days * 3;
+  /// @inheritdoc INSAuction
   bytes32 public constant OPERATOR_ROLE = keccak256("OPERATOR_ROLE");
 
   /// @dev Gap for upgradeability.
   uint256[50] private ____gap;
-
   /// @dev The RNSUnified contract.
   INSUnified internal _rnsUnified;
   /// @dev Mapping from auction Id => event range
@@ -190,7 +191,7 @@ contract RNSAuction is Initializable, AccessControlEnumerable, INSAuction {
     if (msg.value < beatPrice) revert InsufficientAmount();
     address payable bidder = payable(_msgSender());
     // check whether the bidder can receive RON
-    if (!RONTransferHelper.send(bidder, 0)) revert BidderCannotReceiveRON();
+    if (bidder != tx.origin) revert ContractBidderIsForbidden();
     address payable prvBidder = auction.bid.bidder;
     uint256 prvPrice = auction.bid.price;
 
@@ -207,13 +208,13 @@ contract RNSAuction is Initializable, AccessControlEnumerable, INSAuction {
   /**
    * @inheritdoc INSAuction
    */
-  function bulkClaimBidNames(uint256[] calldata ids) external returns (bool[] memory claimeds) {
+  function bulkClaimBidNames(uint256[] calldata ids) external returns (uint256[] memory claimedAts) {
     uint256 id;
     uint256 accumulatedRON;
     EventRange memory range;
     DomainAuction memory auction;
     uint256 length = ids.length;
-    claimeds = new bool[](length);
+    claimedAts = new uint256[](length);
     INSUnified rnsUnified = _rnsUnified;
     uint64 expiry = uint64(block.timestamp.addWithUpperbound(DOMAIN_EXPIRY_DURATION, MAX_EXPIRY));
 
@@ -222,15 +223,15 @@ contract RNSAuction is Initializable, AccessControlEnumerable, INSAuction {
       auction = _domainAuction[id];
       range = _auctionRange[auction.auctionId];
 
-      if (!auction.bid.claimed) {
+      if (auction.bid.claimedAt == 0) {
         if (!range.isEnded()) revert NotYetEnded();
         if (auction.bid.timestamp == 0) revert NoOneBidded();
 
         accumulatedRON += auction.bid.price;
         rnsUnified.setExpiry(id, expiry);
         rnsUnified.transferFrom(address(this), auction.bid.bidder, id);
 
-        _domainAuction[id].bid.claimed = claimeds[i] = true;
+        _domainAuction[id].bid.claimedAt = claimedAts[i] = block.timestamp;
       }
 
       unchecked {