Merge pull request #13 from ronin-chain/feat/refactor

feat: refactor
ronin-chain · Sep 18, 2024 · 0f0bb47 · 0f0bb47
2 parents 18265ab + 39a6e9d
commit 0f0bb47
Show file tree

Hide file tree

Showing 60 changed files with 421 additions and 496 deletions.
diff --git a/foundry.toml b/foundry.toml
@@ -7,7 +7,7 @@ libs = ["lib"]
 # See more config options https://github.com/foundry-rs/foundry/blob/master/crates/config/README.md#all-options
 
 solc = '0.7.6'
-optimizer_runs = 800
+optimizer_runs = 250
 evm_version = 'istanbul'
 use_literal_content = true
 

diff --git a/lib/openzeppelin-contracts b/lib/openzeppelin-contracts
diff --git a/logs/contract-code-sizes.log b/logs/contract-code-sizes.log
@@ -4,50 +4,49 @@
 | AddressStringUtil                  |       86 |     24,490 |
 | AuthorizationLib                   |       86 |     24,490 |
 | Base64                             |       86 |     24,490 |
-| BeaconProxy                        |      785 |     23,791 |
+| BeaconProxy                        |      759 |     23,817 |
 | BitMath                            |       86 |     24,490 |
 | BytesLib                           |       86 |     24,490 |
 | CallbackValidation                 |       86 |     24,490 |
 | ChainId                            |       86 |     24,490 |
-| ERC20                              |    2,578 |     21,998 |
+| Create2                            |       86 |     24,490 |
+| ERC20                              |    2,546 |     22,030 |
 | ERC20Mock                          |    3,518 |     21,058 |
-| ERC721                             |    6,650 |     17,926 |
+| ERC721                             |    6,557 |     18,019 |
 | EnumerableMap                      |       86 |     24,490 |
 | EnumerableSet                      |       86 |     24,490 |
 | FixedPoint128                      |       86 |     24,490 |
 | FixedPoint96                       |       86 |     24,490 |
 | FullMath                           |       86 |     24,490 |
 | HexStrings                         |       86 |     24,490 |
-| KatanaGovernanceMock               |    2,346 |     22,230 |
-| KatanaInterfaceMulticall           |    1,334 |     23,242 |
+| KatanaGovernanceMock               |    2,388 |     22,188 |
+| KatanaInterfaceMulticall           |    1,295 |     23,281 |
 | KatanaV2Library                    |       86 |     24,490 |
 | KatanaV2LibraryTestnet             |       86 |     24,490 |
-| KatanaV3Factory                    |    2,945 |     21,631 |
-| KatanaV3FactoryProxy               |    5,541 |     19,035 |
-| KatanaV3Pool                       |   22,864 |      1,712 |
-| KatanaV3PoolDeployer               |      288 |     24,288 |
-| KatanaV3PoolProxy                  |    1,248 |     23,328 |
-| KatanaV3PoolProxyBytecode          |    2,931 |     21,645 |
+| KatanaV3Factory                    |    4,132 |     20,444 |
+| KatanaV3Pool                       |   21,545 |      3,031 |
+| KatanaV3PoolBeacon                 |    4,246 |     20,330 |
+| KatanaV3PoolProxy                  |    1,209 |     23,367 |
 | LiquidityAmounts                   |       86 |     24,490 |
 | LiquidityMath                      |       86 |     24,490 |
 | LowGasSafeMath                     |       86 |     24,490 |
-| MixedRouteQuoterV1                 |    7,328 |     17,248 |
-| MixedRouteQuoterV1Testnet          |    7,328 |     17,248 |
-| NFTDescriptor                      |   23,833 |        743 |
+| MixedRouteQuoterV1                 |    7,220 |     17,356 |
+| MixedRouteQuoterV1Testnet          |    7,220 |     17,356 |
+| NFTDescriptor                      |   23,372 |      1,204 |
 | NFTSVG                             |       86 |     24,490 |
-| NonfungiblePositionManager         |   24,381 |        195 |
-| NonfungibleTokenPositionDescriptor |    5,219 |     19,357 |
+| NonfungiblePositionManager         |   24,452 |        124 |
+| NonfungibleTokenPositionDescriptor |    5,097 |     19,479 |
 | Oracle                             |       86 |     24,490 |
 | OracleLibrary                      |       86 |     24,490 |
-| PairFlash                          |    5,510 |     19,066 |
+| PairFlash                          |    5,428 |     19,148 |
 | Path                               |       86 |     24,490 |
 | PoolAddress                        |       86 |     24,490 |
 | PoolTicksCounter                   |       86 |     24,490 |
 | Position                           |       86 |     24,490 |
 | PositionKey                        |       86 |     24,490 |
 | PositionValue                      |       86 |     24,490 |
-| Quoter                             |    3,944 |     20,632 |
-| QuoterV2                           |    7,103 |     17,473 |
+| Quoter                             |    3,822 |     20,754 |
+| QuoterV2                           |    7,019 |     17,557 |
 | SafeCast                           |       86 |     24,490 |
 | SafeERC20Namer                     |       86 |     24,490 |
 | SafeMath                           |       86 |     24,490 |
@@ -56,16 +55,16 @@
 | SqrtPriceMathPartial               |       86 |     24,490 |
 | Strings                            |       86 |     24,490 |
 | SwapMath                           |       86 |     24,490 |
-| SwapRouter                         |   10,213 |     14,363 |
+| SwapRouter                         |    9,945 |     14,631 |
 | Tick                               |       86 |     24,490 |
 | TickBitmap                         |       86 |     24,490 |
-| TickLens                           |    1,349 |     23,227 |
+| TickLens                           |    1,322 |     23,254 |
 | TickMath                           |       86 |     24,490 |
 | TokenRatioSortOrder                |       86 |     24,490 |
 | TransferHelper                     |       86 |     24,490 |
-| TransparentUpgradeableProxy        |    2,107 |     22,469 |
+| TransparentUpgradeableProxy        |    2,091 |     22,485 |
 | UnsafeMath                         |       86 |     24,490 |
-| UpgradeableBeacon                  |    1,208 |     23,368 |
+| UpgradeableBeacon                  |    1,153 |     23,423 |
 | UpgradeableProxy                   |      738 |     23,838 |
-| V3Migrator                         |    6,084 |     18,492 |
+| V3Migrator                         |    6,529 |     18,047 |
 
diff --git a/remappings.txt b/remappings.txt
@@ -1,4 +1,3 @@
 @openzeppelin/contracts/=lib/openzeppelin-contracts/contracts/
 @uniswap/lib/contracts/=lib/solidity-lib/contracts/
 base64-sol/=lib/base64/
-@katana/v3-contracts/=src/
diff --git a/script/DeployKatanaV3Core.s.sol b/script/DeployKatanaV3Core.s.sol
@@ -2,14 +2,19 @@
 pragma solidity ^0.7.6;
 
 import { Script, console } from "forge-std/Script.sol";
-import { KatanaV3Factory } from "@katana/v3-contracts/core/KatanaV3Factory.sol";
-import { KatanaV3FactoryProxy } from "@katana/v3-contracts/core/KatanaV3FactoryProxy.sol";
+import { KatanaV3Factory } from "src/core/KatanaV3Factory.sol";
+import { KatanaV3Pool } from "src/core/KatanaV3Pool.sol";
+import { KatanaV3PoolBeacon } from "src/core/KatanaV3PoolBeacon.sol";
+import { TransparentUpgradeableProxy } from "@openzeppelin/contracts/proxy/TransparentUpgradeableProxy.sol";
 
 abstract contract DeployKatanaV3Core is Script {
+  address public sender;
   address public proxyAdmin;
   address public governance;
   address public treasury;
 
+  address poolImplementation;
+  address public beacon;
   address public factory;
 
   function setUp() public virtual {
@@ -22,12 +27,18 @@ abstract contract DeployKatanaV3Core is Script {
   function run() public virtual {
     vm.startBroadcast();
 
+    poolImplementation = address(new KatanaV3Pool());
+    beacon = address(new KatanaV3PoolBeacon(poolImplementation));
+
     address factoryImplementation = address(new KatanaV3Factory());
     factory = address(
-      new KatanaV3FactoryProxy(
-        factoryImplementation, proxyAdmin, abi.encodeWithSelector(KatanaV3Factory.initialize.selector, governance, treasury)
+      new TransparentUpgradeableProxy(
+        factoryImplementation,
+        proxyAdmin,
+        abi.encodeWithSelector(KatanaV3Factory.initialize.selector, beacon, governance, treasury)
       )
     );
+
     console.log("KatanaV3Factory deployed:", factory);
 
     vm.stopBroadcast();

diff --git a/script/DeployKatanaV3Periphery.s.sol b/script/DeployKatanaV3Periphery.s.sol
@@ -3,13 +3,12 @@ pragma solidity ^0.7.6;
 
 import { Script, console } from "forge-std/Script.sol";
 import { TransparentUpgradeableProxy } from "@openzeppelin/contracts/proxy/TransparentUpgradeableProxy.sol";
-import { NonfungibleTokenPositionDescriptor } from
-  "@katana/v3-contracts/periphery/NonfungibleTokenPositionDescriptor.sol";
-import { NonfungiblePositionManager } from "@katana/v3-contracts/periphery/NonfungiblePositionManager.sol";
-import { V3Migrator } from "@katana/v3-contracts/periphery/V3Migrator.sol";
-import { TickLens } from "@katana/v3-contracts/periphery/lens/TickLens.sol";
-import { QuoterV2 } from "@katana/v3-contracts/periphery/lens/QuoterV2.sol";
-import { KatanaInterfaceMulticall } from "@katana/v3-contracts/periphery/lens/KatanaInterfaceMulticall.sol";
+import { NonfungibleTokenPositionDescriptor } from "src/periphery/NonfungibleTokenPositionDescriptor.sol";
+import { NonfungiblePositionManager } from "src/periphery/NonfungiblePositionManager.sol";
+import { V3Migrator } from "src/periphery/V3Migrator.sol";
+import { TickLens } from "src/periphery/lens/TickLens.sol";
+import { QuoterV2 } from "src/periphery/lens/QuoterV2.sol";
+import { KatanaInterfaceMulticall } from "src/periphery/lens/KatanaInterfaceMulticall.sol";
 import { DeployKatanaV3Core } from "./DeployKatanaV3Core.s.sol";
 
 abstract contract DeployKatanaV3Periphery is DeployKatanaV3Core {
@@ -47,7 +46,9 @@ abstract contract DeployKatanaV3Periphery is DeployKatanaV3Core {
         abi.encodeWithSelector(NonfungiblePositionManager.initialize.selector)
       )
     );
-    require(NonfungiblePositionManager(payable(nonfungiblePositionManager)).governance() == governance, "governance mismatch");
+    require(
+      NonfungiblePositionManager(payable(nonfungiblePositionManager)).governance() == governance, "governance mismatch"
+    );
     console.log("NonfungiblePositionManager deployed:", nonfungiblePositionManager);
 
     v3migrator = address(new V3Migrator(factory, wron, nonfungiblePositionManager));

diff --git a/script/local/DeployKatanaV3Local.s.sol b/script/local/DeployKatanaV3Local.s.sol
@@ -4,20 +4,22 @@ pragma solidity ^0.7.6;
 import { Script } from "forge-std/Script.sol";
 import { console } from "forge-std/console.sol";
 import { DeployKatanaV3Periphery } from "../DeployKatanaV3Periphery.s.sol";
-import { MixedRouteQuoterV1 } from "@katana/v3-contracts/periphery/lens/MixedRouteQuoterV1.sol";
-import { KatanaGovernanceMock } from "@katana/v3-contracts/external/KatanaGovernanceMock.sol";
+import { MixedRouteQuoterV1 } from "src/periphery/lens/MixedRouteQuoterV1.sol";
+import { KatanaGovernanceMock } from "src/external/KatanaGovernanceMock.sol";
 import { ERC20Mock } from "@openzeppelin/contracts/mocks/ERC20Mock.sol";
 
 contract DeployKatanaV3Local is DeployKatanaV3Periphery {
   address mixedRouteQuoterV1;
 
   function setUp() public override {
     proxyAdmin = makeAddr("ProxyAdmin");
-    governance = address(new KatanaGovernanceMock(address(0), address(0), true));
+    governance = address(new KatanaGovernanceMock(true));
     treasury = makeAddr("RoninTreasury");
     wron = address(new ERC20Mock("Wrapped Ronin", "WRON", address(this), 10 ** 9 * 10 ** 9));
     factoryV2 = makeAddr("KatanaV2Factory");
 
+    sender = DEFAULT_SENDER;
+
     super.setUp();
   }
 

diff --git a/script/ronin-mainnet/DeployKatanaV3Mainnet.s.sol b/script/ronin-mainnet/DeployKatanaV3Mainnet.s.sol
@@ -4,7 +4,7 @@ pragma solidity ^0.7.6;
 import { Script } from "forge-std/Script.sol";
 import { console } from "forge-std/console.sol";
 import { DeployKatanaV3Periphery } from "../DeployKatanaV3Periphery.s.sol";
-import { MixedRouteQuoterV1 } from "@katana/v3-contracts/periphery/lens/MixedRouteQuoterV1.sol";
+import { MixedRouteQuoterV1 } from "src/periphery/lens/MixedRouteQuoterV1.sol";
 
 contract DeployKatanaV3Mainnet is DeployKatanaV3Periphery {
   address mixedRouteQuoterV1;
@@ -16,7 +16,7 @@ contract DeployKatanaV3Mainnet is DeployKatanaV3Periphery {
     wron = 0xe514d9DEB7966c8BE0ca922de8a064264eA6bcd4; // WRON
     factoryV2 = 0xB255D6A720BB7c39fee173cE22113397119cB930; // Katana V2 Factory
 
-    vm.rememberKey(vm.envUint("MAINNET_PK"));
+    sender = vm.rememberKey(vm.envUint("MAINNET_PK"));
 
     super.setUp();
   }

diff --git a/script/ronin-testnet/DeployKatanaV3Testnet.s.sol b/script/ronin-testnet/DeployKatanaV3Testnet.s.sol
@@ -4,7 +4,7 @@ pragma solidity ^0.7.6;
 import { Script } from "forge-std/Script.sol";
 import { console } from "forge-std/console.sol";
 import { DeployKatanaV3Periphery } from "../DeployKatanaV3Periphery.s.sol";
-import { MixedRouteQuoterV1Testnet } from "@katana/v3-contracts/periphery/lens/MixedRouteQuoterV1Testnet.sol";
+import { MixedRouteQuoterV1Testnet } from "src/periphery/lens/MixedRouteQuoterV1Testnet.sol";
 
 contract DeployKatanaV3Testnet is DeployKatanaV3Periphery {
   address mixedRouteQuoterV1Testnet;
@@ -16,7 +16,7 @@ contract DeployKatanaV3Testnet is DeployKatanaV3Periphery {
     wron = 0xA959726154953bAe111746E265E6d754F48570E6;
     factoryV2 = 0x86587380C4c815Ba0066c90aDB2B45CC9C15E72c;
 
-    vm.rememberKey(vm.envUint("TESTNET_PK"));
+    sender = vm.rememberKey(vm.envUint("TESTNET_PK"));
 
     super.setUp();
   }
+0 −4		CHANGELOG.md
+3 −4		contracts/access/TimelockController.sol
+1 −1		contracts/package.json
+1 −1		package-lock.json
+1 −1		package.json