Skip to content

Comments

HumpyComp wrapper#1

Open
matejfalat wants to merge 3 commits intomainfrom
add-humpy-comp
Open

HumpyComp wrapper#1
matejfalat wants to merge 3 commits intomainfrom
add-humpy-comp

Conversation

@matejfalat
Copy link
Collaborator

@matejfalat matejfalat commented Feb 20, 2026

Related https://github.com/api3dao/oev-searcher/issues/3032

Moved from api3dao/contracts#741, updated the deployment script to use hardhat-deploy tags.

Copy link

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

👍 LGTM.

Ugur told me we should not call it HumpyCOMP, so I'll reach out why.

Copy link

@Siegrift Siegrift left a comment

Choose a reason for hiding this comment

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

Let's rename the contract.

/// @param initialOwner Address of the initial owner
/// @param initialDelegatee Address of the initial delegatee for COMP voting power
constructor(
address initialOwner,

Choose a reason for hiding this comment

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

Also, we'll likely set the owner to the address that Humpy (the borrower) will tell us. Better for us not to manage anything in this contract.

/// @title Humpy COMP
/// @author API3 DAO
/// @notice 1:1 wrapper token for COMP built on OpenZeppelin ERC20Wrapper
contract HumpyComp is ERC20Wrapper, Ownable2Step {

Choose a reason for hiding this comment

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

Let's call it dComp instead (name and symbol can be the same)

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.

2 participants