Skip to content

[PoI]: Add Proof of Inclusion support - #305

Open
itsyaasir wants to merge 80 commits into
mainfrom
feat/poi-implementation
Open

[PoI]: Add Proof of Inclusion support#305
itsyaasir wants to merge 80 commits into
mainfrom
feat/poi-implementation

Conversation

@itsyaasir

@itsyaasir itsyaasir commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Description of change

Links to any relevant issues

Type of change

  • Bug fix (a non-breaking change which fixes an issue)
  • Enhancement (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Fix

How the change has been tested

Change checklist

  • I have followed the contribution guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes
  • I have updated the CHANGELOG.md, if my changes are significant enough

@itsyaasir itsyaasir self-assigned this Jul 3, 2026
@itsyaasir itsyaasir linked an issue Jul 3, 2026 that may be closed by this pull request
itsyaasir added 26 commits July 7, 2026 06:23
[PoI]: Define proof envelope and API surface
[PoI]: Add verifier dispatch and transaction verification
…ruction

feat: Add gRPC client support for Proof of Inclusion and enhance error handling
…port

feat: Add support for object proofs
feat: Add support for event proofs in source and enhance related error handling
…tion

feat: Implement committee resolution
@itsyaasir
itsyaasir marked this pull request as ready for review August 17, 2026 03:13
- Removed obsolete service-info example.
- Added multiple examples demonstrating the creation and verification of transaction proofs, multi-target proofs, object proofs, and event proofs.
- Implemented a main entry point for running examples with command-line arguments.
- Enhanced proof structure to include explicit transaction, object, and event targets.
- Updated utility functions for loading the mainnet genesis blob and measuring elapsed time.
- Improved WASM bindings to expose proof targets and their properties.
- Updated tests to validate new proof target functionality and ensure correctness.
@UMR1352
UMR1352 self-requested a review August 21, 2026 08:27

@UMR1352 UMR1352 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks very well built, only nitpicks.

Comment thread bindings/wasm/poi_wasm/examples/src/02_create_and_verify_multi_target_proof.ts Outdated
Comment thread bindings/wasm/poi_wasm/examples/src/02_create_and_verify_multi_target_proof.ts Outdated
Comment thread bindings/wasm/poi_wasm/src/committee.rs Outdated
Comment thread bindings/wasm/poi_wasm/src/error.rs
Comment thread poi-rs/src/source.rs
Comment thread poi-rs/src/proof.rs
Comment thread poi-rs/src/proof.rs Outdated
Comment thread poi-rs/src/proof.rs Outdated
Comment thread poi-rs/src/proof.rs
Comment thread poi-rs/src/proof.rs Outdated
- Updated the transaction proof structure to use ProofV1, encapsulating the chain, targets, checkpoint summary, and transaction proof.
- Modified the transaction.json fixture to reflect the new structure, ensuring compatibility with the updated proof format.
- Adjusted proof construction tests to utilize the new ProofV1 structure, including changes to how targets and transaction proofs are accessed.
- Enhanced proof serialization tests to validate the new proof structure and ensure proper version handling.
- Updated utility functions to create proofs with the new structure, ensuring all tests pass with the revised proof format.
- Cleaned up imports and organized code for better readability and maintainability.
docs: add Proof of Inclusion documentation
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.

[PoI]: Add Proof of Inclusion support

2 participants