@zoralabs/limit-orders 0.2.1 → 0.2.4

package/.abi-stability

@@ -0,0 +1,70 @@
+=======================
+👁👁 ABI STABILITY snapshot 👁👁
+=======================
+
+===============================
+DEPRECATED FUNCTIONS
+===============================
+
+(none)
+
+===============================
+➡ IZoraLimitOrderBook
+===============================
+
+--- EVENTS ---
+
+╭-------------------------------------------------------------------------------------------------+--------------------------------------------------------------------╮
+| Event                                                                                           | Topic                                                              |
++======================================================================================================================================================================+
+| LimitOrderCreated(address,address,bytes32,bool,int24,int24,uint128,bytes32)                     | 0x36c14a3cb6751895ef6ab64f123dcca47c492ec447d69f86bb560eb5ad0e47a3 |
+|-------------------------------------------------------------------------------------------------+--------------------------------------------------------------------|
+| LimitOrderFilled(address,address,address,uint128,uint128,address,uint128,bytes32,int24,bytes32) | 0x59d8d839dc2a16f78d0e321652546b26bd10065beade9541895b54e0ec058a37 |
+|-------------------------------------------------------------------------------------------------+--------------------------------------------------------------------|
+| LimitOrderUpdated(address,address,bytes32,bool,int24,uint128,bytes32,bool)                      | 0x73544fee6e45e543d5464c5d945a19046ad773dabfc95c7d1c119b135d947dde |
+|-------------------------------------------------------------------------------------------------+--------------------------------------------------------------------|
+| MakerBalanceUpdated(address,address,uint256)                                                    | 0xa942ad2d47c64ce0859147575881ee76ce335036216e9378b100614f683006cb |
+╰-------------------------------------------------------------------------------------------------+--------------------------------------------------------------------╯
+
+
+--- FUNCTIONS ---
+
+╭-------------------------------------------------------------------------------+------------╮
+| Method                                                                        | Identifier |
++============================================================================================+
+| balanceOf(address,address)                                                    | f7888aec   |
+|-------------------------------------------------------------------------------+------------|
+| create((address,address,uint24,int24,address),bool,uint256[],int24[],address) | eb07ed24   |
+|-------------------------------------------------------------------------------+------------|
+| fill(((address,address,uint24,int24,address),bool,bytes32[])[],address)       | 7a9cf273   |
+|-------------------------------------------------------------------------------+------------|
+| fill((address,address,uint24,int24,address),bool,int24,int24,uint256,address) | 5f8fbad6   |
+|-------------------------------------------------------------------------------+------------|
+| getMaxFillCount()                                                             | d8c31772   |
+|-------------------------------------------------------------------------------+------------|
+| setMaxFillCount(uint256)                                                      | c96d060b   |
+|-------------------------------------------------------------------------------+------------|
+| withdraw(bytes32[],address,uint256,address)                                   | c2ec68ea   |
+╰-------------------------------------------------------------------------------+------------╯
+
+
+===============================
+➡ ISetLimitOrderConfig
+===============================
+
+--- EVENTS ---
+
+╭-------+-------╮
+| Event | Topic |
++===============+
+╰-------+-------╯
+
+
+--- FUNCTIONS ---
+
+╭--------------------------------------------+------------╮
+| Method                                     | Identifier |
++=========================================================+
+| setLimitOrderConfig((uint256[],uint256[])) | 18ed2974   |
+╰--------------------------------------------+------------╯
+

package/.turbo/turbo-build$colon$js.log

@@ -1,17 +1,17 @@
 
-> @zoralabs/limit-orders@0.2.1 build:js /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
+> @zoralabs/limit-orders@0.2.4 build:js /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
 > pnpm run wagmi:generate && pnpm run copy-abis && pnpm run prettier:write && tsup
 
 
-> @zoralabs/limit-orders@0.2.1 wagmi:generate /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
+> @zoralabs/limit-orders@0.2.4 wagmi:generate /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
 > pnpm run build:contracts:minimal && wagmi generate && pnpm exec rename-generated-abi-casing ./package/wagmiGenerated.ts
 
 
