merge main

.forge-snapshots/add liquidity to already existing position with salt.snap

@@ -1 +1 @@
-144615
+144639

.forge-snapshots/addLiquidity CA fee.snap

@@ -1 +1 @@
-170904
+170928

.forge-snapshots/addLiquidity with empty hook.snap

@@ -1 +1 @@
-274182
+274240

.forge-snapshots/addLiquidity with native token.snap

@@ -1 +1 @@
-135108
+135120

.forge-snapshots/create new liquidity to a position with salt.snap

@@ -1 +1 @@
-292807
+292831

.forge-snapshots/donate gas with 1 token.snap

@@ -1 +1 @@
-106309
+106333

.forge-snapshots/donate gas with 2 tokens.snap

@@ -1 +1 @@
-145724
+145748

.forge-snapshots/erc20 collect protocol fees.snap

@@ -1 +1 @@
-57439
+57451

.forge-snapshots/initialize.snap

@@ -1 +1 @@
-60009
+51533

.forge-snapshots/poolManager bytecode size.snap

@@ -1 +1 @@
-24140
+23688

.forge-snapshots/removeLiquidity CA fee.snap

@@ -1 +1 @@
-141192
+141194

.forge-snapshots/removeLiquidity with empty hook.snap

@@ -1 +1 @@
-130567
+130603

.forge-snapshots/removeLiquidity with native token.snap

@@ -1 +1 @@
-112513
+112523

.forge-snapshots/simple addLiquidity second addition same range.snap

@@ -1 +1 @@
-98826
+98850

.forge-snapshots/simple addLiquidity.snap

@@ -1 +1 @@
-161371
+161395

.forge-snapshots/simple removeLiquidity some liquidity remains.snap

@@ -1 +1 @@
-92962
+92986

.forge-snapshots/simple removeLiquidity.snap

@@ -1 +1 @@
-85075
+85099

.forge-snapshots/simple swap with native.snap

@@ -1 +1 @@
-108422
+108434

.forge-snapshots/simple swap.snap

@@ -1 +1 @@
-123239
+123263

.forge-snapshots/swap CA custom curve + swap noop.snap

@@ -1 +1 @@
-124633
+124635

.forge-snapshots/swap CA fee on unspecified.snap

@@ -1 +1 @@
-154684
+154686

.forge-snapshots/swap against liquidity with native token.snap

@@ -1 +1 @@
-105557
+105569

.forge-snapshots/swap against liquidity.snap

@@ -1 +1 @@
-116622
+116646

.forge-snapshots/swap burn 6909 for input.snap

@@ -1 +1 @@
-129261
+129285

.forge-snapshots/swap burn native 6909 for input.snap

@@ -1 +1 @@
-118660
+118672

.forge-snapshots/swap mint native output as 6909.snap

@@ -1 +1 @@
-139727
+139739

.forge-snapshots/swap mint output as 6909.snap

@@ -1 +1 @@
-155080
+155104

.forge-snapshots/swap skips hook call if hook is caller.snap

@@ -1 +1 @@
-206285
+206251

.forge-snapshots/swap with hooks.snap

@@ -1 +1 @@
-132260
+132274

.forge-snapshots/swap with return dynamic fee.snap

@@ -1 +1 @@
-145611
+145577

.forge-snapshots/update dynamic fee in before swap.snap

@@ -1 +1 @@
-147891
+147869

.github/CODEOWNERS

@@ -0,0 +1 @@
+* @uniswap/protocols

README.md

@@ -13,7 +13,7 @@ If you’re interested in contributing please see our [contribution guidelines](
 
 ## Whitepaper
 
-A more detailed description of Uniswap v4 Core can be found in the draft of the [Uniswap v4 Core Whitepaper](./docs/whitepaper-v4.pdf).
+A more detailed description of Uniswap v4 Core can be found in the draft of the [Uniswap v4 Core Whitepaper](./docs/whitepaper/whitepaper-v4.pdf).
 
 ## Architecture
 

src/PoolManager.sol

@@ -112,11 +112,7 @@ contract PoolManager is IPoolManager, ProtocolFees, NoDelegateCall, ERC6909Claim
     }
 
     /// @inheritdoc IPoolManager
