Skip to content

Comments

Add Cannon#69

Open
kaze-cow wants to merge 8 commits intomainfrom
cannon
Open

Add Cannon#69
kaze-cow wants to merge 8 commits intomainfrom
cannon

Conversation

@kaze-cow
Copy link

@kaze-cow kaze-cow commented Feb 23, 2026

This contract is particularly challenging because it has . I wanted to replicate the checks that we do in the script (ex. checking that wrappedNativeToken is actually a WETH contract or something that could concievably be one), but this has turned out to be challenging with the current limitations.

Out of Scope

The CI will come later.

Native Token verification in the Cannonfile turned out to be quite challenging as things are, so saving this for a future PR as we approach more networks.

How to Test

To build the cannon package, it currently cowprotocol/contracts#257, so go over there and follow the build instructions for the cannon package there first. (once the package is published there, it will no longer be necessary to build separately)

Then build:

yarn install
yarn build # runs cannon build
yarn cannon:record

You may need to create the cannon/ directory for the last command to succeed.

Also check the docs, see if they are reasonable 👍

wrapped native token is set from a dummy token and the cowi mport to ensure that the contract can be deployed on local test network
@kaze-cow kaze-cow self-assigned this Feb 23, 2026
@github-actions
Copy link

github-actions bot commented Feb 23, 2026

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@kaze-cow
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Feb 23, 2026
@kaze-cow kaze-cow marked this pull request as ready for review February 23, 2026 08:41
@kaze-cow kaze-cow requested a review from a team as a code owner February 23, 2026 08:41
@socket-security
Copy link

socket-security bot commented Feb 23, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​usecannon/​cli@​2.26.0-alpha.081100859370

View full report

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.

1 participant