Open
Conversation
clement-ux
requested review from
naddison36,
shahthepro and
sparrowDom
as code owners
Fund account with enough ETH to cover the WETH deposit amount instead of the default 100 ETH.
shahthepro
reviewed
Feb 25, 2026
Collaborator
shahthepro
left a comment
shahthepro
left a comment
There was a problem hiding this comment.
> sol2uml diff 0xAe9A68E82e76caa35F8ac3f4e494cdb7438BBac1 .,node_modules --apiKey $ETHERSCAN_API_KEY --network base
Compared the "OETHBaseVault" contract with address 0xAe9A68E82e76caa35F8ac3f4e494cdb7438BBac1 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/SafeCast.sol
match @openzeppelin/contracts/utils/math/SafeMath.sol
match contracts/governance/Governable.sol
match contracts/interfaces/IBasicToken.sol
match contracts/interfaces/IStrategy.sol
match contracts/interfaces/IVault.sol
match contracts/token/OUSD.sol
match contracts/utils/Helpers.sol
match contracts/utils/Initializable.sol
match contracts/utils/StableMath.sol
match contracts/vault/OETHBaseVault.sol
match contracts/vault/VaultAdmin.sol
match contracts/vault/VaultCore.sol
match contracts/vault/VaultInitializer.sol
match contracts/vault/VaultStorage.sol
shahthepro
reviewed
Feb 25, 2026
Collaborator
shahthepro
left a comment
shahthepro
left a comment
There was a problem hiding this comment.
> sol2uml diff 0x0929C0fbFF88e129ACaA51Bba0C959491325b4aD .,node_modules --apiKey $ET
HERSCAN_API_KEY --network base
Compared the "BridgedWOETHStrategy" contract with address 0x0929C0fbFF88e129ACaA51Bba0C959491325b4aD 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/SafeCast.sol
match @openzeppelin/contracts/utils/math/SafeMath.sol
match contracts/governance/Governable.sol
match contracts/interfaces/chainlink/AggregatorV3Interface.sol
match contracts/interfaces/IBasicToken.sol
match contracts/interfaces/IOracle.sol
match contracts/interfaces/IStrategy.sol
match contracts/interfaces/IVault.sol
match contracts/interfaces/IWETH9.sol
match contracts/strategies/BridgedWOETHStrategy.sol
match contracts/token/OUSD.sol
match contracts/utils/Helpers.sol
match contracts/utils/Initializable.sol
match contracts/utils/InitializableAbstractStrategy.sol
match contracts/utils/StableMath.sol
match contracts/vault/VaultStorage.sol
> sol2uml diff 0xe3B3b4Fc77505EcfAACf6dD21619a8Cc12fcc501 .,node_modules --apiKey $ET
HERSCAN_API_KEY --network base
Compared the "BaseBridgeHelperModule" contract with address 0xe3B3b4Fc77505EcfAACf6dD21619a8Cc12fcc501 on base
to local files under folders ".,node_modules"
match @chainlink/contracts-ccip/src/v0.8/ccip/interfaces/IRouterClient.sol
match @chainlink/contracts-ccip/src/v0.8/ccip/libraries/Client.sol
match @openzeppelin/contracts/access/AccessControl.sol
match @openzeppelin/contracts/access/AccessControlEnumerable.sol
match @openzeppelin/contracts/access/IAccessControl.sol
match @openzeppelin/contracts/access/IAccessControlEnumerable.sol
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/Context.sol
match @openzeppelin/contracts/utils/introspection/ERC165.sol
match @openzeppelin/contracts/utils/introspection/IERC165.sol
match @openzeppelin/contracts/utils/math/SafeCast.sol
match @openzeppelin/contracts/utils/math/SafeMath.sol
match @openzeppelin/contracts/utils/Strings.sol
match @openzeppelin/contracts/utils/structs/EnumerableSet.sol
match contracts/automation/AbstractCCIPBridgeHelperModule.sol
match contracts/automation/AbstractSafeModule.sol
match contracts/automation/BaseBridgeHelperModule.sol
match contracts/governance/Governable.sol
match contracts/interfaces/chainlink/AggregatorV3Interface.sol
match contracts/interfaces/IBasicToken.sol
match contracts/interfaces/IOracle.sol
match contracts/interfaces/ISafe.sol
match contracts/interfaces/IStrategy.sol
match contracts/interfaces/IVault.sol
match contracts/interfaces/IWETH9.sol
match contracts/strategies/BridgedWOETHStrategy.sol
match contracts/token/OUSD.sol
match contracts/utils/Helpers.sol
match contracts/utils/Initializable.sol
match contracts/utils/InitializableAbstractStrategy.sol
match contracts/utils/StableMath.sol
match contracts/vault/VaultStorage.sol
match lib/openzeppelin/interfaces/IERC4626.sol
shahthepro
approved these changes
Feb 25, 2026
Collaborator
shahthepro
left a comment
shahthepro
left a comment
There was a problem hiding this comment.
- 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
Collaborator
|
naddison36
approved these changes
Feb 25, 2026
Collaborator
naddison36
left a comment
naddison36
left a comment
There was a problem hiding this comment.
I've verified the deployment
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
3 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.
Deployments
deploy scripts
Governance (Base)
Deploy checklist
Two reviewers complete the following checklist: