Skip to content

feat(ssa_fuzzer): pushing generated program and witness to redis queue #9375

Merged
defkit merged 7 commits intomasterfrom
sn/ssa_fuzzer_proving
Aug 5, 2025
Merged

feat(ssa_fuzzer): pushing generated program and witness to redis queue #9375
defkit merged 7 commits intomasterfrom
sn/ssa_fuzzer_proving

Conversation

@defkit
Copy link
Contributor

@defkit defkit commented Aug 1, 2025

Description

Adds functionality for the ability to save generated programs for which a witness has been found.
This will be used to verify that the program is provable and verifiable on backend.

Problem*

Resolves

Summary*

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@defkit defkit changed the title feat(ssa_fuzzer): bb verification of generated programs feat(ssa_fuzzer): bb proving of generated programs Aug 1, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Test Suite Duration'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 4530242 Previous: 639d802 Ratio
test_report_AztecProtocol_aztec-packages_noir-projects_noir-protocol-circuits_crates_private-kernel-lib 2 s 1 s 2
test_report_AztecProtocol_aztec-packages_noir-projects_noir-protocol-circuits_crates_reset-kernel-lib 2 s 1 s 2
test_report_zkpassport_noir_rsa_ 2 s 1 s 2

This comment was automatically generated by workflow using github-action-benchmark.

CC: @TomAFrench

Copy link
Member

@TomAFrench TomAFrench left a comment

Choose a reason for hiding this comment

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

Tests around proving should be in aztec-packages rather than in this repository.

@defkit defkit changed the title feat(ssa_fuzzer): bb proving of generated programs feat(ssa_fuzzer): pushing generated program and witness to redis queue Aug 4, 2025
@defkit defkit requested a review from Rumata888 August 4, 2025 08:52
github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request Aug 5, 2025
…_fuzzer (#16188)

In noir-lang/noir#9375 new functionality has
been added that allows adding noir programs for which a witness has been
found to a redis queue.

This PR introduces workers, that consumes programs and witnesses from
the queue, proofs and verifies them in order to check that all programs
generated from SSA and solved could be proven and verified.
@defkit
Copy link
Contributor Author

defkit commented Aug 5, 2025

@TomAFrench I removed everything about proving, now its just a feature allowing to push programs and witness to redis

Copy link
Member

@TomAFrench TomAFrench left a comment

Choose a reason for hiding this comment

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

Removing block

@defkit defkit added this pull request to the merge queue Aug 5, 2025
Merged via the queue into master with commit 532b512 Aug 5, 2025
123 checks passed
@defkit defkit deleted the sn/ssa_fuzzer_proving branch August 5, 2025 13:43
github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request Aug 11, 2025
Automated pull of nightly from the
[noir](https://github.com/noir-lang/noir) programming language, a
dependency of Aztec.
BEGIN_COMMIT_OVERRIDE
chore: Release Noir(1.0.0-beta.10)
(noir-lang/noir#9311)
feat(ssa_fuzzer): arrays support
(noir-lang/noir#9427)
chore: add link to issue on TODOs
(noir-lang/noir#8307)
fix(ownership): Increment reference count for nested array get in LHS
assignment (noir-lang/noir#9347)
chore: add some mem2reg unit tests
(noir-lang/noir#9405)
chore(as_slice_length): Various unit tests
(noir-lang/noir#9419)
chore(simplify_cfg): Additional unit tests
(noir-lang/noir#9426)
chore: create directory when writing witness artefact
(noir-lang/noir#9383)
fix: throw error if foreign call returns the wrong number of fields
(noir-lang/noir#9286)
chore: update ex in docs (noir-lang/noir#9385)
chore: add some `assert_constant` tests
(noir-lang/noir#9413)
chore: error on non constant inputs for Pedersen generators
(noir-lang/noir#9389)
chore(inlining): Unit tests for global values and conditional inlining
(noir-lang/noir#9411)
chore(ssa_fuzzer): refactor fuzzing modes + add fuzzing mode without DIE
pass (noir-lang/noir#9401)
chore(inlining): Various unit tests
(noir-lang/noir#9388)
feat(ssa_fuzzer): pushing generated program and witness to redis queue
(noir-lang/noir#9375)
fix(ssa_gen): Generate code for index before the collection
(noir-lang/noir#9332)
chore: Regression test for calling a mutable closure inside a mutable
closure (noir-lang/noir#9384)
fix: some nargo expand fixes
(noir-lang/noir#9324)
fix: disable comptime printing when requesting json output
(noir-lang/noir#9381)
chore: address TODO comments
(noir-lang/noir#9379)
feat: type alias for numeric generics
(noir-lang/noir#7583)
chore: bump external pinned commits
(noir-lang/noir#9291)
END_COMMIT_OVERRIDE

---------

Co-authored-by: AztecBot <tech@aztecprotocol.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
AztecBot pushed a commit to AztecProtocol/barretenberg that referenced this pull request Dec 3, 2025
…_fuzzer (#16188)

In noir-lang/noir#9375 new functionality has
been added that allows adding noir programs for which a witness has been
found to a redis queue.

This PR introduces workers, that consumes programs and witnesses from
the queue, proofs and verifies them in order to check that all programs
generated from SSA and solved could be proven and verified.
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.

3 participants