Skip to content

Conversation

@JuArce
Copy link
Collaborator

@JuArce JuArce commented Jun 26, 2025

Migrate go-circom-prover-verifier to go-rapidsnark

Description

go-circom-prover-verifier is deprecated in favor of go-rapidsnark https://github.com/iden3/go-rapidsnark

How to Test

  1. Start Ethereum Package
make ethereum_package_start
  1. Start batcher
cargo clean --manifest-path ./crates/batcher/Cargo.toml
make batcher_start_ethereum_package
  1. Start aggregator
make aggregator_start_ethereum_package ENVIRONMENT=devnet
  1. Start operator

Note: You need to rebuild ffis

make build_all_ffi
make operator_full_registration CONFIG_FILE=config-files/config-operator-1-ethereum-package.yaml ENVIRONMENT=devnet
make operator_start CONFIG_FILE=config-files/config-operator-1-ethereum-package.yaml ENVIRONMENT=devnet
  1. Start explorer
make explorer_clean_db
make explorer_start
  1. Send proofs
make aligned_install_compiling
make batcher_send_circom_groth16_bn256_task
make batcher_send_circom_groth16_bn256_burst
  1. Check the proofs are verified in the explorer (http://localhost:4000/)

Note: Examples in docs won't work in Holesky until the deploy is done.

Type of change

  • Refactor

Checklist

  • “Hotfix” to testnet, everything else to staging
  • Linked to Github Issue
  • This change depends on code or research by an external entity
    • Acknowledgements were updated to give credit
  • Unit tests added
  • This change requires new documentation.
    • Documentation has been added/updated.
  • This change is an Optimization
    • Benchmarks added/run
  • Has a known issue
  • If your PR changes the Operator compatibility (Ex: Upgrade prover versions)
    • This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
    • This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

@JuArce JuArce self-assigned this Jun 26, 2025
JuArce added 3 commits June 26, 2025 15:07
…m-prover-verifier-to-go-rapidsnark

# Conflicts:
#	Makefile
#	crates/batcher/src/zk_utils/mod.rs
@MarcosNicolau MarcosNicolau added this pull request to the merge queue Jun 26, 2025
Merged via the queue into staging with commit b1a1ba6 Jun 26, 2025
5 of 7 checks passed
@MarcosNicolau MarcosNicolau deleted the 2001-refactor-migrate-go-circom-prover-verifier-to-go-rapidsnark branch June 26, 2025 20:01
@JuArce JuArce linked an issue Jul 15, 2025 that may be closed by this pull request
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.

refactor: migrate go-circom-prover-verifier to go-rapidsnark

4 participants