Add propose_vault_strategy_moves actions for Ethereum and Base#2945
Open
naddison36 wants to merge 1 commit into
Open
Add propose_vault_strategy_moves actions for Ethereum and Base#2945naddison36 wants to merge 1 commit into
naddison36 wants to merge 1 commit into
Conversation
naddison36
requested review from
clement-ux,
shahthepro and
sparrowDom
as code owners
July 21, 2026 05:39
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.
Summary
Adds
proposeVaultStrategyMoves, a Talos action for proposing ordered Vault strategy deposits, withdrawals, and rebalances through the Strategist 2/8 Safe.Supported Vaults:
This uses Safe Transaction Service delegation—not a Safe module. Proposals still require two Safe owner confirmations.
Execution flow
By default, the action:
vault.rebase().checkDelta.The proposed batch is always:
Any simulation, movement, checker, estimation, or nonce validation failure stops before proposal submission.
Movement interface
Movements use an ordered semicolon-separated list:
--moves "withdraw:OUSDMorphoV2StrategyProxy:500000;deposit:OUSDCurveAMOProxy:250000"Supported operations:
deposit:<strategy-name-or-address>:<amount>withdraw:<strategy-name-or-address>:<amount>withdrawAll:<strategy-name-or-address>Deployment names and addresses are supported. Strategy approval and asset compatibility are validated before simulation.
Value Checker
Expected profit and Vault change are derived independently from the local fork unless explicitly overridden.
Using
--skip-forkrequires explicit--expected-profitand--expected-vault-change. Expected values never silently default to zero.Nonce handling
Talos integration
Adds separate disabled manual schedules because Talos locks
--network:propose_vault_strategy_moves_mainnetpropose_vault_strategy_moves_baseRequired configuration:
SAFE_API_KEYPROVIDER_URLfor Ethereum forksBASE_PROVIDER_URLfor Base forksTesting
pnpm lint:tspnpm hardhat test test/tasks/vault-strategy-moves.js— 17 passinggit diff --checkAuthenticated Safe submission was not performed because a production
SAFE_API_KEYwas unavailable in the development environment.Code Change Checklist
To be completed before internal review begins:
Internal review: