From a5f4bb5242711c177c18a98bb20bdf8c95a45373 Mon Sep 17 00:00:00 2001 From: garykocsis Date: Wed, 10 Jun 2026 01:52:01 -0400 Subject: [PATCH] docs: remove resolved lint cleanup item from roadmap The forge lint style notes (mixed-case, screaming-snake-case, asm-keccak256) are now suppressed via the foundry.toml [lint] config, so the roadmap item is no longer outstanding. The erc20-unchecked-transfer work keeps its own separate bullet. Co-Authored-By: Claude Opus 4.8 (1M context) --- README.md | 6 ------ 1 file changed, 6 deletions(-) diff --git a/README.md b/README.md index 5cb2d002..a8d269dd 100644 --- a/README.md +++ b/README.md @@ -396,12 +396,6 @@ Both pitfalls were diagnosed from on-chain state and are now documented as manda - [ ] Volatility-responsive dynamic fee - [ ] Separate vault contract for buffer custody - [ ] Multisig admin -- [ ] **Code style cleanup:** Resolve `forge lint` notes — - SCREAMING_SNAKE_CASE immutables, mixedCase variables/functions - in test harnesses, unused imports, and the `asm-keccak256` - note on `_positionKey()` (minor gas optimization, correctness - unaffected). Zero impact on security; deferred to mainnet - hardening pass. - [ ] **`DemoLPRouter.sol` hardening:** Add ERC20 transfer return-value checks (currently unchecked — acceptable for MockUSDC testnet demo, not production-ready).