Skip to content

feat: add v6.0.0 verifier contracts#68

Closed
fakedev9999 wants to merge 1 commit intomainfrom
fakedev9999/v6-contracts-clean
Closed

feat: add v6.0.0 verifier contracts#68
fakedev9999 wants to merge 1 commit intomainfrom
fakedev9999/v6-contracts-clean

Conversation

@fakedev9999
Copy link
Member

Summary

  • Upgrade sp1-sdk from 5.0.0 to v6.0.0 (git dependency) and update artifacts.rs for async SDK API
  • Generate fresh v6.0.0 Plonk and Groth16 verifier contracts from S3 circuit artifacts
  • Apply manual fixes for 4 known S3 artifact issues (see verification table below)
  • Run forge fmt across all contracts for consistent formatting
  • Fix stale contracts/README.md overview to reflect current architecture

Verification Results

The following checks were run against the regenerated artifacts:

# Check Result Details
1 VK_ROOT is 32 bytes FAIL → fixed S3 emitted 33-byte literal; trimmed leading 00 byte
2 VERSION is "v6.0.0" FAIL → fixed S3 emitted "v6.0.0-rc.1"; replaced in both files
3 Groth16 contract name matches import FAIL → fixed Contract named Verifier; renamed to Groth16Verifier
4 Groth16 uses verifyProof FAIL → fixed Generated this.Verify(); changed to this.verifyProof()
5 VERIFIER_HASH values match PASS Plonk: 0xbb1a6f29..., Groth16: 0xc7f26b1b...

Key Values

  • VERIFIER_HASH (Plonk): 0xbb1a6f2930e94bfe8b35e794faf43133214534a17d2ad8e51358cad437b3c317
  • VERIFIER_HASH (Groth16): 0xc7f26b1bf195b7e1a9bde70e92d1014a520ba06fe9e27a069d57c8abdad35189
  • VK_ROOT (both): 0x008cd56e10c2fe24795cff1e1d1f40d3a324528d315674da45d26afb376e8670
  • VERSION (both): "v6.0.0"

Test plan

  • cargo run --bin artifacts --release succeeds
  • All 5 verification checks pass (4 with manual fixes)
  • forge build compiles cleanly
  • forge fmt --check passes
  • forge test passes (16/16 tests)

Supersedes #66.

@fakedev9999
Copy link
Member Author

Superseded by new PR with corrected S3 artifacts

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.

1 participant