Skip to content

feat(wallet): add vault cli commands#511

Open
Arjentix wants to merge 3 commits into
mainfrom
arjentix/add-vault-wallet-commands
Open

feat(wallet): add vault cli commands#511
Arjentix wants to merge 3 commits into
mainfrom
arjentix/add-vault-wallet-commands

Conversation

@Arjentix

@Arjentix Arjentix commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

🎯 Purpose

While running manually our sequencer and wallet I've noticed that retrieving funds supplied in genesis from wallet cli is not implemented yet. This PR fixes this.

⚙️ Approach

  • Add wallet vault transfer and wallet vault claim commands
  • Use them in test fixtures instead of manual tx construction
  • Expand Justfile

🧪 How to Test

just run-sequencer standalone
just wallet-import-test-accounts
just run-wallet account get --account-id Public/CbgR6tj5kWx5oziiFptM7jMvrQeYY3Mzaao6ciuhSr2r

🔗 Dependencies

None

🔜 Future Work

Maybe rethink private accounts import, because currently it's not handy

📋 PR Completion Checklist

  • Complete PR description
  • Implement the core functionality
  • Add/update tests
  • Add/update documentation and inline comments

@Arjentix Arjentix force-pushed the arjentix/add-vault-wallet-commands branch from e042247 to 9cddce9 Compare June 5, 2026 20:49
@Arjentix Arjentix changed the title Arjentix/add vault wallet commands feat(wallet): add vault cli commands Jun 5, 2026
@Arjentix Arjentix force-pushed the arjentix/add-vault-wallet-commands branch from 9cddce9 to a6a8131 Compare June 8, 2026 11:37

@moudyellaz moudyellaz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm, thanks

@Pravdyvy Pravdyvy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm

@schouhy schouhy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

lgtm thanks for the completions

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