Update changeset to document V3 removal and interface changes + more register upgrade gate tests (#1338)

<!--- Provide a general summary of your changes in the Title above -->

## Description

<!--- Describe your changes in detail -->

## Motivation and Context

<!--- Why is this change required? What problem does it solve? -->
<!--- If it fixes an open issue, please link to the issue here. -->

## Does this change the ABI/API?

- [ ] This changes the ABI/API

<!-- If so, please describe how and what potential impact this may have -->

## What tests did you add/modify to account for these changes

<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, tests ran to see how -->
<!--- your change affects other areas of the code, etc. -->

## Types of changes

<!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: -->

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New module / feature (non-breaking change which adds functionality)
- [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

## Checklist:

<!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->

- [ ] My code follows the code style of this project.
- [ ] My change requires a change to the documentation.
- [ ] I have updated the documentation accordingly.
- [ ] I added a changeset to account for this change

## Reviewer Checklist:

- [ ] My review includes a symposis of the changes and potential issues
- [ ] The code style is enforced
- [ ] There are no risky / concerning changes / additions to the PR

Author: oveddan

.changeset/famous-pens-sneeze.md

@@ -5,10 +5,12 @@
 Remove Uniswap V3 support and refactor coin architecture
 
 **Removal of V3 Support:**
+- Removed support for creating coins based on Uniswap V3 - only V4 coins are supported
+- Default coin deployment now creates Uniswap V4 coins when no config is provided (previously created V3)
 - Removed V3-specific test files and utilities
 - Updated remaining tests to use V4 deployment methods  
 - Removed V3 configuration functions and encoders
-- Added revert logic for V3 deployment attempts
+- Added revert logic for V3 deployment attempts in factory deploy functions
 
 **Architecture Refactoring:**
 - Merged BaseCoinV4 functionality into BaseCoin.sol to consolidate Uniswap V4 integration

packages/coins/test/LiquidityMigration.t.sol

@@ -282,4 +282,110 @@ contract LiquidityMigrationTest is BaseTest {
         vm.expectRevert(abi.encodeWithSelector(Ownable.OwnableUnauthorizedAccount.selector, nonOwner));
         hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
     }
+
+    function test_hookUpgradeGate_registerMultipleUpgradePaths() public {
+        address baseImpl1 = makeAddr("baseImpl1");
+        address baseImpl2 = makeAddr("baseImpl2");
+        address baseImpl3 = makeAddr("baseImpl3");
+        address upgradeImpl = makeAddr("upgradeImpl");
+
+        address[] memory baseImpls = new address[](3);
+        baseImpls[0] = baseImpl1;
+        baseImpls[1] = baseImpl2;
+        baseImpls[2] = baseImpl3;
+
+        // Register multiple upgrade paths at once
+        vm.prank(hookUpgradeGate.owner());
+        vm.expectEmit(true, true, false, false);
+        emit IHooksUpgradeGate.UpgradeRegistered(baseImpl1, upgradeImpl);
+        vm.expectEmit(true, true, false, false);
+        emit IHooksUpgradeGate.UpgradeRegistered(baseImpl2, upgradeImpl);
+        vm.expectEmit(true, true, false, false);
+        emit IHooksUpgradeGate.UpgradeRegistered(baseImpl3, upgradeImpl);
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+
+        // Verify all paths are registered
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(baseImpl1, upgradeImpl));
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(baseImpl2, upgradeImpl));
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(baseImpl3, upgradeImpl));
+    }
+
+    function test_hookUpgradeGate_registerEmptyArray() public {
+        address upgradeImpl = makeAddr("upgradeImpl");
+        address[] memory baseImpls = new address[](0);
+
+        // Register with empty array - should succeed but do nothing
+        vm.prank(hookUpgradeGate.owner());
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+
+        // No events should be emitted, no state changes
+    }
+
+    function test_hookUpgradeGate_removeNonexistentUpgradePath() public {
+        address baseImpl = makeAddr("baseImpl");
+        address upgradeImpl = makeAddr("upgradeImpl");
+
+        // Try to remove a path that was never registered
+        vm.prank(hookUpgradeGate.owner());
+        vm.expectEmit(true, true, false, false);
+        emit IHooksUpgradeGate.UpgradeRemoved(baseImpl, upgradeImpl);
+        hookUpgradeGate.removeUpgradePath(baseImpl, upgradeImpl);
+
+        // Should still be false (was already false)
+        assertFalse(hookUpgradeGate.isRegisteredUpgradePath(baseImpl, upgradeImpl));
+    }
+
+    function test_hookUpgradeGate_registerSamePathTwice() public {
+        address baseImpl = makeAddr("baseImpl");
+        address upgradeImpl = makeAddr("upgradeImpl");
+
+        address[] memory baseImpls = new address[](1);
+        baseImpls[0] = baseImpl;
+
+        // Register once
+        vm.prank(hookUpgradeGate.owner());
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(baseImpl, upgradeImpl));
+
+        // Register again - should succeed and overwrite (true -> true)
+        vm.prank(hookUpgradeGate.owner());
+        vm.expectEmit(true, true, false, false);
+        emit IHooksUpgradeGate.UpgradeRegistered(baseImpl, upgradeImpl);
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(baseImpl, upgradeImpl));
+    }
+
+    function test_hookUpgradeGate_zeroAddressHandling() public {
+        address[] memory baseImpls = new address[](2);
+        baseImpls[0] = address(0);
+        baseImpls[1] = makeAddr("baseImpl");
+        address upgradeImpl = address(0);
+
+        // Should allow zero addresses (no validation in contract)
+        vm.prank(hookUpgradeGate.owner());
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(address(0), address(0)));
+        assertTrue(hookUpgradeGate.isRegisteredUpgradePath(makeAddr("baseImpl"), address(0)));
+    }
+
+    function test_hookUpgradeGate_isAllowedHookUpgradeMapping() public {
+        address baseImpl = makeAddr("baseImpl");
+        address upgradeImpl = makeAddr("upgradeImpl");
+
+        // Test direct mapping access
+        assertFalse(hookUpgradeGate.isAllowedHookUpgrade(baseImpl, upgradeImpl));
+
+        // Register upgrade path
+        address[] memory baseImpls = new address[](1);
+        baseImpls[0] = baseImpl;
+        vm.prank(hookUpgradeGate.owner());
+        hookUpgradeGate.registerUpgradePath(baseImpls, upgradeImpl);
+
+        // Test direct mapping access
+        assertTrue(hookUpgradeGate.isAllowedHookUpgrade(baseImpl, upgradeImpl));
+
+        // Should match isRegisteredUpgradePath
+        assertEq(hookUpgradeGate.isAllowedHookUpgrade(baseImpl, upgradeImpl), hookUpgradeGate.isRegisteredUpgradePath(baseImpl, upgradeImpl));
+    }
 }