Permissioned Pools#476
Merged
Merged
Conversation
| address permissionedToken1 = | ||
| WRAPPED_TOKEN_FACTORY.verifiedPermissionedTokenOf(Currency.unwrap(poolKey.currency1)); | ||
| if (permissionedToken0 != address(0)) { | ||
| IWrappedPermissionedToken(permissionedToken0).isAllowed(msgSender()); |
Contributor
There was a problem hiding this comment.
Question here: Given we are dealing with a permissioned/wrapped binary, why are we casting the permissioned token to an interface for the wrapped token? Shouldn't this be on poolKey.currency0 (and poolKey.currency1 for line 56)? Unless we are expecting the permissioned tokens to have the same function? I apologize if I am missing something, which definitely may be the case 😄 .
Contributor
Author
There was a problem hiding this comment.
Good catch, thanks!!
* Initial commit: wrapped token tests * fix imports
* initial commit * touch ups, rm shadowed vars * bring in wrapped change * format * nit change to pure * external, pure nit * reset getCode paths * relative imports * fix isolated mint_clear_revert test * use relative paths for mock allowlist * use BaseAllowList not BaseAllowlist * change allowList to allowlist * move to permissioned tests folder * all tests working * merging new changes from permissioned-pools * touch ups, rm shadowed vars * bring in wrapped change * format * nit change to pure * external, pure nit * reset getCode paths * relative imports * fix isolated mint_clear_revert test * use relative paths for mock allowlist * use BaseAllowList not BaseAllowlist * change allowList to allowlist * move to permissioned tests folder * all tests working * replace mockAllowlist with permissionedPoolsBase * rm mock allowlist * run linter * rm unused * change casing on const * reorganize vars * add most of the tests mentioned in comments * all paths covered * seperate hooks contract instead of delegate router * minor cleanup * use all combinations of permissioned and normal in liq tests * spacing nit * nit, edit note * add clarifying comment to wrapped factory balance check * remove circular dependency from permissioned pools implementation (#482) * initial commit * rm fallback from permissioned v4 router, leave receive * fix github ci * use currency instead of ierc20 in some tests * add comment to hooks on why verifyAllowlist is sufficient * inherit from BaseHook contract * remove unused import, generate gas snapshot --------- Co-authored-by: gretzke <daniel@gretzke.de>
* Initial commit: forced transfers by admins * fix merge conflict * remove event
* Initial commit: forced transfers by admins * add granular allow list and multiple hooks and fix revert tests * add coverage for receive * permissioned v4 router test spacing nits * change allowed hook check * allowedHooks on pposm * nit * add inheritdoc to transferFrom * fix merge conflict * make setAllowedHook external * fix test resulting for resolving conflict * simplify checkAllowedHooks * fix snapshot --------- Co-authored-by: gretzke <daniel@gretzke.de>
* initial commit * linted * make dispatch internal * changed revert for command not implemented * add underscore to internal/prive functions on permissioned v4 router * applied fmt to permissioned v4 router
* Add check whether swapping is enabled * Gas snapshot * Add natspec to event
gretzke
force-pushed
the
permissioned-pools
branch
from
November 3, 2025 10:54
7522c84 to
1154a7a
Compare
…ositionManager (#523) * fix(permissioned-hooks): enforce permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * tests(permissioned-hooks): adding unit tests to validate enforcement of permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * update gas snapshot for PermissionedV4Router * fixing snapshots and unused variable nit * fixing snapshots one more time * nit: updating comments in permposm and permv4router
…ction (#524) * fix(permissioned-hooks): enforce permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * tests(permissioned-hooks): adding unit tests to validate enforcement of permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * update gas snapshot for PermissionedV4Router * fixing snapshots and unused variable nit * fixing snapshots one more time * fix: validate mint recipient has LIQUIDITY_ALLOWED in PermissionedPositionManager
…use underlying permissionedToken balances (#525) * fix(permissioned-hooks): enforce permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * tests(permissioned-hooks): adding unit tests to validate enforcement of permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * update gas snapshot for PermissionedV4Router * fixing snapshots and unused variable nit * fixing snapshots one more time * fix: validate mint recipient has LIQUIDITY_ALLOWED in PermissionedPositionManager * fix(permissioned-position-manager): add _mapSettleAmount override for CONTRACT_BALANCE
…erm-posm to be different from regular posm (#529) * fix(permissioned-position-manager): updating the name and symbol to be different from regular posm * nit: fixing test name * nit: adding dev-comment to specify eip712 domain separator
…-200, ECO-222, ECO-226) (#530) * fix(permissioned-position-manager): updating the name and symbol to be different from regular posm * nit: fixing test name * fix: address audit findings N-01, N-02, N-06 * lint: running linter and snapshots * nit: fixing error notes
…m (ECO-196) (#532) Apply the same LIQUIDITY_ALLOWED check used in _mint to the `to` address on admin-initiated transfers, so admins cannot move a permissioned position to a recipient that is not allowlisted for each permissioned currency in the pool. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…apter._unwrap (ECO-198) (#534) * fix(permissioned-pools): use SafeERC20.safeTransfer in PermissionsAdapter._unwrap (ECO-198) Replaces the bare PERMISSIONED_TOKEN.transfer call with SafeERC20.safeTransfer so that tokens which return false on failure (or silently fail under pause/freeze logic) revert the whole transaction instead of burning adapter supply without paying out the underlying. Fixes ECO-198. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * upating snapshot * refactor(permissioned-pools): use solmate SafeTransferLib in PermissionsAdapter._unwrap Switches from OpenZeppelin's SafeERC20 to solmate's SafeTransferLib for consistency with the rest of the repo (e.g. WstETHHook). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * refactor(permissioned-pools): alias OZ ERC20 so solmate ERC20 keeps the plain name Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * refactor(permissioned-pools): alias solmate ERC20 as SafeERC20 to avoid collision Keeps OZ's ERC20 plain (it's the inheritance base) and aliases solmate's ERC20 as SafeERC20 for the SafeTransferLib using directive. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore(permissioned-pools): update PermissionedV4Router snapshot after SafeTransferLib swap solmate SafeTransferLib is slightly cheaper than OZ SafeERC20 on the unwrap path. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Co-authored-by: SocksNFlops <91764028+SocksNFlops@users.noreply.github.com>
…#537) * fix(permissioned-posm): enforce hook allowlist on liquidity increases (ECO-218) * chore(permissioned-posm): reformat with forge fmt 1.4.3 to satisfy CI * nit(tests): fixing function name to be more clear
…allow-revert behavior (#540) * doc(permissioned-v4-router): adding natspec to execute() to clarify allow-revert behavior * Update src/hooks/permissionedPools/PermissionedV4Router.sol Co-authored-by: ens_408 <eric.sanchirico@uniswap.org> --------- Co-authored-by: ens_408 <eric.sanchirico@uniswap.org>
… PositionManager (#528) * feat: emit Swap and ModifyLiquidity events from V4Router and PositionManager Surface user-facing Swap and ModifyLiquidity events so indexers can attribute actions to the actual end user (msg.sender) rather than the router/posm contract address logged by PoolManager. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: formatting and update gas snapshots Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * feat(PermissionsAdapter): pass tokenAddress to allowlist checker and emit verification deposit event - Add `tokenAddress` parameter to `IAllowlistChecker.checkAllowlist` so a single allowlist checker can serve multiple assets without a round-trip back into the adapter to resolve which asset is being checked. - Add `depositForVerification(amount)` and `VerificationDeposit` event to PermissionsAdapter, giving asset issuers a filterable on-chain signal for the initial verification deposit instead of a vanilla ERC20 transfer. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * test(MockAllowlistChecker): name tokenAddress param instead of commenting it out Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * test(MockAllowlistChecker): derive token from tokenAddress param The mock now matches the new interface's purpose — resolve per-token allowlist lookups from the tokenAddress Uniswap passes in, rather than a constructor-provided token. This also uncovered test_permission_flag_partial_permissions relying on the old mock's single-token quirk; the PermissionedHooks _verifyAllowlist actually requires both currencies to be allowed, so inverted the test (and renamed with _reverts suffix) to match the real behavior, consistent with test_permission_flag_partial_permissions_swap on the router side. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: update permissioned router gas snapshot Reflects removal of the MockAllowlistChecker token SLOAD now that the mock reads tokenAddress directly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * feat(V4Router): gate Swap event emission to permissioned-token pools Plumb PermissionsAdapterFactory through the V4Router constructor and only emit Swap events when at least one of the pool currencies is a verified permissioned token. Pools composed of unpermissioned tokens no longer pay the ~32k gas tax for event emission. Moves the Swap event tests from V4RouterTest to PermissionedV4RouterTest, where they exercise the only path that now emits, and updates gas snapshots. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * refactor(PermissionedHooks): emit Swap event from afterSwap hook Moves the permissioned-pool Swap event emission out of V4Router and into PermissionedHooks._afterSwap. V4Router no longer needs a reference to PermissionsAdapterFactory, and unpermissioned pools pay zero incremental gas. Emission lives where permissions are already enforced, and the hook reads the end-user address via IMsgSender(sender).msgSender(). - Reverts V4Router constructor signature and removes Swap event wiring - Adds AFTER_SWAP flag (bit 6) to PermissionedHooks - Removes afterSwap from test_hooks revert expectations - Swap event tests now assert emission from the hook address Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * refactor(IV4Router): drop unused Swap event declaration No router in this repo emits Swap anymore — emission moved to PermissionedHooks in the previous commit. The IV4Router declaration was stale (NatSpec claimed router emission) and redundant with the one on PermissionedHooks. Tests now reference PermissionedHooks.Swap directly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * style(V4Router.t.sol): remove trailing blank line Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * test(PositionManager): assert ModifyLiquidity emits exactly once per action Mint runs through the MINT_POSITION path and increase/decrease/burn run through _modifyLiquidity. They're distinct paths today, but a future refactor that unifies them could silently double-emit. Record logs over each lifecycle action and assert exactly one event from address(lpm). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * style(test): drop unused amount1 decode in swap event helper Use the blank positional slot in destructuring instead of decoding and discarding. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: update gas snapshots and apply forge fmt after merge Regenerate snapshots from the test run and reformat long Swap event emit and helper signatures after merging origin/permissioned-pools. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * chore: regenerate gas snapshots with --isolate (CI profile) Snapshots were previously written without --isolate, which the CI test job runs with. Regenerated using forge 1.3.6 + FOUNDRY_PROFILE=ci to match the CI snapshot check. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…rawClaim` functionality (#542) * fix(permissioned-position-manager): removing transferability of perm-posm positions * feat(permissioned-position-manager): adding unwind-position + withdraw-claim functionality * cleaning up implementation * updating action names to be consistent * fix(permissioned-position-adapter): removing default-recipient parameter in unwind-position fn * fix(permissioned-position-manager): updating position-unwound event to emit once per leg to provide more info about actual recipient and asset distribution during the unwind * nits: cleaning up nits * nit: fixing natspec of withdrawClaim()
…encies (ECO-197) (#533) * fix(permissioned-position-manager): sweep underlying for adapter currencies (ECO-197) When TAKE(adapter, ADDRESS_THIS, ...) runs, PermissionsAdapter._update auto-unwraps on transfer out of the PoolManager, so POSM ends up holding the underlying permissioned token rather than the adapter token. The base _sweep only looks at the adapter currency balance (0) and leaves the underlying stranded, where a later caller could claim it. Override _sweep in PermissionedPositionManager to resolve verified adapter currencies to their underlying token before sweeping. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * nit: running --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com> Co-authored-by: SocksNFlops <91764028+SocksNFlops@users.noreply.github.com>
* fix(permissioned-position-manager): unsubscribing in unwind-position * nit: adding subscribe action placeholder
* fix(permissioned-hooks): unverified-adapter follow-up * chore: trigger CI
…tion (#546) PoolManager already declares an `event ModifyLiquidity(PoolId, address, int24, int24, int256, bytes32)` with the same parameter types as the one PR #528 added to PositionManager. Any downstream contract that inherits from both — e.g. universal-router's `ImportsForTypechain.sol` shim that pulls both into the hardhat build — fails to compile with `DeclarationError: Event with same name and parameter types defined twice`. Rename the periphery event to `ModifyPosition` so downstream consumers don't have to work around the collision. Same parameters, same emit sites, same semantics (`sender` is the end user, not the position manager). Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…t contract (#526) * fix(permissioned-hooks): enforce permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * tests(permissioned-hooks): adding unit tests to validate enforcement of permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * fixing snapshots and unused variable nit * refactor(permissioned-routing): strip PermissionedV4Router to abstract contract * fix(ci): format MockPermissionedRouter * fix(ci): format MockPermissionedRouter and fix snapshot * lint: fixing lint issue in tests * Add per hop slippage checks (#497) * remove hooks and move to hook repo (#510) * remove hooks and move to other repo * remove hook script * fix snapshots * fix: resolve zizmor GitHub Actions security findings (#515) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * Add warning comments to deltas functions (#517) * feat: add per-hop slippage to single swaps and flip to output/input ratio (#516) * fix: increase per-hop slippage precision from 1e18 to 1e36 The 1e18 precision caused the price calculation to truncate to 0 for token pairs with extreme value ratios (e.g. WBTC/PEPE), silently disabling the slippage check. 1e36 provides sufficient precision while remaining overflow-safe (uint128_max * 1e36 < uint256_max). Adds regression test with an extreme price pool verifying the check still fires for high ratio pairs. * feat: add per-hop slippage checks to single swap commands Adds uint256 maxHopSlippage field to ExactInputSingleParams and ExactOutputSingleParams. When non-zero, enforces a price-based (amountIn * PRECISION / amountOut) slippage check. This prevents sandwich attacks when single swaps follow V2/V3 hops with positive slippage in the Universal Router. Introduces V4TooLittleReceivedPerHopSingle and V4TooMuchRequestedPerHopSingle errors for the single swap case. * refactor: flip per-hop slippage to output/input ratio with min check Changes price formula from amountIn/amountOut (revert if > max) to amountOut/amountIn (revert if < min). This makes 0 a natural disabled value for single swap maxHopSlippage since any real output/input ratio is always > 0, removing the need for an explicit != 0 guard. * chore: update gas snapshots * lint * OZ L04 * snapshots * oz L05 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * oz N08 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * PR comments * oz l07 * bytecode hash snapshot --------- Co-authored-by: Alice Henshaw <henshawalice@gmail.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Alice <34962750+hensha256@users.noreply.github.com> * bump version (#518) * docs: update BaseHook in README (#519) * fix: pin npm to specific version in deploy workflow (#520) * fix: pin npm to specific version in deploy workflow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: pin npm install to git commit SHA instead of registry version Use github:npm/cli#<sha> to bypass the npm registry entirely, preventing supply chain attacks on the package manager itself. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(migration): migrating out PermissionedPoolHooks dependency --------- Co-authored-by: Daniel Gretzke <daniel@gretzke.de> Co-authored-by: diana <diana.kocsis@uniswap.org> Co-authored-by: David Gilman <david.gilman@uniswap.org> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Alice Henshaw <henshawalice@gmail.com> Co-authored-by: Alice <34962750+hensha256@users.noreply.github.com> Co-authored-by: Alan Wu <60207036+alanhwu@users.noreply.github.com>
# Conflicts: # test/shared/HookMiner.sol
#548) * fix(permissioned-pools)[eco-354]: dedupe duplicate statements in PermissionedPositionManager._pay() * chore: updating snapshots
…Checker validation (#549)
…pters (#550) * fix(permissioned-position-manager): reject mints in ordinary pools (ECO-344) * chore: running forge fmt
…quirement (#551) * docs(permissioned-pools)[eco-343]: document allowedWrappers trust anchor on IPermissionsAdapter * nit: adding back in example natspec comments
…ction executor (#555) * fix(permissioned-position-manager): require BURN_6909 owner matches action executor * chore: fix snapshot and shorten test name for forge fmt
…d UNWIND_WITH_FALLBACK actions (#556) * fix(permissioned-hooks): enforce permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * tests(permissioned-hooks): adding unit tests to validate enforcement of permission-flags in both payer modes of PermissionedV4Router + PermissionedPositionManager * fixing snapshots and unused variable nit * refactor(permissioned-routing): strip PermissionedV4Router to abstract contract * fix(ci): format MockPermissionedRouter * fix(ci): format MockPermissionedRouter and fix snapshot * lint: fixing lint issue in tests * Add per hop slippage checks (#497) * remove hooks and move to hook repo (#510) * remove hooks and move to other repo * remove hook script * fix snapshots * fix: resolve zizmor GitHub Actions security findings (#515) Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> * Add warning comments to deltas functions (#517) * feat: add per-hop slippage to single swaps and flip to output/input ratio (#516) * fix: increase per-hop slippage precision from 1e18 to 1e36 The 1e18 precision caused the price calculation to truncate to 0 for token pairs with extreme value ratios (e.g. WBTC/PEPE), silently disabling the slippage check. 1e36 provides sufficient precision while remaining overflow-safe (uint128_max * 1e36 < uint256_max). Adds regression test with an extreme price pool verifying the check still fires for high ratio pairs. * feat: add per-hop slippage checks to single swap commands Adds uint256 maxHopSlippage field to ExactInputSingleParams and ExactOutputSingleParams. When non-zero, enforces a price-based (amountIn * PRECISION / amountOut) slippage check. This prevents sandwich attacks when single swaps follow V2/V3 hops with positive slippage in the Universal Router. Introduces V4TooLittleReceivedPerHopSingle and V4TooMuchRequestedPerHopSingle errors for the single swap case. * refactor: flip per-hop slippage to output/input ratio with min check Changes price formula from amountIn/amountOut (revert if > max) to amountOut/amountIn (revert if < min). This makes 0 a natural disabled value for single swap maxHopSlippage since any real output/input ratio is always > 0, removing the need for an explicit != 0 guard. * chore: update gas snapshots * lint * OZ L04 * snapshots * oz L05 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * oz N08 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * PR comments * oz l07 * bytecode hash snapshot --------- Co-authored-by: Alice Henshaw <henshawalice@gmail.com> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Alice <34962750+hensha256@users.noreply.github.com> * bump version (#518) * docs: update BaseHook in README (#519) * fix: pin npm to specific version in deploy workflow (#520) * fix: pin npm to specific version in deploy workflow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: pin npm install to git commit SHA instead of registry version Use github:npm/cli#<sha> to bypass the npm registry entirely, preventing supply chain attacks on the package manager itself. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix(migration): migrating out PermissionedPoolHooks dependency * fix(permissioned-position-manager)[eco-424]: authorize UNSUBSCRIBE and UNWIND_WITH_FALLBACK actions * chore(permissioned-position-manager): tighten unwind params comment * refactor(permissioned-position-manager): inline UNWIND_WITH_FALLBACK auth predicate * chore: running forge fmt * chore(permissioned-position-manager): remove issue ref from test comment --------- Co-authored-by: Daniel Gretzke <daniel@gretzke.de> Co-authored-by: diana <diana.kocsis@uniswap.org> Co-authored-by: David Gilman <david.gilman@uniswap.org> Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com> Co-authored-by: Alice Henshaw <henshawalice@gmail.com> Co-authored-by: Alice <34962750+hensha256@users.noreply.github.com> Co-authored-by: Alan Wu <60207036+alanhwu@users.noreply.github.com>
… emitting ClaimWithdrawn (#557) * fix(permissioned-position-manager): resolve recipient sentinel before emitting ClaimWithdrawn * chore: revert stray gas snapshot drift
…ion (#558) * fix(permissions-adapter): use safeTransferFrom in depositForVerification * chore(snapshots): bump router gas snapshot for safeTransferFrom adapter bytecode shift
…sitionManager size
ericneil-sanc
approved these changes
May 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request introduces a significant update to the codebase by adding support for permissioned pools and wrapped permissioned tokens. The changes include implementing new contracts, interfaces, and functionality to enforce allowlist checks, manage permissioned tokens, and integrate these features into existing components like
PositionManagerandV4Router.Updates to
PositionManagerandV4Router:Enhanced
PositionManagerfunctionality:_mintand_paymethods virtual to allow overriding in derived contracts. (src/PositionManager.sol, [1] [2]PermissionedPositionManagercontract, which overrides_mintand_payto enforce permissioned token checks and wrapping logic. (src/hooks/permissionedPools/PermissionedPositionManager.sol, src/hooks/permissionedPools/PermissionedPositionManager.solR1-R74)Permissioned Router:
PermissionedV4Routercontract, which integrates permissioned token wrapping and allowlist checks into the router's payment logic. (src/hooks/permissionedPools/PermissionedV4Router.sol, src/hooks/permissionedPools/PermissionedV4Router.solR1-R53)Implementation of Wrapped Permissioned Tokens:
Wrapped Permissioned Token Contract:
WrappedPermissionedTokencontract to manage wrapping and unwrapping of permissioned tokens, enforce allowlist checks, and restrict token transfers to the pool manager. (src/hooks/permissionedPools/WrappedPermissionedToken.sol, src/hooks/permissionedPools/WrappedPermissionedToken.solR1-R104)Wrapped Permissioned Token Factory:
WrappedPermissionedTokenFactorycontract to facilitate the creation and verification of wrapped permissioned tokens. (src/hooks/permissionedPools/WrappedPermissionedTokenFactory.sol, src/hooks/permissionedPools/WrappedPermissionedTokenFactory.solR1-R42)New Interfaces for Permissioned Pools:
Allowlist Checker Interface:
IAllowlistCheckerinterface to standardize allowlist checks for permissioned tokens. (src/hooks/permissionedPools/interfaces/IAllowlistChecker.sol, src/hooks/permissionedPools/interfaces/IAllowlistChecker.solR1-R8)Wrapped Permissioned Token Interface:
IWrappedPermissionedTokeninterface to define the behavior of wrapped permissioned tokens, including wrapping, unwrapping, and allowlist checks. (src/hooks/permissionedPools/interfaces/IWrappedPermissionedToken.sol, src/hooks/permissionedPools/interfaces/IWrappedPermissionedToken.solR1-R60)Wrapped Permissioned Token Factory Interface:
IWrappedPermissionedTokenFactoryinterface to define methods for creating and verifying wrapped permissioned tokens. (src/hooks/permissionedPools/interfaces/IWrappedPermissionedTokenFactory.sol, src/hooks/permissionedPools/interfaces/IWrappedPermissionedTokenFactory.solR1-R48)