Skip to content

Cli type check #23

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.

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

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Cli type check #23

wants to merge 14 commits into from

Conversation

servalD
Copy link
Collaborator

@servalD servalD commented Jun 26, 2025

Linked issues

Dependencies

  • Added @commander-js/extra-typings to have typed options

Changes

  • Includes Unix keystore wrapper #21, which would be merged before this PR.
  • Speeds up type checking by using the type maps instead of generics.
  • Adds a proxy to each contract interface, injecting safeWrite, which simulates each call before sending a write transaction.
  • Hardcodes or centralizes some variables.
  • Validates all CLI inputs (CB58, addresses, private keys, BLS PoPs and public keys...), and casts them before performing actions.

Breaking changes

  • All Token and AVAX Input values are converted to 1e9 BigInt (To be adapted)

servalD added 14 commits June 11, 2025 18:08
Signed-off-by: gaetan cabaret <[email protected]>
Signed-off-by: gaetan cabaret <[email protected]>
Signed-off-by: gaetan cabaret <[email protected]>
Signed-off-by: gaetan cabaret <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant