-
Notifications
You must be signed in to change notification settings - Fork 396
feat: mina verifiers #2163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
feat: mina verifiers #2163
Changes from 250 commits
Commits
Show all changes
344 commits
Select commit
Hold shift + click to select a range
524e0fa
Use Mainnet and adapt to new state size
gabrielbosio fed2ad4
Update test files and remove bad consensus test
gabrielbosio feeb6aa
Add state size check in batcher
gabrielbosio cdb2438
Merge pull request #7 from lambdaclass/consensus
xqft a00d8c6
Read state size in batcher
gabrielbosio 703fe27
Make operator read state size in pub input
gabrielbosio be1f6b9
Update Mina test make cmds
gabrielbosio 49e60b1
Update operator tests
gabrielbosio e0e42d2
Merge pull request #16 from lambdaclass/state_size_in_pub_input
gabrielbosio c99e66c
Change order of pub inputs, simplify fns and names
xqft 5c11fa3
Update test files
xqft 1e86a2a
Update batcher pre-verification
xqft 3ca8bc8
Change Holesky Aligned contract address
gabrielbosio b602274
Log batcher gas cost
gabrielbosio 8873369
Log aggregator gas cost
gabrielbosio e4bc95d
Merge branch 'main' into mina
gabrielbosio 1882cb3
Merge branch 'mina' into mina_holesky
gabrielbosio db616f7
Merge pull request #17 from lambdaclass/tip_hash
gabrielbosio 5f3540f
Merge branch 'mina' into mina_holesky
gabrielbosio a47350d
Update Holesky contract
gabrielbosio 12ac9aa
Add Mina build step
gabrielbosio b75e1e2
Measure block verification time
gabrielbosio ec48580
Add Holesky Aligned SM contract addr disclaimer
gabrielbosio 08a220c
Merge pull request #18 from lambdaclass/mina_holesky
gabrielbosio 19d2783
Copy mina operator
xqft 6a0ca58
Mockup account verifier
xqft 7e63c68
Update gitignore
xqft 17c9f0c
Added new proving system to batcher
xqft 4160da8
Fix Makefile
xqft a598c68
Deserialize merkle proof, added verify_merkle_proof()
xqft 66fc5d3
Remove proof slicing
xqft 43522ae
Fix Mina verifier linker args in Linux
gabrielbosio a90a4ca
Update go modules
xqft 958cf2e
Change yetanotherco modules to lambdaclass
xqft b5526cb
Revert "Change yetanotherco modules to lambdaclass"
xqft 59ee79a
Fix go modules
xqft d2af2d6
Change go module name
xqft 5d0618c
Change libmerkle_tree path
xqft 2b353e2
Update lib merkle tree path
xqft b0c70c2
Update go mod replace
xqft a9ab0f2
Fix case
xqft 43da621
Revert "Update lib merkle tree path"
xqft 9597aca
Revert "Change libmerkle_tree path"
xqft a9577df
Add mina_account operator import, fix test module name
xqft 3b36365
Temporal fix for internal package import
xqft b447638
Fix merkle proof deserialization
xqft ce8a206
Added clarifying comment
xqft 38755a2
Added candidate ledger hash to state proof pub inputs
xqft 3f7dbd1
Fix proof prechecks
xqft a9fa34e
Add support for account_id_hash
xqft 141356b
Added verify merkle test
xqft 1e6a457
Fix go binding test
xqft b0b6ce7
Remove unused test module
xqft 2ca07ab
Fix CI
xqft 5163c73
Merge pull request #20 from lambdaclass/mina_linux_ld_args
gabrielbosio c3c7b84
Update Makefile
xqft 4bd6140
Update operator/mina_account/lib/src/lib.rs
xqft 06bc10c
Update .env.dev
xqft ce5f842
Merge pull request #19 from lambdaclass/account_inclusion_verifier
gabrielbosio 2c1cd40
Fix Mina account Linux linker directive
gabrielbosio 5242815
Adapt to new Mina Proof of State
xqft 43c1f64
Added merkle list verification
xqft 4e127d2
Update comment:
xqft 893bb71
Update batcher pre-verification checks
xqft dbcb0d0
Fix operator for new Mina proof
xqft 0afb987
Fix comment
xqft 82e32bb
Add long range fork rule
xqft 3bb8196
Fix clippy
xqft 7d89a32
Better error msg
xqft a85d7f9
Merge pull request #22 from lambdaclass/relative_finalization
xqft 6554ca0
Fixed mina and mina_account tests and moved test files
xqft f00c05c
Merge branch 'mina' into fix_mina_account_ld
gabrielbosio 9378fc2
Merge tag 'v0.5.2' into update_to_0_5
gabrielbosio edc8441
Merge branch 'long_range_consensus' into update_to_0_5
gabrielbosio a0c893e
Change staged ledger hash with snarked ledger hash
gabrielbosio c43d80f
Merge tag 'v0.6.0' into update_to_0_6
gabrielbosio 62df493
Log batch inclusion result
gabrielbosio 741a15f
Make Anvil start in Aligned 'hard fork'
gabrielbosio 710a9e8
Temporarily fix batch indentifier
gabrielbosio 3c75d7f
Handle new account proof
xqft 36db55f
Add new account test files
xqft b498ed8
Fix Makefile
xqft 8fdcab7
Add mina account send task rule
xqft 17ba105
Use new proof
xqft 912387a
Fix leaf_hash
xqft 4fcc72e
Remove debug log
xqft 44c32f6
Update SM bindings
gabrielbosio 57e92b0
Add test files
xqft 7ec2b4c
Update cargo lock
xqft 6fe90f6
Add new mina account proof
xqft e901104
Handle new account proof
xqft 09ace35
Fix mina state proof test files'
xqft c65702a
Revert change of eigenlayer submodule
xqft fad946f
Temporarily fix batch id bug in devnet and testnet
gabrielbosio f815d62
Update contract bindings
gabrielbosio 59eb621
Merge branch 'new_graphql_account' into solidity_account
xqft abe4672
Merge pull request #32 from lambdaclass/new_graphql_account
gabrielbosio e2dd410
Update Cargo.toml
xqft e1f956f
Merge pull request #33 from lambdaclass/solidity_account
xqft 590dd19
Merge branch 'mina' into update_to_0_5
gabrielbosio d247f1b
Merge branch 'update_to_0_5' into update_to_0_6
gabrielbosio 8f1f4bd
Remove result log
gabrielbosio 9c73929
Lock Mina account packages
gabrielbosio c866c04
Fix new task test
gabrielbosio a157f80
Add comment about temporary solution
gabrielbosio d5b0885
Merge pull request #30 from lambdaclass/update_to_0_6
gabrielbosio 7a2b555
Merge pull request #31 from lambdaclass/update_to_0_5
gabrielbosio 1d5265b
Add tests for empty proof and empty pub inputs
gabrielbosio 824a10b
Add tests for empty inputs in Mina verifier
gabrielbosio caf7e32
Update Mina test files
gabrielbosio 664a813
Test empty inputs in Mina FFI
gabrielbosio a67bb78
Make unchanged mut vars immutable
gabrielbosio 6b14da0
Update Mina account test files
gabrielbosio a979e1f
Add Go tests for empty inputs in Mina account FFI
gabrielbosio ad9aa08
Test empty inputs in Mina Account Rust FFI
gabrielbosio 1c85ce2
Add checks for huge input sizes
gabrielbosio e46b6f5
Refactor input size check
gabrielbosio 05eccaf
Merge pull request #35 from lambdaclass/input_checks
gabrielbosio 453d9b4
v0.7.0 (#980)
MauroToscano 7853f73
Update Aligned to 0.7.0
gabrielbosio 2c60d62
Fix SM test
gabrielbosio 2fe0dac
Update Holesky SM and batcher address
gabrielbosio 752e7fd
Add consensus state input checks
gabrielbosio 2adffd2
Comment input checks
gabrielbosio af63afb
Remove print statement
gabrielbosio a649f09
Fix typo in 'long'
gabrielbosio 2b00561
Revert verifier index change
gabrielbosio 50ed685
Add pre-verification of Mina state
gabrielbosio ec5800e
Add pre-verification of Mina account inclusion
gabrielbosio 1501e75
Fix Go workflow
gabrielbosio 94eaaff
Add feature flag for Mina network
gabrielbosio c732554
Compute SRS statically
gabrielbosio 9bac7bc
Update addresses
xqft 6d5e724
Parameterize Batcher and SM addresses
gabrielbosio ee913d0
Merge tag 'v0.7.2' into update_to_0_7_2
gabrielbosio 88de323
Change runner in Go and Rust workflows
gabrielbosio e170af3
Add HOME env var in Go and Rust workflows
gabrielbosio 97dbced
Add Rust installation step in Go workflow
gabrielbosio 26497cf
Fix proving system match
gabrielbosio 1f69ee8
Add Rust caching in Go workflow
gabrielbosio c06c37b
Merge pull request #37 from lambdaclass/update_to_0_7_for_real
gabrielbosio 903c90b
Merge pull request #38 from lambdaclass/consensus_state_input_checks
gabrielbosio a18cc0a
Merge branch 'mina' into static_srs
gabrielbosio 2a68b77
Merge pull request #41 from lambdaclass/static_srs
gabrielbosio 40feee0
Merge branch 'mina' into parameterize_sm_and_batcher
gabrielbosio 8d28206
Merge pull request #42 from lambdaclass/parameterize_sm_and_batcher
gabrielbosio 7dc08d4
Merge branch 'mina' into parameterize_network
gabrielbosio 6f61538
Merge pull request #40 from lambdaclass/parameterize_network
gabrielbosio 7d03160
Merge branch 'mina' into pre_verification
gabrielbosio 3426bb4
Merge pull request #39 from lambdaclass/pre_verification
gabrielbosio bfe122c
Merge branch 'mina' into update_to_0_7_2
gabrielbosio 7c903cb
Compile both Verification keys
gabrielbosio 586f0c4
Clarify BATCHER_PAYMENT_SERVICE_ADDR error
gabrielbosio 56ac3a2
Read BATCHER_ETH_ADDR in SDK
gabrielbosio 7744149
Merge pull request #44 from lambdaclass/use_two_vks
gabrielbosio 06d4b4e
Point bridge repo to aligned branch
gabrielbosio 9444937
Merge pull request #45 from lambdaclass/point_bridge_to_aligned
gabrielbosio ecc2fa0
Fix Mina test files
gabrielbosio db0b1ca
Fix proving system used to test Mina Account verifier
gabrielbosio d177042
Update Mina Account test files
gabrielbosio 3a4ed6e
Merge pull request #46 from lambdaclass/fix_test_proof_files
gabrielbosio 916a171
Merge branch 'mina' into update_to_0_7_2
gabrielbosio b68aba7
Merge tag 'v0.8.0' into update_to_0_8_0
gabrielbosio f6fe98c
Add Rust setup step
gabrielbosio a9e2cbb
Make test job use custom runner
gabrielbosio 9992a53
Merge pull request #43 from lambdaclass/update_to_0_7_2
gabrielbosio e47fa65
Merge pull request #47 from lambdaclass/update_to_0_8_0
gabrielbosio 16e80cf
Merge remote-tracking branch 'upstream/staging' into mina
gabrielbosio 78408e5
Fix runner
gabrielbosio 2fac66c
Remove Rust setup steps in Go workflow
gabrielbosio 456c712
Remove HOME env var in Go workflow
gabrielbosio 102c0c2
Change to Aligned runner in Go workflow
gabrielbosio f4b2299
Change to Aligned runner in gas diff workflow
gabrielbosio 0824e2a
Revert "Change to Aligned runner in gas diff workflow"
gabrielbosio ffb0bae
Remove info log on is_proof_verified error
gabrielbosio 0e1820b
Revert SM contract format
gabrielbosio e17b371
Revert Batcher contract format
gabrielbosio 8725233
Revert SM test changes and format
gabrielbosio 5ebc68a
Revert contract bindings
gabrielbosio 7ab4b9b
Revert Anvil initial state
gabrielbosio 456cf32
Revert changed docs
gabrielbosio 56bd1a8
Revert Go module replacements
gabrielbosio eef7115
Revert indentation in ethers dependency
gabrielbosio 0f21213
Add Mina task send docs
gabrielbosio 56c8a76
Normalize operator ignore files
gabrielbosio d93a343
Remove unused account test files
gabrielbosio 80b76dc
Fix verifier index test
gabrielbosio 9abb4dd
Add Mina test workflow
gabrielbosio fc8f1d4
Rename generated libs
gabrielbosio d4b1559
Fix paths in Mina workflow trigger
gabrielbosio 454a33c
Merge branch 'staging' into mina
gabrielbosio 4115688
Add Mina account and Mina ld lib path
gabrielbosio c62c5e6
Repatch Arkworks
gabrielbosio 032bec8
Use LC's fork of proof systems
gabrielbosio 93428c1
Ignore correct Mina Account FFI generated lib
gabrielbosio 52116ae
Update deps
xqft d7bc504
Merge pull request #48 from lambdaclass/update_batcher_deps
gabrielbosio 239cae0
Merge branch 'staging' into mina
gabrielbosio 1b8e225
Format Aligned batcher lib file
gabrielbosio e6b30ed
Fix tests
gabrielbosio 319d3b6
Merge branch 'staging' into mina
gabrielbosio 19f4b50
Add mina verifiers to operator dockerfile path
gabrielbosio 78b5461
Add Mina and Mina Account build steps for Docker
gabrielbosio e2fc65b
Ignore Mina account verifier binary
gabrielbosio c3365b0
Remove unused Mina module
gabrielbosio 2326f5f
Add Mina verifier copy cmd in Batcher Dockerfile
gabrielbosio e3b6559
Move Mina verifier copy cmds
gabrielbosio 458a55e
Use commit hashes of Mina Bridge dep instead of branches
gabrielbosio 99904e0
Check result of computing Merkle Root of Mina Account
gabrielbosio 614012d
Add Mina and Mina Account steps to send all cmd
gabrielbosio 59b1fe0
Merge pull request #51 from lambdaclass/add_mina_proofs_to_send_all_cmd
gabrielbosio 2ed9faa
Handle Mina pub input errors
gabrielbosio 0cf098b
Improve verification key error handling
gabrielbosio 5adec30
Replace eprintln calls with logger
gabrielbosio 7e0b2ff
Merge pull request #50 from lambdaclass/check_merkle_root_result
gabrielbosio b8edf9e
Merge pull request #52 from lambdaclass/handle_verify_internal_errors
gabrielbosio 555e28c
Merge pull request #53 from lambdaclass/improve_vk_error_handling
gabrielbosio cebeee7
Merge pull request #54 from lambdaclass/replace_eprintln_calls
gabrielbosio e881bc4
Use rev instead of branch for OM and Bridge deps
gabrielbosio 220546a
Merge pull request #55 from lambdaclass/use_rev_instead_of_branch
gabrielbosio 279aa75
Improve Mina verification FFI
gabrielbosio 4fe4a2f
Fix Mina FFI tests
gabrielbosio c7eb450
Fix Mina account FFI
gabrielbosio c26ad54
Merge pull request #56 from lambdaclass/improve_ffi_arg_types
gabrielbosio ccbe588
Fix batcher clippy issues
gabrielbosio 836ce51
Remove pub input len log
gabrielbosio 27c2f85
Test null ptr cases in Mina verifier
gabrielbosio e973022
Add empty test case for Mina FFI
gabrielbosio 260c051
Add Mainnet case for Mina FFI
gabrielbosio 761d383
Add tests for Mina account FFI
gabrielbosio df5c035
Fix tests
gabrielbosio 8755559
Merge pull request #57 from lambdaclass/add_tests
gabrielbosio a82fd67
Document Mina verifier
gabrielbosio 10fe27c
Document Mina Account verifier
gabrielbosio 83f0b1b
Merge pull request #58 from lambdaclass/improve_mina_docs
gabrielbosio 10bd8c6
Merge branch 'staging' into mina
gabrielbosio 21f45f8
v0.20.0 (#2159)
JuArce a0ebc25
Merge remote-tracking branch 'aligned/testnet' into mina
MarcosNicolau 6d8a3a2
fix: makefile targets
MarcosNicolau 477736a
merge error in verfier
MarcosNicolau 1aa74de
fix compilation failure due to dependency versions mistmatch by bumpi…
MarcosNicolau c38c31f
fix go lint
MarcosNicolau 6386260
Update Makefile
MarcosNicolau a54c2de
revert build and test go runner
MarcosNicolau f2dfdf6
copy mina folder in batcher docker
MarcosNicolau ccfc8be
move mina docker targets to the bottom
MarcosNicolau 11ca8d0
apply patches to Batcher cargo.toml
MarcosNicolau 6bf89dd
fix ci stuck because of low max fee in mina account docker task
MarcosNicolau 3d3036b
fix docker sent proofs in makefile
MarcosNicolau 8d00024
Update docs/3_guides/6_setup_aligned.md
MauroToscano File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| name: test-mina | ||
|
|
||
| on: | ||
| merge_group: | ||
| push: | ||
| branches: [main] | ||
| pull_request: | ||
| branches: ["*"] | ||
| paths: | ||
| - 'operator/mina/**' | ||
| - 'operator/mina_account/**' | ||
| - '.github/workflows/test-mina.yml' | ||
|
|
||
| jobs: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: actions/setup-go@v5 | ||
| with: | ||
| go-version: '1.22' | ||
| cache: false | ||
| - uses: actions-rs/toolchain@v1 | ||
| with: | ||
| toolchain: stable | ||
| - name: Test Mina Rust | ||
| run: make test_mina_rust_ffi | ||
| - name: Test Mina go bindings | ||
| run: make test_mina_go_bindings_linux | ||
| - name: Test Mina Account Rust | ||
| run: make test_mina_account_rust_ffi | ||
| - name: Test Mina Account go bindings | ||
| run: make test_mina_account_go_bindings_linux |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.