Replace dual-impl deployment with version selector

Rather than deploying both V1 and V2 implementations, the script now
reads stark_exchange_migration.version from the config (1 or 2) and
deploys only the selected implementation. Config files updated to add
the version field; input configs set to version 2 (V2 not yet deployed),
deployed configs set to version 1 (V1 currently deployed).

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>

Author: lfportal

config/deploy/mainnet/eth_config_input.json

@@ -8,7 +8,7 @@
     "axelar_gateway": "0x4F4495243837681061C4743b74B3eEdf548D56A5"
   },
   "stark_exchange_migration": {
-    "implementation_address": "0x58b5484F489f7858DC83a5a677338074b57de806",
-    "v2_implementation_address": "0x0000000000000000000000000000000000000000"
+    "version": 2,
+    "implementation_address": "0x0000000000000000000000000000000000000000"
   }
 }

config/deploy/mainnet/eth_deployed.json

@@ -8,7 +8,7 @@
     "axelar_gateway": "0x4F4495243837681061C4743b74B3eEdf548D56A5"
   },
   "stark_exchange_migration": {
-    "implementation_address": "0x58b5484F489f7858DC83a5a677338074b57de806",
-    "v2_implementation_address": "0x0000000000000000000000000000000000000000"
+    "version": 1,
+    "implementation_address": "0x58b5484F489f7858DC83a5a677338074b57de806"
   }
 }

config/deploy/sandbox/eth_config_input.json

@@ -8,7 +8,7 @@
     "axelar_gateway": "0xe432150cce91c13a887f7D836923d5597adD8E31"
   },
   "stark_exchange_migration": {
-    "implementation_address": "0x9F9b4A495A62191A4225759Ae7C00906Cc6417B8",
-    "v2_implementation_address": "0x0000000000000000000000000000000000000000"
+    "version": 2,
+    "implementation_address": "0x0000000000000000000000000000000000000000"
   }
 }

config/deploy/sandbox/eth_deployed.json

@@ -8,7 +8,7 @@
     "axelar_gateway": "0xe432150cce91c13a887f7D836923d5597adD8E31"
   },
   "stark_exchange_migration": {
-    "implementation_address": "0x9F9b4A495A62191A4225759Ae7C00906Cc6417B8",
-    "v2_implementation_address": "0x0000000000000000000000000000000000000000"
+    "version": 1,
+    "implementation_address": "0x9F9b4A495A62191A4225759Ae7C00906Cc6417B8"
   }
 }

script/DeployEthContracts.s.sol

@@ -12,8 +12,7 @@ import {StarkExchangeMigrationV2} from "../src/bridge/starkex/StarkExchangeMigra
  * @title DeployEthContracts
  * @notice Deploys the migration contracts on Ethereum:
  *         1. VaultRootSenderAdapter
- *         2. StarkExchangeMigration (implementation only, no proxy)
- *         3. StarkExchangeMigrationV2 (implementation only, no proxy)
+ *         2. StarkExchangeMigration implementation (version selected via config)
  *
  * @dev The StarkExchangeMigration is deployed as an implementation contract only.
  *      The existing StarkEx bridge proxy on-chain will be upgraded to point to this
