Skip to content

fix: liquidation pool rounding error #619

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
nieldeckx wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix: liquidation pool rounding error #619

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c23742d
fix: liquidation pool rounding error
nieldeckx Apr 27, 2026
033004c
fix: test
nieldeckx Apr 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions clarity/contracts/vaults-v2/arkadiko-vaults-pool-liq-v1-2.clar
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,9 @@
(define-constant ERR_CLAIM_FAILED u950002)
(define-constant ERR_INVALID_REWARD_TOKEN u950003)
(define-constant ERR_INSUFFICIENT_USDA u950004)
(define-constant ERR_POOL_BALANCE_TOO_LOW u950005)

(define-constant MIN_POOL_USDA u10000000000)

;; ---------------------------------------------------------
;; Variables
Expand Down Expand Up @@ -397,6 +400,7 @@
(new-usda-balance (- usda-balance amount))
(new-fragments-per-token (/ (var-get fragments-total) new-usda-balance))
)
(asserts! (>= new-usda-balance MIN_POOL_USDA) (err ERR_POOL_BALANCE_TOO_LOW))
(var-set fragments-per-token new-fragments-per-token)

(try! (as-contract (contract-call? .arkadiko-dao burn-token .usda-token amount tx-sender)))
Expand Down
Loading
Loading