@venusprotocol/liquidity-hub 1.0.0 → 2.0.0-dev.1

package/README.md

@@ -439,6 +439,6 @@ Ordering is guaranteed by the call-array index (`addHub` before any `addYieldGro
 
 **Reconciliation:** a Hub is live the moment script 06 runs but discoverable only after its VIP — `deploy/ReconcileHubRegistry.s.sol` (read-only, unnumbered) diffs the committed `deployments/<network>/Hub_*.json` set against `registry.getHubs()` in both directions and reverts on any mismatch (shadow Hubs deployed-but-unregistered, or registered-but-untracked). Run it after every onboarding VIP and periodically from monitoring.
 
-`deploy/config/<network>.json` holds only public addresses and parameters — `acm`, `governance`, `fluxLendingResolver`, optional `wbnb` / `vBNB` (the Migrator's native path; omit or zero to disable it), and per-asset `key` / `asset` / `name` / `symbol` / `decimalsOffset` / `initialMaxWithdrawalSize` / `blocksPerYear` / `feeRecipient`. The committed `bscmainnet.json` ships with `0x0` placeholders for governance/acm/resolver/recipient and must be filled before a real deploy. Secrets (deployer key, RPC URLs, Etherscan key) live only in `.env` (see `.env.example`). Deployed addresses are committed under `deployments/<network>/<name>.json`; `broadcast/` is gitignored and is not the source of truth.
+`deploy/config/<network>.json` holds only public addresses and parameters — `acm`, `governance`, `fluxLendingResolver`, `wbnb` / `vBNB` (the Migrator's native path; both keys are required, set them to `0x0` to deploy with that path disabled), and per-asset `key` / `asset` / `name` / `symbol` / `decimalsOffset` / `initialMaxWithdrawalSize` / `blocksPerYear` / `feeRecipient`. The committed `bscmainnet.json` ships with `0x0` placeholders for governance/acm/resolver/recipient and must be filled before a real deploy. Secrets (deployer key, RPC URLs, Etherscan key) live only in `.env` (see `.env.example`). Deployed addresses are committed under `deployments/<network>/<name>.json`; `broadcast/` is gitignored and is not the source of truth.
 
 **Networks:** RPC aliases are pre-wired for `bsc_mainnet`, `bsc_testnet`, `ethereum`, `sepolia` (one Etherscan V2 key across chains). v1 targets BNB Chain.

package/contracts/interfaces/IMigrator.sol

@@ -7,10 +7,8 @@ pragma solidity 0.8.25;
  * @notice One-click migration from an existing Venus Core position into a Liquidity Hub. In a
  *         single atomic transaction the Migrator pulls a user's Core vToken, redeems it to the
  *         underlying, and deposits that into the matching Hub, minting share tokens to `receiver`.
- * @dev A single immutable, permissionless deployment serves every registered Hub — the target is
- *      chosen per call via the `hub` argument and must be present in the HubRegistry, so a migration
- *      can only ever land in a governance-registered Hub. Two paths exist because Venus splits its
- *      markets by asset type:
+ * @dev A single immutable, permissionless deployment serves every Hub — the target is chosen per
+ *      call via the `hub` argument. Two paths exist because Venus splits its markets by asset type:
  *      ERC-20-backed markets (vUSDT, vUSDC, ...) go through {migrateFromCore}; the native market
  *      (vBNB) redeems raw BNB and so has the dedicated {migrateFromCoreBNB} path that wraps to WBNB
  *      before depositing into the WBNB Hub.
@@ -38,8 +36,6 @@ interface IMigrator {
     error ZeroAddress();
     /// @notice A required amount argument was zero.
     error ZeroAmount();
-    /// @notice The target `hub` is not a governance-registered Hub in the HubRegistry.
-    error HubNotRegistered(address hub);
     /// @notice The vToken's underlying does not match the Hub's asset.
     error AssetMismatch(address vTokenUnderlying, address hubAsset);
     /// @notice The Core market returned a non-zero (failure) error code on redeem.
@@ -48,6 +44,8 @@ interface IMigrator {
     error NothingReceived();
     /// @notice The minted shares were below the caller's `minShares` floor.
     error InsufficientShares(uint256 shares, uint256 minShares);
+    /// @notice The native (vBNB → WBNB) path is unavailable: this deployment has no WBNB/vBNB wired.
+    error BNBMigrationUnsupported();
     /// @notice Native value was received from an address other than the configured vBNB market.
     error OnlyVBNB();
 
@@ -97,8 +95,8 @@ interface IMigrator {
     /**
      * @notice Migrate a native vBNB position into the WBNB Hub.
      * @dev Caller must first `approve` this Migrator to transfer `vBNBAmount` of the configured
-     *      vBNB market. The redeemed BNB is wrapped to WBNB before deposit. Reverts unless `hub` is a
-     *      registered WBNB Hub (`IHub(hub).asset() == WBNB`) and `shares >= minShares`.
+     *      vBNB market. The redeemed BNB is wrapped to WBNB before deposit. Reverts unless this
+     *      deployment has WBNB/vBNB wired, `IHub(hub).asset() == WBNB`, and `shares >= minShares`.
      *      This is the no-consent path — use {migrateFromCoreBNBWithConsent} to record consent.
      * @param vBNBAmount vBNB receipts to pull and redeem.
      * @param hub The WBNB Hub to deposit into.

package/contracts/migrator/Migrator.sol

@@ -6,7 +6,6 @@ import { SafeERC20 } from "@openzeppelin/contracts/token/ERC20/utils/SafeERC20.s
 import { ReentrancyGuard } from "@openzeppelin/contracts/security/ReentrancyGuard.sol";
 
 import { IHub } from "../interfaces/IHub.sol";
-import { IHubRegistry } from "../interfaces/IHubRegistry.sol";
 import { IMigrator } from "../interfaces/IMigrator.sol";
 import { IVBep20 } from "../interfaces/external/IVBep20.sol";
 import { IWBNB } from "../interfaces/external/IWBNB.sol";
@@ -15,16 +14,12 @@ import { IWBNB } from "../interfaces/external/IWBNB.sol";
  * @title Migrator
  * @author Venus
  * @notice One-click migration from a Venus Core position into a Liquidity Hub. A single immutable,
- *         permissionless deployment serves every registered Hub; the target Hub is supplied per call
- *         and must be present in the HubRegistry.
+ *         permissionless deployment serves every Hub; the target Hub is supplied per call.
  * @dev Stateless by design — no proxy, no admin, no funds held between calls. Each migration is
  *      atomic: pull the user's vToken, redeem it to the underlying, deposit that into the Hub, and
  *      mint shares straight to the receiver. If any leg fails the whole call reverts and the user
  *      keeps their vToken.
  *
- *      The target `hub` is checked against the immutable {HUB_REGISTRY} before any funds move, so a
- *      migration can only land in a Hub governance has registered.
- *
  *      Accounting is done by balance delta, never by a self-reported amount: Core `redeem` returns
  *      a Compound error code (not the redeemed amount) and silently nets out any Comptroller
  *      treasury fee, so the Migrator measures what actually arrived and deposits exactly that. The
@@ -39,24 +34,16 @@ import { IWBNB } from "../interfaces/external/IWBNB.sol";
 contract Migrator is IMigrator, ReentrancyGuard {
     using SafeERC20 for IERC20;
 
-    /// @notice Governance-curated directory of valid Hubs.
-    IHubRegistry public immutable HUB_REGISTRY;
-    /// @notice Canonical wrapped-native token (WBNB).
+    /// @notice Canonical wrapped-native token; `address(0)` disables the native path.
     IWBNB public immutable WBNB;
-    /// @notice Canonical Venus native market (vBNB).
+    /// @notice Canonical Venus native market (vBNB); `address(0)` disables the native path.
     IVBep20 public immutable VBNB;
 
     /**
-     * @param wbnb_ Wrapped-native token (WBNB). Mandatory; the constructor reverts on the zero address.
-     * @param vbnb_ Venus native market (vBNB). Mandatory; the constructor reverts on the zero address.
-     * @param hubRegistry_ The HubRegistry every migration target is validated against. Mandatory;
-     *        the constructor reverts on the zero address, since without it no Hub would be accepted.
+     * @param wbnb_ Wrapped-native token, or `address(0)` on chains/deployments with no WBNB Hub.
+     * @param vbnb_ Venus native market (vBNB), or `address(0)` to leave the native path disabled.
      */
-    constructor(IWBNB wbnb_, IVBep20 vbnb_, IHubRegistry hubRegistry_) {
-        if (address(wbnb_) == address(0) || address(vbnb_) == address(0) || address(hubRegistry_) == address(0)) {
-            revert ZeroAddress();
-        }
-        HUB_REGISTRY = hubRegistry_;
+    constructor(IWBNB wbnb_, IVBep20 vbnb_) {
         WBNB = wbnb_;
         VBNB = vbnb_;
     }
@@ -124,7 +111,6 @@ contract Migrator is IMigrator, ReentrancyGuard {
             revert ZeroAddress();
         }
         if (vTokenAmount == 0) revert ZeroAmount();
-        if (!HUB_REGISTRY.isHub(hub)) revert HubNotRegistered(hub);
 
         address asset = IHub(hub).asset();
         address vTokenUnderlying = IVBep20(vToken).underlying();
@@ -149,9 +135,11 @@ contract Migrator is IMigrator, ReentrancyGuard {
         private
         returns (uint256 shares)
     {
+        if (address(WBNB) == address(0) || address(VBNB) == address(0)) {
+            revert BNBMigrationUnsupported();
+        }
         if (hub == address(0) || receiver == address(0)) revert ZeroAddress();
         if (vBNBAmount == 0) revert ZeroAmount();
-        if (!HUB_REGISTRY.isHub(hub)) revert HubNotRegistered(hub);
 
         address asset = IHub(hub).asset();
         if (asset != address(WBNB)) revert AssetMismatch(address(WBNB), asset);

package/deployments/bscmainnet/Migrator.json

@@ -1,6 +1,6 @@
 {
-  "address": "0xb0D77DC86d4E6a8282eECF3d4d456B3b94F2a7aD",
-  "block": 115268335,
+  "address": "0xfe6b8BEf1215C19Cd247FbF495ef560932F1Eb9B",
+  "block": 112781054,
   "chainId": 56,
   "contract": "Migrator",
   "abi": [
@@ -16,11 +16,6 @@
           "name": "vbnb_",
           "type": "address",
           "internalType": "contract IVBep20"
-        },
-        {
-          "name": "hubRegistry_",
-          "type": "address",
-          "internalType": "contract IHubRegistry"
         }
       ],
       "stateMutability": "nonpayable"
@@ -29,19 +24,6 @@
       "type": "receive",
       "stateMutability": "payable"
     },
-    {
-      "type": "function",
-      "name": "HUB_REGISTRY",
-      "inputs": [],
-      "outputs": [
-        {
-          "name": "",
-          "type": "address",
-          "internalType": "contract IHubRegistry"
-        }
-      ],
-      "stateMutability": "view"
-    },
     {
       "type": "function",
       "name": "VBNB",
@@ -310,14 +292,8 @@
     },
     {
       "type": "error",
-      "name": "HubNotRegistered",
-      "inputs": [
-        {
-          "name": "hub",
-          "type": "address",
-          "internalType": "address"
-        }
-      ]
+      "name": "BNBMigrationUnsupported",
+      "inputs": []
     },
     {
       "type": "error",

package/deployments/bscmainnet_addresses.json

@@ -24,7 +24,7 @@
     "Hub_U": "0x0e5AA174d4F31b757a237eb1999DE151596788B0",
     "Hub_USDC": "0x9D2D9592cF8DFbf59107fAab703d08494BE14617",
     "Hub_USDT": "0x18AfDACF30F8671021dec4b78297E39d2FE87226",
-    "Migrator": "0xb0D77DC86d4E6a8282eECF3d4d456B3b94F2a7aD",
+    "Migrator": "0xfe6b8BEf1215C19Cd247FbF495ef560932F1Eb9B",
     "YieldGroupCoreImpl": "0x3BccED778CAaf97AE8ff8Bedc16d2165aA15771b",
     "YieldGroupFRVImpl": "0x72D115b1Be69a2065fED0B0574F95B3E46BD4502",
     "YieldGroupFluxImpl": "0xFD49a719E0bF7f588F77bBEdf10dd8E843774499"

package/deployments/bsctestnet/Migrator.json

@@ -1,6 +1,6 @@
 {
-  "address": "0x907d344E53A6e51Cb3E01cb76495D6C5325d72e8",
-  "block": 124402350,
+  "address": "0x343D518d8C89f9B5D770000F1ed80f45bF1419f5",
+  "block": 119460060,
   "chainId": 97,
   "contract": "Migrator",
   "abi": [
@@ -16,11 +16,6 @@
           "name": "vbnb_",
           "type": "address",
           "internalType": "contract IVBep20"
-        },
-        {
-          "name": "hubRegistry_",
-          "type": "address",
-          "internalType": "contract IHubRegistry"
         }
       ],
       "stateMutability": "nonpayable"
@@ -29,19 +24,6 @@
       "type": "receive",
       "stateMutability": "payable"
     },
-    {
-      "type": "function",
-      "name": "HUB_REGISTRY",
-      "inputs": [],
-      "outputs": [
-        {
-          "name": "",
-          "type": "address",
-          "internalType": "contract IHubRegistry"
-        }
-      ],
-      "stateMutability": "view"
-    },
     {
       "type": "function",
       "name": "VBNB",
@@ -310,14 +292,8 @@
     },
     {
       "type": "error",
-      "name": "HubNotRegistered",
-      "inputs": [
-        {
-          "name": "hub",
-          "type": "address",
-          "internalType": "address"
-        }
-      ]
+      "name": "BNBMigrationUnsupported",
+      "inputs": []
     },
     {
       "type": "error",

package/deployments/bsctestnet_addresses.json

@@ -17,7 +17,7 @@
     "HubRegistryImpl": "0x4D2C18fB4520c2e4f7C754979e9a4F3BbC1BCe92",
     "HubRegistryProxyAdmin": "0x9f8413eEE33D434F6D4f40C83181f32A831c9ef7",
     "Hub_USDT": "0x7cE6ADF754D0eC81A6CF8ACd9C7454F45077dc61",
-    "Migrator": "0x907d344E53A6e51Cb3E01cb76495D6C5325d72e8",
+    "Migrator": "0x343D518d8C89f9B5D770000F1ed80f45bF1419f5",
     "YieldGroupCoreImpl": "0x96b839765b7bc062e354f57c8F9f7Be328C4fb63",
     "YieldGroupFRVImpl": "0x4fC5cFafCa279777DfF65f7D7e19205d3785304C",
     "YieldGroupFluxImpl": "0xEe52fb9ec5bcD6868a3D7f2FBb3e71e9D6Fc97dd"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@venusprotocol/liquidity-hub",
-  "version": "1.0.0",
+  "version": "2.0.0-dev.1",
   "description": "Per-asset ERC-4626 allocator vault",
   "author": "Venus",
   "license": "BUSL-1.1",