FEAT: support vyper contract verification #10864
Conversation
(cherry picked from commit e41786b)
crates/verify/src/verify.rs
Outdated
| @@ -410,6 +421,20 @@ impl VerifyArgs { | |||
| } | |||
| } | |||
|
|
|||
| pub fn detect_language(args: &VerifyArgs, ctx: &VerificationContext) -> ContractLanguage { | |||
There was a problem hiding this comment.
can we make this a function of VerifyArgs instead?
Another-DevX
requested review from
DaniPopes,
klkvr,
grandizzy,
yash-atreya and
zerosnacks
as code owners
Another-DevX
changed the title
crates/verify/src/verify.rs
Outdated
| #[derive(Copy, Clone, Debug, Eq, PartialEq, ValueEnum)] | ||
| pub enum ContractLanguage { | ||
| Solidity, | ||
| Vyper, | ||
| } |
Another-DevX
force-pushed
the
feature/vyper-support
branch
from
93a3dfe to
e049932
Compare
|
Btw @mattsse, we should probably open an issue on the foundry-docs to properly mention this new feature, right? |
|
hello |
There was a problem hiding this comment.
cool, last pedantic nits
and would like another review by @grandizzy or @zerosnacks
| } | ||
| ContractLanguage::Vyper => { | ||
| // Execute vyper -f solc_json {target_path} | ||
| let output = Command::new("vyper") |
There was a problem hiding this comment.
I guess that's ok for now but wonder if we should upstream a vyper.solc_json fn to compilers as it is basically doing similar as here https://github.com/foundry-rs/compilers/blob/5846d3ca5181ecf4515997eecdeac52fdda22d29/crates/compilers/src/compilers/vyper/mod.rs#L118
There was a problem hiding this comment.
Mmm, let me try to use the functions already existing on compilers or to create a new one for the solc_json generation
Closes: #10702
This PR enables verification of Vyper contracts via
forge verify-contract.✅ Summary of the approach
--languagethat allows selecting betweensolidityandvyper.solc_json(standard input JSON) through a small bypass, avoiding the need to route throughfoundry-compilers..vy, the appropriate arguments expected by Etherscan for Vyper are applied.🔧 Dependencies
This implementation depends on:
foundry-rs/block-explorers#91
Please make sure to merge and update that crate before merging this PR.
🧪 Test
Verified on Sepolia:
https://sepolia.etherscan.io/address/0x538d309a74c7201803db2eff456d7ebba56022b2#code