Skip to content

Add automatic AztecScan contract verification after devnet deployment#243

Open
FilipHarald wants to merge 1 commit intoAztecProtocol:nextfrom
aztec-scan:next
Open

Add automatic AztecScan contract verification after devnet deployment#243
FilipHarald wants to merge 1 commit intoAztecProtocol:nextfrom
aztec-scan:next

Conversation

@FilipHarald
Copy link

  • Add verify_on_aztecscan.ts with artifact and instance verification using raw fetch
  • Update deploy_contract.ts to verify PodRacing on AztecScan after deploy (best-effort)
  • Use raw Noir compiler JSON artifact for verification (not codegen'd ContractArtifact)
  • Add AztecScan config to devnet.json and config.ts
  • Fix proverEnabled in setup_wallet.ts (required for devnet, disabled for local)
  • Fix hardcoded 120s deploy timeout in deploy_account.ts to use config-based timeouts

Tested end-to-end on devnet: artifact verification 201, instance verification 200.

- Add verify_on_aztecscan.ts with artifact and instance verification using raw fetch
- Update deploy_contract.ts to verify PodRacing on AztecScan after deploy (best-effort)
- Use raw Noir compiler JSON artifact for verification (not codegen'd ContractArtifact)
- Add AztecScan config to devnet.json and config.ts
- Fix proverEnabled in setup_wallet.ts (required for devnet, disabled for local)
- Fix hardcoded 120s deploy timeout in deploy_account.ts to use config-based timeouts

Tested end-to-end on devnet: artifact verification 201, instance verification 200.
@FilipHarald
Copy link
Author

Hey @critesjosh

I don't know if you would like to keep this repo to a minimal, or more like a general tool. I made it work with our latest release on devnet at least. Copying most of what I did in https://github.com/aztec-scan/aztec-scan-sdk

Copy link
Collaborator

@critesjosh critesjosh left a comment

Choose a reason for hiding this comment

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

I like this idea. Can you add to the README additional instructions about what verifying on aztecscan is and why do it?

},
"aztecscan": {
"apiUrl": "https://api.devnet.aztecscan.xyz",
"apiKey": "temporary-api-key"
Copy link
Collaborator

Choose a reason for hiding this comment

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

im assuming this key is retrieved from your site?

Copy link
Author

Choose a reason for hiding this comment

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

Well, yes, but we don't have signup yet. So this will work for all users for now.

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