Skip to content

100 and 128 bit security end to end - #138

Merged
shreyas-londhe merged 17 commits into
mainfrom
zkfr/e2e-security
Oct 12, 2026
Merged

shreyas-londhe merged 17 commits into
mainfrom
zkfr/e2e-security

Conversation

@zkfriendly

@zkfriendly zkfriendly commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

This PR reduces PCS configuration to one choice: 100-bit or 128-bit.

The code calculates the remaining parameters internally from the padded witness size. Callers no longer choose backend profiles or tune individual parameters.

For reviewers, start with profiles.rs.
This is the main change. It contains both profiles, their calculations, and tests covering every supported witness size.

  • The 100-bit profile uses list decoding and OOD checks.
  • The 128-bit profile uses unique decoding and additional grinding.

Most other changes update the prover, verifier, CLI, and tests to use this interface. Some also update OOD sampling, transcript binding, and grinding to support the selected target.

All 72 PCS tests, Clippy, and formatting checks passed.

These are BitZ PCS targets. Spartan security is not dynamic.

@zkfriendly
zkfriendly changed the base branch from main to sl/initial-ood September 29, 2026 11:03
@zkfriendly
zkfriendly added this pull request to stack #139 September 29, 2026 11:03
Base automatically changed from sl/initial-ood to main September 30, 2026 09:13
@zkfriendly
zkfriendly marked this pull request as ready for review October 6, 2026 11:21

@shreyas-londhe shreyas-londhe left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM, thanks for the PR!

@shreyas-londhe
shreyas-londhe merged commit d7e7257 into main Oct 12, 2026
3 checks passed
@shreyas-londhe
shreyas-londhe deleted the zkfr/e2e-security branch October 12, 2026 04:57
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