-    function initialize(PoolKey memory key, uint160 sqrtPriceX96, bytes calldata hookData)
-        external
-        noDelegateCall
-        returns (int24 tick)
-    {
+    function initialize(PoolKey memory key, uint160 sqrtPriceX96) external noDelegateCall returns (int24 tick) {
         // see TickBitmap.sol for overflow conditions that can arise from tick spacing being too large
         if (key.tickSpacing > MAX_TICK_SPACING) TickSpacingTooLarge.selector.revertWith(key.tickSpacing);
         if (key.tickSpacing < MIN_TICK_SPACING) TickSpacingTooSmall.selector.revertWith(key.tickSpacing);
@@ -129,14 +125,13 @@ contract PoolManager is IPoolManager, ProtocolFees, NoDelegateCall, ERC6909Claim
 
         uint24 lpFee = key.fee.getInitialLPFee();
 
-        key.hooks.beforeInitialize(key, sqrtPriceX96, hookData);
+        key.hooks.beforeInitialize(key, sqrtPriceX96);
 
         PoolId id = key.toId();
-        uint24 protocolFee = _fetchProtocolFee(key);
 
-        tick = _pools[id].initialize(sqrtPriceX96, protocolFee, lpFee);
+        tick = _pools[id].initialize(sqrtPriceX96, lpFee);
 
-        key.hooks.afterInitialize(key, sqrtPriceX96, tick, hookData);
+        key.hooks.afterInitialize(key, sqrtPriceX96, tick);
 
         // emit all details of a pool key. poolkeys are not saved in storage and must always be provided by the caller
         // the key's fee may be a static fee or a sentinel to denote a dynamic fee.
@@ -178,7 +173,7 @@ contract PoolManager is IPoolManager, ProtocolFees, NoDelegateCall, ERC6909Claim
         BalanceDelta hookDelta;
         (callerDelta, hookDelta) = key.hooks.afterModifyLiquidity(key, params, callerDelta, feesAccrued, hookData);
 
-        // if the hook doesnt have the flag to be able to return deltas, hookDelta will always be 0
+        // if the hook doesn't have the flag to be able to return deltas, hookDelta will always be 0
         if (hookDelta != BalanceDeltaLibrary.ZERO_DELTA) _accountPoolBalanceDelta(key, hookDelta, address(key.hooks));
 
         _accountPoolBalanceDelta(key, callerDelta, msg.sender);
@@ -221,7 +216,7 @@ contract PoolManager is IPoolManager, ProtocolFees, NoDelegateCall, ERC6909Claim
         BalanceDelta hookDelta;
         (swapDelta, hookDelta) = key.hooks.afterSwap(key, params, swapDelta, hookData, beforeSwapDelta);
 
-        // if the hook doesnt have the flag to be able to return deltas, hookDelta will always be 0
+        // if the hook doesn't have the flag to be able to return deltas, hookDelta will always be 0
         if (hookDelta != BalanceDeltaLibrary.ZERO_DELTA) _accountPoolBalanceDelta(key, hookDelta, address(key.hooks));
 
         _accountPoolBalanceDelta(key, swapDelta, msg.sender);

src/ProtocolFees.sol

@@ -6,7 +6,6 @@ import {IProtocolFeeController} from "./interfaces/IProtocolFeeController.sol";
 import {IProtocolFees} from "./interfaces/IProtocolFees.sol";
 import {PoolKey} from "./types/PoolKey.sol";
 import {ProtocolFeeLibrary} from "./libraries/ProtocolFeeLibrary.sol";
-import {BipsLibrary} from "./libraries/BipsLibrary.sol";
 import {Owned} from "solmate/src/auth/Owned.sol";
 import {PoolId} from "./types/PoolId.sol";
 import {Pool} from "./libraries/Pool.sol";
@@ -17,18 +16,13 @@ abstract contract ProtocolFees is IProtocolFees, Owned {
     using ProtocolFeeLibrary for uint24;
     using Pool for Pool.State;
     using CustomRevert for bytes4;
-    using BipsLibrary for uint256;
 
     /// @inheritdoc IProtocolFees
     mapping(Currency currency => uint256 amount) public protocolFeesAccrued;
 
     /// @inheritdoc IProtocolFees
     IProtocolFeeController public protocolFeeController;
 
-    // a percentage of the block.gaslimit denoted in basis points, used as the gas limit for fee controller calls
-    // 100 bps is 1%, at 30M gas, the limit is 300K
-    uint256 private constant BLOCK_LIMIT_BPS = 100;
-
     constructor() Owned(msg.sender) {}
 
     /// @inheritdoc IProtocolFees
@@ -66,42 +60,6 @@ abstract contract ProtocolFees is IProtocolFees, Owned {
     /// @dev this is overridden in PoolManager.sol to give access to the _pools mapping
     function _getPool(PoolId id) internal virtual returns (Pool.State storage);
 
-    /// @notice Fetch the protocol fees for a given pool
-    /// @dev the success of this function is false if the call fails or the returned fees are invalid
-    /// @dev to prevent an invalid protocol fee controller from blocking pools from being initialized
-    /// the success of this function is NOT checked on initialize and if the call fails, the protocol fees are set to 0.
-    function _fetchProtocolFee(PoolKey memory key) internal returns (uint24 protocolFee) {
-        if (address(protocolFeeController) != address(0)) {
-            uint256 controllerGasLimit = block.gaslimit.calculatePortion(BLOCK_LIMIT_BPS);
-
-            // note that EIP-150 mandates that calls requesting more than 63/64ths of remaining gas
-            // will be allotted no more than this amount, so controllerGasLimit must be set with this
-            // in mind.
-            if (gasleft() < controllerGasLimit) ProtocolFeeCannotBeFetched.selector.revertWith();
-
-            address toAddress = address(protocolFeeController);
-
-            bytes memory data = abi.encodeCall(IProtocolFeeController.protocolFeeForPool, (key));
-
-            bool success;
-            uint256 returnData;
-            assembly ("memory-safe") {
-                // only load the first 32 bytes of the return data to prevent gas griefing
-                success := call(controllerGasLimit, toAddress, 0, add(data, 0x20), mload(data), 0, 32)
-                // if success is false this wont actually be returned, instead 0 will be returned
-                returnData := mload(0)
-
-                // success if return data size is 32 bytes
-                success := and(success, eq(returndatasize(), 32))
-            }
-
-            // Ensure return data does not overflow a uint24 and that the underlying fees are within bounds.
-            protocolFee = success && (returnData == uint24(returnData)) && uint24(returnData).isValidProtocolFee()
-                ? uint24(returnData)
-                : 0;
-        }
-    }
-
     function _updateProtocolFees(Currency currency, uint256 amount) internal {
         unchecked {
             protocolFeesAccrued[currency] += amount;