Skip to content

Conversation

@aalavandhan
Copy link
Member

No description provided.

dependabot bot and others added 24 commits October 17, 2024 08:31
Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 2.3.0 to 2.3.3.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](coverallsapp/github-action@v2.3.0...v2.3.3)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 2.3.3 to 2.3.4.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](coverallsapp/github-action@v2.3.3...v2.3.4)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* removed pricing strategies in favor of meta oracle

* Bill broker refactor, using new meta oracle

* Charm vault refactor, using new meta oracle

* Updated Interfaces and Errors

* updated unit tests

* code review changes
Apply suggestions from code review

Co-authored-by: Brandon Iles <brandon@fragments.org>

review fix

caching bool instead of prev deviation
* deployed new oracle and managers

* updated readme
* staking subgraph, refreshing store on rebase

* charm wampl vault subgraph
* adds timestamp to swap events subgraph

Signed-off-by: nms-7 <nixon.ubud@gmail.com>

* linting

Signed-off-by: nms-7 <nixon.ubud@gmail.com>

* fixed null swap creation with 0 timestamp

Signed-off-by: nms-7 <nixon.ubud@gmail.com>

---------

Signed-off-by: nms-7 <nixon.ubud@gmail.com>
Bumps [coverallsapp/github-action](https://github.com/coverallsapp/github-action) from 2.3.4 to 2.3.6.
- [Release notes](https://github.com/coverallsapp/github-action/releases)
- [Commits](coverallsapp/github-action@v2.3.4...v2.3.6)

---
updated-dependencies:
- dependency-name: coverallsapp/github-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* using fee factor instead of percentages to support -ve fees

* unit tests

* code review

* convergent fee curve
* fee curve update

* code review fixes
* rollover fee handles undersubscribed vault

* bumped up hardhat

* code review comments
@openzeppelin-code
Copy link

openzeppelin-code bot commented Mar 9, 2025

Release V5.0.0

Generated at commit: 0ba831ea47a2855f7f7346b82afdf3d6b8fd16d1

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
1
0
0
3
24
28
Dependencies Critical
High
Medium
Low
Note
Total
0
0
0
0
0
0

For more details view the full report in OpenZeppelin Code Inspector

@aalavandhan aalavandhan changed the title WIP: Release V5.0.0 - Under Development Release V5.0.0 - Under Review Apr 18, 2025
@aalavandhan aalavandhan added the in-review Currently in review label Apr 18, 2025
return 0;
}

// We compute equal value of perp tokens going out.
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
// We compute equal value of perp tokens going out.

// The mint fees are waived, because single sided deposits
// push the pool back into balance.
uint256 percOfAmtInSwapped = ONE.mulDiv(
usdAmtIn - (s.usdBalance + usdAmtIn).mulDiv(mintAmt, totalSupply_),
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
usdAmtIn - (s.usdBalance + usdAmtIn).mulDiv(mintAmt, totalSupply_),
usdAmtIn - (s.usdBalance + usdAmtIn).mulDiv(mintAmt, totalSupply_ + mintAmt),

)
: 0;
// Compute mint amount.
mintAmt = (totalReserveVal > 0)
Copy link
Member

Choose a reason for hiding this comment

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

Can remove the 0 check, given we already do it right above.

// The mint fees are waived, because single sided deposits
// push the pool back into balance.
uint256 percOfAmtInSwapped = ONE.mulDiv(
perpAmtIn - (s.perpBalance + perpAmtIn).mulDiv(mintAmt, totalSupply_),
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
perpAmtIn - (s.perpBalance + perpAmtIn).mulDiv(mintAmt, totalSupply_),
perpAmtIn - (s.perpBalance + perpAmtIn).mulDiv(mintAmt, totalSupply_ + mintAmt),

* fee policy restructure

* updated perp fee and rebalance handling, audit cleanup

* updated vault fee and rebalance handling, audit cleanup

* interface and lib updates

* updated unit tests

* code review fixes #1

* code review fixes #2

* code review fix #3

* Removed subscription ratio for a global vault:perp tvl (or system ratio) (#254)

* Using a system ratio target

* using system state to calculate fees

* using system state for fees and removed subscription liquidity check

* interface updates

* unit test updates

* rename
@aalavandhan aalavandhan changed the title Release V5.0.0 - Under Review Release V5.0.0 May 8, 2025
@aalavandhan aalavandhan merged commit 25a4544 into main May 8, 2025
1 check passed
@aalavandhan aalavandhan deleted the development branch May 8, 2025 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in-review Currently in review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants