Skip to content

fix(jolt): update to rev 2e05fe88, enable ZK, fix no-std platform#305

Merged
han0110 merged 5 commits intoeth-act:masterfrom
0xAndoroid:fix/jolt
Mar 9, 2026
Merged

fix(jolt): update to rev 2e05fe88, enable ZK, fix no-std platform#305
han0110 merged 5 commits intoeth-act:masterfrom
0xAndoroid:fix/jolt

Conversation

@0xAndoroid
Copy link
Contributor

@0xAndoroid 0xAndoroid commented Mar 7, 2026

Summary

  • Update all jolt deps from 35d46f5 to 2e05fe88 (includes no-std jolt-common fix)
  • Add Curve generic parameter to JoltProverPreprocessing/JoltVerifierPreprocessing (new rev API)
  • Enable zk feature by default when zkvm feature is active
  • Add zk feature to ere-platform-jolt (forwards to jolt-sdk/zk)
  • Remove set_std(true) call (removed in new rev)
  • Replace jolt::puts with jolt::print! macro (cross-platform)

Test plan

  • cargo build -p ere-jolt passes
  • cargo build -p ere-platform-jolt --no-default-features passes (no-std)
  • csp-benchmarks sha256/keccak/ecdsa prove+verify pass with this ere rev

- Update all jolt deps from 35d46f5 to cddd96fe
- Add Curve generic parameter to JoltProverPreprocessing/JoltVerifierPreprocessing
- Enable zk feature by default when zkvm feature is active
- Remove set_std(true) call (removed in new rev)
- Remove puts() call in platform (removed in new rev)
@0xAndoroid 0xAndoroid changed the title refactor(jolt): use #[jolt::provable] macro, make jolt actually ZK, remove ere-platform-jolt fix(jolt): update to rev 2e05fe88, enable ZK, fix no-std platform Mar 7, 2026
Copy link
Collaborator

@han0110 han0110 left a comment

Choose a reason for hiding this comment

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

Thanks for the update! Just some small fix needed for the docker image to work, pushed it at 00dac79 (also updated it to always turn on the zk feature).

@han0110 han0110 merged commit 6573843 into eth-act:master Mar 9, 2026
59 of 60 checks passed
@0xAndoroid 0xAndoroid deleted the fix/jolt branch March 9, 2026 15:37
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