Minor issues (#14 from audit) (SC-1032)#130
Merged
lucas-manuel merged 10 commits intodevfrom Jun 27, 2025
Merged
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR addresses minor clean-up items from the recent audit, primarily updating import paths to use OpenZeppelin contracts, removing an unused helper import, and fixing a small typo in a comment.
- Switched
IERC20imports (andIERC4626) fromforge-stdtoopenzeppelin-contracts - Removed the now-unused
RateLimitHelpersimport inPSMLib.sol - Corrected a typo in the CurveLib comment (“multipled” → “multiplied”) and reorganized controller imports with a compliance note for
IERC7540
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/libraries/PSMLib.sol | Updated IERC20 import path; removed unused RateLimitHelpers import |
| src/libraries/CurveLib.sol | Fixed typo in comment (“multipled” → “multiplied”) |
| src/libraries/CCTPLib.sol | Updated IERC20 import path |
| src/MainnetController.sol | Reordered imports; added compliance comment for IERC7540; updated IERC20/IERC4626 imports |
| src/ForeignController.sol | Reordered imports; added compliance comment for IERC7540; updated IERC20/IERC4626 imports |
Comments suppressed due to low confidence (2)
src/MainnetController.sol:7
- [nitpick] Consider clarifying why
IERC7540remains imported fromforge-std(e.g., lack of OpenZeppelin support) and reference the specific version or commit used for audit traceability.
// This one has been reviewed, and is compliant with the specs: https://eips.ethereum.org/EIPS/eip-7540
src/libraries/PSMLib.sol:4
- After removing
RateLimitHelpers, please verify that there are no remaining references to its functions to avoid compilation errors or broken logic.
import { IERC20 } from "openzeppelin-contracts/contracts/interfaces/IERC20.sol";
lucas-manuel
requested changes
Jun 27, 2025
lucas-manuel
requested changes
Jun 27, 2025
Comment on lines
6
to
+7
| import { IRateLimits } from "../interfaces/IRateLimits.sol"; | ||
| import { IALMProxy } from "../interfaces/IALMProxy.sol"; | ||
| import { IALMProxy } from "../interfaces/IALMProxy.sol"; |
Contributor
There was a problem hiding this comment.
Needs to be alphabetically ordered
lucas-manuel
approved these changes
Jun 27, 2025
lucas-manuel
requested changes
Jun 27, 2025
src/ForeignController.sol
Outdated
| import { IERC4626 } from "openzeppelin-contracts/contracts/interfaces/IERC4626.sol"; | ||
|
|
||
| import { AccessControl } from "openzeppelin-contracts/contracts/access/AccessControl.sol"; | ||
| import { IMetaMorpho, Id, MarketAllocation } from "metamorpho/interfaces/IMetaMorpho.sol"; |
Contributor
There was a problem hiding this comment.
metamorpho should be above openzeppelin (alphabetical)
|
Coverage after merging sc-1032-minor-issues into dev will be
Coverage Report
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
lucas-manuel
approved these changes
Jun 27, 2025
barrutko
added a commit
to grove-labs/grove-alm-controller
that referenced
this pull request
Jul 17, 2025
* feat: Add and rename audits (#96) * feat: add cantina * fix: change all names * feat: Refactor staging deployments scripting to work for full deployment on Base and Arbitrum (#97) * feat: do initial refactor * feat: initial deploy script working * feat: update with working script * feat: staging deployment working locally * fix: cleanup * feat: update to add full input files * feat: add arbitrum test coverage * feat: add more testing and update arbitrum file * fix: update to remove deps from output, updagte lib * feat: add live staging contracts * fix: update sub and blocks * feat: add release jsons * fix: add mainnet staing * fix: arb filename * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * fix: Update swap rate limit calculation (#94) * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * feat: initial logic working * feat: update to add testing * fix: cleanup --------- Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> * feat: Update staging PSM addresses, fix morpho usdc staging address (#98) * feat: update addresses * fix: logs * fix: rm unconfigured psms * feat: Add v1.4.0 audits (#99) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * feat: Deploy v1.4.0 staging (#103) * feat: deploy v1.4.0 staging * fix: newlines * fix: update testing * feat: Update testing to use USDT pools, focus on USDT approval functionality (#101) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * fix: refactor tests * test: add allowance assertions * test: add coverage for swap rate limits * fix: update to use higher swaps * refactor: Remove `setRateLimitData` from RateLimitHelpers (SC-962) (sparkdotfi#108) * refactor: Split out Curve and PSM logic into libraries (SC-971) (#104) * refactor: Init library to take multiple relayers (SC 961) (sparkdotfi#109) * refactor: Spin out logic of CCTP into a library (SC-970) (#105) * feat: Adds forceApprove logic (SC-978) (sparkdotfi#110) * feat: Deploy Unichain (SC-992) * cleanup: Remove BUIDL redeem facility (SC 994) * chore: Change `CENTRIFUGE_REQUEST_ID` var to constant (sparkdotfi#118) * chore: change var to constant * use internal * feat: Add LayerZero integration (SC-982) (sparkdotfi#119) * feat: layerzero integration * fix review * change note * fix followup review * add success tests * fix review * add gas cost test * chore: linting * add tests for setter functions * chore: linting * chore: linting * fix: Update README (sparkdotfi#121) * fix: Order params consistently (sparkdotfi#122) * fix: interface (sparkdotfi#124) * fix: Use more robust `_approve` (SC-1036) (sparkdotfi#126) * feat: add requires * fix: add full test coverage * feat: Gas optimize `approve` (SC-1037) (sparkdotfi#128) * feat: add requires * fix: add full test coverage * feat: reuse `approveData` in (Mainnet|Foreign)Controller._approve. * fix: typo (causing compilation error). --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * Minor issues (#14 from audit) (SC-1032) (sparkdotfi#130) * chore: rm `IMetaMorpho` imports in MainnetController.sol. * chore: rm `AccessControl` import in CurveLib.sol. * chore: rm `RateLimitHelpers` import in PSMLib.sol. * fix: `multiplied` spelling. * chore: change IERC(20|4626) to openzeppelin. * chore: rm import from ForeignController.sol. * fix: alphabetical ordering. * fix: re-order imports. --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Make `transferTokenLayerZero` payable (SC-1039) (sparkdotfi#127) * fix: make transfer function payable * fix: review --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Set LayerZero recipients in deploy (SC-1038) (sparkdotfi#129) * feat: setLayerzeroRecipients in deploy * fix: review * fix: build * fix: align --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Add missing approval (SC-1035) (sparkdotfi#123) * fix: add missing approval * add comments * fix: comments * fix: comments * fix: comment --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Reuse approveData in CurveLib (sparkdotfi#131) * update * remove script * update * chore: remove irrelevant deployments * test: skip arbitrum deployment tests * adjust deployment inputs * reorg * change staging psm address --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> Co-authored-by: supercontracts <bonjour.tabish@gmail.com> Co-authored-by: Dom <18601956+hacker-DOM@users.noreply.github.com>
kevinlichan
pushed a commit
to grove-labs/grove-alm-controller
that referenced
this pull request
Jul 17, 2025
* feat: Add and rename audits (#96) * feat: add cantina * fix: change all names * feat: Refactor staging deployments scripting to work for full deployment on Base and Arbitrum (#97) * feat: do initial refactor * feat: initial deploy script working * feat: update with working script * feat: staging deployment working locally * fix: cleanup * feat: update to add full input files * feat: add arbitrum test coverage * feat: add more testing and update arbitrum file * fix: update to remove deps from output, updagte lib * feat: add live staging contracts * fix: update sub and blocks * feat: add release jsons * fix: add mainnet staing * fix: arb filename * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * fix: Update swap rate limit calculation (#94) * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * feat: initial logic working * feat: update to add testing * fix: cleanup --------- Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> * feat: Update staging PSM addresses, fix morpho usdc staging address (#98) * feat: update addresses * fix: logs * fix: rm unconfigured psms * feat: Add v1.4.0 audits (#99) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * feat: Deploy v1.4.0 staging (#103) * feat: deploy v1.4.0 staging * fix: newlines * fix: update testing * feat: Update testing to use USDT pools, focus on USDT approval functionality (#101) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * fix: refactor tests * test: add allowance assertions * test: add coverage for swap rate limits * fix: update to use higher swaps * refactor: Remove `setRateLimitData` from RateLimitHelpers (SC-962) (sparkdotfi#108) * refactor: Split out Curve and PSM logic into libraries (SC-971) (#104) * refactor: Init library to take multiple relayers (SC 961) (sparkdotfi#109) * refactor: Spin out logic of CCTP into a library (SC-970) (#105) * feat: Adds forceApprove logic (SC-978) (sparkdotfi#110) * feat: Deploy Unichain (SC-992) * cleanup: Remove BUIDL redeem facility (SC 994) * chore: Change `CENTRIFUGE_REQUEST_ID` var to constant (sparkdotfi#118) * chore: change var to constant * use internal * feat: Add LayerZero integration (SC-982) (sparkdotfi#119) * feat: layerzero integration * fix review * change note * fix followup review * add success tests * fix review * add gas cost test * chore: linting * add tests for setter functions * chore: linting * chore: linting * fix: Update README (sparkdotfi#121) * fix: Order params consistently (sparkdotfi#122) * fix: interface (sparkdotfi#124) * fix: Use more robust `_approve` (SC-1036) (sparkdotfi#126) * feat: add requires * fix: add full test coverage * feat: Gas optimize `approve` (SC-1037) (sparkdotfi#128) * feat: add requires * fix: add full test coverage * feat: reuse `approveData` in (Mainnet|Foreign)Controller._approve. * fix: typo (causing compilation error). --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * Minor issues (#14 from audit) (SC-1032) (sparkdotfi#130) * chore: rm `IMetaMorpho` imports in MainnetController.sol. * chore: rm `AccessControl` import in CurveLib.sol. * chore: rm `RateLimitHelpers` import in PSMLib.sol. * fix: `multiplied` spelling. * chore: change IERC(20|4626) to openzeppelin. * chore: rm import from ForeignController.sol. * fix: alphabetical ordering. * fix: re-order imports. --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Make `transferTokenLayerZero` payable (SC-1039) (sparkdotfi#127) * fix: make transfer function payable * fix: review --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Set LayerZero recipients in deploy (SC-1038) (sparkdotfi#129) * feat: setLayerzeroRecipients in deploy * fix: review * fix: build * fix: align --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Add missing approval (SC-1035) (sparkdotfi#123) * fix: add missing approval * add comments * fix: comments * fix: comments * fix: comment --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Reuse approveData in CurveLib (sparkdotfi#131) * update * remove script * update * chore: remove irrelevant deployments * test: skip arbitrum deployment tests * adjust deployment inputs * reorg * change staging psm address * feat: add Centrifuge functions to ForeignController * test: add ForeignController tests on fake Avalanche (ETH Mainnet) * refactor: add minor fixes * remove transfer --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> Co-authored-by: supercontracts <bonjour.tabish@gmail.com> Co-authored-by: Dom <18601956+hacker-DOM@users.noreply.github.com>
barrutko
added a commit
to grove-labs/grove-alm-controller
that referenced
this pull request
Jul 25, 2025
* Add arbitrum deployment (#84) * add arbitrum deployment * newline * feat: Add and rename audits (#96) * feat: add cantina * fix: change all names * feat: Refactor staging deployments scripting to work for full deployment on Base and Arbitrum (#97) * feat: do initial refactor * feat: initial deploy script working * feat: update with working script * feat: staging deployment working locally * fix: cleanup * feat: update to add full input files * feat: add arbitrum test coverage * feat: add more testing and update arbitrum file * fix: update to remove deps from output, updagte lib * feat: add live staging contracts * fix: update sub and blocks * feat: add release jsons * fix: add mainnet staing * fix: arb filename * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * fix: Update swap rate limit calculation (#94) * feat: Add audits (#26) * feat: add audits * fix: update file * fix: update staging test * fix: Update Centrifuge tests (#25) * Adapt to latest vault address * Update block number * Remove unused setUp --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * feat: Add Curve support, remove Morpho allocator logic (SC-938) (#23) * feat: add basic curve deposit/withdraw * feat: Refactor to use internal functions and modifiers (#24) * feat: refactor to use internal functions and modifiers * fix: use view * fix: move helpers * fix: rm todo * feat: Add DaiUsds swaps (SC-940) (#22) * feat: add dai usds swaps * fix: rm rate limits * test: add testing for failure modes * feat: tests all passing * fix: update broken staging test * test: add invalid order coverage, cleanup * fix: rm morpho functionality * fix: rm morpho * feat: tests passing * fix: rm console * feat: refactor to use rlusd pool * feat: refactor to use new slippages, remove tokens params * feat: add remove liquidity working * feat: tests passing * fix: rm console * fix: update remaining fixes * fix: reorder some code, add zero slippage test coverage * fix: update test names * feat: add simplified calculation * fix: rm unused function * fix: formatting * fix: update slippage * fix: comment * fix: cache param * fix: update spacing * feat: Update to round up for rate limits and minimums (SC-955) (#31) * feat: Use `get_virtual_price` (SC-959) (#29) * feat: add comments * feat: add virtual price * feat: add stress test * feat: add starting fuzz tests * fix: delete fuzz test * fix: add comments * fix: update comments * fix: Add line about transferAsset (#33) * feat: Add swap rate limiting in `addLiquidityCurve` (SC-952) (#30) * feat: add initial structure * feat: add rate limit for swap * feat: tests passing * fix: ordering and test * fix: update optimizer runs * fix: update comment * feat: Add index input validation (SC-960) (#34) * feat: add index input validation * feat: update to add testing * ifx: update to use n coins * feat: initial logic working * feat: update to add testing * fix: cleanup --------- Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> * feat: Update staging PSM addresses, fix morpho usdc staging address (#98) * feat: update addresses * fix: logs * fix: rm unconfigured psms * feat: Add v1.4.0 audits (#99) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * feat: Deploy v1.4.0 staging (#103) * feat: deploy v1.4.0 staging * fix: newlines * fix: update testing * feat: Update testing to use USDT pools, focus on USDT approval functionality (#101) * fix: rm redundant audits, add chainsecurity * feat: add cantina audit * fix: refactor tests * test: add allowance assertions * test: add coverage for swap rate limits * fix: update to use higher swaps * refactor: Remove `setRateLimitData` from RateLimitHelpers (SC-962) (sparkdotfi#108) * refactor: Split out Curve and PSM logic into libraries (SC-971) (#104) * refactor: Init library to take multiple relayers (SC 961) (sparkdotfi#109) * refactor: Spin out logic of CCTP into a library (SC-970) (#105) * deploy optimism (sparkdotfi#111) * feat: Adds forceApprove logic (SC-978) (sparkdotfi#110) * feat: Deploy Unichain (SC-992) * cleanup: Remove BUIDL redeem facility (SC 994) * chore: Change `CENTRIFUGE_REQUEST_ID` var to constant (sparkdotfi#118) * chore: change var to constant * use internal * feat: Add LayerZero integration (SC-982) (sparkdotfi#119) * feat: layerzero integration * fix review * change note * fix followup review * add success tests * fix review * add gas cost test * chore: linting * add tests for setter functions * chore: linting * chore: linting * fix: Update README (sparkdotfi#121) * fix: Order params consistently (sparkdotfi#122) * fix: interface (sparkdotfi#124) * fix: Use more robust `_approve` (SC-1036) (sparkdotfi#126) * feat: add requires * fix: add full test coverage * feat: Gas optimize `approve` (SC-1037) (sparkdotfi#128) * feat: add requires * fix: add full test coverage * feat: reuse `approveData` in (Mainnet|Foreign)Controller._approve. * fix: typo (causing compilation error). --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * Minor issues (#14 from audit) (SC-1032) (sparkdotfi#130) * chore: rm `IMetaMorpho` imports in MainnetController.sol. * chore: rm `AccessControl` import in CurveLib.sol. * chore: rm `RateLimitHelpers` import in PSMLib.sol. * fix: `multiplied` spelling. * chore: change IERC(20|4626) to openzeppelin. * chore: rm import from ForeignController.sol. * fix: alphabetical ordering. * fix: re-order imports. --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Make `transferTokenLayerZero` payable (SC-1039) (sparkdotfi#127) * fix: make transfer function payable * fix: review --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * feat: Set LayerZero recipients in deploy (SC-1038) (sparkdotfi#129) * feat: setLayerzeroRecipients in deploy * fix: review * fix: build * fix: align --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Add missing approval (SC-1035) (sparkdotfi#123) * fix: add missing approval * add comments * fix: comments * fix: comments * fix: comment --------- Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> * fix: Reuse approveData in CurveLib (sparkdotfi#131) * feat: Add v1.5 audit reports (SC-1044) (sparkdotfi#132) * feat: add cantina audit * fix: update spark-address-registry * feat: add new audits * fix: rm ds store files and add to gitignore * feat: bring back superstate & maple; remove centrifuge crosschain transfers * cleanup deployments --------- Co-authored-by: Sam MacPherson <xylom44@gmail.com> Co-authored-by: Lucas Manuel <lucasmanuel.tech@gmail.com> Co-authored-by: Jeroen <1748621+hieronx@users.noreply.github.com> Co-authored-by: supercontracts <bonjour.tabish@gmail.com> Co-authored-by: Dom <18601956+hacker-DOM@users.noreply.github.com>
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
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.
No description provided.