Skip to content

Add DEX/oracle slippage protection to insurance collection#152

Merged
jordanschalm merged 6 commits intomainfrom
jord/139-insurance-swapper-slippage
Feb 10, 2026
Merged

Add DEX/oracle slippage protection to insurance collection#152
jordanschalm merged 6 commits intomainfrom
jord/139-insurance-swapper-slippage

Conversation

@jordanschalm
Copy link
Copy Markdown
Member

Extract inline DEX/oracle deviation check from manualLiquidation into a shared assertDexOracleDeviation helper and apply the same check to collectInsurance, preventing insurance swaps at manipulated DEX prices.

jordanschalm and others added 4 commits February 9, 2026 15:53
Extract inline DEX/oracle deviation check from manualLiquidation into a
shared assertDexOracleDeviation helper and apply the same check to
collectInsurance, preventing insurance swaps at manipulated DEX prices.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jordanschalm jordanschalm marked this pull request as ready for review February 10, 2026 17:51
@jordanschalm jordanschalm requested a review from a team as a code owner February 10, 2026 17:51
@@ -1069,7 +1069,9 @@ access(all) contract FlowCreditMarket {
/// @param reserveVault: The reserve vault for this token type to withdraw insurance from
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we add @param comments for the new parameters?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, done in 7b050e6

@jordanschalm jordanschalm merged commit 1dc50bd into main Feb 10, 2026
1 check passed
@jordanschalm jordanschalm deleted the jord/139-insurance-swapper-slippage branch February 10, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants