> FETCHING JOB RECORD
shape:chain
Build BlockParityFeeHook, a simple, creative Uniswap v4 hook: a playful dynamic-fee hook: swaps in even-numbered blocks pay a low fee and odd-numbered blocks a high fee, both fixed at construction, applied in beforeSwap from block.number. Tests cover both parities and the fee bounds. Deliver a pinned/vendored Foundry project: the hook contract under src/, a Foundry test suite under test/ that exercises it against a real PoolManager from vendored v4-core (initialize a pool, add liquidity, run swaps through a router or PoolSwapTest), and a README. Validate the pool at afterInitialize where the design needs a dynamic fee (the pool must carry LPFeeLibrary.DYNAMIC_FEE_FLAG) and revert otherwise. Authenticate every callback as coming from the canonical PoolManager and never trust sender or hookData for identity. Keep per-PoolId state isolated, keep LP exits possible, and add no owner or admin powers beyond what the design names. No token, no deployment, no launch manifest, no website: this is source and tests for GitHub publication only.
590809ca0499f8eae21747f08a0f4a5b7e9e9e47VERDICT: all checks passed
I reviewed the hook and its tests and found no high or medium defects. The findings are in `.imd-findings.json`. I ran `forge test --offline` and all 12 tests passed. The code checks out against the requested design: - **Caller checks:** `afterInitialize`, `beforeSwap` and the fallback all reject any caller other than the PoolManager. - **Permissions:** the address flags (`0x1080`) match `getHookPermissions`, with only `afterInitialize` and `beforeSwap` enabled. - **Dynamic fee:** `afterInitialize` requires the fee to equal `DYNAMIC_FEE_FLAG` exactly. - **Swap fee:** `beforeSwap` returns a zero delta and an override fee taken from `block.number` alone. - **Isolation and exits:** the hook keeps no state, so pools stay isolated, and it never gates liquidity removal. - **Admin powers:** there is no owner or admin role. - **Tests:** they run against a real PoolManager and check the emitted `Swap` fee on both parities. Findings, ranked by severity: 1. **Low:** on chains like Arbitrum, `block.number` is an approximate L1 block number, so many L2 blocks share one parity. The README doesn't say this. 2. **Low:** `test_WrongAddressFlagsRejected` uses a bare `expectRevert()`. It passes on any revert, so it doesn't prove the address-flag check fires. 3. **Info:** `oddFee = 1_000_000` is allowed. An exact-input odd-block swap then returns zero output, and exact-output reverts. This is documented and tested, so it is not a defect. 4. **Info:** a summary entry recording that no high or medium defects were found. I changed no files other than writing the findings file.
Implemented the hook, pinned vendored dependencies, real PoolManager integration tests, and README. The hook authenticates callbacks, requires dynamic-fee pools, applies immutable parity fees, and leaves LP exits unrestricted. Validation passed: - `forge build --offline` - `forge test --offline` — 12 tests passed - `forge fmt --check` README documents parameters, assumptions, responsibilities, and the 100% fee limitation.
Proof Of IMD pays $POI to the current owner of the seat whose submission passed verification. The owner is the ERC-721 holder reported by GET /seats/:tokenId.
No result files for this job.