Skip to content

feat: rust sdk#459

Merged
MauroToscano merged 148 commits intomainfrom
442-feat-rust-sdk
Jul 1, 2024
Merged

feat: rust sdk#459
MauroToscano merged 148 commits intomainfrom
442-feat-rust-sdk

Conversation

@uri-99
Copy link
Copy Markdown
Contributor

@uri-99 uri-99 commented Jun 21, 2024 •
edited by NicolasRampoldi
Loading

Description

  • This PR creates an SDK for Aligned with 3 methods: submit, verify_proof_onchain and get_verification_key_commitment.

Test the SDK

  • Start anvil with block time, the aggregator, the operator and the batcher.
  • Inside the aligned-sdk directory, run cargo test.
  • Whenever the test freezes, run make batcher_send_infinite_groth16 and keep sending proofs in the background.

Test the client that now uses the SDK

  • Run make install_aligned_compiling.
  • Run which aligned and make sure you are using the .cargo/bin/aligned version.
  • Make sure the client works as expected, run Submit, VerifyProofOnChain and GetVerificationKeyCommitment.

Important

  • I made an minimum example using the SDK, I will add it in another PR because I need the SDK merged in main.
  • The client won't work with the batcher deployed on Testnet yet. We need to re-deploy the batcher for it to send the current version to the client.

entropidelic and others added 30 commits June 17, 2024 13:54
@entropidelic
First iteration of client-side commitment calculation working
74d7d48
@entropidelic
Client side verification working
24af202
@entropidelic
Add types module for batcher client
e2f5c7a
@entropidelic
Polish code
06279e8
@entropidelic
Refactored some functionality into save_response function
6cf4031
@entropidelic
Remove unused import
1fcaa05
@entropidelic
Add comment
7ba86f1
@MauroToscano
Change logs
06bd9be
@uri-99
feat(wip): building sdk of submit()
fabf809
@uri-99
fix: add sdk to cargo.toml
1be2ee5
@uri-99
Merge branch '407-feat-batcher-should-not-return-commitment' into 442…
e346c76 
…-feat-rust-sdk
@uri-99
feat(wip): test file that uses sdk
e1c6de1
@entropidelic
Save work in progress
e6da4bb
@uri-99
feat(wip): sdk for test example
56541bc
@entropidelic
Finish first iteration of payment system
5e982f8
@entropidelic
Delete unused functions
b2094be
@entropidelic
Fix upgrader scripts and tests
5557013
@entropidelic
Merge branch 'main' into 412-feat-implement-payment-system
5ed3a37
@entropidelic
Add config file for aggregator in anvil
88d2b20
@entropidelic
Add keys files
46f107a
@entropidelic
Merge remote-tracking branch 'origin/412-feat-implement-payment-syste…
66a8171 
…m' into 412-feat-implement-payment-system
@entropidelic
Merge branch 'main' into 412-feat-implement-payment-system
df7c610
@uri-99
feat: submit test by 1, must run twice
d8c9f9a
@uri-99
remove: test.rs file, now running directly on lib.rs
7070b39
@uri-99
fix: update cargo.lock
61fe057
@entropidelic
Update Makefile
412f4ef
@entropidelic
remove unused import in aligned contract
2929de3
@entropidelic
Merge branch 'main' into 412-feat-implement-payment-system
a720faa
@entropidelic
Change initialGasLeft in function
13a3d57
@entropidelic
Use anvil address & keys for aggregator
6ea90c7
MauroToscano
MauroToscano approved these changes Jun 28, 2024
View reviewed changes
Comment thread batcher/aligned-sdk/src/errors.rs Outdated
entropidelic and others added 16 commits June 28, 2024 11:54
@entropidelic
Fix signature bug:
0c46cac
@entropidelic
Merge branch 'main' into batcher-interact-with-payment-service
541e36d
@NicolasRampoldi
fix: remove unused errors
593cb70
@NicolasRampoldi
Merge branch 'main' into 442-feat-rust-sdk
dcb03e9
@entropidelic
Get payment service address from file
b14d48a
@entropidelic
Polish code
840ac94
@NicolasRampoldi
feat: add submit_multiple and submit
85ae366
@NicolasRampoldi
Merge branch '442-feat-rust-sdk' of github.com:yetanotherco/aligned_l…
aa384a9 
…ayer into 442-feat-rust-sdk
@entropidelic
Add script for for funding batcher
2815f14
@entropidelic
Change script name
0aea9e5
@entropidelic
Change script name
da99fff
@MauroToscano
Clippy
8458f92
@MauroToscano
Clippy
cdd7f28
@entropidelic
Remoe unused crate
7f2ce7e
@entropidelic
Add keystore_path arg to send burst script
9ff219c
@NicolasRampoldi
Merge branch 'refs/heads/batcher-interact-with-payment-service' into …
4389d34 
…442-feat-rust-sdk

# Conflicts:
#	batcher/Cargo.lock
#	batcher/aligned-batcher/src/lib.rs
#	batcher/aligned-sdk/Cargo.toml
#	batcher/aligned-sdk/src/eth.rs
#	batcher/aligned/Cargo.toml
NicolasRampoldi
NicolasRampoldi suggested changes Jun 28, 2024
View reviewed changes
Copy link
Copy Markdown
Contributor

@NicolasRampoldi NicolasRampoldi left a comment

Choose a reason for hiding this comment

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

Wait for #519 to merge

@NicolasRampoldi
fix: cargo.lock
eeafbda
@NicolasRampoldi
Merge branch 'refs/heads/main' into 442-feat-rust-sdk
48c5871 
# Conflicts:
#	batcher/Cargo.lock
#	batcher/aligned-batcher-lib/Cargo.toml
#	batcher/aligned-batcher/src/lib.rs
#	batcher/aligned-sdk/src/eth.rs
#	batcher/aligned/Cargo.toml
#	batcher/aligned/generate_proof_and_send.sh
#	batcher/aligned/src/main.rs
@NicolasRampoldi
fix: merge conflicts
de50d5d
@NicolasRampoldi
Merge branch 'refs/heads/main' into 442-feat-rust-sdk
6c7e626 
# Conflicts:
#	batcher/aligned-batcher/src/lib.rs
#	batcher/aligned/src/main.rs
@NicolasRampoldi
fix: merge conflicts
eda16ae
@MauroToscano MauroToscano merged commit dbbadfa into main Jul 1, 2024
@MauroToscano MauroToscano deleted the 442-feat-rust-sdk branch July 1, 2024 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MauroToscano MauroToscano MauroToscano approved these changes

+2 more reviewers

@NicolasRampoldi NicolasRampoldi NicolasRampoldi approved these changes

@taturosati taturosati taturosati approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@NicolasRampoldi NicolasRampoldi

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

feat: rust sdk

6 participants

@uri-99 @MauroToscano @NicolasRampoldi @taturosati @entropidelic @JuArce