@zoralabs/limit-orders 0.2.1 → 0.2.2

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.2 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.2 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.2 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.13s
 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.2 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.2 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 449ms (unchanged)
+src/IZoraLimitOrderBook.sol 54ms (unchanged)
+src/libs/LimitOrderBitmap.sol 57ms (unchanged)
+src/libs/LimitOrderCommon.sol 210ms (unchanged)
+src/libs/LimitOrderCreate.sol 374ms (unchanged)
+src/libs/LimitOrderFill.sol 244ms (unchanged)
+src/libs/LimitOrderLiquidity.sol 250ms (unchanged)
+src/libs/LimitOrderQueues.sol 43ms (unchanged)
+src/libs/LimitOrderStorage.sol 13ms (unchanged)
+src/libs/LimitOrderTypes.sol 3ms (unchanged)
+src/libs/LimitOrderViews.sol 114ms (unchanged)
+src/libs/LimitOrderWithdraw.sol 109ms (unchanged)
+src/libs/Permit2Payments.sol 17ms (unchanged)
+src/libs/SwapLimitOrders.sol 173ms (unchanged)
+src/router/ISetLimitOrderConfig.sol 1ms (unchanged)
+src/router/SwapWithLimitOrders.sol 301ms (unchanged)
+src/ZoraLimitOrderBook.sol 111ms (unchanged)
+test/gas/LimitOrderFillGas.t.sol 792ms (unchanged)
+test/gas/LimitOrderSwapGas.t.sol 283ms (unchanged)
+test/LimitOrderAccessControl.t.sol 396ms (unchanged)
+test/LimitOrderBitmap.t.sol 212ms (unchanged)
+test/LimitOrderCreate.t.sol 364ms (unchanged)
+test/LimitOrderFill.t.sol 999ms (unchanged)
+test/LimitOrderLibraries.t.sol 371ms (unchanged)
+test/LimitOrderLiquidityPayouts.t.sol 420ms (unchanged)
+test/LimitOrderV4Pools.t.sol 141ms (unchanged)
+test/LimitOrderWithdraw.t.sol 665ms (unchanged)
+test/SwapWithLimitOrders.t.sol 88ms (unchanged)
+test/SwapWithLimitOrdersRouter.t.sol 626ms (unchanged)
+test/unit/LimitOrderBitmapUnit.t.sol 80ms (unchanged)
+test/unit/LimitOrderCreateUnit.t.sol 193ms (unchanged)
+test/unit/SwapLimitOrdersUnit.t.sol 601ms (unchanged)
+test/unit/SwapLimitOrdersValidation.t.sol 228ms (unchanged)
+test/unit/SwapWithLimitOrdersUnit.t.sol 63ms (unchanged)
+test/utils/BaseTest.sol 593ms (unchanged)
+test/utils/MockWETH.sol 34ms (unchanged)
+test/utils/TestableZoraLimitOrderBook.sol 41ms (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 35ms
+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,17 @@
 # @zoralabs/limit-orders
 
+## 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": []
+  }
+]