Skip to content

Deploy Base 46: upgrade cross-chain remote strategy#2830

Merged
sparrowDom merged 4 commits intomasterfrom
sparrowDom/deploy_base_045
Mar 9, 2026
Merged

Deploy Base 46: upgrade cross-chain remote strategy#2830
sparrowDom merged 4 commits intomasterfrom
sparrowDom/deploy_base_045

Conversation

@sparrowDom
Copy link
Member

Deploy 46

  • Deploys a new implementation of CrossChainRemoteStrategy supporting an accurate withdrawAll from the Morpho V2 vault

Deployed contracts:

Deployed contracts

contract name contract address
CrossChainRemoteStrategy 0xAA8af8Db4B6a827B51786334d26349eb03569731
#Diff source code with the previous source
sol2uml diff 0x5f81a5a22375ebcc2075b162d23a5b16a1e7a92d 0xAA8af8Db4B6a827B51786334d26349eb03569731  --network base --apiKey [API_KEY]

#Verify source code 
sol2uml diff 0xAA8af8Db4B6a827B51786334d26349eb03569731 .,node_modules --network base --apiKey [API_KEY]

Deploy checklist

Two reviewers complete the following checklist:

- [ ] All deployed contracts are listed in the deploy PR's description
- [ ] Deployed contract's verified code (and all dependencies) match the code in master
- [ ] Contract constructors have correct arguments
- [ ] The transactions that interacted with the newly deployed contract match the deploy script.
- [ ] Governance proposal matches the deploy script
- [ ] Smoke tests pass after fork test execution of the governance proposal

Copy link
Collaborator

@shahthepro shahthepro left a comment

Choose a reason for hiding this comment

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

Compared the "CrossChainRemoteStrategy" contract with address 0xAA8af8Db4B6a827B51786334d26349eb03569731 on base
to local files under folders ".,node_modules"

match   @openzeppelin/contracts/token/ERC20/extensions/IERC20Metadata.sol
match   @openzeppelin/contracts/token/ERC20/IERC20.sol
match   @openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol
match   @openzeppelin/contracts/utils/Address.sol
match   @openzeppelin/contracts/utils/math/Math.sol
match   @openzeppelin/contracts/utils/math/SafeCast.sol
match   contracts/governance/Governable.sol
match   contracts/governance/Strategizable.sol
match   contracts/interfaces/cctp/ICCTP.sol
match   contracts/interfaces/IBasicToken.sol
match   contracts/interfaces/IMerkl.sol
match   contracts/interfaces/IStrategy.sol
match   contracts/interfaces/IVault.sol
match   contracts/interfaces/morpho/IMorphoV2Adapter.sol
match   contracts/interfaces/morpho/IVaultV2.sol
match   contracts/strategies/crosschain/AbstractCCTPIntegrator.sol
match   contracts/strategies/crosschain/CrossChainRemoteStrategy.sol
match   contracts/strategies/crosschain/CrossChainStrategyHelper.sol
match   contracts/strategies/Generalized4626Strategy.sol
match   contracts/strategies/MorphoV2VaultUtils.sol
match   contracts/token/OUSD.sol
match   contracts/utils/BytesHelper.sol
match   contracts/utils/Helpers.sol
match   contracts/utils/Initializable.sol
match   contracts/utils/InitializableAbstractStrategy.sol
match   contracts/vault/VaultStorage.sol
match   lib/openzeppelin/interfaces/IERC4626.sol

shahthepro
shahthepro previously approved these changes Mar 6, 2026
Copy link
Collaborator

@shahthepro shahthepro left a comment

Choose a reason for hiding this comment

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

  • All deployed contracts are listed in the deploy PR's description
  • Deployed contract's verified code (and all dependencies) match the code in master
  • Contract constructors have correct arguments
  • The transactions that interacted with the newly deployed contract match the deploy script.
  • Governance proposal matches the deploy script
  • Smoke tests pass after fork test execution of the governance proposal

Copy link
Collaborator

@clement-ux clement-ux left a comment

Choose a reason for hiding this comment

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

Compared the "CrossChainRemoteStrategy" contract with address 0xAA8af8Db4B6a827B51786334d26349eb03569731 on base
to local files under folders ".,node_modules"

match @openzeppelin/contracts/token/ERC20/extensions/IERC20Metadata.sol
match @openzeppelin/contracts/token/ERC20/IERC20.sol
match @openzeppelin/contracts/token/ERC20/utils/SafeERC20.sol
match @openzeppelin/contracts/utils/Address.sol
match @openzeppelin/contracts/utils/math/Math.sol
match @openzeppelin/contracts/utils/math/SafeCast.sol
match contracts/governance/Governable.sol
match contracts/governance/Strategizable.sol
match contracts/interfaces/cctp/ICCTP.sol
match contracts/interfaces/IBasicToken.sol
match contracts/interfaces/IMerkl.sol
match contracts/interfaces/IStrategy.sol
match contracts/interfaces/IVault.sol
match contracts/interfaces/morpho/IMorphoV2Adapter.sol
match contracts/interfaces/morpho/IVaultV2.sol
match contracts/strategies/crosschain/AbstractCCTPIntegrator.sol
match contracts/strategies/crosschain/CrossChainRemoteStrategy.sol
match contracts/strategies/crosschain/CrossChainStrategyHelper.sol
match contracts/strategies/Generalized4626Strategy.sol
match contracts/strategies/MorphoV2VaultUtils.sol
match contracts/token/OUSD.sol
match contracts/utils/BytesHelper.sol
match contracts/utils/Helpers.sol
match contracts/utils/Initializable.sol
match contracts/utils/InitializableAbstractStrategy.sol
match contracts/vault/VaultStorage.sol
match lib/openzeppelin/interfaces/IERC4626.sol

Copy link
Collaborator

@clement-ux clement-ux left a comment

Choose a reason for hiding this comment

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

  • All deployed contracts are listed in the deploy PR's description
  • Deployed contract's verified code (and all dependencies) match the code in master
  • Contract constructors have correct arguments
  • The transactions that interacted with the newly deployed contract match the deploy script.
  • Governance proposal matches the deploy script
  • Smoke tests pass after fork test execution of the governance proposal

@codecov
Copy link

codecov bot commented Mar 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 41.10%. Comparing base (bbb45b6) to head (01dcf72).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2830   +/-   ##
=======================================
  Coverage   41.10%   41.10%           
=======================================
  Files         110      110           
  Lines        4634     4634           
  Branches     1263     1263           
=======================================
  Hits         1905     1905           
  Misses       2726     2726           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sparrowDom sparrowDom merged commit 5244488 into master Mar 9, 2026
10 of 17 checks passed
@sparrowDom sparrowDom deleted the sparrowDom/deploy_base_045 branch March 9, 2026 09:56
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.

3 participants