@etherisc/gif-next 3.0.0-c419b59-617 → 3.0.0-c46a89a-335

package/contracts/upgradeability/ProxyManager.sol

@@ -5,14 +5,14 @@ import {ProxyAdmin} from "@openzeppelin/contracts/proxy/transparent/ProxyAdmin.s
 import {ITransparentUpgradeableProxy} from "@openzeppelin/contracts/proxy/transparent/TransparentUpgradeableProxy.sol";
 
 import {Blocknumber, BlocknumberLib} from "../type/Blocknumber.sol";
-import {IVersionable} from "./IVersionable.sol";
+import {IUpgradeable} from "./IUpgradeable.sol";
 import {NftId} from "../type/NftId.sol";
 import {NftOwnable} from "../shared/NftOwnable.sol";
 import {Timestamp, TimestampLib} from "../type/Timestamp.sol";
 import {UpgradableProxyWithAdmin} from "./UpgradableProxyWithAdmin.sol";
-import {Version, VersionLib} from "../type/Version.sol";
+import {Version, VersionPart, VersionLib} from "../type/Version.sol";
 
-/// @dev manages proxy deployments for upgradable contracs of type IVersionable
+/// @dev manages proxy deployments for upgradable contracs of type IUpgradeable
 contract ProxyManager is
     NftOwnable
 {
@@ -27,14 +27,15 @@ contract ProxyManager is
         address activatedBy;
     }
 
-    event LogProxyManagerVersionableDeployed(address indexed proxy, address initialImplementation);
-    event LogProxyManagerVersionableUpgraded(address indexed proxy, address upgradedImplementation);
+    event LogProxyManagerProxyDeployed(address indexed proxy, address initialImplementation);
+    event LogProxyManagerProxyUpgraded(address indexed proxy, address upgradedImplementation);
 
     error ErrorProxyManagerAlreadyDeployed();
     error ErrorProxyManagerNotYetDeployed();
 
     error ErrorProxyManagerZeroVersion();
     error ErrorProxyManagerNextVersionNotIncreasing(Version nextVersion);
+    error ErrorProxyManagerNextVersionReleaseInvalid(Version nextVersion);
 
     UpgradableProxyWithAdmin internal _proxy;
 
