Skip to content
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

Provide tool for users to get all the ABIs they need #72

Open
banool opened this issue Nov 9, 2023 · 1 comment
Open

Provide tool for users to get all the ABIs they need #72

banool opened this issue Nov 9, 2023 · 1 comment

Comments

@banool
Copy link

banool commented Nov 9, 2023

As it is today Surf provides DefaultABITable. This is helpful if users only use the modules included in it, but otherwise it has two issues:

  1. It doesn't include everything in the framework. Keeping it up to date might be challenging.
  2. It doesn't include ABIs for any other modules my module might depend on.

I suggest a tool where the user can provide either a Move package on disk or a module on chain as input and as output it recursively discovers all the modules those modules depend on and outputs their ABIs. This would make it easy for users to ensure they've got all the ABIs they need.

I actually implemented something similar to this already in Rust: aptos-labs/aptos-core#9405.

@SamuelQZQ
Copy link
Member

Good point. We also considered this before, will add it. Thank you!

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

No branches or pull requests

2 participants