[VPD-1100]: TokenBuyback migration proposal and simulation#700
Draft
Debugger022 wants to merge 2 commits intomainfrom
Draft
[VPD-1100]: TokenBuyback migration proposal and simulation#700Debugger022 wants to merge 2 commits intomainfrom
Debugger022 wants to merge 2 commits intomainfrom
Conversation
Replaces the legacy Token Converter system (7 contracts) with 10 ACM-authorized TokenBuyback proxies on BNB Chain mainnet and testnet. Includes RiskFundV2 upgrade, ACM grants, PSR rewiring, and fork simulation scaffolding.
Debugger022
changed the title
Contributor
|
from hashdit audit L02: call pauseAuctions() defensively as part of the VIP to keep the surface closed post-upgrade just in case |
Add Shortfall.pauseAuctions() to the VIP-800 proposal on both bscmainnet and bsctestnet, with matching pre/post-state assertions in the simulations. Isolated pools are wound down so no live or upcoming auctions exist, but pausing closes the surface post-RiskFundV2-upgrade as defense in depth and removes any need for an auctionsPaused or transferReserveForAuction balance check in the upgrade path.
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
2 participants
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.
Summary
XVSVaultConverter, WBNBBurnConverter, ConverterNetwork) with 10 ACM-authorized
TokenBuyback proxies operated by a finance-team cron.
(
updatePoolState,sweepTokenFromPool,poolAssetsFundsmapping).eliminating the oracle-coupled 50% premium and ConverterNetwork routing layer.
Changes
vips/vip-800/bscmainnet.ts— proposal definition: RiskFundV2 upgrade, acceptOwnership on 10buybacks, PancakeSwap router allowlisting, ACM grants for cron operator,
legacy converter drain + ACM revoke, PSR distribution repoint
bsctestnet.ts— identical flow targeting testnet addressessimulations/vip-800/bscmainnet.ts/bsctestnet.ts— fork simulation scaffolding: pre/post-VIPstate assertions for ownership, router allowlist, ACM permissions, PSR config,
and legacy converter balances
abi/— ABIs for AccessControlManager, DefaultProxyAdmin, ERC20,ProtocolShareReserve, SingleTokenConverter, TokenBuyback
Test plan