Skip to content

Conversation

@briwylde08
Copy link
Contributor

@briwylde08 briwylde08 commented Jan 28, 2026

Copilot AI review requested due to automatic review settings January 28, 2026 14:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new documentation page covering zero-knowledge proof primitives (BN254 + Poseidon) for Stellar and exposes it via the built docs routes list.

Changes:

  • Added a new ZK Proofs on Stellar documentation page under Build → Apps.
  • Updated routes.txt to include the new docs route.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
routes.txt Adds the generated route entry for the new ZK docs page.
docs/build/apps/zk.mdx Introduces new ZK-focused documentation content and links to related resources.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

Copy link
Contributor

@ElliotFriend ElliotFriend left a comment

Choose a reason for hiding this comment

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

Looks like a great start! Left some thoughts, but there's very little that I would consider deal-breakers.

Thanks!

Comment on lines 18 to 20
- `g1_add` - performs point addition in G1 (point on curve)
- `g1_mul` - performs scalar multiplication in G1 (point on curve)
- `pairing_check` - performs pairing operation on a vector of G1 points and a vector of G2 points
Copy link
Contributor

Choose a reason for hiding this comment

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

These descriptions seem a little "inside baseball" for someone who might be less well-versed in ZK arithmetic. Would probably be useful to make them a little simpler

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @ElliotFriend! Accepted suggestions and tried to make the descriptions a little better. LMK what you think.

briwylde08 and others added 6 commits January 28, 2026 14:59
Co-authored-by: Elliot Voris <elliot@stellar.org>
Co-authored-by: Elliot Voris <elliot@stellar.org>
Co-authored-by: Elliot Voris <elliot@stellar.org>
Co-authored-by: Elliot Voris <elliot@stellar.org>
@stellar-jenkins
Copy link

4 similar comments
@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

@stellar-jenkins
Copy link

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.

4 participants