workspace,starknet_proof_verifier: bump privacy-{prove,circuit-verify} to 039e52c, add v0/v1 aliases#14017
Conversation
PR SummaryMedium Risk Overview Introduces explicit verifier aliases Reviewed by Cursor Bugbot for commit aff094e. Bugbot is set up for automated code reviews on this repo. Configure here. |
d445589 to
845affe
Compare
6ac567b to
963202d
Compare
845affe to
bd31b43
Compare
1a88611 to
ef43d9f
Compare
b4fb627 to
afe1573
Compare
ef43d9f to
67dfc82
Compare
afe1573 to
f09af06
Compare
67dfc82 to
430f38c
Compare
f09af06 to
dc465e6
Compare
42bbbd4 to
c5110f2
Compare
a274215 to
85967fb
Compare
c5110f2 to
6e47d4b
Compare
AvivYossef-starkware
left a comment
There was a problem hiding this comment.
Can be in a different PR, but I think we need to add cases for virtual snos prover test (round-trip proving and verifying in the CI) .
and maybe change/add cases to the os tests for tx with proof facts
@AvivYossef-starkware reviewed 4 files and all commit messages, and made 1 comment.
Reviewable status: 4 of 6 files reviewed, 1 unresolved discussion (waiting on Yoni-Starkware).
6e47d4b to
df403f4
Compare
c37c0fe to
eb4e814
Compare
0b391c7 to
32350e2
Compare
eb4e814 to
acaa303
Compare
6fcaefc to
80f6045
Compare
80f6045 to
0bd2e48
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 0bd2e48. Configure here.
af06eb0 to
4377e32
Compare
AvivYossef-starkware
left a comment
There was a problem hiding this comment.
@AvivYossef-starkware reviewed 10 files and all commit messages, and made 1 comment.
Reviewable status: 10 of 15 files reviewed, all discussions resolved.
.github/workflows/starknet_transaction_prover_ci.yml line 118 at r7 (raw file):
- name: Run virtual OS multicall proving test working-directory: crates/starknet_transaction_prover run: cargo test --release -p starknet_os_flow_tests --features starknet_transaction_prover/stwo_proving prove_and_verify_multicall_tx -- --ignored --test-threads=1
why?
Code quote:
--test-threads=1
Yoni-Starkware
left a comment
There was a problem hiding this comment.
@Yoni-Starkware partially reviewed 2 files and made 1 comment.
Reviewable status: 2 of 15 files reviewed, all discussions resolved (waiting on AvivYossef-starkware).
.github/workflows/starknet_transaction_prover_ci.yml line 118 at r7 (raw file):
Previously, AvivYossef-starkware wrote…
why?
Similar to @avi-starkware's test above
# Run tests sequentially (no parallelism) since proving is a heavy CPU and memory intensive task
e9aef6a to
a6beb27
Compare
…} to 039e52c, add v0/v1 aliases Picks up the keccak builtin simulation in the privacy bootloader (starkware-libs/proving-utils#342 by @yuvalg), allowing virtual-OS proofs that exercise the keccak builtin to be generated and verified. Renames the workspace alias `privacy-circuit-verify` to `privacy-circuit-verify-v1` (with `package = "privacy-circuit-verify"`) to match the existing `privacy-circuit-verify-v0` sibling and the `ProofVersion::{V0, V1}` enum naming; the proof verifier dispatches on proof version and uses the corresponding alias. Regenerates the example proof fixtures (`apollo_transaction_converter/resources/example_proof.bin` and the `0.14.3` regression copy in `starknet_proof_verifier/resources/regression_test/0.14.3/`) against the new prover output. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
a6beb27 to
aff094e
Compare
AvivYossef-starkware
left a comment
There was a problem hiding this comment.
@AvivYossef-starkware reviewed 16 files and all commit messages, and made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on Yoni-Starkware).
.github/workflows/starknet_transaction_prover_ci.yml line 118 at r7 (raw file):
Previously, Yoni-Starkware (Yoni) wrote…
Similar to @avi-starkware's test above
# Run tests sequentially (no parallelism) since proving is a heavy CPU and memory intensive task
It was needed when we had 2 proofs in parallel, it's not the case now, so you can keep it for safety or remove it


Stack:
Summary
Bumps the workspace's
privacy-proveandprivacy-circuit-verifydependencies to95bc227and adds a v0 alias dispatch. Picks up the keccak builtin simulation in the privacy bootloader (starkware-libs/proving-utils#342, @yuvalg) plus regenerated circuit verifier constants (starkware-libs/proving-utils#343).Regenerates the example proof fixtures to match the new prover output format:
crates/apollo_transaction_converter/resources/example_proof.bincrates/apollo_transaction_converter/resources/example_proof_facts.jsoncrates/starknet_proof_verifier/resources/regression_test/0.14.3/example_proof.bincrates/starknet_proof_verifier/resources/regression_test/0.14.3/example_proof_facts.jsonThe rev points at
Yoni-Starkware/proving-utilsbecause the relevant fix commits are not yet merged upstream; switch to astarkware-libsrev once proving-utils#343 (and its base PRs #341, #342) land.