@@ -44,17 +45,19 @@ contract ProxyManager is
 
     /// @dev convencience initializer
     function initialize(
-        address registry,
         address implementation,
         bytes memory data,
         bytes32 salt
     )
         public
         initializer()
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
-        versionable = deployDetermenistic(
-            registry,
+        address initialOwner = msg.sender;
+
+        __NftOwnable_init(initialOwner);
+
+        upgradeable = deployDetermenistic(
             implementation, 
             data,
             salt);
@@ -62,19 +65,20 @@ contract ProxyManager is
 
     /// @dev deploy initial contract
     function deploy(
-        address registry, 
         address initialImplementation, 
         bytes memory initializationData
     )
         public
         virtual
         onlyInitializing()
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
-        (
-            address currentProxyOwner, 
-            address initialProxyAdminOwner
-        ) = _preDeployChecksAndSetup(registry);
+        if (_versions.length > 0) {
+            revert ErrorProxyManagerAlreadyDeployed();
+        }
+
+        address currentProxyOwner = getOwner(); // used by implementation
+        address initialProxyAdminOwner = address(this); // used by proxy
 
         _proxy = new UpgradableProxyWithAdmin(
             initialImplementation,
@@ -82,15 +86,14 @@ contract ProxyManager is
             getDeployData(currentProxyOwner, initializationData)
         );
 
-        versionable = _updateVersionHistory(
+        upgradeable = _updateVersionHistory(
             initialImplementation, 
             currentProxyOwner);
 
-        emit LogProxyManagerVersionableDeployed(address(versionable), initialImplementation);
+        emit LogProxyManagerProxyDeployed(address(upgradeable), initialImplementation);
     }
 
     function deployDetermenistic(
-        address registry, 
         address initialImplementation, 
         bytes memory initializationData, 
         bytes32 salt
@@ -98,12 +101,14 @@ contract ProxyManager is
         public
         virtual
         onlyInitializing()
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
-        (
-            address currentProxyOwner, 
-            address initialProxyAdminOwner
-        ) = _preDeployChecksAndSetup(registry);
+        if (_versions.length > 0) {
+            revert ErrorProxyManagerAlreadyDeployed();
+        }
+
+        address currentProxyOwner = getOwner(); // used by implementation
+        address initialProxyAdminOwner = address(this); // used by proxy
 
         _proxy = new UpgradableProxyWithAdmin{salt: salt}(
             initialImplementation,
@@ -111,11 +116,11 @@ contract ProxyManager is
             getDeployData(currentProxyOwner, initializationData)
         );
 
-        versionable = _updateVersionHistory(
+        upgradeable = _updateVersionHistory(
             initialImplementation, 
             currentProxyOwner);
 
-        emit LogProxyManagerVersionableDeployed(address(versionable), initialImplementation);
+        emit LogProxyManagerProxyDeployed(address(upgradeable), initialImplementation);
     }
 
     /// @dev upgrade existing contract.
@@ -124,7 +129,7 @@ contract ProxyManager is
         public
         virtual
         onlyOwner()
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
         bytes memory emptyUpgradeData;
         return upgrade(newImplementation, emptyUpgradeData);
@@ -135,7 +140,7 @@ contract ProxyManager is
         public
         virtual
         onlyOwner()
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
         if (_versions.length == 0) { 
             revert ErrorProxyManagerNotYetDeployed();
@@ -150,11 +155,11 @@ contract ProxyManager is
             newImplementation, 
             getUpgradeData(upgradeData));
 
-        versionable = _updateVersionHistory(
+        upgradeable = _updateVersionHistory(
             newImplementation, 
             currentProxyOwner);
 
-        emit LogProxyManagerVersionableUpgraded(address(versionable), newImplementation);
+        emit LogProxyManagerProxyUpgraded(address(upgradeable), newImplementation);
 
     }
 
@@ -164,14 +169,14 @@ contract ProxyManager is
 
     function getDeployData(address proxyOwner, bytes memory deployData) public pure returns (bytes memory data) {
         return abi.encodeWithSelector(
-            IVersionable.initializeVersionable.selector, 
+            IUpgradeable.initialize.selector, 
             proxyOwner, 
             deployData);
     }
 
     function getUpgradeData(bytes memory upgradeData) public pure returns (bytes memory data) {
         return abi.encodeWithSelector(
-            IVersionable.upgradeVersionable.selector, 
+            IUpgradeable.upgrade.selector, 
             upgradeData);
     }
 
@@ -180,7 +185,7 @@ contract ProxyManager is
     }
 
     function getVersion() external view virtual returns(Version) {
-        return IVersionable(address(_proxy)).getVersion();
+        return _versionHistory[_versions[_versions.length]].version;
     }
 
     function getVersionCount() external view returns(uint256) {
@@ -195,42 +200,29 @@ contract ProxyManager is
         return _versionHistory[_version];
     }
 
-    function _preDeployChecksAndSetup(address registry)
-        private
-        returns (
-            address currentProxyOwner,
-            address initialProxyAdminOwner
-        )
-    {
-        if (_versions.length > 0) {
-            revert ErrorProxyManagerAlreadyDeployed();
-        }
-
-        address initialOwner = msg.sender;
-        __NftOwnable_init(registry, initialOwner);
-
-        currentProxyOwner = getOwner(); // used by implementation
-        initialProxyAdminOwner = address(this); // used by proxy
-    }
-
     function _updateVersionHistory(
         address implementation,
         address activatedBy
     )
         private
-        returns (IVersionable versionable)
+        returns (IUpgradeable upgradeable)
     {
-        versionable = IVersionable(address(_proxy));
-        Version newVersion = versionable.getVersion();
+        upgradeable = IUpgradeable(address(_proxy));
+        Version newVersion = upgradeable.getVersion();
 
         if(newVersion == VersionLib.zeroVersion()) {
             revert ErrorProxyManagerZeroVersion();
         }
 
         if(_versions.length > 0) {
-            if(newVersion.toInt() <= _versions[_versions.length-1].toInt()) {
+            Version version = _versions[_versions.length-1];
+            if(newVersion.toInt() <= version.toInt()) {
                 revert ErrorProxyManagerNextVersionNotIncreasing(newVersion);
             }
+
+            if(newVersion.toMajorPart() != version.toMajorPart()) {
+                revert ErrorProxyManagerNextVersionReleaseInvalid(newVersion);
+            }
         }
 
         // update version history

package/contracts/upgradeability/{Versionable.sol → Upgradeable.sol}

@@ -3,39 +3,40 @@ pragma solidity ^0.8.20;
 
 import {Initializable} from "@openzeppelin/contracts-upgradeable/proxy/utils/Initializable.sol";
 
-import {IVersionable} from "./IVersionable.sol";
-import {Version, VersionLib} from "../type/Version.sol";
+import {IUpgradeable} from "./IUpgradeable.sol";
+import {Version} from "../type/Version.sol";
+import {Versionable} from "../shared/Versionable.sol";
 
-
-abstract contract Versionable is 
-    Initializable,
-    IVersionable 
+abstract contract Upgradeable is
+    Initializable, 
+    Versionable,
+    IUpgradeable 
 {
     constructor() {
         _disableInitializers();
     }
 
-    function initializeVersionable(
+    function initialize(
         address activatedBy,
         bytes memory data
     )
-        public
-        initializer()
+        external
     {
-        _initialize(activatedBy, data);
+        if(_getInitializedVersion() != 0) {
+            revert InvalidInitialization();
+        }
+        __Upgradeable_init(activatedBy, data);
     }
 
-    function upgradeVersionable(
+    function upgrade(
         bytes memory data
     )
         external
-        reinitializer(VersionLib.toUint64(getVersion()))
+        reinitializer(getVersion().toInt())
     {
         _upgrade(data);
     }
 
-    function getVersion() public pure virtual returns(Version);
-
     // IMPORTANT each version must implement this function 
     // each implementation MUST use onlyInitialising modifier
     // each implementation MUST call intializers of all base contracts...
@@ -59,4 +60,14 @@ abstract contract Versionable is
     {
         revert ErrorVersionableUpgradeNotImplemented();
     }
+
+    function __Upgradeable_init(
+        address activatedBy,
+        bytes memory data
+    )
+        private
+        reinitializer(getVersion().toInt())
+    {
+        _initialize(activatedBy, data);
+    }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@etherisc/gif-next",
-  "version": "3.0.0-c419b59-617",
+  "version": "3.0.0-c46a89a-335",
   "description": "This is the repository for the next version of the Generic Insurance Framework (GIF) smart contracts. ",
   "main": "index.js",
   "scripts": {
@@ -50,6 +50,7 @@
     "antlr4ng": "^3.0.4",
     "ethers": "^6.7.1",
     "ethers-decode-error": "^2.1.3",
+    "simple-git": "^3.27.0",
     "solidity-docgen": "^0.6.0-beta.36",
     "winston": "^3.10.0"
   }

package/artifacts/contracts/shared/ContractLib.sol/IInstanceAdminHelper.dbg.json

@@ -1,4 +0,0 @@
-{
-  "_format": "hh-sol-dbg-1",
-  "buildInfo": "../../../build-info/a11472d155cc1e3900b16d2850f59142.json"
-}

package/artifacts/contracts/shared/ContractLib.sol/IInstanceAdminHelper.json

@@ -1,24 +0,0 @@
-{
-  "_format": "hh-sol-artifact-1",
-  "contractName": "IInstanceAdminHelper",
-  "sourceName": "contracts/shared/ContractLib.sol",
-  "abi": [
-    {
-      "inputs": [],
-      "name": "getInstanceAdmin",
-      "outputs": [
-        {
-          "internalType": "contract ITargetHelper",
-          "name": "",
-          "type": "address"
-        }
-      ],
-      "stateMutability": "view",
-      "type": "function"
-    }
-  ],
-  "bytecode": "0x",
-  "deployedBytecode": "0x",
-  "linkReferences": {},
-  "deployedLinkReferences": {}
-}

package/artifacts/contracts/shared/ContractLib.sol/ITokenRegistryHelper.dbg.json

@@ -1,4 +0,0 @@
-{
-  "_format": "hh-sol-dbg-1",
-  "buildInfo": "../../../build-info/a11472d155cc1e3900b16d2850f59142.json"
-}

package/artifacts/contracts/shared/ContractLib.sol/ITokenRegistryHelper.json

@@ -1,40 +0,0 @@
-{
-  "_format": "hh-sol-artifact-1",
-  "contractName": "ITokenRegistryHelper",
-  "sourceName": "contracts/shared/ContractLib.sol",
-  "abi": [
-    {
-      "inputs": [
-        {
-          "internalType": "ChainId",
-          "name": "chainId",
-          "type": "uint96"
-        },
-        {
-          "internalType": "address",
-          "name": "token",
-          "type": "address"
-        },
-        {
-          "internalType": "VersionPart",
-          "name": "release",
-          "type": "uint8"
-        }
-      ],
-      "name": "isActive",
-      "outputs": [
-        {
-          "internalType": "bool",
-          "name": "",
-          "type": "bool"
-        }
-      ],
-      "stateMutability": "view",
-      "type": "function"
-    }
-  ],
-  "bytecode": "0x",
-  "deployedBytecode": "0x",
-  "linkReferences": {},
-  "deployedLinkReferences": {}
-}

package/artifacts/contracts/upgradeability/IVersionable.sol/IVersionable.dbg.json

@@ -1,4 +0,0 @@
-{
-  "_format": "hh-sol-dbg-1",
-  "buildInfo": "../../../build-info/a11472d155cc1e3900b16d2850f59142.json"
-}

package/artifacts/contracts/upgradeability/Versionable.sol/Versionable.dbg.json

@@ -1,4 +0,0 @@
-{
-  "_format": "hh-sol-dbg-1",
-  "buildInfo": "../../../build-info/a11472d155cc1e3900b16d2850f59142.json"
-}