refactor: remove v prefix from SemVerMixin (#1385)

0xClandestine · web-flow · commit a518026de4bc · 2025-05-22T09:34:48.000-04:00
**Motivation:**

Slashing was deployed with using a `v` prefix, thus we're simply going
to drop the prefix moving forward.

**Modifications:**

Prefix removed, and tests updated.

**Result:**

SemVerMixin no longer requires a `v` prefix.
diff --git a/src/contracts/interfaces/IRewardsCoordinator.sol b/src/contracts/interfaces/IRewardsCoordinator.sol
@@ -258,7 +258,7 @@ interface IRewardsCoordinatorTypes {
      * @param MAX_RETROACTIVE_LENGTH The maximum retroactive length of a rewards submission
      * @param MAX_FUTURE_LENGTH The maximum future length of a rewards submission
      * @param GENESIS_REWARDS_TIMESTAMP The timestamp at which rewards are first calculated
-     * @param version The semantic version of the contract (e.g. "v1.2.3")
+     * @param version The semantic version of the contract (e.g. "1.2.3")
      * @dev Needed to avoid stack-too-deep errors
      */
     struct RewardsCoordinatorConstructorParams {
diff --git a/src/contracts/interfaces/ISemVerMixin.sol b/src/contracts/interfaces/ISemVerMixin.sol
@@ -6,6 +6,6 @@ pragma solidity ^0.8.0;
 /// @dev Follows SemVer 2.0.0 specification (https://semver.org/)
 interface ISemVerMixin {
     /// @notice Returns the semantic version string of the contract.
-    /// @return The version string in SemVer format (e.g., "v1.1.1")
+    /// @return The version string in SemVer format (e.g., "1.1.1")
     function version() external view returns (string memory);
 }
diff --git a/src/contracts/mixins/SemVerMixin.sol b/src/contracts/mixins/SemVerMixin.sol
@@ -11,12 +11,12 @@ abstract contract SemVerMixin is ISemVerMixin {
     using ShortStringsUpgradeable for *;
 
     /// @notice The semantic version string for this contract, stored as a ShortString for gas efficiency.
-    /// @dev Follows SemVer 2.0.0 specification (https://semver.org/). Prefixed with 'v' (e.g., "v1.2.3").
+    /// @dev Follows SemVer 2.0.0 specification (https://semver.org/).
     ShortString internal immutable _VERSION;
 
     /// @notice Initializes the contract with a semantic version string.
-    /// @param _version The SemVer-formatted version string (e.g., "v1.2.3")
-    /// @dev Version should follow SemVer 2.0.0 format with 'v' prefix: vMAJOR.MINOR.PATCH
+    /// @param _version The SemVer-formatted version string (e.g., "1.2.3")
+    /// @dev Version should follow SemVer 2.0.0 format: MAJOR.MINOR.PATCH
     constructor(
         string memory _version
     ) {
@@ -29,10 +29,10 @@ abstract contract SemVerMixin is ISemVerMixin {
     }
 
     /// @notice Returns the major version of the contract.
-    /// @dev Supports single digit major versions (e.g., "v1" for version "v1.2.3")
-    /// @return The major version string (e.g., "v1" for version "v1.2.3")
+    /// @dev Supports single digit major versions (e.g., "1" for version "1.2.3")
+    /// @return The major version string (e.g., "1" for version "1.2.3")
     function _majorVersion() internal view returns (string memory) {
         bytes memory v = bytes(_VERSION.toString());
-        return string(bytes.concat(v[0], v[1]));
+        return string(abi.encodePacked(v[0]));
     }
 }
diff --git a/src/contracts/mixins/SignatureUtilsMixin.sol b/src/contracts/mixins/SignatureUtilsMixin.sol
@@ -21,7 +21,7 @@ abstract contract SignatureUtilsMixin is ISignatureUtilsMixin, SemVerMixin {
     using SignatureCheckerUpgradeable for address;
 
     /// @notice Initializes the contract with a semantic version string.
-    /// @param _version The SemVer-formatted version string (e.g., "v1.1.1") to use for this contract's domain separator.
+    /// @param _version The SemVer-formatted version string (e.g., "1.1.1") to use for this contract's domain separator.
     /// @dev Version should follow SemVer 2.0.0 format with 'v' prefix: vMAJOR.MINOR.PATCH.
     ///      Only the major version component is used in the domain separator to maintain signature compatibility
     ///      across minor and patch version updates.
diff --git a/src/test/harnesses/AllocationManagerHarness.sol b/src/test/harnesses/AllocationManagerHarness.sol
@@ -13,7 +13,7 @@ contract AllocationManagerHarness is AllocationManager {
         uint32 _DEALLOCATION_DELAY,
         uint32 _ALLOCATION_CONFIGURATION_DELAY
     )
-        AllocationManager(_delegation, _pauserRegistry, _permissionController, _DEALLOCATION_DELAY, _ALLOCATION_CONFIGURATION_DELAY, "v9.9.9")
+        AllocationManager(_delegation, _pauserRegistry, _permissionController, _DEALLOCATION_DELAY, _ALLOCATION_CONFIGURATION_DELAY, "9.9.9")
     {}
 
     function deallocationQueueAtIndex(address operator, IStrategy strategy, uint index) external view returns (bytes32) {
diff --git a/src/test/harnesses/DelegationManagerHarness.sol b/src/test/harnesses/DelegationManagerHarness.sol
@@ -20,7 +20,7 @@ contract DelegationManagerHarness is DelegationManager {
             _pauserRegistry,
             _permissionController,
             _MIN_WITHDRAWAL_DELAY,
-            "v9.9.9"
+            "9.9.9"
         )
     {}
 
diff --git a/src/test/integration/IntegrationDeployer.t.sol b/src/test/integration/IntegrationDeployer.t.sol
@@ -41,7 +41,7 @@ abstract contract IntegrationDeployer is ExistingDeploymentParser {
     bool isUpgraded;
     uint mainnetForkBlock = 21_616_692; // Post Protocol Council upgrade
 
-    string version = "v9.9.9";
+    string version = "9.9.9";
 
     // Beacon chain genesis time when running locally
     // Multiple of 12 for sanity's sake
@@ -352,14 +352,14 @@ abstract contract IntegrationDeployer is ExistingDeploymentParser {
         );
         avsDirectoryImplementation = new AVSDirectory(delegationManager, eigenLayerPauserReg, version);
         eigenPodManagerImplementation =
-            new EigenPodManager(DEPOSIT_CONTRACT, eigenPodBeacon, delegationManager, eigenLayerPauserReg, "v9.9.9");
-        strategyFactoryImplementation = new StrategyFactory(strategyManager, eigenLayerPauserReg, "v9.9.9");
+            new EigenPodManager(DEPOSIT_CONTRACT, eigenPodBeacon, delegationManager, eigenLayerPauserReg, "9.9.9");
+        strategyFactoryImplementation = new StrategyFactory(strategyManager, eigenLayerPauserReg, "9.9.9");
         slashingWithdrawalRouterImplementation =
-            new SlashingWithdrawalRouter(allocationManager, strategyManager, eigenLayerPauserReg, "v9.9.9");
+            new SlashingWithdrawalRouter(allocationManager, strategyManager, eigenLayerPauserReg, "9.9.9");
 
         // Beacon implementations
-        eigenPodImplementation = new EigenPod(DEPOSIT_CONTRACT, eigenPodManager, BEACON_GENESIS_TIME, "v9.9.9");
-        baseStrategyImplementation = new StrategyBase(strategyManager, eigenLayerPauserReg, "v9.9.9");
+        eigenPodImplementation = new EigenPod(DEPOSIT_CONTRACT, eigenPodManager, BEACON_GENESIS_TIME, "9.9.9");
+        baseStrategyImplementation = new StrategyBase(strategyManager, eigenLayerPauserReg, "9.9.9");
 
         // Pre-longtail StrategyBaseTVLLimits implementation
         // TODO - need to update ExistingDeploymentParser
diff --git a/src/test/integration/tests/eigenpod/SlashBC_OneBCSF.t.sol b/src/test/integration/tests/eigenpod/SlashBC_OneBCSF.t.sol
@@ -34,7 +34,7 @@ contract Integration_SlashBC_OneBCSF is IntegrationCheckUtils {
     function _init() internal override {
         // 1. etch a new implementation to set staker's beaconChainSlashingFactor to 1
         EigenPodManagerWrapper eigenPodManagerWrapper =
-            new EigenPodManagerWrapper(DEPOSIT_CONTRACT, eigenPodBeacon, delegationManager, eigenLayerPauserReg, "v9.9.9");
+            new EigenPodManagerWrapper(DEPOSIT_CONTRACT, eigenPodBeacon, delegationManager, eigenLayerPauserReg, "9.9.9");
         address targetAddr = address(eigenPodManagerImplementation);
         cheats.etch(targetAddr, address(eigenPodManagerWrapper).code);
 
diff --git a/src/test/mocks/EigenPodMock.sol b/src/test/mocks/EigenPodMock.sol
@@ -6,7 +6,7 @@ import "../../contracts/interfaces/IEigenPod.sol";
 import "../../contracts/mixins/SemVerMixin.sol";
 
 contract EigenPodMock is IEigenPod, SemVerMixin, Test {
-    constructor() SemVerMixin("v9.9.9") {}
+    constructor() SemVerMixin("9.9.9") {}
 
     function nonBeaconChainETHBalanceWei() external view returns (uint) {}
 
diff --git a/src/test/unit/AVSDirectoryUnit.t.sol b/src/test/unit/AVSDirectoryUnit.t.sol
@@ -36,7 +36,7 @@ contract AVSDirectoryUnitTests is EigenLayerUnitTestSetup, IAVSDirectoryEvents,
         avsd = AVSDirectory(
             address(
                 new TransparentUpgradeableProxy(
-                    address(new AVSDirectory(IDelegationManager(delegationManager), pauserRegistry, "v9.9.9")),
+                    address(new AVSDirectory(IDelegationManager(delegationManager), pauserRegistry, "9.9.9")),
                     address(eigenLayerProxyAdmin),
                     abi.encodeWithSelector(
                         AVSDirectory.initialize.selector,
@@ -51,7 +51,7 @@ contract AVSDirectoryUnitTests is EigenLayerUnitTestSetup, IAVSDirectoryEvents,
         bytes memory v = bytes(avsd.version());
         bytes32 expectedDomainSeparator = keccak256(
             abi.encode(
-                EIP712_DOMAIN_TYPEHASH, keccak256(bytes("EigenLayer")), keccak256(bytes.concat(v[0], v[1])), block.chainid, address(avsd)
+                EIP712_DOMAIN_TYPEHASH, keccak256(bytes("EigenLayer")), keccak256(abi.encodePacked(v[0])), block.chainid, address(avsd)
             )
         );
 
diff --git a/src/test/unit/AllocationManagerUnit.t.sol b/src/test/unit/AllocationManagerUnit.t.sol
@@ -64,7 +64,7 @@ contract AllocationManagerUnitTests is EigenLayerUnitTestSetup, IAllocationManag
         strategyMock = StrategyBase(
             address(
                 new TransparentUpgradeableProxy(
-                    address(new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "v9.9.9")),
+                    address(new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "9.9.9")),
                     address(eigenLayerProxyAdmin),
                     abi.encodeWithSelector(StrategyBase.initialize.selector, tokenMock)
                 )
diff --git a/src/test/unit/DelegationUnit.t.sol b/src/test/unit/DelegationUnit.t.sol
@@ -113,7 +113,7 @@ contract DelegationManagerUnitTests is EigenLayerUnitTestSetup, IDelegationManag
 
         // Deploy mock token and strategy
         tokenMock = new ERC20PresetFixedSupply("Mock Token", "MOCK", tokenMockInitialSupply, address(this));
-        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "v9.9.9");
+        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "9.9.9");
         strategyMock = StrategyBase(
             address(
                 new TransparentUpgradeableProxy(
@@ -1160,7 +1160,7 @@ contract DelegationManagerUnitTests_Initialization_Setters is DelegationManagerU
             abi.encode(
                 EIP712_DOMAIN_TYPEHASH,
                 keccak256(bytes("EigenLayer")),
-                keccak256(bytes.concat(v[0], v[1])),
+                keccak256(abi.encodePacked(v[0])),
                 block.chainid,
                 address(delegationManager)
             )
diff --git a/src/test/unit/EigenPodManagerUnit.t.sol b/src/test/unit/EigenPodManagerUnit.t.sol
@@ -43,7 +43,7 @@ contract EigenPodManagerUnitTests is EigenLayerUnitTestSetup, IEigenPodManagerEv
 
         // Deploy EPM Implementation & Proxy
         eigenPodManagerImplementation =
-            new EigenPodManager(ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "v9.9.9");
+            new EigenPodManager(ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "9.9.9");
         eigenPodManager = EigenPodManager(
             address(
                 new TransparentUpgradeableProxy(
@@ -234,7 +234,7 @@ contract EigenPodManagerUnitTests_ShareUpdateTests is EigenPodManagerUnitTests {
 
         // Upgrade eigenPodManager to wrapper
         eigenPodManagerWrapper = new EigenPodManagerWrapper(
-            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "v9.9.9"
+            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "9.9.9"
         );
         eigenLayerProxyAdmin.upgrade(ITransparentUpgradeableProxy(payable(address(eigenPodManager))), address(eigenPodManagerWrapper));
     }
@@ -425,7 +425,7 @@ contract EigenPodManagerUnitTests_WithdrawSharesAsTokensTests is EigenPodManager
 
         // Upgrade eigenPodManager to wrapper
         eigenPodManagerWrapper = new EigenPodManagerWrapper(
-            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "v9.9.9"
+            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "9.9.9"
         );
         eigenLayerProxyAdmin.upgrade(ITransparentUpgradeableProxy(payable(address(eigenPodManager))), address(eigenPodManagerWrapper));
     }
@@ -546,7 +546,7 @@ contract EigenPodManagerUnitTests_BeaconChainETHBalanceUpdateTests is EigenPodMa
 
         // Upgrade eigenPodManager to wrapper
         eigenPodManagerWrapper = new EigenPodManagerWrapper(
-            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "v9.9.9"
+            ethPOSMock, eigenPodBeacon, IDelegationManager(address(delegationManagerMock)), pauserRegistry, "9.9.9"
         );
         eigenLayerProxyAdmin.upgrade(ITransparentUpgradeableProxy(payable(address(eigenPodManager))), address(eigenPodManagerWrapper));
     }
diff --git a/src/test/unit/EigenPodUnit.t.sol b/src/test/unit/EigenPodUnit.t.sol
@@ -58,7 +58,7 @@ contract EigenPodUnitTests is EigenLayerUnitTestSetup, EigenPodPausingConstants,
         beaconChain = new BeaconChainMock(EigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL);
 
         // Deploy EigenPod
-        podImplementation = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "v9.9.9");
+        podImplementation = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "9.9.9");
 
         // Deploy Beacon
         eigenPodBeacon = new UpgradeableBeacon(address(podImplementation));
@@ -302,7 +302,7 @@ contract EigenPodUnitTests is EigenLayerUnitTestSetup, EigenPodPausingConstants,
 
 contract EigenPodUnitTests_Initialization is EigenPodUnitTests {
     function test_constructor() public {
-        EigenPod pod = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "v9.9.9");
+        EigenPod pod = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "9.9.9");
 
         assertTrue(pod.ethPOS() == ethPOSDepositMock, "should have set ethPOS correctly");
         assertTrue(address(pod.eigenPodManager()) == address(eigenPodManagerMock), "should have set eigenpodmanager correctly");
@@ -330,7 +330,7 @@ contract EigenPodUnitTests_Initialization is EigenPodUnitTests {
     }
 
     function test_initialize_revert_emptyPodOwner() public {
-        EigenPod pod = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "v9.9.9");
+        EigenPod pod = new EigenPod(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "9.9.9");
         // un-initialize pod
         cheats.store(address(pod), 0, 0);
 
@@ -1711,7 +1711,7 @@ contract EigenPodHarnessSetup is EigenPodUnitTests {
 
         // Deploy EP Harness
         eigenPodHarnessImplementation =
-            new EigenPodHarness(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "v9.9.9");
+            new EigenPodHarness(ethPOSDepositMock, IEigenPodManager(address(eigenPodManagerMock)), GENESIS_TIME_LOCAL, "9.9.9");
 
         // Upgrade eigenPod to harness
         UpgradeableBeacon(address(eigenPodBeacon)).upgradeTo(address(eigenPodHarnessImplementation));
diff --git a/src/test/unit/RewardsCoordinatorUnit.t.sol b/src/test/unit/RewardsCoordinatorUnit.t.sol
@@ -119,7 +119,7 @@ contract RewardsCoordinatorUnitTests is EigenLayerUnitTestSetup, IRewardsCoordin
                 MAX_RETROACTIVE_LENGTH: MAX_RETROACTIVE_LENGTH,
                 MAX_FUTURE_LENGTH: MAX_FUTURE_LENGTH,
                 GENESIS_REWARDS_TIMESTAMP: GENESIS_REWARDS_TIMESTAMP,
-                version: "v9.9.9"
+                version: "9.9.9"
             })
         );
 
@@ -145,7 +145,7 @@ contract RewardsCoordinatorUnitTests is EigenLayerUnitTestSetup, IRewardsCoordin
         token2 = new ERC20PresetFixedSupply("jeo boden", "MOCK2", mockTokenInitialSupply, address(this));
         token3 = new ERC20PresetFixedSupply("pepe wif avs", "MOCK3", mockTokenInitialSupply, address(this));
 
-        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "v9.9.9");
+        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "9.9.9");
         strategyMock1 = StrategyBase(
             address(
                 new TransparentUpgradeableProxy(
diff --git a/src/test/unit/SlashingWithdrawalRouterUnit.t.sol b/src/test/unit/SlashingWithdrawalRouterUnit.t.sol
diff --git a/src/test/unit/StrategyBaseTVLLimitsUnit.sol b/src/test/unit/StrategyBaseTVLLimitsUnit.sol
@@ -24,7 +24,7 @@ contract StrategyBaseTVLLimitsUnitTests is StrategyBaseUnitTests {
         StrategyBaseUnitTests.setUp();
 
         // depoloy the TVL-limited strategy
-        strategyBaseTVLLimitsImplementation = new StrategyBaseTVLLimits(strategyManager, pauserRegistry, "v9.9.9");
+        strategyBaseTVLLimitsImplementation = new StrategyBaseTVLLimits(strategyManager, pauserRegistry, "9.9.9");
         strategyWithTVLLimits = StrategyBaseTVLLimits(
             address(
                 new TransparentUpgradeableProxy(
diff --git a/src/test/unit/StrategyBaseUnit.t.sol b/src/test/unit/StrategyBaseUnit.t.sol
@@ -55,7 +55,7 @@ contract StrategyBaseUnitTests is Test {
 
         underlyingToken = new ERC20PresetFixedSupply("Test Token", "TEST", initialSupply, initialOwner);
 
-        strategyImplementation = new StrategyBase(strategyManager, pauserRegistry, "v9.9.9");
+        strategyImplementation = new StrategyBase(strategyManager, pauserRegistry, "9.9.9");
 
         strategy = StrategyBase(
             address(
@@ -160,7 +160,7 @@ contract StrategyBaseUnitTests is Test {
         // Deploy token with 1e39 total supply
         underlyingToken = new ERC20PresetFixedSupply("Test Token", "TEST", 1e39, initialOwner);
 
-        strategyImplementation = new StrategyBase(strategyManager, pauserRegistry, "v9.9.9");
+        strategyImplementation = new StrategyBase(strategyManager, pauserRegistry, "9.9.9");
 
         strategy = StrategyBase(
             address(
diff --git a/src/test/unit/StrategyFactoryUnit.t.sol b/src/test/unit/StrategyFactoryUnit.t.sol
@@ -46,12 +46,12 @@ contract StrategyFactoryUnitTests is EigenLayerUnitTestSetup {
 
         underlyingToken = new ERC20PresetFixedSupply("Test Token", "TEST", initialSupply, initialOwner);
 
-        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "v9.9.9");
+        strategyImplementation = new StrategyBase(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "9.9.9");
 
         strategyBeacon = new UpgradeableBeacon(address(strategyImplementation));
         strategyBeacon.transferOwnership(beaconProxyOwner);
 
-        strategyFactoryImplementation = new StrategyFactory(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "v9.9.9");
+        strategyFactoryImplementation = new StrategyFactory(IStrategyManager(address(strategyManagerMock)), pauserRegistry, "9.9.9");
 
         strategyFactory = StrategyFactory(
             address(
diff --git a/src/test/unit/StrategyManagerUnit.t.sol b/src/test/unit/StrategyManagerUnit.t.sol
@@ -46,7 +46,7 @@ contract StrategyManagerUnitTests is EigenLayerUnitTestSetup, IStrategyManagerEv
             IDelegationManager(address(delegationManagerMock)),
             ISlashingWithdrawalRouter(address(slashingWithdrawalRouterMock)),
             pauserRegistry,
-            "v9.9.9"
+            "9.9.9"
         );
         strategyManager = StrategyManager(
             address(
@@ -84,7 +84,7 @@ contract StrategyManagerUnitTests is EigenLayerUnitTestSetup, IStrategyManagerEv
         public
         returns (StrategyBase)
     {
-        StrategyBase newStrategyImplementation = new StrategyBase(_strategyManager, _pauserRegistry, "v9.9.9");
+        StrategyBase newStrategyImplementation = new StrategyBase(_strategyManager, _pauserRegistry, "9.9.9");
         StrategyBase newStrategy =
             StrategyBase(address(new TransparentUpgradeableProxy(address(newStrategyImplementation), address(admin), "")));
         newStrategy.initialize(_token);
@@ -243,7 +243,7 @@ contract StrategyManagerUnitTests_initialize is StrategyManagerUnitTests {
             abi.encode(
                 EIP712_DOMAIN_TYPEHASH,
                 keccak256(bytes("EigenLayer")),
-                keccak256(bytes(bytes.concat(v[0], v[1]))),
+                keccak256(abi.encodePacked(v[0])),
                 block.chainid,
                 address(strategyManager)
             )
diff --git a/src/test/unit/mixins/SemVerMixin.t.sol b/src/test/unit/mixins/SemVerMixin.t.sol
@@ -18,12 +18,12 @@ contract SemVerMixinTest is Test {
     SemVerMixinMock public semVer;
 
     function test_version_returnsCorrectVersion() public {
-        semVer = new SemVerMixinMock("v1.2.3");
-        assertEq(semVer.version(), "v1.2.3");
+        semVer = new SemVerMixinMock("1.2.3");
+        assertEq(semVer.version(), "1.2.3");
     }
 
     function test_majorVersion_returnsCorrectMajorVersion() public {
-        semVer = new SemVerMixinMock("v1.2.3");
-        assertEq(semVer.majorVersion(), "v1");
+        semVer = new SemVerMixinMock("1.2.3");
+        assertEq(semVer.majorVersion(), "1");
     }
 }
diff --git a/src/test/unit/mixins/SignatureUtilsUnit.t.sol b/src/test/unit/mixins/SignatureUtilsUnit.t.sol
@@ -16,7 +16,7 @@ contract MockSigner {
     }
 }
 
-contract SignatureUtilsMixinUnit is Test, SignatureUtilsMixin("v0.0.0") {
+contract SignatureUtilsMixinUnit is Test, SignatureUtilsMixin("9.9.9") {
     uint signerPk;
     address signer;
     bytes32 hash;
diff --git a/src/test/utils/EigenLayerUnitTestSetup.sol b/src/test/utils/EigenLayerUnitTestSetup.sol

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,6 @@ pragma solidity ^0.8.0;`
`6`	`6`	`/// @dev Follows SemVer 2.0.0 specification (https://semver.org/)`
`7`	`7`	`interface ISemVerMixin {`
`8`	`8`	`/// @notice Returns the semantic version string of the contract.`
`9`		`- /// @return The version string in SemVer format (e.g., "v1.1.1")`
	`9`	`+ /// @return The version string in SemVer format (e.g., "1.1.1")`
`10`	`10`	`function version() external view returns (string memory);`
`11`	`11`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ contract AllocationManagerHarness is AllocationManager {`
`13`	`13`	`uint32 _DEALLOCATION_DELAY,`
`14`	`14`	`uint32 _ALLOCATION_CONFIGURATION_DELAY`
`15`	`15`	`)`
`16`		`- AllocationManager(_delegation, _pauserRegistry, _permissionController, _DEALLOCATION_DELAY, _ALLOCATION_CONFIGURATION_DELAY, "v9.9.9")`
	`16`	`+ AllocationManager(_delegation, _pauserRegistry, _permissionController, _DEALLOCATION_DELAY, _ALLOCATION_CONFIGURATION_DELAY, "9.9.9")`
`17`	`17`	`{}`
`18`	`18`
`19`	`19`	`function deallocationQueueAtIndex(address operator, IStrategy strategy, uint index) external view returns (bytes32) {`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ contract DelegationManagerHarness is DelegationManager {`
`20`	`20`	`_pauserRegistry,`
`21`	`21`	`_permissionController,`
`22`	`22`	`_MIN_WITHDRAWAL_DELAY,`
`23`		`- "v9.9.9"`
	`23`	`+ "9.9.9"`
`24`	`24`	`)`
`25`	`25`	`{}`
`26`	`26`