-> @zoralabs/limit-orders@0.2.1 build:contracts:minimal /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
+> @zoralabs/limit-orders@0.2.4 build:contracts:minimal /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
 > forge build src/ --no-metadata
 
-Compiling 92 files with Solc 0.8.28
-Solc 0.8.28 finished in 8.26s
+Compiling 93 files with Solc 0.8.28
+Solc 0.8.28 finished in 7.24s
 Compiler run successful!
 - Validating plugins
 ✔ Validating plugins
@@ -26,51 +26,50 @@ Compiler run successful!
 ✅ Updated ./package/wagmiGenerated.ts (3 replacements)
 ✨ All files processed successfully!
 
-> @zoralabs/limit-orders@0.2.1 copy-abis /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
+> @zoralabs/limit-orders@0.2.4 copy-abis /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
 > pnpm exec bundle-abis
 
 
-> @zoralabs/limit-orders@0.2.1 prettier:write /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
+> @zoralabs/limit-orders@0.2.4 prettier:write /home/runner/work/zora-protocol-private/zora-protocol-private/packages/limit-orders
 > prettier --write 'src/**/*.sol' 'test/**/*.sol'
 
-src/access/SimpleAccessManaged.sol 345ms (unchanged)
-src/access/SimpleAccessManager.sol 253ms (unchanged)
-src/IZoraLimitOrderBook.sol 62ms (unchanged)
-src/libs/LimitOrderBitmap.sol 66ms (unchanged)
-src/libs/LimitOrderCommon.sol 207ms (unchanged)
-src/libs/LimitOrderCreate.sol 482ms (unchanged)
-src/libs/LimitOrderFill.sol 243ms (unchanged)
-src/libs/LimitOrderLiquidity.sol 232ms (unchanged)
-src/libs/LimitOrderQueues.sol 37ms (unchanged)
-src/libs/LimitOrderStorage.sol 16ms (unchanged)
-src/libs/LimitOrderTypes.sol 2ms (unchanged)
-src/libs/LimitOrderViews.sol 132ms (unchanged)
-src/libs/LimitOrderWithdraw.sol 91ms (unchanged)
-src/libs/Permit2Payments.sol 32ms (unchanged)
-src/libs/SwapLimitOrders.sol 161ms (unchanged)
-src/router/SwapWithLimitOrders.sol 269ms (unchanged)
-src/ZoraLimitOrderBook.sol 136ms (unchanged)
-test/gas/LimitOrderFillGas.t.sol 851ms (unchanged)
-test/gas/LimitOrderSwapGas.t.sol 310ms (unchanged)
-test/LimitOrderAccessControl.t.sol 402ms (unchanged)
-test/LimitOrderBitmap.t.sol 217ms (unchanged)
-test/LimitOrderCreate.t.sol 392ms (unchanged)
-test/LimitOrderFill.t.sol 1071ms (unchanged)
-test/LimitOrderLibraries.t.sol 381ms (unchanged)
-test/LimitOrderLiquidityPayouts.t.sol 436ms (unchanged)
-test/LimitOrderV4Pools.t.sol 147ms (unchanged)
-test/LimitOrderWithdraw.t.sol 677ms (unchanged)
-test/SimpleAccessManager.t.sol 303ms (unchanged)
-test/SwapWithLimitOrders.t.sol 125ms (unchanged)
-test/SwapWithLimitOrdersRouter.t.sol 636ms (unchanged)
-test/unit/LimitOrderBitmapUnit.t.sol 86ms
-test/unit/LimitOrderCreateUnit.t.sol 172ms (unchanged)
-test/unit/SwapLimitOrdersUnit.t.sol 637ms (unchanged)
-test/unit/SwapLimitOrdersValidation.t.sol 229ms (unchanged)
-test/unit/SwapWithLimitOrdersUnit.t.sol 69ms (unchanged)
-test/utils/BaseTest.sol 635ms (unchanged)
-test/utils/MockWETH.sol 33ms (unchanged)
-test/utils/TestableZoraLimitOrderBook.sol 37ms (unchanged)
+src/access/PermittedCallers.sol 462ms (unchanged)
+src/IZoraLimitOrderBook.sol 66ms (unchanged)
+src/libs/LimitOrderBitmap.sol 64ms (unchanged)
+src/libs/LimitOrderCommon.sol 249ms (unchanged)
+src/libs/LimitOrderCreate.sol 402ms (unchanged)
+src/libs/LimitOrderFill.sol 234ms (unchanged)
+src/libs/LimitOrderLiquidity.sol 256ms (unchanged)
+src/libs/LimitOrderQueues.sol 49ms (unchanged)
+src/libs/LimitOrderStorage.sol 38ms (unchanged)
+src/libs/LimitOrderTypes.sol 3ms (unchanged)
+src/libs/LimitOrderViews.sol 105ms (unchanged)
+src/libs/LimitOrderWithdraw.sol 109ms (unchanged)
+src/libs/Permit2Payments.sol 18ms (unchanged)
+src/libs/SwapLimitOrders.sol 163ms (unchanged)
+src/router/ISetLimitOrderConfig.sol 1ms (unchanged)
+src/router/SwapWithLimitOrders.sol 253ms (unchanged)
+src/ZoraLimitOrderBook.sol 137ms (unchanged)
+test/gas/LimitOrderFillGas.t.sol 859ms (unchanged)
+test/gas/LimitOrderSwapGas.t.sol 306ms (unchanged)
+test/LimitOrderAccessControl.t.sol 388ms (unchanged)
+test/LimitOrderBitmap.t.sol 222ms (unchanged)
+test/LimitOrderCreate.t.sol 389ms (unchanged)
+test/LimitOrderFill.t.sol 1062ms (unchanged)
+test/LimitOrderLibraries.t.sol 395ms (unchanged)
+test/LimitOrderLiquidityPayouts.t.sol 442ms (unchanged)
+test/LimitOrderV4Pools.t.sol 160ms (unchanged)
+test/LimitOrderWithdraw.t.sol 691ms (unchanged)
+test/SwapWithLimitOrders.t.sol 89ms (unchanged)
+test/SwapWithLimitOrdersRouter.t.sol 631ms (unchanged)
+test/unit/LimitOrderBitmapUnit.t.sol 91ms (unchanged)
+test/unit/LimitOrderCreateUnit.t.sol 193ms (unchanged)
+test/unit/SwapLimitOrdersUnit.t.sol 667ms (unchanged)
+test/unit/SwapLimitOrdersValidation.t.sol 251ms (unchanged)
+test/unit/SwapWithLimitOrdersUnit.t.sol 66ms (unchanged)
+test/utils/BaseTest.sol 640ms (unchanged)
+test/utils/MockWETH.sol 28ms (unchanged)
+test/utils/TestableZoraLimitOrderBook.sol 40ms (unchanged)
 CLI Building entry: package/index.ts
 CLI Using tsconfig: tsconfig.json
 CLI tsup v8.4.0
@@ -79,9 +78,9 @@ test/utils/TestableZoraLimitOrderBook.sol 37ms (unchanged)
 CLI Cleaning output folder
 CJS Build start
 ESM Build start
-CJS dist/index.cjs     22.04 KB
-CJS dist/index.cjs.map 37.57 KB
-CJS ⚡️ Build success in 24ms
-ESM dist/index.js     20.87 KB
-ESM dist/index.js.map 37.45 KB
-ESM ⚡️ Build success in 25ms
+CJS dist/index.cjs     25.65 KB
+CJS dist/index.cjs.map 44.14 KB
+CJS ⚡️ Build success in 36ms
+ESM dist/index.js     24.48 KB
+ESM dist/index.js.map 44.03 KB
+ESM ⚡️ Build success in 35ms

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # @zoralabs/limit-orders
 
+## 0.2.4
+
+### Patch Changes
+
+- 3a8a5a05: Fixed infinite loop bug in batch fill when order hasn't crossed tick range
+- 53957bf7: Skip order creation when swaps hit hard price boundaries using sqrt price sentinel checks.
+
+## 0.2.3
+
+### Patch Changes
+
+- 98e02520: Fix dual positive deltas consolidation in burnAndRefund
+
+  - Consolidate payouts to single currency when burning limit orders with dual positive deltas
+  - Extract path-building logic into reusable `_buildSingleHopPath` helper function
+  - Ensure users receive proceeds in their original deposit currency by swapping counter-assets
+  - Align burnAndRefund behavior with burnAndPayout for consistent payout consolidation
+
+- 91a82b2f: Fix withdraw crossed order check to use consistent logic with fills. The `hasCrossed` check for currency1 orders now uses strict `<` comparison instead of `<=`, preventing false positives at the tick boundary. Consolidated `hasCrossed` and `currentPoolTick` helpers into `LimitOrderCommon` for shared use across fill and withdraw paths.
+- e9ffd038: Fix premature limit order fills due to Uniswap v4 tick boundary handling
+
+  Previously, limit orders for non-currency0 tokens could be filled prematurely when the current tick was exactly equal to the order's lower tick boundary. This fix ensures orders are only filled when the pool tick has fully crossed the order's range, preventing users from receiving fewer output tokens than intended.
+
+## 0.2.2
+
+### Patch Changes
+
+- bb2c2b76: Replace AccessManager with PermittedCallers + Ownable2Step for access control in ZoraLimitOrderBook and SwapWithLimitOrders contracts.
+
+  - Remove AccessManager and SimpleAccessManaged pattern
+  - Use OpenZeppelin Ownable2Step for two-step ownership transfer
+  - Add `permittedCallers` mapping to gate `create()` access (public by default, owner can restrict)
+  - Add `isPermittedCaller()` getter and `setPermittedCallers()` batch setter functions
+  - Owner (multisig) retains access to `setMaxFillCount()` and `setLimitOrderConfig()` functions
+
 ## 0.2.1
 
 ### Patch Changes

