Skip to content

[WIP] E2E interactive cli #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
franklynot merged 43 commits into from
Feb 24, 2025
Merged

[WIP] E2E interactive cli #67

franklynot merged 43 commits into from
Feb 24, 2025

Conversation

@StarDumpling
Copy link

@StarDumpling StarDumpling commented Feb 19, 2025

No description provided.

franklynot
franklynot requested changes Feb 19, 2025
View reviewed changes
StarDumpling and others added 16 commits February 19, 2025 22:52
@StarDumpling
Add missing commands to client CLI, fix statuses
632ed3f
@franklynot
Clean up Esplora URL handling
aa65d4f
@franklynot
Add a comment
bbdba10
@franklynot
Create L2 chain adaptor outside client constructor
38b40ed
@franklynot
Update comment
09a1a21
@StarDumpling
Add `set-chain-service' command and broadcast disprove command
eab41fd
@StarDumpling
Refactor get_proof() fn
c2122f5
@franklynot
Fix conflicting 'set-chain-service' command flag
0808e3b
@franklynot
Unblock the CLI using mock chain adaptor
dbc3461
@franklynot
Revert the bridge key dir name change
75c43e5
@franklynot
Add an error message when funding UTXO is missing
4d70592
@franklynot
Fix the Alpen signet URL 🙈
a8b8c0f
@franklynot
Add missing client sync before certain commands
2f1a02b
@franklynot
Replace direct MockAdaptor instantiation with a helper call
1020b85
@franklynot
Rename "set-chain-service" command to "mock-l2-pegout-event" for clarity
f38f658
@franklynot
Merge pull request #69 from elementlabs42/mock-peg-out
6876357 
Feed destination network peg out event to CLI for the demo
@franklynot
Copy link

franklynot commented Feb 21, 2025 •
edited
Loading

The readme.md also needs to be updated. It's missing some existing commands and ones introduced in this PR.

franklynot and others added 12 commits February 21, 2025 15:32
@franklynot
Remove '.' after graph ID in console output for easier copying in ter…
a434727 
…minal
@franklynot
Add graph type in console output for clarity and make it consistent a…
01af739 
…cross messaging
@franklynot
Refactor key command handling to display current key configuration wh…
4a5b9ee 
…en no arguments are provided

Remove hardcoded secrets from base graph module and redefine them in the test setup
@franklynot
Remove remaining hardcoded secrets
ac4fa41
@franklynot
Add missing data sync and flush when mocking L2 event
8b18c11
@franklynot
Add missing data flush after boradcasting peg-out
cf65c46
@franklynot
Change the Alpen signet endpoint to one that accepts large txs
0b14fd7
@franklynot
Remove duplicate info in console output for transaction broadcasting
3d51adb
@franklynot
Add error message for missing verifier public keys in client command
5622674
@StarDumpling
Add commands for assert_initial_* with invalid proof
adbe5e9
@aristotleee
removed compression profiling
1cbee0d
@aristotleee
Merge branch 'rc' into e2e-interactive-cli
8b528a1
aristotleee
aristotleee approved these changes Feb 24, 2025
View reviewed changes
@franklynot franklynot marked this pull request as ready for review February 24, 2025 16:49
@franklynot franklynot merged commit 884c915 into rc Feb 24, 2025
0 of 2 checks passed
@franklynot franklynot deleted the e2e-interactive-cli branch February 24, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@franklynot franklynot franklynot requested changes

@aristotleee aristotleee aristotleee approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@StarDumpling @franklynot @aristotleee