[01/05] Split stage 6 sumchecks#1564
Closed
RadNi wants to merge 18 commits into
Closed
Conversation
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Update the stack spec to match the final branch slices, moved claim-reduction implementation scope, and expanded local CI coverage. Co-authored-by: Cursor <cursoragent@cursor.com>
Restore the feature spec text to its original contents; only the stack plan should describe the updated PR split and local CI scope. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep the stack plan focused on PR scope and ownership; verification details are tracked outside this spec. Co-authored-by: Cursor <cursoragent@cursor.com>
Use phase-specific stage6a/stage6b sumcheck params and proof wiring so zk transcript and constraints stay aligned after splitting stage6. This preserves the split design while fixing zk failures without introducing bytecode-commitment features. Made-with: Cursor (cherry picked from commit e9ec21c) Co-authored-by: Amirhossein Khajehpour <khajepour.amirhossein@gmail.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
This was referenced May 21, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Posted by Cursor assistant (model: GPT-5.5) on behalf of the user (Amirhossein Khajehpour) with approval.
Replacement for LayerZero-Research#16 so the first stack PR targets
a16z/jolt:maindirectly.Stack position:
01/05Base branch:
a16z/jolt:mainSource implementation reference:
amir/bytecode-commitment-merged@a351d5078b83577db8f3264fa53dfaddcf5ed687Owned paths:
The stack plan is preserved in this PR description instead of being checked into the repository. The checked-in spec source of truth is
specs/1344-committed-bytecode-program-image.md.Committed Bytecode Stack Plan
Source implementation reference:
amir/bytecode-commitment-merged@a351d5078b83577db8f3264fa53dfaddcf5ed687Stack base:
a16z/jolt:mainGoal: split the current committed bytecode/program-image branch into smaller stacked PRs while preserving the final behavior of the source branch.
Stack
stack-plana16z/jolt:mainstage6-splitprecommittedprogram-foundationbytecode-program-reductionsurface-docsSlice Checklist
00: Stack Plan
Owned paths:
Checklist:
a16z/mainbranch.01: Stage 6 Split
Owned paths:
Checklist:
stage6a_sumcheck_proofandstage6b_sumcheck_proof.02: Precommitted
Owned paths:
Checklist:
BytecodeChunk,ProgramImageInit, and committed-program reductions.jolt-core/src/subprotocols/sumcheck.rsexactly equal toorigin/main.03: Program Foundation
Owned paths:
Checklist:
ProgramModeandProgramPreprocessing.04: Bytecode Program Reduction
Owned paths:
Checklist:
BytecodeValStage(i)values.ProgramImageInitContributionRw.05: Surface And Docs
Owned paths:
Checklist:
Note
High Risk
High risk because it introduces a new polynomial commitment scheme and substantially refactors
jolt-program’s instruction decoding/expansion and program-image building, which are correctness-critical for proof generation and verification.Overview
Adds a new PCS implementation. Introduces
crates/jolt-hyperkzg, implementing HyperKZG (Gemini-to-KZG reduction) with batch open/verify, error/types, benches, fuzz targets, and integration tests.Refactors program image + expansion pipeline.
jolt-programnow builds programs usingJoltInstructionProfileand stores expanded bytecode asJoltInstructionRow(notNormalizedInstruction), adds aserializationfeature (enabled by default) gating serde/ark-serialize on trace types, and rewrites expansion to a two-phase recipe/materialization model with stricter allocator tracking (bitset, recursion-depth/capacity checks) and new error cases.Updates RISC-V lookup-table integration and CI.
jolt-lookup-tablesswitches toJoltInstructionRow/JoltInstructionRowData, enablesjolt-riscv’sserializationfeature, and test helpers are updated accordingly; CI now conditionally enablestest-utilswhen present and runstracertests with--features test-utils.Housekeeping. Bumps workflow
typosaction, updates dependencies (blake3,sysinfo, alloy crates, etc.), and updates README test command tocargo nextest run -p jolt-core --features host.Reviewed by Cursor Bugbot for commit 4624ea6. Configure here.
Made with Cursor