defi-amm-security

Security checklist for Solidity AMM contracts, liquidity pools, and swap flows. Covers reentrancy, CEI ordering, donation or inflation attacks, oracle manipulation, slippage, admin controls, and integer math. Use when auditing or writing Solidity AMM, liquidity pool, or swap code.

By affaan-m · 2,822 installs

npx skills add affaan-m/ecc --skill defi-amm-security

Source repository · Upstream listing

DeFi AMM Security Critical vulnerability patterns and hardened implementations for Solidity AMM contracts, LP vaults, and swap functions. When to Use Writing or auditing a Solidity AMM or liquidity pool contract Implementing swap, deposit, withdraw, mint, or burn flows that hold token balances Reviewing any contract that uses token.balanceOf(address(this)) in share or reserve math Adding fee setters, pausers, oracle updates, or other admin functions to a DeFi protocol How It Works Use this as a checklist plus pattern library. Review every user entrypoint against the categories below and prefer the hardened examples over hand rolled variants. Execution Safety The shell commands in this skill are local audit examples. Run them only in a trusted checkout or disposable sandbox, and do not splice untrusted contract names, paths, RPC URLs, private keys, or user supplied flags into shell commands. Ask before installing tools or running long fuzzing/static analysis jobs that may consume significant local or paid resources. Never include secrets, private keys, seed phrases, API tokens, or mainnet signing credentials in command examples, logs, or reports. Examples Reentrancy: enforce CEI order Vulnerable: Safe: Do not write your own guard when a hardened library exists. Donation or inflation attacks Using token.balanceOf(address(this)) directly for share math lets attackers manipulate the denominator by sending tokens to the contract outside the intended path. Track internal accounting and measure actual tokens received. Oracle manipulation Spot prices are flash loan manipulable. Prefer TWAP. Slippage protection Every swap path needs caller provided slippage and a deadline. Safe reserve math For large reserve math, avoid naive a b / c when overflow risk exists. Admin controls Prefer explicit acceptance for ownership transfer and gate every privileged path. Security Checklist Reentrancy exposed entrypoints use nonReentrant CEI ordering is respected Share math does not depend on raw balanceOf(address(this)) ERC 20 transfers use SafeERC20 Deposits measure actual tokens received Oracle reads use TWAP or another manipulation resistant source Swaps require amountOutMin and deadline Overflow sensitive reserve math uses safe primitives like mulDiv Admin functions are access controlled Emergency pause exists and is tested Static analysis and fuzzing are run before production Audit Tools