Skip to content

[zk-sdk-wasm-js] Create wasm wrappers for the rest of the sigma proof types#131

Merged
samkim-crypto merged 1 commit intosolana-program:mainfrom
samkim-crypto:wasm-js-proofs
Oct 13, 2025
Merged

[zk-sdk-wasm-js] Create wasm wrappers for the rest of the sigma proof types#131
samkim-crypto merged 1 commit intosolana-program:mainfrom
samkim-crypto:wasm-js-proofs

Conversation

@samkim-crypto
Copy link
Copy Markdown
Contributor

@samkim-crypto samkim-crypto commented Oct 12, 2025

Summary of Changes

This is a follow-up to #128. All the encryption types that we want to export out to wasm was added in #78 and #128. What remains are sigma proof types and range proof types.

I was thinking of adding wrapper types for both sigma proof types and range proof types in this PR. It seemed like there are some non-trivial design choices for creating wrapper types for range proof, so I decided to focus on sigma proofs in this PR for now.

I apologize for committing in a lot of code, but most of the code is quite repetitive and mechanical.

  • I created wrapper types for the proof data type and the proof context type.
  • Added proof generation and verification functions for the proof data
  • Added basic functions for serializing and deserializing for the context type

@samkim-crypto samkim-crypto changed the title [zk-sdk-wasm-js] Create wasm wrappers for the rest of the sigma proof types and range proof types [zk-sdk-wasm-js] Create wasm wrappers for the rest of the sigma proof types Oct 13, 2025
@samkim-crypto samkim-crypto marked this pull request as ready for review October 13, 2025 09:08
@samkim-crypto samkim-crypto requested a review from grod220 October 13, 2025 09:09
Copy link
Copy Markdown
Member

@grod220 grod220 left a comment

Choose a reason for hiding this comment

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

Great job! 🚢

@samkim-crypto samkim-crypto merged commit 1e0c41d into solana-program:main Oct 13, 2025
9 checks passed
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