Skip to content

#129 Update SMT to use version 6a instead of 3o#130

Merged
b3y0urs3lf merged 5 commits into
mainfrom
issue-129
Jul 9, 2026
Merged

#129 Update SMT to use version 6a instead of 3o#130
b3y0urs3lf merged 5 commits into
mainfrom
issue-129

Conversation

@martti007

Copy link
Copy Markdown
Collaborator

No description provided.

@martti007
martti007 requested a review from MastaP July 6, 2026 13:40

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request introduces the pathToRegion utility to pack a node's key prefix into a 32-byte region, updating the hashing logic in InclusionCertificate and FinalizedNodeBranch. It also refactors SparseMerkleTree and SparseMerkleSumTree to simplify the buildTree method and changes path lengths from bigint to number. Feedback is provided to add validation for the depth parameter in pathToRegion to prevent potential runtime errors or out-of-bounds writes.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/smt/SparseMerkleTreePathUtils.ts
@MastaP MastaP linked an issue Jul 6, 2026 that may be closed by this pull request
MastaP
MastaP previously approved these changes Jul 7, 2026
Base automatically changed from issue-127 to main July 8, 2026 11:06
@b3y0urs3lf
b3y0urs3lf dismissed MastaP’s stale review July 8, 2026 11:06

The base branch was changed.

@b3y0urs3lf
b3y0urs3lf merged commit 5bfbb67 into main Jul 9, 2026
1 check passed
@b3y0urs3lf
b3y0urs3lf deleted the issue-129 branch July 9, 2026 08:32
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.

Switch SMT inclusion verification from version 3o to 6a

3 participants