package/abis/ISetLimitOrderConfig.json

@@ -0,0 +1,27 @@
+[
+  {
+    "type": "function",
+    "name": "setLimitOrderConfig",
+    "inputs": [
+      {
+        "name": "config",
+        "type": "tuple",
+        "internalType": "struct LimitOrderConfig",
+        "components": [
+          {
+            "name": "multiples",
+            "type": "uint256[]",
+            "internalType": "uint256[]"
+          },
+          {
+            "name": "percentages",
+            "type": "uint256[]",
+            "internalType": "uint256[]"
+          }
+        ]
+      }
+    ],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  }
+]

package/abis/{SimpleAccessManaged.json → Ownable.json}

@@ -1,7 +1,7 @@
 [
   {
     "type": "function",
-    "name": "authority",
+    "name": "owner",
     "inputs": [],
     "outputs": [
       {
@@ -14,10 +14,17 @@
   },
   {
     "type": "function",
-    "name": "setAuthority",
+    "name": "renounceOwnership",
+    "inputs": [],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "transferOwnership",
     "inputs": [
       {
-        "name": "newAuthority",
+        "name": "newOwner",
         "type": "address",
         "internalType": "address"
       }
@@ -27,12 +34,18 @@
   },
   {
     "type": "event",
-    "name": "AuthorityUpdated",
+    "name": "OwnershipTransferred",
     "inputs": [
       {
-        "name": "authority",
+        "name": "previousOwner",
         "type": "address",
-        "indexed": false,
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "newOwner",
+        "type": "address",
+        "indexed": true,
         "internalType": "address"
       }
     ],
@@ -40,10 +53,10 @@
   },
   {
     "type": "error",
-    "name": "AccessManagedInvalidAuthority",
+    "name": "OwnableInvalidOwner",
     "inputs": [
       {
-        "name": "authority",
+        "name": "owner",
         "type": "address",
         "internalType": "address"
       }
@@ -51,7 +64,13 @@
   },
   {
     "type": "error",
-    "name": "AccessManagedUnauthorized",
-    "inputs": []
+    "name": "OwnableUnauthorizedAccount",
+    "inputs": [
+      {
+        "name": "account",
+        "type": "address",
+        "internalType": "address"
+      }
+    ]
   }
 ]

package/abis/Ownable2Step.json

@@ -0,0 +1,115 @@
+[
+  {
+    "type": "function",
+    "name": "acceptOwnership",
+    "inputs": [],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "owner",
+    "inputs": [],
+    "outputs": [
+      {
+        "name": "",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "stateMutability": "view"
+  },
+  {
+    "type": "function",
+    "name": "pendingOwner",
+    "inputs": [],
+    "outputs": [
+      {
+        "name": "",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "stateMutability": "view"
+  },
+  {
+    "type": "function",
+    "name": "renounceOwnership",
+    "inputs": [],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "transferOwnership",
+    "inputs": [
+      {
+        "name": "newOwner",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "event",
+    "name": "OwnershipTransferStarted",
+    "inputs": [
+      {
+        "name": "previousOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "newOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      }
+    ],
+    "anonymous": false
+  },
+  {
+    "type": "event",
+    "name": "OwnershipTransferred",
+    "inputs": [
+      {
+        "name": "previousOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "newOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      }
+    ],
+    "anonymous": false
+  },
+  {
+    "type": "error",
+    "name": "OwnableInvalidOwner",
+    "inputs": [
+      {
+        "name": "owner",
+        "type": "address",
+        "internalType": "address"
+      }
+    ]
+  },
+  {
+    "type": "error",
+    "name": "OwnableUnauthorizedAccount",
+    "inputs": [
+      {
+        "name": "account",
+        "type": "address",
+        "internalType": "address"
+      }
+    ]
+  }
+]

package/abis/PermittedCallers.json

@@ -0,0 +1,181 @@
+[
+  {
+    "type": "function",
+    "name": "acceptOwnership",
+    "inputs": [],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "isPermittedCaller",
+    "inputs": [
+      {
+        "name": "caller",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "outputs": [
+      {
+        "name": "",
+        "type": "bool",
+        "internalType": "bool"
+      }
+    ],
+    "stateMutability": "view"
+  },
+  {
+    "type": "function",
+    "name": "owner",
+    "inputs": [],
+    "outputs": [
+      {
+        "name": "",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "stateMutability": "view"
+  },
+  {
+    "type": "function",
+    "name": "pendingOwner",
+    "inputs": [],
+    "outputs": [
+      {
+        "name": "",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "stateMutability": "view"
+  },
+  {
+    "type": "function",
+    "name": "renounceOwnership",
+    "inputs": [],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "setPermittedCallers",
+    "inputs": [
+      {
+        "name": "callers",
+        "type": "address[]",
+        "internalType": "address[]"
+      },
+      {
+        "name": "permitted",
+        "type": "bool[]",
+        "internalType": "bool[]"
+      }
+    ],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "function",
+    "name": "transferOwnership",
+    "inputs": [
+      {
+        "name": "newOwner",
+        "type": "address",
+        "internalType": "address"
+      }
+    ],
+    "outputs": [],
+    "stateMutability": "nonpayable"
+  },
+  {
+    "type": "event",
+    "name": "OwnershipTransferStarted",
+    "inputs": [
+      {
+        "name": "previousOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "newOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      }
+    ],
+    "anonymous": false
+  },
+  {
+    "type": "event",
+    "name": "OwnershipTransferred",
+    "inputs": [
+      {
+        "name": "previousOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "newOwner",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      }
+    ],
+    "anonymous": false
+  },
+  {
+    "type": "event",
+    "name": "PermittedCallerUpdated",
+    "inputs": [
+      {
+        "name": "caller",
+        "type": "address",
+        "indexed": true,
+        "internalType": "address"
+      },
+      {
+        "name": "permitted",
+        "type": "bool",
+        "indexed": false,
+        "internalType": "bool"
+      }
+    ],
+    "anonymous": false
+  },
+  {
+    "type": "error",
+    "name": "CallerNotPermitted",
+    "inputs": []
+  },
+  {
+    "type": "error",
+    "name": "OwnableInvalidOwner",
+    "inputs": [
+      {
+        "name": "owner",
+        "type": "address",
+        "internalType": "address"
+      }
+    ]
+  },
+  {
+    "type": "error",
+    "name": "OwnableUnauthorizedAccount",
+    "inputs": [
+      {
+        "name": "account",
+        "type": "address",
+        "internalType": "address"
+      }
+    ]
+  },
+  {
+    "type": "error",
+    "name": "PermittedCallersLengthMismatch",
+    "inputs": []
+  }
+]