@@ -38,11 +37,9 @@ contract DeployEthContracts is Script {
     address private axelarGasService;
     address private axelarGateway;
 
-    // StarkExchangeMigration
+    // StarkExchangeMigration implementation (version 1 or 2, selected via config)
     address private migrationImpl;
-
-    // StarkExchangeMigrationV2
-    address private migrationV2Impl;
+    uint256 private migrationVersion;
 
     string private configFilePath;
     string private outputFilePath;
@@ -67,9 +64,7 @@ contract DeployEthContracts is Script {
 
         // --- StarkExchangeMigration ---
         migrationImpl = vm.parseJsonAddress(config, "$.stark_exchange_migration.implementation_address");
-
-        // --- StarkExchangeMigrationV2 ---
-        migrationV2Impl = vm.parseJsonAddress(config, "$.stark_exchange_migration.v2_implementation_address");
+        migrationVersion = vm.parseJsonUint(config, "$.stark_exchange_migration.version");
     }
 
     /// @notice Deploys the Ethereum contracts.
@@ -91,24 +86,19 @@ contract DeployEthContracts is Script {
             console.log("Using existing VaultRootSenderAdapter:", senderAdapter);
         }
 
-        // 2. Deploy StarkExchangeMigration implementation (if not already deployed)
+        // 2. Deploy StarkExchangeMigration implementation (if not already deployed).
+        //    The version is selected via stark_exchange_migration.version in the config.
         //    The constructor only calls _disableInitializers(). No proxy is deployed here.
         //    The existing StarkEx bridge proxy will be upgraded separately via governance.
         if (migrationImpl == address(0)) {
-            migrationImpl = address(new StarkExchangeMigration());
-            console.log("Deployed StarkExchangeMigration implementation:", migrationImpl);
-        } else {
-            console.log("Using existing StarkExchangeMigration implementation:", migrationImpl);
-        }
-
-        // 3. Deploy StarkExchangeMigrationV2 implementation (if not already deployed)
-        //    The constructor only calls _disableInitializers(). No proxy is deployed here.
-        //    The existing StarkEx bridge proxy will be upgraded separately via governance.
-        if (migrationV2Impl == address(0)) {
-            migrationV2Impl = address(new StarkExchangeMigrationV2());
-            console.log("Deployed StarkExchangeMigrationV2 implementation:", migrationV2Impl);
+            if (migrationVersion == 1) {
+                migrationImpl = address(new StarkExchangeMigration());
+            } else {
+                migrationImpl = address(new StarkExchangeMigrationV2());
+            }
+            console.log("Deployed StarkExchangeMigration v%d implementation:", migrationVersion, migrationImpl);
         } else {
-            console.log("Using existing StarkExchangeMigrationV2 implementation:", migrationV2Impl);
+            console.log("Using existing StarkExchangeMigration v%d implementation:", migrationVersion, migrationImpl);
         }
 
         vm.stopBroadcast();
@@ -131,7 +121,8 @@ contract DeployEthContracts is Script {
             require(axelarGateway != address(0), "vault_root_sender_adapter.axelar_gateway is zero address");
         }
 
-        // StarkExchangeMigration: no constructor params to validate (only calls _disableInitializers)
+        // StarkExchangeMigration: validate version selector
+        require(migrationVersion == 1 || migrationVersion == 2, "stark_exchange_migration.version must be 1 or 2");
     }
 
     /**
@@ -144,17 +135,13 @@ contract DeployEthContracts is Script {
 
         vm.writeJson(vm.toString(senderAdapter), outputFilePath, ".vault_root_sender_adapter.address");
         vm.writeJson(vm.toString(migrationImpl), outputFilePath, ".stark_exchange_migration.implementation_address");
-        vm.writeJson(
-            vm.toString(migrationV2Impl), outputFilePath, ".stark_exchange_migration.v2_implementation_address"
-        );
 
         console.log("Deployment output written to:", outputFilePath);
     }
 
     function _logSummary() private view {
         console.log("--- Ethereum Deployment Summary ---");
         console.log("VaultRootSenderAdapter:", senderAdapter);
-        console.log("StarkExchangeMigration (impl):", migrationImpl);
-        console.log("StarkExchangeMigrationV2 (impl):", migrationV2Impl);
+        console.log("StarkExchangeMigration v%d (impl):", migrationVersion, migrationImpl);
     }
 }

script/README.md

@@ -7,7 +7,7 @@ Solidity scripts for deploying and configuring the migration contracts using Fou
 | Script | Chain | Purpose | Executed By |
 |--------|-------|---------|-------------|
 | `DeployZkEVMContracts.s.sol` | zkEVM | Deploy VaultEscapeProofVerifier, VaultRootReceiverAdapter, VaultWithdrawalProcessor | Deployer |
-| `DeployEthContracts.s.sol` | Ethereum | Deploy VaultRootSenderAdapter, StarkExchangeMigration, and StarkExchangeMigrationV2 implementations | Deployer |
+| `DeployEthContracts.s.sol` | Ethereum | Deploy VaultRootSenderAdapter and StarkExchangeMigration implementation (version selected via config) | Deployer |
 | `GenerateAssetMappings.s.sol` | - | Generate asset mappings by querying bridge contract | Utility (offline) |
 | `VerifyAssetMappings.s.sol` | - | Verify asset mappings against bridge contract | Utility (read-only) |
 | `RegisterTokenMappings.s.sol` | zkEVM | Register token mappings on the processor | TOKEN_MAPPING_MANAGER |
@@ -88,9 +88,8 @@ forge script script/DeployEthContracts.s.sol \
 **Output:** Creates `config/eth_deployed.json` with deployed addresses populated in:
 - `vault_root_sender_adapter.address`
 - `stark_exchange_migration.implementation_address`
-- `stark_exchange_migration.v2_implementation_address`
 
-**Note:** Both `StarkExchangeMigration` and `StarkExchangeMigrationV2` are deployed as implementations only (no proxy). The existing StarkEx bridge proxy on-chain must be upgraded to point to an implementation separately (e.g., via governance), at which point `initialize` is called with the appropriate parameters.
+**Note:** The `StarkExchangeMigration` is deployed as an implementation only (no proxy). The existing StarkEx bridge proxy on-chain must be upgraded to point to this implementation separately (e.g., via governance), at which point `initialize` is called with the appropriate parameters. Set `stark_exchange_migration.version` to `1` for `StarkExchangeMigration` or `2` for `StarkExchangeMigrationV2`.
 
 ### Step 4: Generate Asset Mappings (zkEVM)
 
@@ -192,8 +191,8 @@ Deploys the Ethereum contracts for the migration system.
     "axelar_gateway": "0x..."
   },
   "stark_exchange_migration": {
-    "implementation_address": "0x0000...",
-    "v2_implementation_address": "0x0000..."
+    "version": 2,
+    "implementation_address": "0x0000..."
   }
 }
 ```